Note that Python will evaluate an empty string or None to False, in addition to the False that's returned by `get` if the "contributor" property does not exist.


Hmm. Maybe :) It probably wouldn't be too hard to do, but I don't want to get into the situation where *every* new type needs a parser/formatter... we don't have one for DateInterval, for example. Worth chatting about further.


No source changes, so the version stays the same.


I haven't put my dirty hands on this file yet :laughing:
So don't be afraid and take all the credits (and responsibility) for this :wink:


Its a nice solid toolchain that works with my kernel. AOSP GCC doesn't work
for some reason.
On Oct 15, 2015 12:55, "Park Ju Hyung" <notifications@github.com> wrote:

> Um, why are you even using this toolchain?
> On Oct 16, 2015 1:58 AM, "Paul Keith" <notifications@github.com> wrote:
>
> > I keep getting this Error when Compiling Kernels with this TC
> > CC scripts/mod/empty.o
> > HOSTCC scripts/kallsyms
> > HOSTCC scripts/selinux/genheaders/genheaders
> > HOSTCC scripts/selinux/mdp/mdp
> > HOSTCC scripts/bin2c
> > HOSTCC scripts/mod/mk_elfconfig
> > Host TableGen: libLLVMWrap (gen-intrinsic) <=
> >
> frameworks/compile/libbcc/bcinfo/Wrap/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > Warning: AndroidManifest.xml already defines minSdkVersion (in
> > http://schemas.android.com/apk/res/android); using existing value in
> > manifest.
> > Warning: AndroidManifest.xml already defines targetSdkVersion (in
> > http://schemas.android.com/apk/res/android); using existing value in
> > manifest.
> > Host TableGen: libLLVMBitReader_2_7 (gen-intrinsic) <=
> >
> frameworks/compile/libbcc/bcinfo/BitReader_2_7/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > arm-eabi-gcc: internal compiler error: Illegal instruction (program as)
> > Please submit a full bug report,
> > with preprocessed source if appropriate.
> > See http://gcc.gnu.org/bugs.html for instructions.
> > make[4]: *** [scripts/mod/empty.o] Error 4
> > make[4]: *** Waiting for unfinished jobs....
> >
> > This doesn't happen when I reset --hard to before this commit
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13798312
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13799470>
> .
>


That's because it makes sense to make it depend on the attacked creature. For example, that would allow to always fight to kill when attacking natural enemies.


er, ignore that comment. deleting


What would you use instead? I guess the REPLACE isn't needed any more since the version column is dropped..


Feel free to change what you want once this one is merged


Waarom check weghalen van de levens van de speler?


could rename variables a and b to be explicit on what the represent? Also may need to have a check incase either is negative and also if offset can be negative?


Seriously, improving is good, but let's always keep in mind the cross-compatbility (especially Android and iOS)
Possibly I don't want to end up to ifdef code.
In this case memset is very basic and should be supported by every desktop and mobile platform.
Good job ! :+1:


That might be a good idea. You can go for it if you want.


Indentation here is off. I'm seeing something different for line 37 and line 38


[Reference here](https://gist.github.com/3TUSK/5ab7f75b18c942e3732e). Will port from gist to this repository.


[Issue #12439](https://code.google.com/p/googleappengine/issues/detail?id=12439) created.


The current message is just fine IMO, my addition was just for fun :)


Ерунда вышла, а разобраться в чем причина пока не могу. В общем обратил внимание, что класс называется levelUpWindow, т.е. с маленькой буквы. Исправил на  LevelUpWindow и теперь при попытке залить из Intelij Idea изменения на гит я получаю сообщения об ошибке:
Error:pathspec 'out/production/ARLG/com/rommax/MonsterSet.class' did not match any file(s) known to git.
error: pathspec 'src/com/rommax/LevelUpWindow.java' did not match any file(s) known to git.
error: pathspec 'out/production/ARLG/com/rommax/levelUpWindow.class' did not match any file(s) known to git.
error: pathspec 'src/com/rommax/ItemSet.java' did not match any file(s) known to git.

Я попробовал удалить этот файл на гите, т.к. решил что проблема в названии файла, но из этого ничего не вышло. Восстановить обратно его мне не удалось. Как все это побороть я еще не разобрался, т.к. в будни основная работа забирает все время.


Waarom wordt de shortPower niet meer gereset?


C  -> I, I hasn't IC.


I was simply wondering if you've experienced the same error
On Oct 15, 2015 12:55, "Park Ju Hyung" <notifications@github.com> wrote:

> Um, why are you even using this toolchain?
> On Oct 16, 2015 1:58 AM, "Paul Keith" <notifications@github.com> wrote:
>
> > I keep getting this Error when Compiling Kernels with this TC
> > CC scripts/mod/empty.o
> > HOSTCC scripts/kallsyms
> > HOSTCC scripts/selinux/genheaders/genheaders
> > HOSTCC scripts/selinux/mdp/mdp
> > HOSTCC scripts/bin2c
> > HOSTCC scripts/mod/mk_elfconfig
> > Host TableGen: libLLVMWrap (gen-intrinsic) <=
> >
> frameworks/compile/libbcc/bcinfo/Wrap/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > Warning: AndroidManifest.xml already defines minSdkVersion (in
> > http://schemas.android.com/apk/res/android); using existing value in
> > manifest.
> > Warning: AndroidManifest.xml already defines targetSdkVersion (in
> > http://schemas.android.com/apk/res/android); using existing value in
> > manifest.
> > Host TableGen: libLLVMBitReader_2_7 (gen-intrinsic) <=
> >
> frameworks/compile/libbcc/bcinfo/BitReader_2_7/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > arm-eabi-gcc: internal compiler error: Illegal instruction (program as)
> > Please submit a full bug report,
> > with preprocessed source if appropriate.
> > See http://gcc.gnu.org/bugs.html for instructions.
> > make[4]: *** [scripts/mod/empty.o] Error 4
> > make[4]: *** Waiting for unfinished jobs....
> >
> > This doesn't happen when I reset --hard to before this commit
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13798312
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13799470>
> .
>


Or maybe we should increase it on the other icons. IMHO, we don't even see the effect in most of them.


It means the same as:

```
public string Name {
  get {
    return Info.Name;
  }
}
```


Good Thanks.


we don't have `E_`


Ziet er goed uit staan nog wel enkele syso's in


Ja, die Hidekis gehen und der Eurochron auch.

Dankeschön

Josef

----- Ursprüngliche Nachricht -----
Von: "sidey79" <notifications@github.com>
Gesendet: 15.10.2015 19:39
An: "RFD-FHEM/RFFHEM" <RFFHEM@noreply.github.com>
Cc: "hjgode" <hjgode@gmail.com>
Betreff: Re: [RFFHEM] Sorry, I reverted my changes as they were not build onthe HEAD commit (88be9d7)

Kein Problem, es läuft jetzt wieder.

Normalerweise hätte dein client das erkennen sollen und dich auffordern ein pull zu machen. Keine Ahnung, was da passiert ist.

Die controls Datei erzeuge ich bei mir mit einem macro bei jedem commit, so kann ich es nicht vergessen.

Gehen denn jetzt alle TFA und Arduino Sensoren?

Grüße Sidey

hjgode <notifications@github.com> schrieb am 15.10.2015:
>Soll ich das jetzt noch machen nachdem schon wider neue Änderungen drin
>sind.
>
>Die controls_duino.txt sollte man nach JEDEM update (commit/push)
>aktualieren. Ich denke mal nach den weiteren Änderungen ist das jetzt
>zu spät. Ausserdem habe ich gerade erfolgreich ein update force auf
>dev-rawIn gemacht. Das bedeutet, die controls_duino.txt ist aktuell
>OK.
>
>Warum ich mit dem update force und anschließenden perönlichen
>Änderungen plus commit allerdings so viele Ändeungen hatte kann
>eigentlich nur daran liegen, dass sich zwei commits überschnitten
>haben. Obwohl git dann eigentlich meckern müsste (von wegen host is
>ahead ud stash etc.).
>
>Ich weiss nicht was da passiert ist, scheint jetzt aber wieder im Lot
>zu sein.
>
>Tschuldigung nochmal.
>
>On 10/15/15, sidey79 <notifications@github.com> wrote:
>> Jo, da musst Du beim Zusammenführen wohl manuell auswählen, welcher
>Block
>> von wo kommt. Kannst Du bitte komplett auf die funktionierende
>Revision
>> zurücksetzen, damit alle Änderungen zurückgesetzt werden, also auch
>> controls.txt.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>https://github.com/RFD-FHEM/RFFHEM/commit/88be9d7ccb164a498d1b9980b6e3b8176c1994d4#commitcomment-13785518
>
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/RFD-FHEM/RFFHEM/commit/88be9d7ccb164a498d1b9980b6e3b8176c1994d4#commitcomment-13799076

--
Diese Nachricht wurde mit K-@ Mail gesendet.
—
Reply to this email directly or view it on GitHub.


FILE directives here is not necessary


When Info only has a setter, it is the equivalent of a ```readonly``` field - just for properties.


Staat nog een syso


develop (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1378 of 1378 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)


Hi, I have added "Write optimisation", turkey test does not provide anything new to the API so I decided to not include this. I will add some info about "timeparser, jdbc result writer". Probably I will do that tomorrow.


Also `${generated_file}`


`${file}`


Ook nog een syso


caused [JENKINS-30941](https://issues.jenkins-ci.org/browse/JENKINS-30941)


Because it is a member from the class RoomLibrary. But I may change that a bit later


That's why I removed it ;-)


Ага, понятно :) Скайп есть? Если что мой - sergiytkach Можно в чате обсудить рог. Сам недельки 2 назад начал изучать джаву.


There's no support for `n_val_one` in the main ts branch.


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


I think you want to call `user.save()` after calling this right?


Thanks ^^


Wtf?


The problem with run is that it wasn't killing Jupyter reliably.


Confirmed. I did test this, and the message was only coming out once but it seems to depend on the region (if there are objects changed after the first backup).  But I retested more carefully just now and confirmed the backup command is redundant; there's a hard-coded call in Scene.Close. Will remove the file again, or perhaps it may be better to include an empty file (a bit more self-documenting then on how to add shutdown commands).


it looks like all of the code in this file assumes one phone number per user, is that intentional?


Yes that's another possibility. Let's see what our Icon lore master @Bertram25 thinks about it :)


Note that I've removed mEffect->isForcedToWork thanks to the behaviour plugins. That was something I was not very happy with and the reason why I went for it


The problem is just that `run` and `spawn` have different defaults as to STDIO passthrough.


m_getBufferError is always >= 0, so you only need to check if (m_getBufferError < 5), right?


That's an unwritten rule: if a function has a too long name, it probably means that the logic can be improved :P


> The lich has no material, or it also relies on the undead* stuff?

Yes

> Same here, file names should be CamelCase.

That's because it was like that in the original file. I will try to change that


These error messages are likely not very helpful to an administrator. The admin would not know the difference between 'current broker' and 'broker host'.

Perhaps something along the lines of:
```
do_fail("SSL cert issuer doesn't match between localhost and #{broker_host_env} defined in /etc/openshift/env/OPENSHIFT_BROKER_HOST")
```


I'd move tests for statics to `statics` dir


nit: extra space before '='.


We could do, but I think that's overkill given that we are talking about one single case.

What does "new LocalDate(2001, 2, 29)" do?


In this error output, perhaps show that we attempted to connect to #{broker_host_env} and failed. Otherwise, the errors for curling localhost and #{broker_host_env} are the same.


Backports are 'enabled' but you have to ***explicitly*** tell the system to install from it.  I.E:  `sudo apt-get install swig3.0/trusty-backports` would install it from backports.  Otherwise it'll fail.  To not have to deal with this longer command string, the PPA will just 'work' once it's added.


@Sacrifice78 he has retail information and he is already working on it, also there is @HorridoJoho's implementation in forums.


we don't check which `foo` resolved


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


Ok. I will try. Like the lich, I've just taken the mesh as is


`val x: Int = foo()` ?


The commit is great indeed.


Exactly ;-)


Because T resolved to generic parameter. (diagnostic will be fixed later in new resolve)


good catch! guessing when i told it to update view positions it did this... investigating...


msvc12 is compiler version 12 which corresponds to MSVS2013. MSVS2012 is msvc11. Easy, isn't it ^^


for web users, we also need check which phone number is default and set that as the default number?


saw approval from lukebakken
at https://github.com/basho/riak_pb/commit/8ea6e80acf3ad6b38844ddf0c18cc4b1671d254f


merging basho/riak_pb/bugfix/bch/tweak-python-msg-generation = 8ea6e80a into borshop-integration-138-bugfix/bch/tweak-python-msg-generation


basho/riak_pb/bugfix/bch/tweak-python-msg-generation = 8ea6e80a merged ok, testing candidate = 833ed9a0


nice catch


`// !DIAGNOSTICS: -UNUSED_VARIABLE`


ccc


this will cause python to import an installed module called "hello_world", which should be this module, so I don't think the import will cause a print like we want.  I haven't taken the time to run this yet, though.  Will pull/reject after some testing :-)


Maybe:

"1. Create at least one admin user. See the ..."


Mmm I'm not so sure. I'd like to keep them protected until we find a use for them to be public. Making them more visible is always easier than the other way around.


:cry:


cdfdf


This comment is no longer true actually. Will remove.


I haven't used it yet.
On Oct 16, 2015 3:00 AM, "Paul Keith" <notifications@github.com> wrote:

> I was simply wondering if you've experienced the same error
> On Oct 15, 2015 12:55, "Park Ju Hyung" <notifications@github.com> wrote:
>
> > Um, why are you even using this toolchain?
> > On Oct 16, 2015 1:58 AM, "Paul Keith" <notifications@github.com> wrote:
> >
> > > I keep getting this Error when Compiling Kernels with this TC
> > > CC scripts/mod/empty.o
> > > HOSTCC scripts/kallsyms
> > > HOSTCC scripts/selinux/genheaders/genheaders
> > > HOSTCC scripts/selinux/mdp/mdp
> > > HOSTCC scripts/bin2c
> > > HOSTCC scripts/mod/mk_elfconfig
> > > Host TableGen: libLLVMWrap (gen-intrinsic) <=
> > >
> >
> frameworks/compile/libbcc/bcinfo/Wrap/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > > Warning: AndroidManifest.xml already defines minSdkVersion (in
> > > http://schemas.android.com/apk/res/android); using existing value in
> > > manifest.
> > > Warning: AndroidManifest.xml already defines targetSdkVersion (in
> > > http://schemas.android.com/apk/res/android); using existing value in
> > > manifest.
> > > Host TableGen: libLLVMBitReader_2_7 (gen-intrinsic) <=
> > >
> >
> frameworks/compile/libbcc/bcinfo/BitReader_2_7/../../../../../external/llvm/include/llvm/IR/Intrinsics.td
> > > arm-eabi-gcc: internal compiler error: Illegal instruction (program as)
> > > Please submit a full bug report,
> > > with preprocessed source if appropriate.
> > > See http://gcc.gnu.org/bugs.html for instructions.
> > > make[4]: *** [scripts/mod/empty.o] Error 4
> > > make[4]: *** Waiting for unfinished jobs....
> > >
> > > This doesn't happen when I reset --hard to before this commit
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13798312
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13799470
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arter97/linaro-lto/commit/5b3acad714156ff58b301e6af87368c2ab5410f3#commitcomment-13799581>
> .
>


Got it, didn't realize we remove the background style sheet.


Is this safe or should we be bumping to 2.1.1 and carrying what we need?  Not sure of the implications here but this commit is from 14 days ago whereas the latest release is Aug 12.  Just want to double check that this is indeed the right thing to do.


Got it. Added the comment and moved these back to the "Create a new Buffer" block.


Ага, я тоже в процессе изучения =) В скайпе два sergiytkach, один из Киева, а второй из Чернiвцi


Do we want to keep this check in this test or pull it out to a separate test?


:ok:


Will clean up wording.


Yes, It helped me alot.


staticFun?


LGTM


:trumpet: :balloon:


Daniel, I tried to replace all the std::vector with the DenseVector. I have
a problem that in the FiniteStrainCrystalPlasticity, the
MaterialProperty<std::vector<Real>
> & _gss;  is not easy to do as it is somehow related to the auxkernel
MaterialStdVectorAux Do you think I should rewrite a auxkernel or just
change others?


Gil wrote:
We do not have countable condition registers in our architecture (CC_BUILD, CC_INDEX, CC_NAME0, CC_NAME1).
That's why we had to statically use the indexes and could not discover them on run-time.

Please remember that our perf block design is very simple with much less flexibility than vannila ARC. We wished to save here gates on NPS chip.

Sent by MailWise<http://www.mail-wise.com/installation/4> – Your emails, with style.:)

-------- Original Message --------
From: Alexey Brodkin <notifications@github.com>
Sent: Thursday, October 15, 2015 05:09 PM
To: EZchip/linux <linux@noreply.github.com>
Subject: Re: [linux] ARC: [plat-eznps] Use dedicated performance monitor (922d723)
CC: Noam Camus <noamc@ezchip.com>

What I really don't like here is hard-coded HW event indexes. Because as ARC700 databook says:

The set of countable conditions available on any particular core configuration can vary. For this reason, registers are provided to allow a debugger or natively executing program to discover the supported conditions.

That is it is mandatory to do that run-time discovery of available counters and then do its mapping, see how we do it in our perf code, https://github.com/torvalds/linux/blob/master/arch/arc/kernel/perf_event.c#L487

Moreover since HW events are the same for vanilla ARC and EzChip platform we may [and I believe we should really] reuse the same code that does parsing and mapping of HW events to those known to perf. But for that I need to rework ARC perf code so it has generic parts and plugins for different perf counters.

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/922d72324008490c9843494e7c16f33d27ed62b8#commitcomment-13793658>.


I removed the file again. I concluded the primary purpose is to remove all the redundant extra files that aren't normally needed, and providing an empty file seemed to undo that effort. It's gone. ;)


Big fixes were missing super calls in a lot of VC lifecycle methods. Also tweaked some action method names (e.g. `saveButtonTapped:` over `save:`), and gave a little better explanation of why in the readme.


This is different. But thanks :)


Change the commit message to `UPSTREAM: docker/distribution: <carry>: *commit title*`

If there was a respective PR upstream, that would be `UPSTREAM: docker/distribution: #pr-number: *commit title*`


There might be some confusion between 'name' and 'slug' (which is only set AFAICT on DigitalOcean provided images but not snapshots) remaining which prevents salt-cloud from creating DigitalOcean VMs based on user snapshot images.


Do you have odd numbers for lead/trail constraints? We might need to
revisit the design if so.
On Thu, Oct 15, 2015 at 14:13 montehurd <notifications@github.com> wrote:

> good catch! guessing when i told it to update view positions it did
> this... investigating...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wikimedia/wikipedia-ios/commit/607ce6cf6055635795228580dfc01587e3773aee#commitcomment-13799846>
> .
>


> This has nothing to do with terminal window width. It has to do with readability. Lines should really be broken at 60-70 characters, because long lines got awkward to read otherwise

In general, I'd respectfully disagree.  In this case, it looks long because everything else is short (and should be changed in this case because it is so much relatively shorter).  In general, though, most text I've seen in different sources expands further to the right than this and doesn't really tire the eyes that much.  It also can be a bit more difficult to account for this width in code, too.

The Doom Larry changes have already taken a lot of time, so I think it would be better to focus on the SFML changes than the width inconsistency in the file for now, however.


Or, if the user hits this problem (lack of "disk" space, which can happen
to other directories), he can set his favorite one (say `/var/tmp`) with
the standard environment variable.

Thoughts?
On Oct 15, 2015 2:39 PM, "Lucas Meneghel Rodrigues" <
notifications@github.com> wrote:

> @clebergnu <https://github.com/clebergnu> @ldoktor
> <https://github.com/ldoktor> - I remember why I put this here. It turns
> out that now Fedora has tmp on tmpfs, and given that the space available in
> /tmp is usually restricted, any large-ish file tends to cause problems.
> I've decided then to use /var/tmp, that's still filesystem backed to
> avoid trouble, unless the user explicitly sets TMPDIR, in which case I'm
> assuming the user knows what he/she is doing.
>
> Of course, we can put this back in and then have some mysterious problem
> down the line that will take a while to figure out, and after investigation
> we get to the conclusion that it's a problem due to the small amount of
> actual space on /tmp. It's a judgment call, maybe these days this is not
> such a problem anymore.
>
> So I think this patch should be NACKed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clebergnu/avocado/commit/7b1ac444aca4dc276f8cab26ddaf3b0be67d0e18#commitcomment-13799224>
> .
>


`UPSTREAM: docker/distribution: 1050: *commit title*


This is what I hate about github. this was an intermediate code.


@bgerstle turns out canvas location is just where in the IB window the *overall* view is placed.


Fixed with 8179123 - should be good for 101 years.


Sorry, I meant that if this is a widespread issue, then we can ask design
to revisit and fix all at once in Beta.
On Thu, Oct 15, 2015 at 14:19 Brian Gerstle <brian.gerstle@gmail.com> wrote:

> Do you have odd numbers for lead/trail constraints? We might need to
> revisit the design if so.
> On Thu, Oct 15, 2015 at 14:13 montehurd <notifications@github.com> wrote:
>
>> good catch! guessing when i told it to update view positions it did
>> this... investigating...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/wikimedia/wikipedia-ios/commit/607ce6cf6055635795228580dfc01587e3773aee#commitcomment-13799846>
>> .
>>
>


Ahhhh 👍🏻
On Thu, Oct 15, 2015 at 14:20 montehurd <notifications@github.com> wrote:

> @bgerstle <https://github.com/bgerstle> turns out canvas location is just
> where in the IB window the *overall* view is placed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wikimedia/wikipedia-ios/commit/607ce6cf6055635795228580dfc01587e3773aee#commitcomment-13800000>
> .
>


![mbbg vu to dam dang - phim sex khong che](https://cloud.githubusercontent.com/assets/15105885/10523317/133f324a-73a4-11e5-96d8-43877e9a686c.gif)


Agree, preparing a patch...


Next time do dos2unix in a separate commit. It makes viewing other changes a hassle.


commit message here should be
`bump(github.com/stevvoe/resumable) 51ad44105773cafcbe91927f70ac68e1bf78f8b4`


i think this is missing the part that checks multipleDecorations and adds the opening bracket?


The previous maintainer didn't seem to have a gist for 24.4 or would have obviously used it :)


wait @an-OK-squirrel does this work or no?


nice use of array!


2 seconds seems like an awfully short time to wait for cleanup. It would be nice to have this configurable, but that the very least I'd say something closer to 15-20 seconds would be good.


nice function! how are you planning to invoke/call it?


I think there was consensus to not have `BlobKeeper` upstream right? If so, ping me when you will update the OS code for review.


Я из Черновцов.


take another look at the condition (second expression). Will your loop ever execute?


great!


@plugz what do you think about this ?
Unfortunaltey I couldn't find an easier solution

'''

void ArtNetPlugin::init()
{
    QList<QNetworkInterface> interfaces = QNetworkInterface::allInterfaces();

    if (m_netInterfaces.isEmpty())
        m_interfacesTotalCount = 0;

    if (interfaces.count() == m_interfacesTotalCount)
        return;

    m_interfacesTotalCount = interfaces.count();

    for(int i = 0; i < interfaces.count(); i++)
    {
        QNetworkInterface interface = interfaces.at(i);

        foreach (QNetworkAddressEntry entry, interface.addressEntries())
        {
            QHostAddress addr = entry.ip();
            if (addr.protocol() != QAbstractSocket::IPv6Protocol)
            {
                ArtNetIO tmpIO;
                tmpIO.IPAddress = entry.ip().toString();
                if (addr == QHostAddress::LocalHost)
                    tmpIO.MACAddress = "11:22:33:44:55:66";
                else
                    tmpIO.MACAddress = interface.hardwareAddress();
                tmpIO.controller = NULL;

                bool alreadyInList = false;
                for(int j = 0; j < m_IOmapping.count(); j++)
                {
                    if (m_IOmapping.at(j).MACAddress == tmpIO.MACAddress)
                    {
                        alreadyInList = true;
                        break;
                    }
                }
                if (alreadyInList == false)
                {
                    m_IOmapping.append(tmpIO);
                    m_netInterfaces.append(entry);
                }
            }
        }
    }
}
'''


TEST_SET_BYTE(value, config_buffer, 0)
TEST_SET_UINT32(value, config_buffer, 1)
TEST_SET_UINT24(value, config_buffer, ...)
TEST_SET_UINT16(value, config_buffer, 5)
o algo similar. entonces todo el shifting lo haces en el macro.


Excellent work! Any reason for adding the `1` when you're logging to console?


Jerem, t'oublieras tout le temps les `,`, jsais pas ce qu'elles t'ont fait... tu dois les trouver moche j'en sais rien... Oo


>> 56 no deberia ser posible.


Even if it doesn't have a lot of meaning, let's have this godoc'd as well. Makes golint happier.


cool!


Thank you!


I think, this is not necessary. Give me a false log on dsmeg.


this still looks weird to me. I think using curly braces to denote variables in URLs is a pretty common idiom in API documentation, so let's stick with that. or you could do what Google does in their docs and italicize it: https://developers.google.com/maps/documentation/geocoding/intro

Like https://api.careerbuilder.com/core/taxonomy/*taxonomy name*

Either way, you don't want that space in there, or the trailing slash.


Should we always return true, regardless of animating state?


convention: We keep imports separate into logical groups: 1. stdlib imports 2. third party imports (including Kube imports) 3. OpenShift imports


Done.


lol wow.

I tried to not use `urlsplit` but you cannot modify the members of a `urlparse`, and so the only alternative is to manually build up a url from its constituent parts.  I figured `urlsplit` + `urlunsplit` is the safest way to do that.


I propose to create a small function that create a SDL_HapticEffect with good default value.

It would compact a bit the code of this function, and it would highlight the difference between the various effects.


Hi Stefano,

Could you consider testing these values against the patch "pawn_asymmetry4" which passed STC and LTC, instead of against the current master ? This way, if your patch also passes, we would have two sequentially consistent improvements instead of two conflicting patches against the old master, as last time :-)

Good luck with your tests !


Jikes you are right... leftover from the old code... will fix


LGTM


I think there's an issue with GIT allowing raw access to a file the way we linked it. I ended up placing the AJAX plugin locally and changed the url for the geojson to look locally. It works and I updated the map (master and gh-pages). Let me know if it works to close the issue.


this will fail as you're `i` is not yet defined :(


Файл стоит рассматривать почти с нуля, т.к. я внес много мелких изменений в структуру кода:
- поменял порядок выполнения функций на более логичный, на мой взгляд;
- объединил циклы for (удаление активного переключателя фильтра) и switch (установка сохраненного переключателя фильтра) в один;
- дал функции setClass более говорящее название setDefaultFilter;
- изменил комментарии :-P .

Подозреваю еще, что установку сохраненного переключателя, которую из switch перенес в for тоже можно как-то упростить, на что вроде бы в т.ч. намекает втой предыдущий комментарий относительно этого участка кода. Но не осил. =(


In this assignment, you will write a Python program to compute the
multiplication table without the multiplication operator *.
In other words, you are computing the results of each multiplcation as a
repeated addition using an accumulator and a for loop. For example to
compute 4*3, we would proceed as follows:
Initialize an accumulator with 0
Loop 4 times
Add 3 to accumulator
At the end of for loop, accumulator contains the result of the multiplication
Your program will first ask the user for n in the multiplication table
of size n by n from 1 to n. The program will then display the result each multiplication on
a separate line, without using the multiplication operator.


you're close!


I thought host regex was allowed - why not?


cool!


This looks a bit odd - I'd expect either
```c++
wchar_t *sxFileName = reinterpret_cast<wchar_t *>(wParam);
```
or
```c++
LPWSTR *sxFileName = reinterpret_cast<LPWSTR>(wParam);
```


Write a program that will allow the user to draw four different colored triangles using mouse clicks. More specifically, your program should use a loop that iterates through a list of colors, given here:

colors = ['red', 'green', 'blue', 'orange']
In the body of the for loop, you should allow the user to click somewhere in the window, and record that point using the GraphWin object's getMouse() method that returns a point object. The syntax is as follows:
 p = win.getMouse()
where p is a Point object. Right after the user clicks, you will use that point to create a red circle with radius 4, and draw it. When the user has clicked three times, your program will then draw the triangle at those points, with a fill color given by the loop index.


man, you killed it! Great job!


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1011](https://teamcity.mesosphere.io/viewLog.html?buildId=59146&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:58


Paging @smarterclayton, this is the only thing left here AFAIK.


This is a *great* use of function!


Should be `PromiseInterface` if only using `then` and not any of the advanced features.


Don't we usually backport all fixes to the stable version?


a fix has been pushed...  stay tuned.


Hehe, straightforward as usual, like Windows 8 with is actually 6.2 or 6.3.


Yes, we have only ever supported redhat based operating systems with this module because debian and ubuntu did not have cgit packages (though that may have recently changed).


The question is: how often is init() called ? Because calling QNetworkInterface::allInterfaces() is not a fast operation.

I think it is only when loading the plugin at startup and then going to the I/O tab, so this patch is good :)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1012](https://teamcity.mesosphere.io/viewLog.html?buildId=59148&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 180 Build time: 00:04:30


Thanks for looking into this John. The solution I am working on involves
adding methods to ArgDef and ArgList that handle these 'flag' type
parameters correctly.

--
Haines, Chris

On Thu, Oct 15, 2015 at 1:18 PM, John Fereira <notifications@github.com>
wrote:

>
> Okay I have a pattern that works now and will commit the code in just a
> bit.
>
> Basically, when you changed the Match(ArgList opts) method and added
> Boolean.parseBoolean(opts.get("r”) to parse the “rename” Param
> It was calling the ArgList.get(String) method with the assumption that the
> option had a parameter.
>
> If you look at the getParser() method you’d see that the parser for the
> –r/rename option did not include a “withParameter()” statement. If that
> line is changed to:
>
>
> parser.addArgument(new
> ArgDef().setShortOption('r').setLongOpt("rename").withParameter(true,
> "RENAME").setDescription("rename or remove the matched entity from
> scoring").setRequired(false));
>
>
> It will configure the option such that it expects a parameter (true or
> false). I also changed the Boolean.parseBoolean(opts.get(“r”)
>
>
> To:
>
>
> (opts.has("r")?Boolean.parseBoolean(opts.get("r")):false),
>
>
> That way if the option is not included in the command line or the <Param
> name="rename”>true|false</Param> is not included in a configuration file,
> renameRes will be set to false.
>
>
> I also added log.debug statements to report the values of rename and
> clear-literals.
>
>
> The only gotcha here is that if one specifies the option on the command
> line but does not follow it with “true” or “false” the parser will fail.
>
>
> Before I change some of the other tools I know of one case in the example
> harvester which would break if command line option for a true/false
> parameter required a value.
>
> From: MassuguGo
> Reply-To: vivo-project/VIVO-Harvester
> Date: Tuesday, October 13, 2015 at 7:54 PM
> To: vivo-project/VIVO-Harvester
> Cc: John Fereira
> Subject: Re: [VIVO-Harvester] fix bug where all values (including 'false')
> in config was still treated as 'true' (only omitted parameter resulted in
> 'false') (874aeee)
>
> I believe this is him reporting that my attempt at resolving this bug had
> unintended side effects.
>
> Not certain why it would hang beforehand though.
>
> --
> Haines, Chris
> On Oct 13, 2015 7:46 PM, "John Fereira" <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> > Yes,that is a bug and exists in several of the tools. For now, you'll
> have
> > to comment out the parameters in order to specify the value as false.
> >
> > John Fereira
> > Cornell University, Ithaca, NY
> >
> > Sent from TypeMail
> >
> >
> >
> >
> > -------- Original Message --------
> > From: Ben <notifications@github.com<mailto:notifications@github.com>>
> > Sent: Tue Oct 13 18:02:26 EDT 2015
> > To: vivo-project/VIVO-Harvester <VIVO-Harvester@noreply.github.com
> <mailto:VIVO-Harvester@noreply.github.com>>
> > Subject: Re: [VIVO-Harvester] fix bug where all values (including
> 'false')
> > in config was still treated as 'true' (only omitted parameter resulted
> in
> > 'false') (874aeee)
> >
> > I get an error from Match when trying to run the Harvest Person Data
> from
> > CSV option in VIVO.
> > 2015-10-13 15:53:47.666 DEBUG [o.v.h.u.a.ArgList] running Match
> > 2015-10-13 15:53:47.669 DEBUG [o.v.h.u.a.ArgList] command line args: -i
> > vivo/config/h2-sdb.xml -ImodelName=http://localhost/vivo/harvested-data
> >
> -IdbUrl=jdbc:h2:/usr/local/vivo/harvester/vivo/harvested-data/csv/30FE126DFA5BFC23B376FC4B3D490754/score-data/match
>
> > -r -t
> > [snip.....jenaconnect log..... snip]
> > 2015-10-13 15:53:48.414 ERROR [o.v.h.s.Match] r has no parameters
> > 2015-10-13 15:53:48.539 DEBUG [o.v.h.s.Match] Stacktrace:
> > java.lang.IllegalArgumentException: r has no parameters
> > at org.vivoweb.harvester.util.args.ArgList.get(ArgList.java:177)
> > ~[harvester.jar:na]
> > at org.vivoweb.harvester.score.Match.<init>(Match.java:140)
> > ~[harvester.jar:na]
> > at org.vivoweb.harvester.score.Match.<init>(Match.java:119)
> > ~[harvester.jar:na]
> > at org.vivoweb.harvester.score.Match.main(Match.java:443)
> > ~[harvester.jar:na]
> > 2015-10-13 15:53:48.543 INFO [o.v.h.s.Match] Match: End
> >
> > If I try the prior commit, the script just hangs. Any ideas?
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/vivo-project/VIVO-Harvester/commit/874aeee1c22589d772ed65be0fbf3ce10aa7ff87#commitcomment-13753409
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/vivo-project/VIVO-Harvester/commit/874aeee1c22589d772ed65be0fbf3ce10aa7ff87#commitcomment-13754984>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub<
> https://github.com/vivo-project/VIVO-Harvester/commit/874aeee1c22589d772ed65be0fbf3ce10aa7ff87#commitcomment-13755102>.
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vivo-project/VIVO-Harvester/commit/874aeee1c22589d772ed65be0fbf3ce10aa7ff87#commitcomment-13798747>
> .
>


Wouldn't it be clearer to have CDragDropAttachment declared and defined within AddEdit_Attachment.{h,cpp} instead of in their own files, since they're only used there?
Alternately, we can add the class to the existing DDSupport.{h,cpp} files.


It's for ILSGateway. Probably it will need the same changes as for EWS but I will check this later


Feel free to amend as necessary.

It does work as is but as long as it works afterwards, then I have no issue with you changing it.

So you seem to have all my commits pre and post my pull request in the same request.

Must remember not to push any more commits until you have closed the outstanding pull request.

David

On 19:29, 15 Oct 2015, at 19:29, ronys <notifications@github.com> wrote:
>This looks a bit odd - I'd expect either
>```c++
>wchar_t *sxFileName = reinterpret_cast<wchar_t *>(wParam);
>```
>or
>```c++
>LPWSTR *sxFileName = reinterpret_cast<LPWSTR>(wParam);
>```
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/c-273/pwsafe/commit/683d531e3ccdb52071d9b6e6b75917d56bf316fa#commitcomment-13800227


I submitted a pull request.


This is a great way of solving this clue, but can you think of maybe a better one? Hint: Did you know you can retrieve object values in JS like this too: `object["key"]`?


The only one that would be needed is for the callback call. You only check if a call has been made, and `MakeCallback` can't have been called w/o passing a `Function`.


by convention, normal functions begin with a lowercase letter (or underscore)


No, saving verified number before user is saved doesn't work.


Hi there,

Thanks for trying out Buildtime Trend!
I noticed you enabled the Buildtime Trend webhook in Travis CI to analyse the build process data and timings of your project.

If you have questions or feature requests, don't hesitate to ask. We have a mailing list and a twitter account where you can reach us : https://buildtimetrend.github.io/#contact

I'm also interested to hear why you started using Buildtime Trend, what your expectations are, or how you found out about Buildtime Trend.

I hope you find using Buildtime Trend useful and that it can help you gain some insights about your build process!

Kind regards,

Dieter
Buildtime Trend founder and developer


Oh right, good point


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


### New Functionality: MeshClient.prototype.login(credentials)

Previous client initialisation still works. (With a deprecation warnins)

See [Using the Client](https://github.com/happner/happner/blob/feature/login/docs/client.md)

Release pending.


Pourquoi `!important`?


Un debug oublié?


Rony,

Again, I don't mind but would prefer in
AddEdit_Attachment.{h,cpp} rather than the much bigger DDSupport.{h,cpp} which deal with much more detailed D&D.

David

On 19:33, 15 Oct 2015, at 19:33, ronys <notifications@github.com> wrote:
>Wouldn't it be clearer to have CDragDropAttachment declared and defined
>within AddEdit_Attachment.{h,cpp} instead of in their own files, since
>they're only used there?
>Alternately, we can add the class to the existing DDSupport.{h,cpp}
>files.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/c-273/pwsafe/commit/683d531e3ccdb52071d9b6e6b75917d56bf316fa#commitcomment-13800345


Excellent!


mark this fixed.


This works on Android?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1014](https://teamcity.mesosphere.io/viewLog.html?buildId=59152&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:14


super nit, but it came up in a different PR:  most of our flags are "sync-period" not "resync-period"


I don't think a normal user will ever need to know about `imageStreamDeletions` so this shouldn't get printing and describing. Instead add the type in describe/describer_test.go#DescriberCoverageExceptions and describe/printer_test.go#PrinterCoverageExceptions and explain with a comment in each case why we don't need it.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_pb/builds/185


Since I'm starting to think this may be a bit too easy for you, can you think of a more dynamic way of doing this? Maybe with array's and loops?


Yes, I will change it.


...and subscribed!
Hoping this goes open source soon


s/Http/HTTP everywhere, sorry


We most probably won't need this function anymore


s/Url/URL everywhere


Should be a string resource for I18N (but not a prereq. for pulling this).


reverse


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


Assalamu'alaikum pak, saya dari RPL-1 pas mau nge push ke repo TugasMingguan gak bisa akses terus.
Username for 'https://github.com': gunawan001
Password for 'https://gunawan001@github.com':
remote: Permission to adfbipotter/TugasMingguan.git denied to gunawan001.
fatal: unable to access 'https://github.com/adfbipotter/TugasMingguan.git/': The
requested URL returned error: 403


Oops, remet la, ca fait trop collé sinon.


Good catch, this is all wrong.


:+1: Thanks for tackling this!


Sorry for these. I had the fixes committed locally but forgot to push…


I think we need to probably have a special backward-compat test for this, actually.


Probably. Can't see any reason why that event should be allowed to bubble up.


minor: s/Ig/IG


shouldn't we also use target width and target height parameters when actually getting image? that way we won't need to crop out as much


Тормознул, ты мне сам написал решение filter[value].setAttribute('checked', 'checked');
Но у меня не получается его применить:"Uncaught TypeError: Cannot read property 'setAttribute' of undefined". При этом я переименовал переменные filterChrome, filterSepia и filterNone в такие же, но полностью в нижнем регистре.

Завтра на свежую голову попробую разобраться еще.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1015](https://teamcity.mesosphere.io/viewLog.html?buildId=59155&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:04:13


Hrm, actually, I think this may be okay.  Will dig.


This shouldn't actually be in this commit.


NIce work!


make sure to start regular function with lower case letter


"k8" means nothing.  It should be either "k8s" or "kube" or "kubernetes".  k8s means "K, 8 letters, S" (kubernetes).  k8 means "kubernete"


En quoi cette ligne est differente de celle juste au dessus?


I guess that's reasonable, although you know that people will shout at you angrily when something fails because of that, because they probably won't debug the test to reach this conclusion. So yes, sure, let's get back to /tmp.


oops, should only be > 0


load FAQ from database


I see many changes like this one, adding one space at the end. Can you explain why/when they are needed?


How about just `unsubscribe()`?


Why remove the delay?


I'd rather use java.util.ServiceLoader that's part of the jdk instead of grizzlie's ServiceFinder.


Maybe we can uncomment this lines and introduce a new "read intro", after the if-else block, to allow the users read the instructions before starting the donation process.


It is a major version, so without deprecation if fine, just need a CHANGELOG entry


Okay, after giving this some thought, I think this is okay, because we don't expose PodSecurityContext in the v1beta3 api, which is appropriate.  I do think this merits a note in a comment, though.


done


Yeah, I think it works and there is no defaulting going on so technically it's fine.  Comment is fine with me.


this looks reasonable. does it work?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1016](https://teamcity.mesosphere.io/viewLog.html?buildId=59161&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:03:58


r+


The parameter is not unused - even if not used in the Control base class. It resides here due to documentation reason. Many controls like sap.m.Dialog, sap.m.ListBase and many more override the invalidate function and require that origin parameter.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/56527bb0b6bb37c4f0097ae46e9bf57aba89486f


merging graydon/stellar-core/queue-publish-meter = 56527bb0 into auto


graydon/stellar-core/queue-publish-meter = 56527bb0 merged ok, testing candidate = 6666625b


> Can we get rid of...

No. This cfg file is the default for the haproxy package. Is not related to kubernetes/contrib.
This is one of the reasons why I prefer to use wget to get the haproxy apk.


fast-forwarding develop to borshop-integration-138-bugfix/bch/tweak-python-msg-generation = 833ed9a0


With the `delay(1)` the value is 57 instead of 97.
I thought that this may have something to do with the speed difference, but I certainly could be wrong.


With all due respect, why?


Sorry Dietmar, reverted your commit in order to resolve merge conflicts in #13


Hey @adrianbk, I'm curious why you use `buildScript """` here vs. the more-prevalent `buildFile << """` approach. @burtbeckwith and I were talking about this earlier today. I'm not sure there's actually any functional difference between the two, but the latter is definitely more common. Maybe there's something I don't know about the former?


The content of the src directory was copied from [alpine aports](http://git.alpinelinux.org/cgit/aports/tree/main/haproxy/APKBUILD) and modified to update haproxy to 1.6 and add the lua support.


Sieht gut aus! Bravo :+1:


oh NO


Instead of "Service" let's have something more descriptive like "RouterInterceptor" or "RouterExtension"


Not sure which string you mean but I didn't add too many as not sure if they are all staying.

I assume I18N is the last thing to be done and then all strings (there are hardcoded error messages too) will need to be reviewed.

David

On 19:39, 15 Oct 2015, at 19:39, ronys <notifications@github.com> wrote:
>Should be a string resource for I18N (but not a prereq. for pulling
>this).
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/c-273/pwsafe/commit/683d531e3ccdb52071d9b6e6b75917d56bf316fa#commitcomment-13800460


notice the `1.6.5` part @henriquea


Nope that's sufficient. It takes care of it.


@bastianallgeier Could this be simpler:

```
# block all files in system folders from being accessed directly
RewriteRule ^(site|kirby|panel)/(.*) index.php [L]
```


Настройки по подключению к БД должны быть в *.ini файле


Hier fehlt noch ein Fall, ist Dir das bewusst? Was passiert mit den Zahlen aus "dFeld", wenn wir ins neue Array erfolgreich das "d" eingefügt haben? Was ist mit dem Index?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1017](https://teamcity.mesosphere.io/viewLog.html?buildId=59162&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:08:01


Bleib dran, denk auch noch an die main()-Methode. Weiter so!


Tener en cuenta el ancho y alto de las imágenes todas deben ser de la misma medida


We should use an SPI here too, something like "ScaffolderExtension", and remove all the extension model files to the service implementation


That was a bit short, sorry. What I mean is, what is wrong with this commit message? It seems very clear to me what it does. What would be gained from taking the time to adopt a new set of commit conventions?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1021](https://teamcity.mesosphere.io/viewLog.html?buildId=59178&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:45


This may be a merge error. The error location is helpful to have the the error message.


Done


Since this is pretty much a minor issue, but one that's hard to have a
clear consensus, let's see what Lukas thinks about it. No need to rush.
On Oct 15, 2015 3:44 PM, "Lucas Meneghel Rodrigues" <
notifications@github.com> wrote:

> I guess that's reasonable, although you know that people will shout at you
> angrily when something fails because of that, because they probably won't
> debug the test to reach this conclusion. So yes, sure, let's get back to
> /tmp.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clebergnu/avocado/commit/7b1ac444aca4dc276f8cab26ddaf3b0be67d0e18#commitcomment-13800526>
> .
>


It's deleted in the message handler


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


OK for that, but you are on the wrong code, strange. This one is the commit before I rebased and force pushed


I don't see a need to use the appending version (`buildFile <<`) unless I'm going to have multiple appends like:

`buildFile << "a"`

`buildFile << "b"`


>  The heuristic is also susceptible to non-terminal character devices being the target of redirection, e.g. /dev/null.

Windows still uses CHAR device check. Do you know if NUL returns true for that (I think it does)? If it does then this change will introduce a subtle difference of behavior between UNIX and Windows, in places where we explicitly throw exceptions when things are redirected etc.


We can remove this `android:theme` on the activity now, as we've only added it for testing.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1022](https://teamcity.mesosphere.io/viewLog.html?buildId=59179&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:04:02


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


Not the fault of this PR, but we could change this from environ.get() to environ['PTERO_WORKFLOW_DB_STRING'].


Ah mmm so you are right it isn't but fixing it means chaning abstractnetworkjob and the account to support generic requets... basically the term davRequest is a bit misleading. We should fix it. But not in this PR I think


Note to self: libwvdrmengine.so is misplaced


I think it boils down to personal preference, and in no way I was trying to push mine.

In this instance, it was just the effort I had to put in to reach 120, or 80 was quite the difference. I was just lazy I guess. Anyway, I could disable this cop for now?


No worries.


I opened it this morning. I don't switch off my pc anymore (only suspend 2 ram). It likely misses a refresh on my side. Got too much open tabs to refresh all of them ;)


My laptop stayed pretty stable from when I pushed this commit up till last night. Last night, every time notifyd restarted it immediately jumped to 99% cpu. I was able to restart the OS cleanly. For now I'm using atom and millions of iterm2 windows. :stuck_out_tongue_closed_eyes:


Hi,

looks like there is a bug there, in the creation I try to assign _all_ids[i].res (whereas for ids that do not exist this would throw an exception). also when closing we should make sure that we haven't closed yet, and deleting the object without calling close is another problem, but since JS has no destructors we have to be careful.

However I'm a little confused.
You've asked me about automated testing, and if I'd expect you to do the testing for me (or at least that what was implied). I'm a senior engineer that's worked across many projects. Normally I also ask people to pay for challnages and test (as otherwise I feel people don't respect my time, which has been the case each time I did free "test" thus far, each time I had paid test, people were impress). Your challenge seemed fun and not something that's too overbearing like spending a full weeks of work. Hence I decided to take it. I've dedicated time to thinking about it, setting up a git repo for you, sending you the email, following your exact instructions. Now you're asking me to debug and iterate my versions a few times. After our conversations and iterations so far? do you have doubt that I'd be able to do that? Have you looked at my profile or background? Do you believe that I won't be able to fix my code to work? Don't you think that if you believe I can code, we'd have a call? Maybe tell me about you a little, see if we are even a personality fit?

At this stage, I feel that without some even keel approach I'd rather give it a miss...


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1019](https://teamcity.mesosphere.io/viewLog.html?buildId=59165&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:09:04


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1018](https://teamcity.mesosphere.io/viewLog.html?buildId=59164&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:09:20


caveat: .sort on list sorts the list in place and returns None. You need to use `sorted(glob.glob(niimg))` to get  here.


Ah, I'd be happy to do that after I get this project in better shape :-)


This same change will be needed in CursorVisible setter as well. It writes the string using Console.Write(). I will update that :)


1K iterations?


Initialization should be outside of `#ifdef X86OPT` or the value on non-x86 would be undefined.


Gulp as a build process allows for keeping everything (including images, manifest file, etc.) in the `/src/` folder, and only creating `/build/` as necessary. It also means that the final, distribtion `manifest.json` Chrome extension file can be automatically populated with information (such as version number) from package.json.

Here's a rundown of the build commands I've covered in the [updated readme](https://github.com/JeromeDane/chrome-extension-auto-reload/blob/master/README.md):

Generate a `./build/` folder which can be loaded into Chrome as an unpacked extension:
```
gulp build
```

Automatically rebuild modified files into `/build/` when they're saved:
```
gulp watch
```

Generate `./dist/chrome-extension.zip`, which can be uploaded to the Chrom web store:
```
gulp dist
```


@fbaligant If I get it right, the ``block`` value should not be reused as the key is not unique when the vaultfile is encrypted.


I'ts very good. I was with this problem.


Got it.  I removed it.  I'll push the change to the master after I complete the list assignment.


Fixes baudehlo/node-phantom-simple#90


I'm not understanding... I already changed CursorVisible just below:
https://github.com/stephentoub/corefx/commit/98552707c585bc0f0c138fdeed991c28f00a0ac5#diff-cb11c7a5ca3a475a8382bf2353b2c115R105

Is there another one besides that I missed?


Aktueller Stand 15_10_15 21:07Uhr:
Damit sind alle Klassen vervollständigt, wir benötigen jetzt noch

J-Unittests sowie Javadoc-Kommentare für die Klassen: Spieler,Spielfeld,Spielfigur


That's the one :). I missed the change was to the CursorVisible property :). Thanks!


i guess phaseid of all npcs is wrong, it's not the same like phasemask in 3.3.5


```Console.Is*Redirected``` returns true for nul, so that seems consistent with the new behavior.  Was that the only concern?


My takeaway from all this is that we should keep the internal down-mixing and allow people to use splitters if they want single-channel analysis of a multi-channel source. That way we get simple obvious analyzer behavior that does not ignore any channel of the input if people don't do anything fancy. Any problems with that approach?


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


Pls indent the `do:` (sorry)


too much space after `distinct` (sorry)


Well, Let's make it `_aosc_bashrc_vcs_git`..


OK, I see. kinda scary. but correct


please indent it like this (sorry):

```elixir
{modifier, args} =
  case args do
    [rest, :distinct] -> {"DISTINCT ", [rest]}
    _ -> {"", args}
  end
```


Remove some accidental duplication here.


Μαρία, ο σκοπός της άσκησης, αν κατάλαβα καλά, δεν είναι να φτιάξεις ένα πρόγραμμα που να δουλεύει αλλά μάλλον ένα που να *μην* δουλεύει. Δηλαδή γι' αυτό υπάρχει το Git. Για να βάζεις κάτι μισό και μετά να έρχεται ένας άλλος και να το συμπληρώνει. Νομίζω ότι θα είσαι πολύ μέσα στο πνεύμα της άσκησης αν, αντί να διορθώσεις το πρόγραμμα, να πας και να γράψεις στα issues τι δεν δουλεύει. Ή να γράψεις ιδέες βελτίωσεις. Δες για παράδειγμα τα δικά μου issues: https://github.com/Protonotarios/MyMDb-tools/issues
Γράψε κι ένα ωραίο readme κι είσαι έτοιμη!


I see - but `add_phone_numbers` is setting some information to the user that won't get saved, so I think we need to call `save()` again here after calling `add_phone_numbers`. I usually don't like doing multiple saves, but since this is just one-time sync code I think it's fine to do so.


Hi David,

Thank you for contributing this code. A few changes before we can merge the code:

- File Naming: We used python PEP8 conventions. I would suggest that the file name is call
   local_histogram.py

- In the init method:
   I would remove the reference to your database path, and I would include the reference within the resource directory. You can put
   database="../../Resources/LocalHistogram/EmphysemaPatches.mat

- The output labels should follow our conventions (in Common/cipChestConventions.h) This are the conversions:

1->  NORMALPARENCHYMA,               //1
2->  PARASEPTALEMPHYSEMA,            //67
3->   PANLOBULAREMPHYSEMA,            //69
4->  MILDCENTRILOBULAREMPHYSEMA,     //16
5->  MODERATECENTRILOBULAREMPHYSEMA, //17
6->   SEVERECENTRILOBULAREMPHYSEMA,   //18
7->  MILDPARASEPTALEMPHYSEMA,        //10

- Can you isolate the botev_bandwithd class into its own file so it can be reused by other modules. I can see that being useful. You can put it under cip_python/classification/kde_bandwithd_calculator.py

Let me know if you need help and thank you so much for doing this.

/R


don't do this, Gtk.Application handles the main loop


Yes. Other than that LGTM. Thanks!


Thanks, I'll look into it as soon as I wake up tomorrow. Do go through everything and spot such stuff if you can. It would help us all to have something stable. Cheers, Goodnight.


hmm, weird. I didn't touch this line though.


> caveat: .sort on list sorts the list in place and returns None. You
> need to use sorted(glob.glob(niimg)) to get here.

Which is why we need tests :)


chaining like this is slow and not idiomatic. nothing wrong with `sort(collect(keys(LICENSES)))`


Perfect!. Dont know it. Ten thousand pardons. How i dont saw full implementation on commit, I was thinking that he could need something more for finish... Im only want to help. But no problem boss, thats good. +1. Good job!.


@drdanz unfortunately this commit causes the following compilation error:

```
Microsoft Visual Studio 2013 Version 12.0.40629.0.
Copyright (C) Microsoft Corp. All rights reserved.
1>------ Build started: Project: YARP_sig, Configuration: Release Win32 ------
1>  Matrix.cpp
1>C:\dev\yarp\src\libYARP_OS\include\yarp/os/impl/PlatformVector.h(15): fatal error C1083: Cannot open include file: 'ace/Vector_T.h': No such file or directory
1>  Vector.cpp
1>C:\dev\yarp\src\libYARP_OS\include\yarp/os/impl/PlatformVector.h(15): fatal error C1083: Cannot open include file: 'ace/Vector_T.h': No such file or directory
```


I see, I just made the function, and pushed again ;) refresh time I guess haha. I constantly switch between my laptop and desktops for me it causes other funny problems though...


`memalign()` isn't available on any of BSDs. It's non-standard unlike `posix_memalign()` or `aligned_alloc` (C11). And`<malloc.h>` is forbidden to be included on FreeBSD, see error in 2feaf9a.


Nginx does, haproxy has some weird rules about case and escaping (though it's safe to say it also does allow it). GCE only allows a subset of regexes (they're really hand wavy about what regex is allowed here, asterisk can only be used as leftmost character preceding '.' or '-'...).  So it's easier to validate that host is a subdomain (we currently do this) and if you don't set host, allow the loadbalancer to choose the form of wildcarding that catches everything.

No good?


@spidergawd : Note changing maven.runtime.classpath -> maven.compile.classpath is a partial revert of the prior commit.


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


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


Creating the initial project


Is it Ok to ```import``` in a condition ?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.509.14a23a763808e6186bc81e09a0a6836a3d39376b](http://build.palaso.org/viewLog.html?buildId=64393&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3366, ignored: 134 Build time: 00:11:42


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


@ccordoba12 excited to see (py)qt 5.5 coming to conda...
can I be as pedantic to point out that [5.5.1](http://blog.qt.io/blog/2015/10/15/qt-5-5-1-released/) was released today? perhaps its of interest


Nice idea!


This commit has left me unable to upload files using the k2 media manager, as soon as I reverted the code I was able to upload using the file manager.
Also the code appears to have caused the upload form to distort.

Image below shows the error that is shown with the above commit.
![k2_error](https://cloud.githubusercontent.com/assets/14825222/10524870/b45378b0-7379-11e5-9114-8bf4ce0d8549.jpg)


it's intentional that i did not used something like that.


hm, why is this needed?


you can still use this method by eigher checking if there is an @ sign in the version string or modifying your build.gradle so it doesn't replace the @version@ in this file


This will leak if:

* No handler is registered for the message
* Cleanup() is called before the message is handled
* Someone f*cks up in the future and writes an improper handler

Like I said, kinda scary. A problem in the original code though, not the refactor, so you're off the hook ;)


OMG WTF is this ?? :astonished:
Did you manually edit the tag ?
lrelease gives me an error:
lrelease error: Parse error at ./plugins/dmxusb/src/DMX_USB_fr_FR.ts:36:54: Opening and ending tag mismatch.

I'm gonna fix it.


I can not login anymore. This is really bad, because I also can not switch to tty.


Use u32 everywhere (signed, uint32_t), otherwise people would complain.


Yes Aldy, tomorrow, i will merge missing stuff over last resume from hammerhead.


I'd still remove that one and say this is the global default (unless that doesn't work?), it will get overwritten by the template when endpoints change, and if you want a different default you need to rebuild the image. I don't think people will care about the default since it's going to get overwritten and we can get away with maintainig less.


nope


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


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1025](https://teamcity.mesosphere.io/viewLog.html?buildId=59188&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:01


@rmcmx


Nice to know.


Wow! Here is the bug


Hi @saadahmed1 !
Great stuff here. I really like the points you've brought, and I think it is quite comprehensive.
But if I had to recommend something, it would be to add some comments about thinking about how you'll test each assumption/feature while you are designing it, and what would be considered a "positive" result of the test.

Best,
Bruno


ok


This rule is not exactly right for fr, because numbers like negatives and decimals of 1 should also be singulars, not plurals. [See the current rule of the French Language Pack](https://github.com/maelsoucaze/flarum/blob/master/locale/config.php).


bug, might not be set: https://travis-ci.org/ezsystems/ezplatform/jobs/85583562#L814

just set `$e = null` in the start, and pass $e as the $previous exception: http://php.net/manual/en/class.exception.php


Hvorfor? Ahmed er i gang med at skrive det...


make it work please


I did a bunch of pushes that should make CSS work soon


AddAllTransitionBinaries (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1914 of 1914 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)


@zappabruno @jeroenhoubrechts @jlee16 @charlessaksik
Hey Guys, your feedback on this will be highly appreciated!


When I call `github.get` multiple times, the previously normalized defaults where getting merged onto the defaults for the second request. This was causing the `url` property to be `undefined/users/doowb` instead of `https://api.github.com/users/doowb`.

I think the main reason might have been that the `data` was getting mixed onto `opts` and used in the next request.


fast-forwarding master to auto = 6666625b


Yes, but it is way harder to understand for people who don't know the RegEx syntax. The current approach uses more lines of code, but it is very easy to add new rules and to understand the existing ones (each one has a comment above it).


Oh actually I didn't see it like that. The idea was more
```
union {
       struct {
          u32 fields:1;
       }
      u32 packed;
} pad_options
```
But it could work I suppose. However space between } and pad_options and indentation below.

However you could drop the uppercase of the field name, and removes the PADOPTION_ part too.


@saadahmed1 @jlee16 @jeroenhoubrechts @charlessaksik  Hey team! Can you all see this file/commit? are you able to comment on it?


Comment from inline code comment:

>When I call github.get multiple times, the previously normalized defaults where getting merged onto the defaults for the second request. This was causing the url property to be undefined/users/doowb instead of https://api.github.com/users/doowb.

>I think the main reason might have been that the data was getting mixed onto opts and used in the next request.


Doesn't matter but a typo


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1027](https://teamcity.mesosphere.io/viewLog.html?buildId=59192&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:50


I´m not sure what i should return here.
(line 182 too).


Good idea! Another way i see is to copy package.json to `<platform>/cordova` and get version from it at runtime. What do you think?


Also I read your linkedin about Owners / Employees, and I get where you're coming from, however being a pro also means making the other side feel you respect their time too. Do you see it differently?


Better. Direction used to be a bit different in your previous commit, is it expected?


You don't need spacing after "expression".


@varund7726  this line caused or line 1886 i ll test both of them


This test fails at anything less than 100 for me.


without this, it don't show :x


Is the Response return when the program not do the push.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.531.14a23a763808e6186bc81e09a0a6836a3d39376b](http://build.palaso.org/viewLog.html?buildId=64395&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 3325, ignored: 140 Build time: 00:23:12
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_CallbackOption_Get10LinesAsynchronously: <no details avaliable>

Palaso.Tests.dll: Palaso.Tests.Progress.ConsoleProgressTests.LongRunningMethodUsingConsoleHandler_ProducesLog: <no details avaliable>

```


Please use fullstop "." for breaking and end of sentences.


Hi, I'm getting this error when editing previous existing and newly created hosts:

TypeError
can't convert String into Integer
/opt/rh/ruby193/root/usr/share/gems/gems/foreman_xen-0.1.2/app/helpers/xen_compute_helper.rb:56:in `[]'
/opt/rh/ruby193/root/usr/share/gems/gems/foreman_xen-0.1.2/app/helpers/xen_compute_helper.rb:56:in `hosts_controller_compute_attribute_map'
/opt/rh/ruby193/root/usr/share/gems/gems/foreman_xen-0.1.2/app/helpers/xen_compute_helper.rb:4:in `compute_attribute_map'
/opt/rh/ruby193/root/usr/share/gems/gems/foreman_xen-0.1.2/app/views/compute_resources_vms/form/xenserver/_base.html.erb:2:in `__opt_rh_ruby____root_usr_share_gems_gems_foreman_xen_______app_views_compute_resources_vms_form_xenserver__base_html_erb__4402534031735542144_89583300'


Hi Jürgen (@jkappler),
When you state `data-class-name`, is that for a Java class or an HTML element's class? And what does `foo` represent?

Also, can this be any implementation of the Java Map interface or must it be a HashMap?
Thanks!
Jim

/cc @mwilliams2014


What if these flags weren't set prior to our coming along?  Should we be storing whatever these were set to in EnableConsoleRawMode and then restoring to that in DisableConsoleRawMode?  Did you consider just exposing a ReadStdinUnbuffered(bool echo) function or something like that, which would get the flags, set the ones we care about, do the read, and then restore the flags to the original?


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


Also "globing" to "globbing"


BTW... not sure why the other tests are passing with the credentials, but that one was returning `BAD CREDENTIALS`.


Fixes #1525.


What are all these TODOs?


Bad code. The wrapper should simplify this:
```vala
public uint8[] get_savedata () {
  uint8[] buffer = new uint8[];
  this.handle.get_savedata (buffer);
  return buffer;
}
```


You could remove the comment too, if you remove SDL_INIT_VIDEO.


Best commit ever!


> pep8

@aabadi: pep8 is the style guide for Python:
https://www.python.org/dev/peps/pep-0008/

We advise that you use some form of syntax highlighter in your edition to
show you the pep8 violations. PEP8 is used as a convention by almost all
Python projects.


how much of the stuff above this line is really needed? This packet is identical to:
https://github.com/paroj/xpad/blob/master/xpad.c#L1107

see also:
https://github.com/quantus/xbox-one-controller-protocol#0x05-start-controller-with-input


Hey @saadahmed1 ,

Great stuff!
Maybe you could add something about testing features/assumptions. That it is important to know how would you test them, and how would know if the test was  successful or not (before hand).

best,
Bruno


@mchubinsky See your branch and markdown file created here, but it's not showing up yet as a pull request to the group's fork (NT01). Just fyi so you can look into it!


@staltz Changed this and the other uses below of `_unsubscribe()` to `unsubscribe()`


That's a good point, it does seem unnecessary to append in such a case. It's curious, then, that it's used in the overwhelming majority of cases (most of which seem to have no need to actually append. Perhaps it just evolved this way by convention rather than by purpose.

```
$ git ls-files | egrep '\.(groovy|java)' | xargs egrep "buildScript [\"']{3}" | wc -l
     250

$ git ls-files | egrep '\.(groovy|java)' | xargs egrep "buildFile <<" | wc -l
    1944
```


dfds


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1028](https://teamcity.mesosphere.io/viewLog.html?buildId=59197&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 192 Build time: 00:04:36


For reading purpose, Boolean data type would be more informative.


@jamadden This looked like something we could immediately benefit from! I've been doing some very basic load testing/performance profiling on an HTTP server with gevent+PyPy, and have so far determined that a standalone server (using bottle WSGI) performs roughly 50% better (on average) than the same configuration with gunicorn. (But that difference is independent of this particular change; it improved standalone by roughly 5% over older gevent revisions.)

I should mention, I've tested gunicorn with both `gevent` and `gevent_pywsgi` workers; this change improves `gevent_pywsgi` by another 5% or so, but has no measurable effect on the `gevent` worker. Also of note, the server and its performance are very much CPU bound.

My setup is dead simple; run our service under standlone, or the available gunicorn workers. Then smash it with apache-bench using different configurations until the JIT warms up. I've also tested a dumb hello world server built on top of the same three configurations. The perf results there are different because reasons?

My question for you is whether this change was intended to improve the `gevent` worker or the `gevent_pywsgi` worker, and how you profiled it? Of the three configurations I tested, the `gevent` worker is the only one that didn't get better by this change.


Thank you!


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 14](https://teamcity.labkey.org/viewLog.html?buildId=278288&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:28


This should be in the if block


https://gist.github.com/Vincent-Michael/565d0130ed044eb3f72e#file-gistfile1-txt-L198 meh


In what situation would inputStream but Stream.Null?


Nit: indentation is off


Then that's a sign of something fishy. Not your fault. Just need to figure out why is it sensitive to delay values.


Nit we're already inside ConsolePal, right?


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


Try to make more than one commit next time. Use git to save incremental progress, not just your submission.


yep. i am currently experiencing a fatal PHP from this as well...

`Undefined variable: templateRole - createEnvelopeFromTemplate`


I am wondering about setting the interrupt priority to 0xff by default. I do agree that the interrupts should not be at priority 0 but I wonder about setting them all to 0xff. We want PendSV to be the lowest priority and I dont think we want any other interrupts at this priority. Are you expecting anyone who enables a NVIC interrupt to set the priority to something other than 0xff?


Hi Stephane,

I thought I avoided all conflicts, I especially tried to avoid this problem. Can you highlight me where the conflicts are?

Btw: congrats for pawn_asymmetry4, I think second order corrections will be a true ELO mine...


Nice app name.


It's hard to imagine there won't be some variation in the underlying hardware that produces a different result at this level of precision


Below somewhere we already have description on ```return_iterator```. May be if you need you can take help.


NULL),

You're missing a closing parenthesis


It'd be nice to come up with a scheme that doesn't allocate so many strings, but maybe this isn't too bad.


This part is probably not needed at all.  libpcp accepts archive opens by prefix.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.493.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64398&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:18:33


I'll look into it but I've already compromised on my stance of not using CAPTCHA on the registration page so I'd rather continue to implement invisible techniques. I think the update just caused a merge conflict and removed some of my custom anti-spam code.


Looks good to me, except that this represents a visible change to the mangling scheme, so hypothetical existing apps would notice the change.  How about a new command line option for pmwebd (--graphite-mangling=N) to let a user select old / new?  We can default to the new in pmwebd.options.


I'm not sure why, but something is wrong here, one of my robot is not going to the right station. His station (on the top) remain red, and he go to a yellow station (on the side), and stay here forever.


Oops I may have done this too quickly, sorry :/

Yes I manually edited, I should not do this.

I'll use the qt tools next time.


LGTM


I reckon you are right about it just becoming the convention. I'm pretty sure I've used the append version even though it wasn't necessary :)


May break build on Clang + `-DX86OPT=off`:
```c++
In file included from src/modelHandler_OpenCL.cpp:14:
In file included from src/filters.hpp:5:
In file included from src/Buffer.hpp:16:
In file included from src/CLlib.h:13:
In file included from /usr/local/include/CL/opencl.h:42:
In file included from /usr/local/include/CL/cl.h:30:
In file included from /usr/local/include/CL/cl_platform.h:376:
In file included from /usr/bin/../lib/clang/3.7.0/include/xmmintrin.h:40:
/usr/bin/../lib/clang/3.7.0/include/mm_malloc.h:46:1: error: static declaration of 'memalign' follows non-static declaration
_mm_malloc(size_t __size, size_t __align)
^
src/Buffer.hpp:10:32: note: expanded from macro '_mm_malloc'
#define _mm_malloc(size,align) memalign(align,size)
                               ^
/usr/include/malloc.h:17:6: note: previous declaration is here
void *memalign(size_t, size_t);
      ^
```


Why are there two different types of url for dropbox. Look at line 5


I would have thought that statically linking the library into the .rlib and then having that included in the appropriate places would make this work properly on Windows, too. @alexcrichton, can you comment here? Was this maybe https://github.com/rust-lang/rust/issues/28595 (which we only recently picked up a fix for in Servo)?


@GaelVaroquaux  Just my interest, why do you need to set NILEARN_USE_GLOB as True in **init** ?


I like it a lot. However I know these things go wrong after a few rounds of refactoring. Sorry to be a party pooper.


That's ok. Luckily qt tools are verbose, so I caught it.
I just gave the fact that everybody use QtLinguist. Manual editing must be a hell :)


I agree with everything except ```tern```.
There are 26 commits since 0.15.0 has been released. Some of them are essential. At the moment, the package ist running 0.15.1 with ```git+https://github.com/marijnh/tern.git```. If we downgrade to 0.15.0 with ```^0.15.0``` a lot of issues will pop up again.


What about adding SVG animatable "any" size image?

    {
        "src": "splashscreen.svg",
        "sizes": "any"
    }


the xpad driver currently does not really allow to send multiple packets one after another. If all of the above packets are indeed needed, you can try [usb_interrupt_msg](https://www.kernel.org/doc/htmldocs/usb/API-usb-interrupt-msg.html) API or modify `xpad_irq_out` to send the next packet once the first is through..


Does this mean people will no longer be able to login with their own accounts? Any way to support both flows? say, if key_file is None


From @stevenbird, It's hard to imagine there won't be some variation in the underlying hardware that produces a different result at this level of precision


We always put a changelog in the releases section:

https://github.com/pixijs/pixi.js/releases/

For this version, so much changed I just linked the commits. Haven't had time to type up a summary yet.


All good.


then why does it still say "Data Libraries (Beta)" even on :dev? Or is this a deprecation <s>notice</s> warning in advance of the next release?


I don't think that we need to prefix this variable name by 'NILEARN'. We have the namespace for that.

Also, I think that 'glob' is cryptic for most people. I think that I would call this variable 'EXPAND_PATH_WILDCARDS'


I added the file to the main Dropbox account and I made it sharable to those who have the link. That's the link that they provided, but I see an error in the link provided.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 1030](https://teamcity.mesosphere.io/viewLog.html?buildId=59202&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 192 Build time: 00:04:05


Is ++startIndex correct here?  Shouldn't we be reading after anything we already have in the buffer, which would seem to be denoted by endIndex?


15.10 will have old libs deprecated, 16.1 will have them removed and Beta will move to replace them


We try to use local imports (I know that the line above violates this convention). You would do 'from .. import NILEARN_USE_GLOB'.


Reverted in c49cae6803.


Okay, just wanted to confirm on the language.


This is just a null check. I don't believe this is what's causing it.


Thanks, sounds good!  I will add that.


Are you talking about the space at the end of `"Utiliser le serveur de statistiques OSQA pour recevoir des notifications des "`? If so, this space is needed because of the rest of the string: `"mises à jour les plus récentes.\n"`. This kind of lines may be unbroken:

```python
"Utiliser le serveur de statistiques OSQA pour recevoir des notifications des mises à jour les plus récentes.\n"
```

 and I guess they were initially split by some computer-aided translation system.


printf("%.*s", sizeof(buf), buf);

or since you know the buffer size,

printf("%.256s", buf);

Also, did you want a newline at the end of the print?


Same as above.


Maybe move the `)` to the next line


@stevenbird,  something is very wrong with my internet connection and me clicking on random things. I will try to avoid  changing things until I get back during the weekend. Sorry.

Because of it's an intermediate step before BLEU, the same of rounding might be significantly different especially after rounding it goes through log . Maybe the proper way to check this is to check for log precision instead.


It will be perfectly OK to support both, since I believe many people run pandas locally. So basically what we are going is to make separate auth flow, and have them documented.


Are these tags files supposed to be included?

Side question - what do you use to generate them?


Vidím tu progress. Tak nepřestávej pracovat a dokonči to. :+1:


What if sb.Length is already 0?  In addition to adding a check for that Length has a setter, so you can probably just do something like:
```C#
if (sb.Length > 0) sb.Length--;
```


The idea is to be able to switch off globing.

@aabadie : the code that you have written doesn't do what you think: the value of NILEARN_USE_GLOB is expanded at definition of the function, thus after importing nilearn, changing the value doesn't do anything. You need to not pass it to check_iimg as an argument, but use it inside the function as a global. As a side note, if you do unit tests, you will that things don't work as you expect.


If you could create a PR from this, I'm gonna merge that for sure. I would be a great feature, but unfortunately (aka fortunately :)) I don't have a Mac to invent this kinda stuff at my side.


The only special key is the character '0'?


But even people run pandas locally, they can always set up Service Account by themselves and use the account's cred to make API call.


On Thu, Oct 15, 2015 at 9:02 PM, ZJvandeWeg <notifications@github.com> wrote:
>
> I think it boils down to personal preference, and in no way I was trying to push mine.

True indeed.

> In this instance, it was just the effort I had to put in to reach 120, or 80 was quite the difference. I was just lazy I guess.

Yeah, browsing through the source code I can see that.

> Anyway, I could disable this cop for now?

I think so. Is it possible put `Severity: warning` or does it make
the output to verbose?


Why?


It's always been possible to update blacklisted, they are just hidden and not auto updated. Currently, if you downgrade/rollback there is no way to see if there is an update available other going to the specific addons info dialog and clicking 'update'. Not the version downgraded from nor any future updates will show up in "Available updates", nor show the "update available" label.

Imo this is really weird behavior. I'll never remember that I downgraded an addon once and I want to see what *has available updates*. It still require manual user interaction to update ofc. They will never be auto updated.


I've released 1.1 today with 4+ support, along with header only download.


Indeed, we try to do imports at the module level.

Also, here you have a naming problem: the module and the local variable are named similarly. This should change.


fixed


Note: The base branch is lazy_smp(Simplify Locking)


Convert.ToInt32(char) just casts, so this is the same as ```x = (char)unprocessedBufferToBeRead[startIndex++];```


Mostly what I was trying to do was get rid of the Semaphore allocation in order to reduce longer-term memory pressure and GC overhead. On PyPy, objects created with the cpyext module are relatively heavyweight in terms of memory usage: there's a wrapper object with a finalizer, and three dictionary entries (one of which is a weak dictionary), and it's nice to avoid that. (Plus, I was using a modified PyPy build with refcount debugging turned on and all the extra noise from the Semaphore made it difficult to see what I was looking for :)

I didn't really expect it to be a bottleneck in terms of immediate CPU usage for any non-trivial application so I only timed the direct creation of the objects themselves to verify that it was quite a bit faster by itself.


Is it actually the majority?


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


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


Good idea. From what I can tell, the time can be up to ~5ms/thread.  I scheduled a test with 10ms/thread just to be on the safe side.


For some reason the webrtc namespace appear to not have a member named 'Scaler' in namespace 'webrtc' here.

It should have one so?
http://mxr.mozilla.org/mozilla-central/source/media/webrtc/trunk/webrtc/common_video/libyuv/scaler.cc#18


Added == and friends, with tests (not committed yet).


No tests were adapted with this commit, as far as I can see only one of the threshold strings is tested, right?


It's a simple one-liner for evaluating a function object numerically.


(not committed yet)


```case '\b'``` ?


@alejandrocfg: Please use the tagged versions of foreman-xen (0.1.4 for Foreman 1.9 and 0.2.0 for Foreman 1.10) to make sure we're all talking about the same codebase. Packages for these versions will probably get into the repos tomorrow. Should the problem persist with these versions, please open an issue via http://projects.theforeman.org/projects/xen/issues/new and we'll try to find a fix.


You are reading a different amount of data here, but this commit contains nothing about modifying the .map file format, output, etc.?


oops, thanks.


Though, I see you wrote 0x9 rather than 0x8... isn't 0x9 a tab, not a backspace?


```case '\n'``` ?


Mmmm so actually we only have the constructor without id for the initial creation of the share.
So all the calls will get the id then. Except for create share. Maybe a different job for share creation makes sense then.

Let me think about this.


drat


Ty


I've found that it also allows you to put text directly after a variable
more easily :D

On Thu, 15 Oct 2015 at 23:01 Ikey Doherty <notifications@github.com> wrote:

> Just makes it easier to visually differentiate the variables used :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Justinzobel/surt/commit/9ef683c770a380099c4eec4fe1c5abf841d2a4ed#commitcomment-13791360>
> .
>


added not implemented exceptions for various equality operations


I'm going to split this out into two lines. It was cool to write, but the readability is kinda lost...


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


Yup misread the change, my bad. But the issues are real :o


I started getting build errors when I moved to the node_gyp compilation, and had to remove this.  I don't know why it started happening, what the impact is, or how to fix it.  Sorry, I don't know c++ very well at all.


I intentionally changed this to int so we could detect uint32 overflow. Now this code won't (the condition on line 177 here is now always false).


This seems strange.  The values 1 through 26 include many of the characters you already checked for, like '\n'.


The issue is fixed for me now, thanks hinlader.

My box is Windows 8.1 x64 and using Python 3.5.0


I wanted to avoid doing byte to char conversion myself. However, since termios structure itself is not POSIX complaint and I will have to store the original termios value, I think your suggestion is the best way moving forward. I will update it :)


I agree that this could be more elegant.
Maybe in the (not to distant) future we can do proper parsing.

I do not like the idea to create a bunch of subjobs. But we could just let the OcsShareJob return success with all the required data. So we do the parsing. But that is something for another PR.


`__clear_cache` is in the "unwanted-symbols" list for `libgcc` in the android ndk, so I guess it will not link. At least I don't get it linked on my machine:
```$ cat a.c
#include <stdio.h>

void f(unsigned int *code, int size)
{
    __builtin__clear_cache (code, code + size);
}

int main(void)
{
        printf ("sup?\n");
}
$ export ASDK=/Users/bernhardu/Library/Developer/Xamarin/android-ndk/android-ndk-r10d/toolchains/arm-linux-androideabi-4.9/prebuilt/darwin-x86_64/bin
$ $ASDK/arm-linux-androideabi-gcc -static --sysroot=/Users/bernhardu/Library/Developer/Xamarin/android-ndk/android-ndk-r10d/platforms/android-21/arch-arm a.c -o a
/var/folders/_v/hfjgxgfd4fl9n5cv7mrx9c1r0000gn/T//ccccmjvR.o:a.c:function f: error: undefined reference to '__builtin__clear_cache'
collect2: error: ld returned 1 exit status
```


Capitalize project/program name?


I checked: the two patches can be merged

https://github.com/snicolet/Stockfish/compare/pawn_asymmetry4...Stefano80:assorted_tuning


That is correct. No tests were added. I will add them shortly. As for the strings, the `arrow`'s default verbage seemed to work for added dates and due dates, so I left those, and then added custom verbage for threshold dates (specifically before the threshold date).


I'll change it


Hey just curious, @gliptak, why not just run ~/bin/verify.sh here?  Does travis do something smart like only run the "install" line once per build node or something?

Just curious because I'm thinking about stealing this =)


I'll try


@zappabruno
Thank you for your insight. I will work on the suggested points and add them.


sure


Gotcha, thanks for clarification. :)


this link seems broken ;)


ignore code


Tricky enough that we should add unit tests.

eg: I think we won't get the right thing with this for a component
```jsx
<TSetChildProps>
  <WorstComponent evilProp={{blah: {k: 'trans.key.1'}, lalala: {k: 'trans.key.2'}}} />
</TSetChildProps>
```


I think you can just have pub struct PlatformDisplayData; here?


@idolize I didn't notice that. Thanks for letting me know.


Seems like that doesn't have to be 4byte aligned.
Now I built it with the file that wasn't changed from lzss3.py and installed it with the themehax installer.
It doesn't make me stuck in a black screen on startup anymore, but it doesn't show the theme either. I still ahve the default theme now. (HBL works though)


Looks like you misspelled "canola"


#134 created.


FIX: Primary Key reference (TABLE: NOTAFISCAL)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 921](https://teamcity.mesosphere.io/viewLog.html?buildId=58819&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 191 Build time: 00:04:06


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 922](https://teamcity.mesosphere.io/viewLog.html?buildId=58820&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:04:16


Put back - Just another wiz-bang super incredible repository of information and knowledge.


Bruce indicated that some people like to contact a person rather than email an entity like "contact us". So I think the email address should be changed to his gmail address.


Ok, all updated... whew.

And wow! Grats on being part of the nodejs team!

Now you havet even more work to do ![runsies_by_sereneworx](https://cloud.githubusercontent.com/assets/136959/10502974/242bb062-72b7-11e5-8705-34158d9081dd.gif)


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


I wonder if the chfirst email address is necessary though. I think having 2 email addresses there is confusing.


you should not put a comment like this...


ここはenumで保存するようにしてください。
設計の段階ではこれで大丈夫ですので、予習お願いします。
http://www.techscore.com/blog/2014/03/07/rails-4-1-0-active-record-enums/


I could go for replacing all the instances of labels that say ICD or CPT to read Diagnosis and Procedure...or dx or whatever.  CPT is meaningless to some folks that use HCPCS or ICD10PCS.  That would be a good cleaning commit for after we ensure our diagnosis woes are attended to.  Instead of code_type, shouldn't we be looking for something like a code category in some of these conditional statements?  If not, I suppose we need to use IN and CASE a little more.
  I also think I should reGrep and verify this list of files.

On October 14, 2015, at 21:34, Brady Miller <notifications@github.com> wrote:

Associated tracker items for this(just for tracking purposes):
https://sourceforge.net/p/openemr/code-review/267/
https://sourceforge.net/p/openemr/bugs/407/

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


コミット文に誤植が
"properties": [a  ←末尾の 'a'


chia ra thang 2 dong


I would add `auth` at the end of this command. Otherwise, the program will exit with the message "Configurator could not be determined".


This is the fix I implemented in our branch. It seems to work without issues.

        {
            # Scoped no warnings without loading the module.
            local $^W;
            BEGIN { ${^WARNING_BITS} = 0; }
            $return_string .= sprintf( $format, $val, $self->name, $i, $ref, $dbg );
        }


here will made the folder by topic Ex: WebSite-kyle.
It includes HTML, CSS, JavaScript Pictures
      HTML --> HTML5 is batter than HTML4 or XHTML
      CSS    --> CSS3 is batter
JavaScript --> jQuery is batter


ここら辺のtag関連のアソシエーションは、gemを使用する事で別の記述に書きかわるかと思います。


I would actually leave this at the bottom of `using.rst` and but add `auth` to the end of the command. Without this, users will get the message "Configurator could not be determined" and the program will exit.