This commit introduces new rules for classes. However, these classes are unnecessary, as the rules had already existed previously. We should revert these changes.

`class="reviews-display"` does not make any apparent change to how it's display.

`class="review-content"` is redundant because `<p>` can do the same thing and is more universal. This [link](http://getbootstrap.com/css/#type-body-copy) goes into detail about the `<p>` tag.

`class="time-tag"` does nearly the same `class="text-info"` so it was a redundant rule, as the original should be used.


@eroneko  https://github.com/mi3-dev/android_kernel_xiaomi_msm8x74pro/commits/lollipop-mr1

内核的代码有升级就会提交的！


Yep

Sent by Outlook for Android

On Wed, May 20, 2015 at 7:00 PM -0700, "Wasabi Fan" <notifications@github.com> wrote:
This seems to be a recurring theme :wink:

---
Reply to this email directly or view it on GitHub:
https://github.com/kyranstar/Hunt-The-Wumpus/commit/ec73fa7659f9c7f26de1edcad5760ea6c3b6a139#commitcomment-11297223


Query or note?


Pushed the assets

Sent by Outlook for Android

On Wed, May 20, 2015 at 6:51 PM -0700, "Wasabi Fan" <notifications@github.com> wrote:
I'm seeing errors like this:
```
The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/Wumpus.png' does not exist!
```

That probably means that you didn't commit the "80's shit" folder. My guess is that the issue that you are facing is something else, but I can't see it until you add those assets.

---
Reply to this email directly or view it on GitHub:
https://github.com/kyranstar/Hunt-The-Wumpus/commit/ec73fa7659f9c7f26de1edcad5760ea6c3b6a139#commitcomment-11297147


I think that the following functions are not adapted to this grid right?


For a while there as I was getting back into C++ and learning the managed C++ stuff. The cyclic dependencies in .h files for managed classes were giving me headaches. I think breaking the function bodies into .h and .cpp files helped me resolve one of those cases, but that was after I had added the `IBrowserAdapter` stuff.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


tested and working


Good input today @iamvery


`ripple::make_Manifest` returns a `boost::optional` and we shouldn't blindly just "dereference" it here.


I couldn't reach the delete query method in those commits, but we have to pay attention about authors being able to delete editors queries. Without any extra validation inside the delete method, that could happen.


Yes. And aoMaps use the 2nd set of UVs -- just like lightMaps do -- in case you want to give it a try.


```
error : The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/gold.png' does not exist!
error : The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/Pit0.png' does not exist!
```


It is, but the way ASP.NET Web.API handles it is different based on the content type. so for everything to work as expected it's better if the content type is set to 'application/json'

See: http://stackoverflow.com/a/20226220/2287840


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


It was @kmanalo I believe.


Ok - NeoBundler is frustratingly vague on the problem


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Nit: Quotes should be double. Though ignore.

Nit: Module is unused.


I like it better


The AUTHs should probably not be hard-coded, since the ISUPPORT message tells us what these are.


rax retry tests


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I was working on that but did not get it done before your release.


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


Not bad dude. You figured all the git out! Have $1 on me via @changetip


Hi @jcstrang-edu, @MaxFangX sent you a Bitcoin tip worth 4,450 bits ($1.00), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/845558)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Should be on the prototype, no?


test for pullrequest!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


extractall has this really annoying bug:
https://bugs.python.org/issue15795

since i upgraded to salt 2015.5.x it brook a lot of my formulas that were downloading zipfile containing binaries, as now they lost their executable bits.

https://github.com/saltstack/salt/issues/24006


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Fixed.... I think

Sent by Outlook for Android

On Wed, May 20, 2015 at 7:35 PM -0700, "Wasabi Fan" <notifications@github.com> wrote:
```
error : The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/gold.png' does not exist!
error : The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/Pit0.png' does not exist!
```

---
Reply to this email directly or view it on GitHub:
https://github.com/kyranstar/Hunt-The-Wumpus/commit/ec73fa7659f9c7f26de1edcad5760ea6c3b6a139#commitcomment-11297506


eita... obrigado por corrigir, @RicardoMurad!
você pode arrumar a indentação antes de mergearmos?


I looked it up.  Yes, that's the standard place to put it:
http://www.scalatra.org/2.2/getting-started/project-structure.html


i would keep this var, and only register the hotkey here, where chat_frame is instantiated


This changeset isn't controversial.
You should check this one into master.
I'm not going to touch anything until you're ready, but on an active project the longer you wait, the harder it gets to resolve the merge conflicts.


What are these improvements to?
Please be specific, especially if we can't see what it is or it's not obvious.
I only ask because depending on what objects it is, I may need to pass them through to bushkit.ac, damage.ac and pontoon.ac, thanks.


can just do

register_hotkey(TOGGLE_CHAT_KEY, toggle_chat);


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-5.29%25-green.svg)


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


@jensens, you forgot to add `payment_text` to the list of possible template callbacks. This results in the error described in issue #24.


What was the problem that you were seeing that caused this fix?


:+1: It's faster, and more randomerified. (Not a real word.)


this commit can replace previous
everything works


I don't understand this class so I assume you did it correctly.
I'm surprised there isn't a built-in thing in Scala for error messages.


厉害！


Need to edit the "+" which is not creating the bullet points as intended


Two things:
- This is not against the latest development code, if you wanted to contribute it.
- I'm not sure this change makes sense. The position of the nozzle after homing is supposed to include the offset. If you move to where the offset is in this way, it undoes the offset. To accomplish what you're trying to do here, I believe you should do a direct move to (0, 0, 0) by issuing `G0 X0 Y0 Z0`.


rax retry tests


Bad doge!  I love it!  :dog2:


You can delete this line and the ExampleSpec file.
(Also Hello.scala and ExampleTest.scala.)


Let's say you index a list of scalar, can you index a scalar after? It's the same thing here I think.


> If (and only if) I'm not mistaken, then you have to include the gamemap header in the corresponding cpp file, right?

If I'm not mistaken, you need the header but not they binary. That means that you don't have to add GameMap.cpp to the source list of the test (and all the other needed files that comes with it).


Why did we make this change?


Yes, this is very wrong in many ways :-).

The scenario where browseFile does not work is when you deploy the project to Azure Websites (see the recent blog post by @shanselman). The way this works is that IIS calls Suave (via httpPlatformHandler) and so the response from Suave goes to IIS and then back to the client.

I never got a perfectly clear error log out of this, but it seems that IIS *sometimes* cannot handle the responses from Suave. IIS returns “HTTP ERROR 502.3 Bad Gateway” with a note that “CGI application does not return a valid set of HTTP headers, or when a proxy or gateqay was unable to send the request…”.

So, I suspect IIS might be not understanding something from the Suave response, like compression or something… but I don’t know what exactly is wrong.

You should be able to reproduce this just by deploying an older version of https://github.com/tpetricek/Fun3D.Testing (before this fix).


`strategy.computeHas("secret", "S@l7", "SHA-512")`

If you use a keystore you could have:

`strategy.computeHas("secret", "S@l7")`

So if the server is compromised in terms of security it will not be obvious the algorithm used to hackers will not quickly reverse it to try to crack the user database...


This should be a nop - see comment on last RedisException case


@michbert gotcha, thanks—maybe you could open up a PR against our repo and we could discuss the tests there? (It just preserves the history of our conversation better.)


> Why generating a SVG filter when a spread value is here, as it's not defined in spec ? Shouldn't it be better to leave untouched ?

Didn't know. Good point. My original objectif was to avoid the error thrown by this module.

> do you have feedback of cssnext users about this feature ? Is it not already obsolete ?

No idea, I don't really care. I've introduced a feature, Firefox ESR probably still need this for now so I am not going to introduce a breaking change by removing it. We should better just do nothing to fix this bug (no error when using spread - some browsers already support it)


@DamianEdwards Can we make ```includeAncestorCultures``` optional parameter with default value = true?


Since this is something all users would need to do as well, at some point
we'll need migrate scripts. We could write some (this one should be pretty
simple) to start getting in the habit. We can then dogfood the process as
developers and get a good sense of what our customers will want.

-Joe

On Thu, May 21, 2015 at 1:52 PM, Nathaniel Kofalt <notifications@github.com>
wrote:

> I'm quickly losing track of these snippets.
>
> Would you be open to rolling these into release notes?
> https://github.com/scitran/api/releases/new
>
> Thoughts @gsfr <https://github.com/gsfr> @josschne
> <https://github.com/josschne> @andynemzek <https://github.com/andynemzek>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scitran/api/commit/56b0a18657e7062bc0857e79d2889e08ddb27c84#commitcomment-11313458>
> .
>


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


CSS outline not supported by: Opera Mini (5.0-8.0)


@ryanbalfanz I've made many different changes to the css and html to try and modify the web view for responsive mobile view, but I still can't figure out why my changes are not showing up on mobile view. I feel like I might be missing some kind of dependency, but I've checked through the links and scripts and I still can't figure this one out. Hopefully it's just a gotchya.


Trying to create a new answer without a value results in an error. I thought trying to edit an answer to be empty would want to do the same.


I think you are right, since I did not load it through json.loads I think
it is giving me a string of '[]' I'm running it right now with jsonFormat
set to false.

On Thu, May 21, 2015 at 3:57 PM, Jon Hall <notifications@github.com> wrote:

> Try printing the type of intentsRemaining. My guess is it is actually a
> string with the value '[]'. You probably want to run it through json.loads
> first?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/08900013e47bd70f3a420ea0880939f7f7d926df#commitcomment-11314723>
> .
>


for sorting the NamedQuery BuiltinUser.findAll


Feel free to make a pull request to https://github.com/gliaskos/freebsd-chromium when you feel ready.

Keep up the good work :+1:


CBECC (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2041 of 2158 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-94.58%25-yellow.svg)


By the way I have forked this repo and updated it to r30 + some minor fixes

https://github.com/syepes/freebsd-ports/tree/master/influxdb


@mojyt: this is the error we talked about for Action_Density.

for some reasons, DataFileList did not write data to disk right away. So it  turned out that `dslist.read_data(filename)` just read an empty file and made cpptraj raise Error.

So I made a nested function call and  things ok now.

You can check `calc_density` in `pytraj/common_actions.py`.

Hai


Hmm. For now I suppose, sure. We can revisit blank answers later with the non-response answers feature.


I meant .mo must still have old translation until new .mo is generated, which is now achieved. I updated .po files with 'fuzzy' line on last push.


I don't think that the support library version correspond to API levels. As far as I know the important part is 'v4' (meaning that this allows compatibility with API level 4 phones). Details for this are [here](http://developer.android.com/tools/support-library/features.html). Please correct me if I'm wrong here this is something I always thought was badly documented.


Awesome you are right! Thanks Jon, was going insane because of this...

On Thu, May 21, 2015 at 4:04 PM, Kelvin Flores <kelvin@onlab.us> wrote:

> I think you are right, since I did not load it through json.loads I think
> it is giving me a string of '[]' I'm running it right now with jsonFormat
> set to false.
>
> On Thu, May 21, 2015 at 3:57 PM, Jon Hall <notifications@github.com>
> wrote:
>
>> Try printing the type of intentsRemaining. My guess is it is actually a
>> string with the value '[]'. You probably want to run it through json.loads
>> first?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/opennetworkinglab/ONLabTest/commit/08900013e47bd70f3a420ea0880939f7f7d926df#commitcomment-11314723>
>> .
>>
>
>


Why did you remove this?


If we want to go with JS, we can use this http://stackoverflow.com/a/901144 and be done with it.


That's what I thought, too :)


Agreed, thanks.


Oops, that was unintended. I will push a fix.


@ryanbalfanz


хранится где-то еще в Интернете
лучше просто "хранится где-то в Интернете"


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


thanks a lot!


This commit provides a better and safer solution to the problem where links were interpreted as comments.


Some issues with multiple changes within one rock stepping onto one another.


J'ai annulé mon commit précédent et supprimé la fonction test pour voir si les vidéos sont présentes. Est-ce que c'est bon pour vous ?


Thank you, but this should be:
```
LDFLAGS += $(shell pkg-config mlt++ --libs-only-L)
LDLIBS += $(shell pkg-config mlt++ --libs-only-l)
```
Since I do not have a compilation failure on any of my systems, can you please verify my version still work on your system and update this commit if so?


Que vergonha


Note that if you ever decide to use action caching, you will need to create separate views for admins and non-admins. You can use partials to avoid duplicating code.


Nice! Do we need this inside a synchronized block? Same as isUiThread that I (hope) is thread safe...


This is a regression — it should be possessive, not plural.


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8), iOS Safari (7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), UC Browser for Android (9.9)


CSS outline not supported by: Opera Mini (5.0-8.0)


В первом предложении не хватает причины, для чего это нужно ( размещение сайта, приложения и т.д )


Why would you wait for the timeout to trigger when you can fail fast ?


`s/Lister/List/`?


skype: yakikhadafi


Ok, fixed in d86892f


Fixed~


Throw an error if it is a number stage is passed in.


Ah, no, using webpack for everything now :) Will remove on next commit


Пархоменко Александр


Just because globals are bad... I don't want to see something like User model and then you define var User = 'adfasdfasdf'; => all hell is broken lose.

Do-not-use-globals


CBECC (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


not your changes but thread creation is not checked


@artpard I've just noticed that there is a suspicious looking space in a value of 'name' argument. Is that line correct?


So, I found this on Stack Overflow… as I understand it, `Integer` is a class, and I'm passing `user_input` as a parameter and if it _is_ an Integer, it will return the same number as an instance of Integer, but if it isn't, it'd throw an argument error. Which is where `rescue` comes in… by bypassing the error and returning `false` instead?

I would've used `integer?` to return a boolean, but it seems to return a method error instead of `false` (plus, I want to test floats too). And testing `user_input` with  `[0, 1, 2, 3, 4, 5, 6, 7, 8, 9].include? (user_input)` wouldn't check for floats but would allow an input like `afj02`… is there a better way? I feel like I'm using `Integer` improperly here…


Ok, I understand your point. It is just this code is already done like that lock in random places... :<


You forgot to add the "," behind 'CourierKilledAlert'


:clap:


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Tests suggest that 3 helps.


Looks good! :D


I'm not sure I get what you mean.


return status would suffice here ?


Good call. How's this? I tweaked the language a bit to separate providing
and accepting into separate clauses, but the spirit of the suggestion is
maintained, I believe.

On Thu, May 21, 2015 at 11:14 AM, Satrajit Ghosh <notifications@github.com>
wrote:

> provide and accept constructive criticism on the work of others or our own
> work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arokem/nipy.github.com/commit/a87a87406f359fc628f27e486311727369efcebe#commitcomment-11312565>
> .
>


@ExSport this branch should make it so the framerate is taken from MediaInfo for all renderers https://github.com/UniversalMediaServer/UniversalMediaServer/pull/522


Thank you!

I got chromium to build and run successfully, but had to turn off the Video4Linux feature because I don't believe that FreeBSD has V4L2 in the ports tree. Other than that I'll be sending you a PR shortly!


The matcher uses flux to sort sources and reference objects. I was just fixing an outstanding bug in the unit test. I already spent 4 hours cleaning up the commit history, and a few things slipped through.


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8), iOS Safari (7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), UC Browser for Android (9.9)


CSS outline not supported by: Opera Mini (5.0-8.0)


Agree with damienlevin : we can return right away with a failure because we know it will fail. By implementing this way, you give the option to ignore this exception or handle it therefore getting some feedback right away.
If you ignore the request when disconnected, you are making a decision that will force the user to wait for a timeout.
With this implementation, timeout or imminent feedback are both possible in the user implementation.


Hummm… the conditionals were supposed to be erlang conditionals. We should try to always use `=<` which is the erlang version.


How about handling the ippicv special case outside of the generic case? something like:
```cmake
foreach (dep ...)
  ...
    if (NOT _isimported)
      list(INSERT ${_3rdpaty} 0 ${dep})
    else()
      list(INSERT ${_extra} 0 ${dep})
    endif()
  ...
endforeach()
# ippicv specific handling
list(FIND ${_extra} "ippicv" ippicv_idx)
if (${ippicv} GREATER 0)
  list(REMOVE_ITEM ${_extra} ippicv)
  list(INSERT ${_3rdpaty} 0 ippicv)
endif()
```
Since it is ipp, most likely no other  3rd-party should depend on it so this should be valid.


Note that you can also try
--help

I've tried with ubuntu and it worked ^^


there's a typo in the file name


gotta remember to use `next()` instead of `iter.next()` to be a cool python bro


Oops, also forgot that I'm listing the due dates on the book show view so people sorta know when to expect their hold to be all good


Nice! I almost committed the same thing in scheduler.py:
##[MerMod] Add .DLL and .EXE extensions to avoid error in .NET applications.
            if "DLL" in options["file_type"]:
                if '.DLL' not in options["file_name"][-4:].upper():
                    options["file_name"] = options["file_name"] + '.DLL'
            elif "PE32" in options["file_type"] or "MS-DOS" in options["file_type"]:
                if '.EXE' not in options["file_name"][-4:].upper():
                    options["file_name"] = options["file_name"] + '.EXE'
        return options


Hey Tom-- do you want to keep the debug stuff here? And are there any advantages to Kryo vs. native Java for all of the other stuff Spark is going to serialize along w/the data in the RDD?


This revised logic looks correct. I think you should remove the `this.close()` call, though--it should be unnecessary (`this.eos()` should return true as soon as `child.close()` is called), and operators don't open and close themselves.


We should probably just use "must not".


It is expedient to drop support for version 0. We don't have any version 0 data any longer and some of these keys don't exist in any form in version 0 tables.


Instead of invoking the reflection stuff every time, get this from the executor


heh. I'll destroy this. :)


Huh, did I forgot this? I could swear I set it to v120_xp in the first place.


0.89004 = GradientBoostingClassifier(n_estimators=200, learning_rate=0.1, min_samples_split=3, max_depth=9, verbose=1) & ytest.loc[idx, 'prediction'] = 0.045 # pure hack


@adasikow This space definitely should not be here. I will take care of it in next PR. As far as I know there should not be any errors, because this name is not in use.


Strip out the calculation logic from the constructor and move to the respective getter methods


I can't seem to find the method `Reaction.getDiffusionFactor(T)` in the current version of master, while the method is being called here.


Makes sense. I will keep it disabled and manually enable it when I want to play in the console. Thanks.


This file is 47 bytes, the archive is empty :)


With 0.13.2, I was passing props and query manually as follows:

```
Router.run(routes, Router.HistoryLocation, function (Handler, state) {
  React.render(<Handler params={state.params} query={state.query} />, document.getElementById('app'));
});
```

And could then access `this.props.params` and `this.props.query` within my components. I have tried updating to 0.13.3 and changed my loader to:

```
Router.run(routes, Router.HistoryLocation, function (Handler) {
  React.render(<Handler />, document.getElementById('app'));
});
```

And I'm not seeing these props come through, as suggested by the changelog. What am I missing?


Example for the getter for the details, it's the same method you have implemented but processing is being done on demand


Maybe something about "may modify results" instead. Possibly explicitly state that this is entirely at their discretion.


sounds good to me.


"Examples of incident that [X]:"


This is just SF


Added a new method for convenience


OK on the name change. My existing code makes it easier to add additional tests, but I'll change it.


They could, but this plugin will allow all OPS, and kick all non-OPs. It Is useful if there is one of those "staff testing only" moments.


I was reffering to
> I still would not remove the m_ImageProcessingLock lock just because we are already locked in the calling function.

I meant in EVR code it is random if it comes to locks. Like with this `m_SampleQueueLock` which wasn't lock always when it make sense. Or other cases where lock is duplicated or not. Nevermind. (it just looks like it was glued to somehow work)


@artpard Ok, I didn't know that this name is not in use yet.


Good catch


btw, this docstring is wrong. probably my fault.


BackgroundTaskDisplay will only hold reference to the backgroundTask and the backgroundTaskStatus, and the specific implementations will deal with how to transform the status to a UI consumable format or state like the ExportImportBackgroundTaskDisplay


To be honst here. I would use reference everywhere I could.
Like PGEX_Entry& someData = f_section.subTree[subtree].subTree

The reason why this is faster overall is that with the current function you always query multiple list. So you have to read the list for an item, which is another list where you look for an item, and then another list where you finally get the item and this takes a lot of time.


I think subscription means here that the user activated (subscribed to) the 2FA service. I wonder about it when I was working on the german translation. But on the day doing this I haven't see the translation in the frontend on my system so I did not changed it. Or is that the "do not use 2FA on this computer" subscription?
I agree with you that subscription is not a good use here. Let us change it.


What do you mean?
I'd say the require() is probably the only bit where we need a synchronized, so that we reach for the compiler sequentially, not risking to do weird things.
Or maybe you were talking about the getTracer() calls, and  suggesting that we start synchronization after the check?


Where it makes sense let's have two kind of getters, one with and one without Locale so it's providing more flexibility to the API consumer


We have a TriStateCheckBoxCell class which we use in cell views. We can't re-use that here but perhaps we can just make sure to share the images (or maybe we already do).


I rebased this commit out of master, to keep a linear history.


Actually not anymore. Relic from when state was a struct member.


strange, "0 is faster, but had a bug before 3.0.5" but master isn't on 3.0.5 of cholmod yet, is the comment referring to the overall suitesparse version number, or just wrong?


I don't know which commit caused this, but currently the modloader just doesn't work after starting a level.


Interesting point. It's worth a try adding interpolated to the good criteria (though I think saturated is still a valid test).

I'm not sure what your point is about interpolated.center and interpolated.any; do you think interpolatedCenter is good enough for rejecting sources with poor centroids?


i.e. for https://github.com/ceylon/ceylon.language/issues/687


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8), iOS Safari (7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), UC Browser for Android (9.9)


CSS outline not supported by: Opera Mini (5.0-8.0)


Не так давно, для запуска своего "проекта" в сети вам пришлось бы купить компьютер, настроить его на исполнение вашего бэкенд-кода и подключить его к интернету. В наши дни вы не только можете хостить собственные песни, файлы и почту в облаке, но так же можете запускать и свои сайты с серверов, которые хостит кто-то другой (возможно, даже в другой стране). Теперь услуги по поддержке серверов, необходимых для работы и поддержания вашего приложения онлайн, предоставляет множество компаний (например [Heroku](http://www.heroku.com), услугами которой мы воспользуемся позже, или [Amazon Web Services (AWS)](http://aws.amazon.com/)). Все, что от вас требуется єто "отправить" свой код необходимым способом.


Shouldn't `post` be named `firstPost`?


@jonboulle I like grouping const for related consts (like enums). These are not related. What's the advantage of using of using this form here?


Ok. I'll leave it here for now so that it can still be built until it has been proxied in our nexus.


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1145 of 1145 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.79%25-green.svg)


Actually, maybe not. In this case the third state is "use document default" and I'm not sure what the best look and feel for that is.


You did set it to v120_xp, though I apparently accidentally changed it back
at some point without meaning to.

On Thu, May 21, 2015 at 1:23 PM, Kevin Waldock <notifications@github.com>
wrote:

> Huh, did I forgot this? I could swear I set it to v120_xp in the first
> place.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wohlhabend-Networks/LunaDLL/commit/409e3a96d5692f072b32f106399bce0a4097464f#commitcomment-11315172>
> .
>


I realize this is just a test, but better style would be:

`List<TupleBatch> sourceList = ImmutableList.of(tbb1.popAny(), tbb2.popAny());`


Once I figure out what they do, yes, gladly.


Can you change text: String to stage: Stage?


r+, looks better, thanks


For now I'll use the first pattern, as its simpler for testing purposes (as it doesn't require the use of a jar).


71693606-CompilerWarnings1 (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@SubJunk You don't understand. The renderer conf is already MediaInfo capable as it is PanasonicTV. FFmpeg don't use info parsed by MediaInfo so it is autodetected by FFmpeg itself because there is no -r param (fps) before (specifying input file) filename or after it (specifying output file) as it was with MEncoder what I remember from past. In this case incorrect 50fps is detected for input file and reused as an output fps. For most people unwatchable(realtime is no way) for 1080p as in my case. Not sure if problem is rare or not but what I spotted when tested it, file was interlaced so maybe bug is visible only here. My father reported more files stuttering but only this one he remembered so problem can be wider or he watched mainly interlaced files at this day :-))


Double space -> single space
I would actually separate the second sentence into a new Guideline, though.


PHP shouldn't be too hard to use with Jekyll as long as we use `{% raw %}`
tags and can get the page to be processed by PHP.


This seems to be missing some of the additional functionality cgkineo added for how the progress would be shown on menus depending on what was optional in the children. I think this was mainly related to nested contentObjects:
https://github.com/cgkineo/adapt_framework/commit/7ecb057ae51ee91fe8b7df83f56acadaab014349
https://github.com/cgkineo/adapt-framework-kineoFork/issues/5

Might be we don't want that level of complication for adaptlearning and it may need some tweaks anyway, but the initial concept of how it should work was approved by Paul W at the time.


Thank you http://www.raywenderlich.com/84434/sprite-kit-swift-tutorial-beginners


I'd like to move those calculations to plp which is responsible for rendering the them. Keep everything in one place?


The way my code is written now, 2,147,483,647 is my magicNumber. It decreases by and consumes memory while doing it.


Nei, ned so ;) nullpointerexception dörfed eigetli ned uftrete, wenn die uftritt isch en programmierfehler passiert. Du rüefsch jetzt user.getSignature() uf und catchisch de nullpointer. Das heisst aber er chan überall uftrete (ned so wie spezifischeri exceptions) also heisst das user cha null sie oder innerhalb vode implementierig vo getSignatire tritt en nullpointer uf. Du willsch ja nur luege, dass user ned null isch. D implementierig dörf kei NPE werfe, will das ja en programmierfehler isch. Das verschleierisch somit. Darum immer null checks mache uf genau die variable wod bruchsch und ned en catch NPE. Das verursacht grossi debug problem ;)


I don't think you should get rid of the loop, even though this will certainly work. In principle, `TupleSource` should be able to return `null`, or any number of results less than the expected final count, on any invocation of `nextReady()`, as long as it returns all expected results before `eos() == true`. In fact, we should probably write a version of `TupleSource` that does just this (returns `null` or a subset of results randomly), and see which tests are broken.


Nitpick: indentation here is off. 2 spaces after `syntax-parse` is standard. May also want to put `#:literals` on the next line.


Actually, I would say the way you're doing is the most accurate. Using `.to_i` on a string that contains a mix of numbers and letters will succeed by just stripping out the letters, which may not be the behavior you actually want. If `Integer` finds anything that isn't a number, it barfs and throws an error, which the rescue you're using catches and executes its statement (which returns the value `false`).

And you're correct about array based checking for float types is error prone.

tl;dr: You're doing it right :100:


Can this be deleted?


This comment no longer applies if you're not closing the operator in `fetchNextReady()`.


I made a sort of work around, and I think it will work? Basically I just multiply the floats by a big number and cast them to ints so that they are unique ints to a certain amount of decimal points. It seems to be working, so I don't need it.


@Underground78 @XhmikosR

What do you think? Disabled item is the one selected as every time event in options.

![git](https://cloud.githubusercontent.com/assets/1126053/7758466/30e78346-000a-11e5-8973-87023e11005e.png)


Thanks. The comment is not relevant anymore. The old behavior was a preparation for the future as Tim Davis fixed a bug in `cholmod_symmetry` in version 3.0.4, but he forgot to bump the version in the C header. However, it was discovered in #11387 that, when using the `0` option, the function `cholmod_symmetry` doesn't just test for symmetry, but also fails on negative or imaginary diagonal elements. Hence, we'll have to use `1` even in newer versions. I'll remove the comment.


@nikolai-b That's really nice work. Well done!
Like you, I too would prefer JavaScript over ShinyJS package. Great work once again.
I was recently looking at the map-based applications when I came across http://www.censusprofiler.org/
I believe we can learn from this app, as it already has quite a few controls. What say?


I really like that you didn't hardcode colors or strings that is definitely something I myself need to work on


I think you should change your icon for your horoscope app if you keep working on it. I def need to do this as well


boolean formType - этот параметр здесь не нужен


Yes, sorry, that's correct, I should have deleted this. I had previously played around with some ways to correct rate for diffusion, but end up never using this method. Instead it's corrected in ```Reaction.getRateCoefficient()```, which uses methods in ```rmgpy/kinetics/diffusionLimited.py```


Are type definitions supported or not? If they're not supported, it sounds like we should delete this.


fixed


Если в createContact явно прописал rebuildCash(); то здесь кэш обнулять уже не нужно.


Delete comments, also line 5.


I was saying...that with introduction of the ConcurrentHashMap, do we still need to sync on require()? But I was wrong as ConcurrentHashMap does not guarantee that .contains sees concurrent .puts ...aka it does not lock the whole data structure...tricky...if you removed synchronized we would risk requiring twice che same namespace..


расскоментировать


I can remove the extended debug info - no need to keep it there. Kryo is generally recommended, so I'll put it back. New commit coming up.


@saghul, at this point, would it be possible to have a minor version bump with these changes (1.6.0)?


It is now before `require.js` is loaded!?


Au lieu de mettre `with asso_name=app.site.association.name` ici et pour les deux autres qui suivent, il serait plus propre d'utiliser [le module `set`](http://zds-site.readthedocs.org/fr/latest/front-end/template-tags.html#le-module-set) une fois juste avant le bloc `title` ! ;)


сразу пишем
return driver.getCurrentUrl().contains("/group.php") && driver.findElements(By.name("new")).size() > 0


Hey Alison,
I think that you should keeo working on this project on your own and try to use other widgets just to play with them and get comfortable with them. I really like that you didn't hardcode anything. I've been really bad about setting strings in the values strings file and keeping it separate from layout. Also, remember to add comments. I feel like a hypocrite saying this because I am also not good with comments. Just some stuff for us both to work on.


No one said it's hard. But again,  why use PHP if we can do the same with
simple Javascript code.
On May 21, 2015 11:35 PM, "Underground78" <notifications@github.com> wrote:

> PHP shouldn't be too hard to use with Jekyll as long as we use `{% raw %}`
> tags and can get the page to be processed by PHP.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/39cbfa563e7aed1f9397f74e87a1d508d3064ae3#commitcomment-11315381>
> .
>


Pareil ici, le module `set` ;)


Is this still needed? Its use is commented out above.


kyle was here


klçjklç


UI code is poorly structured, many duplications and unused files/methods... some names are also not clear and confusing. I decided to clean it out now and this is the first step.. aaa is just a temp folder that could be anything after i finish, but i just wanted that folder stay alphabetically at top.


tight


the only advantage is then I won't complain in code reviews...

I don't care that much but still consider it cleaner to frame them in a single block, with blank lines separating unrelated ones - c.f. import blocks.


Right now users of the client library have to handle timeouts in any case in order to ensure correct operation. I don't think the added complexity of an extra exception at the level of individual requests is worth the trouble, especially since there is a notification mechanism for clients that can pick up whenever a redis connection is lost or re-established. If you want to throw a new exception here, at least give it a distinct type (maybe RedisDisconnectedException?)


@nikolai-b You know what? While implementing tooltips on the server side, I found this bug in ShinyBS package, that if you don't have at least one tooltip defined on the client side, the server side won't work. To overcome this issue, I had introduced this empty tooltip. I guess now you've removed all tooltips, so of course, it is not needed any more.


Is this still broken, and which part doesn't work yet?


Yes it does.
For which command does it fail for you?

For example `ps aux FF` and `ps aux PE` both work for me.


You should indent your code.


@nikolai-b This looks really fabulous. Great work!


Being require() syncronized at this point ConcurrentHashMap is maybe not useful here..thanks for making my brain working on these issue, I am enjoying it :smile:


But when I do... http://i.imgur.com/heCWiGQ.jpg


Hmm, indeed there does not seem to be v4l2 support in the Ports Tree, other than GStreamer plugins.


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I don't understand this fixme comment. Can you expand the explanation or fix it?


Ditto. Seems to be 8 spaces instead of 4.


Ok,  I'll consider it and will let you know tmrw.  I'm still not sure if I
understand  the problem correctly.  Do we need the bash popen because the
pipe commands  in egs use  the bash syntax (wildcards expansion,  variable
substitution and so on)? Or is there some other,l reason I fail to see?
AFAIK,  one can run cmd pipe even using cygwin  commands without any
problem.

About the paths  tinkering -- why do we need it? Kirill,  can you provide a
specific  example?  I thought  the paths are mapped transparently by the
cygwin runtime.
Y.
On May 20, 2015 11:57 PM, "Kirill Katsnelson" <notifications@github.com>
wrote:

> @danpovey <https://github.com/danpovey>: No, this would mean no egs/
> scripts. These need bash even on Windows.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-11294456>
> .
>


Ditto


For me it's a server side matter but I'm guess I'm old-fashion when it comes to web.


I think I prefer `change`, but I'm not a native English speaker.
For me, "switch" is more about "turning on / off", while "change" is like the meaning of "change clothes" or "change bus lines"


Now that's a solution which everyone agrees on. Nice job!


This export and definition should probably be removed.


We also want the form disabled if the formState is 'saveStarted'


It's a shame this had to be reverted.  Is there news on why and if it's going to make it in 1.9.3?


This was supposed to be fixed in the PR that added this explanation, but it somehow got lost in a rebase.


Tipp: Solche Updates kann man ganz einach vermeiden, indem man Jekyll den job erledigen lässt:

```html
<p>© {{ site.time | date: '%Y' }} Sitename
```


@gutard there it is, this is deployed on the node you know ;)


suggestion: remove "to begin"


Certainly for centroids, but also in general, you don't care about bad pixels in the remote outskirts of an object (which would set `…interpolated.any`), so `…interpolated.center` works fine for rejecting bad centroids.


Valeu @junior-ales! Um detalhe: cuidado com a formatação do XML! Estamos tratando o texto do bloco `<descricao>` como pré-formatado, então qualquer espaço em branco ali vai parar na página. Usar um bloco `CDATA` resolve: https://github.com/servicosgovbr/guia-de-servicos/commit/32564bcbed8107cdacc9e1385998d45ffb48974b


We want the confirmation message to appear if the state is 'saveStarted' (which is also a disabled state)


LOVE this quote!


71693606-CompilerWarnings1 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1656 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Most recent change: iframe now docks at the bottom of the screen. Opening (via focus on input box) and closing (via button on the iframe) works as before.


Hmm I'm not sure what to think. It makes sense I guess.


>16 additions and 68 deletions.

Awesome! :+1:


@cmdcolin  / @deepakunni3   Not sure if the functional tests will be helpful or not.   Was trying to reduce startup for something else.  I think I'll still end up with integration tests.


It's more that each bullet point has a specific and good reason, and none of them should be superseded by popularity.


After looking at this package in PyPI I'm not sure that this is relevant.  We don't adhere to the google style guide.  Should we add this?


May read a little more smoothly if you change the first like to want: "you want into a manifest and apply them as you like.


@rodfersou please study this code carefully


Suggestion: Start this with the However, rather than interjecting it.


Cool, I learned something :smile_cat:.  I was conflating **no** initializer (you get junk) with value initialization, which does something sensible.

I had previously submitted a pull request that fixed some compile warnings, and my misunderstanding caused me to use `std::fill` to iniailize some arrays.  I will revert those back to value initialization.


@gbadner +1, and beyond: query comments are usually not needed at all, unless they get logged. Could they be generated only as needed? In that case you could make the message as nice as you want - even using string concatenation.


Okay, now I am not sure that I can edit your commit like I thought... I am just not going to commit it.  Looking through your changes, I just see that for the leptons the only relevant lines are addition of the max eta requirement in the main config file.  I'll just add these by hand, since its just a few lines.


Good catch about the MAX_INT. The point I'm trying to get at is this: If the number is greater than 18, it is set to 0. If it is less than 0, it is left alone and the main function will run to completion. Do you know of any more efficient way to perform the behavior I've just described? (hint: a while loop that decrements by 1 n amount of times is NOT the most efficient way)


funny that the '// TODO' was there all this time :)


@iSaulius

I have just checked my OpenCart menu, and I can't reproduce your bug. It is working for me.
I have tried to debug you website (mouse right click), but it seems that you disabled this option.

If you need help, you will need to enable the mouse right click option so I will be able to debug your website for a solution.

BTW: are you using the default OpenCart menu or an addon?


This is the main area locating workflow. You will need to save it to the appropriate directory on your system


Feel free to revert this line/change the way I do it, I'm just pushing this before I go to bed.


Ah excellent, that sounds good to me :) :+1:


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


"have an RSpec-style syntax"

then this is the one I am trying to supersede. not sure how best to do it. Golang testing isn't rspec but it is the std way.


See above comment. Also, vanilla CC checks checks that it *isn't* a cable, whilst I'm checking it *is* a modem with cable - as that is the only time you can right click to activate it.


Put another way, what's the right thing to do when you get a RedisException from a request you've made? If there are multiple possible causes (not just disconnections) you would at least like an exception that is more specific about it being a fail fast caused by a disconnection. And then, what's the right thing to do? Back off and wait before reattempting the request? At least by allowing the initial request to timeout, there's no temptation to spam the request over and over again without waiting for the connection to be reestablished (and this could just be the connection to a single shard, only affecting some requests that the client is making). If the request fails fast with a new exception that isn't a timeout, now the client is responsible for implementing extra recovery logic beyond what it would use for normal timeouts - or it is at least responsible for finding a way to fit handling the new failure type into the ordinary timeout recovery logic


Got a little lost with pronouns and antecedents here: "As you saw when you applied the `graphite`class in the Power of Puppet quest, classes can take parameters which allow you to adapt their function to match your needs."


I've got this commented out until I get around to explicitly defining the mappings for this document. Otherwise, it ends up getting indexed and breaks specs.


It's not that far away, https://github.com/ContinuumIO/dask/blob/master/dask/async.py#L383-397


Hello tout le monde,

J'ai exactement le meme probleme Nibeck1309, en manuel il passe en Snatched mais en automatique meme en passant par Manage Searches il ne trouve absolument rien. Tester avec Cpasbien et KickAssTorrents.

Test sur un Raspberry Pi 2 et sur Macmini OSX avec la branch Master & Developper idem.

Par contre avec la version de base du site https://github.com/SiCKRAGETV/SickRage ca fonctionne sans probleme.


What, commits aren't markdown supported???


Tönt ganz geil ;) :+1:


Hi myk002. Compilation of this line produces a warning (or an error with -Werror which is enabled per default in the autoconf compilation method).

src/Interface/ComboBox.cpp: In member function 'size_t OpenXcom::ComboBox::getHoveredListIdx() const':
src/Interface/ComboBox.cpp:245:9: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]


"we will" - usually you use the contraction, I think.


All an `Importer` needs to do is implement this method.


check is redundant.


redundant


I kind of agree, we would have a lot more flexibility to generate different text and maybe screenshots.


Du hesch das wahrschinli eifach mal für de afang mit strings gmacht oder? Da gäbtis suscht besseri datetype..


"a few different" ways: don't need different ... suggestion: "There are several ways ..."


Should define those at point of use


delete "though"


Test


All an `Importer` needs to do is implement the `available_version` method.
I've got the `update_metadata` call commented out until I figure out how to tell ES not to index this new document (bad things happen, specs break, etc.).


Indeed if only ever read and written from synchronized code, having the HashMap be Concurrent is a redundant protection. But keep in mind that if you want to add read-only access to it, then (e.g. for consulting which namespaces are currently loaded from a user plugin - just musing here -), then you'll have to also do that call from a synchronized block. So reads will block writes.
With the ConcurrentHashMap, you can just let some "read-only" calls do their stuff without the need of synchronizing access to the reference.


In this case I was talking about the progress indicators you may have on menus rather than the pageLevelProgress extension. If the progress is displayed via various modules/extensions then it may make sense for the model to be responsible for the calculation so you don't have duplicated code, but you have a better overview of everything than I do.


Now I get to start a -test=0.   I wonder how well that will go, lol


Thanks!  You need the latest version of leaflet for this to work but we now have access to the Leaflet map object so can do loads of funky stuff that isn't available in Shiny.


Thanks, I'll do some work today.

On Thursday, May 21, 2015, PEMapModder <notifications@github.com> wrote:

> whatever you like, just to remind you [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/1d876d44609e32d30c3bda30745555d52a45ab49#commitcomment-11312050>
> .
>

--


Will do. Although not major. Only adds a label. Should we wait until
settings are fully implemented?
On 21 May 2015 20:56, "Evan Lucas" <notifications@github.com> wrote:

> ah missed this one. I guess I had already pulled the patch. Mind opening a
> new PR with this commit?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jwrb/paycoin-electron/commit/35dbedf7150ac14e435a591c14c9d7440e611e9f#commitcomment-11314719>
> .
>


I also need animations at load!


I think it makes sense too. Although, last time we changed this stuff we had people complain, but you know :p


Below (I could not comment in line), I think there is an extra "a": "It may not be a mission critical software ..."


I now have "xclip -o" working with no hacks.


Also:
> with Sphinx under sphinx.ext.napoleon. The sphinxcontrib.napoleon extension will continue to work with Sphinx <= 1.2.


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1656 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


```arc_evict_ghost()``` does actually exist in the previous commit "Illumos - 5408 managing ZFS cache devices requires lots of RAM".


Hey, I’d like to point out that this is totally a breaking change. Just a patch bump for this? The format of the data is totally different! Could have saved me an hour or two of head-scratching.


@jordansissel
Don't you also want to implement this same fix fix the Elsif block? My config wouldn't compile until I implemented this fix for `If` and `Elsif`


Ha!

So I feel bad about removing all the tooltips as you spent quite a long time getting them all working and looking nice.  It can be easily reverted but I wanted see how the map felt without them for a bit as they kept obscuring parts of my screen, half appearing then vanishing, not vanishing when I my mouse moved away.


any reason why the files are coffee, but the tests are javascript?


Typo in commit message "Te 'sslCiphers' option "


"Должно быть вы удивитесь, но вы увидите некоторое структурное подобие Rails,"
"Возможно вы удивитесь, увидев некоторое структурное подобие Rails,"


Well, for me, it's a static site and should stay that way. And if we really need something like that, then we should start reconsidering the drdump choice...

**Again**, make a PR and we can discuss it there, along with the actual patch. Now, it's just theory.


@datakurre where is the image stored? in collective.cover we have a huge issue right now with this and the use of plone.app.iterate; at some point we end with zillions of empty blob files and we haven't been able to find the source of the bug.

@cguardia can give more information on this.


Multi-line `else if` and `else1 conditionals should be tested too.


suggestion: "At this point you already have both packages ..."


to be symmetric, shouldn't also you check if `/usr/bin/kadmin` exists too?


that "ignore that for now" note does not need to be there; as an in-progress project people can assume this'd be fixed.


We should use double type because  13! exceeds unsigned integer maximum of about 4,000 million on 32-bit platforms.


github ftw


I'm not an expert here, so just out of curious, why is this line so long?


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


That will work, but you don't need a regex there, and you don't need a grouped regex.


This is not C free(). Are you really sure you can pass a NULL ctx to it? Docs don't state that you can, so I'm assuming you can't. The OpenSSL example code also had NULL checks, FWIW.


See above.


I guess he should have searched the pull requests before submitting :P

This patch is almost a year old; I've moved on from both winston AND syslog, and don't appear to have this code available anymore. Judging by the other patch, a one line change shouldn't be too troublesome for you if you decide to merge this one.


Good point. I'll do that. I've also disabled the code with #if 0 instead and documented the parameters as requested above.


com.google.gwt.thirdparty.json.JSONException: No value for file
	at com.google.gwt.thirdparty.json.JSONObject.get(JSONObject.java:355)
	at com.google.gwt.thirdparty.json.JSONObject.getJSONObject(JSONObject.java:574)
	at SearchUs.server.logic.SearchManager.getSearchResults(SearchManager.java:72)
	at SearchUs.server.dispatch.SearchActionHandler.execute(SearchActionHandler.java:24)
	at SearchUs.server.dispatch.SearchActionHandler.execute(SearchActionHandler.java:15)
	at com.gwtplatform.dispatch.rpc.server.AbstractDispatchImpl.doExecute(AbstractDispatchImpl.java:154)
	at com.gwtplatform.dispatch.rpc.server.AbstractDispatchImpl.execute(AbstractDispatchImpl.java:110)
	at com.gwtplatform.dispatch.rpc.server.AbstractDispatchServiceImpl.execute(AbstractDispatchServiceImpl.java:87)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at com.google.gwt.user.server.rpc.RPC.invokeAndEncodeResponse(RPC.java:587)
	at com.google.gwt.user.server.rpc.RemoteServiceServlet.processCall(RemoteServiceServlet.java:333)
	at com.google.gwt.user.server.rpc.RemoteServiceServlet.processCall(RemoteServiceServlet.java:303)
	at com.google.gwt.user.server.rpc.RemoteServiceServlet.processPost(RemoteServiceServlet.java:373)
	at com.google.gwt.user.server.rpc.AbstractRemoteServiceServlet.doPost(AbstractRemoteServiceServlet.java:62)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:755)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:848)
	at com.google.inject.servlet.ServletDefinition.doServiceImpl(ServletDefinition.java:287)
	at com.google.inject.servlet.ServletDefinition.doService(ServletDefinition.java:277)
	at com.google.inject.servlet.ServletDefinition.service(ServletDefinition.java:182)
	at com.google.inject.servlet.ManagedServletPipeline.service(ManagedServletPipeline.java:91)
	at com.google.inject.servlet.FilterChainInvocation.doFilter(FilterChainInvocation.java:85)
	at com.google.inject.servlet.ManagedFilterPipeline.dispatch(ManagedFilterPipeline.java:119)
	at com.google.inject.servlet.GuiceFilter$1.call(GuiceFilter.java:133)
	at com.google.inject.servlet.GuiceFilter$1.call(GuiceFilter.java:130)
	at com.google.inject.servlet.GuiceFilter$Context.call(GuiceFilter.java:203)
	at com.google.inject.servlet.GuiceFilter.doFilter(GuiceFilter.java:130)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1474)
	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:499)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:137)
	at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:557)
	at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:231)
	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)
	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:428)
	at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)
	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)
	at org.eclipse.jetty.server.handler.RequestLogHandler.handle(RequestLogHandler.java:68)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)
	at org.eclipse.jetty.server.Server.handle(Server.java:370)
	at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:489)
	at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:960)
	at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1021)
	at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)
	at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)
	at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:668)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:52)
	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)
	at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)
	at java.lang.Thread.run(Thread.java:745)


Why did you change the lines to the exact state as before?
I think anyway your topic should not be a pull request. For personal question I would suggest a private message.


Something's not right here. This is a list, but you don't any list options -:
eg -:
```xml
			<Item>
				<Value>1</Value>
				<Label lang="en">Map OFF status to all devices in group 1, Double click on key 1 will send ON to all devices in group 1, all dimmers set to prev.value</Label>
			</Item>
```


We can't differentiate between syntactic errors and the requested ciphers not being available. Maybe we can squeeze that in there somehow.


Probably, but I don't really know what to fall back on if neither exist. And I'm figuring we'll remove this once we upgrade everything to jessie anyway.


Weiss ned eb das so en gueti idee isch alles als string zspeichere und denn als zahl umzwandle ;) aber isch wahrschinli nur temporär so. Chasch ja ame es todo dratue


Ok - so the items are down here - I suspect you're just missing the first ```<item>``` tag. Have you actually checked that the XML is formatted correctly?


We could avoid duplication here by doing something like:

```
if [ -n "$CHUNK_SIZE" ]; then
    CHUNK_ARG="-n $CHUNK_SIZE"
fi
lvconvert -y --zero -n -c $CHUNK_ARG --thin ...
```


Ok - my mistake :) The formatting confused me - it's probably ok...


Typo : it is "flags"


Where? minDistSq is not const, and the arguments are POTs and so need not be const (except the first one, which is const).


Ciao @Ithildir !
Certo che mi ricordo di te! Allora hai fatto il grande salto oltreoceano ? :+1:
(FeedingKnowledge continua.....ho da poco ultimato upgrade dalla 6.1 alla 6.2!) :)

Ok...I made this PR after James comments on http://www.liferay.com/web/james.falkner/blog/-/blogs/liferay-6-2-ce-ga4-now-available about gradle in SDK....so I didn't know this move on [Gradle plugins] (https://github.com/liferay/liferay-portal/tree/master/modules/sdk)....I will try them!

Thanks!


@rzarzynski you can squash this one into the first commit


Links for related pages will need to be inserted when we get the URLs settled.


remove a ;


Not actually related directly to this, but I discovered today that the rawhde installer at least crashes if you try to use `--chunksize` in kickstart files.  Possible fix: https://github.com/rhinstaller/blivet/pull/121


Links to GJ pages will need to be updated to wttf pages.


remove the é;


Thanks for making the file - sorry about the confusion with the review - I missed the top Item tag due to the formatting being a bit screwy.


Sorry took me a while to get to this.

Looks good. I agree this is the right way to handle beatCount - so it is always a valid value, and semantically is the "start" of a beat, like the drop of a kick drum.


You could just say

`assertEquals(limit, tb.numTuples());`


I admit I have no idea where the 5 comes from. I'd rather not touch it without a deeper understanding of the code.


makes sense. Will change.


why do we need to have this microoptimized? surely we can use a more human readable value, e.g., number of seconds to round to?


This assertion should stay, but the following comment should be deleted (since you shouldn't be closing the operator itself).


I relied on the OpenSSL documentation for those functions. See https://www.openssl.org/docs/ssl/SSL_CTX_free.html


I relied on the OpenSSL documentation for those functions. See https://www.openssl.org/docs/ssl/SSL_free.html


What about caching eviction ? https://code.google.com/p/guava-libraries/wiki/CachesExplained Or is the number of jars limited enough to avoid having to manage memory ?


This comment no longer applies if you're not closing the operator in `fetchNextReady()`.


Yes, I see now too. It is a my problem with tabs and blank spaces. Shall i correct it?


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2132 of 2211 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-96.43%25-yellow.svg)


As you want, it's just I'm a const/static addicted and I can't do that if I declare it at the beginning. But there is no need for const here.


Thanks. I see that it does say that, now.


maybe we should have an io context for the log pool ready on the RGWRados class? (we have that for other pools)


I tweaked the earlier patches in the stack to fix this.  We didn't have 5056 ported when I started working on this.


Alright, that's plausible :)
Will Stick to that.


Sure! I will change it ASAP


Yep.


Oh that's great. I have just updated leaflet and it seems to be working fine.


@hvelarde They get stored into annotations of the content object, next to the tile annotation:

    {u'plone.tiles.data.afc7d64755b345e68ba31b217829522a': {'image': <plone.namedfile.file.NamedBlobFile object at 0x10dd44500>, 'scale': 'mini'},
      'plone.tiles.scale.afc7d64755b345e68ba31b217829522a': {'e471a8ca-2e3a-4190-980b-d4ce994bb8e3':{'mimetype': 'image/png', 'uid': 'e471a8ca...

But now when you said that, I see that nothing currently removes those scale annotations. They need a custom delete form, which I must add later... (and adding possibility of custom delete form was the backwards incompatible change in p.a.tiles 1.x and 2.x).


really... that is so op


sos_recon_coeff.dat is in DATFILES directory in most Hall C replay. So change PARAM to
DATFILES.


@hastalafiesta-
https://github.com/ChronoMonochrome/android_device_samsung_codina/commit/3b7f9ecb9af931439f1beb53af74ca41adfc97d6

Il file init.u8500.rc ce l'hai in android_device_samsung_u8500-common/rootdir/, mentre l'autore del fix lo ha portato direttamente in android_device_samsung_codina.


Right, and there are a few reasons for that point:

1. RSpec syntax is extremely readable, and with the `it` statements, usually does a good job of being a temporary replacement for documentation of a package or module.
2. Standardizing a testing syntax across languages at elos would make the barrier to entry for contribution lower than it otherwise would need to be. If I want to contribute to the golang codebase, it would help if the syntax for the tests was familiar. One less thing to learn.
3. In a similar vein, figuring out the usage of a module would be much simpler if the testing was in a familiar syntax.
4. It is easier to establish best practices company-wide for testing.

Not using a standard syntax is a loss in all of these respects, and I don't see how popularity could outweigh that.


It’s ok - I’ve merged, but I was just thinking I might run the formatter over the complete database.


Oh well, we might bring them back together. To be honest, I was thinking of having a configuration tab (or some control) or something where users can switch off additional help (like tooltips). If you are an advanced user of this app, you wouldn't require these extra bits any more.


Is this ever going to become official? :cake:


Would be good to drop a TODO here to be clear why this is commented out, and when this expected to be removed. (Looks like this is a separate change though)


Feel free to make a PR.


This comment no longer applies if you're not closing the operator in `fetchNextReady()`.


You could also write this as

```clojure
(->> chain
  char-seq
  (map dna->rna)
  (join ""))
```


Is this `do` necessary?


yeah, saw that


I've made [this gem](https://github.com/localhots/subzero) as a joke four years ago, and now Rails got serious about it. Marvelous!


Seems nice. I like the idea at least. I let @deepakunni3 know that the net.sf json modules were failing and recommended jsonbuilder instead so that our travis-ci gets fixed again


If you restore the `close()` here (it's not strictly necessary for a test, but good form nonetheless), you should be sure it comes after we test `limiter.eos()`.


> If I'm not mistaken, you need the header but not the binary. That means that you don't have to add GameMap.cpp to the source list of the test (and all the other needed files that comes with it).

Yep, that's exactly what I'd like to test. But once again, let's not wait for me since I don't have much time lately.


Du hesch ja noimet gschriebe dassmer jetzt nachem checkout vo oisem tool na manuelli apassige münd mache. Findsch das e schöni lösig? Ich weiss ja ned was gnau gänderet, bzw welles file genau, aber ich hettis versuecht als overlay zmache. Bim style isch das ja zimli easy, bim index.html settis aber au ga mit de richtige inclusion patterns. Oder suscht als eigete fork vo swagger-ui wo denn als jar verpackt in maven rückimportiersch. Isch momentan sicher e pragmatischi lösig, aber villicht gahts ja na konzeptionell schöner :)


How would I go about private messages on github?


Fixed misspelling


did you mean canols?


Jetzt bisch ja voll de angular pro :stuck_out_tongue_winking_eye:


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Probably not :grin:


Isn't this exactly the same as V3 (just formatted a little differently?)  Would be good to have it similar to how versions 1+2 are documented (Frames for protocol versions 3+4 are defined as:)


Single ampersand by intent? You should add a comment then.


Vinicio Guachamin


Chaschmer denn das mal na gnau erchläre was du da machsch?


SHould this be commented?


this will never throw, use
```js
if (!(type === 'invoice'))
```


Why wouldn't the `_target` be `readonly`?


First Attempt at making the function


Sorry ned gseh dasses uskommentiert isch :stuck_out_tongue_closed_eyes:


That line is one of our protractor test files. The gulp build task takes the whole /master directory and processes it to generate the /app directory. As a part of that processing, the js is minified and uglified, hence the long line.


There will be one, but I want to get some more changes in first, so no ETA yet.


Hahaha I registered but this is suuuper random. It's an HR software
training session??!

On Thu, May 21, 2015 at 2:40 PM, botbotbotbot <notifications@github.com>
wrote:

> Wanna go to this thing when you're visiting? It might be boring, I don't
> know anything about it, but there's free drinks and food
> https://learn.zenefits.com/2015-roadshow-registration.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11313171>
> .
>


One large commit for linux, one hard push for linux kind.


no


https://github.com/docker/docker/pull/10682  It was rejected.


When i try to execute autogen.sh, it finishes with

```
config.status: error: cannot find input file: `Makefile.in'
```

a Makefile won't be generated.


You don't need the else statement on line 20 of Formula.java as the x values are already set to -999 at initialisation.


Отлично! Только поправьте отчество и старайтесь осмысленный комментарий к коммиту вводить :+1:


provato


Can one of you guys update this; I'm curious about what you're working on


the `parameters` field should be `_parameters` to be consistent with other field declarations, and can be `readonly`; its reference doesn't need to be reassigned anywhere.


Good idea.


@datakurre well, you really have to take care of that before it becomes an issue.

some references:

https://github.com/collective/collective.cover/commit/bf386feef911e0456b86a14edc5dd58a3ee20fec
https://github.com/collective/collective.cover/commit/6b0216b81847590a979dc8b94b1c680c5e971862


Made changes to how the elements change to the size of the browser.
-Still need to work on title


Would be awesome if we could use trailing closure syntax here...

```swift
PFFacebookUtils.logInInBackgroundWithReadPermissions(permissions: [AnyObject]?) { (user: PFUser?, error: NSError?) -> Void in
  <#code#>
}
```


Dear Dvir-Julius,
i disabled copy protection, menu is default OpenCart.
Thank You!


Need to add `link = decodeURIComponent(link);` before the `var method...` line
Without it, some search queries (eg. involving email addresses) don't return the proper results because the query string is encoded when sent to gmail


add another line


The reason your validation is not firing properly is that you are targeting an ID called `form-group`, but there is no such thing on the page. The ID you should be targeting is `order-form` -- which is the ID for the form itself. Each form input is contained in a div with the class `form-group`.

Other than that original target, it looks like your code is accurate, so fixing that line should fix the issue.


To Do:
- Save all variables
- Load all variables


Hello Unknown! How is Unknown today?


Yann pourrais tu changer les deux images de fonds de la page d'accueil de Edgefest. merci
![fondbleu](https://cloud.githubusercontent.com/assets/7852945/7759549/a53f9eb0-0012-11e5-89db-14ca3fc6bb02.png)
![Uploading LOGOedgefest-2015.png…]()


If msgpack::move() is supposed to be a polyfill for std::move() then I should be able to use it both with `MSGPACK_USE_CPP03` defined and without (in C++11 mode). Is this #ifdef actually necessary or can both just use the same code? See also other functions for the same pattern.


Probably use `"/ipns/"` (with the trailing slash) in the conditional.

I'd prefer:

    if strings.HasPrefix(p.String(), "/") || !strings.HasPrefix(p.String(), "/ipfs/")

so we can eventually support other protocols besides the current “try all our resolvers” logic we bind to `/ipns/…`.  For example, early passes in #1208 had the DNS link stuff under `/dns/…`.  That was too much to bite off in #1208, but I still think we'll end up with an explicit-protocol system like that eventually, and I want to keep that door easy to re-open when we do decide to move in that direction.

Drop `/ipfs/` from this comment to catch up with the condition change.  Since this reduces our resolved protocols to just the multi-protocol /ipns/, maybe rephrase the comment to:

    // namespaced non-/ipfs/ path (e.g. /ipns/...)

On the other hand, I really want to roll the pathresolver logic into the recursive namesys helper, since resolving and parsing paths really has very little to do with how your entries are stored.  That's a bigger refactoring, so I don't mind tearing up a bit of IPFS special-casing here when I get around to writing it up ;).


Important change. Major security flaw fixed.


Dude you should not remove it, Command Blocks and Enchantment Tables are solid


This could be an exception actually. Feeding anything other than a parameter declaration to this constructor would be a programming error.


braces?


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-5.29%25-green.svg)


For managed deployments, Flywheel will take care of the upgrade and migration. For unmanaged deployments, we'll need to provide migration scripts. For now, Michael is managing our deployments, and developers should just reset their environments whenever an incompatible API/DB change happens. I'll also try to be more mindful of communicating changes. None of these are on the demo branch, though.


Typo here, should read "push" ;)


Çok zorlama bir kelime gibi geldi bana.


Nice! It's so simple, but makes such a big difference. And then you can style them, too!


Initial Commit


Change function so it returns whether it was successfully added.


Got it.


Oh I see what you mean. I would argue that updates in Task (I can only think of tracking symbolic variables) will be different from the updates in the Optimizer (updating the model parameters).


Thank you for changing this!


Eh si, e' quasi un anno ormai che mi sono trasferito a Los Angeles! :sunglasses: Son contento sia tutto ok, ogni tanto torno su FeedingKnowledge e me lo riguardo un po'... ero proprio contento di quel lavoro!

Fammi sapere! Grazie ancora :blush:


71693606-CompilerWarnings1 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


'd.name' or 'd.metadata.name'?


Fine by me, as long as @Danimal696 still has a copy of the normal somewhere. Ideally we'll fix this issue that normals and opacity don't work together so that we can use both :)


That's true, thanks!


Sorry for the trouble @andrewbranch! Like I said in the readme, this library will follow semantic versioning only after JSON API hits 1.0 (which is happening today) and this library is updated to conform to the new 1.0 changes, which should happen by end of week. Until then, the JSON API spec has been changing so fast (and I'm still playing catch up on the new features), that, if I made a major release for every breaking change, we'd be at some insanely high version number by now, and all the old versions would be obsolete (because not spec-compliant) in a few days anyway.

If you're using this in production, I'd suggest waiting a few days until this library hits 3.0, which will match with the 1.0 version of the spec (again, that's being finalized today), and then make all the changes to your client at once!


@kemitchell I don't think you actually need this delay because the rate limits are bucketed by the hour, you can check your limit like so (assuming you have `pip install httpie`):

`http GET https://api.github.com/rate_limit?access_token=$GITHUB_TOKEN`

You'll want at least ~3,000 calls left.


It is interesting we thought about that and at the time we were afraid it would be too slow doing a some stopping criteria testing  at each update. I'll add an issue for that as we shouldn't care about speed at this point.


Edited to fully describe what this project is about and where the data should be.


I didn't look into it yet.


@iSaulius

Got it. Change your owl.carousel.css (starting at line 81) as follow:
Instead of:     z-index: 1;
change it to:   z-index: 2;

It will solve your problem.
Let me know.


Haha, yeah, no clue, someone invited Allie and she forwarded it to me

On Thursday, May 21, 2015, jxcache <notifications@github.com> wrote:

> Hahaha I registered but this is suuuper random. It's an HR software
> training session??!
>
> On Thu, May 21, 2015 at 2:40 PM, botbotbotbot <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > Wanna go to this thing when you're visiting? It might be boring, I don't
> > know anything about it, but there's free drinks and food
> > https://learn.zenefits.com/2015-roadshow-registration.html
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11313171
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11316698>
> .
>

--
latest Vidcode:
*Vidcode in Intel
<http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
*Vidcode in Fast Company
<http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls>*

*Leandra Tejedor, *Designer and Front-end Developer

*vidcode.io <http://www.vidcode.io/> / @vidcode
<https://twitter.com/vidcode>* / *Insta*
<https://instagram.com/vidcode>* /** FB
<https://www.facebook.com/myvidcode>*


I think you're right. ``yield``(ing) anything but a ``self.call(0`` or ``self.wait(0`` call is useless in terms of coroutines.


FIXME


@hvelarde In stdtiles, I'll handle it in the edit form.

About empty blob files. I'm seeing such being generated for versioned DX content blobs (into portal_historiesstorage).


// l


you also killed `ace-jump-buffer` when changing to `avy` as well


Why **MUST** this? this makes having any parameters a breaking change in the future


Modifications:
- adding an empty space for the first fprintf with one less loop iteration
- removing the if statement
- adding argv[argc-1] for the last fprintf (with the \n)

Purpose: eliminate argc executions of the if statement and argc-1 calls of fprintf

Improvement: the produced machine code will have less instructions and less function calls


At the moment I think we've got an opportunity to have a quick review of it all. I want to see whether plp / core / menu's are best suited to it. It could also be more of a 'menuLevelProgress'. We got options anyhow. The code we have is still applicable, just where is the question.


https://www.openssl.org/docs/ssl/SSL_CTX_new.html says that method is deprecated. Do we need it for backwards compat?


:+1:

Detail: je zou alleen nog `$('#robots').html('')` kunnen doen voor de `for` loop en daarna evengoed `append()` gebruiken, is misschien iets netter.


object names are all lower cased, so replicationConroller >> replicationcontroller. See https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/api.md


Hi, I am interested to apply rabit to my project. Currently I use MPI Allreduce to handle everything, which is good. But I do want to add fault tolerance to the framework. A clear challenge I can see is the checkpoint method can only handle small local model which can be cheaply copied between different nodes in RAM.

However, it is not the case in my project, and not the case of many machine learning algorithms, either. (say in MCMC, one has to maintain a large set of latent variables or in ADMM, one may maintain a large set of dual coordinates). I am wondering if there exists an interface in rabit to warmly calculate those local models (possibly in multiple extra rounds in the recovered node) when recovered from a global model, rather than to restore them from previous save.

Thanks.
-Jianbo


@iSaulius
I found another issue with your website.
Daniel will kill us if I will continue to help you here.

Can you supply your private email, and I will continue to help you from there?


Why are you putting it in the bridge if it isn't even implementing a bridge? It's implementing a listener! Not a bridge!


Woah I really love the colorful one! It was suuuuper disorienting :D

On Thu, May 21, 2015 at 5:51 PM, botbotbotbot <notifications@github.com>
wrote:

> Haha, yeah, no clue, someone invited Allie and she forwarded it to me
>
> On Thursday, May 21, 2015, jxcache <notifications@github.com> wrote:
>
> > Hahaha I registered but this is suuuper random. It's an HR software
> > training session??!
> >
> > On Thu, May 21, 2015 at 2:40 PM, botbotbotbot <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> > > Wanna go to this thing when you're visiting? It might be boring, I
> don't
> > > know anything about it, but there's free drinks and food
> > > https://learn.zenefits.com/2015-roadshow-registration.html
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11313171
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11316698
> >
> > .
> >
>
>
> --
> latest Vidcode:
> *Vidcode in Intel
> <http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
> *Vidcode in Fast Company
> <
> http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls
> >*
>
>
> *Leandra Tejedor, *Designer and Front-end Developer
>
> *vidcode.io <http://www.vidcode.io/> / @vidcode
> <https://twitter.com/vidcode>* / *Insta*
> <https://instagram.com/vidcode>* /** FB
> <https://www.facebook.com/myvidcode>*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11316995>
> .
>


71693606-CompilerWarnings1 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1656 of 1656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


They are deprecated in favor of TLS_method(). I think those are OpenSSL 1.0.2 only? Maybe?... Will check...


Wat doet dit? Kun je dit misschien met jQuery oplossen? `$('#dropdown-title')....`


d.metadata.name - but it this function is actually not used right now. #8332 makes an update to fix the path for v1beta2, so there will be a merge conflict. I'll list it as a dependency.


Did you solve them both??

On Thursday, May 21, 2015, jxcache <notifications@github.com> wrote:

> Woah I really love the colorful one! It was suuuuper disorienting :D
>
> On Thu, May 21, 2015 at 5:51 PM, botbotbotbot <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > Haha, yeah, no clue, someone invited Allie and she forwarded it to me
> >
> > On Thursday, May 21, 2015, jxcache <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> > > Hahaha I registered but this is suuuper random. It's an HR software
> > > training session??!
> > >
> > > On Thu, May 21, 2015 at 2:40 PM, botbotbotbot <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>
> > > <javascript:_e(%7B%7D,'cvml','notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>');>>
> > > wrote:
> > >
> > > > Wanna go to this thing when you're visiting? It might be boring, I
> > don't
> > > > know anything about it, but there's free drinks and food
> > > > https://learn.zenefits.com/2015-roadshow-registration.html
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11313171
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11316698
> > >
> > > .
> > >
> >
> >
> > --
> > latest Vidcode:
> > *Vidcode in Intel
> > <http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/
> >*
> > *Vidcode in Fast Company
> > <
> >
> http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls
> > >*
> >
> >
> > *Leandra Tejedor, *Designer and Front-end Developer
> >
> > *vidcode.io <http://www.vidcode.io/> / @vidcode
> > <https://twitter.com/vidcode>* / *Insta*
> > <https://instagram.com/vidcode>* /** FB
> > <https://www.facebook.com/myvidcode>*
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11316995
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/1cc7c84e293152e3532992e98457a01cd4fff9da#commitcomment-11317106>
> .
>


Throwing the error on timeout was the one we absolutely cannot throw, because timeout errors happen alot over CAN. At least with my previous experience. Maybe RoboRIO CAN is different, but still.


I also want to review menu level steplocking as part of this. Especially as we'll be making cover and hotspot menus available to open source.


package Figuras;

/**
 *
 * @author Usuario
 */
public class Rectangulo {

    public  static double base;
    public static double altura;
    public static double area;
    public static double perimetro;

   //Mètodos get & set
    public double getBase() {
        return base;
    }
    public void setBase(double base) {
        this.base = base;
    }
    public double getAltura() {
        return altura;
    }
    public void setAltura(double altura) {
        this.altura = altura;
    }
    public double getArea() {
        return area;
    }
    public void setÁrea(double area) {
        this.area = area;
    }
    public double getPerimetro() {
        return perimetro;
    }
    public void setPerimetro(double perimetro) {
        this.perimetro = perimetro;
    }
    // CALCULO DE ÀREAS Y PERÌMETROS

    public static double CalcularPerimetro(){
     perimetro=base+base+altura+altura;
     return perimetro;
    }
    public static double CalcularArea() {
     area=(base*altura);
     return area;
    }

    public  static void Imprimir(){
    System.out.println("EL ÁREA ES: "+area+"\n EL PERÍMETRO ES: "+perimetro);
    }
}


I wonder what the wording here is like in other specs.


@Bertram25 In any case, if we refer to
https://github.com/OpenDungeons/OpenDungeons/blob/development/source/tests/CMakeLists.txt#L47

Tile includes the gamemap and the gamemap is not in the source list so I guess it works as said...


Ok. Thanks.


ObjexxFCL_ElimWarnings (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yes, no references to TLS_method() in our OpenSSL tree.

When implementing servers that speak multiple TLS versions you actually use SSLv23_server_method() today, and then restrict it further with flags (NO_SSL3, etc.).

So this should be fine.


Did you solve them both?


I'd either get rid of this block, or factor it so it can be used for the other ones (e.g. app-properties etc.).  The idea is that someone can use node-amqp-encoder to encode simple JSON with type annotations that allow it to create AMQP-specific types (e.g. AMQPMap, AMQPSymbol). Using the encoder allowed a higher level client (e.g. node-sbus) to encode something that can be decoded into appropriate types by multiple lower-level implementors (e.g. node-amqp10, node-qpid).


cc @byroot


:ship:


Is `localhost:4000` legit? I'd expect 3000


So 1.0 clients will ignore all parameters.

Let's say we introduce the `ext` and `supported-ext` parameters in 1.1. So 1.0 clients will continue to ignore those parameters. However, this text will change for 1.1 to say something like:

> Clients **MUST** ignore all parameters, except `ext` and `supported-ext`, for the `application/vnd.api+json` media type received in the `Content-Type` header of response documents.

In this way, 1.1 clients can recognize the new parameters and 1.0 clients will continue to ignore them.

Note that the server logic below is a little more complicated, since 1.0 servers actively need to defend against negotiating with 1.1 clients.


I just tried compiling my fix with haxe 3.1.3, turns out it won't compile (only 3.2.0 will).
If I copy a haxe 3.2.0 compiled run.n to my haxe 3.1.3 system, it still fixes the stacktraces (which is not a real surprise, since the neko vm is the same)


It think it would be nice doesn't remove copyrights (and respect someone else's work). Thank you.


Good point.  My team never got CAN working so I wasn't sure about that.  You can just remove the code instead of commenting it out.


This is just a local javascript variable name, so it can be anything. The api is using the updated lower case endpoint.


Hmm, I think you're right. :+1:


Bump major version


I think this rule is causing trouble in any system that has `box-sizing: border-box` (including Desmos). If you use that box-model, you get to remove those two lines (and don't need the padding, which is why I didn't see this when testing in Desmos).

Proposals:
(1) put all of mathquill on `box-sizing: border-box`, because it's totally sweet and way easier to reason about
(2) override any parents' rules about box-sizing in mathquill, by adding a rule like:

```
.mq-math-mode, .mq-editable-field {
  * {
   box-sizing: content-box;
  }
}
```


Please set up your editor to use spaces instead of tabs. We use 4 spaces as a normal indent.


The ratings will need to be updated when OGRS is adopted.


will fix.  thx


Thanks for the explanation.


# great code @carenvaleria


Can you use the Unicode escape sequence here is the string literal instead of the Unicode character?


Alright... I keep changing my code editor configs everytime someone in a project alerts me about that. That's one annoying problem to handle when we participate in a variety of projects simultaneously, each one with a different indentaion style :-)

I'll fix that. Thanks for alerting me about that.


NIT: You might just us a NoCase suffix here on the variable names, to align with the others.


CSS3 Box-sizing not supported by: IE (10,11)


I think it would be ideal to pull this out into it's own test case around embedded nulls and maybe augment it with cases like: "ab\0x00c" if we don't already have them?


you mean for tests.. dunno. some people utilize :4000 on test


Scalebar is looking awesome - is this due to some upstream changes in rstudio/leaflet? Keeping ahead of the game!


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-87.45%25-red.svg)


This is a good question! I think it is possible, what you can do is not save those local state, and only save global state, and in the code

```c++
int version = LoadCheckPoint(&global_model).

if (version != 0) {
     // some warmup on local stats
}
running the training.
```


You can leave an editor setup file in the root if that helps. We have already https://github.com/metapolator/metapolator/blob/master/.dir-locals.el for emacs. It would be nice if the filename starts with a dot.


Agreed - fantastic work @nikolai-b


My bad, misread


Feel free to send the UI for it before :]


So you actually have the `select` input written correctly here, with the full 50 states in `option` elements. Kudos!

The issue you probably ran into is that you left the text `input` with the name "your-state" in there. You should remove that `<input type="text">` and add the class, name, and ID attributes to the `select` element. I'm happy to talk more if it helps. That would be a very cool enhancement.


Right, they are just another method.


on the otherhand, the local checkpoint was only replicated over a ring over a few nodes. It is not globally replicated over all the machine's RAM.  So I think it is also fine to use local_model when needed


sorry your is ok .i had issues from other translations


possible de stocker la liste dans un fichier plutôt que directement dans le code ?


This should be...
```
TwoD(const std::vector<OneD<_T> >& v) :
    mCoef(v)
}}}
```


This should be a flag, not? Here 3 (0b11) means it is `HaveConfigDialog & PreferRightAlignment`

 So one only one bit should be 1 => this `SingleInstance` should be for example 4 (1 << 2 that means 0b100)


I love the images you chose for compatibility as well as the descriptors for the signs! :)


dummie lle :P


Since this is pretty simple, just change this to
```
return derivative()(x);
```


Since this is pretty simple, just change this to
```
return derivative().derivative()(x);
```

That is, do it in one line and you don't need any of the extra ()'s I don't think


updated, already generate a new key so this should be fine to export


This isn't a directory?


@madebymunsters This could be reduced to one line like - > `return number++;`


@madebymunsters can also be reduced to one line - > `return number1 + number2;`


@madebymunsters DWMR. Looks good. Press on.


Header_footer style change fixes overlap issue with long student dropdown menus and the homepage slideshow.


it's perfectly acceptable to access a properties underlying instance variable within an init method.  In fact its highly encouraged.  There are rare but possible problems if there is some KVO happening on the property your referencing you haven't returned an instance yet.

Outside the init method I would say almost always use self to access the property.


Really nice work - your code and the result looks great!


`updates_from_param_modifier` contains all updates related to a particular `param_modifier`. There is no way they would be overridden as the symbolic variables being updated are created locally in the `param_modifier`.


haha same question as earlier, yeah i would like @stevvooe and @dmcgowan to make sure this is fine. I replaced it with the `di.size = desc.Length` on https://github.com/tiborvass/docker/commit/03a139104e9a2750682f7c6a6384bcccd1a13395#diff-7cbe4277b9a08ded689d34eaae37655eR489


CSS3 Box-sizing not supported by: IE (10,11)


I think this may need fixing as well. Is it the `sconsUtils` back door to working out where "afwdata" is located?


Very clear, easily understandable logic here, which is awesome.


Yes you are right, l'll changed it. This is probably a relic of ADAGRAD.