I believe Doug is right, this is the missing piece.  You'll see where the ther `register*` methods are being called there, you'll just need to do something similar.


If by evensquares you mean pawns on files b d f or h, then...

A White Bc1 (dark squares) controls the even files (bdfh) rank 8
A White Bf1 (light squares) controls the odd files (aceg) rank 8
A Black Bc8 (light squares) controls the even files (bdfh) rank 1
A Black Bf8 (dark squares) controls the odd files (aceg) rank 1

So line 330 should start with
score += square_color(s) != Us ...

Or I missed something ?!


This condition seems to have no sense: we check archive names but use previously calculated filenames. And old code don't work with multifile packages. In general, there is API inconsistence, as `archiveName()` can return several names. But I don't know project good enough to change API, so I decided to fix problem in place.


I prefer the set version with union / difference.
Using filter means you need to create an other variable to build the toRemove set.


Perhaps the better question is do we care to maintain an API and web site? It's just us, maybe ditch one. You like the API stuff and auth perhaps we should throw together an ember client that replicates the three pages we have.


Good


iteration (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2344 of 2384 tests passed)

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


Hey Jay;

I was able to get your "with-dirwalk" sample to work in my testing!! I took the code from your with-dirwalk.py script and tested it with the os.walk directory set to current working directory and i noticed the print out was occurring twice for the commands contained in the file router.txt. What I discovered after commenting out stuff and retesting is that I didnt need this section under def main():
for dirname in dirs:
            if fname.endswith('.txt'):
                process_file(os.path.join(root,fname))

Once i commented out this section and added parenthesis around the print functions to account for 3.x syntax it worked like a champ and extracted the voice translation instances from all the sub-directories.Thanks dude, this is awesome!!!


misspelled "canola"!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11026](https://teamcity.mesosphere.io/viewLog.html?buildId=102215&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:12


It's a dependency size issue. Trim was too heavy and worked its way into `toNumber` which then is used all over the place. So I had to lighten it up. I tried built-ins but that's only a sure thing for `trim` and not `trimRight` and `trimLeft` so I went to the good old fallback.


Good to know, @gberaudo wrote:
>A partir de la semaine prochaine, IE9 sera uniquement supporté par Microsoft sur Windows vista:
https://support.microsoft.com/en-us/lifecycle#gp/Microsoft-Internet-Explorer
Windows vista sera encore supporté pendant 1 an, jusqu'au 11 avril 2017 :
http://windows.microsoft.com/en-us/windows/lifecycle
A partir de cette date, la version minimale de IE pour le bureau Windows sera IE 11 (WEBGL).

Perhaps it's reasonable to drop those IE9 compat fixes?


Now that you know we won't ever run the `telegraf_config` resource from other cookbooks (just the outputs and plugins), do you need these file resources?  or can we go back to using the `telegraf_outputs` and `telegraf_plugins` resources here?  This doesn't seem very DRY.  :droplet:


@matcracker did it.


Even by file index, yes this can be confusing!
Even : File A,C,E,G
Odd : File B,D,F,H


In fact I got the issue number wrong. Should be DOCS-374.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25114/testReport/) for this commit**


Nice :-) Player can't wait. Thanks you so much.


111101900_MSHeatPump (Nigusse) - x86_64-MacOS-10.9-clang: OK (2015 of 2016 tests passed)

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


I’d just say something like “Requires `juttle` 0.2.0.”. The changes are not user visible and they don’t influence anything, so no need to mention them IMO. If you really want to mention the work, I’d say something high-level like “Adapted to changes in `juttle` 0.2.0.”

The point is that the changelog should be primarily useful to users and contain things that are of interest to them. If anyone cares about more details, he/she can always look into the commits.


HTTP


Check out how I edited your codeblock to show formatting above, using triple backticks.

Click on Styling with Markdown :point_down: for more!


Maybe «Додатки»?


@dastgir what is this update for? (me not updated on renewal xD)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11027](https://teamcity.mesosphere.io/viewLog.html?buildId=102216&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:22


NitfImageSegmentHeader used to be an class and it's now an interface.  Although it has the same name, it's actually a new file.


Ah yeah, makes sense. Thanks.


Not sure if replicating the changes structure from `juttle` (Major Changes/MinorChanges/Bug Fixes) is needed in backends where the number of changes will typically be small. I’d just put everything in one big clump.


It works. Perhaps that's an improvement in Laravel 5?


How about «Приватні чати» or «Приватні розмови»?


«безплатні» → «безкоштовні»
«конфіденційнимию» — «конфіденційними»


Yes indeed, I think that was the question I had at the end of the first comment I made on this in regards to it turning into a lot of effort for something that we may ditch anyway and just use the api.

In fact to be honest at this point you can do everything the html pages can so I'm sort of ok with ditching it and maybe revive vilive and let it house the ember stuff?

I revisited the post with the ember/elixir stuff and it doesn't really look like that much to enable the CORS stuff.


LGTM (already in master).


According to the spec, we have to support IE9. But I don't think it's reasonable to waste time on supporting IE 9. We should check with the client.


Hey, would you mind contacting me?


According to our stats on c2c.org, IE9 represents less than 1.8% of visitors. 3 months ago, it was (still) 3%. My guess is it's safe to assume that IE9 will not be of any concern when we release V6.

Current statistics:
FF 32%
IE6 0.0
IE7 0.1
IE8 0.8
IE9 1.7
IE10 0.9
IE11 13.2
Edge 2.2
Chrome 37
Safari 11.5
Opera 0.6
Others 0.1


Sorry for the breakage.
Fixed by https://go-review.googlesource.com/#/c/18430/.

On 8 January 2016 at 09:13, Siarhei Navatski <notifications@github.com>
wrote:

> How to get stringer using in golang 1.4 after this commit? I'm using
>
> go get golang.org/x/tools/cmd/stringer
>
> It happens on clean environment without stringer previously installed
>
> docker run --rm golang:1.4 go get golang.org/x/tools/cmd/stringer
> package golang.org/x/tools/cmd/stringer
>     imports go/constant <https://goto.google.com/constant>: unrecognized import path "go/constant <https://goto.google.com/constant>"
> package golang.org/x/tools/cmd/stringer
>     imports go/importer <https://goto.google.com/importer>: unrecognized import path "go/importer <https://goto.google.com/importer>"
> package golang.org/x/tools/cmd/stringer
>     imports go/types <https://goto.google.com/types>: unrecognized import path "go/types <https://goto.google.com/types>"
>
> docker run --rm golang:1.5 go get golang.org/x/tools/cmd/stringer#works without errors
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/golang/tools/commit/542ffc7e75cf45ac7ad456664a710cd96338fda6#commitcomment-15340163>
> .
>


We would need to find a better name than manipulator ;)


bad indent


“Itegrated” → «Інтегрований»


Fine !

If your const Bitboard EvenSquares = 0x5555555555555555ULL;
represents File A, C, E, G, (I can't  check it now)
then your test should do as advertised.

Good luck !


Raw string literals can be more readable for this:

```rust
format!(r#"{{"instance": {} }}"#, instance)
```


(I'm sort of a client, but I assume Stéphane should be the one to contact in order to validate this, although you can use my support for arguing)


You have a typo


Comentários e Strings em código a gente tem traduzido. Pode traduzir esse aqui tbm @mauriciomaniglia?


Esse aqui também.


@TreyE Do you think it might be a good idea to print this into a text file instead of printing straight into console?


http://stackoverflow.com/a/24371533

Seems as though `Promise` is supported natively as of `0.12`, which is very old now... but old versions are being built/tested by Travis CI; so yeah, probably worth shimming for backwards compatibility.


does not directly throw exception


syslog-ng 3.7.1 compiled from source.

It is indeed running in threaded mode.

--
Jean-Baptiste Fuzier
0670657841
http://twitter.com/jbfuzier

On Fri, Jan 8, 2016 at 3:49 PM, Balazs Scheidler <notifications@github.com>
wrote:

> We just recently bumped into a similar issue, which might be the same as
> yours. What is your syslog-ng version? Are you running it in threaded mode?
>
>
> --
> Bazsi
>
>
> On Fri, Jan 8, 2016 at 11:52 AM, jbfuzier <notifications@github.com>
> wrote:
>
> > Sorry I made a mistake and did a pull request to your repo's master
> instead
> > of my fork master.
> >
> > So far I doubt those changes are useful for others, I am in the process
> of
> > investigating problems in the DNS cache. Basically we have around 1000
> > hosts logging to the server so we need the cache and what we are seeing
> is
> > that even with cache and expiration set to a high value; the daemon is
> > still constantly doing dns reverse lookup every 5-10secondes for the same
> > hosts.
> >
> > So that is why I am adding those quick & dirty debugging log..
> >
> > --
> > Jean-Baptiste Fuzier
> > 0670657841
> > http://twitter.com/jbfuzier
> >
> > On Fri, Jan 8, 2016 at 11:25 AM, Balazs Scheidler <
> > notifications@github.com>
> > wrote:
> >
> > > A bit more clarity in the messages would be useful, e.g. the fact that
> it
> > > is coming from the DNS cache would be useful.
> > >
> > > On the code side, proper indentation would be better:
> > >
> > > msg_debug("DNS Cache full, removing one entry",
> > > evt_tag_str("hostname", &cache_first.next->key),
> > > NULL);
> > >
> > > With those, it could even be merged to master.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15336583
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15337038
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15340979>
> .
>


SamsungBCMRIL itself a part of AOSP code so i added to see any difference
Just wondering why it doesn't give a stderr when building :D


XD usunales i dodales te same linie  ?


We can add a config setting, but I feel that would be polluting the config area with details.
Any suggestions how to handle semi random constants like this?


@egor-kalinin will be fixed by https://github.com/doctrine/dbal/pull/2286


unack, hey :) Your commit message is a bit weird:

1. As GitCop already mentioned your body contains lines that are too long.
2. For the shortlog, how about: `Constants: Move out of class`? That's way shorter and more concise IMO.
3. Your spacing is weird. Please put no space before the colon (`Constants:` is better than `Constants :`) and put a space after periods and commas.

Looking at the code roughtly that looks good. We can probably merge that after the commit message gets ready :) You'll be accustomed to this style in no time and then it'll be easy for you to write nice commit messages - they're always challenging in the beginning.


this if check should be:
```
if (S==CXX && res == NULL)
```
because otherwise we would raise a C++ exception even though a CAPI exception is specified...


Acho que aqui não precisa da vírgula depois do `desta forma`.


Yes, but then every line is screaming "hey, see this `?`? this is because something ten lines above might have gone wrong, but the execution continued, and if you want to know why you need to know about the test suite runner thingy, Travis and whatnot". I'd argue that it is preferable to exit the path, either cleanly by using a guard and normal, explicit control flow or just crashing.


Allocating thread objects has overhead, and thread objects also use a lot of memory.


@stustison - this is how you now obtain the principal from the message.  We still have the fall back scenario just in case.


> It means you can't express an open path that ends on its first point

at least, that seems to be the author's (@LettError) original intention, but I'm not 100% sure.


修改成功


Fixed.


Any particular reason why this instance of {couchbase_server_home_path}/bin was not changed to {couchbase_server_bin_path}?


Acho que em:
>antes de a extensão ser definida.

talvez pudesse ficar:
>antes da extensão ter sido definida.

O que acha?


My thought was that it reflects the semantic versioning structure. But I'm OK with a longer list...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11028](https://teamcity.mesosphere.io/viewLog.html?buildId=102220&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:15


Маленькое замечание в образовательных целях однострочные названия переменных это плохо. Фиг поймешь потом чего ты и зачем назвал. переименую переменную `u` в `user`


Maybe better to translate it as «Gitter доступний у всіх сучасних браузерах, так само як і для мобільних пристроїв та персональних комп’ютерів»?


this should be ```throwCAPIException()``` instead of ```return NULL```


111303866_ATSingleDuctReheat (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2014 of 2014 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)


Well, I agree with it is preferable to exit the path then I would prefer doing this:

``` swift
guard let result = result else {
    XCTFail("Result is nil")
    return
}
expect(result.hasNext).to(beFalse())
```


Conseil : Lire ce Commit avec l'option w=1 pour la lisibilité

https://github.com/ngonzalez/mapotempo-web/commit/76880e1aafc7e5b053364fd9e41c45c0b71ec765?w=1


Sharing options usually open in popups like this. We have no preference, but this is how most sites does it.
As this hijacks the a tag and uses the href it should work as a standard link anyway.
We will add the `_blank` in templates right away.


idk why but this crashs my Server.


// ACHIEVEMENT_CRITERIA_TYPE_COMPLETE_ARCHAEOLOGY_PROJECTS = 3
+struct
+{
+uint32 unused;                                 // 3
+uint32 count;                                  // 4
+} archaeology;

-// ACHIEVEMENT_CRITERIA_TYPE_REACH_GUILD_LEVEL      = 125
-struct
-{
-uint32  unused;                                 // 3
-uint64  level;                                  // 4
-} reach_level;
+struct
+{
+uint32  unused;                                 // 3
+uint32  level;                                  // 4
+} reach_level;


This breaks the install instructions for OSX.
You need to add a statement to Install .md to the effect
copy scsynth from Macintosh HD/Applications/SuperCollider/SuperCollider/Contents/Resources/scsynth to
<SONIC PI HOME>/app/server/Native/osx/

Secondly, if you move scsynth from the supercollider app, you also must have the .scx plugins folder also in <SONIC PI HOME>/app/server/Native/osx/ This must also be made clear in the install.md
Up until now I have been running with scsynth and the plugins inside the supercollider app install.
I have about three or four different builds on my mac. This means I now have to have three or four different plugin folders, whereas before they were all centralised. So this is an annoying change, although I suppose it mirrors the Windows and Pi installs more closely so is probably overall a good thing.

A possible bug....
First time I ran without having moved scsynth I was in dark mode.
a) the time out seemed excessively long before the error dialog came up.
b) the buttons were blank ovals until they were clicked. (second time I tried, the Details button showed text but the OK one was still blank
Both buttons showed OK in normal mode.


fillInMissingOdds以后把Id全部设为null再save, 之前存在的odds就会又存一遍吧？


very cool Dominik!


Sorry I forget to mention that the script was made by @sirikfoll, thanks to him.


I don't think this can cause a `ConcurrentModificationException` as the only code calling this method already is wrapped in a `synchronized (rangedRegionState). I think the comment "Need to check if region is null in case it was removed from the collection by another thread during iteration" can be removed.


This error message is a bit misleading if `path` is a directory:
```
$ cat /
cat: /: No such file or directory
```
GNU `cat` prints a different error message in that case:
```
$ cat /
cat: /: Is a directory
```


Hi @rbnpi,

this change was made to make things easier to debug remote systems - just in case their local copy of SuperCollider was used due to some error with the packaged version. The aim of the code base is to make things easier for the users not the developers, so while I understand your pain, I'm happy with making things simpler and more straight forward.

Still, you should be able to recreate your old behaviour using some symlinks - one for the scsynth exe and another for the ugen folder.

Not sure what to make out about the bug report. If you can replicate it, then please open it up as a separate issue :-)


Check row 38 staticAmmo there is RHS Ammo in it


I only run regular Menda to check compatibility.
![cetme-l](https://cloud.githubusercontent.com/assets/8630254/12201670/55e4291c-b62e-11e5-8544-0e75ebe2f68c.png)


thx to @matcracker


ok, so what I should do then, is to maybe not ask "instanceof StreamableSearchResult" and add a `getStreamURL()` method on the `UISearchResult` interface if there's such a thing and ask if that's non null instead.


If visibility never matters for friends then that is easy to do.


Since this discussion is on a tiny change where I was syncing an example to match a rule, here's [the actual rule](https://github.com/NoRedInk/ruby-style-guide#consistent-classes) we're discussing.

> I just have never seen the style of putting the initialize method above the macros, and am not sure what the advantage is with that.

Here are my thoughts about `initialize()`.  I like to use it as the dividing line between class stuff (i.e. `extend`, Rails macros like `has_many`, and class methods) and instance stuff (i.e. `attr_*` declarations, public methods, and private methods).  I feel like it's ideal for that for a couple of reasons:

* It it the transition point in that `initialize()` is an instance method, generally first called for you by the class method `new()`
* It's the interface you must satisfy to get an object to be able to call any of the other instance methods

> I'm not sure I follow why macros should be placed with instances methods.

Now here is where I think there's some confusion and we may be discussing two different things.  Again I recommend consulting [the style rule that prompted this change](https://github.com/NoRedInk/ruby-style-guide#consistent-classes).

Most macros are still above `initialize()`.  Using Rails as our example, associations and validations would still be above `initialize()`.  I moved only small subset, the `attr_*` attribute macros, below `initialize()`.  Because…

> I like the macros at the top to see what the expected attributes are at a glance, and then encountering initialize to confirm.

I feel like you pretty much nailed my reasoning right here.  I also like to see the "expected attributes" first.  In a Rails model, that's typically the validations and those are above `initialize()` (which you often won't explicitly have).  In any other Ruby object, the expected attributes are the parameters of `initialize()`.  That's why I want to see it first.

In comparison, the `attr_*` declarations are really just normal methods (they're not really _macros_, because Ruby doesn't have such a concept) that create normal methods.  In other words, this code:

```ruby
attr_reader :var
```

is just a shortcut for:

```ruby
def var
  @var
end
```

All style guides agree that the latter version goes after `initialize()`, because it's just a method, so why wouldn't `attr_*` declarations?

_I hope it's useful to have me explain my thinking behind the changes I made like this.  However, if I start to annoy you at any point, please do say so!  If we, as a team, decide that any of these changes that I made is without good cause, I am happy to undo it._


The `node.exe` we've had in this repo for a while now is pretty old. Bumping us up to the latest node version, which @Andrew-Hagedorn-ZocDoc and I have been using when developing on Bundler.


@jcfr this should be resolved by #443


Added a `package.json` file to this repo. This way we can start tracking exactly what dependencies we require, and what versions we're using. This will help us down the road if we migrate to gulp or another system.


faktycznie dzieki


Oh nice! :smile:


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1138](http://teamcity.monogame.net/viewLog.html?buildId=13447&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:27:43


This is the minification task. I originally wanted to just call `uglify.minify()` directly, but it has some limitations when passing in a string of code rather than a file. Specifically that will make it harder to generate source maps down the line, as we can't specify what the original file was. The steps here are taken directly from uglify's docs.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1139](http://teamcity.monogame.net/viewLog.html?buildId=13451&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Implemented new Trace method. Update Codon Specific Parameter Trace did not work for RFP since it is done on a per codon basis, not amino acid. I changed updateCodonSpecificParameter to updateCodonSpecificParameterForAA and wrote updateCodonSpecificParameterForCodon for the RFP model.

Reorganized the GABE section of main to run.

Added mutation prior to FONSE restarts files to attempt to fix the problem with starting from restart files. Not the problem, although it needed to be add.


?? :laughing:
![gun](https://cloud.githubusercontent.com/assets/11800714/12201752/5ef38ce4-b626-11e5-98f4-504082297424.png)


I think this should be in the `org.altbeacon.beacon.service` package not the `org.altbeacon.beacon.service.scanner` package.  It doesn't have to do with scanning really, and is not used by any class in that package.


Lembrar de Eliminar a Tela automática e dar mensagem de sucesso


menda-quite-shockingly-dark ... ...


Also see d83100a


Pipelining of `nowait` operations is certainly expected ;)


Technically looks great - my only question relates to why we change the notification from the logical to the physical name here:
   alert("Bag " + bagName + " successfully transmitted " +
 +                    "to " + destDir.toString() + ".");
The design intent originally was to expose to HB users only the logical name. I realize we have had to relax that, since we need to surface errors in the destinationUrl (physical name). But I'm unclear why we would in the 'success' case.

+1


Why can't we use the dismiss behaviour of notices?


Since -0 is a nonsense (mathematically speaking), wouldn't it be better to investigate why shared->min can be set to -0 ? Just asking.


... and for Ocarina of Time, it uses the C buttons for items. In that game, I use the right stick for the c pad

What if we allowed multiple driver primitives to be mapped to a single button? That way, the right stick could be the cpad, and X and Y could also be c left and c down.

Until then, the best we can do is collect a whole bunch of button maps, and default to the most popular one.

Per-game button maps would be a cool idea. This depends on a game database though, so it'll have to wait. As long as we're adding a button map database, though, we should build a database of which buttons map to which game commands. So when you're configuring SSB64, for example, you could prompt the user for "Jump", "hit", "special", "pause" instead of "C left", "A", "B", "Start". It would be cool if we could crowd-source this data and cover most games.


@spadafour It looks like at this point, you are picking up where Kari left off last semester, right?


New classes should follow the Android convention of prefixing class members with "m", so this should be `mContext`.  This applies to other members as well.


This method would make it possible for Unicorn to work on
* CentOS 6.6
* using older **gcc** 4.4.7 20120313 (Red Hat gcc 4.4.7-16) and
* older **ld** 2.20.51.0.2-5.43.el6 20100205

I am not yet able to confirm if these same weak-symbol support works on newer versions of GCC toolchain.


Thanks for the figures @gottferdom
@stef74 What do you think? Can we drop IE9 support?


wow


@ebeshero he is editing his own unfinished file from last semester :+1:


whats goin on with this one?


created https://github.com/KnpLabs/KnpMenu/releases/tag/v2.1.1


cluster_init just updated to use {{ couchbase_server_bin_path }} throughout


Thanks, @RJP43 :+1:


Con sshpass solicionado bicho


I have been kicking around the idea of making this a legit node setup. So delete everything out of node_modules and have developers just use NPM.  Thoughts?

The only thing stopping me is whether or not that would work in a future CI world (which maybe I should just ignore and figure out at that point).


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1468 of 1468 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-70.62%25-green.svg)


Ok, added pcl::normal into the picture, blue discs, that are correct!
cyan/green is old lib, purple is new lib.
![screenshot from 2016-01-08 16 40 26](https://cloud.githubusercontent.com/assets/1950251/12201841/b8d36a36-b626-11e5-83c6-cff88a022aad.png)

I reduced the number of sampled discs to 3 (reduce caos).
pcl looks correct, other two are not.
Also i computed error between the 3 N vectors (euclidean distance)
Turns out that sometimes, for old lib some N computed are NAN (like in this case...).

Errors:
pcl-newlib -> 1.58, 0.8, 0.8
pcl-oldlib  -> 1.72, 1.54, nan


This should be debug level not error level.


зачем этот метод?


clang ain't dumb, it's smart.  assert is *never* to be used as a control flow directive, as its behavior is differet based on compilation mode.  `assert(false) != abort()`


I'd definitely like to get us to a place where we can use NPM properly in dev, CI, and prod. It'd make using node modules in the monolith/edge services a lot more realistic.


cool!


crash is fixed in 5be217ff27a1c72a8588e791d82ae0624908f0a6


Phần 4. Chọn công nghệ


if its not needed, why first like check that ```target &&
if (target && target->IsAlive())```


@aldenml saw where the real issue was :)

Thank you. Fixing it all.


Good that this will ignore deserialization errors.  So if you upgrade an app with a newer version of the library with updated classes deserialization will just fail.


interesting..


hey, what's that comment doing here? That information is in the commit message and doesn't really belong here does it?


I believe I just always picked the source part? I'm not sure I will just change back to parent.


... Oops. This `1` should have been `count`.

Issue #1074.


откуда берутся проперти? где они лежат?


Какой тип связи используешь?


Give an error ```Data Structure ERROR: No ['ROOT']['el'] element found in flex form definition.````
if using "config" as name for a field or for a sheet.


No, it seems to be a bug. It seems like the python template we downloaded was still in beta, because the new one on the website has these bugs fixed.


Renamed to `name` to reduce mixup with Tile `id`s


@samveen  I see that you matched this output with what is in the manpage,. Thanks.
Wondering if that format is something that is good for you, or maybe it does not matter..
```
HOSTS -------------------------------------------------------------------------
c910f02c05p06, c910f02c05p07
-------------------------------------------------------------------------------
```

Or should we put HOSTS: into the same line with the names?
```
-------------------------------------------------------------------------------
HOSTS: c910f02c05p06, c910f02c05p07
-------------------------------------------------------------------------------
```

if it doesn't matter, let's not change it from what you sync'ed up.


Have you run the Hipchatter's tests to ensure that nothing is broken?


looks good otherwise, assuming tests pass


Windows vanilla works


что может быть в поле айди нового ентити до того, как это поле просетали в ифе?


Karolis Spring


Do we plan to have different community formats? If not, this will probably be changed in the final version. Storing json data is only a good idea for *dynamic* fields. In records it make sense because the metadata schema will be very different from one record to another.


A flask ...


I'm not positive whey this line is needed -- I guess that if an app unbinds to the service and stops it (because the scanning is not needed anymore) it will start fresh with a new state when beacon monitoring is restarted.


Please avoid unnecessary noise in commits by whitespace changes that are not required.
Also: Indentation is 4 spaces, not with tabs.


I've read various posts on the internet saying alternately "subqueries are bad for performance," "subqueries are good for performance," and "subqueries don't impact performance". Not sure what to think, but using a subquery instead of a join here is _way_ easier to work with (portable, can be inserted just about anywhere, and doesn't have unintended consequences on the rest of the query). So I'm sticking with this approach unless there's a compelling reason not to.


I did not know that using $vars in the commit message would leave those spots blank. I guess I gotta escape those.


Is it possible to move out Flask parts from here?
The idea was to split web-related code and real functionality. Everything with "flask" in name is web-related (flask is a web framework), and this class just implements the (business) functionality...


@rogerhutchings shouldn't this be `app.jsx`?


`VersionOne`, without a space


Yea. Sry, try latest


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11031](https://teamcity.mesosphere.io/viewLog.html?buildId=102245&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:30


Why not extend copyright as "2014-2016" ? :)


Shaman Tools? Or is Incantation a fancy word?

(because the quests dialogue say 'shaman tools' atm)


Dit mag in Cultuurnet\UiTPASBeheer\Properties staan (niet PassHolders dus)


subornates = servants


This looks good, thanks!!


1st Floor = 1F


@arteam so does this imply that the next release we'll have java 8 support? Sorry, perhaps, I missed an announcement ... pretty excited about this.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11033](https://teamcity.mesosphere.io/viewLog.html?buildId=102247&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:46


holy... You're right. I work with nwjs most of the time, and I never noticed the native `Promise` :o but it works!


the function still exists in ipsec.inc and can be zapped now


Hier kan je ook gewoon naar string casten normaal gezien als je HumanReadableGender StringLiteral laat extenden


FIx #255, not #225


It's not related to shared->min as far as I can tell, it just happens to be done after the test on shared->min. The issue is more likely at line 72 in Math::round(p_val). If p_val happens to be -0.00000000872, it seems to be rounded to -0 (would maybe need some print_line()s to double check that btw).

So then the question is more why does Math::round not return 0 instead of -0, as it's the exact same thing mathematically speaking, and the latter looks weird to humans?


Well, it's a controversial issue. The header part is always better (more correct/precise) for documentation purposes. On the other hand, the source part sometimes uses a more concise name, which makes the implementation more readable. So you shouldn't blindly pick one or the other, but always consider both and think about what's the better option.


@cschneider I use Kubuntu 14.04. Can you describe what environment properties would cause it to not run the default build configuration?


May be this solution is better:
```php
if (isset($dataStructure['config']) && isset($dataStructure['config']['type']) && $parentIndex !== 'TCEforms') {
	$dataStructure = array('TCEforms' => $dataStructure);
}
```


oops, my mistake... kind of rushed changing the heading :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11029](https://teamcity.mesosphere.io/viewLog.html?buildId=102231&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:12:25


Mag StringLiteral extenden


closes #3


Fixed


nah its an else if, static analysis will complain xd


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11030](https://teamcity.mesosphere.io/viewLog.html?buildId=102244&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:06:31


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2053 of 2054 tests passed)

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


So many different changes in one commit. Not good at all...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11032](https://teamcity.mesosphere.io/viewLog.html?buildId=102246&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:19


Je zou de code hier wat kunnen vereenvoudigen door een array te gebruiken met een key EN en een key NL, en daarbinnen weer een array met een key MALE en een key FEMALE. Dan moet je in principe maar $this->readableGenders[$languageCode][$genderCode] aanroepen (wel een isset() check voorzien dat die bestaat en anders een lege string gebruiken)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1481 of 1481 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)


thanks, will do :)


does it work when you start, if there is no default value?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1184.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67688&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 1 (1 new), passed: 811, ignored: 7 Build time: 00:36:12
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Ah, sorry I had not read this before the call.  Glad we are in agreement now :)


Je kan de omzetting van genderCode naar de human-readable string hier al kunnen doen. Zekers als je StringLiteral zou extenden is dit vereist omdat je dan een $value aan de constructor van StringLiteral moet meegeven, die de uiteindelijke waarde (human-readable gender) moet voorstellen. Je moet toNative() dan ook niet overschrijven want StringLiteral implementeert dat al en returnt daar in zelf de $value.


You forgot to resolve the conflict. You must select wich part of code should be kept and delete the `<<<<<<< HEAD`, `=======` and `>>>>>>> 62acf0addbf82bd71216f8b28170a2901829d167` separators.


can you add the settings to gradle.properties


polygon !!!!


I think it's correct that there are "two" zeroes (called signed zero). It's part of the IEEE standard as well and so you get -0 after Math::round a negative value.


[_countdownBtn startWithTime:5 title:@"获取验证码" countDownTitle:@"s" mainColor:[UIColor colorWithRed:84/255.0 green:180/255.0 blue:98/255.0 alpha:1.0f] countColor:[UIColor lightGrayColor]];


111101900_MSHeatPump (Nigusse) - 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)


Well, there's always the fear of something going wrong and the dependent files (css, js) could be missing. But if all the assumptions hold true, this should be fine.


:+1: Thanks, that was bothering me.


This looks right, with two caveats:
 1. It may be hard to time since scoring may be sub-millisecond for a single data point. We may want to switch to micros.
 2. It may be expensive to keep calling start() and stop(). This will come out in profiling.


opened


Length = 5 would be plenty convincing for me.  There's nothing wrong with length = 10 except (possibly) test run time.  I leave it up to you.


hi bro aapav01 and LGaljo
i have small request
one small question why you are making this rom has pre rooted because you know everything
why because in my opinion the non rooted build has more stability with stock experience but
pre rooted roms not get that great stability like non rooted rom because
i have used lot of pre rooted roms in starting stage it is very fast after few days use it getting slow down
due to giving root permissions to unwanted stuff apps like etc

im using your rom about 2 weeks it is non rooted build i have downloaded lot of apps and games and unwanted stuffs etc i never faced any lag and slowing down problems im really impressed about that

what i want to say is after completed the rom please make one non rooted build because some people wants to use this rom for long time they chose non rooted build but some one wants to develop it so they choose pre rooted

hey devs please dont misunderstand me
you know everything which one is good for users and you are the legends of android
if i said anything wrong im very soory


"two" -> "to"


Jewel


same


You forgot to complete the sentence of what happens when there's a heavy state.


Looks good!


TeamCity pull requests :: netty [Build 1005](https://garage.netty.io/teamcity/viewLog.html?buildId=4624&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6365, ignored: 23 Build time: 01:23:30


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11034](https://teamcity.mesosphere.io/viewLog.html?buildId=102255&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:48


Ideally we'd break down the summarization time for streaming into maintenance (lines 198-213) and querying (line 220). Basically, the maintenance keeps the streaming data structures up to date, then the final call (getItemsets) actually computes the itemsets from the data structures.

Fine for now.


Likewise, length = 5 would be fine and I leave it up to you.


@mhalle

Also we need this one for slicer3 - thanks!

http://www.slicer.org/pages/Special:SlicerDownloads


Incomplete sentence?


Updated as recommended.


Same about length.


aliases are scoped by network. hostConfig maintains only the configs of the network to which the container is first connected to. and this alias represent that network alone (`--net`).
This is similar to `Links`... the `Links` we see in hostConfig is only for the network behind the `--net`.

What shall we call this instead ? Any suggestions ?


its the same as `--link`.
`docker network connect --link <container-name>:<alias> <network> <container>`


reserved for what? (and why not define a constant for it anyway, at least than noone will use it unintentionally)


Out of curiosity, should the version & readme be bumped to reflect the dependency on ICU 56? (I’m not sure that’s actually the way the lib is versioned :sweat_smile:)


почему эти проперти лежат в вебе, если относятся к базе данных?


will fix


One other useful note here would be the addition of the body braces around conditions.


A lot better now :+1:


this was the issue, thanks @aldenml


I'm thinking to make it easier on us (since it's just two of us working on it), the ember app should exist inside of yggdrasil until they get mature enough to be maintained separately. I would expect that we'll be churning on both a lot and it would just be a hassle to commit and PR in two separate repos.

As to the structure of that, no idea. Maybe generate an ember 2.x app in a new directory at the top called `client` or something. Either way we'll probably want to use all the stock ember-cli stuff for the ember server and tooling, Cross origin stuff will need to be setup to allow ember and the api to talk to each other though.

You feel up to getting that started? I'll be willing to help of course.


On the other hand, the definition of Math::round() seems to enforce the usage of the negatively-signed zero: https://github.com/godotengine/godot/blob/master/core/math/math_funcs.cpp#L136-L144
The if does not consider 0 as a "positive number" (p_val > 0), so 0 is treated as a negative number and then ```Math::round(0)``` will always return -0.


pull request #1061


This file needs to be cleaned up


I don't see using `LDFLAGS` instead of `LIB` as a semantically correct approach to solve this.


Roxtanun and the next line says Roxtaunun.

Rocktanun?


Flammable


如果你不关心对象里同时出现 setter 和 getter，你可以关闭这个规则。


the Sapper Master


I wouldn't say that these are options... (pid is definitely not an option, I think). Maybe `ProcessInfo`...
(But we are talking about two members, and the original one was more readable IMHO.)


Are you building with the normal process? I mean:
```bash
waf configure --board linux
waf
```


yep, added in latest commit


i can confirm this works with Ubuntu 15.10 using GCC 5.3.0


To answer your earlier question, `isr` contains the results. There is a prettyPrint function that we could use, or we could create an `uglyPrint` option :) https://github.com/stanford-futuredata/macrobase/blob/6d57db8f073f21b4d2f663d5a7397eb2a557ff87/src/main/java/macrobase/runtime/standalone/batch/MacroBaseBatchCommand.java#L60


Looks good!


`sort keys`


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1139](http://teamcity.monogame.net/viewLog.html?buildId=13451&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:29:46


1 or a constant?


Yaaaaay


The chief's ...


space after `,` (everywhere)


Half-burnt Journal Memo (does that work?)


A good **asynchronous** neovim alternative for syntastic is neomake (https://github.com/benekastah/neomake) if you are looking for a replacement


Wäre nicht eine `config.json` option besser?


I like the unlink rename change, can we call this "assign certificate" for symmetry?


I wonder why it does work for me here... The order is the same:
```
14:04:00 runner ['/usr/bin/g++', '-Wl,--gc-sections', 'libraries/AP_Math/benchmarks/benchmark_matrix.cpp.1.o', '-o', '/home/gjsousa/ardupilot/build/linux/libraries/AP_Math/benchmarks/benchmark_matrix.linux', '-Wl,-Bstatic', '-L.', '-lap', '-Wl,-Bdynamic', '-L/home/gjsousa/ardupilot/build/linux/gbenchmark/lib', '-lm', '-lpthread', '-lrt', '-lbenchmark']
```


It is not strictly reserve but there is the risk that one day somebody checks that the value is non 0 (true)  and this may cause a problem. This is why I kept the existing numbering starting at 1 (the value was useless in fact until now). I can define a constant like `GROUP_DISABLED_MEMBER` but some reviewer may complain that I defined a constant not used anywhere!


`sort keys`, also you can use a `,` instead of `.` in the reporters


A journal recorded with the thoughts of Medzio Diena four years ago.


The dots (...) printing out when ``-q`` is not specified does not get formatted very pretty...  Should we change this since we're in the file?

Currently, it looks like:
```
....................HOSTS -------------------------------------------------------------------------
c910f02c05p06
-------------------------------------------------------------------------------
```

Suggesting something like....
```
Processing: ....................
HOSTS -------------------------------------------------------------------------
c910f02c05p06
-------------------------------------------------------------------------------
```

If you like that, here's a patch file..
```
diff --git a/xCAT-client/bin/xdshbak b/xCAT-client/bin/xdshbak
index e384424..b241f73 100644
--- a/xCAT-client/bin/xdshbak
+++ b/xCAT-client/bin/xdshbak
@@ -101,6 +101,10 @@ if ($::opt_q)
 select(STDOUT);
 $| = 1;

+if (!($quiet)) {
+    print "Processing: ";
+}
+
 LINE:
 while (<STDIN>)
 {
@@ -158,6 +162,9 @@ while (<STDIN>)
     }
 }

+if (!($quiet)) {
+    print "\n";
+}
 #
 # If compression specified:
 # Print the lines for each set of hosts with identical output preceded
```


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/997640dc35fe963d4696ef100aa7a9783f377ebc/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


Maybe you can try with
```
def _send_request(url, data):
    if settings.CELERY_ENABLED:
        requests.post(url, json=data)
        return

    try:
        requests.post(url, json=data)
    except Exception:
          logger.error("Error sending request to slack")
```

According to http://docs.python-requests.org/en/latest/user/quickstart/#more-complicated-post-requests and https://github.com/kennethreitz/requests/blob/3d33f30436f75a5822d33b99d9db92555cac8c83/requests/models.py#L406 this method use the correct content-type in the request.


That's a good point, and yeah I think it would be best to put it at the root under `client` and let it be self contained. Not sure why I didn't think about that, and in fact was kind of why I was thinking another repo, but client would work just fine and let us stabilize it.

Are you good with me producing that on this branch? I was already thinking about playing with ember for the json-api stuff shouldn't take long to hook up what we have already, and I can clean up the elixir stuff keeping only api related files around.


If you are looking for a ctrlp alternative look at this: https://github.com/junegunn/fzf

Also works with the Silver Searcher and some other things


many to many, через табличку.


mage=wizard?


I would prefer to just remove the mknod option entirely.  It is essentially useless.


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/57ed54d2ce5335b6be2f1630bc73ab1ff8de075a/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/57ed54d2ce5335b6be2f1630bc73ab1ff8de075a/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/57ed54d2ce5335b6be2f1630bc73ab1ff8de075a/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/6ec6aba1ebdadb2c30cdcd0a6638b1142ea00d93/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


please add a space between if and (


逐步走近单例模式。。
-------------------------------
public class Singleton {
	private static Singleton uniqueInstance;   //最原始的单例，采用延迟实例化，单线程下安全
	private Singleton(){}
	public static Singleton getInstance(){
		if(uniqueInstance==null){
			uniqueInstance = new Singleton();
		}
		return uniqueInstance;
	}
        ==============================================================================
       	private static  Singleton uniqueInstance = new Singleton();//关键字也可加上final，本身是安全的
	private Singleton(){}                                                            //缺点是在类加载时就实例化，假如
	public static Singleton getInstance(){                                  //这个对象一直没被使用，会造成资源
		return uniqueInstance;                                                //的浪费.
	}
        ==============================================================================
        //加同步，多线程安全，缺点是每次调用方法都需要加同步，会造成程序执行效率降低
        private static Singleton uniqueInstance;
	private Singleton(){}
	public static synchronized Singleton getInstance(){
		if(uniqueInstance==null){
					uniqueInstance = new Singleton();
				}
		return uniqueInstance;
	}
==============================================================================
         //采用双重检查加锁，各方面都不错，但由于JDK1.4及之前版本JVM对volatile关键字的实现会导致
          双 重检查锁失效，建议JDK1.5及以后使用此单例写法。
          volatile关键字确保当uniqueInstance变量被初始化成Singleton实例时，多个线程可以正确处理
          uniqueInstance变量。

        private volatile static Singleton uniqueInstance;
	private Singleton(){}
	public static Singleton getInstance(){
		if(uniqueInstance==null){
			synchronized(Singleton.class){
				if(uniqueInstance==null){
					uniqueInstance = new Singleton();
				}
			}
		}
		return uniqueInstance;
	}
================================================================================
        另外网上也有人贴出 静态内部类 和 枚举 实现单例。代码略。。。。。。。


Once you accept this, I can add some code to add the inlier and outlier counts (first two args). It won't be super hard, you can also try if you want.


the Elementalist


This else is pretty useless.


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/825cb19aa48d80b18e51f2f925eafa306fba8f4b/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


I don't think this is a good idea to add one more unnecessary `push` and `pop` call in here. You could just push the current one into the queue, and resume the new created one directly.


What if we'd just call it `inherit_environment`? (`yes` means enable and `no` means disable).


Consider refactoring those two strings to final static fields.


Тогда давай пример данных которые отправляешь и получаешь ошибку.


Remove unused imports.


This was actually authored and committed by Luke zhang


@mnrasul yes, we flipped to using java 8 as the default - https://github.com/dropwizard/dropwizard/commit/e4380dcfa457a75ef2e323411f4f69d8dd556f6e


Please do not reformat this, it is set to stay within 80 columns.


默认情况下，`setWithoutGet` 总是被设置为  `true`


Hi Stephen,
Thanks for giving the read permission.
I created new branch and added new methods for organization membership and update user identity.  I want to raise the push my branch to remote and then raise the pull request.  I am still getting the error as below:

Prems-MacBook-Pro:zendesk-java-client prem$ git push origin  zendesk-java-client_Jan8th
remote: Permission to cloudbees/zendesk-java-client.git denied to prem-jason.
fatal: unable to access 'https://github.com/cloudbees/zendesk-java-client.git/': The requested URL returned error: 403

Please help me on this.


Your changes are included in the newly tagged v.2.1.0.

Thank you very much for your contribution!


@kalyneramon creio que essas instruções poderiam deixar de ser inline.


And, if you use `co` you can get a really neat async programming style, like:
```js
co(function *() {
 fw = filewalker(examplesDir);
 fw.on('dir', function(p) { ... })
 yield fw.walk() // wait to finish async Promise with yield!
 console.log('async traversal is all done!')
});
```


Not a very useful log message, particularly for such a large set of changes!


It seem like it would be better to go the other way around and define a negotiate_succeeded boolean set to false, and then change it to true only if negotiate was successful.
For 2 reasons:
1. fails safe this way
2. you need to assign it only once or twice so there are less code changes


I think we want a timeout on execute, not on initializer


Yeah, it’s not the most obvious thing – I often end up adding a case to the Simplifier or Typer, then realize halfway through that it should be in the other.


Shouldn't the type of `x` here be switched with the type of `x` in the abstract definition? Because a class which includes `Moo` will not always be an instance of `Child`, but a subclass of `Child` will always include `Moo`. Or am I missing something?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11035](https://teamcity.mesosphere.io/viewLog.html?buildId=102261&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:25


This is what is causing the html parse error. Get rid of this line and the line underneath. Not sure why you need this.


This method is on the class, not the instance... so you can't use `@timeout`


@biyeun @kkrampa why is multifield.css_id needed in order to display the help bubble?


Wäre nicht ein Button/Haken im UI besser, und in der `config.json` wird höchstens der Default festgelegt? Denn eigentlich ist das ja keine Entscheidung der Community, sondern der Betrachter der Karte.

Ich hatte diese Änderung einfach mal auf dem Webserver gemacht um zu gucken, wie vielen das auffällt und wie das Feedback dazu ist. @mortzu hat dann hier das Repo angelegt und das gepusht. Das war eigentlich nicht meine Absicht, da ja die Menge an Forks des Meshviewers gering bleiben sollte. Von daher ist hier nicht der Ort, sowas zu diskutieren IMO, sondern wenn das Upstream Repo.


I am wondering if we really need the renegotiate option, in what case it would useful ?
When NTLM is used it is used generally through SPNEGO so fallbak is part of the negotiation.
I think we can do away with the option and just drop the header unconditionally on mechanism failures.


@frenche do you have any opinion on this ?


You can even simplify this to:
```py
corpus = Corpus.from_file('bookexcerpts')
```


How do you handle headers.from headers.x-my-header, etc.?


Yes, good idea. I'll also fix the ampersand then... ;)


ok


You cannot click on the input because the footer is getting the handle. Remove the footer to validate this. You will need to change your css for the footer if you want to keep it.


update-copyright-year (jasondegraw) - x86_64-MacOS-10.9-clang: OK (0 of 0 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)


done .. it fixed


Hmm, since you have already use a `Box` to manage the `Coroutine`, why not use this dropping strategy:

* When the coroutine is dropping (calling the `Drop` trait), it resume itself directly in the `Drop::drop` function and call `panic!(ForceUnwind)` directly. I implemented it once in the past, it works well.

    You could resume the coroutine directly with a dummy context (empty `Coroutine`).

Then all things could be done directly by the compiler and no memory leaks will happens.

(I can't see the necessary of those `set_drop_allowed` functions.)


:heavy_plus_sign: :one:


these should refer to paths under `Godeps/`?


Executor use Thread class so it works in the same way:
https://github.com/fanhongtao/JDK/blob/master/src/java/util/concurrent/Executors.java#L556

Propably using raw Thread class is faster :stuck_out_tongue_winking_eye:


Ok, I see the problem. It's probably different objects but we can live with you solution


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11036](https://teamcity.mesosphere.io/viewLog.html?buildId=102262&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:08:56


Maybe make this an `else if`?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11037](https://teamcity.mesosphere.io/viewLog.html?buildId=102269&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 212 Build time: 00:02:35
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


we should add 'use strict' function here


Oh wow this is why I wasn't able to use `api.imply` here. Good fix.


Can we instead embed the user code as a herestring and then call `Invoke-Command` against it?

That should allow us to capture parsing errors, right?

```
C:\Users\Administrator> $code = 'Write-Host Hi, James'
C:\Users\Administrator> Invoke-Command -ScriptBlock ([ScriptBlock]::Create($code))
Hi James
```


we should add 'use strict' in head of any function.


right, I must fix it, and handle the case insensitivity of headers


当 `getWithoutSet` 设置为`true`时，如果只定义 getter 而不定义 setter，该规则将发出警告（默认为 `false`）。
当 `setWithoutGet ` 设置为`true`时，如果只定义 setter 而不定义 getter，该规则将发出警告（默认为 `true`）。


Looks like a comment didn't get removed at some point.

![image](https://cloud.githubusercontent.com/assets/138615/12202938/aee03bfc-b62c-11e5-892d-dea3c1553f10.png)


Do we want this for Tex as well?


Why do we call this many times with same arguments?


I suggest maybe the unlink flag should be designed to be changeable  from outside to avoid auto unlink unix domain socket file. It can be useful while implementing the graceful restart. The graceful restart need the new process listen on the same file and then close the old one. If the old one auto unlink the socket file, it will cause the socket is not available from the client. The graceful restart can be found here `https://github.com/facebookgo/grace`


oh you did in your next commit


Blz. É que eu estava seguindo o arquivo deinitialization.md, lá está em inglês tbm.


how do you iterate on a stream in a filter?


Okay, ich mach den config default wert okay? :P

On 01/08/2016 05:20 PM, Jan-Philipp Litza wrote:
>
> Wäre nicht ein Button/Haken im UI besser, und in der |config.json|
> wird höchstens der Default festgelegt? Denn eigentlich ist das ja
> keine Entscheidung der Community, sondern der Betrachter der Karte.
>
> Ich hatte diese Änderung einfach mal auf dem Webserver gemacht um zu
> gucken, wie vielen das auffällt und wie das Feedback dazu ist. @mortzu
> <https://github.com/mortzu> hat dann hier das Repo angelegt und das
> gepusht. Das war eigentlich nicht meine Absicht, da ja die Menge an
> Forks des Meshviewers gering bleiben sollte. Von daher ist hier nicht
> der Ort, sowas zu diskutieren IMO, sondern wenn das Upstream Repo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FreifunkBremen/meshviewer/commit/6d0143dcc25b38f4fc58090f566e3cbf1c3dfdab#commitcomment-15342988>.
>


Sounds good to me. Just make sure I can figure out the exact versions used so I can make sure my environment is consistent.


`gboolean` arguments are not the best (at the call site a `TRUE` or a `FALSE`  is not very informative, and could violate single responsibility principle or tell don't ask principle ).

I'd suggest to split the function into two separate functions (or, maybe initiate an operation mode enum... but that's more complicated then we need).

`_exec_program(const gchar *cmdline)` and
`_exec_program_with_empty_environment(const gchar *cmdline)` (`empty` -> `clean`)


We found an old proposal for the name of the option: `inherit-envvars`.


Oh, I wasn't sure. I _did_ look but overlooked `get_sample_corpora_dir()`. Updated.


I don't think it is a good solution to but this is the only one I found...

I do
````
waf configure --board linux
waf
````


Almost nobody sees Github commit comments here. We only use github as a source mirror. We do use Github for issues, though.

Please file an issue or post to the mailing list.


该规则强制一种编码风格：对于每个属性，如果定义了setter，也必须定义相应的 getter。


No consolekit support in deepin?
I haven't tried yet all these new DEs.


Makes sense, OK


OK


There it shouldn't be that harmful to leave the runlist be, as it was created on those old nodes and should be valid.


Good point I will pull this tag https://github.com/ember-cli/ember-cli/tree/v2.2.0-beta.3 and use it to gen the ember stuff. That way we don't end up with subtle differences.


that's four newlines after the import, we only want two, sorry but that needs correction.


So am I, but get a different representation.

If you checkout out latest gh-pages and run `make clean all` are there modifications to the imagery.* files?


Ficou melhor assim mesmo. Vou mudar aqui.


two newlines too much


Sure no problem,
but for now the code isn't very clean.
I'm doing this because I want to migrate a blog from wordpress.com to a self hosted ghost installation


three newlines too much


That is even better :-)


I have given up openrc for deepin for the time being - there are too many issues on top of the other already existing ones in deepin ;) :laughing:
I have no idea, yet how much all the deepin software relies on systemd.
That's a project for the future ... ... ;)


LOL - weird bug in Thus - I edited the screenshot to remove extraneous stuff, and the order of files illogically changed - selecting was fine, but dragging selected the files happened in their original position, so I grabbed a CETME-L build by HMG pic which previously occupied the spot of that Screenshot.

Anyway - the GTK3 menubar items in the actual screenshot have black text on black background when unselected.


Então, isso é algo provisório pra gente mostrar aqui pro pessoal, estamos trabalhando ainda em cima disso ;)


Thanks!


This commit is being reviewed as a part of review issue #159.


Just asked to satify my curiosity. :)
I know that cinnamon seems to have ck support.
I basically only really know qt based DE and a bit OB.


This is basically the important part.


@roystgnr,

```pre
  CXX      src/systems/libmesh_dbg_la-system_projection.lo
../src/systems/fem_context.C: In member function ‘virtual void libMesh::FEMContext::pre_fe_reinit(const libMesh::System&, const libMesh::Elem*)’:
../src/systems/fem_context.C:1586:27: error: ‘const class libMesh::DofMap’ has no member named ‘old_dof_indices’
         sys.get_dof_map().old_dof_indices (&(this->get_elem()), this->get_dof_indices());
```

This function no longer works with --disable-amr, but this has been masked for a while with other problems with that config I just fixed (--disable-amr and libmesh_override have been problematic.)

This method's foreign enough I'm not confident guessing a fix - will you take a stab at your convenience?


`return true;` here to keep validateXXX contract uniform?


111303866_ATSingleDuctReheat (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2020 of 2020 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 am still not quite sure why this is a better than an additional node state (it really is that). But don't worry too much about that I guess I am fine with it :).


I was referring to the statement `map(_validate, values)`, which will assume values is a list.

Stepping through with a debugger, I see `None` values will never make it this far, they exit above at:
```
        if value is None:
            return None
```

So this is ok.


Ah of course, I should have considered the fact that some of these changes were very likely to require a newer ReShade version too, that explains it then. :)


It´s better to use the XmlRpcDataMarshaller and work with clean Pojos instead of hiding the vector implementation.

I will change it soon


ok go ahead, I do not use those Averages but we do histo filling ourself and just use hadd


strictly speaking, we don't need to `return true` in any of these, as validation checks rely on exceptions, not on return value


Sorry, I don't understand the second clause of the sentence.   It says none should be used when using none?
Can we just say "Use profile shaderc_profile_none for GLSL verisons that do not define profiles, e.g. versions below 150."  I think that covers both directions of the implication.


remove the stacktrace printing


UI code for https://github.com/BioroboticsLab/biotracker_core/issues/52


-1 one last clarification on the blurb.


saw approval from bkerley
at https://github.com/basho/riak-ruby-client/commit/946fde9a9a03fa1c0fc26c59eb0ffa13e9b00e42


merging basho/riak-ruby-client/features/lrb/rts-797 = 946fde9a into borshop-integration-259-features/lrb/rts-797


basho/riak-ruby-client/features/lrb/rts-797 = 946fde9a merged ok, testing candidate = 3df27f7c


This was a left over from ORC's news item.


It's less code. We need the first role (crowbar_upgrade) anyway, so I was thinking of reusing it


111101900_MSHeatPump (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2021 of 2022 tests passed)

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


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/1d2185a307605284eab532fb1d157800f59a245d/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


LGTM


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/a570be4d71406b44497d573e4e2ebd9c7faab044/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


@yurloc would test scope be sufficient here?


As there are implementations of SLF4J API other shipping in SLF4J distribution or logback, we can't just declare SINGLETON as private. However, we can accommodate your Clojure logging framework by adding a public getter.

Note that the StaticLoggerBinder, StaticMarkerBinder and StaticMDCBinder classes in slf4j-api are pruned out during the build when generating slf4j-api.jar. Admittedly, it's pretty ugly. The StaticLoggerBinder class actually used is the product of the binding process. There is a StaticLoggerBinder class shipping with every slf4j implementation. So your change would need to be added for every instance of the class.

Given the above, this PR should *not* be merged.


You need to create your own github fork and push to your fork. then create
a PR from your fork and there you go

On 8 January 2016 at 16:16, prem-jason <notifications@github.com> wrote:

> Hi Stephen,
>
> Thanks for giving the read permission.
>
> I created new branch and added new methods for organization membership and
> update user identity. I want to raise the push my branch to remote and then
> raise the pull request. I am still getting the error as below:
>
> Prems-MacBook-Pro:zendesk-java-client prem$ git push origin
> zendesk-java-client_Jan8th
> remote: Permission to cloudbees/zendesk-java-client.git denied to
> prem-jason.
> fatal: unable to access '
> https://github.com/cloudbees/zendesk-java-client.git/': The requested URL
> returned error: 403
>
> Please help me on this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cloudbees/zendesk-java-client/commit/d66200c13c4721cb2d8c8e7cc533bff283d9becb#commitcomment-15342906>
> .
>

--
Stephen Connolly ∙ Software Developer & Architect ∙ CloudBees, Inc.
<http://www.cloudbees.com/>


Use a `use` statement for `\UnexpectedValueException`


dashes and underscores are equivalent in the option parsing.  I think consistently using -'s for the options is more familiar and looks better...


**Line contains following spacing inconsistencies:
- No newline at EOF.
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `pythoncheck`.* *Related to file `coalib/misc/Constants.py`, line 78.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/misc/Constants.py
+++ b/coalib/misc/Constants.py
@@ -75,4 +75,4 @@
 try:
     os.makedirs(TAGS_DIR)
 except:  # pragma: no cover
-    pass+    pass
```


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (948 of 948 tests passed)

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


Do we want to remove these yet? Or are all of these cases cleaned up in the previous commit?


It wouldn't help much, I want to still name them for their specific uses like lmc-work or lmc-tmpdir


Careful with a . in the message, that is a regular expression matched.


quasi-related, would it be possible for an admin to upload 1.0.1 to the google hosted library?


If `Bar#foo` is restricted to `Moo` then it certainly can handle `Child` (because it includes `Moo`), so it complies with the abstract def requirement. Then it can handle other types that include `Moo`, but it doesn't conflict with the above passed check.

(unless, of course, I'm missing something too)


Firstly stop spaming .. i will not update anything in open .. do u want things go private ..


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11038](https://teamcity.mesosphere.io/viewLog.html?buildId=102288&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `pythoncheck`.* *Related to file `coalib/misc/Constants.py`, line 78.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/misc/Constants.py
+++ b/coalib/misc/Constants.py
@@ -75,4 +75,4 @@
 try:
     os.makedirs(TAGS_DIR)
 except:  # pragma: no cover
-    pass
+    pass
```


You're not calling `parent::__construct()` here.


ID should probably be '1864106' because '1864107' is already used next line.


A good think for this section (which would also help improve this weakness) is section headers. Maybe try writing out headers before you start writing.


Not to be harsh but this PR shows pretty poor understanding of SLF4J binding mechanism. To reassure you, I will readily admit to authoring the said mechanism and inflicting it upon the world, a much more serious offense in itself.


Pretty pleased with how well this works!


Looks good to me!


These are places that partially instantiated terms are built


Maybe link in this problem solving skill to industry work and creative coding. Good programmers (this is covered in the book Think Like A Programmer) don't use cook books they solve problems.


ok moving this into message, allowing to remove fromMessage in builder
I think this is specific to message. At serialization time we handle only generic properties filtering. Anyway I don't see howto to filter this at serialization time.


removed, see upper


Thanks! Fixed in latest.


Should I remove the entire AFProgramDriverOptions structure and have 3 redundant fields in AFProgramDestDriver and AFProgramSourceDriver or just rename the struct?

The chosen name wasn't a good idea, sorry.


This list is specific to subwikis flavor. Would have been nicer to put the common part here and extend it in xwiki-enterprise-ui-mainwiki-all and xwiki-enterprise-ui-wiki-all.


fixes #35


ack


@mstevens83, how does this version check work? Where it is getting those requirements from?


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/436


@JanWielemaker

This is one Idea I am testing..  it does not need foreign wakeup   here it is used   https://github.com/SWI-Prolog/swipl-devel/commit/2ac5d51a25d16dcb709b4a32ee1358bd98864934#commitcomment-15343575

https://github.com/SWI-Prolog/roadmap/issues/14


hey @FlorianPalm, just a hint: you could reuse the unit-tests in the other chunking_* branches


@borshop merge


That's a good question; I took a look at other uses of `changeFileMode` when I made the change and didn't see any where we'd also need to do this.

This change is directed at a problem wherein modifying a file's access bits has the side effect of clearing its ACL mask, so we only need to do it when it's important that the ACL mask is immediately correct (masks get reset whenever project sharing settings change). This event will get picked up in Pro and handled by recomputing the mask.


111101900_MSHeatPump (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2021 of 2022 tests passed)

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


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/50c6358a69dd7b2fc5ea15cb535da45713042bd7/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/50c6358a69dd7b2fc5ea15cb535da45713042bd7/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/50c6358a69dd7b2fc5ea15cb535da45713042bd7/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


@davidtolsma If I understand correctly, I think this declaration is extraneous.  Because the directive does not include the required options, the method will not add anything to the `custom_props` array.  Said differently, I think this directive is equivalent to not including it at all, right?

I'm working on some other changes today.  If you agree, I'll handle removing the line.  Just wanted to double-check with you first.


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/865a68afa60fba58c77916f38d511f32fb401cac/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


@aberels  what's the benefit of adding this doctype and the head information?


Maybe pass `*args **kwargs` of the `klass` parameter, instead of just a stream?


드로어 담당자가 힘들어하겠군...


Ah é? Então precisamos mudar lá também :)

Valeu por avisar :+1:


we could just use `new Fraction(1)` here


brilliant.


Of course, I am probably abusing gTop incorrectly.    Sorry the commit was full of the other code .. I was just having a little fun


I used it with

```
new Promise(function (resolve, reject) {
  var files = [];
  filewalker(dir, opts)
     .on('file', function (f) {
          files.push({
              some: f.path
              other: f.size
           });
     })
     .on('done', function () {
          resolve(files);
     }).walk()
});
```

Is that not good?


Good for me then.


df


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[R/zzz-autoroxy.R:1:1:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/R/zzz-autoroxy.R#L1) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
{
^
```
[R/zzz-autoroxy.R:8:16:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/R/zzz-autoroxy.R#L8) *warning:* **Do not use absolute paths.**
```r
    if (grepl("/.*[.]Rcheck/00_pkg_src/.*$", normalizePath(getwd(),
               ^~~~~~~~~~~~
```
[R/zzz-autoroxy.R:14:1:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/R/zzz-autoroxy.R#L14) *style:* **lines should not be more than 80 characters.**
```r
        warning("Cannot load roxygen2. Package documentation will be unavailable.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/e37a30798a97e29de7de5ffdf442ab59d77472c7/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11040](https://teamcity.mesosphere.io/viewLog.html?buildId=102290&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:50


breaking pwm
#1382


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11039](https://teamcity.mesosphere.io/viewLog.html?buildId=102289&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


I'm not too optimistic, but it would make more sense only for passed pawns,
or pawns which are not opposed ?

As it is, I'm afraid this is just random noise that cancels out in most cases.


こちらのコード、pmodeが1の時に[+preview_url+]のURLに対してGETとPOSTリクエストがそれぞれ1回ずつ発生する事になります。
なのでpmodeが1の時はわざとabout;blankでウィンドウを開くように修正したのですが、何か問題がありましたでしょうか。


Ah, got it! Makes complete sense.

On Fri, Jan 8, 2016 at 11:49 AM, Jonathan <notifications@github.com> wrote:

> That's a good question; I took a look at other uses of changeFileMode
> when I made the change and didn't see any where we'd also need to do this.
>
> This change is directed at a problem wherein modifying a file's access
> bits has the side effect of clearing its ACL mask, so we only need to do it
> when it's important that the ACL mask is immediately correct (masks get
> reset whenever project sharing settings change). This event will get picked
> up in Pro and handled by recomputing the mask.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/be26806d1c82d9cd302e9c0c5c75ba5fc0451110#commitcomment-15343662>
> .
>


It seems that you misspelled "canola"!


What is the compilation issue? Has it been fixed upstream already?

It would be great to update the dependencies to the latest stable version (that is 1.6.7.1 now)


is right?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1140](http://teamcity.monogame.net/viewLog.html?buildId=13465&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


@AlbertoSONIC Why do you not add a png of advanced optionwith this file manager?


I'd prefer to see a new method here (`$from->equals($to)` like in [Money\Currency](https://github.com/mathiasverraes/money/blob/master/lib/Money/Currency.php#L50)).

Although, I actually prefer the method name `sameValueAs` from [nicolopignatelli's ValueObjectInterface](https://github.com/nicolopignatelli/valueobjects/blob/master/src/ValueObjectInterface.php).


I know


Can you add some comments here


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11041](https://teamcity.mesosphere.io/viewLog.html?buildId=102291&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:00


[R/group.R:63:9:](https://github.com/krlmlr/MakefileR/blob/9b247b060202fd632340c9b184fd0c9016dc11a0/R/group.R#L63) *style:* **Use <-, not =, for assignment.**
```r
  rules = list(...)
        ^
```
[R/group.R:74:1:](https://github.com/krlmlr/MakefileR/blob/9b247b060202fd632340c9b184fd0c9016dc11a0/R/group.R#L74) *style:* **Variable and function names should be all lowercase.**
```r
`+.MakefileR_group` <- function(x, y) c(x, y)
^~~~~~~~~~~~~~~~~~~
```
[tests/testthat.R:2:9:](https://github.com/krlmlr/MakefileR/blob/9b247b060202fd632340c9b184fd0c9016dc11a0/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(MakefileR)
        ^~~~~~~~~
```


I think `$numeratorDenominatorPair` is a better name than `$conversionValues`.


merge.to.master.


There's no need to introduce a new role for a new state. Just add the state to the element_states. In the recipe you'd just use node[:state] instead of node["crowbar_wall"].

I guess you're right about less code, though.


@dorimanx
Any idea how to deal with below build error (after iommu updates)
  LD      vmlinux.o
drivers/built-in.o: In function `.LANCHOR2':
sensors_ssc.c:(.data+0x1c940): multiple definition of `iommu_ops'
arch/arm/mm/built-in.o:arch/arm/mm/proc-v7.o:(.data+0x60): first defined here
/home/neobuddy89/android/kernels/../toolchains/android-toolchain/bin/arm-LG-linux-gnueabi-ld: Warning: size of symbol `iommu_ops' changed from 72 in arch/arm/mm/built-in.o to 88 in drivers/built-in.o
/home/neobuddy89/android/kernels/shamu/Makefile:847: recipe for target 'vmlinux' failed
make[1]: *** [vmlinux] Error 1
Makefile:133: recipe for target 'sub-make' failed
make: *** [sub-make] Error 2


Tried mine 4.9 as well as your 5.3 toolchain.. same irritating error.


Could you add a comment at either the module level or for `ObligationForest` saying a few words about the purpose and use of the data structure?


@laceylin2010 for style points, focus on appropriate indentation ;)


@laceylin2010 same indentation issue here - the best way to think about indentation is to always indent direct children and keep siblings on the same line


Very true they are all handy!  I think systemless is the future though.  Which supersu are you using?


main modification would be that `DocumentFragment` contains the path, making it possible to use an instance as a mutation input.


@laceylin2010 best practice is to hyphenate or underscore your class names - reserve camel casing for javascript variables


@laceylin2010 nice instantiation of your JobsWorked object!


I implemented IDisposable for all these types and disposed them everywhere they are used. Writing to the console in finalizers is a bad idea anyway...
What I did not check yet is if there are other places where IDisposable should be fixed. That's what I meant with "I'm not sure if I'm done with this". But I'm sure there are other places.

BTW, I rebased this branch onto master a few minutes ago, so this commit is not in the history anymore.


Really good to see Shout gaining more and more community! :)


I would add a little safety. If we already did it on the Tex side we would make sure that chef-client will stay happy after upgrading the admin node even before touching the run_list.


I'm proposing that the `LookupResultSpec` also contains the path and operation, so that it can be clearly and easily linked to a `LookupSpec` or even used to retry a new LookupSpec.


Thank you for your observations, yes I wanted to see how it perform like this, and then go with something like give 2 times the bonus for not opposed pawns and 4 times the bonus for not opposed pawns with no enemy pawns on the sides. Also I think I will move it in the pawn evaluation, as they are already flagged by type there.


The message and parameter name are reversed. (Damn you `ArgumentException`!)


use `.stdout.strip()` instead of `.stdout[:-1]`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11042](https://teamcity.mesosphere.io/viewLog.html?buildId=102294&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:14


Correction: "[_posts/2016-01-07-1-o-inicio.md] Add"...


I've thought a bit about this and I think it makes more sense to have a dedicated result type (or well defined tuple) rather that using the `Document` interface. This will allow to convey the error type, index, etc... more clearly and draws a parallel with what is done in multiLookup.


Nice catch :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11043](https://teamcity.mesosphere.io/viewLog.html?buildId=102296&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:08


Also '\\' is an escape character. So if you actually need to write .DB "Testing \x 123" you would instead need to write .DB "Testing \\x 123" just like in ANSI-C. Thus '\\<' and such skips the special command.

One last note, the .DL support is not added to structs and such yet. I figured I'd get around to this soon enough for full .DL/.LONG support. Keep in mind i'm not going to add .DD/.DWORD because none of the architectures WLA supports use 32-bit addressing. Thus I see no use in .DD support.


I vote no on this unresolved question, top-level methods and sub-document methods should be kept distinct (and `Document` and `DocumentFragment` should not be interchangeable types).


show error message to user


Of course, the communities should be properly normalized and put in the database, this is just a quick basic implementation.


nice ideas.
multiple driver primitives as first step would be great.
I'm not sure how complex this should be designed, actually I play mostly n64 multiplayer.
My most concern will be to load different maps for different players, don't know how this can be combined with per game maps.
Also while the idea to map "jump" and "special" etc I often don't know how often a certain skill is to be used and if my buttonmap makes sense before actually playing it.
Crowd sourcing this may turn this concern invalid.


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1486 of 1486 tests passed)

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


Nice work man :)

Is possible contact you by email or skype?
Thank you.


Ok no problem then :)


Tested and working. Will update PR now


Am I correct that in Python you can't document "Attributes" where thy are defined? That seems to make it more difficult to keep the documentation in sync with the implementation.


If you want to use the method in higher plotting functions you might want to set the style in the higher function or app and not for every single plot.


Whoops. Looks like my text editor did some cleanup on the spaces too.


Completely agree that Console.WriteLine shouldn't be in there. I figure we could abuse it until all of the disposable stuff was cleaned up and then get rid of all of them :)

I'm assuming that this isn't all of them, but it could be. I just haven't looked into it enough. This was on my short todo list, so this definitely helps me out :)


If you want, you could add two names: mine and yours, as both of us have worked on this and this is a good time to get recognized for your work (just my thoughts).


109626134b-AvailabilityManagerOutputUnits (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2013 of 2014 tests passed)

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


path is wrong


The exceptions may be caused by bad internet connection etc. I have slightly changed it in the 3rd commit


Pero esos campos no son **requeridos**, para que ponerles Validators?


I thought it may be good to have not valid xml even in test-files


111303866_ATSingleDuctReheat (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)


Good stuff, should this resolve [this issue](https://github.com/daid/EmptyEpsilon/issues/86) with the build or is it just better error handling?


Indentation for all comments should be the same.


Indentation


Sugestão aplicada em https://github.com/plonegovbr/portal.buildout/pull/39


FYI - I used this link to update the docs in confluence.


Why do we need a path for communities? Is it to enable community hierarchies?
I would prefer UUIDs. The UI can still show a hierarchy if needed.
That way we still have the format `http://...<resource_type>/ID/<resource_type>...`


As I understand it, it is easier to manipulate the ax-object and then return it for the use in heigher plotting functions or apps.


@mightycid can we remove this `cid` parameter? I'd like to avoid anything that looks like you're tracking our users.

We pass along the referer as DuckDuckGo so you should be able to know they came from DDG that way


Of course, thanks.


Well done! Do you like utility.py functions?


Time to rebuild /lib LKG?


Cool. Thanks.


Would be helpful to mention where `Err` and `Ok(None)` results are coming from, i.e. from selection.


fixes #345


Added support for (real) private messaging


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11044](https://teamcity.mesosphere.io/viewLog.html?buildId=102301&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:50


**Code Quality: Nitpick**

Indentation needs to be done better here.


thx ;3


Should this start at 7, end at 10?


If you want to plot a bus it is enough to pass the uid of the bus. In my opinion it is a source of error to slice over the bus_uid and the bus_type.


These will write to the seen database on a user being active or quitting


Nice Job! I was literally just cloning the repo to make these changes.


@krtek4 Is that standard? I'm so used to have `ll='ls -la'`...


Just a matter of taste :smile:


ack, for the sake of a real good commit you would remove the "for consistency" out of the shortlog and mention that in the body (it's a reason, not what you do); because the shortlog is 51 chars (one char about the "soft" maximum) however, we can accept this :) One would also think about correcting the backspace in the other commit so you'd have one backspace correction commit and one for the equal sign. However this commit is rather simple so that's ok too.


ack


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1481 of 1481 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 would fix this directly in the file as part of this PR.