I think if possible this should be solved by just putting the problem on
the user to use a command line that can be interpreted by whatever shell is
going to be invoked by default.
Dan

On Wed, May 20, 2015 at 5:53 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> @danpovey <https://github.com/danpovey>, @jtrmal
> <https://github.com/jtrmal>: The pipes in windows are normal pipes for
> all practical purposes. There is a _popen() in the CRT that tries to hide
> differences (and there is actually very little; the mechanism is called
> anonymous pipes, q. v.). The problem is not how pipes work from the C
> program standpoint, but rather with the command line that kaldi passes to
> popen(). The command line is interpreted by cmd.exe in Windows, and the
> syntax is generally incompatible with bash. Pipes like "gunzip -c | sox |"
> will work provided you have a port of lame and sox, which are widely
> available. Pipes like "cat * |" will work provided you have a port of
> cat(1), and this practically means you need cygwin, or, at the least, I am
> unaware of a standalone port of cat. Absent unix-style shell, who needs it.
> To sum up, I think the best a pproach with pipes would be as follows:
>
>    - Run the command line to popen() through bash if the calling process
>    is launched by [ba]sh;
>    - Do not use bash if launched as a native process, use the default cmd
>    - Be able to override both of the above default behaviors. A little
>    bit complex set of rules, but the user should think of the first two
>    options as "always using the default launching shell to interpret pipe
>    command lines." This looks like a reasonable default to me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-11294375>
> .
>


Thank you for pointing this out! I've removed the now-redundant code.


Right. Is WebStorm the same guys that make phpStorm and pyCharm?


Removed from the pom.xml (see line 90). See 9685e8c.


@danpovey: No, this would mean no egs/ scripts. These need bash even on Windows.


> .g. A CHANGELOG entry mentions: "Added improvements for commit calender". Then I found a bug in the commit calender (regression). A reference to the commit makes it a lot easier to find the commit then :)

The syncthing folks do that: https://github.com/syncthing/syncthing/releases/tag/v0.11.0
It is quite useful.


ah right.  good call


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (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)


I think I might delay and update tomorrow, I don't feel like breaking anything this late in the day.


@deizel Thanks for the correction. Not sure what happened here - the prior hash was correct at the time. :confused: Found another reference to it on https://hash-list.com.


@stopiccot Does this look ok to you? I thought it would help make the jenkins instance more discoverable.


probably a good idea

--
Tim Whidden | VP, Engineering
(e) tim@1stdibs.com
(p) 212.627.3927 (ext. 178)

On Wed, May 20, 2015 at 5:58 PM, peterborghard <notifications@github.com>
wrote:

> I think I might delay and update tomorrow, I don't feel like breaking
> anything this late in the day.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prestifidibs/sinopia/commit/2f684ca52a9945200239f5d0949ac76b90ade1d3#commitcomment-11294473>
> .
>


It's an option. I'm more of the opinion the FW should only act to higher levels (on your example: launch an exception on problems like writing permissions, services down, etc.). The responsibility on treating the exceptions depend on the app, not the FW.

In the case of SIFO, anyhow, I think there's an intermediate point: the existence of the sifo-common-app (at least actually) let you apply this desirable feature for any "default app" using the default error-catching controller. What do you think?


Old school 1st(I had a Ubuntu Mate 15.04 laying around already)
![screenshot](https://cloud.githubusercontent.com/assets/6667703/7737559/869a31aa-ff3b-11e4-8806-6f4a5c3c2b53.png)


I'll take care of it.


Well, it's not as pretty as travis, but it's easy to set up on one's own servers, and better at testing multiple platforms, which is important for C code.

Can you review https://github.com/rillian/ogg/commit/fbe55b0f60515be4dd9c90733b76d912a8a469c4 ?


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)


Hey man thanks a lot! First pull request on a repo I started :)


@seelabs Makes sense.


cmd


My position is that if people want to use Kaldi to train on Windows, they
should compile in cygwin.  I see no advantage in compiling using MSVC for
this purpose.  It just creates a ton of extra hassle.  You are the only
person who has wanted to do this, and as far as I can see it was as a kind
of personal challenge, rather than for a practical purpose.
Dan

On Wed, May 20, 2015 at 5: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>
> .
>


Thanks for looking over it, I'll work on those changes tomorrow morning and think you're right, I was using the console a little too much to check if the code was working.. so maybe some katas are a good idea.


Good fuckin job Freddy, Got the button working, next I'll start on getting the info panel/Flocks Starting at the Right Time/ Game Class W/ Ettlin and Finally, we'll try and get everything working.


I don't find it less readable. But there are a few reasons I'm using `styles` wherever possible.

* consistency
* performance gains from `styles` prop caching in the virtual DOM
* [react-style-webpack-plugin](https://github.com/js-next/react-style-webpack-plugin) uses the `styles` prop for css extraction

I'm still not sure whether or not react-styles will make it to geoshare, I still have to play with it a bit more.


What's the Sample at the top for?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: 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)


Oh! interesting. That makes sense since the .map() just runs the logic and loop itself.


Updates to meet SAGA 2.1.3's [changes](http://sourceforge.net/p/saga-gis/wiki/Compatibility%202.1.3/) handling of Boolean parameters.

Earlier versions of SAGA may still use the "omitting" style in command line calls.


.. but if the fix is easy I'm willing to to consider it.
You say you want it to "Run the command line to popen() through bash if the
calling process is launched by [ba]sh;".  How would this be accomplished?
Dan

On Wed, May 20, 2015 at 6:03 PM, Daniel Povey <dpovey@gmail.com> wrote:

> My position is that if people want to use Kaldi to train on Windows, they
> should compile in cygwin.  I see no advantage in compiling using MSVC for
> this purpose.  It just creates a ton of extra hassle.  You are the only
> person who has wanted to do this, and as far as I can see it was as a kind
> of personal challenge, rather than for a practical purpose.
> Dan
>
>
> On Wed, May 20, 2015 at 5: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>
>> .
>>
>
>


This is such a nice bit of cleanup even if it is mostly just renaming and moving a few things.  II particularly like getting all of these flags in one place and naming them the same.  I finally had a chance to go over this fairly carefully and it looks good to me.  I'll be working my way through the rest of the patch stack.


It will fire the doneTyping callback after the user has stopped typing for a a partial second so I'm not providing the results constantly. Or at least that was it's intention.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: 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)


makes sense.


appends the previous commit:

Changes made to the mtasc compilation commands.


`doneTyping()` is called every keypress on line 133, so it appears that the timeout is redundant?!


Yeah I was kinda :confused: with the namespace, but didn't want to put it root. Also, thought about `Profiler`


This is a copy of my portfolio prior to adding the unfinished blog page.


Shauren, could you check Ice Tomb AoE Damage https://github.com/TrinityCore/TrinityCore/issues/14304

Thanks for the fix btw.


Everything except "emptyIterator" is unrelated to update to Java 7 and done by 6149348dddaa5493f10ae0b3e8f7598ec4c0d290


Awesome.  I will take a look at this in a few.


develop (lefticus) - i386-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)


Whehey... nice! Really, really nice! :+1: If this commit actually solves the ice tomb issues; /salute /bow /cheer!


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1649 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-99.94%25-yellow.svg)


yup you're right.


This is a very unfortunate decision to revert synaptics. It is practically impossible to type without this update. The battery drain is insignificant.


- Se ha añadido un paquete para facilitar el scroll infinito, dado que el scroll hecho anteriormente no era reactivo y habría sido laborioso reordenar o filtrar la lista. Se han quitado el sistema de scroll infinito creado en el commit anterior.
- Ahora se pueden ver los detalles de una publicación y sus opciones disponibles. Para ello se ha creado una nueva template.
- Se ha añadido la opción de editar una publicación. Para ello se ha creado una nueva template.
- Se ha añadido la opción de borrar una publicación.
- Todos los métodos se han pasado al servidor.
- Se han cambiado las rutas y el controlador de publicaciones para incluir las rutas de ver los detalles de una publicación o para acceder a su formulario de edición.
- Se ha cambiado el nombre de la template de creación de publicación.
- Se ha cambiado el tema de Bootstrap.


A test that fails blocks forward progress.  Someone has to look at it.  Merges won't occur.  etc.  If they're considered 'ok' to fail and also 'ok' to pass, then what value has been provided by its existence?  A test failure by definition is "something went wrong, figure it out and make it not fail"


@glevand sorry I don't mean to imply we used `uname -m` for all of these, just for the ARM architectures specifically after chatting with ARM (the company).  amd64 vs x86_64 was basically a coin flip at the time since there are reasonable arguments both ways (`uname -m` is arguably too specific to rely on generally)

https://github.com/appc/spec/pull/33#issuecomment-67081990


@teschnei  I have check out as is, commit as is, to preserve darkstars pain in the rear **mixed line ends**. Not only have you guys got diff files using diff types, some files even multiple line end types in the same file from editors who are just clueless about line ends (or don't care).


Re-numbering these in a public header is asking for trouble.  It's a good thing there aren't any out of tree consumers for  these interfaces I'm aware of.  We can get away with this at the moment and we should so we can stay in sync.


lgtm, although I think I did add the sort mostly because it appeared that the order was completely 'random' in our cases... it ended up changing from run to run, where I'd really want premake to be deterministic. maybe I need to look further up for the problem in this though.


Thanks! See you tonight.


Too hard to police everything, do what we can


Inside the ring, the one that drops onto the center part.

On Sat, May 16, 2015 at 5:14 PM, Iceflower <notifications@github.com> wrote:

> Which drop shadow?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IceflowRE/data/commit/4cf87895e0c602319bc84ff0f88b2df4639d4401#commitcomment-11227623>
> .
>


Did some work on a custom admin base with a colleague and remembered this situation.  I think there is a better solution that doing a type check and assertion error.

We should just force cast it into a list.  The actual django ModelAdmin does this in tons of places already like `get_list_display` or `get_fields`.

```
fields = list(line.get('fields'))
```


I think when you changed this, you broke `unistd::pivot_root()`. `unistd::pivot_root()` just takes the arguments of the input closures to `with_nix_path()` and passes them right to `unistd::pivot_root`. These arguments used to be `*libc::c_char`s, but now it's passing in `&OsStr`s. And there don't appear to be any tests which cover `unistd::pivot_root()`.


You don't actually use the `do?` binding here. Might as well remove it, unless you think it's useful for pseudo-documentation?


Currently working my way through the code to get apollo_parse to work


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)


Пока просьба не принимать ветку. Тесты реализованы, но не удаётся ими добавить новую запись.


kek


If you submit a pull request for this change, I will accept it.


@wKovacs64 The package might’ve been updated in place. Happens frequently with a few apps.


@cvium This broke tests? http://ci.flexget.com/job/Test-Develop/386/


Guava's Preconditions.checkNotNull() is different from Objects.requireNotNull () - see https://code.google.com/p/guava-libraries/wiki/PreconditionsExplained But even if we don't utilize this fact would be better to stay with usage of Preconditions for consistency, unless invocations of others methods also replaced or removed.


I just used integers for when values might be assigned to them later in the code. I thought we would most likely use integers rather than decimal numbers with float. C and D are small functions apart of the main phi function. I mostly just put this on as reference for when we input the transfer functions with all of the variables.


Gemfile.lock не редактировал. Возможно, он изменился при установке "selenium-webdriver".


Or not reference but notes on a computer instead of staying on paper.


The removal of the last " on line 40 seems to be a typo...

pcbsd-build script bombed on the buildworld at this file until I added it back.


With an option like "ls -l"?
Weight global + min/max could be great.


added, thanks!


What is the issue that needs to be fixed?


Shoving the app menu in the window seems fine.. but that is an odd choice for icon... Otherwise looks fine. I believe we can force the menu elsewhere if we want but im not sure what would be ideal.


Not directly related to this patch.  But this got me wondering about @DeHackEd's changes in #1971, perhaps instead of hashing with a crc64 we could change `b_freeze_lock` and `b_cv` to indexes (or pointers) in to an fixed array.  That would give us a constant tiny/free lookup cost and a `uint32_t` index packed next to `b_datacnt` would cost us zero space on a 64-bit system.


I was receiving [this error](https://github.com/atom/atom/issues/6870) when trying to convert my markdown to a pdf. Removing the parentheses in line 50 of lib/markdown-pdf.js fixed it. The package is working fine for me now.


Should this be reversed logic? if composite_protocols, then don't split the string.


s/toble/table


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (633 of 633 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.3%25-green.svg)


This doesn't modify equal. equal will always be true


Done with that? Fine :)


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/c32af523c39c62b74300f0a10335cbcedc64d08f/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/c32af523c39c62b74300f0a10335cbcedc64d08f/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/c32af523c39c62b74300f0a10335cbcedc64d08f/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/c32af523c39c62b74300f0a10335cbcedc64d08f/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/c32af523c39c62b74300f0a10335cbcedc64d08f/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Yeah it's quite nice. Maybe if you add OS X and Windows builders to your Jenkins setup we can make nice build status table like https://github.com/dotnet/coreclr has


What's this for?


since this is a list (with in general one or more elements), maybe a bit better name is its plural, so instead of 'proto' use 'protocols' for a bit better semantics?


I know. I'm not really sure why I thought this was a good idea.


> finalColor = material.color * material.map * material.lightMap;

@mrdoob That is not how ligntmaps work. Lightmaps are now additive -- as a result of a change implemented by @bhouston several months ago.

I am willing to work with you, but we have to do something that makes sense.

This is the pseudo-code for how `MeshPhongMaterial` computes outgoing light due to incoming ambient light:

	vec3 totalAmbientLight = vec3( 0.0 );

	totalAmbientLight += ambientLight.color; // from AmbientLight source

	totalAmbientLight += material.lightMap * material.lightMapIntensity; // lightMap is additive to ambient light

	totalAmbientLight *= ( material.aoMap - 1.0 ) * material.aoMapIntensity + 1.0; // aoMap modulates total ambient light

	outgoingLight += material.color * material.map * totalAmbientLight;

Now, for `MeshBasicMaterial`, we do essentially this, instead:

	vec3 totalAmbientLight = vec3( 1.0 ); // hardwired

	outgoingLight += material.color * material.map * totalAmbientLight;

The only thing that makes any sense at all is to add support for ambient occlusion like so:

	vec3 totalAmbientLight = vec3( 1.0 ); // hardwired

	totalAmbientLight *= ( material.aoMap - 1.0 ) * material.aoMapIntensity + 1.0; // aoMap modulates total ambient light

	outgoingLight += material.color * material.map * totalAmbientLight;

Lightmaps are another source of ambient light, and are not appropriate for `MeshBasicMaterial`.

Quite frankly, I am ambivalent regarding adding AO support to `Basic`, but will implement it if you want. This will leave `Lambert` with out it, however.


It should continue, but at the end it should return an error.
An example, with standard unix commands:
```
$ touch fileA
$ rm fileB fileA
rm: cannot remove 'fileB': No such file or directory
$ echo $?
1
$ ll fileA
ls: cannot access fileA: No such file or directory
```
fileA is removed, but the return code of rm is 1 because it encounters an error.

```
$ rm fileB fileA
rm: cannot remove 'fileB': No such file or directory
rm: cannot remove 'fileA': No such file or directory
$ echo $?
1
```
The return code is still 1 if it envounters multiple errors.


yeah, it is reversed.


Internally, I don't think we have the "OSEnvironment" property defined.  You may need to define it in open.targets for any of our referenced relative projects to use.


> Shoving the app menu in the window seems fine..

I don't really have a problem with it. But who knows how other DE's will handle it?

> but that is an odd choice for icon...

I know right. I thought it was another close button at 1st,lol!!!

> I believe we can force the menu elsewhere if we want but im not sure what would be ideal.

I don't know, maybe see how other DE's handle it?


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)


After giving this post a read, it looks sane to me, but I'd consider breaking it into separate image building and image using parts.


```
parser.add_argument('inputs', metavar='INPUT', nargs='+')  # one or more inputs
parser.add_argument('output', metavar='OUTPUT', nargs='1')  # one output
```


Right now I still like the current UI better. It looks pretty much the same on all DE's.


I'd use `computeIfAbsent`


I wanted to propose [appveyor](http://www.appveyor.com/) for windows CI. But now I'm sure you need extend your jenkins setup


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)


@vitorgalvao Yeah, I figured that was what happened after verifying against the provided GPG signature.

```console
$ wget https://download.electrum.org/electrum-2.2.dmg
$ wget https://download.electrum.org/electrum-2.2.dmg.asc
$ gpg --verify electrum-2.2.dmg.asc
gpg: assuming signed data in `electrum-2.2.dmg'
gpg: Signature made Wed May 20 19:40:39 2015 BST using RSA key ID 695506FD
gpg: Good signature from "Animazing <animazing@gmail.com>"
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 9914 864D FC33 499C 6CA2  BEEA 2245 3004 6955 06FD
```

@wKovacs64 Note the prior hash has a date of May 1, but the GPG signature is dated May 20... :sweat_smile:


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)


@brainydexter I am facing 2 issues -

1. what to return from the function ? as I am already returning true and false from the for loop inside.
2. where and how to print the value of j. since scope of j is limited only to  for loop.


Scroll isn't a valid property of the position property. What were you trying to accomplish with it?


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)


This should be readErr, not openErr. What I usually do is use a single "err" variable that I re-use each time.


Alright fixed it.


Thanks. Push to master.


There are no units with font-weight, so you don't need the px.


Why it not inherited from `Volume` ?


Great job!


Ahh, I bet it changed headers which were sent which broke the vcr recording of the session. Deleting and re-recording those tests' cassettes should do the trick I think...


The ordering specified in the script should be maintained by Premake. If that doesn't happen, it's a bug, but I haven't seen any randomness here.


SF


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)


What sense to allow setting mode in RW object (that was designed only for RW)?


@saimonsharif i entered "fixed" at first, but that would make the button move along with the page. To fix that, I changed to "scroll." Is there another way to avoid that from happening?


@saimonsharif Cool, thanks


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)


On an unrelated note. What Icon are you using for the new station pop-out sort button? The default Ubuntu theme doesn't seem to have that icon.
![screenshot from 2015-05-20 17 39 48](https://cloud.githubusercontent.com/assets/6667703/7738193/76e246ae-ff17-11e4-8622-ae97ee07c789.png)


I saw some failures that could be explained from that. But ok for now as I didn't recorded it. At worst, it is compilation crash.


that is the way to go I think.


how about calling this package "multicall" instead of "procflavor"?


exePath ?  path is pretty ambiguous.


Really, @zebron these are probably meant to be blocks of content on the timeline, I think Kandace just put the RTF files in hear real quick as placeholders. FWIW.


view-sort-ascending-symbolic, pretty sure the gnome icon theme added in 3.16.


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)


Why did this need to be reverted?


Oops, thanks :)


That would explain things. Ubuntu 15.04 is bastardized GNOME 3.14.


Thanks for the quick fix!


Ik wil die alert er nog uit hebben alleen zonder doet hij het (nog) niet.


@danpovey: using [this 12-line](https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#diff-35d4c23a4288ae52e707e78f09b77229R148) `popen()` implemenation. As for the detection, bash sets a few distinctive environment variables (e. g. SHLVL)


Funny the icon showed up jut fine in Ubuntu Mate 15.04.


If by next release we don't require 3.16 we could easily just ship our own copy of that icon, sort of silly to depend on newer for an icon.


The story that calls this snippet needs the name and type columns. We need to add "Type" to the select method.


반블럭체크가 이 버전에선 안됩니다


Here, I must say there is something I don't get.
Since you're asking the gamemap as a constructor parameter, can't you use it in the constructor to setup the `mIsOnServerGameMap` member and thus get rid of the boolean in the constructors?

Maybe I missed something though.


Yep... ugh. The original signature was April 24 and your update was May 13, so the new binary must've been uploaded sometime between those two dates (May 20 is today, your output is from the new 2.2 release).

Sorta hate the idea of in-place updates/replacements as it tramples on hash verification, but... c'est la vie.


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: 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)


how about returning the added entrypoint, then making CmdForFlavor a method on the entrypoint, so the args list just includes the args, not the entrypoint/flavor/whatever for args[0].


Ok, now i recall the reason for that. `nullptr` is a C++11 standard and thus requires enabling it in your compiler, which is what this line for.


Looks great, thanks for the help!


This can be reverted because the upstream illumos fix was applied as part of #5497.  Right?


errors?


Ah I see, nice catch.


After this change i see no suitable classes for storing user entered volumes string.
Before change `Volume` has no right parser, after has no mode at all. `VolumesRW` can be used for storing, but saving ro values in RW class sounds weird.


How did you noticed that one in game?


cleanup


capitalized error message


88737258-EPW-Ruby-Access (jasondegraw) - 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)


I think this is best solved by adding a default value for the property so that we do not sprinkle ```|| {}``` all over the code base.


silly mistake; fixed. thanks!


Does this account for the fact that we'll be adding whitespace in between each token?


Not sure if i understood you correctly, but will try to implement it and we'll see if i got you right.


just casting, not LOS issue


Ah nvm, question was answered by reading the tests :]


The equivalent of https://github.com/illumos/illumos-gate/commit/78d7e96 which is probably worth mentioning in the commit.


Looks like good streamlining in any case.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: 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)


Is locale a possible concern here? https://en.wikipedia.org/wiki/Dotted_and_dotless_I


Well, we need to make sure not to *start* a path string with multiple separators (think UNCs).

I suggest a utility function or macro for detecting when we've walked up to the root.  Also, we should stop at $HOME, if we know it.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: 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 understand the fix but I find it weird that we cannot let the creatures go next to the door and attack it with the current code. ? Well, let's try it and we'll see whether this induce other problems...


zamiast "Sommer_Wasniowski_Rudas" powinno być "Sommer_Rdzanowski_Pytlak"


Can you make all the event constructor functions public?


I opted to only make this constructor public; do you think we should do the same for the other event types?


Does this work?  It looks like the img variable is scoped inside the if statements, it wouldn't be defined in line 144


@brigleb Yeah. I threw them in there as placeholders like you said. Originally I entered the content from Trello to TextEdit but it had a bunch of formatting and it got all weird when you were looking at it on Github. So, I changed the files into plain text so whoever was working on these assets could read it.


<grammar nazi>
Maybe have Assertion() message say "beam_info_index (%d) invalid (either <0, >= %d, or not actually a beam)!\n"
</grammar nazi>


Ok I can update when I get to a pc, or someone can throw up a pull request.


[![Ehrenpforte (The Triumphal Arch of Maximilian I)](http://api.artsmia.org/images/116298/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116298)


Very nice!


I would rather name it `addRandomOffsetToPosition()` IMHO. This is not even forcefully linked directly to an entity.


Is there some reason this owns a TabContentView and isn't a TabContentView?


Ah, thanks, I was trying to code the fix, but I still can't code in go :)


develop (lefticus) - i386-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)


The indentation is wrong here.


@TheTincho Me neither - it doesn't stop me though :P


Hum, maybe sometime we will not have the UnitCreated event.
That could happen if UnitEnteringCombat is seen before.

In this case, an 'else' should be preferred in UnitStateChanged ?

```lua
function mod:OnUnitStateChanged(unit, bInCombat, sName)
  ...
  if sName == self.L["Avatus"]  then
    if not encounter_started then
       .... ( Init Code )
    else
       .... ( New Code here )
    end
  elseif sName == self.L["Infinite Logic Loop"] then
```


I don't much like relying on `System.gc` to actually do anything, it is specced as a hint only.


I like these :) I doubt the laziness gains us any speed, but its nice to have it all in one place.


I'd say:

        a*n* object current walk queue with the given path.


hello


Wrong indent here too.


could avoid packaging big.min.js since that isn't needed on the node side.


This commit is related to manifoldjs/ManifoldJS#60


Yeah and because the completion analyzer in elasticsearch ignores whitespace it's all good


The overall idea is clever. :)


var_log_t is declared in system/logging.te.  Need to create an interface there.


cool


This commit is related o manifoldjs/ManifoldJS#60


Oops?


0.89100 - Model = GradientBoostingClassifier(n_estimators=200, learning_rate=0.0995, min_samples_split=3, max_depth=9, verbose=1) & st.loc[idx, 'prediction'] = 0.0.0355 # pure hack

0.88715 -Model = GradientBoostingClassifier(n_estimators=203, learning_rate=0.0982, min_samples_split=3, max_depth=9, verbose=1) &  ytest.loc[idx, 'prediction'] = 0.03515 # pure hack

0.89233 Model = GradientBoostingClassifier(n_estimators=200, learning_rate=0.0962, min_samples_split=3, max_depth=9, verbose=1) & ytest.loc[idx, 'prediction'] = 0.03567 # pure hack


Okay, changes in https://github.com/launchdarkly/go-client/pull/13


Yenda, what do you think about this?
Dan

On Wed, May 20, 2015 at 6:46 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> @danpovey <https://github.com/danpovey>: using this 12-line
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#diff-35d4c23a4288ae52e707e78f09b77229R148>
> popen() implemenation. As for the detection, bash sets a few distinctive
> environment variables (e. g. SHLVL)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-11295133>
> .
>


Ubuntu is really the only thing holding us back. I wouldn't worry about it. Just don't package a version that's broken for Ubuntu in the PPA. Can't let old releases hold us back. If someone decides to run from source then it's really up to them to update to the needed GTK version. Other than that they can run off the PPA.


Can Add be more descriptive?  Add what?


Del what?


Oops, I was pushing to a different branch. I've just updated the PR. It now has additional tests, and the correct signature.


@DamianEdwards can code style changed accepted in this #4 PR


@rassilon I've added this for consistency. Any ideas if we actually need to mark virtual overrides as dll exported?


These aren't needed


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)


88737258-EPW-Ruby-Access (jasondegraw) - 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)


may be slightly cleaner to do:
```
if l := d.getLease(args.ContainerID, conf.Name); l != nil {
        l.Stop()
        return nil
}
return fmt.Errorf("lease not found: %v/%v", args.ContainerID, conf.Name)
```


nice work!


There's always just using "view-sort-ascending" it's in the freedesktop.org icon naming spec.


Haven't yet looked properly through your nice addition, but I confirm it seems to work. Good job!

I don't have this `Iterator0_1` and it doesn't seem to be used anywhere (leftover from you project I guess?!)... ;)


The test method needs to be overridden in all of the connector tests (including Raptor in this commit). See https://github.com/electrum/presto/commit/956911c20f83ab37dd6d29ff17d3aa0ca9a79aae for an example


0.12081 when used trend ^ 1.95 so far best model


Just a comment


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)


Name this `test_rename_column` to not conflict with the table rename test (they run in parallel)


Verify that the column is renamed:
```java
computeActual("SELECT y FROM test_rename_column");


@mafintosh I had to do this to get the cors headers to apply.


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)


Remove the Raptor test override in this commit


Maybe "Fixed beam fired without a valid ship to fire from!".


Whats the advantage of fading in the snapshot vs the real view?


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)


@whothat  So idk how to, if possible, to have two application.html.erb with a different yield environment. like the admin environment isn't going to look like the user's environment when browsing the site. i just don't know what to look up where to find how to do this


Heh. I originally wanted to make this a Dictionary or views [String: UIView] to keep it kinda general (i.e. we could automatically transition views with the same name to match, or at least do special things if we knew the name of a view we cared about), but I'm not going to hold this up trying to keep Transitionable really general.


Does it cast now?


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)


Is this used?


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)


Now comes the part where we sit back and wait....................................


I'm busy with vim,git and a little Ubuntu but once I'm done with that I'll try to merge my initial_site_app


How about
```java
assertEquals(metadata.getColumnHandles(tableHandle).keySet(), ImmutableSet.of("orderkey_renamed"));


I did not find a good value :(    The lower I went, the better the result.  It is a factor in losses, but it probably only applies to rooks and occasionally queens.  Minor pieces can usually find another to trade with.  Cut off rooks have more difficulty.  Queens just find ways to escape easier.


Need to add icons to edit texts still, as well


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)


yep, test forgotten detected ^^


Can you just set titleContainer.alpha to zero here?


Argh, `npm version` decided 2-space was better than 4-space indentation...


Plain old "view-sort-ascending" is U-G-L-Y but "go-down-symbolic" looks nice.
![screenshot from 2015-05-20 18 17 20](https://cloud.githubusercontent.com/assets/6667703/7738664/98f114a0-ff1c-11e4-82e8-0cc4cdbe9c09.png)


ok


Missing `COLUMN` :)


0.12096 when used trend ^ 2.13


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)


Thanks :). I just realised this includes Chinatown et al so I might just run the script you posted in the GG thread. But we'll add credit to you!


Add "in the ``users`` table"


Yes, it works on phase 3.

The other issues related to Ice Tomb are not fixed yet.


Yeah, I've attributed it to our update to lua 5.2.3, which appears to do some 'odd' things with tables sometimes... I've not really looked into it much further though.


It was only a convenience to help the initial merge of the main patch go better.  My intent was to reinstate it after I had things working.  That was the major underlying theme of my recent comment about being "too aggressive" with reverting things.  I'm going through the patch stack again and am un-reverting this as well as some of the other stuff for which I was too heavy-handed.


Its interesting to duplicate these in here. I wonder if we can actually subclass this for them (to avoid creating them for every TabContentView. I worry they won't be in the right state either. We could make them generic views and snapshot the "real" views?


Squash this into the first commit


Yep, that was a mistake on my part. Thanks for the heads up its fixed now.

--
Kris Moore
PC-BSD Software / iXsystems
Enterprise Storage & Servers Driven By Open Source

On Wed, 20 May 2015, cthunter01 wrote:

>
> The removal of the last " on line 40 seems to be a typo...
>
> pcbsd-build script bombed on the buildworld at this file until I added it back.
>
> —
> Reply to this email directly or view it on GitHub.[AB1_IJ0ZqX3yrXSJiFHfa87DQg8k21Psks5oLQEUgaJpZM4Eiglh.gif]
>
>
>


@bnoordhuis @misterdjules  Line 493 (as of commit c601adc5cd37797e7dade92cbd7e79507867d051), specifically, shows an if/else on the right-side of the assignment, probably meant to be on the next line and indented for readability so people aren't overlooking it in the future.

this is what i was referring to in my comment some time back when noting "there appears to be some extra code at the end of line 477" (which now line 493 after patches/merges since.) sorry for the late response, i'm pulled so many directions every day.


Agreed.


I would want this to be more explicit. For example:

This method assumes that a Limiter is always desired, and that the correct configurations for creating the desired Limiter are available in the input State. If this is not the case, the method will throw an exception. If a Limiter is optional in the application, the calling code should first ensure that the Limiter should in fact be created, or handle the exception appropriately.


How about adding the ldflags to the build?

                   @cd ${WRKDIR}/src/github.com/influxdb/ &&  ${SETENV} ${GO_ENV} go build -ldflags "-X main.version ${PORTVERSION} -X main.commit ${GH_COMMIT}" ./...


Cache lines aren't 64-bytes on all architectures.  A better, but gcc specific, way to do this would be to add the `____cacheline_aligned` keyword at the end of the structure.

```
struct multilist_sublist {
        ...
} ____cacheline_aligned;
```


The problem was that if you open your dungeon and add a door on the only entrance, enemy floodfill will grant them access to your dungeon (since the door will not be unpassable for them) but during path computation, the door tile was not considered passable and no way was found. That's why this fix is needed.
The potential problem would be that enemy creatures could go through your door while fighting (ranged creatures looking for a spot). That's why I test for fighting in the creature action list


Well I'm not sure what a "pure js" project is though. As I noted above the two random JS projects I'm working on now are either node only or browser only...


Ah ok, now it's clearer. Thanks for the explanation. :)


Its called partials
On May 20, 2015 4:14 PM, "LinnierGames" <notifications@github.com> wrote:

> @whothat <https://github.com/whothat> So idk how to, if possible, to have
> two application.html.erb with a different yield environment. like the admin
> environment isn't going to look like the user's environment when browsing
> the site. i just don't know what to look up where to find how to do this
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/whothat/Ben_Hough/commit/5a235cbeffbca748bd4135e347c81ba5776a99fe#commitcomment-11295453>
> .
>


But apart from that, yeah. I think @timbotnik moved us in this direction already.


@rift137 We are getting an error here https://sentry.biola.edu/biola-university/profile-publisher/group/86/. I think maybe `.empty?` should be `.blank?`


Any reason to have a second function for this?


neat! so this way i can have a yield environment for admin then for everyone else?


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 13](https://teamcity.labkey.org/viewLog.html?buildId=251101&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **SUCCESS**
Summary: Tests passed: 26 Build time: 00:44:14


maybe `s/\ncommit .*//s`


Oh, I see, just keeping it in one place. Is updateViewConstraints too much?


71693606-CompilerWarnings1 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1655 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-99.94%25-yellow.svg)


Yes.  Good idea.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2116 of 2212 tests passed)

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


medic huron nicht im shop


Excellent choice.  Much better than passing a string to an empty `STAmount`, like the previous interface.  I also like giving the function a name, rather than just having a 2-argument constructor.  This really documents what's happening.


With the crash that happened when unpopping twice. I've seen that the first step was to pop the current action, then we tried something. But if this something failed (because no path or whatever), we ended up popping twice. That's why it seemed straightforward to only pop the current action if the new action was pushed.


Fix_sqlite_queries (macumber) - 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)


Note this will never select neither the tank nor the 2nd in aggro list, due to starting out from position 1, and the filter already checks for victim.


@wjwwood I could not find any reference of `deprecated-register` for gcc and during compilation it outputs:

    warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]

Could this be clang specific? If yes, should this be updated to:

    #pragma clang diagnostic push
    #pragma clang diagnostic ignored "-Wdeprecated-register"
    #pragma GCC diagnostic push
    #pragma GCC diagnostic ignored "-Wall"
    #include <ndds/ndds_cpp.h>
    #include <ndds/ndds_requestreply_cpp.h>
    #pragma GCC diagnostic pop
    #pragma clang diagnostic pop


Updated with this commit, still needs to be tested.
fabricator77/SpaceEngineers@cf8aec048b861512bd0f2aee34f039b0a4d4b824


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1146 of 1146 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-69.8%25-green.svg)


This is the patch I was working on when I called it quits for the day.  Unfortunately, I did the original port so long ago that I forgot and was apparently too lazy to create a better comment at the time :|


Thanks! BitHub has sent payment of  $8.91USD for this commit.


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)


no pb ^^


azrafe7
Opps sorry I need to remove that, will look tomorrow. I considered abstract Iterator but in the end I knew that dirty while loops are currently fastest.
Iterator0_1 is my prefered implementation but it was too slow.
https://github.com/Justinfront/Letters/blob/master/src/letters/pathway/Iterator0_1.hx
```haxe
var iter01 = new Iterator0_1( step );
for( t in iter01 ) {
  v = MathPoints.quadraticBezier( t, arr );
  lineTo( v.x, v.y );
}
```
if you look at divtastic3 you will see the beizer example
http://divtastic.googlecode.com/hg/samples/deploy/bezier_chain/BezierChain.html
it basically is an equation of the curve and it's like a tween but I am instead just sampling points along it and drawing a lines between them.


thats how it is supposed to work, isnt it?


I was thinking of trying to shim the illumos commit into the stack for better documentation.


After pulling and solving the conflicts I committed and pushed.
The result is that along with the conflict fix in `marker.js` I pushed all the changes I got from the pull.
Should I reset to the previous commit, them only commit the conflict change for `marker.js` and amend it?
Or is this acceptable?


well, @oyvindln request was to avoid using the gamemap to not depend too much on it for his tests (allowing to not compile the gamemap). Using it in the constructor would kill all the interest.


I think the LockOSThread() needs to happen in WithNetNSPath() before entering the namespace, and we need to guarantee that we restore the namespace before WithNetNSPath returns as well, because that OS thread will become available for scheduling other goroutines once the locked goroutine returns.

I'd add a bool to WithNetNSPath() to toggle the pre-nsenter LockOSThread(), and in this call site it would be true.  I guess then you'd have WithNETNSPath() UnlockOSThread() when that bool is true as well, before it returns but after restoring the namespace.


This was eating all our exceptions. And the $self->logError() basically does nothing, we'll have to refactor that some other time... for now we should just propagate the exception since this is an import stopping error imo.


These two files got left out because of my malformed .gitignore, so they needed to be added back in.


Oh wait, the exit-tab-tray animation will be  a bit tough then eh?...


I think the name and description make it clear that whoever uses this method desires a *new* `Limiter`, not an optional one, not `null`. But I agree we don need to make it clear that an `IllegalArgumentException` will be thrown if the configuration is incorrect.


I'd leave this as `MAX(boot_ncpus, 1)` and then define `boot_ncpus` to `num_online_cpus()`.


Just propagate the exception. This is an import stopping error, imo, not just a logging one.


IMHO, it doesn't correct the bug you have seen. But it is cleaner ^^


Boy, is this ever easier to read.


We should make this a module option and document it in `zfs-module-parameters.5`.


We should make this a module option and document it in `zfs-module-parameters.5`


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1145 of 1145 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)


my standard `MAVEN_OPTS` is `-Xmx1024m -XX:MaxPermSize=512m -Djava.awt.headless=true` so I hadn't noticed that. The CCP build uses plugin that uses groovy, which I think chews up some perm gen space.


Well it is a MovableEntity function... And the idea is to allow each entity to allow to change that (or not) on client side. Most of them will not do anything (only creatures ATM).
But I was also thinking that we might use the same for dropping creatures/treasuries/... But I need to have a closer look, I'm not sure yet it is feasable.


A likely issue is that you do not have the file extensions displayed in whatever file browser you're using. This makes it so the file is actually named "mcmod.info.txt" to the system, so it does not read it properly.

You could try using Google to figure out how to enable the file extensions in your OS and then make sure it is in fact names "mcmod.info"


Left 2 notes on this commit, everything else looks good.


88737258-EPW-Ruby-Access (jasondegraw) - 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)


We should make this a module option and document it in zfs-module-parameters.5


I think that's mainly why I did it, yes, since these are provided in the JRE.


If you want cubic curves the MathsPoints has the function required.  I just have not added the wrapper code but should be very easy if we feel it's a good decision, I am in two minds motifs does not have it.
http://divtastic.googlecode.com/hg/samples/deploy/cubic_bezier/CubicBezier.html


develop (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)


I really wouldn't mind if "go-down-symbolic" was used as the button for selecting the station in the headerbar ui. Skip the station combo and tuck it up close, make it the same height as the station title and make it on the same level as the same level as the station title.

Something like this.
![mockup](https://cloud.githubusercontent.com/assets/6667703/7738937/da232352-ff1f-11e4-9ac2-c87ae7eb1f86.png)


It could be, but I don't think it necessarily "should" be. Can you think of a good reason to call `setUserError(false)`?


Watch your indentation here. It looks like you've gone 2 spaces too deep.


Check the indentation of the lines below this. Any code that runs within the scope of this `if` statement should be indented 2 spaces, the `else` keyword should line up with this `if`, as should the `end` keyword.


Looks like a rogue single space is throwing off the indentation here.


I put it in intentionally. I wanted to record the jax-ws method that is servicing this request, and this seemed the only reasonable way to do it. (And it can be used, hypothetically, by SSW and DSS, or a jsf app like eTimesheet).


`@question.resolved` returns a true/false value, so perhaps you could consider using that instead of the `==` operator.

eg.

```ruby
if !@question.resolved
  ...
```

The `!` signifies `not` here.


71693606-CompilerWarnings1 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1655 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-99.94%25-yellow.svg)


Commit message should be "Cleaned up __html__() method"


Oops, meant to commit to master.


Since arc_evict_ghost is being removed you could just squash the previous revert in with this commit.


Maybe use a nested switch here for cleaner looking code.

Example: see line sexp.cpp:26948 [master]


Could you suggest how to handle the ones which prefer to use on a modified rom f2fs? Thanks


We should flag an error to the user ideally I think. Otherwise people will log successful tests because it hasn't patched the binary file


@wilson0x4d I'm not familiar with Python, but I think that this `if` expression evaluates to `'6'` or `'default` depending on the return value of `is_arch_armv6()`, and thus is intended to be written that way. I haven't looked at the commits history though.


I know this is just test code.  But I get the sense that you understand a bunch of the magic numbers and conditions in here.  It would be great if you could sprinkle in a few comments.  That would give me a running start if I have to maintain or add to the test.


Left 1 comment on this commit, everything else looks good.


I'll do that, since it seem's we're already committed to gcc-isms (statement expressions, etc.).


Most of the above can be forward declared.


Will do.


indentation


what a messy code!


Ditto.


HOW_TO_RUN_SNAPCHAT

Instructions for starting servers.

1. go to .../275_class_presentation/runtime/mesh>
2. run ./startServer.sh raftserver0.conf
3. run ./startServer.sh raftserver1.conf
4. run ./startServer.sh raftserver2.conf
5. run ./startServer.sh raftserver3.conf
6. run ./startServer.sh raftserver4.conf
7. The leader would be estbalished through election.
8. If you need to adjust random timing for testing, heartbeat can be changed through changing variable sHeartRate from HeartbeatManager.java (it is set as 2000 msec ). Random time can be controlled from Server.java through the method getRaftTimerRandomElectionTimeout()(currently its 5-8 seconds)

Instructions for running clients.

1. You can find five new clients in (client.java, client1.java, ClientTest.java, Client2.java,ClientTest1.java) in ../src/demo
2. Client.java and Client2.java are the receivers. They don’t send any data.
3. Client2.java sends one image to the server it is registered.
4. ClientTest.java sends multiple images to the server to test scalability.
5. ClientTest1.java creates multiple threads and sends images simultaneously.


I think that's going to happen as part of my in-progress tour through the patch stack.


no idea where this came from.


Yup, you're right, one is off by a bit, good catch. Didn't look like it complained, though I was able to get to the else branch successfully while testing it out. Just changed that.


Better save it as `boost::intrusive_ptr<libtorrent::torrent_info>`. We need to miminize the copying/creation of torrent_info objects. It is an expensive operation and with many torrents it adds up. Plus, libtorrent 1.0.x isn't very efficient on this.
See #2968 and libtorrent bug [745](https://code.google.com/p/libtorrent/issues/detail?id=745).
(after this PR we need to investigate on how to cache these objects and request them every time we need them).


is this coming from the rfc?


I think we generally use `0.0f` syntax.


I presume this is from the rfc as well?


Instead of adding a new include directory, can you make the config.h do the right thing for windows using cmake tests?


You missed the actual typo :)


Do we need to do any kind of sanity checks on the parsed times or is trusting them as-is at this point appropriate?


Yup, it looks like the instance var isn't lined up with the `if` by one space in two different parts.


test.judge() returns a sciunit.Score.  The score attribute of a sciunit.Score contains the score itself, e.g. True/False in the case of a BooleanScore, so you can do something like:

```
assert(ten_test.judge(random_model).score)
```


won't this leave the service object mutated?  I'm not sure from context whether the object is yours to mutate or not


The same reason applies here. Take an intrusive_ptr and don't copy below.
Plus, does a QSharedPointer add anything of value in this class vs the intrusive_ptr?


I just wanted to make it clear


You can still restrict them by explicitly listing in the command line parameters.


@santhaseelank https://help.github.com/articles/creating-a-pull-request/


Will it be better to set `mDownY` as this `downEvent`'s y value?

Because using the current event point may have a small risk that when the user has moved his finger vertically by a distance between the touch slop and 2x touch slop. the list will get a click gesture.


We pass in zero to the first chunk. This is indeed the base case.

On each chunk that we upload, we pass along the server modified timestamp to the next. The last chunk passes this modified timestamp out, and we assign it to `lastFetched` on line 256.

The idea of this is twofold:

1. It does the fast-forwarding that every other Sync client does. See the comments on line 251. The zero will never end up as `lastFetched`, but `reduce` requires a base.

2. It allows us to (eventually) pass the last collection modified time as `If-Unmodified-Since` on each upload batch, and between the download and the upload phase. This alone allows us to detect conflicts. See comment on line 238.

In order to implement that, we'd need to chain the date from `getSince` on line 286 in place of the `0` on line 291.


agreed


Massive comment left inline.


If the aim of this was to solve problems for users switching to old versions of BA, would this not produce the same effect for people who follow that same process with current versions? Example: newer versions of the config that are using "getItems" will now no longer work because it is changed to "items" in the code. Essentially, it looks like you just recreated the problem for newer versions of the plugin. If you would really like to provide full backwards compatibility, you should check for both nodes.


There are seven different verbs that would work here:

http://sqlite.org/lang_insert.html

I'll add an enum.


The "throw" issue is because you are only performing these calculations when the element is near the edge. You'll likely want to remove that constraint.


Getting close!


I just noticed @teresafinn was added! Yay! That's gooder, for sure.


Thanks for removing these contact related update since oncontactchange should handle it already \o/


Ideally you'd want to use "whatever is fastest," right?  It'd be more work, but would it be possible to tweak the number of threads at runtime based on actual performance effects?  Human intuition is notoriously unreliable when it comes to performance.


I think this comment is not necessarily true. If you have an enum like for example

```java
enum TestEnum { MYVAL, MYVAL2, MYVAL3 };
```
it won't be using the initial value in all cases. In fact, it leaves subclasses the responsibility of deciding what is the single character that should identify them.


Revisen sus casos de uso que se entregan el día viernes


It is bad to store Mlt::ClipInfo in here because ClipInfo holds references to Mlt::Producers, which will inflate memory usage for producers that might no longer be used. Most producers cache structures, images, threads, and file handles - not just memory affected but also thread and file handles for which there are kernel imposed limits. Next, think about all of the UndoHelpers stored in QUndoCommands on the QUndoStack. All we really use from ClipInfo is frame_in and frame_out. Better to simply store those values.


@jroper This version of typesafe config requires Java 8.

https://github.com/typesafehub/config/issues/322

This has lead to a breakage in our [Java 6 based community build](https://jenkins-dbuild.typesafe.com:8499/view/Shared/job/Community-2.11.x/).

Do you use features from 1.3.0, or could you live with 1.2.1?


Based on the comment above, I would suggest naming the class something like `SingleCharEnumTypeDescriptor`.

You could also place a bound on the `T` parameter: `<T extends Enum>`. Thay way we only allow enumerations into this type.


The behaviour of signed overflow is undefined in C/C++.


:boom:


I would rename this to something like `getIdentifyingChar`, because again, `getInitial` is misleading.


Wow there's a lot pretty out-of-date on that page.

Re the promises, there's no component.json. Are you using this library already at nib?


Good catch, i'll push a fix for this.


So the only part of this that gave me pause, really, was that we're now apparently installing .py files into /usr/lib/python/dist-packages; that was a surprise.  Otherwise this makes obvious sense to me, other than perhaps needing update on the specific version.  Does the -530 need to match the version at the time of merge?


Moved to BrowserDB, which will do for now.


I know this used to be frowned upon because of performance (which probably means it's not an issue anymore) but I still use it for convenience. Any ideas/links to whether this is best practice or not?


```
background: url("/assets/images/bg-footer3.gif") no-repeat right bottom #000;
```
はアセットパイプラインが有効なので
```
background: #000 image_url("bg-footer3.gif") no-repeat right bottom;
```


正しくは、M504


できれば、どの項目をどのように変更したのかを書くと良い


Снова вы правы, а я неправ.
"напечатать в std::cout сведения обо всех фигурах, находящихся в списке, с разбивкой по типам;"

"с разбивкой по типам" надо убрать -- убираю...


This stops processing for all devices because of the failure of a single device. See the previous code and comments that were removed pointing this out.


タイトルが不適切。
つけるならば、「ユースケース記述の作成」というタイトルの
小項目として「仕様ツールの決定」などにすると良い。


floatの解除方法が古いので、今風なのに変えましょう。

floatする子要素を持つ要素に
overflow:hidden;
と
幅を指定

すればfloatは解除されたものと同じ表示になります。

といった無駄なHTMLを書かずに済みます。


Essaye pas des affaires "essais-erreur" juste pour voir si ça marche pis les commit.


IIUC, NamedQueryDefinition.getComment() will return null, unless a comment is specifically provided by one of the following:
* <query comment="..."/>
* <sql-query comment="..."/>
* @org.hibernate.annotations.NamedQuery( comment="..." )
* org.hibernate.annotations.NamedNativeQuery( comment=".."),
* from a org.hibernate.comment query hint.

If NamedQueryDefinition.getComment() returns a non-null value, I agree the query comment should be set to NamedQueryDefinition.getComment(); otherwise, I still think a comment should be set to a non-null value. If creating a new string by concatenation is too expensive, I would prefer setting the comment to queryName.


Can you update the typography documentation portion of the styleguide with these fonts?


case p:Program if(p.parent == null)  =>

Should do the trick an save duplication at the end of the case


Move down inside your foundation comment banner


Same comment applies here...


delete duplicate


Last week, I tested RainTPL4, it works as well. I haven't found any problem yet.
I would you to separate some of files that still contain many classes inside such as below
- RainTpl4.php

By the way, we should apply standard of PSR-4 (http://www.php-fig.org/psr/psr-4/) that recommend us to remove require or require_once in the class file. I see some files still have require_once

 \Rain\Tpl\Plugin\Compress has "require_once"
 \Rain\Tpl\Plugin\ImageResize
 \Rain\Tpl\Plugin\PathReplace

I will apply RainTPL4 to my project this, if I can


I would recommend changing the value of :sort-val to a keyword instead of a string.


Did, I'm working on it. Contacted @pinotree to clear one doubt about Qt L. out. Thanks for all your help.


Can remove (keyword ...) if make change above in the atom and when calling this function.


I'll check it by the end of the week, a little busy right now.


Sorry for my error , it works well . #343 can be closed


If this is filtering out NestedTuplesRef,  I think
immediates.flatten.collect { case NestedTupleRef(colRef,_) => colRef.collection }


If you look at your server log, you'll notice that the question is getting destroyed when the `GET /question/:id/edit` request is processed.  So even though you're seeing the form, the question has already been destroyed.  If you just refresh the page, you'll notice you get the error.  So your bug isn't when it hits the update action....your bug is somewhere around the edit action.

This line is where the bug is.
First, it looks like you're trying to comment out these lines, but I don't think it's having the intended effect.  Every language has a different way to comment out code and the comment syntax that you're using there (`<!--`) is only for html.  In order to comment out embedded ruby, you need to add a `#` after the `<%` and before the `=`.
So, since this line of embedded ruby is not commented out, it's being evaluated before the html gets rendered on the page.  Knowing that, can you figure out why the question is getting deleted when you load the page?


The arrow is nice, but isn't necessary. I would remove altogether, or move the css link to index.html and apply the arrow another way.


You may find you have a bug on this line too, although it has nothing to do with the other bug that's causing questions to be deleted on the edit page.


Börjar bli lite trött på github nu!! Det här är ju saker som jag lade till


If make change in atom above, need to change arguments passed to `update-sort-value` to keywords.


sina weibo api sdk


Seams this replaces the attr-th element of the keys list. This can be done with keys.patch(attr, Seq(Field(...)),1)


// Define the Circle class
public class Circle {    // Save as "Circle.java"
   // Private variables
   private double radius;
   private String color;

   // Constructors (overloaded)
   public Circle() {                   // 1st Constructor
      radius = 1.0;
      color = "red";
   }
   public Circle(double r) {           // 2nd Constructor
      radius = r;
      color = "red";
   }
   public Circle(double r, String c) { // 3rd Constructor
      radius = r;
      color = c;
   }

   // Public methods
   public double getRadius() {
      return radius;
   }
   public String getColor() {
      return color;
   }
   public double getArea() {
      return radius*radius*Math.PI;
   }
}


// Test driver program for the Circle class
public class TestCircle {    // Save as "TestCircle.java"
   public static void main(String[] args) {   // Execution entry point
      // Construct an instance of the Circle class called c1
      Circle c1 = new Circle(2.0, "blue");    // Use 3rd constructor
      System.out.println("Radius is " + c1.getRadius()  // use dot operator to invoke member methods
         + " Color is " + c1.getColor()
         + " Area is " + c1.getArea());

      // Construct another instance of the Circle class called c2
      Circle c2 = new Circle(2.0);          // Use 2nd constructor
      System.out.println("Radius is " + c2.getRadius()
         + " Color is " + c2.getColor()
         + " Area is " + c2.getArea());

      // Construct yet another instance of the Circle class called c3
      Circle c3 = new Circle();             // Use 1st constructor
      System.out.println("Radius is " + c3.getRadius()
         + " Color is " + c3.getColor()
         + " Area is " + c3.getArea());
   }
}


@aurimasv, looks like you forgot to update this. (copy to clipboard fails without this change)


You know that: `(Get-Content $file) -eq $content` returns true if _any one_ of the many lines matches `$content`, right?


Looks good tess!


If something has two translations (view is either *vista* or *visão*) I am free to choose as long as I am consistent, right?


Clearly not thoroughly tested.
1) Generate a tone amplitude 0.8
2) Amplify by -50 dB
Error.
"Value not in range: -51.94 to 48.06"


Whoooops


This doesn't work too well for cases when $MOZ_BUILD_APP does not equal browser and $MOZ_BUILD_APP doesn't seem to get passed to the mozilla configure from Comm-Central configure (looks like it does but s/browser//suite/ doesn't work). Not sure of a better fix besides just generally disabling it until #101 is done.


:clap:


Meus Arquivos/Java - Eclipse/MestreCuca/src/mestreCuca/LivroReceita.java

Quero fazer uma versão melhor que essa. essa esta muito gambiarra...


Looks like I did
On May 19, 2015 10:46 PM, "Rintze M. Zelle" <notifications@github.com>
wrote:

> @aurimasv <https://github.com/aurimasv>, looks like you forgot to update
> this. (copy to clipboard fails without this change)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rmzelle/zotero/commit/0923eb1342e5af4264dfcc04fdeb4686abb7af6a#commitcomment-11275965>
> .
>


Hah! Thanks for stopping by. Guess I had better update again based on the
changes I made on monday...
On May 19, 2015 6:03 PM, "Mike Sinkula" <notifications@github.com> wrote:

> Mike was here. ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bschwartz757/WEB170/commit/da71d52144c9d9062a7cb88e175479b8d030b6c7#commitcomment-11274710>
> .
>


DO NOT COMMIT : netmgrd.te


This functions helps to reduce time computations by caching the inverse of a matrix


please fix netmgrd.te ... because i found some error :(


Switched to `getBoolean`, `getTimestamp`.


The consumer of getDependentEntityInsertActions() should not modify the collection. Would returning Collections.unmodifiableCollection( dependenciesByAction.keySet() ) impact performance?


Bug 1163274.


Yeah, my bad.


It is just an extra (small) allocation. At the moment the consumer can modify the collection through the iterator anyway.


这种方法好高端的样子


This two functions create an object that stores a matrix vector and cache's its inverse.


Is there a reason the list is not initialized with size INIT_QUEUE_LIST_SIZE?


This would be a trailing comma


Should this become ours? IMOS?


You know pull to refresh is a component of the TableViewController right?


Upload to folder named grav in Dropbox.


!(unresolvedInsertions == null || unresolvedInsertions.isEmpty()) should be replaced with hasUnresolvedEntityInsertActions().


An oversight on my part, I will fix


This method shouldn't exist.


The first function makeCacheMatrix <- creates a list containing functions to set and get the matrix, and to set and get the  inverse of the matrix.


inv <- NULL
    set <- function(y) {
        x <<- y
        inv <<- NULL
    }
    get <- function() x
    setinvmatrix <- function(inverse) inv <<- inverse
    getinvmatrix <- function() inverse
    list(set = set, get = get,
         setinvmatrix = setinvmatrix,
         getinvmatrix = getinvmatrix)


(It was part of the partial-deletion logic.)


This has been updated for consistency with the rest of Play. This plugin itself is only used by the builds for Play modules for Play 2.4.x or greater, which all require Java 8. I don't think it makes sense to include this in a jdk6 community build, because it will never be used on jdk6 - it should be part of the same community build that builds Play 2.4.x and associated modules, and all of these require Java 8.  Do you have a Java 8 community build?  If not, you probably need one soon if you want to be building current versions of projects.


Take a look at [https://github.com/Team-Surge/Surge-Client/blob/develop/Surge-Client/PullToRefresh.swift](PullToRefresh.swift). It simply uses a reference to the tableviewcontroller. The goal of breaking it out is to avoid having to rewrite the same code multiple times, and hopefully be able to customize it in the near future.


NVM, sleepy eyes.


The first function makeCacheMatrix <- creates a list containing functions to set and get the matrix, and to set and get the inverse of the matrix.


I don't think these should be here. E.g. for SM I have node only, whereas for havas I have browser only.

We should keep this clean. It's easy to extend one config with another: (see "Extending Configuration Files" here http://eslint.org/docs/user-guide/configuring)


"Other protocols could negotiate protocols differently" is a bit tortured. Suggest: "Other substrates could negotiate the application protocol differently."


Note how I added the `collections` class. This is to make my javascript cleaner and easier.


Yeah, I'm much happier with this Sync client than I am with Android Sync. Turns out it's much harder to write a sync system if you don't control the database!


I also added the `data-collection` attribute to remove all the `if / else` statements in the drag & drop logic.


The second function will return the inverse of the  matrix input to makeCacheMatrix<-


we don't need to cache our books here.


`ids` isn't unused; see line 675.


"is used only for protocol identification" -> "is used here only to identify the application protocol or suite of protocols that the client intends to use in the tunnel."


inv <- x$getinvmatrix()
    if(!is.null(inv)) {
        message("getting cached data")
        return(inv)


I save the complete object of a book onto the `tile` element.


Where else are you having to use it?


"consequently uses a protocol that operates over TLS" -> "conveys a TLS session that in turn encapsulates another protocol"


I need to re-edit this to 'Executive Club's Chairman'


see the `.collections` selector that was added in the HTML?

When a `click` occurs on a `button` we query the `data-collection` attribute. This will be used to retrieve the books from `localStorage`

I made a helper function `getBooks` to make the code easier to read.

I empty the existing books and then render the books retrieved from `localStorage`


Switched to `for row in cursor`.

Iterating the cursor: remember that we have one row per *visit*, which might involve a place multiple times. We walk the cursor to collect the place for an ID and the visits for an ID. We accumulate into a `Set` because the cursor itself will feature each ID multiple times.

We finish by walking the set of IDs, combining the `Place` and the `[Visit]`.


"contains the single protocol" -> "contains a single ALPN Protocol Identifier corresponding to the application protocol"


Contextualise - e.g., "When used in the ALPN header field, ..."


We *could* do a fancy walking-rollup, reliant on the `ORDER BY`, which is what I'd have done in Java or Perl… but it didn't seem to suit Swift's style.


Nice work


see what I mean by how much code goes away.

When we get our `ui.draggable` which is our `.tile` we access the book object we stored when we rendered it.


The `getBooks` helper function


why are you enabeling/disabling ric again and again


I mean, there's no need to gloat really. We need to connect on some ?'s I have if you don't mind. :+1:


RADICAL!?!?!?!?!?!?!


Thanks for the explanation, that makes sense.

Yes, we have a Java 8 community build. So far we the Java 6 and Java 8 versions builds the same set of projects.

I'll fix back to a tag of `twirl` for now. Soon, we'll update our Java 8 community build to track Play/Akka 2.4.


I want to customize it before turning it in. I'm sure we will need refresh control somewhere else eventually haha.


* if the `book` isn't in `localStorage` we save it there.
* retrieve the `collection` from `localStorage` if it doesn't exist create an array.
* Only add the book to the `collection` if it currently does not exist in there.
* Save the `collection`.


Just need to make sure the template.obj.collection references are correct, then rehydration code can be debugged/finished. After that, off to the races with Kue.


A simple wrapper function for saving to `localStorage` it makes sure our data is in a string.


:+1: nice draft and I share your sentiments regarding week 2... :weary:


Why is this operator needed?


* retrieve an item from `localStorage`
* if nothing exists return `null`
* parse the string into a `object`
* `JSON.parse` can throw an exception resulting in an error. Wrapping it in a `try { } catch(e} { }` allows us to handle the error if it happens.
* return the data from localStorage.


如果你的提示文字已修改，请在新文件对应的提示文字修改，以防本地验证失效！
还针对按回车不放的用户优化，预防多余不必要的请求连接。

![20150520114333](https://cloud.githubusercontent.com/assets/11162054/7718719/284008cc-fee7-11e4-9105-5bd66d6c2ed4.png)
![20150520114257](https://cloud.githubusercontent.com/assets/11162054/7718720/2846696a-fee7-11e4-9c24-2fba31d539aa.png)
![20150520114146](https://cloud.githubusercontent.com/assets/11162054/7718721/2848a14e-fee7-11e4-9943-261cc858d871.png)
![qq 20150520114043](https://cloud.githubusercontent.com/assets/11162054/7718722/28504ab6-fee7-11e4-8b48-cefceb327e22.png)
![20150520113910](https://cloud.githubusercontent.com/assets/11162054/7718723/2871cc5e-fee7-11e4-9339-df0c7b2e9afa.png)
![20150520113827](https://cloud.githubusercontent.com/assets/11162054/7718724/288f7bfa-fee7-11e4-8e2a-ec164e90c4e2.png)
![20150520113805](https://cloud.githubusercontent.com/assets/11162054/7718727/28e938c0-fee7-11e4-9eb8-9e572028f107.png)
![20150520113748](https://cloud.githubusercontent.com/assets/11162054/7718728/291d7630-fee7-11e4-92d5-d93a30397396.png)
![20150520113704](https://cloud.githubusercontent.com/assets/11162054/7718725/28a30e18-fee7-11e4-941f-a29502ff06cc.png)

```js
//欢迎回来 是服务器返回的提示文字，如果你的已经修改，请在新文件对应的提示文字修改，以防本地验证失效！
if($("#msg-success-p").eq(0)[0].innerHTML=="欢迎回来"
                    || $("#msg-success-p").eq(0)[0].innerHTML=="你已成功登录，如果页面不跳转，请刷新！"){
                      // 这里第一个是提示文字，如果要改，上面判断的也要一起改。
                      //第二个是显示的样式，分两个： 出错了（error）成功（success ）
                        msg_out("你已成功登录，如果页面不跳转，请刷新！","success");
                        //msg_id=1 不执行（提）登（交）录
                        msg_id=1;
                         $("#msg-error-p").html("");
                }
                if($("#msg-error-p").eq(0)[0].innerHTML=="邮箱或者密码错误"
                    || $("#msg-error-p").eq(0)[0].innerHTML=="邮箱或者密码错误，请重新输入！"){
                     if($("#passwd").val()==inpasswd && $("#email").val()==inemail){
                        msg_out("邮箱或者密码错误，请重新输入！","error");
                        msg_id=1;
                        return false;
                    }
                }
```


OKです


Wut?


I'm assuming this change snuck in here by accident ? :) I'll revert unless you'd rather us keep it this way?


Wasn't intentional, was probably a copy-and-paste error or a brain fart.


That's a good question...


It's fine, harmless.


@Looooong  is ok?


@pavel-kirienko - for get the close operations - will add them in the AM


DS - Store Bounds as a value instead of a pointer, and return by value (e.g. from


@pavel-kirienko Added the Close operations


+1, thanks. :)  It's my goal to only complain about details that matter, as there will be plenty of those.  But I appreciate this attention to detail!


starting to like the new toy @shashiranjan84


Cool! I did get it from CPAN. I've got the new, surgically edited version from here, just as soon as I figure out how to upload it. I've never used github, it tells me "non-fast-forward" updates are rejected when I just try to replace it =x I'll get it.

That may be a good thing, because a user testing it had it fail with a "Bad Captcha" error. That's caused by not having enough link karma on the account. I should probably find a graceful way to handle that, maybe similar to how it handles posts/comments from new accounts that break the 10 minute rule. Having developed bots before, I make sure I'm using an account that has enough karma, so I actually never bumped into any of those errors.

The Python API I've used prompts you for a captcha when that happens, but of course, there's no captcha, it's the command line, so it just fails anyway. Maybe I'll see if a link to a captcha image can be returned, although I doubt it would be useful (you'd have to be using some sort of graphical interface to even see it).


Shouldn't this refer to the OM repo at https://github.com/outware-mobile/JSONKit


Sorry!


np... don't merge!


inv <- x$getinvmatrix()
    if(!is.null(inv)) {
        message("getting cached data")
        return(inv)
    }
    data <- x$getinvmatrix()
    inv <- solve(data, ...)
    x$setinvmatrix(inv)
    inv
}


(I hijacked this pr to test Yichao's question about JLOptions.image_file... tsk tsk to myself)


Sort of defeats the purpose of the log_debug proc but very well.


I think  you shouldn't look at this as if the consumer is aware or not of the fact it is using an ORM or not. My opinion is you should decide if you want to implement Unit of Work pattern or not. I mean even when you are using only stored procedures for your IContentStore implementation can be useful to use UoW pattern. If you want it you will need to include tracking code in your ContentStore. So, do you want to provide UoW it to upper layers or the upper layers should track themselves which Documents has to be updated on db? If the annswer is No, you should always use .AsNoTracking() if you decide that yes then you should offer tracking infrastructure like the one you are using now from EF also for alternate ContentStores.
So, if we agree we are talking about UoW and tracking infrastructure no matter if implementation is done by  an ORM or not. So, to have the possibility of stopping this tracking whenever we need it for performance reasons is a feature that has nothing to do with the fact you are using an ORM or not, it is related with the fact that do you want to  offer a tracking service to upper layers or not.


Thanks for these! Now my eyes can just glaze over on that math and ignore it :)


I think that "Productivity Tools" section is more appropriate.


Yeah can't spell magic whelp


I didn't want the overhead of the checks log_debug() does if I'm not debugging fire. I could have probably left it with world << but this seemed nicer somehow.


This is AMAZING. Thanks so much, Nate. I can't wait to look through it line by line tomorrow!


No need for an extra namespace under mozilla.


Note: we've moved away from PR_GetEnv to using an extra log level, see bug 1164297.

(and PR_LOGGING has been removed, IIUC)


@netj Please check my commit.


Hi Dillbyrne ,
    Really thanks for your contributions.I have been using RAS for quite a while and it is really good.
Normally  I use it for CPA offers testing.
By the way , I'd like to know if it is possible to buy a private version for my own specific purpose.
My skype:newyorkheart2000.
I hope you can add my skype and let's have a chat.


I do apologize I never answered your question. As port 0 is not a valid port I'm not sure how anyone would connect to "any" port. I don't think that 0.0.0.0 and 127.0.0.1 are the same, 127.0.0.1 would only accept connections from the localhost whereas 0.0.0.0 accepts connections from any host. It has been too long and I've forgotten the context of the discussion but seems to be about listening for connections.


On Mon, May 18, 2015 at 2:13 AM, Juan G. <notifications@github.com> wrote:

> Hi @vsingleton <https://github.com/vsingleton> this is wrong and will
> cause the jar to not work in Glassfish 4. Please can you fix this and
> re-send? Thanks!
>
I would love to agree with you.  But the phrase "this is wrong" tells me
nothing.  What _exactly_ is wrong?  The phrase "fix this" does not tell me
what to fix.


Thanks!


@andreas-jonsson This introduced build error in MS VC++ 2010.


Good move :+1: FWIW I find the TypeScript team does the same :rose:


The answer to "if this is needed or not" boils down to how this is even figured out.

In Forge, its based on the running thread. SERVER_THREAD is obviously SERVER and likewise with CLIENT. If you are on another thread accessing platform, it may not be immediately apparently which one it is. This is more an issue with Sponge as SpongeVanilla will just return SERVER for the side.


good morning. debug line? or if/else remove? :D


Yes, you are correct.
As I mentioned in the conversation part, it is not correct to points to any private repo. While in our project (such as SSS), we use local podSpec. So you may notice that I have reverted this file in the following commit.
At this stage, do you want me to change the podSpec? If yes, I will do it.
Thanks.


For run cmd


This closes #1


For project name


jQuery support


These File are not in use in project


For java Web project


I prefer this on two separate lines


You can now do things like this:

```xml
<populationSize spec='PopulationSize'>
  <predicate spec='Predicate' value="abs(i-100) < 50"/>
  X[i] = exp(-(i-100)^2/10)
</populationSize>
```

You can also use functions defined in the model.

Maybe @alexpopinga is interested in this for her simulation studies?


this is good


I prefer the original


this is really clean way of handling it. Super happy to see this!


Are unpaid dues represented as 'Nothing'?


no, just leave as is, I think it's right - the -lrtdm flag was needed by Xeno2 only if you used some RTDM device driver from RT, like the RT-CAN
pretty sure this works with Xeno3 without that flag,but we'll find out once a Xeno3 patched kernel exists and we try an RTDM driver - no urgency


Comment added


Not really a fan of your commit message style. It makes it really hard to parse what was changed just browsing the commit list.


Target has to be choosen while the trigger goes to stack not during resolution.


[this style is soft deprecated](https://github.com/hadley/testthat/blob/master/NEWS.md#testthat-0919000)


I love how you guys are stubbing data out


LGTM， @lijiajia Pls review and notice the change:)


I'm a fan of this code :+1:


If you send a PR I will gladly accept it.


Another way to do this is with the `<istrue>` tag. For a boolean flag like this, you could set it to either `0` or `1` (true). The `<istrue>` tag is equivalent to this in PHP.

```php
<?php
$value = 1;
if(!empty($value)) // is true
```


Might be nice to move the inline styles into css.

I love the use of font awesome!


@Underground78: this is just an adaptation of the Wiki article. Feel free to make any changes.

Also, perhaps we should add a FAQ entry or something so that link is actually discoverable.


Эм... вот в последнем файле прикол... правка шаблона напрямую? Может лучше как-то перекрывать вьюхи оригинальные, а не править напрямую? Это было неожиданно. В следующий раз буду внимательнее.


I really like how you put this into a separate file.


preferred way is <% cache @rails_model_name do %>
second preferred way is, <% cache"colorado_retreat_landing", expires_in: 1.hours, race_condition_ttl: 1.hours do %>
last preferred way is <% cache "something" do %>


@josh [ngx_http_gzip_module](http://nginx.org/en/docs/http/ngx_http_gzip_module.html) does not do *this*. I am sure you know it and that was a sentence just quickly written, but for the archives what the standard compression modules do is compress on-the-fly. On the other hand [ngx_http_gzip_static_module](http://nginx.org/en/docs/http/ngx_http_gzip_static_module.html) is able to serve an already compressed file if present on disk.

I know two use cases for [ngx_http_gzip_static_module](http://nginx.org/en/docs/http/ngx_http_gzip_static_module.html):

1. As with other asset optimization techniques, if you are able to compress text-based assets as part of your deploy workflow then you can apply **maximum** compression because it is going to be done only once and can afford the extra CPU cycles, and **later the server has to do nothing on-the-fly**, just serve the compressed variant from disk.

2. **Serve compressed assets from CDNs** for browsers that are able to understand a compressed asset, because CDNs do not compress on-the-fly.

Regarding Transfer-Encoding, we could argue if theoretically this should be done at that level. But at the practical level that header is useless for the majority of use cases of Sprockets.

The pair Accept-Encoding/Content-Encoding is absolutely dominant and it is what we have to deploy for *today*. On the other hand, in order to be able to indicate compression in Transfer-Encoding the client has to first acknowledge it is willing to accept it via the TE header. As of this writing Safari, Firefox, and Chrome send Accept-Encoding: gzip and **do not** send any TE header, so as a matter of fact the serve just **can't compress via Transfer-Encoding for them. In addition to that, it seems [Chrome is not even considering TE](https://code.google.com/p/chromium/issues/detail?id=94730) for compression. So Transfer-Encoding for compression is useless for all practical purposes.

In my experience [ngx_http_gzip_static_module](http://nginx.org/en/docs/http/ngx_http_gzip_static_module.html) works flwalessly, have used it for years. And that is what motivated me to add this feature [back in 2011](https://github.com/rails/sprockets/commit/51f167d3906dab7201f0661dce5c8345ef0d7229).

That is only a response to the objection to the whole idea of using this technique. Of course, whether this belongs in Sprockets core or not is you call.


LGTM


@Diazonium
I believe we will never miss anything (no need to increase this value) because
1. Input is same. We only tell zlib to output first 1024 bytes of compressed stream first (we continue on later), so it is guaranteed to be same.
2. If there are more than recompTresh bytes different in first 1024 it means there would be more than recompTresh bytes different in whole stream also. We skip full recompression only in this case and in this case recompression is never done anyway so we didn't miss anything i think.


Order to write logs:
`LOG4CXX_DEBUG(logger_, "Groups Ids: " << groups_ids);`


Weird changes on multiple files result from a disastrous operation, coz I unintentionally pull from purcell's repo, which result in a chaos log files.


@Underground78 for next batch


There is a rendering problem with long canvas items. I have noticed this with waveforms where they do not appear. That is not so bad, but this is worse for 2 reasons: this canvas spans the entire length of the track - not just a canvas the size of a clip - and it renders as opaque black. :-( Maybe we need to repeat canvas items of a limited width.


IGNORE, this was accidental


Absolutely. Since the step identifier would be specific to navigation rules and not used outside that scope, I was thinking perhaps

`ORKNavigationRuleIdentifierNull` = `org.researchkit.navigationrule.nullstep` or something similar.


Chris, I benchmarked how you added Python mime type to add Matlab mime type.
1. I added mime-type to filemgr/src/main/resources/etc/mime-types.xml

 <mime-type type="text/x-matlab">
    <_comment>Matlab source code</_comment>
    <magic priority="50">
      <match value="function [" type="string" offset="0"/>
      <match value="%" type="string" offset="0"/>
    </magic>
    <glob pattern="*.m"/>
  </mime-type>

2. I fixed pge/src/main/resources/bin/mime_partitioner/mime_rat_partitioner.py
   mimeTypes = ['x-java-source', 'x-c', 'javascript', 'xml', 'html', 'css', 'x-json', 'x-sh', 'x-fortran', 'csv' 'tab-separated-values', 'x-tex', 'x-asm', 'x-diff', 'x-python', 'x-matlab’]

And I did 3. ‘mvn install’, 4. 'tar -C ../deploy/ -xvzf distribution/target/dms-distribution-0.1-bin.tar.gz’

However, still drat doesn’t audit Matlab files..
Do you find anything wrong with my procedure?


Oh, thats right, by default it will render into an enormous qimage which
wont do. This was not an issue when it was simply a black rect in the first
revision...

On 07:16, Wed, May 20, 2015 Dan Dennedy <notifications@github.com> wrote:

> There is a rendering problem with long canvas items. I have noticed this
> with waveforms where they do not appear. That is not so bad, but this is
> worse for 2 reasons: this canvas spans the entire length of the track - not
> just a canvas the size of a clip - and it renders as opaque black. :-(
> Maybe we need to repeat canvas items of a limited width.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/metellius/shotcut/commit/10684e82f472d0062d60f371b80da034bd567ec8#commitcomment-11276555>
> .
>


Spelling mistake (contents vs contenst)


Course Project
## Heading


the `done` anonymous function is getting kinda long. Nothing wrong with your code. Strictly an organizational suggestion.

```js

function getButtonInfo() {
   $('.nyImageContainer').on('click', '.likeForm input', function(e) {
       e.preventDefault();
       e.stopImmediatePropagation();
       var buttonInfo = $(this).data('id');

       fetchArticle(buttonInfo, renderArticle);
   });
}

function fetchArticle(info, callback) {
  var url = 'http://api.nytimes.com/svc/search/v2/articlesearch.json' +
       '?callback=svc_search_v2_articlesearch&fq=_id:("' +
        encodeURIComponent(info) + '")&api-key=' +      '18a905653b4f0581867fa5199b5156e0%3A5%3A72064799';

   $.ajax({
     url: url,
     context: document.body,
     method: 'GET'}).done(callback);
}

function renderArticle(data){
    var oneWebUrl = data.response.docs[0].web_url;
    var oneArticleImage = data.response.docs[0].multimedia[1].url;

   var img =
    $('<img src="http://www.nytimes.com/' + oneArticleImage + '" />')
       .css({ // this should be moved to css file.
             'border-radius': '10px',
             'margin': '20px',
             'height': '75px',
             'width': 'auto'});

   var link = $('<a href="' + oneWebUrl + '" />').append(img);

    $('.savedImagesDiv').append(link);
}

```

See how I didn't change your code at all but it should be easier to read and follow the logic of what is happening.


I'd prefer an explicit check for the null step identifier. This is neat, but seems unnecessarily fragile.


Ignore my previous comment on naming; this is fine.

However it does need headerdoc. I would also move it to ORKStepNavigationRule.h/m instead. It doesn't belong in the `ORKStep` header since one should *never* use it with `ORKStep`. It also needs mentioning in the `ORKStepNavigationRule` headerdoc too, i.e. wherever it's an acceptable return value.


Leave a line note


LeaveLeave a line note


## This is a markdown file


Translating everything here is a little cumbersome is it necessary?


修复断网重连后提示消息不隐藏的bug


Maybe tumblr's customize_api was changed.
TextPost, ChatPost and etc became excluding `{block:Text}`, `{block:Chat}`, and etc.
(But PhotoPost always includes `{block:Photo}` list as image list, so PhotoPost is OK.)


This pulls in a bunch of the refactoring from https://github.com/18F/C2/pull/346 – minus the schema changes – so that that PR can be as minimal as possible and we can compare apples to apples.


y esto por qué?


There is a Chinese translation. https://github.com/ye11ow/http2-explained-chinese


This is about the connect() call for UDP (SOCK_DGRAM) sockets. UDP doesn’t actually have connections like TCP, but the call still affects which peers are accepted for recv() calls and the default peer for send() calls. The four cases I listed came from experimenting with normal non-proxied UDP sockets on Linux. Perhaps you’re thinking of how bind() works instead, setting the local address rather than the peer address.


Do you mean `GPLv2`? There's no such thing as GNUv2.


Bientos Ricardo :D


guessing *0.90117 - Model = GradientBoostingClassifier(n_estimators=200, learning_rate=0.097, min_samples_split=3, max_depth=9, verbose=1) & st.loc[idx, 'prediction'] = 0.399 # pure hack

0.89362 - model = GradientBoostingClassifier(n_estimators=200, learning_rate=0.0965, min_samples_split=3, max_depth=9, verbose=1) & st.loc[idx, 'prediction'] = 0.399 # pure hack

0.89271 - Model =  GradientBoostingClassifier(n_estimators=200, learning_rate=0.098, min_samples_split=3, max_depth=9, verbose=1) & ytest.loc[idx, 'prediction'] = 0.0399 # pure hack

0.88589 -  Model = GradientBoostingClassifier(n_estimators=200, learning_rate=0.0985, min_samples_split=3, max_depth=9, verbose=1) & ytest.loc[idx, 'prediction'] = 0.0399 # pure hack


The variable `i` should be made local.


reference: http://bugs.python.org/issue10666 and http://stackoverflow.com/questions/7116038/python-tab-completion-mac-osx-10-7-lion


@XhmikosR: Could you change just the English string without reseting the translations?


Ok, I guess a FAQ entry would make sense.


Don't forget to clean up after the test by removing the `test` entry.