Could return http status code 400 - bad request


@anton-afanasyev По поводу массива - пропустил это. Не хватает реально проверки, что бот в чат выплевывает.


Could return http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.4.10


TeamCity Civic Assembly :: Build backend + frontend [Build 74](http://54.229.176.143/tc/viewLog.html?buildId=167&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Again could return new text and just http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.2.2


Gasp... You are right, without lambdas, thenReturn makes it way prettier...


Question regarding NEWLINE:
we can see ```<br />\n```
as well as
```\r\n\r\n```
and even
```\n```

Which one should we use?
Are the encoded somewhere when composing emails contents?


This is the first time I'm using Aurora and I copied a lot of the configurations from the Vagrant example. Are you saying the parameter to announcer-ensemble should have 5 nodes? I'm not exactly sure what this parameter is used for.

Also I've updated the quorum size to 3.


Could do a select count(*) and return count > 0


Yes, its an OB theme, menda package is too big just for window theme.


fixes https://github.com/rundeck/rundeck/issues/1590


we didn't add a kb shortcut yet.. I still propose ctrl+Enter for consistency


This might not be suitable for all users? What if some users do not need a time?


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


No validation here?


doesProductExist gives more reflection on what the return values will be i.e. doesProductExist - true = yes, false = no. Check is a bit confusing you can't really say a yes no answer. Its more reflective of an action rather than a question e.g. ... go check this vs, does this exist?


The issue here is that the lock file is distributed with the source code, so the dependency versions are fixed there. This because I want to be sure that the non-develop dependencies are correct.
As this project uses travis-ci to check the builds in all PHP versions, the only phpunit version that works cross-platform is the 4.8.19.
As I understand Composer do not support conditional dependencies, so I am a bit stuck here.

Eventually I can set the various non-develop dependencies versions and remove the lock file. I'll have a look to this in a bit...


sfasf


If not successful, maybe it would be good to also report what wrong happened?


afaasfasf


See https://github.com/twbs/bootstrap/issues/18520#issuecomment-163890539
(NPM `dist-tags` shows "latest" is bound to alpha v4!)


There are still places where is /usr/libexec/ path hardcoded. I had to put
find %{_builddir}/%{name}-%{version}/src/ -name '*.conf' | xargs -i sed -i 's|/usr/libexec|/usr/lib|g' {}
in my custom openSUSE abrt.spec to replace them with what I have in Tumbleweed (cleaner solution needed).


That should work from the type-hint, but yeah why not making it explicit also


pixel raio ?.. i think its ratio


Please introduce constants instead of magic 6 and 8 numbers


Esto se debe a que realmente, SQL plus no es un editor. Sirve para ejecutar los comandos.

Para el que esté interesado en conocer el editor que usamos: Eclipse.


>  I know about changing default Openshift name so will add it

cool


I misjudjed the nitpick, it had more reasons than just style :) Good point about IEx also, didn't think about that.


@molinch That's true. Sorry for that!


:) myslim ze mozme aj slovensky v takomto projekte.
Hej ja som aj v svojej branch-i mal toto upravene aj s pouzitim require (AMD moduly) ale kedze, je cely projekt pod riadnym developmentom tak som to vzdal kvoli konfliktom - mozno neskorej.

Takisto generovanie select-u v html som mal na zaklade tohoto zoznamu - aj ked tam je teraz pouzity trocha iny list (ID ale sedia).


How it is saved? Where?


RÈGLEMENT GÉNÉRAL DE L'AUTORITÉ DES MARCHÉS FINANCIERS/Livre II/Titre I/Chapitre I/Article 211-2.md

Préciser l'Union **Européenne**


This causes a problem. '+=' accumulates an error which np.arange does not. When missed once, it causes the solver to keep running without writing any files. Must be replaced with self.tout_next = intg.tcurr + self.dt_out.


OK pour la modification mais rajouter mention "Union Européenne"


TeamCity Civic Assembly :: Build backend + frontend [Build 74](http://54.229.176.143/tc/viewLog.html?buildId=167&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 103, passed: 73, ignored: 4 Build time: 00:10:37
### Failed tests
```
Basic Events CRUD Actions: Should Create an Event: <no details avaliable>

Basic Events CRUD Actions: Should Edit an Event: <no details avaliable>

Basic Events CRUD Actions: Should Delete an Event: <no details avaliable>

Page tests: Should not create new page if cancelled: <no details avaliable>

Page tests: Should add new page: <no details avaliable>

Page tests: Should not change page: <no details avaliable>

Page tests: Should change page: <no details avaliable>

Page tests: Should not delete page if cancel pressed: <no details avaliable>

Page tests: Should delete page: <no details avaliable>

Page category tests: Should navigate to Page Categories page: <no details avaliable>

Page category tests: Should not create new page category if cancelled: <no details avaliable>

##### there are 93 more failed tests, see build details
```


/cc @josevalim and @lexmag!


BTW: This method is not fully thread safe. We need to do something about it (eg. lock a bitmap?). I though if we remove a bitmap every time at the beginning - another thread couldn't interfere with it.


> This because I want to be sure that the non-develop dependencies are correct.

composer.lock doesn't work for this, it is ignored for dependencies (only used for main project)


just hostname, yes. This setting is unrelated to Landrush anyways and currently you get the impression it belongs to the Landrush settings.

The problem is that setting the hostname will influence the OpenShift configuration file name, in particular the node config. This becomes important in the case we have when we create and modify the config files prior to starting the actual OpenShift instance.

It makes sense to set the hostname though and I think it should be allowed, but one needs to factor this name into the provisioning scripts first.


I would prefer to keep both both names as they are.


OK pour la modif :+1: en attente de validation collège


oops, next iteration, does not harm much (but icc)


@tumtumtum if I understand this code correctly… shouldn't this be:
```objective-c
Float64 retval = (self->seekTime * self->audioStreamBasicDescription.mSampleRate) + self->framesPlayed;
```

`seekTime * sampleRate + framesPlayed` instead of `seekTime + sampleRate + framesPlayed`

Is that correct?


Evidentne máme rozpracované rovnaké veci :smile: Urobil som iba jednu definiciu a následne som z nej pripravil array pre select2. Ale ako vravíš, kód sa mení príliž rýchlo pod rukami, bez PR a CR takže chvíľku počkám a potom začnem pomaly refactorovať js.


Ok, I see that you did it. That's nice!


See https://getcomposer.org/doc/02-libraries.md#lock-file


В общем админка станет адаптивной под мобильные устройства. Я просто переверстал стилями админку.


I think this change will increase the api resonses alot. I belived i added this so we dont request new info all the time.


Should be `csharp` not `cs`.


has been -> have been


It's already obvious that this is a great idea! Well done!


Pracujeme na rovnakých veciach. :smile: Je potrebné mať iba jednu miesto kde sú mesta definované a používať ich tiež pre select2. V lokálnej branchy som už zmigroval data do objektu a následne z neho vytvoril tiež array pre select2.

Ale ako vravíš, ide to príliš rýchlo a bez PR a CR sa ťažko robia väčšie zmeny. Dočasne sa budem pokúšať opravovať len drobné veci a pomaly migrovať k lepšej štruktúre a následne neskôr sa to hádam preklopí poriadne.


LGTM


In following implementation Pupils.html has two related controllers: pupilsController and permissionController. I would suggest refactroring this:
1) Change permissionController to be service and expose methods like: canShowPupils() { return true/false;}
2) Extend existing pupilsController to have scope property: $scope.canShowPupils and set this property based on value from new permission service


если вы про линию ".ace_maximizer {cursor: pointer;}" то при наведении на http://joxi.ru/12MQgX0SXRRW2J будет показываться курсор как будто он на ссылки, давая знать, что окно можно раскрыть.


I don't see tests deleted where they were moved from.


@evanweaver FYI, hopefully this got in before you built packages


Thanks


ayy lmao


出击1次，需要战舰旗舰~~1只~~


Ok, I added my email in there, thanks!


refs #39


Fixes #12.


https://github.com/bitcoin/bips/pull/261/files

I was incorrect when I wrote the initial script on reddit.

Apparently (according to the BIP proposal, and upon further investigation, the merged code) it should be:

```javascript
return seq < 0xffffffff - 1;
```

I guess they will allow 0xfffffffe to be non-RBF so you can still use Checklocktimeverify etc. (that require non-MAX sequence to work) without requiring you to opt-in to RBF.


Github shit the bed on this.
As you can see the comparisons are all fucked up. I think I've resolved everything properly but if you made a commit between the last time I merged and the next time and you edited index to do it, double check it against your code before pulling.

also, please nobody force push code.


霰？


摩耶改/改二


我就知道这几个人我会弄错


我本来准备放摩耶改的……摩耶改就包含了摩耶改二了。


@isocolsky @matiu @braydonf


I have uploaded a new release containing bindings for both QtQuickWidgets and QtMultimediaWidgets. Thank you for the suggestion.


好【


Committed by me, the incorrect attribution was unintentional.


Haha, peamiselt kasutan ikka Gentood!


已解锁 -> 未上锁


Thanks! Now it makes sense why you didn't release `v0.1.1` even though there is a tag for it already. :+1:


It should be $font: map-get($conf, font);


Wtf


what do you mean?


ok


Report "success"


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 1.1.375.0](http://build01:8888/viewLog.html?buildId=13484&buildTypeId=ConfigInjector_CI) is now running


Thanks, I updated both of the links :)


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 1.1.375.0](http://build01:8888/viewLog.html?buildId=13484&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 26, muted: 1 Build time: 00:00:28


Oh, ok. Fixed in 5edd7e3d.
What's the reason? Is there a problem with serialization?


This changes the behavior, is that intentional? Before this patch we would never return 0 here, but now we may.


The victory.png that was deleted here is actually the updated one.


released in 1.147


Yes. It thought it was `_free_sst(sst);` like in the other "classes". I'll fix it.


Lets change this to "if creating new student".


Play services / Google API Map working on both physical and virtual devices. This was needed as physical devices are not always available


The side effect is that `<ui-layout class="layout-row">` will have a css style `display:flex; flex-direction:row;`. The workaround may need to be:

```css
ui-layout {
   display:block !important;
}
```


pero para que hacer eso?


fix_shell_script (nmerket) - i386-Windows-7-VisualStudio-12: OK (1984 of 1984 tests passed)

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


I dug deeper into my assumptions about the order of operations within an NSOperationQueue, and while they are generally right, Apple says it can't be guaranteed, so you're right, we need to be more explicit about this. Will fix.

From `NSOperationQueue` docs:
> An operation queue executes its queued operation objects based on their priority and readiness. If all of the queued operation objects have the same priority and are ready to execute when they are put in the queue—that is, their isReady method returns YES—they are executed in the order in which they were submitted to the queue. However, you should never rely on queue semantics to ensure a specific execution order of operation objects. Changes in the readiness of an operation can change the resulting execution order. If you need operations to execute in a specific order, use operation-level dependencies as defined by the NSOperation class.


I feel like adding another reminder here about an important point: you have to complete all the promises with either a result or a failure.


done


No problem, sounds good! The most successful study groups are the ones with several people helping to organize and lead, so teaming up with Gustavo is a great plan! I'm looking forward to hearing how things are going soon!


These look reversed. I think you need to subtract the previous total from the current total for the delta. The totals will increase from week to week. This will cause the value to be negative.


I've updated the FFmpeg builds for all operating systems. The OS X build is from http://evermeet.cx/pub/ffmpeg/snapshots/
Please let me know how they go


Same as above.


LGTM other than the the subtraction being reversed.


Just use now() for this. Though see below.


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


vv


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1977 of 1978 tests passed)

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


Flask gives that for hostnames that aren't subdomains of SERVER_NAME.


fix_shell_script (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


Cant' disable the preemption stuff. Does PENDSV exist on M0?


Timestamps are collected in sd_event_wait() above, could we just rely on them rather than taking new ones here?


Minor fix to consider the misclassification of snake case words as UUIDs (eg. `aaa_aa_aaaaaaa_vehicle_discounts`):
```
" Format all uuids
function! FormatUUIDS()
    %s/\<
      \\([0-9A-Za-z]\{8\}
      \\)\([0-9A-Za-z]\{4\}\)
      \\([0-9A-Za-z]\{4\}\)
      \\([0-9A-Za-z]\{4\}\)
      \\([0-9A-Za-z]\{12\}\)
      \\>
      \/\1-\2-\3-\4-\5
      \/g
endfunction
command! FormatUUIDS call FormatUUIDS()
```
Not doing a pull request so your contributors list isn't polluted :P


What system did you test this on? This is broken on Linux:

    src/terra.cpp: In function ‘int terra_initwithoptions(lua_State*, terra_Options*)’:
    src/terra.cpp:247:97: error: invalid conversion from ‘const char*’ to ‘const unsigned char*’ [-fpermissive]
         err =    terra_loadandrunbytecodes(T->L,luaJIT_BC_strict,luaJIT_BC_strict_SIZE, "strict.lua")
                                                                                ^
    src/terra.cpp:150:5: error:   initializing argument 2 of ‘int terra_loadandrunbytecodes(lua_State*, const unsigned char*, size_t, const char*)’ [-fpermissive]
     int terra_loadandrunbytecodes(lua_State * L, const unsigned char * bytecodes, size_t size, const char * name) {
         ^
    src/terra.cpp:248:103: error: invalid conversion from ‘const char*’ to ‘const unsigned char*’ [-fpermissive]
               || terra_loadandrunbytecodes(T->L,luaJIT_BC_terralib,luaJIT_BC_terralib_SIZE, "terralib.lua");
                                                                                ^
    src/terra.cpp:150:5: error:   initializing argument 2 of ‘int terra_loadandrunbytecodes(lua_State*, const unsigned char*, size_t, const char*)’ [-fpermissive]
     int terra_loadandrunbytecodes(lua_State * L, const unsigned char * bytecodes, size_t size, const char * name) {
         ^


On 2015-12-11 18:29, Ricardo Arturo Cabral wrote:
> Did you forget to push g13_lcd.cc?

oops, sorry about that... should be there now and hopefully it's
working, have some other changed I didn't push yet because they break
the joystick.

- James

> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/james-fowler/g13/commit/68cad1e38f8b4af2dbcb71a95f1670064f9008d2#commitcomment-14937515


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (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)


What you are doing is measuring the time sd_event_dispatch() takes, right? Why not just move the logic into sd_event_dispatch() and rely on sd_event_now() for the first timestamp and use now() for the second? That way we only get the overhead of one extra timestamp per iteration in case profiling is enabled.


Oops, you're right, can you add the updated image again? Thanks!


Would it make more sense to add a non-checkbox selection of language_in directly in DebuggerGwtMain.java (like a combobox or something)?

I always feel that selecting the language through checkboxes is like pulling teeth.


We were not going to handle upgrades. It would have made the playing experience complicated.


Agreed :)


yep, here is a working example:

```javascript
function printParkingTicket() {
      var image = new Image();
      image.src = 'img/my-image.jpg';
      image.onload = function() {
        var canvas = document.createElement('canvas');
        canvas.height = 79;
        canvas.width = 70;
        var context = canvas.getContext('2d');
        context.drawImage(image, 0, 0);

        var imageData = canvas.toDataURL('image/jpeg').replace(/^data:image\/(png|jpg|jpeg);base64,/, "");
//DatecsPrinter is the plugin instance
        DatecsPrinter.printImage(imageData, canvas.width, canvas.height , 1);
      };
    }
```


nit: can we be much more explicit with the name?  Like: osVersionRequiresLoadFromCacheHack


The code now uses `name`, so:

```ruby
puts "xcversion simulators --install='#{candidate.name}'"
```


osVersionRequiresManualStoreToCacheHack


About the TODO: your whole logic here relies on sd_event_run() being called repeatedly (presumably from sd_event_loop()), in that case, why not just put the logic in sd_event_loop() itself, then you would know what the first run is. Or, as I prefer (and noted below), why not just do the profiling inside sd_event_dispatch() directly, and never rely on it being called repeatedly by flipping the two measurements around, or am I missing something?


Oh sorry. With the old factor binary `refresh-all make-my-image` should work. Then you get an image which should be about 1k larger than the old one. Then factor needs to be recompiled because the image header is different. Then `./factor -i=boot...image` should work.


Hey Diego, sos una güeba, eso no es un juego de verdad.


Very nice work! Do you have an ETA for the CR1 release?


Use usec_t and now() for any timestamping (should also drop the need for some of the helpers below).


what are we thinking of making this step?


You should also print the name of the event source to blame, we print that elsewhere, so maybe add a helper for that (whether the pointer or the string should be printed).


Can you give me an example of how you are using the load() function in your project code?


Yeah I think we should! we imported random above


The idea makes sense to me. Would love to see something like this. Though it really should be done per-event-source to be truly useful.


ss-local


Also, I think last_run could be avoided by turning around your code to only use local variables for this, but maybe I'm missing something.


Hard coded language?


@arnaudleray Hopefully the copyright notice is good for you.


Please make these per event-source, rather than global. We don't want to print a lot about the same problem over and over, but we do want to get at least one log-message from each event-source that is showing problematic behavior.


ack


i forgot it sry. I'll (try) to care about it tomorow :)


looks good


need comma prior to $msg?


I think it makes more sense to make a "configfiles" or "markups" or some genericly named package - as Dockerfile is too specific.


What's the motivation for excluding 0.10?

Would you consider bumping up package version to 1.2.0 for that? For folks on 0.10.40 it's very much a breaking change.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1983 of 1984 tests passed)

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


You may need to make clean because it doesn't know the makefile changed.

On Fri, Dec 11, 2015 at 4:25 PM, Elliott Slaughter <notifications@github.com
> wrote:

> What system did you test this on? This is broken on Linux:
>
> src/terra.cpp: In function ‘int terra_initwithoptions(lua_State*, terra_Options*)’:
> src/terra.cpp:247:97: error: invalid conversion from ‘const char*’ to ‘const unsigned char*’ [-fpermissive]
>      err =    terra_loadandrunbytecodes(T->L,luaJIT_BC_strict,luaJIT_BC_strict_SIZE, "strict.lua")
>                                                                                ^
> src/terra.cpp:150:5: error:   initializing argument 2 of ‘int terra_loadandrunbytecodes(lua_State*, const unsigned char*, size_t, const char*)’ [-fpermissive]
>  int terra_loadandrunbytecodes(lua_State * L, const unsigned char * bytecodes, size_t size, const char * name) {
>      ^
> src/terra.cpp:248:103: error: invalid conversion from ‘const char*’ to ‘const unsigned char*’ [-fpermissive]
>            || terra_loadandrunbytecodes(T->L,luaJIT_BC_terralib,luaJIT_BC_terralib_SIZE, "terralib.lua");
>                                                                                ^
> src/terra.cpp:150:5: error:   initializing argument 2 of ‘int terra_loadandrunbytecodes(lua_State*, const unsigned char*, size_t, const char*)’ [-fpermissive]
>  int terra_loadandrunbytecodes(lua_State * L, const unsigned char * bytecodes, size_t size, const char * name) {
>      ^
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zdevito/terra/commit/39a6390ca547280566e470ca7ba7a90f0236cbaf#commitcomment-14938170>
> .
>


Changed BANKACCTFROM to CCACCTFROM for Amex statements


I've reverted https://github.com/loup99/WtWSMS/commit/e0b9de0be728f5ddf5ed1198c7432b83290b9113 as it was the only impact ;)


fast-forwarding end-to-end/timeseries to borshop-integration-1304-at/select_column_order = d670013d


Any better way to grab registered custom metadata than this?


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1987 of 1987 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)


Sorry, I misled here by failing to point out that it should be `update --init`.  Because git makes so much sense...


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 1.1.376.0](http://build01:8888/viewLog.html?buildId=13485&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 1.1.376.0](http://build01:8888/viewLog.html?buildId=13485&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 31 Build time: 00:00:37


fix_shell_script (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1984 of 1984 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 Open-source Libraries :: ConfigInjector :: CI [Build 1.1.377.0](http://build01:8888/viewLog.html?buildId=13486&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 1.1.377.0](http://build01:8888/viewLog.html?buildId=13486&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 31 Build time: 00:00:27


everything done

see #681


It's just that it was considerably heavier per avionics than any other core. So, game balance.


Sorry for the extra round.
On Dec 12, 2015 1:51 AM, "Erik Bray" <notifications@github.com> wrote:

> Sorry, I misled here by failing to point out that it should be update
> --init. Because git makes so much sense...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsipocz/pyasdf/commit/f77ea3da57315b81d6933e51e1665c4eb7b3af48#commitcomment-14938390>
> .
>


Ohh, OK fair enough. Should have tried out that too, but dealing with
submodules is still not routine for me :-)
On Dec 12, 2015 1:51 AM, "Erik Bray" <notifications@github.com> wrote:

> Sorry, I misled here by failing to point out that it should be update
> --init. Because git makes so much sense...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsipocz/pyasdf/commit/f77ea3da57315b81d6933e51e1665c4eb7b3af48#commitcomment-14938390>
> .
>


ack


Thanks.


I am a nice comment


I really like how this is coming along! Good job! :+1:


Shouldn't `c` be the package name ?
I mean even the pylint is applicable for Cpython, JPython and Iron Python (all those python variants) - but we still call it python.


It keeps track of the highest lastLedgerSequence of all tx submitted.  It waits for that ledger to pass.  Then it checks the decommissioned wallets that their owner count is 0.  Which is would be, if all has worked.
Only then does it return the XRP.
If it fails to 0 the owner count, it prints a message, but will exit successfully.


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (913 of 913 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-26.26%25-green.svg)


I can't tell what variable you are talking about.


THIS ??


メモリの節約にはならない気がする


This is correct. It's 3 numbers. Math!


checkMissingReportTypes() and checkReportFields() return void, so cannot
"&&". Also, each function in a separate line is more clear of the checking
logic undertaken. :)

On Thu, Dec 10, 2015 at 12:14 AM, Gustavo <notifications@github.com> wrote:

> I think you could simplify this by doing:
> return checkMissingReportTypes() && checkReportFields();
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/googleads/aw-reporting/commit/228377db23570d647a31d5c5dc0f5f0b0414b5b5#commitcomment-14877518>
> .
>


Hey Ben,

Good find.  It looks like "net/VRNetServer.h" already includes <winsock2.h>.  Any idea why this file is not being included with VRNetServer.h?  Also, it looks like if we move #include "net/VRNetServer.h" to the top it works.  What do you think is going on?

Thanks,
Dan


我觉得原来的“boss战”比“boss出击”更清楚易懂一些


Nice catch! Done.


TODO: import into new branch


Never try to modify the comments as a comment, explain the function.


I did it this way to capture the actual time spent between sd_event_run() invocations.  Generally, in well-formed programs, that's going to predominately be time spent in sd_event_dispatch().  But as more general instrumentation capable of *actually* measuring what it purports to measure; time spent between sd_event_run() calls, I opted for this approach.

Consider the possibility of something (mis)using sd-event sticking something potentially blocking in the outer loop isolated from the dispatcher, I expect that to be captured in the profile.


I'm afraid this patch is conflicting with @mstembera's "easy fix":
https://github.com/official-stockfish/Stockfish/pull/521
So, it may have to be retested once easy fix is merged...


Why?


This is initial commit that only draws the skeleton of the logic. It's been fleshed in the next commit. :)


Ah okay.  Got it to work and uploaded new boot images (``factor -run=bootstrap.image.upload``) and it looks like bootstrap is working again using ``build-support/factor.sh update``.


TODO: import into new branch                   always print to console, we may change this at a later stage, this one is really important for me.


我觉得“未上锁”必须再三强调，已经有不计其数的人吃过亏了，新的表述很容易看漏


That fixed it. Sorry about the noise.


TODO: import into new branch, but only Anticipate.java


It's been changed in the subsequent commit. :)


虽然我的本意并非如此……不过如果觉得这反而有可能好的话，可以就这个事情讨论一下……


This block of code means to check if current SSID is in the auto-login SSID list the user set.
I don't quiet understand the logic here.


Ok, that should work for the trustlines, but does it check that the XRP payment validates?


TODO: import into new branch, the Anticipate.java file as it is here! (not the commit itself)


I really love this watch face need an hourly chime.  I have been trying to install the SDK and not been able to :-(  I was able to get the project to load on the beta version of cloud pebble, but it will no compile (even without any changes).  Any chance you can post a compiled version with the hourly chime on?  Thanks


delete the spaces please


the same spaces


spaces


quit the trash code


Quit the trash code


trash code


Is it possible that value is an empty list? If so would we end up with ')' but without '('?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.378.0](http://build01:8888/viewLog.html?buildId=13487&buildTypeId=ConfigInjector_CI) is now running


delete spaces


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.378.0](http://build01:8888/viewLog.html?buildId=13487&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 31 Build time: 00:00:36


Good point. I've updated it in the PR :)


Casi rematada la practica


Not intentional, that looks like an simple oversight in simplifying the return paths.


would love to see this information in Webstorm :+1:


ack


Do you think it might be worth making flagged a scope?
Once the scope is defined in cs_comments_service/models/content.rb
you and other parts of the code could get the flagged content as follows:
Content.flagged

http://mongoid.github.io/en/mongoid/docs/querying.html#scoping


Although a better commit message was needed


We need work in this , the url for conference rooms should not be hard coded


In general delete , the spaces and trash code


I was thinking 100, but wanted to confirm before putting it in. If we do 100, then we wont need to change the overall array at all.


Is the device running? If so you can run a command from terminal to tell us all the sepolicy stuff it needs


I was testing some stuff in the analyzer, but didn't make any official changes. I guess I added a space here? Trying to figure out how to do "add", do we want to add a symbol <+> or ++ to add dists? what about adding a num to the dst?


Could put spaces between each attribute


?Hi @andkov, I'm about to submit the revision but noticed that they want figures to be .tiff or .eps format. So if the paper is accepted on this round, they will want us to re-send them the figures as one of these file types, for better quality. I just saved them from Github at .png.

Thanks,

Amanda

________________________________
From: Andriy V. Koval <notifications@github.com>
Sent: 10 December 2015 21:57
To: IALSA/Diabetes-Tri-Study-public
Cc: Amanda Kelly
Subject: Re: [Diabetes-Tri-Study-public] updated graphs (3e5f6e2)

@adkelly<https://github.com/adkelly> , you can click on the image to get to the hi-res, but you can download all figures at once by following the link<https://github.com/IALSA/Diabetes-Tri-Study-public/raw/master/Studies_report.rar> on the main README<https://github.com/IALSA/Diabetes-Tri-Study-public/blob/master/README.md>.

Yes, the figures have been updated in the same report. You may need to clear your cookies to see the update, or refresh a few times, or view in icognito. So far the changes in Fig 1 and Fig 2 involve only removing the formula. Let me know if anything else is needed

-
Reply to this email directly or view it on GitHub<https://github.com/IALSA/Diabetes-Tri-Study-public/commit/3e5f6e25f81e5717625d683d87b8c473451a3359#commitcomment-14917168>.


See https://github.com/JetBrains/intellij-plugins/pull/314

![image](https://cloud.githubusercontent.com/assets/384794/11759240/9f7cb4a4-a02b-11e5-9085-c00459049519.png)


The Web element should be private


Ah ok cool I was wondering about the `with` statement, kept seeing it around, thanks for the tip! I think I have an irrational fear of `with` because of JavaScript


Cool will do, I'm loving the more English sounding comparisons in python


@myronmarston How does this look? I'm planning to add some comments to explain the deferral to the instance. I'm not sure whether the class methods should be all together or next to each of the instance methods, as they are currently. I'm also not sure about the ```# @private``` and ```# @api private``` - are they needed for the class methods?


Good to know, thanks!


bool is only thing returned or checked


this doesn't seem to be used


Yep I started out going back and fixing up previous days once I learned a better way of doing things, but figured it would be cooler to have snapshots of how the code is progressing over time as I learn more about python. So hopefully day 25 will look much nicer than day 1!


Thanks :) trying to get into the habit of writing in a more functional style


Shouldn't default is no reservation for this one too, at least for now?


I guess it's better to cancel notification in the service, as soon as it gets the `COMMAND_STOP_LISTEN` command, see [here](https://github.com/phy25/BIT_NetCheckin2/blob/master/app/src/main/java/org/bitnp/netcheckin2/service/LoginService.java#L119).
Because the LoginService is in charge of collecting and scheduling all the transactions, and invoking cancel function in components around may leads to a logic confusion.

Modify your code, feel free to pull the requests.


jsではアロー関数は使えないよ
疑問に思ったんだけど、 gulpfileを.tsにした場合に初回のビルドはどうやって行うんだろう？


awesome!!! :dancers:


One question to clarify. Otherwise LGTM!


We use nargs = '+' in the argparser to ensure that there is at least one. So an error should get thrown, but I'm not sure now if I tested this so I'll check.


Jest mały problem, po załadowaniu story Employer zawsze (w 99 na 100 przypadkach po dodaniu CRUDU) jest undefined, jak damy na przycisk odśwież to już jest ok - nie wiem, trzeba dodać jakiś waiting?


I suppose the other type parameters should also be considered?

I find it inconsistent that
* we allow multiple arguments to be supplied
* we assume they will all be the same


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


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


added icon_history classes


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


@merichar those can be set in the _/etc/default/concerto_ override file and then you wont have to alter this script.  Line 30 above reads in the override file.


Related to #103.


iteration (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2336 of 2373 tests passed)

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


I learned a bit about how to write readable code..


You can do it manually by opening the images in a paint editor and saving in a different format.  I am not sure that our studio supports producing images in tiff.

Sent from my iPhone, please excuse typing and T9 errors.

> On Dec 11, 2015, at 5:21 PM, adkelly <notifications@github.com> wrote:
>
> ?Hi @andkov, I'm about to submit the revision but noticed that they want figures to be .tiff or .eps format. So if the paper is accepted on this round, they will want us to re-send them the figures as one of these file types, for better quality. I just saved them from Github at .png.
>
>
> Thanks,
>
>
> Amanda
>
> ________________________________
> From: Andriy V. Koval <notifications@github.com>
> Sent: 10 December 2015 21:57
> To: IALSA/Diabetes-Tri-Study-public
> Cc: Amanda Kelly
> Subject: Re: [Diabetes-Tri-Study-public] updated graphs (3e5f6e2)
>
>
> @adkelly<https://github.com/adkelly> , you can click on the image to get to the hi-res, but you can download all figures at once by following the link<https://github.com/IALSA/Diabetes-Tri-Study-public/raw/master/Studies_report.rar> on the main README<https://github.com/IALSA/Diabetes-Tri-Study-public/blob/master/README.md>.
>
> Yes, the figures have been updated in the same report. You may need to clear your cookies to see the update, or refresh a few times, or view in icognito. So far the changes in Fig 1 and Fig 2 involve only removing the formula. Let me know if anything else is needed
>
> -
> Reply to this email directly or view it on GitHub<https://github.com/IALSA/Diabetes-Tri-Study-public/commit/3e5f6e25f81e5717625d683d87b8c473451a3359#commitcomment-14917168>.
> —
> Reply to this email directly or view it on GitHub.
>


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


支持啊
```
$ ss-local --help

shadowsocks-libev 2.4.2

  maintained by Max Lv <max.c.lv@gmail.com> and Linus Yang <laokongzi@gmail.com>

  usage:

    ss-local

       -s <server_host>           Host name or ip address of your remote server.
       -p <server_port>           Port number of your remote server.
       -l <local_port>            Port number of your local server.
       -k <password>              Password of your remote server.
       -m <encrypt_method>        Encrypt method: table, rc4, rc4-md5,
                                  aes-128-cfb, aes-192-cfb, aes-256-cfb,
                                  bf-cfb, camellia-128-cfb, camellia-192-cfb,
                                  camellia-256-cfb, cast5-cfb, des-cfb, idea-cfb,
                                  rc2-cfb, seed-cfb, salsa20 and chacha20.

       [-f <pid_file>]            The file path to store pid.
       [-t <timeout>]             Socket timeout in seconds.
       [-c <config_file>]         The path to config file.
       [-i <interface>]           Network interface to bind.
       [-b <local_address>]       Local address to bind.

       [-u]                       Enable UDP relay,
       [-A]                       Enable onetime authentication.

       [--fast-open]              Enable TCP fast open.
                                  with Linux kernel > 3.7.0.
       [--acl <acl_file>]         Path to ACL (Access Control List).

       [-v]                       Verbose mode
```


これ、ややこしいですがそちらの挙動は問題ないと判断しました。
Settings.Instance.NWServerIPは(名前に反して)ホスト名に対応する仕様のようなので
ConnectCmd()内での空文字列はWindowsの既定通りローカルアドレスになるのは問題ないと思います。
今回問題としたのはMainWindow()の
`cmd.SetNWSetting("", Settings.Instance.NWServerPort);`
の空文字列で、こいつがローカルアドレスに変換されていましたが、SetNWSetting()は純粋なIPアドレスのみ受け入れる仕様なのでこの点を修正しました。
// なので実はSetNWSetting()の第一引数をSystem.Net.IPAddressにするのがより賢明です。今気づいた


@vlj Voila, c'est bon, c’était compliqué, dis moi si tu trouve ca tout bon :)


position: absolute;


align with spaces to avoid weird indentation issues like this


Yes. An error will be thrown in parse_args() call if no value is provided. But I am not sure when default values are used.


@vlj le seul truc que j'ai pas compris c'est la, normalement je devait mettre "getTargetIRAnalysis" mais rien a faire, impossible de le mettre le "get" avant, et vu que comme ça, ça marche.


Additional commit message for this commit: This version also works for combined english words such as `lawsuit`. if `lawsuit` is the only parsed english words from the url and it is not assigned for any existing url, then prefixes and suffiexes of 'lawsuit' are searched. Upon first find it stops and return the keyword. Later the keyword is assigned the corresponding url in db


I'm just passing req.user to the renderer. req.user.username is a prop there.


Is the MT-FULL case taken care of here?


MT-FULL case may need more logic


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1983 of 1984 tests passed)

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


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1987 of 1987 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)


MT-FULL case may need more logic. Since the encap profile names, shutdown state etc need to be managed


Correct, but I'm guessing this is the cause of the merge conflict as other things in this file have changed.

What I was also saying is that yes, username is a property, but it isn't something that we set in registration so for most users it will be undefined unless you've set it manually in the database.


Very nice :)


ack


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 68](https://teamcity.labkey.org/viewLog.html?buildId=291194&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:15:05


Fine by me


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


There is a `.py~` file - its a temporary file from some editor. That should not be commited


Doug,

I did not check if it is related to this commit, but it seems that we need minor improvement!

There is a minor display issue related to this module: counter value does not return the right value for matching entries. https://gramps-project.org/bugs/view.php?id=9057

Have a working patch:
```diff
--- a/gramps/gui/views/treemodels/treebasemodel.py
+++ b/gramps/gui/views/treemodels/treebasemodel.py

@@ -589,6 +589,7 @@ class TreeBaseModel(GObject.GObject, Gtk.TreeModel, BaseModel):
         pmon.add_op(status_ppl)

         self.__total += items
+        skipped = 0

         with gen_cursor() as cursor:
             for handle, data in cursor:
@@ -598,10 +599,13 @@ class TreeBaseModel(GObject.GObject, Gtk.TreeModel, BaseModel):
                 if not handle in skip:
                     if not dfilter or dfilter.match(handle, self.db):
                         add_func(handle, data)
-                    self.__displayed += 1
+                    else:
+                        skipped += 1
         status_ppl.end()
         status.end()

+        self.__displayed = self.__total - skipped
+
     def add_node(self, parent, child, sortkey, handle, add_parent=True,
                  secondary=False):
         """
```
but it is a quick workaround...

I suppose that I can guess the expected logic, but I do not understand why skip list is always empty or how to properly fix the counter: self.__displayed.

Cursor stuff or change like:
```
                 status_ppl.heartbeat()
                 if not handle in skip:
-                    if not dfilter or dfilter.apply(self.db, [handle]):
+                    if not dfilter or dfilter.match(handle, self.db):
                         add_func(handle, data)
                     self.__displayed += 1
```
sound like customization or optimization methods?

Currently, I can just report that matched entries are ignored
by the counter from TreeBaseModel only. It works for Search
or for Filter on FlatBaseModel.

Also, maybe there is a cosmetic indentation issue on this module?
```diff
--- a/gramps/gui/views/treemodels/treebasemodel.py
+++ b/gramps/gui/views/treemodels/treebasemodel.py
@@ -568,9 +568,9 @@ class TreeBaseModel(GObject.GObject, Gtk.TreeModel, BaseModel):
             # The tree has both primary and secondary data. The navigation type
             # (navtype) which governs the filters that are offered, is for the
             # secondary data.
-                items = self.number_items2()
-                _LOG.debug("rebuild filter secondary")
-                self.__rebuild_filter(dfilter2, skip, items,
+            items = self.number_items2()
+            _LOG.debug("rebuild filter secondary")
+            self.__rebuild_filter(dfilter2, skip, items,
                                       self.gen_cursor2, self.map2, self.add_row2)

     def __rebuild_filter(self, dfilter, skip, items, gen_cursor, data_map,
```

Jérôme


you can see on top references.


Resolve #2


!fixed


This is probably paranoia, but can we change it from just webpack to `$(npm bin)/webpack`?


How did you figure out that init() needed to be done inside setup()? What could be the reason why it doesn't work from constructor? Anyway, thanks for the update. It was useful for me.


Space inside square brackets detected.


Extra empty line detected at block body end.


Inconsistent indentation detected.


Extra blank line detected.


Extra empty line detected at block body beginning.


Space missing inside }.


Looks like a nice idea.


We still need a .babelrc don't we?

```js
{
  "presets": ["es2015", "stage-0", "react"]
}
```


Surrounding space missing for operator '='.


@jacopoc Do you also propose to move the view definitions like ? -
```
<entity-resource type="model" reader-name="main" loader="main" location="entitydef/entitymodel_view.xml"/>
```


'stream' - не очень подходящее название для переменной, в которую сохраняется введённая пользователем строчка. Лучше подойдёт 'input'.


SF


@kingjan1999 Kann man die App fertig machen? Sie ist doch schon so weit :D
Wäre schade diese nie zu veröffentlichen


I am a nice comment


@okungnyo In the main page says 10xx commits, hence you go backwards. ;)


【经验公式】


Very Good @Zealar another time :+1:


Аналогично здесь. По смыслу это команда, которую ввёл пользователь. Лучше подойдёт 'command'.


@anthrotype Hey, will you submit a PR upstream for all of this? Thanks:


@Quintus The links are switched and one is missing.


Why force `PROFILE_TYPE_ADDRESS` to use an address from the profile directory? This command is only useful when you want to load a profile that is not in the directory (in RAM for example).


FAQ 82 should maybe mention that white/blacklists always override the function restriction instead of the deny button. If an item is already whitelisted, no popup should appear, therefor there is nothing to override.


@yorzi, @robwise is correct!


一些android功能及其总结


There is no project files. As far as I can see there is only one readme.rd file. How can I try this control?. Thanks.


Lack of newline


I think jpm install actually supports multiple modules at the same time.  Obviously not clear at all from that message with it's plural and singular mix :)


Weak help


Does this title method exist?


I knew it! Greedy bastards :D


:joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy:


But maybe 1 cup instead of 1/2?


Sounds good


Please combine sentences.


I've put juce in as a submodule - I think - now so you shouldn't need to do this.  Just do a `git submodule init && git submodule update juce`


nice, ack


Space inside square brackets detected.


Extra empty line detected at block body end.


Inconsistent indentation detected.


Extra blank line detected.


Extra empty line detected at block body beginning.


Space missing inside }.


Is there a reason why you broke this line? I'm pretty sure it was below 80 chars before, we automatically ensure that. (You can usually set your editor to show you a line at the 80 chars so you know where that is.)


same here


Please add comment that this is 5 min.


I stand corrected, @alexdowad :)


We don't have to go to the DB to get the user. Its ID can be accessed through `User.GetUserId()`. Please refactor.


It's a python str built in method!


@an0n981 Hmm... I think that's really an additional issue worthy of another FAQ item.  I'll get busy on that.  Note that the popup will appear for certain types of items (contacts, for example) since they behave differently than other types of items (URLs, for example).


Of course I will, i just forgot ;)
On 12 Dec 2015 10:16 a.m., "Adrien Tétar" <notifications@github.com> wrote:

> @anthrotype <https://github.com/anthrotype> Hey, will you submit a PR
> upstream for all of this? Thanks:
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/daltonmaag/feaTools/commit/98aae9d5d8f6748aaf9e80198b43bae7bed9c78b#commitcomment-14940643>
> .
>


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


Extra empty line detected at class body beginning.


Final newline missing.


Extra empty line detected at module body beginning.


Extra empty line detected at module body end.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.389.0](http://build01:8888/viewLog.html?buildId=13502&buildTypeId=ConfigInjector_CI) is now running


I guess it fails when called on None, haha. Shouldn't a None styleMapStyleName be handled gracefully? The original ufo2fdk requires only familyName and styleName, a minima.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.389.0](http://build01:8888/viewLog.html?buildId=13502&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


@patrick91 wrong translation


Celebrating 100 commits!!!
@Karawasa @MNoya Not sure what to do with the top bar. Maybe add a health bar as well?


真没有。。快哭了


Missing space after #.


Oh yes, None is no good.
On 12 Dec 2015 10:28 a.m., "Adrien Tétar" <notifications@github.com> wrote:

> I guess it fails when called on None, haha. Shouldn't a None
> styleMapStyleName be handled gracefully? The original ufo2fdk requires only
> familyName and styleName, a minima.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jamesgk/ufo2ft/commit/a927f5cf8bbd8ef38cf8df914fe81136370f57a4#commitcomment-14940706>
> .
>


I'm not very fond of this overriding a builtin. If you do `from ufo2ft import compile` then you can't use the original compile anymore.


Пожалуй, необходимости в отдельном классе под этот метод нет. Этот метод больше связан с пользовательским вводом, чем абстрактной консолью. Следовательно, лучше просто перенести этот метод в класс UserInput.
В качестве названия методов лучше использовать глаголы, описывающие то действие, которое выполняет данный метод. В данном случае лучше подойдёт 'printCommands'


There's a mistake


"Match Client"


I


I again


Adding new variables in middle of code is not allowed.


Shouldn't this whole function use getAttrWithFallback for all attributes to start with?


ack (for gitmate)


No, the solution is clever. I thought if you keep the hints like this, then you might want to add an extra solution line though.


unack, documentation comment needs shortening


ack


unack, needs work


Идея с рекурсией мне нравится, но в jvm, к сожалению, не реализована оптимизация хвостовых рекурсий. Из-за этого будет непрерывно расти стек программы и в крайних случаях это будет приводить к исключению StackOverflow. Надёжнее будет реализовать цикл, в котором будет производиться повторяющийся вывод списка команд и выполнение введённых пользователем команд.


No problem :)


You still add `py~` ^_^


The second commit deletes the temp file - but adding it is not necessary int he first place. squash these


@mgorny is this better?


Guys, I'll push PR today to tutorial incl babel 6 update, so you can use it. Please wait until this.


From my findings, it uses DCT and Huffman, hence the complicated look.
See https://github.com/lioncash/ExtractData/blob/master/Extract/cpz/JBP1.cpp for reference.


That's something new. Are those binaries built for evm?


There is a problem with plain `is_scalar($html) === FALSE` and `$html = '';` that when you pass eg. an object that can be cast to string via `__toString()` method, this object will be then replaced with an empty string instead of properly converted. This is IMHO far more dangerous BC break than throwing an exception (which would notify the user to covert the object to string before passing it to the `WriteHtml` method).


unexpected token tCONSTANT


Extra empty line detected at block body end.


Inconsistent indentation detected.


Resolves https://github.com/srph/qp/issues/2


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


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2336 of 2373 tests passed)

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


what is <hp> tag?


unexpected token tLSHFT


unexpected token tEQQ


unexpected token tRSHFT<br>unexpected token tIDENTIFIER


Actually now that we have so many headings,.... what you can do is:

```
     <h2 id="should-i-accept-notifications"><a href="#should-i-accept-notifications">Should I accept notifications?</a></h2>

```

Then you can copy the link and send that directly to people.


Need to read fakeCC for add_wishlist_pricehistory as well.

See #864.


That is an excellent observation. I have put very little effort into making beginner levels. I think @gaminginky had some promising-looking levels for those players interested in fun rather than brain ache.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.390.0](http://build01:8888/viewLog.html?buildId=13503&buildTypeId=ConfigInjector_CI) is now running


```<hp> ```


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.390.0](http://build01:8888/viewLog.html?buildId=13503&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:38


Add a link to ```<a href="/login/">registering</a>```, and link to the forum, and link to github issues.


I'd add something like:
While PyInstaller supports Reproducible Build as much as it can, you need ...
... need to to do some thing yoursefl
... need to support it
... do thing, Pyinstaller can not do.


Are you really sure that  this change doesn't cause issues? Basically this would remap channels when doing passthrough.


![accept accept accept](http://mincore.c9x.org/acceptcat.gif)


:+1: I'm now wondering if the compiler should have even allowed you to export functions which refer to `Label` without exporting the type synonym itself, though.


Of course you need to document the value of PYTHONHASHSEED you used. Otherwise others can not reproduce your biuild. Whis is why we recommend setting PYTHONHASHSEED to `0` or `1`


typo


# Ehi zemel #
*How ya doin niggah*


Hello , How to disconnect MQTT client in When mobile network removed


Files should end with a trailing newline


New update for the item list fixed item listing and fixed rail item bugs.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.929](http://teamcity.monogame.net/viewLog.html?buildId=12151&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


@radiospiel Thanks for bringing the issue up in the first place! We have all learned something!


that one was a typo obviously. It should add 1 to the score if bitsperchannel of the formatDesc is >  CAEUtil::DataFormatToBits(format.m_dataFormat) - could you  add this else if with the proper ">" comparison?


Hm, good point... sounds wrong to me, too.


:+1: Good! Whenever you're adding new gem to Gemfile it should be always commited with `gemfile.lock`


First of all, this method is pointless, you can put whole `ArrayList` into Bundle with `putSerializable`

Also this method has other design problems (just drop method, but note the points for future):
* `b` is very undescriptive parameter name
* `b` parameter switches between two class fields, which could be passed directly
* `setBundle` field is used as parameter, it's accessed only in this method and it's caller and is completely unnecessary as You could just pass `outState` as parameter (and don't bother with `putAll` in `onSaveInstanceState`)


:+1: fot arrays please don't use spaces inside.
`[:pending, :rejected, :accepted]` would be ok!


If you're passing the config why not just check it in the function?


currentData and compressedData are misleading names here


Comments should begin with `//{white space}` - this is not a commented code.


new fix


Please push migrations in separate commits ;-)


Please refactor to `this.UserProfile.Id` as commented minutes ago.


Before we let the devEnum object decide if we will do passthrough. With your change you will assume passthrough even if there was no suitableformat found for passing through. (devices which only have 24 bit streams for example - some odd studio equipment or what not). Imo this still needs to be ensured. If there is no passthrough format found we need to fallback to AE_FMT_FLOAT as we did before.


Before commiting the code which is generated by some gem it's worth to check it and modify eventual code smells. For example:
```ruby
has_and_belongs_to_many :users, :join_table => :users_roles
```
check to new syntax:
```ruby
has_and_belongs_to_many :users, join_table: :users_roles
```


Ah ah I love it :D


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 2 to 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 4 to 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


I am fixing it right away


Typically, we use `:start_date` and `:end_date` for such purposes :) Just a tip.


["initial" = "początkowy" (or "podpisać inicjałami")](https://translate.google.pl/#en/pl/initial)
You probably meant ["initialize"](https://translate.google.pl/#en/pl/initialize)


Though I like your structure to put each directive in a separate file, I think this could become a little lengthy here. Maybe we should reduce to one module 'starter.directives'.


@indutny Hi, I think you forgot to backport this after the upgrade to V8 4.7.


Remember also that it shouldn't be possible to add new project that its `:setup_date` starts in the past.


Sweet!


:+1: Good, always commit migration with schema together :)


imo this needs to go into the else branch of

if (format.m_streamInfo.m_type != CAEStreamInfo::STREAM_TYPE_NULL)

else we score twice and maybe even wrong...


Simple but great, thanks.


Extra empty line detected at block body end.


Inconsistent indentation detected.


unexpected token tCONSTANT


Note: f8d5dc06a6a04c8af7c68b7e1519caa844407a9f will redefine BROKER_DEFAULT_EXCHANGE as "".


ack - lets leave the pypy specific code in `deps.osx.sh` - as it would help if we want to add `pypy` back later


ack


hi


Why using a different variable naming here?


done


There is no need for creating these controllers:
- confirmations_controller.rb
- omniauth_callbacks_controller.rb
- passwords_controller.rb
- sessions_controller.rb
- unlocks_controller.rb
We only need: `registrations_controller.rb` which is specified in the `routes.rb` file:
```ruby
devise_for :users, controllers: { registrations: 'users/registrations' }
```


if(a==0) e non if(a=0)


It's ok to use generators but just notice how many files were added that are empty. Actually we do not need them now. For the future, I recommend to use generators only for generating migrations and creating all other files manually.


Please fix all warnings created by hound.


I MADE SO HARD RETXTURE nei_sprites.png. Previous version was poop!


Final newline missing.


What i know of LGS G500 metadata:
Start right after the profile (78 bytes after the beginning of the page) with a 5 byte "LGS02" marker.
Then 29 bytes whose I don't know the meaning (they are zeros, maybe an empty profile name).
Then comes 11 strings (the first two buttons can not be assigned macros in LGS and thus they don't have names).
Each string is 17 wide characters (big endian) counting the null terminating character (so maximum length is 16). Despite using wide characters, LGS uses ISO-8859-1 encoding (or something similar), it may depend on my locale.

I think that @phomes mentioned that on the G9 profile names are stored on page 1.


Unused block argument - `config`. You can omit the argument if you don't care about it.


I tested this out and it works pretty well. The one thing I might suggest is:

1. Labels that have a child with `Force_breaks_rec` or `Always_rec` in either their "left" or "right" component, should propagate upwards as they do.
2. However, the label *itself* should likely only be forced broken if the *right* component has some propagated breakage.

I'll add a test case to explain why I think this is the right convention, and I think I know how I would implement this.


Just a small tip for the future. We will be probably quite often fetching Project objects from database by defining `:owner_id` so it would be good to add index for that column.


Anything to be done on Syro's side to make this case easier to handle? Maybe there could be a `not_found` method?


Yeah, you cannot use backtics (`) in bash shell :)


Inconsistent indentation detected.


syntax error, unexpected '='

*Detected by PHPLintBear with severity 2 while running section `php`.*

*The issue occurred in the file `test.php`, lines 4 to 4.*

```

a = 4;
```


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*Detected by SpaceConsistencyBear with severity 1 while running section `php`.*

*The issue occurred in the lines 5 to 5.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.php
+++ b/test.php
@@ -2,5 +2,5 @@
 $a = 5;

 a = 4;
-
+
 ?>
```


workers_scaling_active definiuje wyłącznie czy w ramach tego Eksperymentu działa jakiś algorytm skalujący. Potem ta informacja pozwala stwierdzić na przykład, że eksperyment jest automatycznie zarządzany, ale algorytm nie istnieje, więc wystąpił błąd.


I think I'll add all the missing methods to Syro in the next minor release, sounds good?


Good point with the `ArrayList`, saving the current state with this little pattern is what I did in the `FeedFragment`, the difference was I was using a `List` and it's not possible to put whole `List` into a `Bundle`, so I didn't really try casting it to an `ArrayList` and apparently it can be done this way. I will also fix this one with next commit of OfflineData.
Nice to know.


umm ... I have local changes which I haven't pushed already fixing this. okay, I'll cherry-pick


Prawdopodobnie by wystarczyło. Trochę mnie powstrzymał fakt, że w module WorkersScaling mamy już klasę o takiej nazwie. Nie będzie żadnych kolizji, ale chciałem wprowadzić rozróżnienie. Mogę zmienić.


W kilku miejscach używamy już tej nazwy - jeżeli chodzi o zwracany klucz. A jeżeli chodzi o nazwę metody, wprowadziłem konwencję, że metoda ma nazywać się tak jak parametr w zapytaniu, który z kolei w większości przypadków nazywa się jak klucz w zwracanym JSONie, który mu odpowiada.


I do low level parsing and handle null without troubles. My major concern is null for arrays as it means 2 different things.

For object there's already null in 2 or 3 not often used places if I remember correctly. (But not sure after Frodo)

But for getItem when we ask for properties that does not exist they are not returned. Same for getDirectory currently it returns error for non existing folder.

This would lead to 3 different result for the same source : requesting non existing data.

I think if there's a major change in API version (7.X) then moving to null for non existing requested object is possible (but please avoid for arrays or be very clean on definition / documentation). But if we are moving to null return = no existent value (That is different from empty value) it should be the case also in all other places (like getItem).

In fact the more I think the more I do not like null, as it's hard to know the meaning, is it empty or error.

IMO this is a larger question about JSON api future and supporting null as a meaning or not.
Then define clearly if null is error or empty data then update the whole API to match the new design.

Now for my point of view null should be avoided. API should be more or less like it is :
Empty data when data does not exist, for optional data, do not return if they do not exist, and error when trying to fetch impossible to get data.


Hi,
I am looking for this firmware.
I have bought used T5220 SPARC but can not download fresh  soft from ORACLE. Can you advice me what to do.


It didn't ask you to modify the `+old+` ?


I cannot actively pull this commit (or I don't know how). Can you create a pull request for this commit?


non ancora risolto , c'è un errore sul casting


And was the missing space after this comma also not reported ?


Nice idea, seems so


下标？


我说的是socks5 的udp功能 SocksCap测试失败 redsocks的udp转发也是失败

2015-12-12 9:49 GMT+08:00 Syrone Wong <notifications@github.com>:

> 支持啊
>
> $ ss-local --help
>
> shadowsocks-libev 2.4.2
>
>   maintained by Max Lv <max.c.lv@gmail.com> and Linus Yang <laokongzi@gmail.com>
>
>   usage:
>
>     ss-local
>
>        -s <server_host>           Host name or ip address of your remote server.
>        -p <server_port>           Port number of your remote server.
>        -l <local_port>            Port number of your local server.
>        -k <password>              Password of your remote server.
>        -m <encrypt_method>        Encrypt method: table, rc4, rc4-md5,
>                                   aes-128-cfb, aes-192-cfb, aes-256-cfb,
>                                   bf-cfb, camellia-128-cfb, camellia-192-cfb,
>                                   camellia-256-cfb, cast5-cfb, des-cfb, idea-cfb,
>                                   rc2-cfb, seed-cfb, salsa20 and chacha20.
>
>        [-f <pid_file>]            The file path to store pid.
>        [-t <timeout>]             Socket timeout in seconds.
>        [-c <config_file>]         The path to config file.
>        [-i <interface>]           Network interface to bind.
>        [-b <local_address>]       Local address to bind.
>
>        [-u]                       Enable UDP relay,
>        [-A]                       Enable onetime authentication.
>
>        [--fast-open]              Enable TCP fast open.
>                                   with Linux kernel > 3.7.0.
>        [--acl <acl_file>]         Path to ACL (Access Control List).
>
>        [-v]                       Verbose mode
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wongsyrone/openwrt-1/commit/fb5187a99c84a7474247c77ae42039f4a11f979d#commitcomment-14938795>
> .
>


```
def post
  @post ||= Post.find(params[:id])
end
```


ack


Thinking about it! :grinning: Maybe it would be good as a sample project for squidlib-util. I'd need to make some more art for it, though.


`'Access-Control-Request-Method' => '*'`


the input format channel map has only RAW channels. Those have to be mapped to the chosen output format


Also, don't cast, change field type to `ArrayList` to clearly indicate that no other `List` type can be used here


Files should end with a trailing newline


не, это вообще не нужно


Extra empty line detected at block body end.


ok


No. But onDeath, $entity->kill() fixes  it.


:+1: Yeah, Hound is right here :)


@IonicaBizau  I'm trying to push strings to a variable. Is that wrong?? :cry:


if AE wants to open a sink for passthrough you can either do so or fail. pretending success and opening pcm is no option.


Final newline missing.


You have commited a schema.rb with a conflict. It should have been resolved before you do a new commit. To resolve conflicts in schema.rb just run `rake db:migrate`.


Yes, I plan to refactor that. Not yet sure which way is better.


right


You have commited conflicted file. I should not have happened. Before commiting all conflicts should be resolved. Remeber about that.


Hm... it's about 8mb, not 800mb.

```
> (percent['3'] - percent['2.5']) / 100 * sum(as.numeric(seqlengths(hg19Ideogram)[chrs]))
       3
-8279155
```


I think `mkdir -p` would be enough, no need for another dependency


That's ok, but you should have done it before :)


Ahh ok - so we fail and AE will initialize again with AE_FMT_FLOAT then? fine then ...


Can you please remove this file and use just `json-formdata.js` in the build script?


Yeah, that a good idea! :+1:


![Why have you no tested IPv6?](http://cdn.meme.am/instances/64570355.jpg)


I had a fast look at the patch and it looks interesting. I'll put some tests on it later to be sure it's ok.


Yeah, that's a good idea to define this method :) :+1:


This breaks the editor directive, it loses it's scope


Use 2 spaces for indentation in an array, relative to the start of the line where the left bracket is.


Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.


Do not write to stdout. Use Rails' logger if you want to log.


There is a better method defined in `devise` gem which is `user_signed_in?`.


:+1: this makes me happy.


Thanks for the suggestion! I decided to still provide `runFlareShow` for testing/debugging.


You can build like this: `mkdir -p dist && browserify src/json-formdata.js -s JSONFormData  -o dist/json-formdata.js -t babelify --presets [es2015]`

`-s JSONFormData` will tell Browserify to create UMD module, which exports `JSONFormData` class.


Да, пожалуй, так будет удобнее.


Extra empty line detected at class body beginning.


You could define a helper method, for example:
```ruby
def admin_signed_in?
  user_signed_in? && current_user.admin?
end
```


Extra blank line detected.


Extra empty line detected at block body end.


Trailing whitespace detected.


`+' interpreted as argument prefix<br>unexpected token tINTEGER


Final newline missing.


If you have `:belongs_to` relation then you have to include a corresponding column in database. Here:
```ruby
:project_id
:user_id
```


Thanks.


Dropdown menu arrow color... not anchor color.


1 what?
```ruby
1.month
1.week
1.year
etc.
```


`+' interpreted as argument prefix
unexpected token tINTEGER
unexpected token tRPAREN


unexpected token tRPAREN


materialize.css（の読み込み）を削除してはいけません。


We have a macro for that 0, it's called VAR.  Please use that and remove the comment.


Looks good.  Thanks.  Please address the minor comment and I'll merge.


Calendarioがbowerの管理対象になっていません。（`bower.json`の`dependencies`に定義がありません。）
 `./node_modules/.bin/bower install https://github.com/deviprsd21/Calendario.git —save`
してください。


SECURITY BREACHED!


これ、閉じタグで合ってる？


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 842](https://teamcity.labkey.org/viewLog.html?buildId=291199&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30 (8 new), passed: 217 Build time: 00:26:08
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testImmuneSpaceStudySubset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testNavigationDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testBackDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCardStudyLinks: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testFinderWebPartAndActionShareFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testRefreshDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

##### there are 20 more failed tests, see build details
```


Hi Pri,
Are you going to make more changes on this branch (histograms)?
The Python wrapper looks far from finished and inconsistent with the C++ header. If you don't mind, I could do an iteration on it.
Igor


Is parity tag global enough to be placed in common Constans file? (And if so, then the comment is too generic)


actually the interface to ae has a flaw because it can't signal back to player (the was also before ActiveAE):
https://github.com/xbmc/xbmc/blob/master/xbmc/cores/AudioEngine/Interfaces/AE.h#L155

this function "should" never fail but the interface designer stays silent for the nasty case it does fail
on the list of trillion things that needs to be changed :)

```

  /**
   * Creates and returns a new IAEStream in the format specified, this function should never fail
   * @param audioFormat
   * @param options A bit field of stream options (see: enum AEStreamOptions)
   * @return a new IAEStream that will accept data in the requested format
   */
  virtual IAEStream *MakeStream(AEAudioFormat &audioFormat, unsigned int options = 0, IAEClockCallback *clock = NULL) = 0;
```

if opening of sink fails, it just opens the null sink


ah ok, how about "add and install module(s)"


I can't compute all s_curve with this value of cst_rad!
???


このjQueryの読み込みは不要です。16行目で読み込み済です。


LLR: 2.95 (-2.94,2.94) [0.00,5.00]
Total: 19739 W: 2910 L: 2721 D: 14108
sprt @ 20+0.2 th 7L

With how many threads should I run the next test?
I propose 20 threads stc.


It's encrypted anyway :)


faviconは無いのでこの行は削除してください。


You added this file by mistake I think


@triblework I changed the files before pushing to the live server but now the subdomain has gone missing. Any ideas?? :crying_cat_face:


Please add "WTL folder/include" path in the project of proxydomo.


Could you delete these lines you've added ? I don't think they are necessary.


One example macro looks like that:

```
    <xml name="dbselect"
        token_name="db_select"
        token_label="NCBI Database to Use"
        >
    <param name="@NAME@" type="select" label="@LABEL@">
```
And you use it like this:
```
   <expand macro="dbselect" name="db_select_from" label="From NCBI Database" />
```

See here: https://github.com/galaxyproject/tools-iuc/blob/master/tools/ncbi_entrez_eutils/elink.xml#L29


Thanks again :smile_cat:


Hi there,
I've implemented configuration via openvpn config file (at last!!!) please feel free to use =)


Hey @F3PiX – I don't recall the issue exactly but I guess you're right: with a default value in place, deleting a season shouldn't be a problem anymore.


Why you no!


@marek-safar there's still one test that fails (apparently only on 64bit mono):

```
Loading verification data from `ver-il-net_4_x.xml' ...
gtest-230.cs...	KNOWN ISSUE (Compilation error)
test-643.cs...	REGRESSION (SUCCESS -> EXECUTION ERROR)
Wrong return code: 2
Done

2494 test cases passed (99.92%)
1 known issue(s)

The latest changes caused regression in 1 file(s)
test-643.cs
make[1]: *** [qcheck2] Error 1
```


The download links seems wrong, i.e. ...linux-64bit.tar.gz instead of ...linux-amd64.tar.gz.
(https://github.com/openshift/origin/releases/download/v1.1.0.1/openshift-origin-client-tools-v1.1.0.1-bf56e23-linux-64bit.tar.gz)


Use `&&` instead of `and`.<br>Use a guard clause instead of wrapping the code inside a conditional expression.


ок, этот  вариант лучше, но я сначала должен вспомить не сломает ли это что то. в любом ли контексте сработает. ты тоже погоняй пока.. если все будет ок -- замержим.


Looks like the test has a bug, I get the same result on MS.NET.


why that?


why moving everything around? Looks suspicious...


Because the pyenv in circle doesnt have 3.5.1 but the latest pyenv has it.
This command updates pyenv to latest version (it's the way to update pyenv)


Hm. seems like my commits are mixed up. give me a moment.


arfff... push next time :)

let me know when I can try again


It does, but it seems to work... Do you fancy submitting a PR with the updated link?


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 32](https://teamcity.labkey.org/viewLog.html?buildId=291196&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12, passed: 36 Build time: 01:04:36
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

##### there are 2 more failed tests, see build details
```


Since you wanted the textures in here there wasn't a reason to revert it again?


oops. I meant @bencz


Still doesn't fix it. Play services FC while using opengapps...


ack


More seriously, issue #225 is what we are talking about. This commit is just a hotfix.


#说明
>本项目采用jersey2.9+spring3.2.5+hibernate4.2+tomact7.0+log4j加强版
>
>部署和安装省略。

>该实现可以为一个jersey实现的模版。
##jwt的实现
>关于jwt的实现，参考我的csdn [http://blog.csdn.net/li563868273/article/details/50277359](http://blog.csdn.net/li563868273/article/details/50277359 "csdn")
##数据库表
>数据库表参考sharebook.sql
##具体访问api

>参照各种资源类的上面的注


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


I wonder if it would have any (noticable) difference in speed, if one `replaceAll()` was used instead (e.g. with `/\r\n?/g`).


Очень хороший комментарий к коммиту, вас такому в гехтсофте учат?


Guess I should try them all. Thanks!


Ha, shit. I wonder how it even worked when I tested it. Thanks! :)


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 10 to 10.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 43 to 43.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 60 to 60.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 69 to 69.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 86 to 86.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*Detected by SpaceConsistencyBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the lines 64 to 64.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/results/SourceRange.py
+++ b/coalib/results/SourceRange.py
@@ -61,5 +61,5 @@
     def overlaps(self, other):
         if self.file != other.file:
             return False
-
+
         return self.start <= other.end and self.end >= other.start
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 11 to 11.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 70 to 70.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 87 to 87.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2337 of 2373 tests passed)

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


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 13 to 13.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (80 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 171 to 171.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


did you handle the case where a new file gets added, i.e. a file is not in the original_file_dict and thus will not be in the dif_dict? (I think you handle that but want to be sure) Did you test that?


you can put this stuff into a function, you basically do two times the same here with different variables, would also shorten your function *a lot*


Чем лучше? Сейчас окончание соотвествует общему стилю локализации прав пользователей.


we're dropping a space here.


maybe this is simpler:

```
if not basics_match:
    continue
if source_ranges_match:
    ...
```

less indentation


IMO easier to follow


Extra empty line detected at class body end.


Use 2 (not 0) spaces for indentation.


Trailing whitespace detected.


Space missing after comma.


> Should'nt that be case 'sqlite': and so on? You already used $serendipity['dbType'] for the switch.

Yes, you're right. Thanks.

> You could as well use an easy if

I wanted to have that in a switch, the if structure was a problem before.


We should add part of this commit's changelog to the toplevel README.md file.


Head only formula for people who can't wait.

![mongodb-1.2.0-beta]
(http://evrima.net/tmp/homebrewphp-700-mongodb1.2.0-beta.png)


why have you done that? is there a benefit from using filters rather than functions?


When I read this it looked like one would be able to select files based on their owner (obviously only if running as admin) and thereby filtering in files from that user only. On further reading it appears that the information is available but not being acted on. This could be one useful addition.


Congratulations. For such quick pass I think that anything below 20 cores is not interesting. However, you can't go much above this neither. Currently we have 21, 23, 23, 23 cores in the framework. Maybe to go with 21?


informative comment, I need that more often! :100:


@jammin84 is there other changes you've made that are not here as an open pr?


I would propose 17 threads due to many users renting such machines which will make the test finish faster.


We usually don't do an empty line after the comment of a function, did we write that down somewhere? (In that case we need to fix it.)


you can use just `redirect_to dish` here and for `create` method


use Slim gapps or Banks Gapps
Open gapps has some issues
for me only 27th nov open gapps works fine
all others after that borked
use these
https://www.androidfilehost.com/?w=files&flid=41043


Inconsistent indentation detected.


Use 2 (not 1) spaces for indentation.<br>Tab detected.


Tab detected.


Final newline missing.


unack, did you see our standards for commit messages? http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits/#how-to-write-good-commit-messages

We're getting to it :) The first patch I wrote is still not applied ;) (that's actually true, somewhen after the third revision they ignored it and didn't continue review :P)


We should only check for the player whose turn it is. It is OK to not have valid moves if it isn't your turn.


note that tests fail because of trailing whitespace here.


so I'd recommend remove this line :)


think about internationalization for button's label (the same below)


Если есть общепринятый стить, то действительно лучше ему следовать.


you might want to check out this branch which adds some more hardware support for X710 NICs: https://github.com/PrincetonUniversity/MoonGen/tree/dpdk2.1


It's not the best solution. Please use just explicit value for migration next time.


Please don't push commented out code to the repository


Indent the first parameter one step more than the previous line.<br>Space missing after comma.


Ok, it's fixed later


I would rename the method to potential_moves and remove this line.


Too big commit. Try to split all the changes to smaller pieces next time ;-)


received


This is specialy not meaking any sense but this is to find out how to document the code  - changes


`Rails.logger.&` is a better solution ;-) for sure


17 STC and if it passes, then pull request. I agree with binky123 that the difference between stc and ltc is meaningless on 17 cores. But maybe to go with regular STC: 10+0.1?


Maybe you should have separate issues for each task still in development and assign those to team members responsible for them?


There is a conflict here! Shouldn't be here.


Trailing whitespace detected.


Unused block argument - `config`. You can omit the argument if you don't care about it.


Final newline missing.


Extra empty line detected at block body end.


Inconsistent indentation detected.


Use 2 (not 1) spaces for indentation.<br>Tab detected.


Tab detected.


Extra blank line detected.


Spec path should end with `dishes_controller*_spec.rb`


nice :+1:


:+1: But this conflict shouldn't have appeared here.


:+1: Hound is right here :)


I would consider moving this into chess_piece so it can be shared with valid_moves. Then we can override it in individual pieces for performance if we like(King already does this I believe)


:+1: Great start.


Смотрю патч,  а почему не была использована game_err_msg? (Изнутри res_not_found_err_msg)
Она формирует буфер err_msg и в том числе обрезает сообщение если это нужно.


Please rename file.


Please remove commented out code.


Oh, also, there should be a space between the ":" and the "Counter". In the throw below too. Yes, a very minor nitpick, I know. :P


Im Prinzip ist sie das, ich wollte eigentlich nur noch:

- Offline-Speicherung hinzufügen, aber das ist zweitrangig, weil ich noch überlegen muss, wie ich das am besten bewerkstellige (Offline-Spiegelung der gesamten DB ist bswp. keine Option)

- Mich nach einem besserem Server umschauen, aber auch das ist schwierig. Der Gegenwärtige ist ein Server auf Abruf, d.h. man sieht beim Aufrufen der App erstmal: nichts & muss dann neuladen. Aber wer schenkt einem schon Leistung?


Makes generating new bottles for taps a bit easier, Cheers!


於首頁添加debug label
takePicture Grid調整
takePicture後端添加try catch、colorBrush檔名改成cameraBrush


Align the parameters of a method call if they span more than one line.


Hi Igor, (cc @parcollet)
The histogram branch needs more work doing on it before it is ready to be merged, and this will not happen before the holidays. Notable, we need to change the MC class in the library to be able to handle more cleanly measures in the moves.
Pri


ack


啊？


What about absence of first_name or/and last_name?
I think you can place first_name and last_name in array and use two methods here: compact and join (check in docs what it does ;))


RecordRow may be better? like index row.


It's a good solution to be the interactive chart with a map operation.
However I think that it should not run a chart animation whenever changing a map extent.
At first time, `animateRotate` is `true` in the `initChart` function.
From the second time, it is `false`.


you check php for skipping here :)


unack, that is


how about `else: groups[variable] = None` then you can drop the get and None below


I think it will be possible to fix the recursive call of updating using `scheduleUpdate($pos, 0)`. According to https://github.com/PocketMine/PocketMine-MP/blob/master/src/pocketmine/level/Level.php#L681-685 it will be run half a tick later.


if the store is nil, the txn can't be none nil, so we can ignore checking c.Store == nil here.


TeamCity pull requests :: netty [Build 846](https://garage.netty.io/teamcity/viewLog.html?buildId=4017&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4801, ignored: 21 Build time: 00:36:59


SyntaxError at ``//``


It's not that important, I find it more intuitive like this where type checks are here and default values are below


Shouldn't we use the Xamarin Studio roslyn preview? These samples _might_ be using C#6, so it might be worth using it.


:-/

Thanks.

P.

On 12 December 2015 at 13:02, Marcin Wojdyr <notifications@github.com>
wrote:

> SyntaxError at //
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pemsley/coot/commit/ce687415dc3d90725aa326d4703ba40aacc890b8#commitcomment-14941300>
> .
>


Just assert that init is non-null before creating the store.


matter of taste, this is ok :)


I mean why would you pirate a cheap game like this which is worth every cent of it :P


that package description isn't really good, that package is for all bears which do CSS analytics, not necessarily wrappers, right?


unack


it seems this may be my problem also (users can't register because of absolute url). i am using version 2.3. how do i correct this. thanks


Selector should have depth of applicability no greater than 3, but was 4


Hmmm... I didn't see the missing pointer.

Maybe it's worth making the codegen for init different than for
expressions. You maybe write it as:

gen_init(llvm::Value* ptr, Init* init)

and then dispatch to an appropriate behavior using a visitor.

Andrew Sutton

On Sat, Dec 12, 2015 at 8:06 AM, Andrew Sutton <andrew.n.sutton@gmail.com>
wrote:

> Just assert that init is non-null before creating the store.
>
>


@necronet Any idea how to replace this line by this format
````
{React.cloneElement(this.props.children, {...this.state})}
````
plus the injection of the emit method as property?


See the message about code gen. Maybe make init evaluation take a reference
to the value being initialized.

Andrew Sutton

On Sat, Dec 12, 2015 at 2:10 AM, Wyatt Childers <notifications@github.com>
wrote:

> @asutton <https://github.com/asutton> So, we've actually got the same
> problem here, that we have with code generation. A "Value" is expected,
> except this time it's not possible to simply return nullptr.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Beaker-Compiler/beaker/commit/dd6deb3364a2a8224278280b945d063af667bd45#commitcomment-14940216>
> .
>


Yeahhh, it was copied from linter package :P I'll delete the desc


Ok it's now defaultcdrecorder


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


Space inside square brackets detected.


제가 cm13을테스트해본결과. 아마도 lg로고에서멈추는이유가 이커밋밑 밑3개정도의커밋이문제인거같습니다.
아마도이커밋을리버트처리시켜야할거같네요.
그리고 flo커널에서 안드로이드 M selinux커밋과 mm,비디오체킹,로우메모리킬러등커밋들이 많이있습니다.
불머리다하시고 시간남으시면부탁드리겠습니다


Line is too long. [106/100]


Do not write to stdout. Use Rails' logger if you want to log.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 69](https://teamcity.labkey.org/viewLog.html?buildId=291202&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 34 (1 new), passed: 257, muted: 2 Build time: 01:25:56
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

##### there are 24 more failed tests, see build details
```


It's removed on the command-perms branch :smile:


Take a look at GoDocumentationProvider. There are quite good functions for retrieving signature owner and type presentation text. The result text is html but it's fixable.


You can do `from ufo2ft import compile as compileUFO`


Моя промашка, почему-то принял if с обрезкой за проверку на переполнение буфера.


ur code's wrong and seems very bad


Inconsistent indentation detected.


Use 2 (not 1) spaces for indentation.<br>Tab detected.


Tab detected.


Final newline missing.


lol, I should merge command-perms branch too :laughing:


Extra blank line detected.


Tab detected.<br>Trailing whitespace detected.


1 trailing blank lines detected.


removed the line


I grab it from you) But I don't see Visualizer) I complied AudioFX and Eleven but it's not help.


not a big fan of setting states as property for other component, I think is more suited for props. if you had in props emit,title and status you could pass everything with the spread operator `{...this.props}`


couldn't find that branch on my IDE


Extra empty line detected at class body end.


Halooooooooooo !!! checking


in zshrc: `setopt histignoredups`


Just fix up formatting to use tabs rather than spaces and I'll merge in.


Hi,
got lots of
>  E/KernelWakelockReader: Failed to parse proc line: !
I use this ROM
http://www.modaco.com/forums/topic/376549-rom60all-cyanogenmod-13-06122015/

It seems that wlBuffer contains a !.
Let me know if you need some more info


Необходимо вместо $item['date_pub'] использовать $first_item['date_pub']
При первой загрузке старницы со слайдом, не верная дата.
При втором показе слайда, дата уже правильная.


Working servlet needs to be cleaned up some


closes #13


Keep a blank line before and after `private`.


Inconsistent indentation detected.


Oh no worries! It's a huge mess currently. Hopefully I can get everything to compile by tomorrow. Then I have to test that everything is acting OK and I'll be able to merge to master


Extra empty line detected at method body beginning.


Extra empty line detected at block body beginning.


Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.


Is this included in the pod already?


back up


Line is too long. [106/100]


Extra blank line detected.


Tab detected.


1 trailing blank lines detected.


Do not write to stdout. Use Rails' logger if you want to log.


이렇게 주석달면됨?


Could you give a specific example? With such a broad question it's hard to narrow down.


guava doesn't allow this to return null :(