I set up a twice-me-multiplayer game, simply used two copies of the game in windowed mode. However, after just a minute of play one game crashed, and than the other game became a computer player (instead of being allowed to play on versus a computer player).

My main problem would that for this and any future problem the question would be 'is this a problem with my setup or a bug in multiplayer'. Simply put, would it be useful to test like this?


Changed to "reuseSyntaxAndDeclarationsManager".


So it sounds like the work is done on the plugin code. I guess this means the plugin code should be moved to its own repo (as I think you've already done), with the appropriate submodule added to ojs-stable-2_4_6 and ojs-dev-2_4. Is that right, @beghelli? The only thing this doesn't cover is user expectations re: the ALM service, which are still a problem. @jmacgreg, your thoughts?


I think "syntax" covers declarations as well, since declarations are just synax-based.

In any case, once you add the additional indirection we talked about we'll need to distinguish between these.


Changed to a method (```GetLazyState()```).


Updated per your comment earlier.


will let gary do some styling, then add to blacklight and do a pull request


Fair enough--made it non-nested.


Justo, alterado!


*asks*


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 475](https://teamcity.labkey.org/viewLog.html?buildId=269485&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:07:27
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


The usage of underscores and spaces is inconsistent here.


Thanks!
Fixed, will send once I have more updates :+1:


This can be marked `const`, and seems like it should be (just a getter).


I can see how we ended up here, but I'm arguing that we should instead follow a more methodical approach informed by well understood and proven principles which have been shown to produce clear and maintainable code in an OO language.

We haven't really ended up with a "stateless parser", we've ended up with "some pure code in a class, and a user of that code which is required to manage the state on its behalf". That might be a good pattern in an FP setting, where we would then encapsulate the state in a monad to produce a complete abstraction, but it doesn't seem appropriate for OO.

The state hasn't been eliminated, it's been moved outside the boundary of encapsulation which the parser class offers. So now we can't test that we parse incoming bytes correctly in a unit test on the parser, we must test the integration of the parser and the connection class which uses it. And anything else which uses it (a general complaint, not one applying specifically to this instance).

So I'd certainly prefer to see:
 - the state around partially parsed buffers entirely **encapsulated** by the parser.
 - the connection not to be involved in an additional **responsibility**
 - the previous two point being followed would lead new eyes on the code to experience the **least surprise** at what they found.

Which is what you get with a pattern like the previous one you had, or this one: https://github.com/pusher/em-megabus-v2/blob/master/lib/em-megabus/marshaller.rb

I'm not arguing that the pattern you've used is terrible to use, or always bad. But that it is a bit archaic (born of the lack of data hiding possible in e.g. C), or unnecessarily esoteric (when this is a pretty standard thing to need to do and pattern to solve it).


This should be marked `override`, and probably `const` as well.


Made initialization of ```SyntaxAndDeclarationManager``` eager and introduced nested, lazy ```State``` (so that instances can be reused across calls to ```Compilation.With*``` methods).


TeamCity FluentMigrator :: FluentMigrator [Build 341](http://teamcity.codebetter.com/viewLog.html?buildId=198378&buildTypeId=bt82) is now running


Lol? Did something hacked your account, @ParticleCore?


Interesting. Thanks for sharing and I hope you get it going! Would love to test it out as well.


I would guess probably not?


I think it should point out certain things:  Is this implementation here a superset, subset, or what relative to the spec?  What are the differences?


Again, I would guess not.


I did not.  I'm pretty new to using gcc.  I mostly went by previous code and documentation to determine what was redundant (O2/O3) and what made at least some sense to me, without being a subject matter expert.

So would stripping all architecture specific flags be the right thing to do if RetroPie is passing that in?  I wasn't aware of that previously.


As all calls to the `ARSAL_Mutex_Implementation` and `ARSAL_Cond_Implementation` are enclosed in `#ifndefs HAVE_PTHREAD_H`, maybe we can declare and implement these functions in the `#else` of the `#if defined(HAVE_PTHREAD_H)`

No big deal because all this code will probably be stripped away by the linker (static & not used), but it would save a few miliseconds of build time ;)


Hmm; instead of duplicating javadocs with PolyshapeReader I suggest pointing to PolyShapeReader (or vice versa).


TeamCity FluentMigrator :: FluentMigrator [Build 341](http://teamcity.codebetter.com/viewLog.html?buildId=198378&buildTypeId=bt82) outcome was **SUCCESS**
Summary: Tests passed: 3114, ignored: 60 Build time: 00:01:34


why is advertiser optional for this one?


@ShevT  repo sync commit has been removed


http://lxr.free-electrons.com/source/include/linux/kdb.h#L159
Indeed, It might be the other one (KDB_MSGSRC_INTERNAL, it is a bit ambiguous) but that is what I understand from comments and commit message.


should this be az_advertiser?


Looks good.


This `cimport` is incorrect because the declarations in the `enc.pxd` file are automatically merged into the corresponding `enc.pyx` file anyway. The best would be to rename the `enc.pxd` file to fix the naming collision (which was unintended, I guess).

Note that this star-import breaks compilation with Cython 0.23.1. Previous Cython versions had a bug that would let this pass undetected, but did occasionally generate broken C code for it. Cython 0.23.1 now correctly implements this star-import and thus fails to assign the value of `XMLSEC_ENC_RETURN_REPLACED_NODE` which is re-imported over the existing definition here.


Mark `override`.


yes.
VS 2013 with CUDA Toolkit 6.5?
You can also use visualstudio 2013 express


I did this a month before you. U srs?


Esse nome ta muito genérico. Acho que você podia ser uma pouco mais específica.


the previous code had lots of gcc options set without any thought and most were not needed which is why I removed them.

We control architecture flags from the retropie build script which is why they are not needed in the makefile (and were removed).


It should be "to some extent", not "to some extend".


whoops, got this backwards, it should be optional for update_campaign.


Aqui idem. Ter um atributo ```flag``` numa classe pode significar qualquer coisa. Não dá pra compreender bem pra que essa flag vai servir. tenta ser mais específica aqui também. Pra quê essa flag serve?


Is there any chance to see dependencies updated to the latest version?


@liewegas XFS.cc includes `<xfs/xfs.h>` which is offered by xfslibs-dev, so it is moved to https://github.com/ceph/ceph/commit/32331ede41ef5b1dc4eb85304d2e86d7c027c75c#diff-f6374c96c3d077b8e1bb54749f4c2991L34 . does this change break the build?


Quando você faz o return em uma função ou método, o que vem depois dele é ignorado, certo? No switch/case é a mesma coisa. Você só precisa do ```break``` se não estiver retornando nada em casa um dos cases.


interesting changes....


Why not just remove the delimiters?


Boost 1.59 was released very recently. Is it really a good idea to make it the default right away (given Boost's compatibility track record)? Also, a reminder that the staging branch is for changes that are unlikely to cause lots of breakage...


Why move the include, are we not trying to keep these alphabetical these days?


Isn't there an enum for this sort of thing, like `klabels.OperatorEq` or somesuch?


Why introduce schema?


# TIY-Assignments

##JavaScript Cheatsheet

###Basic literal types:
*null-basically just a place-holder. ex:null
*Boolean-A True/False value. true/false are special keywords in JS, and don't need quotes. ex:true/false
*Number-A number. Numbers don't have quotes around them. ex:10
*String-A  string of text. To signify that the variable is a string, you should enclose it in quote marks. ex "bob"

###Special constants:
*Infinty:Goes on fo'eva
*NaN: Not a number.
*undefined: Value given t something that has a value, but in this instance, it doesn't.

###Basic Operators:
Arithmetic, e.g. +, -, /, *
Comparison, e.g. >, <=
Assignment, e.g. var, =, +=


Don't we have provider APIs for this stuff?


There is, but I personally find the equals sign more readable. I can change it if you prefer though


grr, didn't see that.  fixing!


Looks good.


@blacktrash still no luck, I have made a screencast of a debug session on my android it it helps:

FYI ```flowplayer.support.zeropreload === false``` , ```player.conf.autoplay === true```

http://youtu.be/ZYwN5IphF4c?hd=1


Yeahh! Ein kleiner aber wichtiger Schritt in die richtige Richtung.


I know it's not likely to change, but it helps future readers know how to look up the operators / expand the tests etc


How about adding just ```. " ($shorturl)"```?


sure, will fix


This wanted to be `import.org.scalajs.dom.experimental.Notification._` :)


I will change it. I saw it as lowercase but will make the compare case insensitive in case it changes in future.


Good to seeing you do the splits. :D


Not for this P/R, but I would love to see `LedgerConsensusImp::accept` broken up into smaller helper functions for readability purposes.


testing


wir leiten von www um auf ohne www - ich würd das besser wieder raus nehmen


Looks good. This is probably something we should do elsewhere too.


sollte das schon tun? aktuell kommt da eine confix-seite.

bis es geht vielleicht auf den netmon leiten?


10 GOTO 10


Is that missing from the metadata now?


Am Dienstag, den 01.09.2015, 10:40 -0700 schrieb Alexander Wunschik:
> Yeahh! Ein kleiner aber wichtiger Schritt in die richtige Richtung.

Ich finde wir haben hier noch gar keinen Grund zum Feiern. Zum einen
sind immer noch etwa 99% der Clients, die den xChange nutzen auf den
alten eingestellt (diese haben jetzt einen großen Batzen
Ausfallwahrscheinlichkeit mehr). Zum anderen haben wir jetzt eine seit
Jahren stabile zentrale Lösung gegen eine neue zentrale Lösung ersetzt.
Eigentlich keine Besserung, im Gegenteil, wir haben hier nicht die volle
Kontrolle über den DNS Server zur Subdomain, und der DNS neigt
bekanntermaßen zu Problemen.

Lass uns feiern, wenn wir das Teil durch eine ordentliche Lösung ersetzt
haben. Und dann feiern wir nochmal, wenn wir den keyXchange abschalten.

>
> —
> Reply to this email directly or view it on GitHub.
>
>


Or not really, as it will crash from the double free most likely,


Thanks Chris…That makes sense and is a better way to write it!  As for the floral white..dumb mistake!

Ts
> On Sep 1, 2015, at 8:54 AM, Christopher Beck <notifications@github.com> wrote:
>
> .links-container .navbar-link:active {
>   color: rgb(233, 50, 117);
>   background-color: white;
> }
>


I assume you mean using curly brace form of `defaultto { :present }` vs the `do ... end` for the block.  `(:present)` is a list of params, which does not work here.


Looks good.


Just making a note to make sure that all of these tags disappearing is not a Zotero issue.


Really cool. Thank you! :D


Constructor shouldn't have "void"


I'm fine with the use of `Type`. it can be const though so I'll change that.


Yup, to set the default correctly in ensurable, have to pass a block to `defaultto`


I didn't actually check the metadata--can do now--but looked at the item itself and this made sense.


We are back in JS here.


Yeah, but the header for the .c file this is in should always be last. i.e. because this is cgroup-util.c it should include cgroup-util.h last.


the series editor is indeed gone from the metadata, but the tags appear to be our issue; those should still be there.


Fixed following error : make: *** No rule to make target 'vendor/samsung/degaswifi/proprietary/bin/MarvellWirelessDaemon', needed by '/home/patrick/android/CM/cm12.1/out/target/product/degaswifi/system/bin/MarvellWirelessDaemon'.  Stop.
make: *** Waiting for unfinished jobs....


Great work! Waiting for @kbknapp approve


Can this also be accomplished with `log/spy`?


r+


Same error as before. I also see that the functions are very similar, but not the same. Why? Shouldn't this be pretty standard for all Linux variants?


@JSchaenzle thanks for the catch. Can you submit a pull request with this change? I would like to get feedback from @TBonnin.


GSL package install does not throw an error, so the installed path is probably not found.
Something like  -I/usr/path/to/libgsl is also missing AS Compiler Argument.
This maybe caused by sub user space of container vm?


Since you're using self for so long you should also use strongSelf in the block. Also, you'll need to move all self's in the block to strongSelf.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 476](https://teamcity.labkey.org/viewLog.html?buildId=269493&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:31
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


TeamCity pull requests :: netty [Build 162](https://garage.netty.io/teamcity/viewLog.html?buildId=1049&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5591, ignored: 21; resource leak detected (new) Build time: 01:11:06


TeamCity pull requests :: netty [Build 164](https://garage.netty.io/teamcity/viewLog.html?buildId=1052&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 163](https://garage.netty.io/teamcity/viewLog.html?buildId=1050&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5594, ignored: 21 Build time: 00:35:26


@zachwise this seems like a strange place to set the window's open flag, right before you're about to close it. Can you recall a reason for not doing it at the end of `windowOn`?


@MrApocalypse

Tested working as advertised.


What? That's a weird rule. Even if the belongs to the source file, it should still be self-inclusive, so alphabetical ordering is just fine. I don't see any benefit in putting it last.


Nice :y:


saw approval from MonsieurNicolas
at https://github.com/codeck/stellar-core/commit/7b8d93a497cec28a11131f68c7af0ecb8aaf3dc6


merging codeck/stellar-core/patch-1 = 7b8d93a4 into auto


codeck/stellar-core/patch-1 = 7b8d93a4 merged ok, testing candidate = 66abf9d3


that is what the first part of the test does -- it uses a different writer (GeoJSON) to write the shapes and compare that -- since GeoJSON limits the number of characters it outputs the results are consistent.  It is only with an 'equals' call that you get the rounding errors

> ShapeWriter writer = getShapeWriterForTests();
> Assert.assertEquals(writer.toString(shape), writer.toString(out));


:ok: :1234:


Change to:
````
SHAMapItem (tx.first->getTransactionID(), std::move(s)), true, false);
````
This won't help today, but I've opened https://ripplelabs.atlassian.net/browse/RIPD-1022 that will turn this into a significant (and easy) optimization.  You can go ahead and add the `move` without waiting for RIPD-1022, and RIPD-1022 can be done whether or not this change is made.  When both are done, the optimization turns on.


@sindresorhus Could you publish this to npm?


I tested the current R-patched, and it looks like the requisite commits fixing this issue have not made it in, despite it reporting an SVN revision up to date with current R-devel.

I think this implies we need something like

    .rs.haveRequiredRSvnRev(69197) && getRVersion() >= "3.2.3"

just to be absolutely sure we're catching the right version.

(In essence, if the user is running R-devel or R compiled from trunk then we want to compare SVN revision; otherwise, I think we want to compare the actual R version).


Please update the README.md in main directory with how to use yammer version.


I suggest making 'Civility at OSU' the link and dropping 'Click Here'


why is there checked in commented out code?


Ja :) Dat is een belangrijke, niet doen wat je gebruiker zegt, maar luisteren, en dan ontwerpen waar de gebruiker mee geholpen is.

Het is prima om dit soort dingen te googlen, je weet precies wat je moest doen om er achter te komen.


What was the reason for changing these from `endpoint_url=None` to `endpoint_url=mock.ANY`?


Passing falsey values didn't turn wifi off. Checking for a defined value, instead of a truthy value resolves this.


I'd like to keep the real meat of the document here:
https://github.com/locationtech/spatial4j/blob/polyshape2/FORMATS.md

rather than try to duplicate in javadoc, but I'll add something to these comments too.

I also have a work-in-progress javascript library that will take the polyshape string and draw it in leaflet


:star:


Fancee!


Fixed following error : make: *** No rule to make target 'vendor/samsung/degaswifi/proprietary/bin/fsck.exfat', needed by '/home/patrick/android/CM/cm12.1/out/target/product/degaswifi/system/bin/fsck.exfat'.  Stop.
make: *** Waiting for unfinished jobs....


Will do.


Thanks. Will do.


TeamCity pull requests :: netty [Build 164](https://garage.netty.io/teamcity/viewLog.html?buildId=1052&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3611, ignored: 12 Build time: 00:07:16
### Failed tests
```
io.netty.channel.local.LocalChannelTest.testWriteWhilePeerIsClosedReleaseObjectAndFailPromise: <no details avaliable>

```


Goeie om dit soort opruimslagen te doen!

Een volgende mooie stap zou zijn om het `Bedrag` door te geven aan de functie, ipv als global variabele op te slaan. Idem voor de `ExBTW` en `BTW` (die je toch alleen in één functie gebruikt).


This is breaking Monodroid master build.


with [Frames cabal](https://github.com/acowley/Frames/blob/master/Frames.cabal) as reference
If I `stack build --flag Frames:demos --only-configure` and then `stack ghci --main-is plot` is there a way to have `boptsFlags` set to what I had "configured"?
I'm trying to `:l demo/Plot` with the `demos` flag set?


This commit gives me something like a 15% perf boost in FEZ-FNA. C#.

Thanks to @nickgravelyn for showing us this and even writing it into FNA like a hundred years ago:

https://github.com/flibitijibibo/FNA/blob/master/src/SDL2/Input/SDL2_KeyboardUtil.cs#L22


You mean in tests... I will remove it

On Tue, Sep 1, 2015 at 10:58 AM, mdubbyap <notifications@github.com> wrote:

> why is there checked in commented out code?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lganzzzo/signalfx-java/commit/101d9579f3c05e1d1504f4cf33e215f2b1343d41#commitcomment-13005619>
> .
>


Als iemand anders (of jij over een paar maanden) deze code leest dan zie je de `replace()` zo over het hoofd. Beter kun je die op een eigen regel plaatsen. En dan zie je misschien ook sneller hoe je de cancel kan laten werken :)


Ok then, I guess I'll change it and leave it as you did, being you the only one who requested it. Maybe in the future we need to add a fourth parameter to humanize or not. ;)


Maybe trim when checking length as well?


+1, want to open a PR?


Reword to `%r is not a valid target for %r`


Reword to `%r is not a valid choice for %r`


Reword to `Attempted to play a card in %r` (zone)


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/78268564


Add parentheses around `self`


lgtm


This is optional? I am guessing by default it gives the calling players characters?


It may also be useful to add a note to the translator's hints of all presets.fields.leaf_type.* strings that it is about multiple trees, and to those of presets.fields.leaf_type_singular.* that it is about just one tree.


allocating these buckets lazily (if they are not always used, can be a win)


Autodesk_Performance (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1804 of 1805 tests passed)

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


I think when editing you don't display the name in a text field, it should just be plain text of the name just like in the old UI, i believe it is the conventional way we do it for fields a user can't edit


wil this work whit the latest v aswell?


@Betsy25 @Piter432 Do not worry, YouTube + is still being developed, I am simply cleaning the repo. Most of these files I no longer need and others simply lost relevance so they were no longer being supported.

My light OCD needs the repo clean and organized to start working on the custom language feature that I am currently working on.


I don't think you need the `+` if you put /include outside of the `{}`


yeah -- disappearing tags are an issue with the bibliontology RDF import. Not sure I quite understand what's going on, though, because they are in newItem.tags just before ```.complete()```


same comment here.


there appears to be much similar code here, could it be reduce into some helpers the build new methods?

We did something similar here: https://github.com/emberjs/ember.js/blob/master/packages/ember-metal/lib/meta.js#L34-L41


these buckets should also likely be descendents of EmptyObject


really out of my area of expertise.  I'd be happy to verify once it's in kettle build.


This seems to still be very Angular specific.


Can this just be the same TYPE as above ^ (`Scope`)


@arthurvr Done. Added you, so feel free to do so in the future.


Likely an issue with data cleanup in Zotero client. I'll take a look later today


maybe we could make that project more generic to handle all third-party deps, and have specific tasks for `gpg`, what do you think?


Thanks for solving [this issue](https://hibernate.atlassian.net/browse/HHH-10065)!

I'll test it when the new version is released.


s/t.kind/ThirdPartyResourceData in the error message


Thanks for the comments! Didn't realize anyone was actually looking at this. I was looking for some direction with this since I'm (slowly) working on getting [postgresql](https://github.com/jerizm/kong/tree/feature/postgres) support working.

This was mainly just a first step towards that goal and I wasn't really sure of the the best way to go about it.

Having a `require "kong.dao"` makes a lot of sense I can take a stab at it and maybe you can tell me what you think.

Postgres support is an entirely other beast.


thanks!


Please make sure $user_row exists before returning it. Init it with false... $user_rows might not always be true (or have value)


nice ;)


fast-forwarding master to auto = 66abf9d3


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


@wluyima where is this new "uicommons.common' dependency coming from?  It's breaking our build because when we attempt to use encounterTypeService and providerService because we aren't including it.


debug, not printf, right?


Is built


is built


How about something like how our logs work, where it prepends the class and selector name automatically?
DDMethodLogRequest(pnll, frmt...) DDLogRequest(pnll, @"[%@ %@]: %@", NSStringFromClass([self class]), NSStringFromSelector(_cmd), [NSString stringWithFormat:frmt, ##__VA_ARGS__])

Therefore, you don't have to manually track selector name like you did in this commit.


Pushed the styling updates. When doing for BL, note that this also included moving a couple of strings to the locale file.

This is a useful addition as is, but longer-term it would be nice to enhance by:

- Adding a category for entries that begin with a number (we would add "#" before "A" in the index)
- Knowing which letters have no entries (so we could show them as disabled/unlinked)
- Knowing which letter is currently being used to filter (again, so we can style to indicate to the user which letter is active)


Ouch, will fix.


I'd leave the testing of ```-Wdate-time``` as is but with the changes I suggested. The test should fail if SWIG does not understand ```-Wdate-time``` which is pretty much all that is needed.


They should be equal, I'll try and put it on utils.


I wish I'd noticed this before. I've just undone this change. Duplicating the PSGI code into this module was a terrible decision. The PSGI stuff is used by ANY backend, not just HTTP::Easy. The whole point of it being in its own module was to NOT have this kind of stuff duplicated across every backend module.


The expression here should equal to the one just one commit before.
It is said there is a weird issue that MsetDg feature always say "Hash mismatch".


Still wrong progression, need info *O*


Thanks.


is repaired


I though about that, but the comments in `presets.yaml` are generated automatically.  So I'm hoping that the key value `leaf_type` vs `leaf_type_singular` provides enough of a hint for translators.


This should actually be "~> 2.2.0" because as written this will allow future versions of ruby like 2.3 and 2.99. I'd prefer we lock to 2.2.x.


Fixed in https://github.com/wso2/carbon-commons/pull/156


OK


Ah, yes ... Then I'll comments (though they aren't as visible) as soon as it gets pushed to Transifex. When would that be?


@paydro does it LGTY?


cool thanks! it will be useful for me.
On Sep 1, 2015 21:02, "Pablo Villoslada" <notifications@github.com> wrote:

> Ok then, I guess I'll change it and leave it as you did, being you the
> only one who requested it. Maybe in the future we need to add a fourth
> parameter to humanize or not. ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Puigcerber/angular-capitalize-filter/commit/9af958f18f1c229a63909da8847bd492d2c3efeb#commitcomment-13005711>
> .
>


Good catch.


@PaulKinlan @petele Hey gents. Do you know if we happen to have any more pushes to prod planned for this week? :tophat:

The previous version of the images we exported here were getting pretty close to 1MB. Solved now but will defer sharing until we have a chance for another push :hear_no_evil: :see_no_evil: No rush :) Just wondering.


the **hook** should be the name of the module, so you probably want
```
islandora_entities_xml_form_builder_get_self_transforms
```


Transifex should pick it up tonight sometime


period at the end of this sentence!


The new option is not quite in alphabetical order.

Also, let's change the text to:
```
Warn should __DATE__ and __TIME__ be replaced, ignored as this never happens
```


also, you need to assume for different directories. Try
```
'cleanup_and_order_mads.xsl' => drupal_get_path('module', 'islandora_entities') . '/xml/cleanup_and_order_mads.xsl',


On this commit as of Tuesday, Sept 1st. Stepping back through to most recent


What are you doing in the lines 11-15 ? :neckbeard:


ImprovePackaging (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2281 of 2343 tests passed)

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


LGTM @drewbanin. Merge, run the tests, and deploy if tests are all green. Let me know when it's out.

Jenkins might be down though, so you'll need to make sure that's still running.


I think strong-service-install@2.0.0-0 was from when I did the install refactor and it just used the local version number at the time I did the link/save.


@boegel I think this patch is already applied in 2.2.31 so you can skip it...


Picky: this line should say "Implements hook_xml_form_builder_get_self_transforms()"


You can also strip all this comment out and just have
```
/**
 * Implements hook_xml_form_builder_get_self_transforms().
 */
```


credits: @dufferzafar


The {{> is not recognized when I try to render the view.


It's guaranteed to be not nil at this point because you refer to `kvp.Key` above - we'll panic before we get here.


It was reported working fine some time ago. Reversing condition solves nothing, hash values must be checked


ok...


I actually didn't understand that it was being overridden. Curly brace form would be desirable as more idiomatic ruby. :+1:


This never got built. Following the release notes link in the release gives a "Page not Found".


Just "iterable". No wishy-washy about it.


Ctl-A messed this up.


I know you are not doing these changes intentionally.
Now I realized that this is "IDE file" and not related to your project.

if it is not at all android related and you don't want to change this often.. make use of gitignore to stop tracking the file.

understand what gitignore is first: https://help.github.com/articles/ignoring-files/
and then look at this file for example - (I think this is for Idea/AndroidStudio)
https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore


Commented out test cases are features that are not supported for
yammer.metrics 2.0.0
those test cannot be re-enabled unfortunately

On Tue, Sep 1, 2015 at 11:12 AM, mdubbyap <notifications@github.com> wrote:

> Yes, can those tests be re-enabled? Looking at your code coverage, it's
> 49.8%, that's pretty low. I would like to see more like 80% or better code
> coverage. Thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lganzzzo/signalfx-java/commit/101d9579f3c05e1d1504f4cf33e215f2b1343d41#commitcomment-13005915>
> .
>


No Desplega las recomendaciones.


Create new issue with a short code example that reproduces the bug, ping me there.


thanks @whikloj ... I think I've got the changes made that you've suggested.


good catch, this was a holdover from when this function still supported nil arguments. will remove


LGTM


nit: What is the rationale for this argument?


You need to run `go fmt` on the files you write. Most decent text editors will do that for oyu.


nit: Missing Python 3 shebang and copyright.


nit: This feels like a constant.


Thanks for your observation


I think it would be better handled by having a gradle plugin e.g. com.android.model.library  and have 2 types of models supported - one with source code that is being compiled into a lib. and one that wraps a lib and headers.

Is the team working on supporting native lib projects?


@ParticleCore: Oh, thank god. You've scared me and probably other users, too :) Don't do that next time, please :)


`if buying`.


nit: Could you use a where clause here?


Sorry, maybe I should have labeled it "Obsolete" or something similar instead of "Discontinued".


I think that the way it's configured now should be fine. It looks like something else is more significantly broken. A snippet from the page source for the homepage:

```
<title> mickens · mickens </title>

  <link rel="stylesheet" href="
<body>

<div class="sidebar">
```

Which is causing obvious problems. I'll fix that and see if this still needs to be resolved, but normally the "//hostname.tld" convention will cause the browser to use the same protocol to access resources specified in that manner.


Ah yes, the spacing from me writing this in GitHub's web based interface - I'd been planning on sorting that out!


Well ezio I think it is good this way. I ported the to lp for the ones who say i didnt, but that is another story. If you want ezio I can tell you  lthe story,it is long


Why is this deprecated? A `FromStr` impl could still be useful...


In the case where we cannot create the typeface from the asset, I would suggest that instead of swallowing the exception and returning null, we should either just not catch the exception, or throw our own exception with a descriptive message such as "Invalid typeface asset".


OK, then please try to write new tests that are supported  thanks.  To run
the code coverage i used clover.  See that here.
https://confluence.atlassian.com/display/CLOVER/Basic+usage


@whikloj, @DiegoPino y'all are awesome. thanks for jumping in here so quick and helping out @dmoses!


not sure, not terribly familiar with how licenses work


Thanks! But there is a reason for it, and that's because I'm preparing stuff for tomorrow when the release is officially going out. So the page will be there tomorrow :)


Good grief, I honestly just don't care about this NONSENSE anymore but looking at the commits it's clear the only thing "sira" did was create a shitload more work and issues with his "axxion improvements" it broke almost everything it tried to fix causing several hours more work NOT done by him. Again I REALLY don't care anymore, so much nonsense on git these days make me just wanna roll out but there is a clear difference between his end result and ours and that's with us helping him out not too mention his complete lack of correct authorship and being a smartass while doing so and thinking it's funny. If you need input from me just message me via email or hangouts because once that twerp starts, and he will I will no longer be following this conversation


s/exercize/exorcize/ =P

But seriously, s/exercize/exercise/?


nit: `if let`


As I understand it this will return both pods and nodes. Do we foresee any place we would want both? I think in many places we will just want one or the other.


EDAPT_Issues (macumber) - Win64-Windows-7-VisualStudio-12: OK (2306 of 2343 tests passed)

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


ImprovePackaging (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


> Is the team working on supporting native lib projects?
yes.

> (or maybe I don't understand your vision) :-) ?

I was thinking that while we wait for proper native lib support, we could advise developer to have a single gradle project called `third-party` to handle fetching their third party libraries.

One of those library could be `gpg`, but if I was adding another lib I would add it in the same project rather than create a new `third-party-foo` one.

What do you think?


Im honest and i say the truth, you man said you took "as reference" my commits but you used my work on the tint, materialize and the statusbar hide feature. The things that you did has your name, I dont know why you want more, the guy (ezio) picked my port , if you did another different port (y) congrats but he picked mine, the things that you did in this commit (not my original one, it is squashed, has your name) and it should be same way if they case is inverse, but it isnt since you keep saying you take references but no credits... That is all


changes made


Just make it call new method instead of copying code.


Some resource types, like `Log`, don't have an `ensure` - and calling `resource.ensure_value` will throw.  However, our generic provider uses `create` to actually execute the code to apply the resource, which is why we have to fallback here to `'present'`.


You can't just break API.


非常好的同步工具啊。


@spegelius A bit late, but I'm running into this issue right now.
You've remove the macloader service here, but as far as I can tell there is no other place where the macloader service is defined, so it doesn't get started anymore which means there won't be a new `/data/.cid.info` file created, which makes wlan fail in my case.

Was removing the macloader service a deliberate move?


Same.


:clap:


yeah at the moment it returns everything under both label-able types. In terms of code it would be trivial to filter to one tree or the other. As for whether returning both is ever useful - my first guess is no. Want a Type argument to this function to only return results of that type?


@dt-ucsd I don't think we want this hardcoded, do we? shouldn't the URLs here be relative?


Autodesk_Performance (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (755 of 755 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-8.89%25-green.svg)


`'genome.wustl.edu'` could be `Genome::Config::get('email_domain')`.


@rapperskull apparently git rm does not work if the directory above it does not exist. What do you think is the best approach, having also that information from stackoverflow?


and also, please, report a failed send action


I will continue this as long you want cause I am right. I see your moods about I am 14 and i do your same work, just a thought. And who was the ones that are private and blocked me? You guys. You cant talk having your source closed please some coherence


returning an object is fine here


Very good progress overall.
Note that Android (with Java) is a very powerful statically typed laguage and we should make sure of its advantage by using appropriate types instead of list_of_primities, List, Strings, Hashses( maps) when they don't make sense.

Do this for all interface methods: (and hence also to implementations)
1. Never use an Implementation unless you clearly specify - HashMap<> is an implementation - Map is an interface, why not just return Map<String, String> (DONT do this here)
2. GetLatest() Appears to be latest Currency - Why not return a currency object instead?
 -- I couldn't suggest more because I don't know what are you expecting here).

Mainly - Documentation is very important - For example I don't know anything at all about this app, but want to provide an implementation for CurrencyDao - What all is required (expectations/suggested-behavior) should be clearly documented so that I can easily provide a new Implementation

similarly correct for all History() methods - and add proper documentation for all interfaces and methods.


Ditto for "foolscap client connection hint string". Users are likely to ask: what are these; why do I need two of them and how are they different?


don't need this


101884090--NewVenting-Availability-Schedule (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1814 of 1814 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)


getExchangeRates -> we should add a check-style for this. (try to search and add so that the IDE won't let you make a mistake.


Hm, that is strange because the difference of http://flowplayer.blacktrash.org/flowplayer-mpegdash/flowplayer.mpegdash.js to the previous version is:
```diff
diff --git a/flowplayer.mpegdash.js b/flowplayer.mpegdash.js
index 2a098ab..ecebd8d 100644
--- a/flowplayer.mpegdash.js
+++ b/flowplayer.mpegdash.js
@@ -105,25 +105,16 @@
                         common.prepend(common.find(".fp-player", root)[0], videoTag);

                         mediaPlayer = new MediaPlayer(context);
-                        mediaPlayer.setAutoPlay(player.conf.autoplay);
+                        mediaPlayer.setAutoPlay(!flowplayer.support.zeropreload && player.conf.autoplay); // handled by fp API
                         mediaPlayer.setScheduleWhilePaused(true);
                         mediaPlayer.startup();
                         mediaPlayer.attachView(videoTag);
                         mediaPlayer.attachSource(video.src);

                         if (player.conf.autoplay) {
-                            player.on("beforeseek", function (e, api, pos) {
-                                // when seeking to unbuffered positions
-                                // dash.js resumes
-                                if (api.paused && pos > api.video.buffer) {
-                                    bean.one(videoTag, "play", function () {
-                                        videoTag.pause();
-                                    });
-                                }
-                            });
-
-                            // https://github.com/flowplayer/flowplayer/issues/910
-                            // Android and Win Firefox
+                            // let the fp API take care of autoplay
+                            // otherwise dash.js triggers play when seeking to
+                            // unbuffered positions
                             videoTag.play();
                         }
                     },
(
```
i.e., mediaPlayer.setAutoPlay() is false now, except for your Android case where it is still `true` like in the previous version which worked. Are you 100% certain you purged the cache (on iOS I to sometimes reset the device to _really_ purge)?


Yes, +1 to a type argument.


Also about the improvements, it is subjective, but indeed it was a good work and i did it that is why im happy and holly proud


`sol_util_fill_buffer()` takes care of ensuring buffer size already.


Yeah, I had that in mind but I was rushing. :smile:


I prefer to have 1 project per dependency vs. 1 project per process.


Actually it might be better to describe these as "tub.listen" and "tub.location" strings, to more clearly link to the config parameters (and to where users should look for their descriptions in the docs).


@mvl22 any idea why this hasn't gone to the server?  Is the deploy script down?


we should not need these objects unless you are caching them (if caching you are probably doing it wrong - so remove). please remove and try to get the same functionality without these variables. Remember about immutable variables. - what happen when multiple threads call getLatest() - they will corrupt the jsonArrayData, currencyData, etc.
1. currencyDao
2. mContext
3. jsonArrayData


We run hive as the hive user and presto as root. Looks like the issue is the write to /user/hive/warehouse:
presto:default> CREATE TABLE nation_hive AS SELECT * FROM tpch.tiny.nation;

Query 20150901_174721_00004_b6ivj, FAILED, 4 nodes
Splits: 10 total, 9 done (90.00%)
0:07 [25 rows, 0B] [3 rows/s, 0B/s]

Query 20150901_174721_00004_b6ivj failed: Failed to rename hdfs://master/tmp/presto-root/79f16b4f-3388-4f97-a083-aa1b2544d7f3 to hdfs://master/apps/hive/warehouse/nation_hive

And in the server log:
com.facebook.presto.spi.PrestoException: Failed to rename hdfs://master/tmp/presto-root/79f16b4f-3388-4f97-a083-aa1b2544d7f3 to hdfs://master/apps/hive/warehouse/nation_hive
        at com.facebook.presto.hive.HiveWriteUtils.renameDirectory(HiveWriteUtils.java:358)
...
Caused by: org.apache.hadoop.security.AccessControlException: Permission denied: user=root, access=WRITE, inode="/apps/hive/warehouse":hive:hdfs:drwxrwxr-x
 at org.apache.hadoop.hdfs.server.namenode.DefaultAuthorizationProvider.checkFsPermission(DefaultAuthorizationProvider.java:257)
...
The directory in HDFS:
[root@cwallin-vsphere-presto-cwallin-1-master prestoadmin]# sudo -u hdfs hadoop fs -ls /tmp/
Found 2 items
drwx-wx-wx   - hive supergroup          0 2015-08-31 13:56 /tmp/hive
drwxr-xr-x   - root supergroup          0 2015-09-01 10:47 /tmp/presto-root

[root@cwallin-vsphere-presto-cwallin-1-master prestoadmin]# sudo -u hdfs hadoop fs -ls /apps/hive/
Found 1 items
drwxrwxr-x   - hive hdfs          0 2015-09-01 10:47 /apps/hive/warehouse

Thanks for the detailed description of the problems involved. I'm also not sure about the best way to write to the database location (/user/hive/warehouse); I was reading that Cloudera recommends mode 1777 for /user/hive/warehouse, but that's not the way our automation configures hive. I can look into this more.


lets do it as a separate commit / task

On Tue, Sep 1, 2015 at 11:36 AM, mdubbyap <notifications@github.com> wrote:

> OK, then please try to write new tests that are supported thanks. To run
> the code coverage i used clover. See that here.
> https://confluence.atlassian.com/display/CLOVER/Basic+usage
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lganzzzo/signalfx-java/commit/101d9579f3c05e1d1504f4cf33e215f2b1343d41#commitcomment-13006394>
> .
>


This is temporary and works fine for samples containing no twins, what to do for twins?


Hmmm - is it possible that you could have a number of would-be queue borrowers that had all acquired the borrow semaphore but that had not yet reduced the lock semaphore yet, leading to the `lockSemaphore.acquire` call succeeding here but borrowers still getting through and emptying out the queue?  The hope instead was that if the `lockSemaphore.acquire` call is successful here that all of the instances have been returned to the queue already, right?


I'm fine leaving it around if you prefer, it just seemed a bit weird to have a `FromStr` impl that can never fail, but you still have to unwrap.


s/demnd/demand/


great job


Unrelated to this patch, but we should calculate ``b32encode(self.nodeid).lower()`` only once.


Fixed following error: make: *** No rule to make target 'vendor/samsung/degaswifi/proprietary/lib/hw/consumerir.mrvl.so', needed by '/home/patrick/android/CM/cm12.1/out/target/product/degaswifi/system/lib/hw/consumerir.mrvl.so'.  Stop.


You can avoid some allocations and copying by creating buffer having `p` as its backing store. It would be something like:

     struct sol_buffer buffer = SOL_BUFFER_INIT_FLAGS(p, sizeof(*p), SOL_BUFFER_FLAGS_MEMORY_NOT_OWNED);

Then no need for `memcpy()` or even `sol_buffer_fini()`.


TeamCity pull requests :: netty [Build 165](https://garage.netty.io/teamcity/viewLog.html?buildId=1063&buildTypeId=pulls_netty&guest=1) is now running


Never catch an exception - And always log for info:
See this for what you are trying to swallow.
http://www.programcreek.com/wp-content/uploads/2009/02/Exception-Hierarchy-Diagram.jpeg


I am pretty sure system headers should not be influenced by local headers, except for config.h.


Okay :)


Variable don't have name and that causes errors


Minor thing but I've never seen anyhting like that in Phobos. Put each statement on its own line?


One way to get around this is using external tables, but since you have to add partitions manually, that doesn't seem like the best option.


Ack, I'll test.


Esp since here they are here


should we warn here?


Noooo :)

I have an extra buildType named "internal". And I only publish that buildType to Play Store and I loved that the files are in the "internal" folder.

I am sure that no one wants multiple folders with the same bootstrapped store files. They will be duplicate. I think neither of solutions are good. What about putting them in a folder with a more general name without using flavor and buildTypes at all.


Ok, so I updated the official release date to today :)

I'll just hold off on some announcements until tomorrow.


sounds reasonable yeah, will push a commit adding that


Might be a good idea in terms of DRYing up the code, the same could be said for local storage. Just a note that your function name is spelled incorrectly `isSessionSorage` should be `isSessionStorage`.


Where did 18600 come from?


Sure, but we anyway split system from local headers. So my question was really just about the order in each block.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 477](https://teamcity.labkey.org/viewLog.html?buildId=269507&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:09
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


@anatol Please submit a pull request to the master branch.


TeamCity pull requests :: netty [Build 166](https://garage.netty.io/teamcity/viewLog.html?buildId=1064&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 165](https://garage.netty.io/teamcity/viewLog.html?buildId=1063&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 3783, ignored: 12; exit code 137 (new)) Build time: 00:04:48


This fix was started by Troester - I reported no joy, no success in triggering calc behavior, but it may be related to the "undefined" result in another calc.  Did you get button groups to work as triggers?  Please see / reply in forum thread http://fabrikar.com/forums/index.php?threads/calc-works-in-php-submit-but-not-in-ajax.41963/


Included in 46d78d92252c3cbb8a5364c49eb74f191a7c2684


What do you mean by spurious here? The untyped `define-values/invoke-unit/infer` form introduces a number of bindings, most of which a user will not be able to reference.

Ideally, I think there should be a better way to handle the `define-values/invoke-unit` forms, but their expansion is extremely complex so I haven't been able to do much better than the current implementation.


TeamCity pull requests :: netty [Build 167](https://garage.netty.io/teamcity/viewLog.html?buildId=1065&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 166](https://garage.netty.io/teamcity/viewLog.html?buildId=1064&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 385; exit code 137 (new)) Build time: 00:02:40


In fact as you say that proof is there. See this https://github.com/AICP/frameworks_base/commit/917729c19770e305337611e783d41be7bc2daa7b
First public commit of "your" floating window work (Saturday 27 june)

AND:
https://plus.google.com/+SiraCuervo/posts/84of4eNtG8a

Oh!? 6 days before! How could I steal their work if it is closed? Come on guys this is becoming funny stop lying... "Reference" lol


never depend on an implementation - just rely on interface:
`private final CurrenyDao currencyDao`
when you are creating activity - use
`new QuandlCurrencyDao();`
so when you have an XYZCurrencyDao implementation, only thing that will change is:
`new QuandlCurrencyDao() --> new XYZCurrencyDao()`

When you are using Dagger-2 to inject these dependencies -- You don't have change your activity at all. Just the dagger module and everything works as expected.


You can combine all those into a single UPDATE statement


how is this value set? - this is null and is used inside the code - WILL cause Null Pointer Exceptions (NPE)


Thanks. I corrected the error


Done, thanks


Hej Leon, nie musisz dla każdej zmiennej z E-Prime'a robić pole w exp, to byłoby uciążliwe. Najwygodniej gdyby mieć w exp tylko te parametry, które są stałe przez czas trwania całego eksperymentu.


ArrayList<BarEntry> price = new ArrayList<>();
should be
List<BarEntry> price = new ArrayList<>();
Please use interfaces - say you want to replace with LinkedList<>() later.. change in only one place..


You'll want to set it back to its original setting to avoid polluting the test environment:
```Typescript
let originalSetting = Plottable.Configs.ADD_TITLE_ELEMENTS;
Plottable.Configs.ADD_TITLE_ELEMENTS = false;
legend.renderTo(svg);
Plottable.Configs.ADD_TITLE_ELEMENTS = originalSetting; // do this before asserts so it will happen even if the test fails

... asserts ...

```


Could we adjust the indenting for the first line?  Otherwise, :+1:


Do we still want to make sure that errors from ``startService`` get into twistd.log and/or cause tests to fail (by calling ``log.err``)? I imagine that tests that create a node will fail anyway if ``startService`` raises an exception, but that isn't guaranteed.


Whoops.  You should just drop this one, it wasn't supposed to go upstream.


'D' is already defined as the 5th short opt, what's this supposed to fix?


To byłoby pewnie wygodniej mieć w pandas DataFrame zamiast cały czas podmieniać i pobierać z exp. W ten sposób na koniec eksperymentu (albo co blok itp) można by zapisywać dataframe do csv albo xls.


Before I press the button "merge", I really really want to know why this should work better with boost_log. Why introduce a new dependency?


Ah! good catch, but this should be the correct configuration, all services need to actually include uicommons.common to inject the necessary global interceptors and configurations that cut across all apps e.g authentication and setting no cache headers. If you want to fix the broken apps on pages you need to include angular-commons.js from the uicommons module


:+1: for adding these to the proposal.


my bad. yeah, `m` could be changed at https://github.com/ceph/ceph/commit/d015d23d9103b4edad9213a0bf6780209289dc48#diff-72747d40a424e7b5404366b557ff12a3R3935 .


Why the type hint here?


When will we create 2 PRs to Gradle:
- `project.getOrDefault(...)`
- `project.getPropOrSysEnv(...)`


refs #7


FWIW, I've backported this to Rails 4.2.3 and am sharing since it was non-trivial https://gist.github.com/bf4/84cff9cc6ac8489d769e


Yep, already did, there were only a couple.  Something to watch for going forward though.


This looks fishy — I'm surprised it didn't break anything else. A more appropriate solution would be to duplicate the states properly, but then we'd end up with three states named 1266, which wouldn't compile anymore. Hmpf. I guess your solution is the pragmatic best. Is this from Gavin btw?


Would you mind submitting a pull request to fix this?


took * -_-


is it possible to nest the project under a `third-party` directory then?

I found the layout clearer:
```
build.gradle
app/
app/build.gradle
third-party/
third-party/gpg-sdk/build.gradle
```


These come from upstream, I'm inclined to leave them as is.


Could the `target` be empty?


I think it's the radius of HSC, in pixels.


tabs are introduced throughout here.


well, it's clearly not, otherwise the patching would fail...

maybe it'll be fixed in the next version?


01 << 63


test it


wouldn't the correct thing be to produce two more OR blocks?


Thanks!


This is slightly misleading because an indexed `orderBy` is parallelized in a sense (it actually just changes how the results from the shard are combined). Should this example use a non-indexed `orderBy` to make the difference clearer?

"`order_by` with indexes" is also mentioned in the list of terms that can be distributed below in line 56.


Great, thanks! :)


`std::move(s)`


@mndvns semi-colon does not belong here


Why set `_fields_` in a separate statement for `struct_ifaddrs`, unlike the other Structures/Unions?


Should this be `filter(person=user, event=self)`?


Agreed.  I've gone for the nuclear option where we scan all the files we can find.  We currently have 55 failures - a bunch from the inability of `bluebutton.js` to identify an xml file without the file declaration, others down to syntax mismatches (dates mostly) which we can attempt to mitigate.  The intent is to have the framework, to be able to make the code more robust


Not sure, but the "is_copy" name feels wrong if it is `False` *after* the `copy()`


fyi, awesome post!
probably be one of my go to reference, when i re-explore shiny.
also, one thing befuddled me, if this `post` is on the github project pages, then why jekyll?


This should be typecast to `(void)` to avoid compiler warnings on older kernels.


We have no idea what a const LedgerMaster would be.


Sorry, not interested. I received a report that the latest Cython release fails to compile this code, and the bug was obvious enough, so I just thought I'd mention it here.


Done, thanks


This is not exactly right. `reduce`, `max`, `min` (there's also a typo there, it says "mix") and `avg` will be parallelized (you also list `reduce` correctly in line 53).
However they stop *subsequent* chained terms to not be parallelized anymore. This is because these terms require "combining" the results from the shards on the query server before they can yield an results themselves. Hence other chained commands will be run on the query server (just as you explain below).


Apparently PEP8 says no....


I agree. What if we add "tests/" or a "unittests/" directory for things like this and integrate in a "make check" call?


Having the Browse Results button under the Models label throws off the alignment of the Datasets and Models lists.  Is there any reason to not tuck the Browse Results button under the Images dropdown?  Or maybe the Models label could be turned into a link to the Results Browser.  The title could then be changed to Models or something more fitting.

That would also make the url 0.0.0.0:5000/models more consistent.


I can downgrade dash.js for testing on the off-chance that something new is interfering, otherwise I'm baffled.


chanel ;)


Mejoras en la vista del mapa


This query actually doesn't work, because an `orderBy` after a `filter` cannot be indexed. Should this example use a non-indexed `orderBy`?


lol


One suggestion: to be paranoid you might want to add tests that exercise the actual overflow part happening around 31st Dec / 1st Jan (doing both a `plus` and a `minus` scenario), and check the years are maintained correctly.  Ignore me if you already have tests covering that elsewhere...


Is the intent that `getCurrentLedger()` should modify the `LedgerMaster`?  If so, don't mark this `const`, else mark it `const`.  The reader of the code needs to know if the function (conceptually) changes the state of `*this`.


But the link is less obvious, I guess.


Verify file can be written to before adding to files list


Nice. I'd still have gone with the dictionary with int keys (in case your enum values aren't sequentially valued starting at 0 as the array assumes), but either way will be better than enum keys in a dictionary.


Hmm, the whole commit is based on the jflte device tree, so the removal has
probably happened on that tree. If the macloader is needed, i don't see no
reason not to re-add it again. Then again, the macloader binary doesn't
seem to be part of the blobs? It would need to be added too...

On Tue, Sep 1, 2015 at 9:40 PM Simon van der Veldt <notifications@github.com>
wrote:

> @spegelius <https://github.com/spegelius> A bit late, but I'm running
> into this issue right now.
> You've remove the macloader service here, but as far as I can tell there
> is no other place where the macloader service is defined, so it doesn't get
> started anymore which means there won't be a new /data/.cid.info file
> created, which makes wlan fail in my case.
>
> Was removing the macloader service a deliberate move?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CyanogenMod/android_device_samsung_jf-common/commit/76f88178bd156a48e508be1db24828dcc8c5694c#commitcomment-13006481>
> .
>


i like that - done :+1:


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 478](https://teamcity.labkey.org/viewLog.html?buildId=269511&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:41
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


It is part of an attempt to decouple the different behaviors from the main purpose of a given test case. Similar changes are also introduced in latest commit for test_client_with_endpoint_url() and test_client_with_response_parser_factory(). This way, we won't need to change a dozen of existing test cases when we add yet another new parameter in future.

In that sense, I should probably have change this test case more completely, to just self.assertTrue(kwargs['is_secure']==False and kwargs['verify']==False), AND at the same also add another dedicate test case for test_client_without_endpoint_url().

Will change the PR soon.


isn't this the code we want?  It's showing up as the old code in this diff.


this should probably become a `feat._text.split(' ').every()` call instead but you get the idea


No idea. I'm still on version 1.14.1 of WeeWX, so I can only confirm that it works up to that version. It doesn't seem likely since WeeWX is now two major revisions ahead. Looking at its changelog, it looks like Tom has made it easier to add extensions, so I have the feeling it's probably easier to add this kind of code into the mix than it was when I created addon. I don't know how soon I might be able to everything upgraded to current and then look into it. (but it's on my list to do)


Cool, thanks!


Sure would do so :)

On Mon, Aug 31, 2015 at 12:24 PM, Daniel Doubrovkine (dB.) @dblockdotorg <
notifications@github.com> wrote:

> Would you be so kind to PR your changes with tests upstream?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nukturnal/spidey/commit/7ef38c82ec5b1c11c98a4f0b3bd06237167344da#commitcomment-12974799>
> .
>


Autodesk_Performance (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1804 of 1805 tests passed)

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


fix #300


Please avoid variable names that shadow bindings from the standard library. If someone misses this line, the next line looks like a tautology.


Using `[0]` seems to be an implemetation detail not specified in the API docs: https://docs.python.org/3.5/library/gc.html#gc.get_referrers

Also:
> Avoid using get_referrers() for any purpose other than debugging.

:-) just saying, actually a nice solution to get kind of R-like delayed code execution...


This line is a bit of a cheat, maybe? We're a computer system, we can tell the difference between no invitations being sent and all of them having been accepted!

At the very least, I can see organizers wanting some way of remembering who they've invited to help troubleshoot the inevitable "I never received/lost/broke it" responses. And also to avoid sending multiple emails to the same person.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 9](https://teamcity.labkey.org/viewLog.html?buildId=269509&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 2 (1 new), passed: 12 Build time: 00:26:11
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

```


LGTM


I'm also wondering if the use of an external library for math would make sense here


I am very sorry :(

I feared that something like this could happen. But for now I'd rather go with this more predictable solution: The buildType is not used to determine the bootstrap target, whether or not the project uses flavors.

I hope this change does not break your build process. Do you run the bootstrap task more than once?


Okay, so the problem with the code above is that the "}" that closes out your large if-statement is in the wrong place. Right now, you've got the 'else' part of it inside of the 'if' statement's body. See my above comment for where it should be.


This is where you want to place your closing '}'


Srclocs should be fine.


provide->toplevel: Cannot find test-func in #s((prefix zo 0) 0
(test-func-2) () insp0)

  context...:

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/nodep.rkt:136:2

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:74:0:
compute-new-modvar

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:89:0:
filter-rewritable-module-variable?

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:141:0:
merge-module

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:36:0:
merge-forms

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:47:0:
merge-form

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/merge.rkt:15:0:
merge-compilation-top

/Users/blake/devel/racket/racket/collects/racket/contract/private/arrow-val-first.rkt:300:18

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/main.rkt:22:0:
demodularize4

/Users/blake/devel/racket/extra-pkgs/compiler-lib/compiler-lib/compiler/demodularizer/batch.rkt:
[running body]

   /Users/blake/devel/racket/racket/collects/raco/raco.rkt: [running body]

   /Users/blake/devel/racket/racket/collects/raco/main.rkt: [running body]

On Tue, Sep 1, 2015 at 10:51 AM Jay McCarthy <notifications@github.com>
wrote:

> Either this or m should record the phase for the higher phase errors.
>
> Can you post the error you get from the phase=2 test?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bdj/racket-compiler/commit/842cac7c154a44c328c892396248734c6cdb5a07#commitcomment-13004189>
> .
>


Thanks @davidanthoff, I pushed the MH bug fix on metadata, so you should be able to use Metropolis Hastings without problems for now on the released version. Let me know if there are still any problems with it.


Because as I understand the libusb documentation, this call only makes sense when polling the FDs directly: http://libusb.sourceforge.net/api-1.0/group__poll.html#ga5bc6f5bc71a43845244912da6679f634

Of course, the documentation could be wrong or misleading, in which case I'd add it again. But it could also be that this only hid a problem; as the old code essentially timed out immediately all the time.


Also, this is unused.


Missing newline here.


Bro, this is wrong when combined with line 120!


Hi!

unfortunately Chris, the original author, is MIA. We've started a fork at https://github.com/zimbatm/vagrant-nixbox/ (currently 404 as we're about to rename it). We've manually transferred your change over there. Thanks for the docs!


I don't think these comments are necessary. Putting them in the commit message is good enough.


Seems to have fixed it - thanks.

Neither cyclescape3/cyclescape5 have auto-rollout at the moment - I do this manually. May restore this soon.


Hello @rakuishi,

Thank you for your contribution!
However, for optimal user experience, could you please take care of the following issues?

 1. The partial `author.html` is referenced in `layouts/index.html` and in `layouts/_default/list.html` but not included, leading to error messages like these:

    ```
ERROR: 2015/09/01 Unable to render [partials/author.html theme/partials/author.html]
ERROR: 2015/09/01 Expecting to find a template in either the theme/layouts or /layouts in one of the following relative locations [partials/author.html theme/partials/author.html]
```

 2. To facilitate the auto-generation of the Hugo theme list, please expand your `theme.toml`, fix the size of your `images/screenshot.png`, and also add a thumbnail image at `images/tn.png`.  For more information, please carefully read the README https://github.com/spf13/hugoThemes, as quite a bit of changes have happened since you last updated your theme in January 2015.

 3. Please consider adding `{{ .Hugo.Generator }}` to your `layouts/partials/header.html`.  Please see https://github.com/spf13/hugoThemes/issues/42 for more information and https://github.com/spf13/hyde/pull/18 for an example.  :-)

Many thanks!

Anthony


`this-syntax` should work here.


Loopback is the whole 127.0.0.0/8, not just 127.0.0.1; at one point, either Debian or Ubuntu (can't remember which) caused us problems by using other loopback addresses for some things.

    return socket.inet_aton("127.0.0.1")[:1] == socket.inet_aton(ipv4_addr)[:1]


Could you add a comment explaining what this pattern represents and where it comes from?


The entire rescue/ensure block can be removed. Since we're not touching rack.input any more, there's no need to rewind it.


Верни обратно


hello @johnosaurus-rex
Emacs and Sublime sound good!


Should this be `var-int`? If so, why aren't the tests failing?


Make sense.


Bardzo drobna uwaga: przy assert_equal pierwszy argument to wartość oczekiwana (expected), a druga "actual".


@johnosaurus-rex
OK, but I did request that you use triple back ticks so your commands were in a list instead of text block format.
Also, you did not give an example of xargs, as requested.


I think this should be reverted


TeamCity pull requests :: netty [Build 167](https://garage.netty.io/teamcity/viewLog.html?buildId=1065&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5594, ignored: 21 Build time: 00:35:02


Thank for the fix.


I would rather try to find a solution that does not pollute the gulpfile

Did you check https://github.com/flycheck/flycheck/issues/92 ?


Bummer


@jasmin-j As I recently found, gcc-4.5.3 still works in ct-ng.
Can you verify this newer version of mpc still builds/works with older versions? Or do we need a constraint so that it only builds with newer versions of gcc?


I think people would read it, if it had a period in the end of the sentence.


It would also be possible to use anonServiceUserId for that as this is a random and unique key inside the keyserver.


CYNGN_TARGET TO PA_TARGET or whatever


What does this have to do with `letrec`?


One more thing: It's best to create a pull request from a separate branch and not from `master`.


I will fix this.


@spegelius Thanks for the quick reply.
Looking at [init.qcom.rc from cm-11.0-old](https://github.com/CyanogenMod/android_device_samsung_jflte/blob/cm-11.0-old/rootdir/etc/init.qcom.rc#L482) the macloader service is in place there as well. Though I obviously don't know if that was the base that you started from.

macloader itself is actually open-source and part of android_hardware_samsung, see https://github.com/CyanogenMod/android_hardware_samsung/tree/cm-11.0/macloader

Do you think macloader can be re-added again? Is there anything I can do to help? Normally I'd make a PR, but I guess this has to go through Gerrit?


TeamCity pull requests :: netty [Build 168](https://garage.netty.io/teamcity/viewLog.html?buildId=1066&buildTypeId=pulls_netty&guest=1) is now running


Missing period.


No, we only want to add the version number to requests sent to graph.facebook.com


Idem. Explain this pattern.


s/tcp/udp


Howdy again, @jbavari

Curious - why are we adding the Splashscreen plugin when adding the crosswalk browser?  Is it a requirement for Crosswalk?


Purpose statement.


Should also be changed for "normal" users instead of using username as ID store


Drobna uwaga: \s* powinno wystarczyć (bez grupowania) - nie będzie tworzenia niepotrzebnych grup


What's the status of this?


I think this could be avoided using:
https://github.com/clayallsopp/react.backbone
Search for "multiple models"


Move this to the top of the case study?


Did you figure out whether that should in fact be the case?


If you're adding a comment that says "need redis?" why not put this in a method called need_redis? :)


Why reverted change for empty username in last commit?


I'm having trouble accessing Jenkins' built in environment variables (e.g. BUILD_NUMBER for use in the plugin's "Release Version"), and I think it might be because 'matcher' here contains '${' ... I understand the desire for the check on line 30, but perhaps we need to remove '${' and '}' from 'candidate' before the lookup?

Put another way, I think we're trying to look for "${BUILD_NUMBER}" say, in the list of environment variables and it's not there (but "BUILD_NUMBER" is).


I think unrefing it would mean that the server could exit on an empty event loop while it was supposed to be waiting for for a re-connection to occur. I'd have to test that theory.

The `CHANNEL_TIMEOUT` is configurable by tests so they can be sped up. What tests are waiting for the full 60? Note that the `.destroy()` above clears all the timeouts.


I think the outer `Promise.resolve` isn't necessary, since the deeply nested `resolve` call should handle `promiseOrValue` in the same way.  IOW, I think you can go with just lines 27-31, and that'll also avoid creating 2 additional promises.


Looks good to me - thanks. @lresende, please merge this pr.


You should probably include a message in the exception to indicate what the problem is.


Hot link for Event Hub SDK?


Autodesk_Performance (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1281 of 1281 tests passed)

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


Here would be a good place to explain what internal and external names are in this context.


Include message in exception


Purpose statement.


Anche dando un occhio online, questa mi sembra la soluzione migliore. Molto intuitiva... Che dici?

    -module(mybinary).
    -export([reverse/1]).

    reverse(Binary) ->
        reverse(Binary, <<>>).

    reverse(<<>>, Accumulator) ->
        Accumulator;

    reverse(<<Head:1/binary, Rest/binary>>, Accumulator) ->
        reverse(Rest, <<Head/binary, Accumulator/binary>>).


@JanSchulz yeh, the ``[0]`` was a *hack* :)

I did read the warning (we are in fact using the ``get_referents`` currently to detect ``SettingWithCopyWarning`` in the first place).

but w/o actually do a lot of ``inspect`` not sure else how to do this


Имхо зря ты этот символ выбрал - он подразумевает минимизацию с глаз долой, а никак не открепление. Да и выглядит не очень. Предлагаю
''''
	.de-btn-close::after { content: "\u2716"; font-size: 15px; vertical-align: -1px; }\
	.de-btn-toggle::after { content: "\u29C9"; font-size: 13px; }\
''''


Odnośnie wyrażeń regularnych, trzeba będzie pewnie kiedyś przeglądnąć wszystkie występujące w systemie pod kątem grup - często są niepotrzebne.


if you want this in, you gotta PR it


that's the current parameter; going to deprecate that and make it an internal paramter I think (it is just ``None`` or a ``weakref`` back to the object)


So... I'm not sure if it really makes sense to split this out, even though it does mean fewer references. But I could go either way on that. (My general philosophy on this point is "do whatever make the test easier to understand and maintain".)

If we do split it out, however, I think the titles should be updated to reflect the fact that we're testing ID selectors, class selectors, and attribute value selectors (since these could all alternatively be using attribute selectors, which are different) and the help links should be updated to reference the relevant sections of CSS2.1/Selectors3 and whatever markup spec defines these things to be case-sensitive. The set probably also should have an additional check for the case-sensitivity of the attribute itself (the current test only tests its value), but I won't hold up on that since it wasn't in the set to begin with.


I'm with @patkar. Please, backport this to the 2.4 branch.


Couldn't you use free id tables instead?


Thanks for doing this! :)

I have just one request before allowing the merge. Can you wrap your new code in preprocessor directives so that 4.X users can still use it?

I'm fine with wrapping it for only 4.5 & 4.6 - like this:
#if UNITY_4_5 || UNITY_4_6
      TheOldCode;
#else
      TheNewCode;
#endif

Thanks!


Added commas after last array item, indenting line 242 (Jenkins said comma missing on line 1, but there should not be any)


@blacktrash I ran the debug session with cache disabled as you can see and the code displayed on the screen is actually the code getting executed (I can breakpoint it if needed), and this code is matching the diff you sent before.

```mediaPlayer.setAutoPlay()``` is true because ```!flowplayer.support.zeropreload && player.conf.autoplay``` evaluates to ```true``` on my device.

Let me know if I'm missing something


I'm confused, why is Guzzle 3 needed? I don't see it used anywhere?


No caso da alteração da entidade TbComunicacaoExterna, com a configuração da propriedade de "fetch = FetchType.EAGER" no atributo tbComunicacaoExternaxAnexoCollection (linha 56 de src/main/java/br/com/xti/ouvidoria/model/TbComunicacaoExterna.java):
- Gostaria de saber se a alteração foi provocada por algum erro da aplicação em tempo de execução ou se foi uma customização específica para a AGR;


the following version of a mutex function in bash/ksh/dash is slightly more reliable (I think) though yours is fine.

mutex () {                      # this variation uses flock(1) and is most reliable
  exec 9>$LOCKFILE
  if ! flock -n 9  ; then
    log "WARN process $pid has the mutex lock"
    return 1
  fi
  # we now have the lock until 9 is closed (it will be closed automatically when we exit)
}
#
# example calling sequence:
# LOCKFILE=/var/tmp/myscriptname.sh
# mutex || {
#   echo "$(date) $0 already running.  Exiting. " >> $LOGFILE
#   echo "$0 already running. Exiting." >&2; exit 1;
# }


svl_query_report is a view and complex to resolve. stl_scan might be a faster option.


well that didn't work; markdown ate the code.


And some internal comments explaining how it does what it does.


OK. If PyCharm says so.


OK.


![Uploading image.png…]()
how to fix this?


May want to generalize this comment as, IIUC, the general idea (matching things that appear inside unit expansion) applies to the next few syntax classes as well.


do you know if this script will run as-is on dd-wrt?  our home routers run dd-wrt; i am willing to switch them to open-wrt to get this awesome functionality.


More the merrier :)

What kind of plans do you have for this Wildstar emu? I couldn't find any private servers so I'm assuming you're the first


The type for this needs to be password.


@temasek
Thank you for confirming...


Entendemos que a revisão no script de carga foi feita visando a adequação dos dados para o contexto interno da AGR. Apesar das otimizações no código sql, entendemos que este script atende necessidades específicas da AGR e por isto pode ser mantido apenas no "Fork" OuvidoriaAGR, pois não faz sentido acrescentá-lo ao script genérico do GOG.


I add the patch few months ago to easybuild and emailed it to the developers so they fix it in the main release.  Developers replied they were aware about the issue and that it would be fixed for next release. They sent me that link. But maybe they skipped the patch?

In any case if it works, then fine....


I think you can just do `resolve_type_vars_if_possible(&obligation)`, no?


Wow, this is ugly, I did not remember about it.
Since all values are constant, why not simply copy a .conf file to /etc/avahi/ ?


It won't break anything until I upgrade the version of the tool. :smile: When I do the upgrade I am thinking to move the folder into `main`.

I don't use bootstrap task more than once. I only used it once. How does it know that the folder "internal" exist?


I believe that is right. I think its an issue it will not show the splashscreen without.

Care to verify that is the case?


Also:

max_service_class_slots: Max number of WLM query slots in the service_class at a point in time.

Would that be the amount requested slots. Let's say the slots is 5 and the max_service_class_slots is 10, it means that at one point there were 5 running and 5 waiting ?


Let's change path to "/orsee"


This message will need to be updated.


I have created PR #8


Trying to "propose new file" to send a pull request to have this style added to the CSL directory.


do we need this?


Nice @zepner
Going forward, please branch and create a PR.


Also, it would be nice if there were a TODO or something here, because currently the vm.authorized is never set in this scope.


Lwet's change these to
orsee_db
orsee_user
orsee_pw


in principle, this case is now impossible, right? perhaps replace with calls to `bug`?


I'm afraid neither the `Cuda` -> `CUDA` nor the `FORTRAN` -> `Fortran` changes are possible sorry. See https://github.com/github/linguist/pull/901 for context.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.944.7b1e6b71e7e48ce82f90264d84609a168c3e384c](http://build.palaso.org/viewLog.html?buildId=62492&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:03:18


huumm right!

Now it is not dynamic anymore.

I'll do it... :)


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2292 of 2343 tests passed)

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


mmm, i prefer out-of-source build.


If you have question you can join the #Multi-Emu channel on rizon :)


I think getting rid of it would be better, but I can't figure out a way to make it work. Does the fact that it's a splicing syntax class make a difference? The problem is that an init-depend clause is optional in the unit syntax, and all of the ways I've tried to remove this syntax class result in breaking all of the test cases.


:+1:
Ref: https://github.com/zsh-users/zsh-syntax-highlighting/issues/126#issuecomment-136625807


Yeah fixing it today

On Wed, 2 Sep 2015 1:40 am Matthias Kuhn <notifications@github.com> wrote:

> @NathanW2 <https://github.com/NathanW2> I guess that is related to your
> recent theming work? Didn't look into it but I disabled it for now since it
> tags pull requests as bad and hides "real" issues. This message is just to
> notify you ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/524716fbcc95fd25b68b5f3839640824909f0988#commitcomment-13002505>
> .
>


Fixed, thanks.


Sloppiness be gone! Good find @labboy0276


Please comment why we need -platform offscrean (in source) and provide link to answer (on stackoverflow or smth).


I was very much thinking that we would have a single load balancer implementation, which is why I changed ports to be `ports []*api.ServicePort`.  I'd still lean that way because of other protocols - even if realistically only TCP & UDP matter, but maybe somebody will want to use SCTP one day!

However, I have since learned that AWS ELB does not support UDP traffic anyway.  We'd have to implement this using Elastic IPs I believe, which is basically reimplementing ELB.  Given that, I think we should not assume that AWS will support this soon (which shouldn't block support on other platforms in my opinion)


SVL_QUERY_REPORT is the only system table which reliably has near or greater than the 604800 rows necessary to generate 7 days worth of timestamps with 1 second granularity. STL_SCAN would be faster, but would generate a smaller range. So long as the range gerated with STL_SCAN goes back further than the oldest record in STL_WLM_QUERY then it would be a better option.

Re: max_service_class_slots, yes the amount of requested slots (executing + queued) at a given time .


Ok. I figured it out. I fixed the problem with this commit

https://github.com/lemire/FastPFor/commit/211585cb55d90c421a5dbe9fca432794836745d4

Can you sync your fork?


Спасибо, годный символ, нраица.


Left it in there by accident


Ok.


From what I got from the comments that @edersondisouza (in the first version of this series) and @barbieri made, the idea was to move this function to a common place (`sol-util.c`) and only have the implementation of something similar to `rand_gen` in the platform specific files.


Autodesk_Performance (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1814 of 1814 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)


Never tried changing a file like this, I expected it to do something since it was possible.


Does this represent the whole expansion of the unit form? If so, may want a comment mentioning it.


Add a test for this function. This function can be static.


Also fixed action for inline email signup


Yes, the correct thing to do would be to create two extra OR(3) blocks; ; however, the proposed diff produces slightly better code than what the current algorithm does (I.E. it would write:

OR(9)
Player5
Player5
Player5
OR(3)
Player6
Player6
Player6
Player7
Player7
Player7
Player8
Player8
Player8


@dinhviethoa @DaveInAus, Sorry, I've been distracted for a few days, but I've now filed new issues for the two issues I'm seeing.


well, PR it. :P


Maybe we could also check if this link brings the user to the right page when following it?


@ctomc : Do you know if this will make the next EAP 7 build?


Please consider linking to the documentation at https://azure.microsoft.com/documentation/articles/virtual-machines-docker-registry-on-azure-blob-storage/ in your blog post. :)


We're not going to bother our selves again with your nonsense sira, Ezio corrected the commit and comment. its all good by me now. dont want to waste more energy on this. Be happy be proud, that your name is mentioned to. the real devs among us know better. thats why they removed your shit remember. But enough said, lets leave ezio do his work again so he doesnt have to read all this bullshit.


Ohh, this forked it. I expected it to only submit the thing, since I didn't click anything indicating a fork. OK, the more I know.


Good. Can you squash this commit on the previous one and force push? (you don't need to close and reopen, just push - - force)

Otherwise, I'll do locally when merging.


edit to commit description:

"by removing /pub from docs() creation, **/pub** is added in docs function"


I am shame


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1532


85723528_Issue1815 (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2291 of 2343 tests passed)

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


Thanks for pointing out. I fixed it in 24059d9


Need to remove


93651760-RowPerObjectReport (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1807 of 1807 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)


TeamCity ReOpenLDAP :: devel-2.4 [Build 185](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=261&buildTypeId=ReOpenLDAP_Devel24&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:43:30


Hi Brady
My answer is attr() because the variable is inside the html tags


Isn't it really the parent class which is copied?


Thanks


101789044_gridviewFilters (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2304 of 2343 tests passed)

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


<img src="https://cloud.githubusercontent.com/assets/11808693/9594681/2d00f344-502c-11e5-821c-74b9a441748a.jpg" width="152">


Mind putting in a comment to specify the unit here (bytes, I imagine)?


brewで入れられるんですね! issue化します．


- [ ] I think this function might be useless too.


No no. A few lines below, I do ```trySleep(Math.min(100 * i, 1000));```, which limits the sleep time to 1 second max between iterations. The ```100 * i``` ms is just to promote the possibility of a quick retry in the beginning, instead of always waiting 1 second, but it quickly caps at 1 second, which brings the total wait time for ```MAX_ATTEMPTS = 120``` at 1m55s.

That being said, ```ServerTestUtils.waitForServerStart``` also introduces its own sleep time... If you're really worried that it might get stuck forever, we could add a max elapsed time component to the retry logic. I'm going to add a constraint for that and submit a new commit in the PR.

-F


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1784 of 1813 tests passed)

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


All done @devinwalker. we'll deploy in the a.m.


Is this intentionally not `unref()`'d? The default 60s timeout pretty much guarantees any test that triggers it will have its process kept alive long enough to cause tap to timeout :-(


Possibly should set `errno = 0` here?


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

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


btw, this slightly breaks windows compat for lib/socket.c, will need to use `WSAGetLastError()` instead.


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - 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)


fast-forwarding develop to borshop-integration-1231-bugfix/cfishy-riak-test-gleaning = 1643c8d2


Hiya Steve - the extensions.displayObject() component referred to here doesn't seem to exist, and is giving me some errors


Does this linter support linting on the fly?


Looks good. /Maybe/ move the eyes up one or two pixels, since Avali eyes are higher on their heads, and the silhouette seems to be looking straight at the viewer.

I'll hold off on that pull request until there's a more polished product ready, though, at the very least a rename of the files :P


'poops'
right this way sir


Yeah good idea

On Mon, Aug 31, 2015 at 1:20 PM, erfangc <notifications@github.com> wrote:

> Hey Chris - Cab we add a check for the existence of .data
>
> Ideally the payload that is resolved is already the series options object.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/erfangc/DecoratedStockChart/commit/678bcf126269d7335bc826e34117cf12fc181fa4#commitcomment-12980969>
> .
>


@xiphiaz That would be really nice if that gets fixed, and would allow us to streamline the processing of logging the data from sync events much better.


why didnt you remove the .txt tags?


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.540](http://teamcity.monogame.net/viewLog.html?buildId=9887&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:25:49


does your binding need an UpdateSourceTrigger maybe


It looks like you've left a note to yourself to document what `CHANGE` is used for?


I think so.

I think that the concern with generally keeping the (Merge)Receive node in the subquery was that it would needlessly generate more-than-2-fragment plans in cases where 2 fragments could have done the job. I don't completely understand how that can be an issue only for the simple cases where we now pull out the (merge)receive but never an issue in the more complex subqueries when we keep it -- or is there some other (join? multiple-subquery?) test case that justifies this behavior -- succeeding when the subquery is simple (and gets further simplified to one fragment, but failing in the cases when the subquery's greater complexity prevents it from being simplified to one fragment? I suppose that's plausible or maybe we just neglect to "negative test" the problem cases.

The edge case you found is a challenge in itself -- we can't quite seem to convince ourselves that the subquery's explicitly requested output order is significant (for a change) and is in fact critical to determinism. In the long term when we have a better handle on both plan node output ordering and determinism (also a work in progress) we may have the proper perspective to decide properly whether to eliminate a MergeReceive, but maybe with our current technology that's too risky?

We do need to balance that with the risk of regressing some more complex edge cases that will now suddenly generate a more-than-2-fragment plan. Maybe we accept that risk knowing that doing an order by but no limit in a subquery in the context of a join is very edgy indeed  -- it feels like the query writer is inappropriately trying to micro-manage the plan -- which may not be a winning long-term strategy as the planner gets smarter, anyway, so maybe they can break the habit now.


With all three of the line items you are adding you start off with a line break. The line break doesn't require an end tag so just using <br> would be sufficient, and it looks like there are two per line in lines 51 to 53.

Another issue is the target for the href. The files we are linking to need to exist on the web server and they don't at this point in time. So we also need to upload them to the server. The best thing to do would be to include them in your pull request. The web site has a downloads folder, although that it isn't in git yet, so why don't we just use that folder.


To preserve the original behavior, you probably need to coerce `str` to String if it isn't already.


We skipped supporting work preserving restart for 0.1. Without it the containers are getting killed when NM is killed. But we will have to consider it if we are to support it later.


有这个异常
```
433-433/dg.shenm233.wechatmod  I/Xposed﹕ java.lang.IllegalArgumentException: Cannot transfer object from class android.content.res.MiuiTypedArray to class android.content.res.XResources$XTypedArray
```
这个应该是小米系统引起的。

我修改后发现可以使用，就提了PR。
其实我不太懂这个，不知道会引发什么问题。


yes, will update


TODO: the workers map must be protected by a lock


@rtpHarry typo here


I strayed off 12001 because other things in the mod may be using that ID. 12000 may have been Ryuu's starting point, so I stepped back one.


I think I get it now.


Not a problem dude, I just like helping out to tell you the truth, makes me learn as I go 😊

Sent from Windows Mail

From: Fevix
Sent: Monday, August 31, 2015 10:19 PM
To: Fevix/Avali
Cc: Luthor614

I strayed off 12001 because other things in the mod may be using that ID. 12000 may have been Ryuu's starting point, so I stepped back one.

—
Reply to this email directly or view it on GitHub.


この変数使ってなさげ．


Thanks!


Note: This still requires that FEE bumps its version compatibility to WP 4.3:
https://github.com/iseulde/wp-front-end-editor/pull/261


I kinda like these friendly Floran lines :D


@guangyy can we just use `chunk_index(i)` here?


Note: This still requires that FEE bumps its version compatibility to WP 4.3:
https://github.com/r-a-y/wp-front-end-editor/commit/86aada0d7435fa7d2af514dca9db94a28ac91bf1


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-MacOS-10.9-clang: OK (1808 of 1808 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)


[NSNull null]...原来是这么用...


Good catch. I thought it would always be translated to SSL_ERROR_ZERO_RETURN, but that's not true.

Actually, the code in so_read will only invoke SSL_get_error when n < 0, so I guess the current code is fine. But it simplifies the code to unconditionally set errno in the failure case.


It seems you aren't entirely up to date? There's quite a few updated stuff that you seem to be replacing with older stuff, like dialogue lines.


then에서 객체를 return하면 다시 then을 사용할 수 있다.
이렇게 해서 처리하는게 좀 더 좋다.


There's quite a few places where I've used errno instead of the appropriate macros. I've been putting off fixing those places; no time to test Windows builds or figure out whether I should be using SetLastError or WSASetLastError.


typo


LGTM


performance & chicken & egg problems in boot

the 9000 redis is a system redis
and should never have a password
this same redis should ONLY listen to localhost so no security issue
it should only be used for jumpscale system related tasks

so hardcoded yes ofcourse because there is never any other alternative
possible
is always local & always no passwd & needs to be always port 9000

On Mon, Aug 31, 2015 at 9:09 PM, Jo De Boeck <notifications@github.com>
wrote:

> How is this better? Has hardcoded values all over the place. What was
> breaking that made you make this change?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Jumpscale/jumpscale_core7/commit/f86dee0d732f946e60156327cd251e2c48714904#commitcomment-12983080>
> .
>

--
------------------------
Kristof De Spiegeleer
+971525609014
+201206927877
+32 475405474
skype: despiegk


or whatever tcp port it was
redis client ofcourse is required for all other configurable redis clients

On Tue, Sep 1, 2015 at 4:30 AM, Kristof De Spiegeleer <kristof@incubaid.com>
wrote:

> performance & chicken & egg problems in boot
>
> the 9000 redis is a system redis
> and should never have a password
> this same redis should ONLY listen to localhost so no security issue
> it should only be used for jumpscale system related tasks
>
> so hardcoded yes ofcourse because there is never any other alternative
> possible
> is always local & always no passwd & needs to be always port 9000
>
>
>
> On Mon, Aug 31, 2015 at 9:09 PM, Jo De Boeck <notifications@github.com>
> wrote:
>
>> How is this better? Has hardcoded values all over the place. What was
>> breaking that made you make this change?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Jumpscale/jumpscale_core7/commit/f86dee0d732f946e60156327cd251e2c48714904#commitcomment-12983080>
>> .
>>
>
>
>
> --
> ------------------------
> Kristof De Spiegeleer
> +971525609014
> +201206927877
> +32 475405474
> skype: despiegk
>

--
------------------------
Kristof De Spiegeleer
+971525609014
+201206927877
+32 475405474
skype: despiegk


Sorry about the EOL changes.


ok. let me investigate on it.


Projeto básico de conexão com Banco de Dados via JDBC.


Uh, so I guess saving a log to the database isn't supported anymore? Bummer, most of my clients want this.


Did you know the issue you raised here has been addressed in
f2fc2b507bb9c55f76c55d2be716db657a056405?


Wait, so its just funciton (...) return ... end? This must sound stupid but, its just returning what it was given?