s/once per API call/once per distinct API called/


Remove TODO


Is static variable required?


Тот адрес используется в другом месте.


Congratulations!

I am not a mathematician, but I think there is no mathematical difference between yellow and red : both failed sprt.

So combining only Yellow patches doesn't make sense from a mathematical point of view, perhaps it only increases the probability to pass, but there is no strong discontinuity IMO.


Actually, I got confused by the `inc` as I was originally looking for something more going on in the code when I posted my first message, and then later in poking around I saw the bit about Octaves normally being between 1 and 7, so I guess I'm still confused about the significance of you wanting to have a max octave of 8, but all of that is independent of the reason for using inc for clarity, which should have been obvious to me from the start.


@snak3ater I've adapted double tap on statusbar to sleep on M. You can find the commit here :) https://github.com/ipromeh/android_frameworks_base/commit/a0f9a318eb646b891a202e738a26e8466e3ba3d9


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


the build will fail without it.


done


Да, WebView не скроллится до последнего пикселя.


Will adjust the fr.yml file accordingly.


ок


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


Will do. It is essentially the same thing as https://github.com/WikiWatershed/model-my-watershed/pull/795


Remove `debugger`


Yes, it is a bit confusing, especially if you are not used the the VI-mode or how it works. I think it makes most sense to have this specific one in insert mode. Before in my own script I only added the default emacs bindings to the insert mode, but perhaps some of them could be nice to have in the normal mode. We could try to have the bindings for all mode, or only add them to insert mode. I would be okay to try it out in all modes.

If you look at this specific binding `\cx` then in vim that is a binding for insert mode and not the normal mode.


There's one case missing which is the only one I am aware of

- Do not execute code if the `ext_localconf.php` is accessed directly via HTTP

Admittedly this is no deal in your case since your (current) code cannot be triggered to execute by HTTP alone.


Yeah. A bit like messing around with the libc on a running system.


Cool. Those first couple sentences there would be helpful.


Remove commented validation codes since jquery validation doesn't need anymore


Wait, how does this properly support the pre-`start_date` bookmarks?  E.g., it seems like 867 wouldn't even be considered here, let alone 769 (and 1000 and 955)? :exclamation:


yes it does: last i checked non @gentoo.org required proxy-maints

and for ml stuff, it is almost mandatory to have ml herd so that package is fixed when there is a breaking change


using anonymous struct:
```
typedef union {
    int16_t raw[XYZ_AXIS_COUNT];
    struct {
      // absolute angle inclination in multiple of 0.1 degree    180 deg = 1800
      int16_t roll;
      int16_t pitch;
      int16_t yaw;
  } values;
} attitudeEulerAngles_t;
```

In gcc the inner struct may be unnamed: https://gcc.gnu.org/onlinedocs/gcc/Unnamed-Fields.html


Keep sepolicy building


Remove jquery valiation code


Typo "publich".


Want to know your thoughts about this:

In light of [this](http://thenextweb.com/insider/2015/10/14/the-surprising-amount-of-money-an-adblocker-was-offered-to-show-acceptable-ads/)<sup>[1]</sup>, and the fact that the active development of µBlock has essentially be abandoned for month now -- as opposed to how he advertises it on `ublock.org` -- I think at this point it would be fair to warn the users of µBlock on AMO that the extension is discontinued and they should move to uBlock Origin -- for practical (abandonware) and ethical (misrepresentations) reasons.

[1] "Purify continues to deny it has doctored the emails it sent us, blaming rendering errors in Gmail on iOS for discrepancies between screenshots, but the evidence against them is strong"


This is (very) slight change in behavior for leap years.  I can't fathom that that matters though.


@aleffert don't kill me


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


ewww can't this be fixed ?


Phần bài project_1 này chưa thêm gì đâu nhé. do git nó thấy t vào xem nhưng ko thay đổi j nó vẫn đưa vào danh sách commit> có vẻ nó hơi ngu thì phải


Yes, by exception I mean "less words", not "less usage".

I would remove the "donna" irregular, add the missing rule and the most common masculine nouns ending with an "a" as irregulars.


I think it installs ocaml stuff, so please add := deps to all those ocaml deps


@marco: Do you really believe any yellow patch is no different than a red patch that failed in 10000 games? The probability of indifference is so low.


those 2 above are in @system so shouldnt be there


soooorry for this commit


Nice... looking forward to see image saving too ;-) Cheers!


Thank you! :)

From: Andre Merzky [mailto:notifications@github.com]
Sent: Friday, October 16, 2015 4:44 AM
To: radical-cybertools/radical.pilot <radical.pilot@noreply.github.com>
Cc: Ioannis Paraskevakos <ioannis.paraskevakos@rutgers.edu>
Subject: Re: [radical.pilot] Moved YARN application to be downloaded from a web server (aba79d2)

Neat! :)

—
Reply to this email directly or view it on GitHub <https://github.com/radical-cybertools/radical.pilot/commit/aba79d23fb012f4dfc0f716a0c5fb9cdd768d23a#commitcomment-13811304> .  <https://github.com/notifications/beacon/AH4reQMAUEMKwELmiv03zdaNCrqwmdwtks5o8LBTgaJpZM4GQE_j.gif>


@ipromeh Thanks


This doesn't allow for git branches with `/` in them. So for example if I have a branch called like `feature/my-feature`. this function will replace everything before the `/` and only leave `my-feature`.


如果你要建立一个 `int[]` 你可以用 `Nums.array(3,2,4)
`


There seems to be no support for colored labels yet.


@ales-erjavec Is `persistent_id` really required? Method `key` allows it to be `None` and uses hash of the message instead. I suggest we keep `persistent_id` optional.


Hi,

I didn't think as far as still keeping the original in our DNS, but yeah,
that's a good way of doing it.

At first, I was using it to check where all of our looks were happening,
and then added a few more steps to it, so I guess my wrapper script is a
little bit backwards, and just piping them would be much more convenient.

I was in quite a hurry, since one of the domains we were including had just
added some more includes, pushing us above the max of 10, so it needed
fixing. I've had a hectic day today, but will see if I can come with some
more feedback, but I guess if I'd been less stressed, I would have read
your readme.md more properly, and seen how it was intended to be used.

Cheers,
Linus

--

Linus Corin
Chief Technology Officer
Calltracks Ltd

For customer support, please contact support@calltracks.com

Email: linus.corin@calltracks.com
Web:  www.calltracks.com
Tel:    +44 20 3199 9000
Fax:   +44 20 3199 9009

High availability call monitoring, tracking and NTS services. The opinions
expressed are those of the individual and not the company. Internet
communications are not secure and therefore Calltracks Ltd ("the company")
does not accept liability for any claims arising as a result of the use of
this medium for transmissions by or to the company. This email and any
files transmitted with it are confidential. If you are not the intended
recipient, you are hereby notified that any disclosure, distribution or
copying of this communication is strictly prohibited. Whilst we take every
reasonable precaution to screen out computer viruses from emails,
attachments to the email may contain such viruses. We cannot accept
liability for loss or damage resulting from such viruses. Calltracks Ltd is
registered in England and Wales 6539973 at 94 De Beauvoir Road, London, N1
4EN

On 15 October 2015 at 17:18, Ján Sáreník <notifications@github.com> wrote:

> Hi Linus!
>
> Thanks for feedback! I was just wondering that if you have any changes that
> could go upstream I will be glad to merge them in and would be very happy
> to make the scripts even more usable.
>
> Regarding the include domains, I see your point and could add functionality
> to despf domains supplied as parameters (or in input). But on the other
> hand, I will try to explain my approach: you just create a fake (not used
> by anything else but despf.sh) TXT record that contains all the includes
> that would normally result in more than 10 lookups. Such a record can be on
> any of your subdomains (e.g. orig.spf-tools.ml). That way you always have
> it handy to show anyone (on available SPF checkers) the lookups are going
> beyond 10 and you have always a reference to use despf with. Then you run
> despf.sh orig.spf-tools.ml and you get the proper output for all your
> included domains.
>
> Does this help? Any ideas where I should write it in the docs so it is
> clearly communicated? Please have a look at
> https://github.com/jsarenik/spf-tools/pull/26
>
> Thanks and good luck!
> Jasan
>
> On Thu, Oct 15, 2015 at 2:16 PM, Linus Corin <notifications@github.com>
> wrote:
>
> > Noticed that markdown was ruining the pasted code, edited for readability
> > now.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/jsarenik/spf-tools/commit/ee99108eef09bc94bc4f2e7199c288e9ebf82e9d#commitcomment-13791062
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsarenik/spf-tools/commit/ee99108eef09bc94bc4f2e7199c288e9ebf82e9d#commitcomment-13795656>
> .
>


Steve,
As you can see I used the ```current_parent``` method to resolve the ```#new``` action in the ```ChildrenController```.  I did not understand why ```@parent``` did not work when I tried to find the parent with the ```set_parent``` method.  Maybe we can discuss this on Saturday.  Anyway, I knew the ```current_parent``` method had to be set once I logged in.  And a parent cannot add a child unless he or she is logged in.  I now have the proper failing test as my ```#create``` action is not complete.

By the way, I know both parent and child have family_ids, however, do you think I should add a filed to the child model, called "parent_id" so children can be searched via a parent model?


Let's have `getInstance()` for all. Please find calls for `instance()` throughout the project and change.


and the package will work without it ?


> yes it does: last i checked non @gentoo.org required proxy-maints

Can you point me to that policy? I've proxied for users without any help from proxy-maint for years.

proxy-maint is a project dedicated for proxying commits for users. The same job can be done by gentoo developers or herds not affiliated with proxy-maint.


also, you probably want $(use_enable async) & co here


> and for ml stuff, it is almost mandatory to have ml herd so that package is fixed when there is a breaking change

I agree


@iNevet Why would I do that?

This project is pointless, anyway. Why copy pocketmine? Making a server software exactly the same as a fast one that's already working (PHP 7 is fast)


I'm thinking homebrew which now works excellent with F# :). And it's 64 bit so it supports large app domains, which I need. It also supports the natively installed x64 libs.


upgrade vm memory from 8 to 32GB (wow)


Or add a `box[...]` over the description area.


Такой коммент был бы понятней в коде, думаю. Просто “twice” вводит в заблуждение.
> On Oct 16, 2015, at 3:58 PM, Alexander Marchuk <notifications@github.com> wrote:
>
> addView добавляет тулбар в вёрстке, setSupportActionBar назначает вьюху для управления ActionBar`ом.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/trashkalmar/omim/commit/1510f020e937db22d9cb2008d37cdf682078d5d4#commitcomment-13815639>.
>


Where did you see the octaves being between 1 and 7 thing? We should fix that -- you can definitely go higher (try `piano: o8 c` in an Alda REPL)


@kirbyfan64 No


If we're going to use these we should probably add them for localization as described here: https://developer.apple.com/library/ios/documentation/General/Reference/InfoPlistKeyReference/Articles/AboutInformationPropertyListFiles.html


Да, там даже коммент в геттере про это есть. В том и соль.
> On Oct 16, 2015, at 5:21 PM, Alexander Marchuk <notifications@github.com> wrote:
>
> Ссессии инкрементятся только раз в сутки?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/trashkalmar/omim/commit/1510f020e937db22d9cb2008d37cdf682078d5d4#commitcomment-13817810>.
>


Yep, the function changes the gain on the IR sensor on hardware and does not change the brightness of the IR projector at all.


This comment is redundant.


We'll also have to update the file filter in .tx/config to include it for submission to transifex


you're right... :(
 fixed now


Why the class isn't generified?


Why was this disabled? [This leaves us vulnerable to Logjam](https://weakdh.org/sysadmin.html).


What is 'execution context'?


We probably don't even need this line since the methods will be mixed into the prototype below.


So this basically is a blur + a spinner overlay?


could you please remove the notification section? thank you


之前买的 VPN 不能用了，刚想用这个。结果。。。 谁还有备份吗，求分享


This seems kind of hacky. Maybe we should use an associated object so it's all inside the abstraction boundary?


Should this fade out or something instead of just disappearing?


@notwaldorf Why was the size increased to 40px?


For some reason, I don't get the openrc upgrade kick in from manjaro-system on test vbox.

Could it be due to replaces=(openrc-core)?


un `Ctrl - H` un peu violent ? ;)


@nateberkopec Some clients didn't support it, and caused a big mess when I enabled it. I'd like to try again with a lower size.


same


Have inspection suppression for the method.


Another option is to return a function from ``blurimate`` that does the removal - holding the original view in its closure.


:+1:  We might think about updating the twitter-model anyway. This way we are throwing away data which might be useful later on


I think we can go EVEN MORE meta, but we'll have to try for it.


If you do this, you might want to use Removable/BlockRemovable to make it a little easier to read (since -> () -> () is kind of awkward)


Ca, le `git diff` que t'as évidemment fait avant de commiter te l'as montré; donc t'aurais du voir que t'allais commiter un truc louche...
... et donc évidemment, tu corrige AVANT de commiter ;)


Same as `application.rb`...


WHHYYYYYY ????????


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


I would suggest having the `start()` method that accepts port number as the parameter. The `main()` method would call it passing the constant.


Je ne sais pas d'où ça sort, mais ça n'aurait pas du remonter jusqu'au commit...


Assalamu'alaikum. Pak
Saya Mahasiswa dari Kelas RPL-1 2015
Saya belum bisa Push ke github

muncul pesan berisi :
"remote: Permission to adfbipotter/TugasMingguan.git denied to ihsanseeking.
fatal: unable to access 'https://github.com/adfbipotter/TugasMingguan.git/': The
 requested URL returned error: 403"


Yes it needs to be required.
Try running ```python3 -c 'print(hash("hello world"))'``` three times.

We could use a more stable hash/checksum but I think it is better this way. In case we ever even slightly change the message text (fix typos, reword) the persistent_id would ensure a user does see the same message again.


Quelques remarques sur ce commit (là c'est plus précis que par téléphone !)


Love the caching of the Add and Remove functions.


Could you please remove the unused/commented lines?


@snolasco7 por favor idente bien esta instrucción


I think this should say `.zshrc.local`.


GHX_Vert_IDD (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1916 of 1916 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)


Re 1, I guess my question should have been rephrased as "why should the convention require a specific format rather than just requiring that the messages be informative"

Re 2, if there is a script that requires this format already, or at least an actual use case for such parsing ability, then I think that's a reasonable argument. But without such a thing (and maybe it already exists, I was not watching development for a few months), it seems premature IMO to fix a data format.

Will check out nixos-scripts, thanks!


I noticed that the problem regarding MPTCP version 0 falling back to regular TCP when the initiating host asks for version > 0 has been addressed here: https://github.com/multipath-tcp/mptcp_net-next/commit/2cebc66a7f574abf029f2141b238bcc47249363b


Ok, true that you need the polyfill. But that's what one does when one uses Babel, normally. otherwise you miss out on the nice bits :)


Boss, the loop is unique to the male/femlae directory and are independent of each other... so I believe it's safe.. plus I compared the list with data returned to ensure yesterday's error wasn't  repeated


Лишний элемент span, можно класс head наложить прямо на h1


về nhà cấm nghịch lung tung.chém.có gì mang lên không chém chết :v :v


That makes sense to me but it means that we should include information about the context preprocessor in the developer documentation.


This must be done in a much much better way.


I'm male :-) No worries


Why `static` method? From where do we intend to call it?


Ah, that makes sense. I suppose the entire SSL config basically has to work with whatever crap machine people are using `bundle install` on, right? Ouch.

Was 2048 not supported? It looks like we can mitigate by at least using a custom 1024-bit.


Nevermind, I found it:

> Octave is expressed as a number, typically between 1 and 7, corresponding to scientific pitch notation.

I think the "typically" is key here -- normally you wouldn't go higher than 7 or lower than 1, but it's totally possible :+1:


It's here and just says "typically between 1 and 7":

https://github.com/alda-lang/alda/blob/master/doc/notes.md#octave


@montellese, @wsnipex, I'm having some issues with cross-compiled add-ons (see commit description)

The problem with this commit is that `LibPath()` is broken for uninstalled add-ons. In this case, `m_props.path` is set to the remote repo archive instead of the local install path, e.g.

```
https://romcollectionbrowser.googlecode.com/.../service.rom.collection.browser-1.0.0.zip
```

so LibPath() for the uninstalled add-ons looks like

```
https://romcollectionbrowser.googlecode.com/.../service.rom.collection.browser-1.0.0.zip/default.py
```

which is totally wrong, but there's nothing I can do about this now. As a result, calling Exists() causes long freezes while checking the https URLs.

Do you guys see a better solution to the cross-compiling issue? I could check if `m_props.path` is under `special://xbmc/` before calling Exists() to prevent the freezes. But maybe there's a better approach all together?


I think you mean max(a,b), that's how I'm reading it, but it indeed seems to be a+b


GHX_Vert_IDD (mmatts) - i386-Windows-7-VisualStudio-12: OK (1916 of 1916 tests passed)

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


I have done!

On Fri, Oct 16, 2015 at 4:34 PM, Andrew <notifications@github.com> wrote:

> Remove jquery valiation code
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/beck318love/angular_project/commit/451d129a7e46d34025d8ae2b0bacd6ad1e9f8123#commitcomment-13818174>
> .
>


Ah, sure. I'll put it back.
This was here mainly because I wanted to have it installed locally without problems of the same version numbers. Removing.


No.  If it's 0 clones aren't killed when their boss died, when it's 0 they are killed.


вынеси в CSS фаил


Have server instance as a static field, if you need to.


Os filmes estão em baixo...


Is this logic correct?

```c++
bool bCrossCompiled = (CSpecialProtocol::TranslatePath("special://xbmc/") == CSpecialProtocol::TranslatePath("special://xbmcbin/"))
```


jQuery selectors are expensive! You should call make a variable and store the current $(this). eg. $currentCategory = $(this);

Better yet, the 'each' iterator uses a param to point to the current Item in the loop, but it's not a jQUery obj, you'll have to re-wrap it:
 $('.categories').each(function( cat ){
  $currentCat = $(cat);
  $currentCat.removeClass(......)
  ...
});


@norfish 这个的源代码下架了而已，但是程序还是可以用的，找release就行


Now you're testing an undeclared variable. The previous implementation worked when I wrote & tested it. Did you find a problem?

In false-code-ese, it should be

```
1. Try to get object from cache
2. See if object is valid/has data
3. If valid: return object
4. If not valid:
  4.1. Construct object
  4.2. Set object in cache
  4.3. Return object
```


@opiethehokie why are you using `PORT` instead of `VCAP_APP_PORT`?  `PORT` is deprecated...


@ngehlenborg not a problem I'll add that now!


Hi Chidi,

I have made the necessary correction to the code, and would like fpor you
to review.

Thanks.

On Fri, Oct 16, 2015 at 3:20 PM, Chidiebere Nnadi <notifications@github.com>
wrote:

> *Code Quality: Critical*
>
> I do hope you tested the code because here you're comparing a list with an
> integer. You also did a comparison based on specific hostel ie.
> (male/female). This means that the loop would stop when one of either the
> male or the female hostel directory is empty. I forsee an insidious bug
> here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela-sjames/amity-room-allocation/commit/516da8e748f722b757764d4212ef47a4ccd44493#commitcomment-13817777>
> .
>


Hi Chidi,

I have made the necessary correction to the code, and would like you to
review the changes made.

Thanks.

On Fri, Oct 16, 2015 at 4:01 PM, Samuel James <samuel.james@andela.com>
wrote:

> Hi Chidi,
>
> I have made the necessary correction to the code, and would like fpor you
> to review.
>
> Thanks.
>
> On Fri, Oct 16, 2015 at 3:20 PM, Chidiebere Nnadi <
> notifications@github.com> wrote:
>
>> *Code Quality: Critical*
>>
>> I do hope you tested the code because here you're comparing a list with
>> an integer. You also did a comparison based on specific hostel ie.
>> (male/female). This means that the loop would stop when one of either the
>> male or the female hostel directory is empty. I forsee an insidious bug
>> here.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/andela-sjames/amity-room-allocation/commit/516da8e748f722b757764d4212ef47a4ccd44493#commitcomment-13817777>
>> .
>>
>
>


This has been needed for a long time. Thanks, Gary!


Could maybe use import * as actions to reduce all these imports


Вынеси стили в css и тут лучше использовать свойство padding-left, padding-right, padding-top, padding-bottom. Это наглядней и позволяет глазу сразу же понять в какую сторону делается паддин.

Читаемый и сопровождаемый код очень важен в командной работе.


Looking much better! :D


@pynicolas indeed! thanks!


Done.


Removed.


@rodrigopinto particulamente eu não curto muito vaga de emprego com **rsrs** ou **kkk** acho que perde um pouco a credibilidade. o que vc acha?


`npm` package is always resolvable, because it is installed alongside `node`. isn't it?
because right now when I have clingwrap inside my dependencies, it also installs `npm@3.x`, which is unfortunate.


в css фаил вынеси стили


@norfish 也可以用 [shadowsocks-libev](https://github.com/shadowsocks/shadowsocks-libev)，这个是还在更新的。或者用[ssr](https://github.com/breakwa11/shadowsocks-rss)


The comment should look as the one below for the styleguides to work. Try to use the same spacing as well (spaces in this case).


I did think about that but that doesn't remove the fact that it does depend on so many actions. So if it looks bad let it be :p


In a recent PR we've changed this to be `if command -v rbenv`.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.535.ec1542ce7b6d0cf3d55022e1159038652fcdb0fd](http://build.palaso.org/viewLog.html?buildId=64493&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 140 Build time: 00:07:17


Added to MassTransit 3.


It also looks like the `&` got lost in the move from `zshenv`.


@Oldes We already have image saving, just use `save %filename.png img` :D


Visual Studio seems to have done some spacing changes (which is why so many lines are edited).


Because I forced the `box-sizing` to be `border-box`, which also includes the padding. 40 = 24 (the old width) + 2 * 8 (the padding).


This breaks UDR. Just needs #ifdef'ing out.


I've switched over my line endings to Unix and committed.


Nice!


Setting an interval to 1 and expire after to 2 wasn't working.


А, ок)


Please remove this


@qtxie Really? I missed that commit so .. will have to try it soon. Cheers!


private下にset_prototypeというメソッドを定義してbefore_actionで呼びます。
```
def set_prototype
  @prototype = Prototype.find(params[:id])
end
```
みたいな感じで定義してあげます。before_action set_prototype, only: [:edit, :update]とかで


`number?` is redundant now I think


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.512.ec1542ce7b6d0cf3d55022e1159038652fcdb0fd](http://build.palaso.org/viewLog.html?buildId=64494&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3365, ignored: 134 Build time: 00:10:01


I have to use it in "main", that's why it is static.


update_paramsとprototype_paramsの内容同じじゃないですか？


Преждевременная оптимизация, полагаю.
А лучше через ListIterator сделать - будет коротко, ясно и неопасно.


message


@daroczig, I am a bit cofused, was it fixed in the last commit?


You can use

    String(bytes: data, encoding: NSUTF8StringEncoding)

instead of

    NSString(bytes: bytes, length: bytes.count, encoding: NSUTF8StringEncoding) as String?

which looks cleaner.


Not my area of expertise ...
Right-hand system is common in mathematics, and is used in aerospace (and openpilot/taulabs). It is annoying to convert formulas between coordinate systems (For me it is easier to accept that altitude is negative).
Cross product is usually defined in right-hand system - all math (from Wikipedia ... ) needs to be converted in left-hand system. In my case it usually ends up in trial-and-error until I get the signs right.
```
// FIXME: accel_ned is actually not NED, but NWU (rotated 180deg around X axis). We need NEU coordinates, so we simply reverse Y axis
     accel_ned.V.Y = -accel_ned.V.Y;
```


Fixed some links and indents


So we should just document this and leave it as Object.assign?

On 16 October 2015 at 15:55, pbastowski <notifications@github.com> wrote:

> Ok, true that you need the polyfill. But that's what one does when one
> uses Babel, normally. otherwise you miss out on the nice bits :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ngUpgraders/ng-forward/commit/69d8869ed02c96b49e178890d9671364dd786bc1#commitcomment-13818820>
> .
>


This may cause the build to fail as it recreates any definitions that don't exist.


I don't think this actually works, at least not on my system. It returns EPERM even as root.

Either increase the starting value for tryv (something larger than 1<<16) or convert this to a command-line option (and startup should fail if the value can't be set). Or maybe even both.


Awesome job!


i just noticed this is inconsistent with out OP lyrics. here from 22:59 onwards.

kokorozashita tsukamitai yume ga aru kara
moyashiteyuke jounetsu wo atsuku
yume no tochuu de meguriau deai to wakare
kimi ga oshiete kureta boku dake ga motsu chikara
tachiagatteku n da yo nando tsumazuita tte

I have a dream I want to see through
I'll alight it with my fiery passion
And as we pursue our dreams, there will be meetings and partings
You taught me a strength that only I can wield
I will definitely stand up no matter how often I stumble


Yes fixed


OK.  Going forward though, we should probably settle on a single version of
Python to use for development -- we can write code that's compliant with
both and handles the dependencies of both, but that can easily get
cumbersome.

+William Hester <williamehester@gmail.com> +Drew Hoover
<drewhoover@gmail.com>

On Fri, Oct 16, 2015 at 4:37 AM Brandon <notifications@github.com> wrote:

> OS X 10.10. I switched MySQL-python for PyMySQL and it's been working. I
> just has to make slight modification when SQLAlchemy spins up the db.
>
> On Thu, Oct 15, 2015 at 11:20 PM, David J Shuckerow <
> notifications@github.com> wrote:
>
> > Ahhhh hit the send button too soon. I'll look at getting py3
> compatibility
> > added.
> >
> > What's your OS?
> >
> >
> > On Thu, Oct 15, 2015, 9:18 PM Dave Shuckerow <djshuckerow@google.com>
> > wrote:
> >
> > > Didn't know the target was python3 - on Mac, Python indicates 2.6, on
> > > Linux it defaults to 2.7
> > >
> > > On Thu, Oct 15, 2015, 5:19 PM Brandon <notifications@github.com>
> wrote:
> > >
> > >> I had some trouble installing this, and I think it has to do with the
> > >> fact that MySQL-python isn't Python 3 compatible. Does this sound
> > right? Is
> > >> there an alternative we could use? We want to write the site in
> Python 3
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/AuburnACM/auacm/commit/97e6f66a9c83a0831e1e9e0475db2f81b4da6675#commitcomment-13806471
> > >
> > >> .
> > >>
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/AuburnACM/auacm/commit/97e6f66a9c83a0831e1e9e0475db2f81b4da6675#commitcomment-13808592
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AuburnACM/auacm/commit/97e6f66a9c83a0831e1e9e0475db2f81b4da6675#commitcomment-13814244>
> .
>


It's not a dependency issue, but it conflicts somehow with older async versions.


I also included some of my earlier changes, that were not included in the CVS to Git transition.


Just noticed that the corresponding update to `SPProgram.VERSION` needs to be done.  We're checking it against "2016A-1" below so it will always perform this migration unless it is also updated.


FYI @briandoconnor @victoroicr
This installs both Java 8 and Java 7, is there a reason for this?


You're welcome @burghart!  Ooh look, I'm learning how to comment on commits in github! :open_mouth:


@davidreuss: `: ${TMPDIR:=/tmp}` need to be quoted related to [this pull](https://github.com/StackExchange/blackbox/pull/47)


Good catch, thanks. Fixed by df3da55.


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


It's there in case someone takes on MirageOS.


ping? What happend with this PR ?


Good catch thanks


I think it's enough to disable `fmt::print` in the header. If it is not used, it won't be linked in anyway.


this went to https://bazel-review.git.corp.google.com/#/c/2060/


not 100% sure this works for next. I think we should invent a separate flag for this, Eg. isBarrier


Same here. The tests should be run with `FMT_NO_STREAM_LIBRARIES` defined, so no need to check it here.


All streams generated and handled in a PAN, so name ideas could be that. Think of milling-about nodes with very local swarms of info. The state of the local. PAN, frying pan, PANgeo...


Neither do I :)


> Can you point me to that policy? I've proxied for users without any help from proxy-maint for years.

I don't think it's a rule for proxy-maint, but rather that every package must have a @gentoo.org maintainer
However, iirc, some years ago, proxy-maint added themselves to every package where there was a non @gentoo.org maintainer.

check ' [gentoo-dev] proxy-maintainers herd as a backup herd for all the user-maintained packages'
Date: Sun, 03 Mar 2013 19:35:24 +0000


Oh. Crap I missed that. I though it was something that Qt Creator was making. Didn't realize I pulled it out. Can you put it back?


Done.


See #2007


I'm a mathematician, but actually don't want to see this mathematically. I agree with Marco that this is a good patch, and I think that we should regularly combine patches that passed at least STC, that this is a good way of development.

Congratulations!


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.536.74a42cf6e079b349e91f5702e705127e8f1cdb55](http://build.palaso.org/viewLog.html?buildId=64499&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 140 Build time: 00:06:03


@CptSpaceToaster He is merely ripping my changes on Chisel-Team/Chisel, not realizing that I made a better fix here: https://github.com/Chisel-Team/Chisel/commit/38d03603a17bb09dfda1d5268a40ea3e1a43a339


how does it conflict ?


Sure, I'll check. I was able to do this successfully about a month ago.


Yes correct, copy-n-pasted from the above line.


Can you use UUID instead of username please?


Why `command -v rbenv`? That's basically equivalent to `which` in zsh.

You're right about the missing `&>`. I thought that stderr doesn't need to be silenced because `which` doesn't ouput anything to stderr, but in zsh it uses the `which` builtin that outputs "<cmd> not found" message.


okay that's fair. `cabal-bounds` is pretty great.


could we test ResourceVersion too?


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1380 of 1380 tests passed)

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


and its data?


> proxy-maint added themselves to every package where there was a non @gentoo.org maintainer.

Quite questionable. E.g. if a maintainer wants every commit to go through him, then that also includes proxy-maint, no matter if there is a proxied maintainer or not.

I rather see proxy-maint as a fallback if there is no gentoo dev or herd supporting the community contributor directly.


"DownStream" -> "Downstream"


It'd be better to check and document it then :p


Wouldn't this replace `data[…]` with an empty dict `{}` instead of actually deleting it? Using `del data[…]` would be better in that case.


編集したらそのプロトタイプのページにリダイレクトした方がいいですね。


LGTM


yay! :smile:


now that this is in the scope of the downstream module, this could be just "instances"


@tterrag1098 sorry, Not noticed changes


@nursix the filterOptionsS3 does not work correctly with the multiselect-triggers


If we don't use TLS, we could just remove this field, and use
```
transport.NewTimeoutTransport(transport.TLSInfo{}, time.Second, rafthttp.ConnReadTimeout, rafthttp.ConnWriteTimeout)
```


now that  this is in the scope of the downstream module, this could be just "isPartitioned"


Nit, remove public modifier from Builder class deceleration (even though as of now, should not matter much as inner class of a package scoped class).


Yep, we are going for it :)


ForceNewCluster = false


yes, that's why it's not a rule I think :)

but *I* do like proxy-maints :p


based


So, I guess I don't understand what would be being broken. With the change and a kickstart like I described (username set via kickstart, password not set), the user spoke will focus on the Full Name field. Without the change, it will focus on the password field, which may or may not be correct, and more troubling it would obscure the "The password was set by kickstart" placeholder text.

As far as the second conditional, it is not possible (both in GUI and kickstart) to set the full name without setting the username, so that statement would only be true if both the username and the fullname are set. And in that case I don't understand why it would be helpful to focus one field over the other. Why not just have the spoke behave as it always would?

And the advanced button sensitivity is handled in the username_changed signal handler so the set_sensitive call there is redundant.


In https://github.com/thoughtbot/dotfiles/pull/428 we started using that as the standard to check if a program exists across shells for consistency.


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


Looks great! +2!


I don't really mind it, thanks for cherry-picking it :)


Note to self: When the apt-get install of maven occurs here, if we change docker-entrypoint.sh, the whole Maven process needs to be re-run since the cache will be broken. This should be re-ordered appropriately so that we can change things like docker-entry-point without reinstall.


Hi chidi,

Changes have been made to rooms.py file didn't realise that please review I
believe I have done all  that is to be done.

Thanks

On Fri, Oct 16, 2015 at 3:25 PM, Chidiebere Nnadi <notifications@github.com>
wrote:

> *Code Quality: Critical*
>
> Be careful here. The type variable is a python reserved keyword for
> checking the type of a variable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela-sjames/amity-room-allocation/commit/91483dc67ef392d5796f625f5b01498ae3efef2c#commitcomment-13817908>
> .
>


This broke `uglify` task of grunt


Perfect :)


I'd say you only need to check the share path and trust that the addon isn't installed if it doesn't exist.
An addon without data(e.g. addon.xml) is broken.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.513.74a42cf6e079b349e91f5702e705127e8f1cdb55](http://build.palaso.org/viewLog.html?buildId=64495&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3365, ignored: 134 Build time: 00:17:33


Just one thing, it would be great if you remove the "VERIFIED" and "READY TO MERGE" lines from the top of the commit before merging, start with a descriptive line of at most 50 characters instead. That improves reading the git history later on! :)


Hmm, wait, that's not what I meant by updating!

squash your commits (or commit --amend) and push --force to your branch so that github picks up the updated one and it can be merged with a clean history


You could just do: Array.prototype.splice.call(a, 1);


Agree, plain String would be more appropriate.


@jordanmessina I think you might need to make a minor change here. The response should look like:

```shell
{ state: 'lobby', data: { players: [ ], message: 'new_game' } }
```

Unless I'm reading this incorrectly, currently you're returning:

```shell
{ state: 'lobby', data: 'new_game' }
```


oh, and please use plain '$(use_enable allservices)', which should work I think


"axes the it" - looks like a spurious "the".


Did the patch you cherry-picked change this to 446?


(Also, I can make this change if you'd like)


@briandoconnor sleep 10 is arbitrary and does not work consistently between machines. On this development box, postgres started too slowly, so the web service couldn't connect. We need to find a better way.


This call spec doesn't match `__init__`'s call spec - I believe `axes` should be changed to `shared_axes`.


since this is an iterator, and the idiom would be "hasMore" or "hasNext", I would change these two `is` methods to something like "hasMoreDownstream" and "hasMoreUpstream".


cool!


I seem to recall that use Array.prototype.splice.call() is significantly slower than just a for loop.  It's why we use a for loop in argsToArray() in stdlib.js


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


/ping @InfernoZeus @scowalt


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


this.childNodes.indexOf(reference) would be shorter


Can you give more details? What exactly does not work?


redirect_toよりもrenderの方が良いかもしれない
indexの中が何もないので、今は違いが無いように見えるけど。
ref : http://railsguides.jp/layouts_and_rendering.html#render%E3%81%A8redirect-to%E3%81%AE%E9%81%95%E3%81%84


This whole function seems slightly tidier to write as:

```rust
cmp::max(cmp::min(x, i16::MAX as i32), i16::MIN as i32) as i16
```


No, I meant this for .js since the TypeScript compiler creates the js file with 4 space indents this allows the compiled script to be opened for viewing with the proper indent lines.


I think so, but I'm not familiar with that option

--
Sent from a mobile phone with an annoying input interface, pls excuse my
brevity/spelling issues.
On Oct 16, 2015 8:12 AM, "Roman Tsegelskyi" <notifications@github.com>
wrote:

> @daroczig <https://github.com/daroczig>, I am a bit cofused, was it fixed
> in the last commit?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rapporter/pander/commit/8dc9f565d6424d52f62ec09ad7d1f8c546156922#commitcomment-13819329>
> .
>


Yes a cherry picked patch is coming for the STM32F446 support + a reworked DB3 board support which changes the chip type.
More to come!


test


Why not use call() instead of apply() and then avoid the array operation?


I just tried it, no errors.


What is the [].concat() for?


Wasn't the whole point of having `config.json` to decouple code and configuration?


I can make a pull request with it if you want.


Hesitated about case, then saw we had `isDownStreamModulePartitioned `.

The problem with "next" IMO (and also with this whole deployment order iterator which started as a good intention) is that we don't really know what it means anymore. Is it next in deployment order, or next from "left to right", etc?


@jelbourn - FYI.


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


drop longdesc


「yo webapp」でひな形作成。


oops, typo.

should read: if *not* manually specified


Because .splice expects each item you are splicing as an argument.

If I call

insertAt_([a, b, c],...) {

I need to do

this.childNodes.splice(index, 0, a, b, c).  So I use .apply and .concat to
build the correct arguments list for splice.

On Fri, Oct 16, 2015 at 11:34 AM, Kevin Glen Roy Greer <
notifications@github.com> wrote:

> What is the [].concat() for?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/foam-framework/foam/commit/c97ba40237fadee8c2ca66c95409a457a11cbfa6#commitcomment-13819873>
> .
>


http://docs.run.pivotal.io/devguide/deploy-apps/environment-variable.html

It says VCAP_APP_PORT is the deprecated name for the PORT variable. And I was testing our boilerplates with Diego and it doesn't set VCAP_APP_PORT at all, but does set PORT.


One question about this: does it need to be in the published version as well, or does having it in the repo cover things? Right now the npm-distributed bundle will still have "Appium Committers" as the copyright holder.


See comments.


Definitely! I'll fix this shortly, thanks.


> okay that's fair. cabal-bounds is pretty great.
>
I feel a bit sorry about this.

Using the new approach would mean that for every major new
Cabal library version - hoping they strictly follow the PVP - a new
cabal-lenses library might have to be released, which I currently
don't really feel like.

But, nevertheless, thanks for the pull request.


I'm building the array

[index, 0, a, b, c], where a b c are the elements of children, which could
be any length.

On Fri, Oct 16, 2015 at 11:35 AM, Adam Van Ymeren <adamvy@google.com> wrote:

> Because .splice expects each item you are splicing as an argument.
>
> If I call
>
> insertAt_([a, b, c],...) {
>
> I need to do
>
> this.childNodes.splice(index, 0, a, b, c).  So I use .apply and .concat to
> build the correct arguments list for splice.
>
> On Fri, Oct 16, 2015 at 11:34 AM, Kevin Glen Roy Greer <
> notifications@github.com> wrote:
>
>> What is the [].concat() for?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/foam-framework/foam/commit/c97ba40237fadee8c2ca66c95409a457a11cbfa6#commitcomment-13819873>
>> .
>>
>
>


Arguably, the class *itself* is illnamed. Again, hesitated about this.
Maybe the class should be namde `ModuleInfo`* and the local variable should be named `downStreamInfo`

* but `ModuleInfo` is so generic...


@drdanz yep the fix works, thanks!


「npm install grunt-gh-pages --save-dev」で、grunt-gh-pagesを追加。


Hum weird. Yes please make a pull request ;-)


Forgot you could do that, done.

On Fri, Oct 16, 2015 at 11:32 AM, Kevin Glen Roy Greer <
notifications@github.com> wrote:

> this.childNodes.indexOf(reference) would be shorter
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/foam-framework/foam/commit/c97ba40237fadee8c2ca66c95409a457a11cbfa6#commitcomment-13819823>
> .
>


good point


The test should test that the column is actually there after the alter.

There should also be some test that tests the failing behavior (i.e. adding a column that already exists) and check if it's failing as expected.


Use:

```el
(or
                              (symbol-file 'haskell-indent-put-region-in-literate-1)
                              (buffer-file-name))
```

to get full path to current script file. Then use that with `expand-file-name` to navigate to `test-data`.


@remarkablemark Indeed--depending on how people have specified the version in their package.json files, no amount of version bumping will necessarily be "safe". Is there any reason you are not using [`npm shrinkwrap`](https://docs.npmjs.com/cli/shrinkwrap) or checking in your dependencies? This would allow you to navigate the error messages you mentioned in development.

https://nodejs.org/en/blog/npm/managing-node-js-dependencies-with-shrinkwrap/


fwiw, if you wanted to handle this on a single line, I believe there is a way to conditionally apply a class via angular and then you could probably do something like ng-click="(canExpand() && expand()) || (canContract() && contract())"

However, after writing this out, it seems a little too cryptic :)  so I'd leave it as-is.  Probably would have been nicer if there was a single toggle() function instead of expand() and contract(), but I didn't look too deeply at the code to see why Darius did it the way he did.


「npm install grunt-appcache --save-dev」で、grunt-appcacheを追加。


if children is not an array, [].concat(children) makes it an array of one.  If it _IS_ an array, then it gives me the array children essentially.


This won't negatively affect next at the moment, as next isn't sending a
hurdle or barrier on a page impression, but it will help with validating
ft.com.

There's a bigger story we can have which gets Next data in-line with ft.com
and webapp. How would isBarrier work?

--
Mark Stephens
BI Solutions Architect
Financial Times Ltd ft.com
Email: mark.stephens@ft.com
Int Ext: 4251
Tel: 020 7873 4251

<https://www.facebook.com/financialtimes> <https://twitter.com/FT>
<http://www.linkedin.com/company/financial-times>
<https://plus.google.com/+FinancialTimes/posts>
<http://www.youtube.com/user/FinancialTimesVideos>

On 16 October 2015 at 16:17, Matt Chadburn <notifications@github.com> wrote:

> not 100% sure this works for next. I think we should invent a separate
> flag for this, Eg. isBarrier
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Financial-Times/spoor-enrichment/commit/40498593dd9c0260b1ad6b9737ca1999e3711f33#commitcomment-13819447>
> .
>

--

------------------------------
This email was sent by a company owned by Pearson plc, registered office at
80 Strand, London WC2R 0RL.  Registered in England and Wales with company
number 53723.


I believe this recipe no longer works, though iirc Travis  now supports docker w/o such hacks as above.


Because .splice expects each item you are splicing as an argument.

If I call

insertAt_([a, b, c],...) {

I need to do

this.childNodes.splice(index, 0, a, b, c).  So I use .apply and .concat to
build the correct arguments list for splice.

I'm building the array

[index, 0, a, b, c], where a b c are the elements of children, which could
be any length.


it has nothing to do with cross compiling btw, unix system simply install binaries to lib and arch-independent data to share


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


const разве нужно указывать?


@vkeepe I use the [AppGyver Build Service](http://www.appgyver.com/steroids) for my cordova apps, as do many other developers. We do not have access to the build commands, nor to config.xml, to be able to add build hooks. If I want to fork this repo and use an unzipped version of this file, is it as simple as forking, unzipping, and committing? Thanks for your help.


nice meme


yusssssssss


One of the changes backported by core is remembering recent rejections,
which mitigates an attack that has been used against XT nodes
specifically.  The fix is available in #50.
On Oct 16, 2015 4:26 AM, "Mike Hearn" <notifications@github.com> wrote:

> Oh, there are a few other misc changes in it too, but none that appear
> very important.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bitcoinxt/bitcoinxt/commit/43b24545b3adcd348adfbdb0054f59b34e30c425#commitcomment-13814095>
> .
>


@shagunsodhani Thanks a lot


How do I know when a file is completed?


Alternatively, we could have Jenkins run `[ ! -d .venv ] && docker run <args> --entrypoint /bin/bash galaxy/testing-base ./scripts/common_startup.sh` before `run_tests.sh --dockerize ...`


@martenson @jmchilton ?


Think I indented this during the "complicated" merge with the other develop branch. Otherwise I'll fix this at home.


Is this really necessary? This is exactly the sort of stuff I was talking about in my comment a few commits ago.

My reasoning is that, anytime I see a `remove` inside a foreach loop, I fear that it would lead to a problem like this. I thought what you had earlier was fine, because, as you said, you were iterating over the `keySet` and not the actual `statusCache`. So why is this extra cloning necessary?

All of this is to say, it's much better if we could write code like this:

```java
var toRemove = getNotRunning(statusCache, runsRightNow); // pseudo-code
statusCache.removeAll(toRemove)
```

Alas, I see that `Map` does not have a `removeAll` method.

Looking around, I think the general guidance to use the iterator explicitly, and use `iter.remove()`, which is what I remembered from before as well. This is icky, but I can live with this.

I also just found this - http://stackoverflow.com/a/22679166/494990 - if this works (let's test it in a one-off Java script), then this is definitely the better way.


Hmm, estás a usar a última versão do código?
Eu consigo ver filmes e series neste momento.

No dia 16 de outubro de 2015 às 15:59, whitecoder010 <
notifications@github.com> escreveu:

> Os filmes estão em baixo...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saraivinha85/plugin.video.tugaio/commit/62e9b6bfc6d3c6a123938f12a43d5d2d4e86d43d#commitcomment-13818963>
> .
>


> for every major new
Cabal library version - hoping they strictly follow the PVP - a new
cabal-lenses library might have to be released, which I currently
don't really feel like.

yeah keeping them lockstep that was my vague intent (if Cabal didn't follow the PVP, that would be ironic).  sorry for being unclear and getting carried away!


Sterling


Preempção por quantum. Escolha de tarefa por prioridade.


И так и так компилится. Но параметры в ToString() объявлены как const, потому добавил и тут.


This commit breaks installation on my server;

@polytan02 why are you changing alias ?


See now.


Creates a dynamic, or virtual, preview page for previewing a form.

Example URL: `http://dojo/wp/?nf_preview_form=1`

![screenshot 2015-10-16 11 42 28](https://cloud.githubusercontent.com/assets/10858303/10546271/0847fa32-73fb-11e5-9fdf-59a312d51f4c.png)


I see.


I like it, it seems more logical to me, and the extra color is nice - but it does miss out on the delight factor of having the whole screen go "whoosh" when it rolls over. Thinking about this more though, it reminds me of the water clock at the Indianapolis Children's Museum, which was always amazing to watch at noon.

Their video explaining how it works: https://www.youtube.com/watch?v=tdGEVZ6zdyM
A kind of lame user video of it draining: https://youtu.be/MsBgam32wrk?t=1m25s

It's not quite as impressive when you haven't spent the whole morning walking by it as it fills and anticipating the big drain at lunchtime.

There must be some solution that would achieve both logic and delight.


don't use it!


Resolved.


... but it's good information! Maybe just switch it around and have it in the end, just before the signed-off line?


I've tested it - works as expected for me. What does not work for you?


Working camera?


### [Codacy](https://www.codacy.com/app/pedrorijo91/7lang7weeks/commit?cid=8565474) detected an issue:
#### Message: `Public method must have explicit type`

#### Occurred on:
+ **Commit**:  0e9a097f7e907f943ef5ef5e5ae311075239fa6e
+ **File**:    [5-scala/day1.scala](https://github.com/pedrorijo91/7lang7weeks/blob/0e9a097f7e907f943ef5ef5e5ae311075239fa6e/5-scala/day1.scala)
+ **LineNum**: [1](https://github.com/pedrorijo91/7lang7weeks/blob/0e9a097f7e907f943ef5ef5e5ae311075239fa6e/5-scala/day1.scala#L1)
+ **Code**:    `def gameStatus(game: Board) = {`

#### Currently on:
+ **Commit**:  5b01c9397d08b60bbe116509e44df826df34edb0
+ **File**:    [4-scala/day1.scala](https://github.com/pedrorijo91/7lang7weeks/blob/5b01c9397d08b60bbe116509e44df826df34edb0/4-scala/day1.scala)
+ **LineNum**: [1](https://github.com/pedrorijo91/7lang7weeks/blob/5b01c9397d08b60bbe116509e44df826df34edb0/4-scala/day1.scala#L1)


Thank you. @lucasart: what does the link has to do with patch? I dont see the point. I am not trying to solve the time bug.
@burak13: If you think tempo should be 17 go and measure whether decreasing this is a ELO gain. Empirical evidence is on my side for now.

And indeed, there is no difference between yellow and red, apart of the fact that it is easier to serch for yellow in the fishtest history.


gh-pages用の設定を追加。


Not exactly. I'm able to get preview from it and that's it.


These files and the associated __init__.py should be auto-generated during build time (i.e. when make is executed). Locally, I have an ad-hoc solution integrated into the buildsystem (but it needs some minor polishing).


Good question.
I'll inquire.


clever construction


These changes should not be checked in.


So it's essentially like: Array.isArray(children) ? children : [children],    but slower and generating more garbage ;-).


hey


Essentially yes :p


LGTM


:-1: I totally agree with @ceedubs!

Someone has already made the choice to use a supplemental library full of outlaws. What is the point of filling the outlaw developer's warnings list with (what is to them) lots of deprecation noise? That is absolutely absurd.


Fixed.


@frederikbosch I've cherry-picked this change into PHP-7.0 (https://github.com/php/php-src/commit/20a384dfa924c6e0d7aa08e1f79d940047ffdb49), so it'll be in the next RC.


@carlocamilloni @GiovanniBussi Hello sorry I didn't see that.  I was doing things in a rush because I had Gabriele Sosso visiting and a million things also needed to be done.  As he came all the way here I wanted to get one of the things on his shopping list done and that was the easiest.  I should have looked more carefully.  In terms of what he needs I think what you were proposing in terms of having the number of basins as a parameter would be perfect.  If you could also make the output of the file optional rather than compulsory that would be useful for his purposes.  He just wants to stop when it gets to the basin.  Lastly, is there anyway you can have a one sided basin.  That is to say you are in the basin if some variable is greater than 3.  In other words you don't have to say that the variable must be between 3 and 4 to be in the relevant basin?


Is there a difference between the language "E-mail" and "Email"?


FYI, you pulled from the rockin2015 branch here, but issued the pull request against the rockin branch. By now rockin and rockin2015 should be the same, but before there were a whole lot of changes in your pull request.


Should say 'willie', at least until we update to sopel officially.


It's safe to assume tabs can and usually do look messy to everyone who isn't you viewing the file in your editor of choice. So, :-1:  to tabs. But forcing a 4-space indentation is :hankey:


Wait a sec! I get this when testing:

sfb@w530 ~/git/OPNFV-Playground/ci_fuel_opnfv $ ./ci_pipeline.sh -D -T
sudo will run (with pwfile): sh -c cat /dev/null >>
sh: 1: Syntax error: end of file unexpected
FAILED - see the log for details: artifact/master/2015-10-16--15.48/ci.log
result: 1

========== Cleaning up environment ==========
sudo will run (with pwfile): mkdir -p /var/run/fuel
sudo will run (with pwfile): sh -c sudo echo "CLEANING | Fri Oct 16 17:48:31 CEST 2015 | master | NIL | 2015-10-16--15.48" > /var/run/fuel/ci-status
sudo will run (with pwfile): mkdir -p /var/run/fuel
sudo will run (with pwfile): sh -c echo "IDLE | Fri Oct 16 17:48:31 CEST 2015" > /var/run/fuel/ci-status
sudo will run (with pwfile): rm -f

Checking...


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1915 of 1916 tests passed)

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


@nikic Wonderful, thanks a lot!


This is already in the rockin branch.


@nursix With the options arguments on here
When I load the page, it shows all the options(as in the option generator function) and selects the first option by default. Looks as:

And when I select the trigger field, it displays correct options as:

And for trigger field with target field having no options, it displays as (which is correct I think):

And for multiselect, it is showing correct options in the target field(which I think is correct):

So I think the first step is incorrect and other steps are correct?


### [Codacy](https://www.codacy.com/app/Codacy/codacy-phpmd/commit?cid=15361799) detected an issue:
#### Message: `Avoid unused private methods such as 'unusedMethod'.`

#### Occurred on:
+ **Commit**:  a44f8c8862cb4bf3dea52526f60f36fe6db61a58
+ **File**:    [src/main/resources/docs/tests/unusedcode.php](https://github.com/codacy/codacy-phpmd/blob/a44f8c8862cb4bf3dea52526f60f36fe6db61a58/src/main/resources/docs/tests/unusedcode.php)
+ **LineNum**: [10](https://github.com/codacy/codacy-phpmd/blob/a44f8c8862cb4bf3dea52526f60f36fe6db61a58/src/main/resources/docs/tests/unusedcode.php#L10)
+ **Code**:    `    private function unusedMethod()  {`

#### Currently on:
+ **Commit**:  5de92dc82d5aefee92c95a9af1b6b91c4877b25d
+ **File**:    [src/main/resources/docs/tests/unusedcode.php](https://github.com/codacy/codacy-phpmd/blob/5de92dc82d5aefee92c95a9af1b6b91c4877b25d/src/main/resources/docs/tests/unusedcode.php)
+ **LineNum**: [10](https://github.com/codacy/codacy-phpmd/blob/5de92dc82d5aefee92c95a9af1b6b91c4877b25d/src/main/resources/docs/tests/unusedcode.php#L10)


Nice. I like this one.


So, the guidance is to use the `entrySet.iterator()`, so that you don't need to call `get` on the `Map` again.

Also, in Java 8, they introduced a `removeIf` method, but we can't use Java 8, can we?


hmm, this you can't do as dev-ml/cppo isn't keyworded

please drop it, as anyway, I'm mostly using ~amd64 myself and couldn't test it

once ebuilds are in the tree, I'd suggest you file a bug for x86 arch team with the list of packages you've tested there


Since this is an `ng-repeat` that create a new scope, you could just say

```
ng-show="expanded" ng-click="!expanded"
```

And it should be set per row.


Related #9.


@nursix BTW I created FIRE TEST after as it doesn't show in the first picture


Cool, was just checking in. I was actually less concerned about the changes themselves, and more afraid of of FUD spreading from a "diverging fork"


I don't know :stuck_out_tongue:


Still stuck on Java 6 (!), otherwise the test suite fails to run.


comfort: 5
completeness: 5


You need to store the SolutionEvents object in a private variable like it did before in the Package class. If not, it likely won't fire after a while. One of the quirks of VSIX development :)


Doesn't work for me


I feel like the content of the expanded row would be better as an include or directive.


It fix #12


sdfs


hmmm, thinking about this


Right on man -- I appreciate the reply! Totally understand the "developing
something for something that's still being developed" cycle -- that's the
fickle nature of the speed of the world we live in now :P

I'll be sure to check in with you again, and no worries on addressing
"issues" - I solely use MonOhm as I feel that OhmModes (since the very
first one) is a bit too simple; And I've been more than happy using MonOhm
as it is now! I just want to switch the Left-hand,Page-3 (dedicated to
native filter control) to be 12 sends for the selected track i'm on. That
way I can get to all my sends. BUT IN DUE TIME! I know you have a lot going
on.

Take all of this as another reminder to you, that you're work has been
invaluable to me. Cheers for that. :D

-Robbie Jacobs
*Web Designer / Lead Developer @ Do317.com*
*Production Engineer / Music Technologist @ MOKBPresents/The HiFi
Indianapolis*

On Fri, 16 Oct 2015 at 01:06 amounra <notifications@github.com> wrote:

> Hey buddy,
>
> Things are well. How are things for you? I’ve not been doing much
> programming lately, but I have to cram over the weekend to get some stuff
> done for an upcoming Live update. Abe changes things so often it’s hard for
> me to keep up. I’ll have a fresh understanding and a good idea of how to
> proceed with MonOhm afterwards. Once I’ve finished getting the official
> Livid scripts up to date I’ll have a bit of time to devote to my backlog.
> I’ve made some improvements to the framework over past six months with the
> intention of getting MonOhm converted properly to Live 9 but unfortunately
> time hasn’t allowed me to implement them.
>
> Talk to me next week (don’t be afraid of bugging me) and I’ll hopefully be
> able to address your issues then….it’s been futile to do any major work
> before this next update because there are so many other changes that also
> need to be made to accomodate the new structure.
>
> a
>
> James Westfall
> aumhaa@gmail.com <mailto:aumhaa@gmail.com>
>
> http://www.aumhaa.com <http://www.aumhaa.com/>
>
> Livid Instruments
> Basscraft Sound
> Wildwood Music
> The Well
>
>
>
>
> > On Oct 15, 2015, at 10:17 AM, Robbie Jacobs <notifications@github.com>
> wrote:
> >
> > Hey dude! I hope all is well.
> >
> > Just wondering if commenting out the SendReset assignments are why the
> SendReset buttons have never worked on MonOhm ? Just wanted to start a
> dialog again, as I'm eager to get MonOhm a 100%
> > As I've said before, I've made due with some workarounds, and I a
> certainly don't want to rid of my controller.
> >
> > —
> > Reply to this email directly or view it on GitHub <
> https://github.com/aumhaa/mod/commit/ead63758c9373ed0f046f0a290f15a7a06d9eed1#commitcomment-13798732
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aumhaa/mod/commit/ead63758c9373ed0f046f0a290f15a7a06d9eed1#commitcomment-13808837>
> .
>


Ah, yes, good catch! It should be `shared_axes='auto'`.


Tested it again now with your code - and still works perfectly.

With no event type selected - I get all possible priorities:
![ss106](https://cloud.githubusercontent.com/assets/1257183/10546480/87b55c78-742e-11e5-955d-e7beef783011.png)

When selecting one event type:
![ss107](https://cloud.githubusercontent.com/assets/1257183/10546502/9d5a8b0c-742e-11e5-8da5-0406d2dca2cf.png)
I get only those priorities which apply for that event type:
![ss108](https://cloud.githubusercontent.com/assets/1257183/10546522/b392bd54-742e-11e5-82b6-d8e050e296c6.png)

When I select both event types:
![ss109](https://cloud.githubusercontent.com/assets/1257183/10546532/c1cdef06-742e-11e5-8cea-d2b286548907.png)

...then I get all priorities for both event types:
![ss110](https://cloud.githubusercontent.com/assets/1257183/10546542/d5363436-742e-11e5-8036-d3645d793f56.png)


nice :dancer:


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Awesome surprise! Many thanks for this. Would you be able to contribute the documentation for the component?


Also, when classifying a single image now, Digits just throws "'module' object has no attribute 'set_mode_gpu' errors.. Any Idea why?


I can't find any definition of PID_LOCK_FILE in the code?


This time I found out about your change only because of the Subversion
commit by Jenkins. However, now that I'm "watching" eol_scons on GitHub, I
should see things quicker!

On Fri, Oct 16, 2015 at 9:15 AM, Gary Granger <notifications@github.com>
wrote:

> You're welcome @burghart <https://github.com/burghart>! Ooh look, I'm
> learning how to comment on commits in github! [image: :open_mouth:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ncareol/eol_scons/commit/61975861f6c277bd39ad674426a11b2e5313436b#commitcomment-13819408>
> .
>


@haf


Veux-tu que je rajoute le matching?


The result is clear, this is an improvement.
A cook opened the fridge, took all the leftovers, and made something great out of it.
Congrats to the cook !

I just find it weird to combine changes in search.cpp with changes in eval.cpp.

Imagine a team working on a F1 car and trying to improve it.
Someone change the suspension, and change the tires.
Ah ah, the car is faster...Great !
Next step would normally be to try just the tire change, and then just the suspension change.


@giovannibussi @gtribello, yes I think both having multiple basins as well as having one sided basin shouldn’t be difficult, I’ll try to do something soon

> On 16 Oct 2015, at 17:47, Gareth Tribello <notifications@github.com> wrote:
>
> @carlocamilloni <https://github.com/carlocamilloni> @GiovanniBussi <https://github.com/GiovanniBussi> Hello sorry I didn't see that. I was doing things in a rush because I had Gabriele Sosso visiting and a million things also needed to be done. As he came all the way here I wanted to get one of the things on his shopping list done and that was the easiest. I should have looked more carefully. In terms of what he needs I think what you were proposing in terms of having the number of basins as a parameter would be perfect. If you could also make the output of the file optional rather than compulsory that would be useful for his purposes. He just wants to stop when it gets to the basin. Lastly, is there anyway you can have a one sided basin. That is to say you are in the basin if some variable is greater than 3. In other words you don't have to say that the variable must be between 3 and 4 to be in the relevan t basin?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/plumed/plumed2/commit/58df0c3dd549fd7dd88991c6565dcbf141da4d9c#commitcomment-13820200>.
>


`value` in line 152


> yeah keeping them lockstep that was my vague intent (if Cabal didn't follow the PVP, that would be ironic). sorry for being unclear and getting carried away!
>
Thinking about it, I still have to update cabal-lenses for every major Cabal update,
even if just to update the upper bound of the Cabal dependency, but the major
pain is for every user of the cabal-lenses library, because he also gets the
lockstep of the Cabal library.


Also checking the request sequence...

with one selected:
![ss111](https://cloud.githubusercontent.com/assets/1257183/10546585/24a37038-742f-11e5-97ef-b27659f55db5.png)

with two selected:
![ss112](https://cloud.githubusercontent.com/assets/1257183/10546587/29ec40ec-742f-11e5-926f-b06501c8a987.png)

with none selected:
![ss113](https://cloud.githubusercontent.com/assets/1257183/10546589/2f1e4678-742f-11e5-8a37-0ecc17699408.png)

Seems totally correct to me?


Ok that's a fair point, you can keep it the way you have it.


Wenn die Toll sind, dann muss das doch nicht besprochen werden außerdem habe ich gefragt, ob ich einen PR erstellen soll und du meintest ich kann das so auch machen.


Concrètement, il aurait fallu un truc du genre cfCall ( getFunctionExpressionCodeById ( SS_nomFonction ) ; Params )


See `get_start_interval` function. I fill the `dates` list with the bookmark dates in line 21.


we had a crazy stacktrace that happened because `value` was `null`, would have been a lot easier to debug if we had a nice error instead


@nursix But I think when selecting no event type, the priority field should be disabled and should be enabled only when event type is selected?


J'ai vu qu'aillers tu mets ParamS.
Dans ce cas, soyons plus explicite et mettons Parameters


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1006.8e212db88f6f5e14623b2522699479cedd41855a](http://build.palaso.org/viewLog.html?buildId=64513&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 679, ignored: 6 Build time: 00:06:26


Looks good. Please forward to Brian


See if scenario is working


Hmm.... never mind. I'm getting the same error.


I just realized my pull request didn't have the list of commits.


Printデバッグで良いので，どこまで処理が進んでいて，どこから期待する結果と異なっているのかを基に切り分けを進めていくのかなあと思います．

ちなみに，以下の部分を削除すると動作するようになる例をいくつかWebで見かけました（Leonardもっていないので試せないのですが…）．

```
+  while (!Serial) {
+    ; // シリアルポートが開くのを待つ。
+  }
```

ちなみに，接続できないというのは，以下の状況という認識で良いですか？
・Advertiseはしている
・iOS側からConnectしようとすると接続できない（なんらかのエラーコードがでる）


If the purpose is just to create a file if non-existing, maybe

sudo sh -c "touch ${PID_LOCK_FILE}"

But this is purely a matter of taste!


Would it be better on this to add the default if the prop is null? That way, if the user misses that prop, it doesn't break?

```javascript
width: this.props.trackStyle.width || 38
```


this caused javascript to quietly fail in confusing ways. >___<


ah, damn, that was so obvious.  my apologies; hadn't had my :coffee: yet.


Hi


Thanks for doing this, I was in a bit of a rush earlier. Do I just need to tag again or do I need a whole new PR?


ok


I've reviewed the changes and they look good.  +1

Thanks,
Chris


these descriptions are clearly wrong (HTML5?)


Oh! Of course. Let me fix that up. I'll make sure all the files are in the inner StockLS directory. My apologies.

You seriously deserve so many cookies for working me through this. If you want, I'm being serious, I will bake you cookies. Probably I'll burn them, and then buy you real cookies, but I really appreciate this.


Thanks! Missed that


Nope this doesn't work :)


More than 2 years later and after hours of search, I discover the solution to my problem. Thanks.


appcache用の設定を追加。


@kensington @waltercool


a isto se chama try hardar. isso e teres 3 commits com o nome "final". se o teu código fosse supremo, não havia necessidade de testes e "bug fixes".


Commit message:
````diff
-  [FEATURE] Add openPOWERLINK PCIe interface support on Linux
+  [FEATURE] Add PCIe interface support on Linux

 - Add stack generic definitions for Linux PCIe interface.
 - Add support for Linux PCIe in dualprocshm library.
 - Add Linux PCIe interface driver for accessing PCP over
-   dualprocshm interface. The PCIe interface module consists
+   dualprocshm interface. The PCIe interface module consists of
   a driver for the PCIe interface and a module to communicate
   to the PCP using dualprocshm.
````


typo – it should be INTELMQ_PYTHON (see H letter)


Is there a git doctor in the house?

eol_scons is pulled into aspen and aeros as a submodule, which means the submodule stays frozen at the original hash. Thus if you use the 'scons --site-dir eol_scons' build method, these useful changes won't be available.

So, does anyone have a simple recipe for updating a submodule link? I've googled, and everything I've seen is really complicated. There must be a straightforward method, don't you think?


I'm Sexy Lolololol


LOL


please check your copyright, I guess there is always a "," between the year and "Kalycito"


@mdesanti Rubocop is detecting this method as an offence because I add that line. I don't think it should be refactored. What do you think? The only idea that came to my mind was to delegate the flash and redirect to another method.


Right, I guess that should be just "Enabled audio support".


c'est pas vraiment comme ça qu'on conçoit une fonction de ce type. Le paramètre ne peut pas changer de type, ça reste une expression à évaluer. Ensuite la façon que tu as de récupérer cette expression peut en effet varier.
Ex :
cfCall ( getFunctionExpressionByFunctionIndex( MyFunctionId ) ; Parameters )
ou
cfCall ( getFunctionExpressionByFunctionName ( MyFunctionName ) ; Parameters )


hola mundo


Added super call.  Don't look too closely at every other test case in the repo...


True, I understand. Sorry about that, it's just much easier and more consistent for me to manage through GitHub releases. It'd be nice to solve this properly, but right now I'd have to manage any contributions to add CHANGELOG entries, etc. It was purely a maintenance change.


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 118](http://teamcity/viewLog.html?buildId=1457&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 4 Build time: 00:00:56


appフォルダに、sample.appcacheを追加。


Actually i don't think we need that class (i originally created it, but this is not used). The only reason why we may need it is for the parser-generate to generate the ruby class automatically.

Also, i should change the name of my method `getBundle` ;-)


Thanks guys.

On Fri, Oct 16, 2015 at 12:50 PM, Frederik Bosch <notifications@github.com>
wrote:

> @nikic <https://github.com/nikic> Wonderful, thanks a lot!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/php/php-src/commit/8d12ff75132644006b394ab02c37944c48d6a920#commitcomment-13820236>
> .
>

--
Regards,
Felipe Pena


acté ok Collaborator :)


another question / possible issue: it looks like you're not taking the extra step of making your date strings lexicographic-comparable.  am I missing some python fu here?

I'd expect to see the parsed date values (everywhere) converted somewhat like `'%04d.%02d.%02d' % (y, m, d)` before ever trying to compare them with standard `str` `__cmp__`.  I definitely had to do this last year when I was writing a character history rewriter for SWMH or else Badness. :fearful:


Also, I'm not sure if you saw this on the chat. But I'll post this here since you are looking at this now. I'm getting a crash with the latest set of commits. I posted a logcat:

https://drive.google.com/file/d/0B57Ioy26LWegaTZWZGp3REtRTE0/view?usp=docslist_api


Where does the `__PCIE__` come from? Is it also used in case of Windows?

The platforms above all include the same header but have different macros!??!


Also here missing "," ?


@angelama2126 looks great, could give an example of a product/ company which would launch an app first?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.74.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64506&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:35:16


This and the next line are wrong!
1. In add_message_type (above) the component ID and message type are used in host byte order. Therefore, no conversion is allowed here.
2. Since this is Python, the input message is really one of the rockin_msgs message and has to have the COMP_ID and MSG_TYPE fields set (just like in the add_message_type). Therefore, just access them from the message instead of passing them as arguments to the function.


Closes #4


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.495.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64511&buildTypeId=bt323) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 861, ignored: 54 Build time: 00:26:46
### Failed tests
```
Chorus.Tests.dll: Chorus.Tests.ChorusSystemTests.CanShowNotesBar: <no details avaliable>

```


We are so cutting edge ;-)


Any reason to include this? =)


Ayé trouvé :)


Hi again @ShawnHymel

I investigate more today and I found a conclusion.
It's not possible to detect the z position of a gesture.
Why ? Simply because the z position take too long time to get computed.
Also, the more you are close to the sensor, the more the calculation need time to be correct.

This simple code demonstrate it.
It display the `z` and the milliseconds elapsed since the last `z` got

```arduino
unsigned long mil = 0;
unsigned long previous;
unsigned long diff;
uint8_t z = 0;

void loop() {

  if ( zx_sensor.positionAvailable() ) {
    z = zx_sensor.readZ();
    if ( z != ZX_ERROR ) {
      previous = mil;
      mil = millis();
      Serial.print(z, DEC);
      Serial.print(" ");
      diff = mil - previous;
      if (diff > 100) {
        Serial.println("----");
      } else {
        Serial.print(diff);
        Serial.println("ms");
      }
    }
  }
}
```

If I hold a vertical ruler and I quickly stick my hand to the ruler (at 10 cm) I gave this kind of result

```
215 ----
170 24ms
130 25ms
99 25ms
77 25ms
59 24ms
47 25ms
37 24ms
29 26ms
24 24ms
20 24ms
17 26ms
15 24ms
13 25ms
12 25ms
11 25ms
10 25ms
10 24ms
9 25ms
9 24ms
9 25ms
```

The real `z` position is between `12`and `9`
But the sensor gave me lots of wrong values from `215` to `12`
I can understand the time to adjust.
But here, it take more than ~300ms to have the correct z position.

So it is absolutely incompatible to compute the z position of a gesture.
To confirm it, just make a gesture above the sensor with this sketch running, a natural gesture can take 100ms or 200ms

Thanks


The parser already does that. I pull them out of the tree as `ck2parser.Date` objects, whose `val` attribute is a `datetime.date`, which compares correctly with other dates


Just keep on making mods that the community can benefit from/enjoy. =)


Hah. Nice function name.


@nickcolev, does this fix and test *all* the issues with the code?


Ah - ok, so *that* is the problem. Small detail, I think I can fix that.


Okay, you commented out stuff that could be added later, right? Maybe add some comment here, so that we know it later..


Can you also add documentation for the component_id and message_type arguments?


Yes but in this case it's actually code.


Ah, here it is the `__PCIE__`! Maybe add info that we are talking about Linux and PCIe if it helps?


@Stefano: If there is no difference between yellow and red, then why don't you try a combo with red ones? There is no need to search if it's hard to search for them just pick randomly. Since there is no difference, it wouldn't make any difference.

There is already a proof that tempo=20 is NO elo gain at LTC! http://tests.stockfishchess.org/tests/view/56158d6e0ebc597e4f23e694
All I'm saying is, this patch could have been more elo gainer without that part. An overall elo gain doesn't prove all ideas in it is elo gainer. I'm not trying to hinder SF's development BTW, only try to help.


The ExecutorService can only run one runnable at the same time. If we use it, the SeekCompletionListener will only get executed after seekTo()
We would be unable to check in (the synchronous) seekTo() if the completion listener has been called


Ah ok. Makes sense.


not only, it enables a bunch of plugins as well: audiodecoder, camerabin, mediacapture, mediaplayer. Don't ask me what they do :P


Ok!


@RandomAltThingy nobody said this project is a copy of pocketmine. BTW, I ve seen a funny show today. A 12-year-old kid banned by MiNet, make himself a laughter in another project just because he think others should not be better than him. It is pertty interesting isnt it?


Here is a "," - good :)


Why "+ 1"?


Travis fails on python 2.x because `attribute '__doc__' of 'instancemethod' objects is not writable`

(my first wrong guess, it's not really an instance method. AFAIR, we do string manipulation for `__doc__` inside the class definition, i.e. indented.)

Travis fails only on line 399, the other ones are ok. I guess this is because `nominal_association` has the `@cache_readonly` decorator.
We never before tried setting `__doc__` for cached attributes, this way, and only used it for methods so far.


Actually I just see smthg, multiple pads config were not handled well before either seems(well only for writing they were). Look it KeyStatus.  Every invocations were only checking if it was enabled in first bits, so for first pad. For force feedback it seems it was handled too yep. conf->options & PADOPTION_REVERSELY is a good example. Should I fix then? Or will it make boom?


hey you said incline to preserve old behavior in https://github.com/yyx990803/vue/issues/640

then you changed it to true?

why is that?


You should write down meaningful GIT commit messages.


Вот ты зря это сделал, из-за этого баг образовался.


Stack inter**f**ace ....


See comment about serialize function above.


Why didn't you prepare generic timing utility routines shared by all programs?


「sample.appcache」をcopy設定に追加。


Use UINT8* instead of char*!


Can you simplify the logic of the following?


Note that you could do something like the following:

```javascript
if (attachedData && attachedData.instance_id) {
   // ...
}
```

If ``attachData`` doesn't have a property ``instance_id``, it will be ``undefined`` - not ``null``. You can see this in a quick console/REPL session:

```javascript
> var attachedData = new Object();
undefined
> attachedData.instance_id
undefined
> if (attachedData.instance_id !== null) console.log("hmm");
 hmm
```

Doing a conditional like I'm suggested would leverage short-circuit boolean evaluation too, so if ``attachedData`` was either `null` or `undefined`, the JS engine won't bother trying to the right hand side of the conditional.


<!DOCTYPE html>
<html lang="zh-CN">
  <head>
    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <!-- 上述3个meta标签*必须*放在最前面，任何其他内容都*必须*跟随其后！ -->
    <title> 流程审核审批操作明细 1314080901129</title>
    <link rel="shortcut icon" href="icons/favicon.ico" />

    <!-- Bootstrap -->


The "bug" is still there, when I make use of the "dynfilter"-option. Then as before table widths are not respected.


There should be no need to instantiate a message here (and set some value, see next line).


Is the cast to UINT64 really required here?


Also here, cast needed?


Bug should be fixed with the last [commit](https://github.com/rdf-ext/rdf-serializer-n3/commit/e75229e39296c69f05f464c3c6d27e240bec7ae8).


This may not be accurate, it doesn't look like we got rid of the auto save functionality on exit.


Should this be `claret-1`


$(use_enable ounit) is probably the one that works


I think E-mail is for those still struggling to get used from  Mail to Email


You have two options:

cd eol_scons
git checkout master
git pull origin master
cd ..
git add eol_scons
git commit -m'update to latest eol_scons'

The second option is to merge the Aspen-dtd-in-memory pull request I
submitted, since that already has the update to the eol_scons submodule.

In aeros I renamed eol_scons to site_scons, so the --site-dir option is
not needed.  Would you like me to do the same for aspen?


callback functions must have "pfn" prefix


Ok


This also removed the sending of whiten/symmetrize/ etc parameters


Replace icons with new ones but do not remove custom icons.
It's really useful to immediatly distiguish between accounts based on image and we cannot remove this, on the contrary we should improve this, allow users to import their ICO\PNG saved as blob in DB.

Please revert\change it as soon as possible before it will be replaced by other changes, and for such big changes we should use pull requests in order to discuss about changes.


I would not make the data to be read "optional" i.e. providing a default. You always know exactly, how much data needs to be read. Therefore, the default value *only* introduces a source for an error.


ReadVarString


WriteVarString


You are correct, I was a bit quick on that one, please remove the commit
The initial way to do is unusual but it works.
Sorry for this !


Ok, it is published now to **unstable**.


Hopefully this fixes the issue.


Return not documented


Leftover print statement.


I'm not sure if I agree that >= 5 can be considered "pleasant".


Thank you for looking into this! As far as I can tell, this has something to do with the firmware within the ZX sensor itself, which I don't have control over (it was written by [XYZ Interactive](http://www.gesturesense.com/xyz/)). I will forward your findings to them to see if they have any suggestions.


Could we just add a default value for the status filter?


buildタスクをもとに、deployタスクを作成。
appcacheとgh-pagesを追加。


Хотелось бы каких-нибудь комментариев, тестовых запусков и т.д.


- Why not?
- Remove commented #define


While npm does get installed with node, I don't think it's resolvable by require.  Here's what I tried:

    // in a directory with no node modules
    $ node
    > require('npm')
    ! Boom

> On Oct 16, 2015, at 8:04 AM, Jakub Korál <notifications@github.com> wrote:
>
> npm package is always resolvable, because it is installed alongside node. isn't it?
> because right now when I have clingwrap inside my dependencies, it also installs npm@3.x, which is unfortunate.
>
> —
> Reply to this email directly or view it on GitHub.
>


probando la sincronizacion de imagenes y otros codigos por primera ves


Would it be a good idea to add some sanity checks somewhere to ensure that we don't turn on `REGISTRY_TOOLSETS` while disabling the registry entirely?


Consider switching to POST since it deletes failed layers.


You're right I picked up this commit from on old branch which obviously had autosave disabled.


nevermind, my bad. your solution is the correct one.


Consider putting the dismiss URL on the layer attributes (Ref: https://github.com/azavea/raster-foundry/blob/develop/src/rf/apps/core/models.py#L169).


Even on mono?


This seems to provide a reasonably simple answer:

http://stackoverflow.com/questions/5828324/update-git-submodule-to-latest-commit-on-origin

Or to cut to the chase:

    git submodule update --remote --merge

will pull in the latest stuff for all submodules. Then commit to solidify
the result in your project.

On Fri, Oct 16, 2015 at 10:17 AM, Gary Granger <notifications@github.com>
wrote:

> You have two options:
>
> cd eol_scons
> git checkout master
> git pull origin master
> cd ..
> git add eol_scons
> git commit -m'update to latest eol_scons'
>
> The second option is to merge the Aspen-dtd-in-memory pull request I
> submitted, since that already has the update to the eol_scons submodule.
>
> In aeros I renamed eol_scons to site_scons, so the --site-dir option is
> not needed. Would you like me to do the same for aspen?
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ncareol/eol_scons/commit/61975861f6c277bd39ad674426a11b2e5313436b#commitcomment-13820899>
> .
>


Does this improve the Network Connection or what exactly?


Consider deleting until needed.


Hi Heshy (@hroskes),
I think, it is reasonable to change the default value to 'True'. However, this makes the whole option useless because its value is always 'True'. Either you don't use this flag and it defaults to 'True' or you use it  resulting in 'True' for this value ('action="store_true"').
Therefore, I would rather delete this option in the long term.

Cheers,
Gregor


Nothing to preview


If `dismiss` returns a promise this could be written as `model.dismiss().then(removeItem...)` which would only remove the model if it was successfully "dismissed".


this is assasins creed rouge


no need to use PauseableHandler here.

```
m.raftHandler = etcdhttp.NewPeerHandler(m.s.Cluster(), m.s.RaftHandler())
```


@keyboardsurfer [SigInActivity L41 is not using ActivityCompat for additional options like here](https://github.com/googlesamples/android-topeka/blob/master/app/src/main/java/com/google/samples/apps/topeka/activity/SignInActivity.java#L41)


An underline probably would've looked nicer, as well as fallen in line with the visual hierarchy established by the already bold headings (`==> Updated formulae`).

SHIT, bikeshed's already painted. :( I'm too late...


Consider renaming `isWorking` to `isProcessed` or `isDoneWorking`? It would make the boolean logic & conditions a little easier to parse.


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (841 of 842 tests passed)

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


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


we could directly fatal here.


Consider leaving this empty until we add the ability to cancel processing.


I'm not sure if this has the effect that you expect?!

EWOULDBLOCK and EAGAIN mean that there currently is no data available on the socket. For instance, assume you want to read 8 bytes. During the first iteration over your while loop, you read 4 bytes, i.e. less than what you want to read. So you do another iteration. Assume you try reading again, but there is no data available yet. Then recv throws an EWOULDBLOCK/EAGAIN. As a result you return None and have lost what is in the received_data buffer.


`D3Chart`?

Oh wait, just read your comment -- choropleth is going to be part of this? That seems strange. I'll check out your choropleth commits as soon as I can get this Punchcard in.

We'll have to sync up on style then too, since there are some differences between the code you and I have written and the whole thing should feel cohesive.


@rcrowder who are you asking this question? It is best to call someone out with `@` explicitly.


should this logic not be included on other views?


antipattern is hey hey hey


The extra argument is only necessary if you implement the other side too :)

https://github.com/JACoders/OpenJK/blob/master/codemp/server/server.h#L198-L218

And all challenge related changes in this file: https://github.com/JACoders/OpenJK/blob/master/codemp/server/sv_client.cpp