From what I can tell, DevDiv has embraced non-MS technology much better than the rest of the company. They've got great support for Python and Node.js, at least. They released their editor for Linux and Mac for free. They support popular test frameworks and GitHub. It's pretty amazing.

> On Oct 27, 2015, at 7:47 AM, Daniel Harms <notifications@github.com> wrote:
>
> I'm surprised that*
>
> (commented at 4:00 am for some reason)
>
> —
> Reply to this email directly or view it on GitHub.
>


When we include these files, we do it in the form:

   #include  <hwref/gm20b/nv_fb_hwref.h>

"hwref" appears twice. The "nv" prefix also sounds superfluous. Could we maybe rename them to

    #include  <hwref/gm20b/fb.h>

?


nv_drf.h could probably be renamed drf.h


Unneeded space after "g,"? (also on other macros in this file)


Really tired right now and plan on continuing this and catching up with your changes tomorrow morning.


Not a big deal since the preprocessor will optimize anyway, but do we need these unused arguments for these macros?


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


:+1: cool. han mal nagluegt wieso jshint kein fehler azeigt bi dem. das isch ageblich erlaubt ide neuste EcmaScript version 5 womer bruched für de jshint validator. naja, isch glich schöner so :wink:


Peut-être y placer l'URL de l'instance quelque part ? Cela permettra de bien comprendre de la part de quel instance on reçoit la notification.


Why are we screwed?


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


This doesn't work on windows.


hesch di alte styles ned welle userüehre?


Main file


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


Love that we don't even host this


I just copied the approach from `preface.xsl` [Line 41](https://github.com/dita-ot/dita-ot/blob/develop/src/main/plugins/org.dita.pdf2/xsl/fo/preface.xsl#L41), but it makes sense to implement this consistently for all  frontmatter sequences.


How does JavaScrupt's `indexOf` handle utf-8 compared to Go's `strings.Index`?


I think here outputType.isAssignableFrom(inputType) must be the opposite: inputType.isAssignableFrom(outputType)
For example:
if outputType = Number, inputType = Long then (outputType.isAssignableFrom(inputType) == true, but not any number can be applied (converted) to the module which accepts Long value on its input.
The opposite is true - to the Number input you can apply Long value.


after the change from you,does it work?


This change broke my configurations, all scripts was renamed to the alias instead of the key name.

I would like a solution where entity_id is always the key name for the script.
And that the alias configuration directive just sets the customized name so that it works like before.

Anyway because this change breaks previous configurations it should be mentioned in the release notes of 0.7.6.

I had to spend a lot of time investigating why everything just stopped working after the upgrade, the logs is also less than helpful when calling script.turn_on for an non existing id.


sed is an command for linux,


shhh :wink:


Peut-être ajouter une champ signupFrom qui aurait comme valeur un éventuel paramètre passé en GET dans l'URL au moment de l'arrivée sur le site.
Je pense à ceci dans l'optique de pouvoir tracer le nombre d'inscriptions issues des installations windows.


Sorry for correction, the default weight is 10.


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1902 of 1912 tests passed)

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


It seems to me it checks for an overlap between comment_lines and intervening_lines. You can just get the line of the dot and the line of the selector and subtract them to see if there's something between them.


I'm thinking we ought to rename the `classDecorator` opts if this one is `withDecorator`? :)


sind die ", x"-Werte nicht eigentlich directions? evtl sollte man da konstanten einführen


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - i386-Windows-7-VisualStudio-12: Tests Failed

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


The `yeoman-app` uses `findup-sync` as well by the way (https://github.com/yeoman/yeoman-app/blob/master/src/browser/yo/yo.js#L5). Used that one as a reference but used your more awesome `find-up` module now :).


Also closes #35 and #37 .


Was meinst du mit Vollständigem Beispiel @JohannesHoppe ?


This cleanup is necessary for both minio-go and minio-go-legacy as well, similar checks are done even in that code.


Koshel is here!


it all looks good, except for my initial comment, thanks @brainstorm ! :-)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1024.e392c276c5663def4323a695e00b75d6cdd2c45b](http://build.palaso.org/viewLog.html?buildId=64924&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 694, ignored: 6 Build time: 00:10:16


The issue I had with these macros is that they sometimes result in very long lines when the registers have long names. E,g:

    nvkm_wr32(device, 0x419e4c, 0x5);

Turns into

    nvkm_wr32(device, drf_reg_offset(PGRAPH, PRI_GPCS_TPCS_SM_HWW_GLOBAL_ESR_REPORT_MASK),
              drf_fld_val_placed(PGRAPH, PRI_GPCS_TPCS_SM_HWW_GLOBAL_ESR_REPORT_MASK, SM_TO_SM_FAULT, REPORT) |
              drf_fld_val_placed(PGRAPH, PRI_GPCS_TPCS_SM_HWW_GLOBAL_ESR_REPORT_MASK, MULTIPLE_WARP_ERRORS, REPORT));

This makes it hard to respect the 80-columns guideline of the Linux kernel.

The long registers names cannot be helped, and I believe they can become an exception to the rule. However since we don't have that many macros and we are likely to use them very often, it would be nice to shorten them a bit.

How about prefixing with exactly the part of "drfv" that the macro applies to? For instance, since drf_reg_offset() does not touch any field, it could be renamed as dr_reg_offset(). Going further, the "reg" part is already carried by the "r" in dr. Thus dr_offset() translates nicely into "offset of that register for that device" and is shorter.

Similarly, the "fld" in drf_fld_val() is redundant -> drf_val(). The _placed suffix is also long to type -> drv_val_p()?

The idea is to have the operand the macro is acting upon (the relevant part of "drfv") as a prefix, followed by the operation performed on it, in a short manner.


nv_rd32() has become nvkm_rd32() on the recent codebase.

We will probably want a drf_wr32() (or maybe dr_wr32(), see other comment) as well?


Yes, will clean that up as well.


Do you like the old SIGNAL/SLOT notation more than the Qt5 one?


I like how clear this explanation is! :+1:


We will probably want to use the final macros names here.

That's about my only remark - this file is super clear and helpful.


This one, that is https://github.com/omniscientjs/omniscient/commit/88fbafedf2fc2db6cf4026e5987734a2b5affeda#diff-2422dd4abef863a7859a280e0c9a15fdR116


Meinst du alle Dateien?

- app.ts
- car.component.ts
- dashboard.component.ts
- car.model.ts
- index.html


i have added my java files to testrepo


Didn' t test, just reviewing the code... Is the `linkHovered` emited also on "unhover"?(there is nothing about that in the doc)


i know it. windows doesn't have sed. so it should use `git rev-parse --abbrev-ref HEAD`


I agree. This shouldn't have happened. I'm about to go to bed but I'm down to put out a new release to fix this regression.

To fix this we have to change [this line](https://github.com/balloob/home-assistant/blob/dev/homeassistant/helpers/entity_component.py#L68-L69) to only be done if no entity_id exists and set an entity id for the script as usual.


In that case, "intend" should be "suggest", surely?


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


OK. That means even a blank line with no comment will prevent the cop from moving the `.`. Is that OK?


Huh.. I don't remember anybody bumping them. If somebody did, the docstring info was not updated (https://github.com/obspy/obspy/blob/master/obspy/core/util/testing.py#L243), and in `imaging` we have some 2 year old images. #929 was the last major work on image comparison I think and there's no mention of a bump.


Thanks for the fix! I ran into this problem too then `git pull origin master` fixed the problem :)


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1301 of 1301 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.13%25-green.svg)


:+1: :smiley:


Credits go to @elafarge and @HerrmannM (#36, #260, #261). :)


es


We should make use of this within templates used for sending emails, e.g. for activation link and shift modifications (whether deletion or time change), as well! :)


wow respekt du hast ja richtig viel geschafft bin jetzt alles durchgegangen und mir ist nichts aufgefallen =)


Thanks! There's more to come (in other PRs).


Oh, you're right, they weren't bumped, but the tolerances were definitely updated just before that PR.


The markup rendered by the plugin would need a serious update IMHO :
```
<p>
<p><div class="noteclassic">
This is my note ! Remember it!!
</p>

<p>
Blah…
</p>

<p>
Blah…

</div></p>
</p>
```
Two nested <p> with a nested <div> ?!
Where's first paragraph opening tag?
Where's last paragraph closing tag?

I don't think it has any influence on the way Mixture handles the plugin but still...


You've been explicitly asked by @gentoo/qa not to do that anymore.


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


ага)


greate i am just waitting for it


wirte some Ruby program


window is only used once, no need to make it a variable here just to create an alias for Date.now()


removed.


bucket is only used once, so no need to create a variable just to alias all(window, false).


correct, this was just a holdover.  removed.


@jsquyres already #include'd ```stdio.h``` in a previous commit


«В коде выделить метод calculate(), в котором будут проводиться всё вычисления, который на вход принимает строчку с выражением, а возвращает посчитанное число.»

В данном случае не реализован параметр метода. Через параметр в метод должна передаваться строчка, содержащая арифметическое выражение, которое в теле метода должно быть вычисленно.

Этот параметр можно назвать expression.


we also need to update the karaf feature.

given that okhttp and okio doesn't seem to be OSGi bundles, we may need to keep the uberjars around so that we can use them in the feature.

other than that, it seems good to me.


this was re-written.


now we just create an S3OutputStream and return it.


noted.


it is implemented now.


healthy is mutated is several places, no need to make this a mutable variable, should be a constant.


Для класса лучше подойдёт имя Calculator


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


this could be rewritten with a reduce instead of an each, and const variables instead of mutable variables being updated inside of the each() block.


It is dependent on libiwinfo-lua.
```
 Symbol: PACKAGE_libiwinfo-lua [=y]                                                                                │
  │ Type  : tristate                                                                                │
  │ Prompt: libiwinfo-lua...................................... libiwinfo Lua binding                                                                                │
  │   Location:                                                                                │
  │     -> Languages                                                                                │
  │ (4)   -> Lua                                                                                │
  │   Defined at tmp/.config-package.in:14558                                                                                │
  │   Selects: PACKAGE_librt [=y] && PACKAGE_liblua [=y] && PACKAGE_libssp [=n] && PACKAGE_libpthread [=y] && PACKAGE_libc [=y] && PACKAGE_libiwinfo [=y]                           │
  │   Selected by: PACKAGE_zabbix-extra-wifi [=n] || PACKAGE_luci [=n]
```


variable culprit is not needed, just an alias for the find() call above.


implemented.


typo, /to/on/


2. Tere maailm button


excludeMemberFromShare


typos in this message, not sure what it says


is there any reason for keeping those parameters here?


You can use ``git clone --recursive`` and you can get rid of the "git submodule init" step.


not sure why we need this additional healthcheck here, this could just be done in the healthcheck middleware, outside of the perf module


В данном случае неправильный тип переменной и, соответственно, неправильная инициализация.

Input - это строка, содержащая арифметическое выражение, а не число, являющееся результатом вычисления этого выражения.

В данном случае 2 + 2 вычисляются ещё на этапе компиляции, а результат записывается в input. И смысла в вызове метода calculate() уже нет.


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


Adding something


收下了~~


We'll see, Travis should now upload on Py3 to Imgur, there was a fail in this PR: https://travis-ci.org/obspy/obspy/builds/87834106
If we end up with fails on 1.4 we might as well bump them (like I said there is no mpl 1.3.1 for Py35 through anaconda).


组件内的元素的class命名不正确，参考这里的文档：https://github.com/idle-dog/upup/blob/master/docs/DEV.md#组件化拆分与css选择器命名

这里应该命名为 ``.w-zoom_btn`` 以下同


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1025.e392c276c5663def4323a695e00b75d6cdd2c45b](http://build.palaso.org/viewLog.html?buildId=64935&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 694, ignored: 6 Build time: 00:07:58


Good idea! Feel free to try :-)


@notspiff @wsnipex viz.goom and viz.projectm both try to find glew too but don't even have it in their depends. Not sure if the solution would be the same there?


> There's a problem with CDVDInputStreamPVRManager::CloseAndOpen

that has ever been a hack and even segfaults. just make it compile. I will think about a solution for it.


... if there is a need for an additional health check, why limit it to a sync function as opposed to a callback based function? ... probably better to keep it out of this module anyway and let the caller of perf.isHealthy() call whatever other health checks it wants to call in addition to this one.


Goom likely can get the glew axe but projectm used it natively.
28. okt. 2015 08:32 skrev "Memphiz" <notifications@github.com>:

> @notspiff <https://github.com/notspiff> @wsnipex
> <https://github.com/wsnipex> viz.goom and viz.projectm both try to find
> glew too but don't even have it in their depends. Not sure if the solution
> would be the same there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/repo-binary-addons/commit/1a4712df0623d9443f2017c7b68658eef6daca40#commitcomment-14031520>
> .
>


Вообще, мы пока можем  обойтись без вывода на консоль — всё значения можно смотреть в отладке. Но если выводить результат, то хорошо бы было выводить и выражение, для которого он посчитан. Выражение брать из переменной input.


onChange的逻辑实现的并不好，最好改成两个函数，zoomUp和zoomDown。
此外，滑块位置的设置，可以利用百分比算出位置，不需要独立的count函数来更新


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


boh


Is this tested with all available CALs/platforms, I mean wouldn't it be better to ignore the call by returning kErrorOk?


Also here, maybe kErrorOk?


Fine to merge.


Yes sir ..


Yep. We still have to change our code (crawler, views). Though this seems to be small potatoes.


"CString" is Go's C-string-type. A cast to "CString" transforms a Go string into a C string.


This started causing build warnings with MSVC, "set_mb_syn_cabac.h(120) : warning C4146: unary minus operator applied to unsigned type, result still unsigned".

What would be the best way of silencing that warning here?


What about server ip?


@SubJunk As you know this came a bit suddenly for me :wink: I just remembered that all the translations from crowdin aren't pulled either, so the improvements made from the previous release aren't in this build. In the future, pulling those need to be part of the release procedure - and it will be quite easy when the crowdin pluin is in place (as it stands now, ```mvn ums-crowdin:pull``` while being checked out at master, and then commit the changes).


Added it in the SaifeUpdater class.  It seemed fitting.


I commonly use "IS_PY3" here, but that's a matter of taste.


very good mate.


It adds an empty line into the code by adding a function call "REMOVE()" which is later removed from the source code string.

I do not yet know how to do that using normal AST operations.


your code is really amazing!
thx for sharing.
There is still a little question I wanna solve it, how could I catch any native crash all over my code without adding COFFEE_TRY_JNI()?


Why remove the trimming at the end now?


"type" is a Go keyword


And here?


ManyToOne (x => x.EmailAddress,x=>{...})
specify MANY-TO-ONE association exists
between MANY instances of VerificationKey
to ONE instance of EmailAddress.

x.Unique(true) - means that the FOREIGNKEY FROM
EmailAddress TABLE TO VerificationKey TABLE WILL BE UNIQUE.
FOREIGN KEY POSTED FROM EmailAddress table ONLY ONCE
IN VerificationKey TABLE WITHOUT REPEATS
thereby satisfying ONE-TO-ONE ASSOCIATION.


I may be wrong but I remember an example in documentation that tells something like: Manager.add(obj) works only if the obj has been saved in the database.


`Framework Ruby on Rails` ->
`Ruby on Rails Framework`


No, I have not yet tested this on all platforms.

Yes, returning `kErrorOk` is fine as in most cases the return value is not checked and the errors are handled by `eventu_postError` handling below.


sds


Shouldn't these cases also consider a None encoding?


Thanks for the hint. I was copying the stuff over from another platform.


Broker on clientside could/should be static. When is one client going to be in 2 games at once? Same goes for map.


This breaks the redirect of followers


From this commit onwards, we will keep the compression and decompression numbers consistent to avoid confusion.


foreach with a break statement if the comparison matches vs. a for loop?


I don't see the debug statement that was supposedly removed here


Okay, if in most cases the return value is ignored that we can return an error. Can you please check (at least by reading the code) if there might be any issue with any of the CALs? Maybe it makes sense to introduce an error code that identifies "shutting down" or so!?


ah doch, hab einmal ausversehen phisisch geschrieben.
Fix ich nachher. ^^

Mit dem OTC System das imcGames implementiert hat, geht das etwas schneller als normal.


would be a mistake of mine sorry ;)

i found them in some source not sure where it was
since my internet was shit i cant fetch all source and done downloading each part and added
else i would be doing cherry pick itself

even in thread i wrote may be i missed some one to list ;)
hope you got me

from these modules only 2 i used and later updated and fixed all with my work mate ;)


This is ok


why should map know the port.


it checks if "fname" starts with "is" or "has" follow by an upper-case character e.g. "isValid" would match "hash" does not.


Root cause behind this change?


Sorry, My fault...


I agree with you on the principle, *but* the talk is filled with a lot of litteral include from the code, so these changes will break the talk.

And we loose locality for the documentation.

And this is an out-of-tree build, so we're free ;-)


Trigger event 'redirect' in the Followers widget?


Are we ok with adding a dependency on Linq here? I wasn't sure how well supported that was across the .net flavors at this point. Do you know what the implications of using linq could be? Are we using it elsewhere?


I will test the commit on other platforms and verify if this introduces any issues. thanks


厉害


Tak je to otázka jak by se to mohlo nasadit. Nejjednodušší by imho bylo, kdyby jsme současně změnili i encrypt, takže by po aktualizaci syrupu začala komponenta šifrovat nově, pak by se ta stará šifra používala už jen chvíli a v další verzi by se to dalo vyhodit úplně. Ale nevím, jestli by to takhle šlo. Stejně to zkusím s tím forcnutím wrapperu v decryptoru, to by molo vypadat líp


Add function on_channel_redirect which opens the channel in the client action


The doc says to use the `-stats` flag. Both work?


Better to use generic check condition for distro version 7.0+, RHEL 7.2 will be released soon


I know it hardly hurts but probably don't want these to be public? :smile:


Unfortunately, can't do this because this breaks the litteral include


Am I wrong in assuming `create()` saves it in the DB?  Compare with https://github.com/ManoSeimas/manoseimas.lt/blob/suggestion-source/manoseimas/scrapy/pipelines.py#L173-L177: there's no `party.save()` in that code.


Paradicka, Kopik!  :)


```
x.network.value = "";
```
This leads to my wifi does not obtain an IP address (DHCP).


just to point out it's not a regular C++ ``dynamic_cast``


We can use `interface Loadable <T extends IAbstractElement>`.


My colleague and I have discussed and we suppose that AMS wants to send zero value as stream id. 1.7765824089018436E-307 and 4.801834657218423E-299 are too small values and they can be got because of the rounding error. AMF0 transmits only double values and these strange values can be passed as-is.
So we consider that createStream and initStream in StreamService should accept Number values, then convert it to integer and internal code should operate with integer stream ids.
Unfortunately I don't know what initStream method should do. Can you explain it, please?


ok :-)


```>>> a1 = Article(headline='Django lets you build Web apps easily')```
You can’t associate it with a Publication until it’s been saved:

```>>> a1.publications.add(p1)
Traceback (most recent call last):
...
ValueError: 'Article' instance needs to have a primary key value before a many-to-many relationship can be used.
```


I think your src_prepare needs dev-ml/oasis, please add it as build dep


`emake update`


Should be fixed now.


> Only requires: href="/#/active". It should be: href="/#/todomvc/active"

You are wrong. Just hover this examples to see resulted url:
`#/todomvc/active` vs `/#/todomvc/active`  vs `/todomvc/#/active`

`baseUrl` works as intended now. I can't see which issue brought by `onClick` hook for you.

No one view (react component) shouldn't use `url` directly for inapp navigation. It should be either signal call, or link with `href` obtained by `signal.getUrl(params)` method call (or `<Link />` component from `cerebral-react`). Links with destinations outside the `baseUrl` scope onthe same `origin` should allow to move out from app (do not prevent default).

PS. `redirect` is edge case and also should allow to move out outside the app.

My conclusion is current setup works as intended. Thought, we should have good documentation on how to construct inapp navigation links


my precioussss
There's a header you can include to get msvc compliance. And out of tree pass are worthless under windows anyway


I would again link there the pull request id as it is a different repo and only link to the issue in the pull request "notes".


dies ist vermutlich ein tippfehler und war nicht beabsichtigt? scheint mit den anderen Änderungen zwecks icons nix zu tun zu haben?


どこも参照してないです！
ハッシュと同じようにしたほうがいいのかと自分で考えました。以後気をつけます！


Thanks! Enum name makes sense, true. Looking forward to it
________________________________
Von: JonathanH-UK
Gesendet: 28.10.2015 00:54
An: JonathanH-UK/thrift
Cc: Jens Geyer
Betreff: Re: [thrift] THRIFT-3397 Implement TProcessorFactory in C# to enable per-client processors - Tests (024c204)

Just about the whole matter of strings... I completely take your point - particularly in the log delegate where I definitely should be using String.Format() for the incoming message.
With enums (e.g. ServerType in this case) I think it's more normal to call ToString() to obtain the string name of the enum value as opposed to it's numeric value. Regardless - I'll make it more consistent :-)

---
Reply to this email directly or view it on GitHub:
https://github.com/JonathanH-UK/thrift/commit/024c20441fd948017b9df455211f053acf86809d#commitcomment-14026966


I fixed it in [this commit](https://github.com/aklomp/base64/commit/965ee9dbac9cf3f87f9fe0077fec43cad6bb4784). If less than 4 bytes remain (or less than 8 for the `uint64_t` case), the encoder falls back on the bytewise encoder in `/lib/enc/head.c` and `/lib/enc/tail.c`.


ご丁寧にありがとうございます！！
一度自分で調べてみます！

ここ含め、入っていないものは全てのファイルに改行を入れます！


It's a shame that the upstream developers rejected your pull request :/


Am 28.10.2015 um 01:53 schrieb Ben Gardiner:
> @aholler @dominicgs
>
> I know this was a [WIP] commit but I'm trying to understand what's the next step here to get the new internal queues working with Injectors. Can you point me in the right direction please?

Either find someone who explains you how they've worked or find it out
yourself.

I assume my changes are relatively easy to understand, as I've just
introduced a(nother type of) queue.

I've seen some pictures / message flow graphs about them, but they
weren't in sync with what I've seen in the source. Therefore, and
because a lack of interest (use case) for the injector functionality I
haven't spend more time. Dominic said he will do it, but I assume he
suffers under the same problem(s) as I do. ;)

Regards,

Alexander Holler


Ball of Duty Server/Domain/BoDService.cs

Return null? And if its null on client side, we know the server didnt recognise id.


Veldig bra jobba, @kristianenge. Og ikke minst: bra spottet!


sync 命令的话

源，只需要考虑 walnut 内部文件就好吧。

我理解的是，你打算本地运行一个 walnut，然后把某个目录树同步到云端的一个 walnut 目录上

现在由于支持  mount 命令，可以把本地目录 mount 成一个 walnut 内部目录（但是它的文件是只读的）
同样，也可以支持更多的 mount 类型

sync 命令应该不用负责这么多种数据，它就面对 walnut 的内部目录就好了


virtual定义虚方法。
在子类中利用override进行方法的重构。


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


again, return null.


Etwas, wo man noch mal möglichst alle Konstrukte in der Anwendung sieht. Wie man das hinbekommt ist ne andere Frage. Viel kürzen (zb. CSS Klassen) und alles in einen einzigen Listingblock hinein. 👍😉

Von meinem Windows Phone gesendet

----- Ursprüngliche Nachricht -----
Von: "Gregor Woiwode" <notifications@github.com>
Gesendet: 28.10.2015 07:58
An: "Angular2Buch/code" <code@noreply.github.com>
Cc: "Johannes Hoppe" <johannes.hoppe@haushoppe-its.de>
Betreff: Re: [code] angular-templating: korrekturen (fa257ff)

Was meinst du mit Vollständigem Beispiel @JohannesHoppe ?
—
Reply to this email directly or view it on GitHub.


Hmm:  A few things
1)  github seems to be bugous: It does not show the `>` in the above new line 67 of `Makefile`.
2) It is *not* the above change which is causing your problems, but one or two changes before that.
3) You did not say it, but your problem must happen at the time of  `make install`, not at the time of `make`, right?
   IIndeed, `make install`  currently only works from *relased* versions of ESS.  as yes, of course, the file  .IS.RELEASE will not be available for non-released versions of ESS .. as the file name suggests.

This is something to be fixed, indeed (but *not* in the `Makefile` you cite above, but rather in `etc/Makefile` !).
Thank  you for reporting


If I remember correctly, this just wreak havoc to replace instruction while  iterating over them. I guess just using the iterator version of ``ReplaceInstWithInst`` will be a better catch.


@SubJunk Done.


`match_re`  is useless here


Toto nieje nejaký veľký problém, ale nette má metódu beforeRender ktorá slúži na takéto veci
https://doc.nette.org/cs/2.3/presenters#toc-beforerender


OMFG !?!? Tu commites des *tests* sur *github* ??? Soit tu as perdu un paris, soit tu t'es fait piraté ton compte !!!! :P

Ou c'est stupeflix qui t'a forcé à basculer du bon côté de la force ;)


Why should the client know how server handles game identification?


Since DateTimeFormatProvider is stored in DI, you can use constructor injection here instead of using StaticContainer.

FYI, I changed the docs here: https://github.com/piwik/developer-documentation/pull/67 to describe how DI should be used. It's not yet published since it is too targeted towards core developers. Since you're a core developer, you may find it useful.


Maybe the fetching of the clock is too expensive for a debug line that most won't look at. I suggest commenting out or putting in an ifdef the time-measurement code.


:clap:


please run your script on dev-ros and ros-meta category; it is correct to run it here

or fix repoman


Minifiying all the files


Hva er grunnen til at du slettet denne testen, @kristianenge? Den tester en annen public metode enn den over?


yet noone considered it important enough to submit a patch


Dont' we need to define at least PHP >= 5.5.9 everywhere, since L5.1 runs on >= 5.5.9?


We use `onlyHash` option when can't provide correct routing configuration on the server-side.
Given `todomvc` example on your site misconfigured for now. `http://www.christianalfoni.com/todomvc/active` is not valid `url` for server for now. You must either configure server to serve all urls under `http://www.christianalfoni.com/todomvc/` with todomvc app html or use `{ onlyHash: true,  baseUrl: '/todomvc/' }` config for router.


hm


Seems reasonable.


Since DateTimeFormatProvider will be stored in DI, there will only be one DateTimeFormatProvider in the container at a time. So this variable does not need to be static. Also, being static could cause problems in the future, since multiple containers are created, this value will be the same for both, even if one container targets an instance w/ a different setting value.


why?


lel @cachemoi


Why is it named preferedPort? In what case would it change?


@morpheyesh merge overview pages(microservices, service, app) and move javascript to single file


Na toto by sa mali používať signály, nie akcie.


Toto netuším na čo ti vlastne ;)


Hmm, also ich kann das machen, wird dann etwas länger der Text.
Sollte kein größerer Aufwand sein.

@d-koppenhagen & @fmalcher Ihr könnt gern auch über den Artikel lesen und Vorschläge bringen.
Ist aber kein Muss. Wir haben genügend ToDos.


We should probably move this conversation to the issue ticket so it's easier for others to read!


I've opened issue #15 to fix the current build failure - will also need an OS-specific frameReceiver config file to be passed to the integration script.


Toto by nemalo byť persistent


Typo. unnecessary


Debugovacie výstupy by sa nemali commitovať


Opäť, nie action ale len signal


…though what does "unnecessary" mean? Not that the old method name was any clearer :)


Metóda createComponent by mala vždy vrátiť nejaký component, a tu nemáš žiaden return.


Hasufell already did the work for you once. We are not your servants.


timothy.ng'ang'a


I know. This is a bit rushed someone-looking-over-my-shoulder-to-make-it-work-type-code.


Lost discussion https://github.com/padrino/padrino-framework/commit/da9b80f732a5951e6541cc46dd2267809f513d30 due to rebase


I agree that it works as it ideally should now, but how the user defines the baseUrl and the hyperlinks is confusing.

### BaseUrl `/todomvc` and `onlyHash: false`
Note that the baseUrl does not have a trailing slash. The hyperlinks has to be: `/todomvc/active`. This results in `/todomvc/active`.

### BaseUrl `/todomvc/` and `onlyHash: true`
Note that the baseUrl now has a trailing slash. The hyperlinks has to be: `#/active`. This results in `/todomvc/#/active`.

To me that is confusing. This is what I think is more friendly:

1. BaseUrl `/todomvc`
2. Hyperlinks with hash: "/#/active"
3. Hyperlinks without hash: "/active"

So I agree that the router works as it should now, but I vote for documenting `getUrl` and using signals, as you say @Guria. I think using plain urls is too inconsistent in regards of base hash/no hash... which is often the case with routing libs


Could you find API examples of real world of `isContains` and `isContaining`? What is found more often?
Read [Principle of least astonishment](https://en.wikipedia.org/wiki/Principle_of_least_astonishment).


Since there is now some indirection here, you could add `Regex.escape` to clearly indicate to the code reader that whatever the value is, it will be treated as a literal string. Then again, this value won't change, so it only helps the nervous reader (=people like me). I think of that as a good quality in a coder, but reasonable people disagree :D


In this case, since we want to abstract away chunk store's implementation details it makes sense to not use an `io::Error`. If we want to use, say, a memory map inside chunk store one day then returning `io::Error` would no longer make sense and the return type would have to be changed, breaking the API.

In general though, return types shouldn't be any less specific than they need to be. If the only way a function can and will ever fail is via an IO error than there's no reason to obscure this behind a broader error type.

Take the `put` function. There's two ways that this function can fail - either the chunk store is at capacity or there was some kind of implementation-side error storing the data. The return type of `put` should reflect these two possibilities so that the caller can handle them accordingly. But for the `get` method a "chunk store already full" error doesn't make any sense. So why should `get` be able to return that as an error?


No, he didn't


Yeah, todomvc demo does not have the server correctly configured. Will change to hash, was just to show that the router supports not hash too :)


But here we're not creating random Python object instances by hand; we're calling `Model.objects.get_or_create()`.


actionWhat? :)


considering libsoxr is default is recent audacity versions, I'd prefer to avoid REQUIRED_USE here and have 'enable libsoxr -> disable libsamplerate'


For now I can merge `ChunkStorePutError` into `ChunkStoreInternalError` if you like, although I've implemented it like this to show you what I have in mind. Also I'm not too keen on the name `ChunkStoreInternalError` (it suggests some kind of internal programming error) but I couldn't think of anything better.


... with something like:
```
$(usex libsoxr "$(use_with libsamplerate)" "--without-libsamplerate")
```

(but someone else might have a better/shorter syntax)


OK, I'll take a look.


To save a few lines, you can extract the try to the `for` level, and use `catch (Exception e)`:

```java
for (Object enumConstant : castClass.getEnumConstants()) {
     try {
         ....
     } catch (Exception e){}
}```


Везде все красиво работает, вроде.

Доска с пустым названием:
http://voile.gensokyo.org/
http://voile.gensokyo.org/1.html

Доска с числовым названием:
http://2ch.hk/8/
http://2ch.hk/8/1.html


@lucabrivio: Please check this on LTC. It seems as a good solution to me.
@joergoster: check your idea as regression tests on standard STC and LTC. If better than this, OK, but I bet that this solution of lucabrivio is more perfect than it looks on the first sight.


Konštanty sa podľa konvencii píšu veľkými písmenami.


I guess I should point out that I only came across this while hooking up one of my old projects on Meteor as an exercise to know Meteor better. While I do agree that most users are on Susy 2, we should try to avoid altering the frameworks/libraries we package for Meteor to avoid further fragmentation and to keep the process of keeping up with the upstream as smooth as possible.

I've seen you've had a few things reorganised for Susy One back when it was a part of the package. I [got this](https://github.com/designorant/meteor-susy) to a point where Susy One sits nicely in place but chokes on `@for-legacy-browser` mixin which is part of Compass 1.X, that hasn't been packaged for meteor yet. This takes us back to rather complicated [francocatena:compass](https://github.com/francocatena/meteor-compass) where we would have to update `compass-core` to 1.0.1, and hope this would fix Susy One complaining.

Whether this is worth pursuing, I don't know. Compass is already dead in the Node based community, and [does not really work without Ruby](https://github.com/sass/libsass/issues/82) anyway. By the time it's decoupled from Ruby there will hardly be any for using it (if anything but supporting legacy stylesheets).

I may take a dive into this to save a few wandering souls trying to get into Meteor smoothly with their old codebases, but I totally get your point of removing Susy One now.

Thanks a lot for packaging this btw!


Hence I put it in the qoutation mark. For some devices you really see a differrence if you combine the sm toolchains with some custom flags. But I know about the fact that some people are critical about that, which is why I disabled the custom flags by default ;)


Cuenteme .. que es esto?

Saludos

--
A/A/
Il 28/Ott/2015 00:06, "Freddy Rondon" <notifications@github.com> ha scritto:

> @Maureen-Hernandez <https://github.com/Maureen-Hernandez> Thanks for the
> contribution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spantons/zebra-parsers/commit/17b1d17ca706176b078b90c50dce82c6cef23cc2#commitcomment-14027171>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.716](http://teamcity.monogame.net/viewLog.html?buildId=10931&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 551, ignored: 6 Build time: 00:21:34


@christianalfoni there is no need in trailing slash in `baseUrl`. Hash can appear without it.


@alkemist, please see the commit message for c14fc791380efa7e690e7e6d58226f5d21a81968, which provides a rationale for *not* wrapping long lines in markdown (and other prose-oriented text files under source control).

If you have a counter-rationale, it would be good to discuss it, that we might all converge on a common convention.


@flavour How can I remove the person-filter that I added as this here yet show the selected person in the S3OptionsFilter dropdown on load?


@flavour Please review


> I'm a bit sad you didn't do -x when you cherry-picked... Now the original commits are not identifiable anymore...

Oops, I didn't know that option :-s At least the commits can still be found by name.

I'll tag and merge -s ours into master, then, shall I?


It's for Python 2 only so I guess it's out of scope. Anyway these methods and their types are provided by the stubs in typeshed.


My preferred refactoring here is to add a `page-sequence.<section>` attribute-set that defines the `format`, but also uses `_force_page_count` so that the actual `<xsl:page-sequence>` element can then just use a single attribute-set.


I don’t care enough about the issue to resist.


scenario 預期結果沒改到。或是這邊的Cost_100指的是每本書單價都是100?


What about this one instead?
try:
   DEBUG = sys.argv[1] == 'debug'
except IndexError:
   DEBUG = False


Follow us on twitter


就 * 0.95 就好。能讓看的人越直覺越好。


Sure, but this looks like a great way for debugging complex tools.


Am 28.10.2015 um 01:53 schrieb Ben Gardiner:
> @aholler @dominicgs
>
> I know this was a [WIP] commit but I'm trying to understand what's the next step here to get the new internal queues working with Injectors. Can you point me in the right direction please?

Basically you have to find out where the messages do come from the
injectors have to put into the queue and when.

Afterwards it's just a call to enqueue().

The problem I've seen was that the pictures about the injectors only
have shown them as a source of messages, but the code looks like they
are in fact listening for messages too (maybe to find out when to inject
something). It also might be possible that they are replacing messages.
I don't know as I haven't really looked at what they do (as said, lack
of interest/use case).


這個 step 顯得有點想得太多了。也可能這是綠燈重構完的結果就是。如果只是為了 pass 第二個 test cases, 就直接判斷書是幾本，決定結果的折扣就好。


You are comparing absolute url's with relative ones
### BaseUrl `/todomvc` and `onlyHash: false`
absolute: `/todomvc/active`
relative: `./active`

### BaseUrl `/todomvc` and `onlyHash: true`
absolute: `/todomvc/active`
relative: `active`

### BaseUrl `/todomvc` and `onlyHash: false`
absolute: `/todomvc#/active`
relative: `#/active`

### BaseUrl `/todomvc/` and `onlyHash: true`
absolute: `/todomvc/#/active`
relative: `/todomvc/#/active`


I think this should cover also ValueError


Thanks for the TIP. I didn't know that.
Going to add it.


It can, though it looks weird :-) `/todomvc#/active`. Usually I do not think things should be too opinionated, but in the case of displaying and defining urls I think it should be. It is just confusing if not I think.

But anyways, if that opinion is to not define urls manually I am happy with that :-) Then you just relate to the baseUrl and everything else works out.


Image source :
http://www.cloudera.com/content/www/en-us/documentation/archive/cdh/4-x/4-2-1/Hue-2-User-Guide/hue2.html


Can you replace with a description of the class?


foo


Не використовувати беззмістовны назви класів


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


Source of your confusion is that you are comparing absolute url's with relative ones
### BaseUrl `/todomvc` and `onlyHash: false`
absolute: `/todomvc/active`
relative: `./active`

### BaseUrl `/todomvc` and `onlyHash: true`
absolute: `/todomvc#/active`
relative: `#active`

### BaseUrl `/todomvc/` and `onlyHash: false`
absolute: `/todomvc/active`
relative: `/active`

### BaseUrl `/todomvc/` and `onlyHash: true`
absolute: `/todomvc/#/active`
relative: `/todomvc/#/active`


真的想太多了，這邊 interface 切出來，太快了。
需求到這邊還屬於很單純的，3本書結果就是 * 90%, 2本書結果就是*95%, 一本書就是*100%。

在修改 production code 以通過眼前這個紅燈的 scenario，請用 baby step 的精神，也就是用最直覺、簡單、少量的程式來通過測試案例，接著才進行重構。這樣可以有效避免 over design。


See https://github.com/square/picasso/issues/364
This is just a temporary workaround waiting for v 2.5.3 of Picasso.


If `o.getClass() == String.class`, you can use [Enum.valueOf](https://docs.oracle.com/javase/6/docs/api/java/lang/Enum.html#valueOf%28java.lang.Class,%20java.lang.String%29).


There is nothing strange in `/todomvc#/active` Hash from the begging have been used in such may. Hash part means: this part only for client, server shouldn't relate on it


Yeah, I agree, I want baseUrl to say: "This is now the root of the application". All hyperlinks should be defined as if they were on the root url. The router takes care of the rest :-) That is less confusing as you do not have to think about the baseUrl when defining urls. You treat your app as running on the root url.


Agree


Restful api 不改有動詞喔，應該改為 /Users


@samdark As the default in `RateLimiter` is to use `yii\web\User` or `Yii::$app->user->identity`, why do you query again for a user instance here? You should already have an instance at this point.

So you could either fetch the values from the current user session or simply `return  [$this->allowance, $this->allowance_updated_at]` in case of an ActiveRecord.


thanks, I will try to fix it


Yeah, # is just a separator, but `/something/#/` reads better than `/something#/`. That is what I meant by opinionated. Choose the most readable format for the user :-)


Thanks for the help, @coryo!


Yep, all those files were moved to NEventSTore.Domain.


Yes and it works so if you are not constructing url by hand. That's why `getUrl` was introduced. The only thing needed to switch used scheme is just change router options,


Are you sure this method to create an enum from its ordinal is really necessary? The ordinal changes if you change the order of the constants in the code. I think that having enums read from strings can be enough. And this code would be more simple.


Google doctest, it looks like the python interactive shell


#8 #32 #41 #42


this name should not be changing for enabling server rendering, fyi.


代码里，还有一些socket 的练习，服务端就的是java mina nio


Don't use speedhacks with online stuff, that's just stupid.


@sol ^


Follow us on twitter


Hi Brad,
I think that the different place of the Direct Download settings you see in the Layer Configuration UI may depend on the Extensions dynamic lookup. Probably they come into a different order on your system. I will redo the screenshot to only contain that isolated section.

In reference to your downloading issue do you have any chance to share the sample, and additional information on it? Could you also please share your configuration files (datastore.properies, auxiliary files, indexer and so on, if any?). I'm wondering if for some reason, the "like" filter doesn't work properly or works differently on some specific DBs.


This is not right. We don't change the name b/c we render on the server. The component name here simply matches what's in the ERB helper call.


Jup, I will document that on the Cerebral site as soon as I get some time on my hands. And then I will make the new version official :-)

Thanks for the discussion, I was a bit hung up on my demo example, but now we are on the same page :+1:


Very nice! Just tried it and this does help a lot. Thanks for addressing the issue.


separator і br зайві


No spaces in file names!


progress


no, not for now.  i can remove them.


You can't access a record member from a pointer -- it has to be deferenced first, so only your `ValueOf` inner class should implement `memberOf()`

Consider the following C code:
```.c
int account_value(struct account *pa) {
   int years = pa->years;
}
```
The `->` operator is equivalent to first dereferencing and then accessing the member:
```.c
int years = (*pa).years;
```
GCC will generate (roughly) the following Gimple for the above:
```
gimple_assign<MEM_REF, T1, *pa>
gimple_assign<COMPONENT_REF, T2, T1.years>
```
The way to think about it in terms of generators is:
```.java
pa = new RecordPtrVarGenerator()
t1 = pa.valueOf();  // returns instance of RecordPtrVarGenerator.ValueOf, representing (*pa)
t2 = t1.memberOf("years"); // returns instance of RecordPtrVarGenerator.Member, representing (*pa).years
```


Quick question,

the new guzzle 6, I only need to change:

```
$connection->client();
```
to
```
$connection->connect();
```

and keep track of ```expires_in``` ? Or are there other breaking changes?


This is a duplicate of line 56 above.


I'm not so sure on the capital letters. Usually, we use lowercase, unless we run out of options.


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


Ah, OK. Then I rest my case.

Thanks!


Why do we have unused code?


I will add more stuff to it later this week. We have removed network from
wifi config in broadcom plugin because we wanted to allow configuring
bridged wifi easily under network bridge config. So in our broadcom release
we add wifi to bridge like any other interface. The openwrt wireless is
currently a copy of a few files from my broadcom plugin so this line that
forces removal of network from wireless config must have creeped in as
well. So thanks for pointing this out. When I get back to this plugin I
will fix it.
On 28 Oct 2015 08:53, "lvzaina" <notifications@github.com> wrote:

> x.network.value = "";
>
> This leads to my wifi does not obtain an IP address (DHCP).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mkschreder/juci/commit/3464685c14d06895fa0ac62b6f1826a085cd5078#commitcomment-14031769>
> .
>


Hm, should the prefix really be `a` if it's a boolean value?


This should be called `.gitkeep` :grinning:


jst dltng


Removed this, as I wanted to do a simple pull request first.


TURN OFF


:)  it's always a struggle to get a good set of descriptive but terse names here.  the scheme you have is fine.  i can recast them.


suggest that matching and textinput should not be used on accessible ios courses as fundamental bugs with ios stop it working properly. have updated component limitations wikis.

textarea, input[text] and select do not receive 'focus' events on selection with voiceover cursor, when the picker or keyboard is closed the focus jumps to the top of the page. this commit circumvents this bug on ios9 but does not work on ios<9.


this is correct, there's a ServerReduxApp and a ReduxApp.

Notice you *import* from ServerApp below:

https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/client/app/startup/serverGlobals.jsx

```javascript
// Example of React + Redux
// Shows the mapping from the exported object to the name used by the server rendering.
import App from './ServerApp';

// We can use the node global object for exposing.
// NodeJs: https://nodejs.org/api/globals.html#globals_global
global.App = App;
```


We need to restructure this. We can't have a different column for all the different types of audio, it's totally not cool


Just added some comments on what the variable names mean


ok, i'll check them.


ileons-3cores is giving abnormal residual....


ok


なんでだろ。　まだ改行が無いよって言われてるね。
でも確かにありそう。
末尾の行は空欄とかも無いまっさらな状態になってる？


Why is this changing? ReactDom is correct.


@kwkhaw I fixed the original code like this to make it work under Xcode 7. How it was working in the prior code without using explicit import statements?


Seems like 'Ã' should be substituted by more than one character (Ä, Ö, Ü ...) which cannot be intended.( But this could also be a rendering issue in my Browser?)


i'm tempted to put variable names in there instead of macro invocations hehe


TODO: icon cache'ing


use the same line

```erb
<%%= react_component("HelloWorld", {}, prerender: <% options.server_rendering %>, trace: true) %>
```


GJ！


Swap this functionality, It should create the user in the database before sending the email. It's better that way in case of any unforeseen contingency regarding the sending of emails


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


See comment below regarding using the same line.


When running this code from the command line the options for maximum hits per interval and maximum interval time can be specified through -H and -T, respectively.

This code iterates over a text file containing paths to multiple ROOT files and stores each one individually as an item in a list (lines 38 - 49). Luminosity section information from the items in this list are then stored in the variable named lumis (line 51).

A time map is constructed from lumis which contains the run number, luminosity section, and total number of hits (lines 61 - 67).

The time map is then iterated over in order to define the intervals based on the number of hits in the interval and the total interval time. Once an interval is defined then the interval index, hit number, luminosity sections, first run time, last run time, first luminosity section, last luminosity section, unix start time, unix end time, and unix mean time are recorded into the interval dictionary (lines 96 - 171). All of this information is then written to a ROOT file and stored in the branches of a tree (lines 185 -212).


Any reason why we have unused code here. Please remove


Yes, we need to check all the requirements before tagging in beta to be sure we didnt miss anything.


Shouldn't this be `this.destroyed` instead?


ServerReduxApp?

BTW, if we're call it "App", we don't need to say "Redux". Please follow the example of the https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/client/app/


as touch devices always have accessibility enabled - because of the lack of tabindex and tabbing support - the course should not overwrite it current accessibility state on the lms.

accessibility state saving is therefore reserved for desktop use as it is irrelevant on touch devices.


1911-8-4-Update (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


@singingwolfboy This doesn't match setup.py:install_requires (nor did it prior)


//test ajout d'un commentaire


StrictDelimiter was added in Delphi 2006 so I think there should be a DELPHI2006UP check added here though FreePascal also seems to have it.


Interesting way of doing this. Learn something new every day :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2682](https://teamcity.mesosphere.io/viewLog.html?buildId=65282&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:14


> なんでだろ。　まだ改行が無いよって言われてるね。
> でも確かにありそう。
> 末尾の行は空欄とかも無いまっさらな状態になってる？

すみません、スペース入りっぱなしでした。
今修正して確認します！


should html escape this (this is part of the "new" security model) using the attr() function:
``````
echo "   <option value='" . attr($code_key) . "'";


yes, will do.


great job! keep going


Should html escape this (this is part of the "new" security model) using the xlt() function:
``````
echo xlt($code_types[$code_key]['label']);


@alexfedoseev We're trying to make a template for using Redux (and maybe React Router), and it seems that it just makes more sense to have this hooked up with some dummy component so you can verify immediately that this works. So I'm wondering if it makes sense to even have the generation option of Redux (and then React Router) without the hello world example.

Also, @robwise is probably not going to know exactly what goes in the Redux template for the hello world example.

Rob, I'm thinking maybe we **ONLY** offer a hello world with Redux, and we can expand that one option later to include React-Router. This is an advanced, opinionated template, and the utility here is if you want to do it like the way we're doing it. We really can't afford to support many options.

@alexfedoseev what's your thought on this?


This is just going to make sense if we have the concrete example.


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


this should not be Alex :smile:


Maybe something like "Change Me!"


Don't worry too much about the html escaping stuff. You'll get the hang of it with more code. They are essentially just short cuts for the php htmlspecialchars() functions. Can see here for more details on the "new" security model(it's actually a couple years old, but we refer to it as "new"):
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan (See Step 4)


[jRWD.pdf](https://github.com/BlackMagic/jRWD/files/22300/jRWD.pdf)

The latest documentation as of 2015-10-28


Додайте /.idea/ в глобальний .gitignore


@robwise the code for redux does look right. Very impressive.


I see that as good. Master lost on time 116 times, and this version only 6 times.


I think this should be 'of', not 'in'. (like the coffeescript had)


Looking great with a few suggestions.

Let's eliminate redux without HelloWorld.


mdr c'est stupeflix qui ma forcé :D

j'append le Swift la ça picote un peu.... surtout les iteration les cast,
les optionnels.... c'est plus safe mais je me sens moins super man de me
prendre des bifles par le compilateur toutes les 5 secondes ....

Sinon c'est cool

Nico

Le 28 octobre 2015 09:06, teriiehina <notifications@github.com> a écrit :

> OMFG !?!? Tu commites des *tests* sur *github* ??? Soit tu as perdu un
> paris, soit tu t'es fait piraté ton compte !!!! :P
>
> Ou c'est stupeflix qui t'a forcé à basculer du bon côté de la force ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/daida/TestTek/commit/6fdeb112f9caef22025e3f985b7e12750216d4d7#commitcomment-14031936>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2683](https://teamcity.mesosphere.io/viewLog.html?buildId=65283&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


the reasoning for this is that i dont think we can serialize a null object


imo the client should which game its connected to. it could be useful for other stuff aswell


I really shouldn't be using unwrap(), but I don't know enough about Rust's complex type-based error handling system to style this code with better usage...


Default it to ICD10 since this is the new thing (note it will get reverted if a user saves globals and does not have ICD10 active, but not a big deal); the next release will set ICD10 as an active code type by default to avoid this from happening.


doesn't `NULL` have to be `vobsub.c_str()` here?


I added some minor comments inline.

http://www-ekp.physik.uni-karlsruhe.de/~tmueller/files/paper.pdf


the client port is dynamic, it probably changes every time a client is launched. the prefered part was me thinking that we may or may not force the client to use a port. or if the client asked us to reply on port 80 we could reject that.


See: https://github.com/JuliaLang/openspecfun/commit/e91925a9fb8e2e6601daccc9417f9fef243d5ac4


because it doesnt belong here. look in Map.Activate...


license missing (:


Probably better to place this at the bottom of the 'Features' tab, and can see what the community thinks where best place is.


Nice code and most importantly it works.


Original codebook was corrupt.


that is up for discussion


FYI @jturcotte


Review is complete. Let me know when next revision is ready (guessing it will be ready for a community review and likely ready to go into the codebase).


Exactly `isContains` or `isContaining` have not found yet. But after reading [Android API on View object](http://developer.android.com/reference/android/view/View.html), I have found out that:
1. After `is` stands adjective i.e. `pressed`, `enabled`, `focused` etc., NOT third-person singular verb, like `contains`, `gets`, `runs`. For example, if we want to find out if specific tab is pressed, we can create method like `tab.isPressed()`... if we had `tab Object` of course.

2. But we have only one Object - `Tabs`, so we should create method, that finds out if it has specific tab or it is containing specific tab. I could offer few names for current method: `Tabs.hasTab()`, `Tabs.isContainingTab()`. BTW looks like we are duplicating with public method `contains` our private method `isContainingTab`. Why would not to leave one public method `Tabs.hasTab(tabsList, propertyName, propertyValue)` or `Tabs.isContainingTab(tabsList, propertyName, propertyValue)` and use it in another public method `add`?!

3. We can go other way: public method name as I offered above - `Tabs.hasTab(tabName)`, `Tabs.isContainingTab(tabName)` and private method something like `checkTab(tabsList, propertyName, propertyValue)`.


hmm hmm


realistically speaking the IP wouldnt change, and since we already need the IP for wdsl...
the real argument though is that the "getip" on server would only work for local ips. so basically this is/was useless atm.


it doesnt, broker does. also this method should be removed. it is a leftover from before i made Broker initializable outside of map


@justin808 @robwise I think generation of example should be similar to `rails scaffold` command: generate easy customisable templates with some placeholders.


I have no idea how to add my assets folder.  Nothing else I changed will work without it.


I think you might want ```!= 0``` here instead of ```< 0```? Otherwise it'll return true if r1 > r2?


bugfix for missing 'type' in selectors

defer the refocus on previous element to overcome ios 7 bug with the voiceover cursor whereby it falls back to the closest element when the popup is removed from the dom.


Yes, you named it that. And you had told me to look at the dummy example: https://github.com/shakacode/react_on_rails/blob/master/spec/dummy/client/app/startup/ServerReduxApp.jsx


The procedure to add a new direct path works but have still some limitations
1. Only half of the path is remapped
2. DPDK applications inside the VMs MUST be running before the direct path is created
3. Some packets are lost due to procedure to copy them when doing the remapping has not been implemented
4. Statistics are reset (This should be easy to solve)

The procedure of removing a direct path is implemented but it does not work, only the first VM is able to remap the ring, the second one shows an error when reading the ivshmem device.


@kwolekr I don't insist on this being solved this way. I would prefer this to stay like this, since it looks cleaner to me, but if compile time is more important to you then I won't object changing it.

As far as I have seen it didn't compile on any platform.


司马光砸缸


Thank you very much ,your juci project is great!!


孙猴子上树


Please reintroduce `onPage` as this made many tests much more readable.


Checkmarx scan results 28 Oct 2015 10:57 UTC +02:00

High
Medium
Low
Info

New
12
64
798
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

f2f26a42fcb8f71329dbabcdf3b1c99e2211e157	@dortal


@justin808 I've got the Redux template for the Hello World example already implemented and working? What do you mean I don't know what goes in it?


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


Yeah, VIM does clamp when you swap with `O`, it's just we have different views on what the end of line is ;-)
Here in VIM a blockwise selection from '8' till '3':
![past-eol-vim](https://cloud.githubusercontent.com/assets/1772013/10783851/c07899a6-7dae-11e5-9863-61738dc0fcbf.png)
After `O`:
![past-eol-vim1](https://cloud.githubusercontent.com/assets/1772013/10783850/c0717cf2-7dae-11e5-9a0b-1d086f39bfce.png)

The core of the problem is in `TextLayoutp.getBounds` it clamps the offsets to _visible_ characters only:

```java
	int length = text.length();
	start = Math.min(Math.max(0, start), length - 1);
	end = Math.min(Math.max(0, end), length - 1);
```
_A solution_ could be to add an average character width when the bounding box of EOL is calculated.
I'll update the code to get rid of '- 1' to cover a few more cases.


I've tweaked `routeTest` to check whether matched url was prevented and vice versa. :)


I encounter different problem when I try to build an ol.source.WMTS from a GetCapabilities with different layers (see the [jsfiddle] (http://jsfiddle.net/thhomas/oayjrxvc/)).
* I see different behaviours between latest ol.js and ol-debug.js version. The ol.js version allows me to build all the layers tilegrids from GetCapabilities when ol-debug.js does not.
* when I use ol-debug.js version with 2 layers with different extents, the first one builds up (in fact its extent is not taken into account for it is filtered in [wmtssource.js] (https://github.com/openlayers/ol3/blob/master/src/ol/source/wmtssource.js#L439-L445)) and second one is not because (I think) of the changes you made here. In fact `Math.floor(x)` has not the same behaviour than `x | 0` and the capabilities extent gives a bad tile coord. As I do not have the necessary understanding of WMTS format, could you please help me.


Could you please send me that patch as a file? Not sure if copying that as a comment changed something in the formatting.
I get a "fatal: corrupt patch at line 6" when trying to apply it.


I think these two lines should instead be an `else if` below the `[arg isEqualToString:@"--rebase-in-progress"]` branch.


use `DataType[]` instead; int key can be an index in array.


Extra blank line?


`Name[]`


IByteBuffer


Should probably move that to the previous commit :)


哦,好,我再改改


I was trying to implement this with minimal work.  I think the CFEngine developers should refactor that code and optimize it, I don't feel comfortable doing it.  If that's a blocker for the `/proc/net` parsing, let's discuss further.


Thanks for this!


Oh this was also applied to binary files, so the prepopulated.db got corrupted


choose a better name as this hides `object.GetType()`


I suggest this instead:

	if([someOptions[kOakCommitWindowContinue] boolValue])
		fprintf(stdout, "TM_SCM_COMMIT_CONTINUE=1\n");

This then only has extra output when the continue action was used, thus should be backwards compatible with bundles that do not expect the extra info.


Same comment as in https://github.com/tzz/core/commit/08057fed88d21557e53191e1b73e01ca821d40cb#commitcomment-14033136

This is so complicated... I can't fix it by myself :)


@DDuarte could you make travis break the build if a file starts with a space ? It seems to be a recurrent issue


Yes indeed, good catch!


Can you tell me how should I adjust the `sscanf` flags?  Thanks.


Doing this would have the `<h1>Server Rendering</h1>` show up even when server rendering isn't enabled. Also `options.server_rendering?` only works in the context of the generator, that's going to give an unitialized constant error.


Oops, typo!


No `GoogleColumnChart` is specific. Renamed it `GoogleChartClass`


@NatashaKSS This behaviour could be dangerous as it bypasses the rest of Logic's executeCommand method which may or may not have side effects in the future.


`system` should inherit the parent process stdin/stdout, so no thread or draining should be needed.
If this is not the case, I would expect many things to break.
I did it because the old approach threw EBADF sporadically.


Two suggestions:

1. Drop the symbolic constants and instead add/remove the `NSAlternateKeyMask` from `self.commitButton.keyEquivalentModifierMask`.
2. Introduce a new `performCommitAndContinue:` action and update `self.commitButton.target` as well, that way we don’t have to check `self.rebaseInProgress && !(NSEvent.modifierFlags & NSAlternateKeyMask)` in `performCommit:`.


The `shutdown`?


`methods` lowercased ;)


Hello @adonishi,

Thanks for multi line support,

I have requirement of two line chart with irregular intervals.

Like One line has 100 points and second one have 20 points.

is there any solution for that ?

Regards
Rahul


can you help me?
my problem: when i zoom map, my polygon knocked off the map


Checkmarx scan results 28 Oct 2015 11:09 UTC +02:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
6
32
399
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

64950ba3148081eaf42ccc64e6f9f6009c3b5c5b	@amitchen22

2a2b89493c5bd00f6329920ffd098b63f4013de1	@amitchen22

c5d0c589d221845f3b2af00a34135c8f87e9d2b7	@amitchen22

fa375d4467c81554879bd4ca6bd50a4637c9db5a	@amitchen22

2f369d4c0639dd229607bd14d84c66cb2142c209	@amitchen22


@JohannesHoppe Hättest du den  Text aufgrund des relative komplexen Beispiels anders aufgebaut, sodass sich das Codebeispiel besser einbetettet?

Auch nur ne optionale Frage.


Please delete the current content and reference the DEVELOPMENT.md. The stuff in here is outdated.


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


That looks like a such rendering issue. Works fine.


I think a git merge will lead to a lot of conflicts if you have changed a lot of code, so I'm afraid you should merge them manually.


We often refer to boolean values as “flags” so I’d go with `continueFlag` instead of that underscore.


Hello Chaveiro, yes well I spent quite a bit of time trying to work out why this part was not working, I was getting a bug where it wasnt joining correctly so I rebuilt it in the way I was familiar with. Then I realised it was actually just the line:
if ($padding_mode=="join")  that was failing which is now replaced with: if ($padding_mode!=null)
But is there any reason to use static? Where else is it used?


Should be `NO` instead of `false`.


1911-8-4-Update (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


:+1: Btw, you should also fix the beginning of the docstring.


This has now been fixed thanks


OK. I want to build Carbon 5.1.1 soon on my device, are there any non-CM related bugs? (like often AOSP keyboard crash in RR).


The ambuguity between many parameters were already bothering me. Index, offset, elemSize, staticSize etc. Thats why I give as staticSİze and keep internally as elemSize


Good catch about `line_size`.  Fixed.

The nesting is because the file format is: header line 1, data line 1; header line 2; data line 2; ...

Here's an example:

```console
% cat /proc/net/netstat
TcpExt: SyncookiesSent SyncookiesRecv SyncookiesFailed EmbryonicRsts PruneCalled RcvPruned OfoPruned OutOfWindowIcmps LockDroppedIcmps ArpFilter TW TWRecycled TWKilled PAWSPassive PAWSActive PAWSEstab DelayedACKs DelayedACKLocked DelayedACKLost ListenOverflows ListenDrops TCPPrequeued TCPDirectCopyFromBacklog TCPDirectCopyFromPrequeue TCPPrequeueDropped TCPHPHits TCPHPHitsToUser TCPPureAcks TCPHPAcks TCPRenoRecovery TCPSackRecovery TCPSACKReneging TCPFACKReorder TCPSACKReorder TCPRenoReorder TCPTSReorder TCPFullUndo TCPPartialUndo TCPDSACKUndo TCPLossUndo TCPLostRetransmit TCPRenoFailures TCPSackFailures TCPLossFailures TCPFastRetrans TCPForwardRetrans TCPSlowStartRetrans TCPTimeouts TCPLossProbes TCPLossProbeRecovery TCPRenoRecoveryFail TCPSackRecoveryFail TCPSchedulerFailed TCPRcvCollapsed TCPDSACKOldSent TCPDSACKOfoSent TCPDSACKRecv TCPDSACKOfoRecv TCPAbortOnData TCPAbortOnClose TCPAbortOnMemory TCPAbortOnTimeout TCPAbortOnLinger TCPAbortFailed TCPMemoryPressures TCPSACKDiscard TCPDSACKIgnoredOld TCPDSACKIgnoredNoUndo TCPSpuriousRTOs TCPMD5NotFound TCPMD5Unexpected TCPSackShifted TCPSackMerged TCPSackShiftFallback TCPBacklogDrop TCPMinTTLDrop TCPDeferAcceptDrop IPReversePathFilter TCPTimeWaitOverflow TCPReqQFullDoCookies TCPReqQFullDrop TCPRetransFail TCPRcvCoalesce TCPOFOQueue TCPOFODrop TCPOFOMerge TCPChallengeACK TCPSYNChallenge TCPFastOpenActive TCPFastOpenActiveFail TCPFastOpenPassive TCPFastOpenPassiveFail TCPFastOpenListenOverflow TCPFastOpenCookieReqd TCPSpuriousRtxHostQueues BusyPollRxPackets TCPAutoCorking TCPFromZeroWindowAdv TCPToZeroWindowAdv TCPWantZeroWindowAdv TCPSynRetrans TCPOrigDataSent TCPHystartTrainDetect TCPHystartTrainCwnd TCPHystartDelayDetect TCPHystartDelayCwnd TCPACKSkippedSynRecv TCPACKSkippedPAWS TCPACKSkippedSeq TCPACKSkippedFinWait2 TCPACKSkippedTimeWait TCPACKSkippedChallenge TCPWinProbe TCPKeepAlive
TcpExt: 0 0 0 0 0 0 0 0 0 0 11500 0 0 0 0 19 54289 6 8595 0 0 4750373 2322451 460670558 0 6737545 225034 2259621 5590408 0 15871 0 0 1 0 13 7 11 79 956 12328 0 589 542 165546 7087 2131 9059 10472 1582 0 579 1 0 8714 11 2851 1 766 1008 0 54 0 0 0 0 0 264 473 0 0 0 0 478934 0 0 0 0 0 0 0 0 935551 47540 0 12 186 187 0 0 0 0 0 0 9 0 559463 0 0 0 22420 43785784 38 1075 29 1512 0 0 7911 0 0 4 32 35140
IpExt: InNoRoutes InTruncatedPkts InMcastPkts OutMcastPkts InBcastPkts OutBcastPkts InOctets OutOctets InMcastOctets OutMcastOctets InBcastOctets OutBcastOctets InCsumErrors InNoECTPkts InECT1Pkts InECT0Pkts InCEPkts
IpExt: 0 0 74500 3908 127790 0 65909575212 65238163889 17334437 195941 15821538 0 0 55521335 2660 162 3598
```


>@lucabrivio: Please check this on LTC. It seems as a good solution to
>me.

Now it seems not top priority but feel free to schedule a LTC test on my repository...


@robwise Good job!


I had a couple crashes recently on g_hash_table_lookup() from janus.c::janus_session_notify_event(), I take it they were due to this problem?


LGTM


Question from @mcltd via slack Oct 27
4. In DnaOligoSchema all characters that are not AMBIG_CHAR are classified as modifications. https://github.com/DeskGen/dgparse/blob/master/dgparse/schema.py#L324 For example, ' ' and '-' come through as modifications. I think this is wrong. I think that at the moment only '*' should be classified as a modification (later, I'll add the other IDT types). I think ' ' and '-' would be better picked up as errors in the subsequenct validation against SequenceSchema (which very strictly rejects NOT_UNAMBIG_DNA i.e. ^AGTCagtc).
Response from @rodoyle Oct 28
4. Yes I agree.


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


lol


I think it is ok to have some variance on style in the code base, so using the above expression is ok for me.

I'm just tuning in to mention that we don't (and shouldn't) apply defensive programming in the syslog-ng code base. That has serious bad effects on readability and increases code redundancy/boilerplates a lot.


It is not consistent somehow with the other operator[] then :/ From the point of optimization thats ok but we will be mostly copying pod types  ( nearly - except uuid and string ) and else return Zerobuf objects.


keep const and create
`const enhancedStyle = { ...this._getCommonStyle(), ...this._getPositionStyle(), ...style };`


sequence.bases from xlsx parser was ending up in properties, unintended


same


why class, AFAIR we had this discussion and the pimpl classes are structs.


I'm not sure about it. Since Python 3.5 typing is bundled and `import typing` will import typing.py from the standard library, not a version installed from PyPI to site-packages. So it looks like it doesn't make sense to ship installable typing for anything but Python 3.4 and earlier.


Possibly we shouldn't have the header for Server Rendering. That's sort of an informational thing. If you like, it, then yes, keep the if statement.

I'm pretty sure that there's template substitution in the `<%=  %>` syntax like ERB.


Modify five_days variable names to fivedays


Testing typing for PyPy and fixing possible bugs could be another pull request.


Question raised @mcltd via slack:
3. the snapgene parser appears to be different from the other parsers under dgparse insofar that it does not return standard schema items such as properties, sequence.bases, name etc. should I add these items to the output, or is there another plan or preferred approach?
Response from @rodoyle Oct 28:
3) At one point the idea was that the snapgene parser should return a json version of the .dna file format. This is then remapped in a second step, the code for which is in snapgene.init.  I don't have a strong opinion right now.

In fact the code is currently working according to 3) so I have just added a test that the converter exists and returns a sequence as expected.


thank you floffel03 for this quick fix.
I patches that line of code.


Checkmarx scan results 28 Oct 2015 11:19 UTC +02:00

High
Medium
Low
Info

New
6
32
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2f369d4c0639dd229607bd14d84c66cb2142c209	@amitchen22


needed for make_response()


Passt schon. Anders wär's bestimmt nur noch länger geworden. Im Buch können wir ja ggf. didaktischer vorgehen. Kurz vs. Verständlich. Ich kenn das Dilemma sehr gut! 😏

Von meinem Windows Phone gesendet

----- Ursprüngliche Nachricht -----
Von: "Gregor Woiwode" <notifications@github.com>
Gesendet: 28.10.2015 10:19
An: "Angular2Buch/code" <code@noreply.github.com>
Cc: "Johannes Hoppe" <johannes.hoppe@haushoppe-its.de>
Betreff: Re: [code] angular-templating: korrekturen (fa257ff)

@JohannesHoppe Hättest du den Text aufgrund des relative komplexen Beispiels anders aufgebaut, sodass sich das Codebeispiel besser einbetettet?
Auch nur ne optionale Frage.
—
Reply to this email directly or view it on GitHub.


getDynamicPtr ? Because you return pointers with Ptr name in Allocator class.


test function that returns json , that's the way it has been working for me so far ,


Checkmarx scan results 28 Oct 2015 11:20 UTC +02:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
6
32
399
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

64950ba3148081eaf42ccc64e6f9f6009c3b5c5b	@amitchen22


This implements exponential truncated backoff (https://cloud.google.com/storage/docs/exponential-backoff), and is what google seems to be using themselves in their client libraries . It only retries on internal server errors and 429 Too Many Requests. These conditions are rare, and require a bit of patience to let google cloud service recover ;-)  Normally, the problem is recovered after one or two retries.

You're right that 10 retries might be overkill. I could reduce that to 5.

About randomising the entire interval, I do not think that matters much. The worst case sleep time stays the same, and the expected will come down by about 50%.

I think we should subtract 1s from the interval, so that at least the first re-try is performed really quickly (0-1s instead of 1-2s). My experience with GCS is that most errors can be recovered from by immediate retry. The ones that cannot be recovered from immediately, usually require a fairly long time, and often fail eventually anyway.


:+1: Thanks !


That line was just copied over from the redesign-reporting-tables branch (https://github.com/fac/freeagent/blob/redesign-reporting-tables/app/assets/stylesheets/talaria/components/_table.scss#L51) so I must confess I hadn't thought about it. Presumably that was just dropped in to match the old reporting table styles? Shouldn't the default table text colour match the rest of the app though, so only variants would apply a different colour...


@dachary
Need to change :
Xie Xingguo <xie.xingguo@zte.com.cn> <258156334@qq.com>
Xie Xingguo <xie.xingguo@zte.com.cn> <xie.xiexingguo@zte.com.cn>
to:
Xie Xingguo <xie.xingguo@zte.com.cn> <258156334@qq.com>
Xie Xingguo <xie.xingguo@zte.com.cn> <xie.xingguo@zte.com.cn>
in the mailmap too.

Thank you for your patience


Thanks @claudiosmweb


Typo in commit message: I apparently missed the "1" key once while typing "10.10".


Oh yeah, it is. Sorry!

I didn't know we have multiple places for Builders, I just saw those in `net.smartcosmos.builder`...


Thank you


just throw out of range exception


`== TRUE` not required


can actually replace with boundary check + type cast to enum


sidebar


The error messages have to be retranslared on crowdin.


скоріше через дефіс


bike, або краще content


немає відступу


Do we really want the status to be printed at 10 Hertz?
There is a --no-status option, too.


@lenguyenthedat what exactly is your concern?

Sent from mobile

> On 28 Oct 2015, at 3:52 PM, Dat Le <notifications@github.com> wrote:
>
> I know it hardly hurts but probably don't want these to be public?
>
> —
> Reply to this email directly or view it on GitHub.
>


S upřímností nejdřív pojdeš ....

Ale jinak dobře napsané.


Jo, für das Buch wird das Kapitel eeh noch mal grundlegen überarbietet.

On Wed, 28 Oct 2015 at 10:30 Johannes Hoppe <notifications@github.com>
wrote:

> Passt schon. Anders wär's bestimmt nur noch länger geworden. Im Buch
> können wir ja ggf. didaktischer vorgehen. Kurz vs. Verständlich. Ich kenn
> das Dilemma sehr gut! 😏
>
> Von meinem Windows Phone gesendet
>
> ----- Ursprüngliche Nachricht -----
> Von: "Gregor Woiwode" <notifications@github.com>
> Gesendet: 28.10.2015 10:19
> An: "Angular2Buch/code" <code@noreply.github.com>
> Cc: "Johannes Hoppe" <johannes.hoppe@haushoppe-its.de>
> Betreff: Re: [code] angular-templating: korrekturen (fa257ff)
>
> @JohannesHoppe Hättest du den Text aufgrund des relative komplexen
> Beispiels anders aufgebaut, sodass sich das Codebeispiel besser einbetettet?
> Auch nur ne optionale Frage.
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Angular2Buch/code/commit/fa257ff63c09f8a14f57fbf45d04a69cc8f383ef#commitcomment-14033469>
> .
>
--
Viele Grüße
Gregor Woiwode
http://woiwode.info


this should be a USE flag, so it is not hard enabled to start. Generally we should not start services by default, especially since a misconfiguration for sshd could cause problems if it was just started without someone noticing :) so the USE should also be default disabled


Move this to start method


Залиш цей файл тільки якщо впевнений, що він тобі потрібний


removed


Possible to merge into a single if statement
`if filter_files and not afile.startswith("."):`


Great, thanks for these changes!

Just a naive question in this line. Is it correct that you set the 'active_count' index here:

double_vector_iset( obs_std   , active_count , summary_obs_get_std( summary_obs ) * summary_obs_get_std_scaling( summary_obs ));

or should it be the step index like this:

double_vector_iset( obs_std   , step, summary_obs_get_std( summary_obs ) * summary_obs_get_std_scaling( summary_obs ));


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


removed ogg and wav divs


Add </br> only if comment and phone


I think that using name `contains` for all cases would be best solution. Why duplicate word `tab` in method name `Tabs.isContainingTab`?


Wie schon getwittert, expression != statement. Ich denke, der korrekte Termini lautet Ausdruck / expression! Mal schauen, was Patrick P. dazu sagt. (wenn er was sagt).

@GregOnNet


Зміна падінгів, марджинів і бордеру призводить до "скакання" блоків. Зміна падінгів взагалі непотрібна. а щодо бордеру - знайди інший спосіб


Deactivate sort on phone column


Add filter on phone


`base.GetHashCode` returns reference-bound hash code (i.e. it has nothing to do with anything but object reference) and should not be used when overriding `GetHashCode`


@kristopherschmidt that is correct!


Where exactly frame doesnt fit in base texture?


@Optimusprimeums cheers mate!


:yellow_heart: Do remember to use the imperative form for your commit messages. The commit message style for the _Kipit_ project is also useful for your checkpoints.


Because the output gets confusing if you're going to fallback to tar. Once you know that it wasn't because rsync is missing, then it makes sense to output.


@danaspiegel Is this a good way to get the current domain and the other parts of the current URL? Or we might need to move those values as constants to different settings files?


iterator find(const key_type& k){
        Node* ret = root;
        while((ret!= nullptr) && (k != (ret->data).first)){
            if(k < (ret->data).first){
                ret = ret->left;
            }
            else{
                ret = ret->right;}
        }
        if(ret == nullptr){
            return end();}
        else {return iterator(ret);}
    }

    const_iterator find(const key_type& k) const{
        Node* ret = root;
        while((ret!= nullptr) && (k != (ret->data).first)){
            if(k < (ret->data).first){
                ret = ret->left;
            }
            else{ret = ret->right;}
        }
        if(ret == nullptr){return end();}
        else {return const_iterator(ret);}
    }

    unsigned int count(const key_type& k) const{
        if(find(k) == end()) {return 0;}
        else return 1;
    }


it is rather taxing to call this method. Refrain from overriding `Equals` as it might get called inadvertently.


why not a simple boolean here? then you could skip the match to "yes" in `checks.rb`


Makes sense.
Thanks for your contribution.


stop working!


What was the problem before, using a or b uninitialized? Does this change keep the old codesize?


the `if ip_addr.ipv6?` could be the main condition above. so why not just replace `if version == :ipv6`?


Checkmarx scan results 28 Oct 2015 11:33 UTC +02:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
6
32
399
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

64950ba3148081eaf42ccc64e6f9f6009c3b5c5b	@amitchen22


I'd make it part of decoding


Good point, but if we move to the pipe implementation it will go away anyhow.


@joelmarcey this looks like you forgot to remove before committing.


Add a space before %>


stupid question why don't we use the `Rails.logger` here as the default?


Looks awsome! I'll merge this right away


Add a new line


check if null is not returned


Remove this line


bad name


@igrekster I've already changed it locally, if you want I'll create a new pull request so we can compare notes.


Hi Snipe many thanks for your help.
I tried to launch

php composer.phar install --no-dev --prefer-source
php composer.phar dump-autoload
php artisan migrate

from documentation but I got nothing to do/migrate

How can I get your edited filed to my server?

Many thanks.
KR


Remove one space


I think this line is an issue since it will mutate the input.  You could do `mpz_abs(r->n, self->n)` to make the copy into r, but maybe we can skip the abs() step altogether?  mpz_mod_ui says "The remainder has the same sign as the dividend" which it sounds like is what we want.

I think we have another issue here though -- we return different hashes for some longs such as `(-1 << 63) - 3`, where we say `-3` and cpython gives a wrapped-around value.  I think the issue is from using mpz_get_si to extract the value -- if the long overflows a signed integer, gmp will keep the sign but only return the bottom 63 bits, which isn't what we want.  I think it might work to do mpz_get_ui and then apply the sign manually.

There also seems to be a [mpz_mmod_ui function](http://web.mit.edu/gnu/doc/html/gmp_4.html) which might be what we want.  But other doc pages don't mention it, so if we can't have that, maybe we can do something like
```C++
mpz_t r; // can save a little bit by not creating the BoxedLong
mpz_init(r);
mpz_mod_ui(r->n, self->n, ULONG_MAX);
long n = mpz_get_ui(r);
if (mpz_sign(r) < 0)
  n = -n;
return boxInt(n);
```

I'm not sure the actual hash value is hugely important (except for the case that the long is also a valid int), but we might want to add some tests just to make sure this code is doing what we want.


check if null is not returned here


hmm all the loggers would now return to `STDOUT`. shouldn't the log happen in the script that uses this lib?


Just do a fresh git pull, or download the updated file and replace it on your server.


Missing space


Hope that makes sense - it took me a long while to understand the old Alphabet stuff. Its something we'd like to clean up at some point, but for now we live with it.


Uhm... what? This basically returns a random value. If you `:fit` the textbox into a width of 5px before doing this, then this will at most return a width of 5. You should at least do `textbox._layout.width = -1` before this to get rid of that previous width (but this still keeps the previous height which could also be just one pixel, that might influence the width...).
This should rather use the API from #535 (if someone says that this is a good API and should be merged...).


assign values explicitly


tiny, but `checkAndThrowCAPIException()` will check PyErr_Occurred for you.  I think we should probably get rid of checkAndThrowCAPIException (since most of the time the caller should have a return value that they can check more cheaply), so maybe convert this one to throwCAPIException()?


Comment test


@albertdev, I'm happy for you push your changes ( my context on this if a few months old :-( ).


чому не inline-block?


Close input tag


minor, but as long as you're changing things, i'd prefer to format this as:
```ruby
options = {
  :allow_nil => true,
  :scope => scope,
  :if => (proc do |record|
     ...etc...
  )
}
```
to make it easier to visually scan the options


I think the cast is unnecessary here, I would rather change the `deserializer` variable type to `Object` and change the `findDeserializer()` signature to return an `Object` as well : it would mirror the `findSerializer()` method signature which is a good thing I guess.


що це тобі дає?


asd


Would be good to wrap at 80 chars


I'd expect two classes: descendant of `BytesToMessageDecoder` and descendant of `MessageToMessageEncoder`. See MQTT codec impl for reference.


Come to think of it, in your case you could actually use https://github.com/sindresorhus/read-pkg-up#readpkgupsyncoptions instead of all this.


Because the lib can called outside of the crowbar env where we have no `Rails.logger`.


illumos code doesn't check for feature_get_refcount() returning ENOTSUP
(which means feature is disabled) in zdb. zfsonlinux added a check in
zfsonlinux@784652c due to #3468. The check
wasn't reintroduced, which is a potential regression.
I still don't see the difference between illumos and Linux code that could cause the bug to appear only on Linux, so I'd like to confirm this is a regression by testing.


path="kernel/" <<<< the / causes havoc on my build machine


this is std functionality on `IByteBuffer` which is core to DotNetty


Залишити цей файл тільки якщо він дійсно потрібен


What does havoc means ? You can remove the "/" if you really want to ^^


Remove comments


Das kommt eben, wenn man NUR klaut. :-D


Remove space


This would be clearer as just

```ruby
options[:case_sensitive] = false
```
Though arguably even clearer would be to simply always include the option:

```ruby
options = {
  :allow_nil => true,
  :scope => scope,
  :if => (proc do |record|
     ...etc...
  ),
  :case_sensitive => !has_case_insensitive_index?(column, scope)
}
```
and that being the case, I guess there'd arguably be no need for the `options` hash any more:

```ruby
validate_logged :validates_uniqueness_of, name,
         :allow_nil => true,
         :scope => scope,
         :if => (proc do |record|
              ...etc...
          ),
         :case_sensitive => !has_case_insensitive_index?(column, scope)
```


illumos has `guid=%s` due to https://github.com/illumos/illumos-gate/commit/57221772c3fc05faba04bf48ddff45abf2bbf2bd.


Bin deiner Meinung... statement wäre an dieser Stelle besser.

Eine Expression ist `{{ name | uppercase }}`


Для чого тут float: left? не потрібен


Commit message error : use launchctl and not apachectl... :bug:


Thanks, will definitely look into it! :)


@gabizou sry for these fails. Dont saw this, think it was Intellij. Will fix it


Valid point. Thanks.
It's fixed now.


The scan failed.
Included commits

Commit ID	Owner

a6f44e41cbec199739cdda9aba94708fd0a2405a	@amitchen22


oh ya


Až na tyto řádky je soubor z originální šablony nezměněný.


Because `Tabs` means all `tab`'s we have or some more complex entity that contains array of tabs and some additional properties.


@ronen Thanks, I put an options hash there is because I only want to add the `case_sensitive` option when it's case insensitive. You know that by default it's case sensitive.


Remove schema.rb from this commit and clean 2 tables in your database


Так можна, але я мабуть би виклав картинку, щоб була можливість іі скачати з сайту


And if you have more tips regarding my plugin, don't hesitate to create an issue or ping me. I'm far from a Yeoman pro :).


breaks kenel building :).. I did, just wanted to point out, if anyone else would have problems


Ups. I made an error when updating the cocoa backend.


No, because then any classes will get defined twice, and will be incompatible. Your builtins symlink is wrong for the same reason (I've masked it in my fork of mypy though), and more likely to cause problems, too.

```python
def foo(BaseHTTPServer.HTTPServer) -> None: ...
def main() -> None:
    # error: expected BaseHTTPServer.HTTPServer, got six.moves.BaseHTTPServer.HTTPServer
    foo(six.moves.BaseHTTPServer.HTTPServer())
```


LGTM


@Acrews5 DWMR (done with my review). Gold star! :star2:


Just wanted to say thanks for helping to maintain this, it has given me lots of hours of enjoyment! :eyes: :tongue: :bomb:


These scripts definitely have more application to an RPM-based installation. The intention was to publish an MVP to the community; most common daemon-like applications provide a file or set of files using the [System V convention](https://www.centos.org/docs/5/html/Installation_Guide-en-US/s1-boot-init-shutdown-sysv.html) (e.g. Postgresql, Hadoop, httpd, etc.)


Unfortunately I don't believe we can get this for them. It's basically your User ID or Group Chat ID, which you can get by using the bot IDBot. Getting the group chat ID is a little more painful, you need to open a link and get it from the json. That's how it works for now. Telegram might change this in the future.


@bmathews thanks for the link! I didn't know the difference between those characters. But I think `>` is the one that make the text easier to deal with. That way, long paragraphs can be reflowed into a readable .yml column without affecting the content.


Version 2.0 USAR ESTE


Also note that symlinks get replaced by their content during `pip install`, so you can't even ask tools to detect if they are a symlink.


Is this something that go fmt or go vet suggested? Or is it just the golang standard?

Should I run `go fmt && go vet` before each commit?


Ok


wonder, what would be an efficient way of figuring out the minimum geohash that encloses a given bounding box. So far all examples are to get the geohash for a single coordinate given the desired precision. I've an Argentinian beer waiting for the part that takes a bounding box and returns its minimum enclosing geohash :)


Opps! I made a mental note to change this before the pull request but it slipped my mind.


Yeah, in `>` mode, the empty lines are necessary to created a bulleted list instead of a continuation of the same line.


Métodos começam com letra Maiúscula no C#


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2764](https://teamcity.mesosphere.io/viewLog.html?buildId=65574&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:16


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2765](https://teamcity.mesosphere.io/viewLog.html?buildId=65575&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


Ah, that makes sense! Thanks.


Alright I'll get on it


Looks like some merge conflict line @mayuoak


I beg your pardon, I confused this with #1981


I appreciate not having to pass on such addresses  when calling your code. My issue (if it is one) is mostly about the IDs themselves and how to track which drone has which XBee ID. I suppose it's easy to find which ID the current drone has, but do you also track the other IDs in any way? How easy would it be to send the exact same custom packet to all other drones, for example? Just a loop over some list of IDs or a range until the maximum ID?-------- Oorspronkelijk bericht --------Onderwerp: Re: [drone-tomography] Implement custom packets for the XBee sensor simulator (89797d3)Van: Tim van der Meij Aan: timvandermeij/drone-tomography Cc: Leon Helwerda Perhaps I'm not understanding the question correctly, but the destination ID mentioned here is literally an ID, not the sensor's address. For instance, if you want to send a packet to sensor 2, just add to_id = 2 to the packet. The code then finds sensor 2's address using https://github.com/timvandermeij/drone-tomography/blob/master/settings.json#L95-L99. The first item (0) is always the ground sensor, the second item (1) corresponds to sensor 1, et cetera.

In short, this code provides the abstraction from XBee device addresses and allows external code to just address a sensor by its ID. The only prerequisite is that the sensors list in the settings JSON file is correct, but that only has to be done once for the entire network. It is also possible to do a node discovery, but that takes quite some time and does not find all sensors in the worst case, so I decided against doing that.

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


LGTM!


I wonder if cleaner to do ``length>>`` instead?


I think this needs to be addressed at the language design level first. We need to decide whether we want to allow unpaired surrogates (like they are allowed in UTF16 string literals) or whether they result in a parse error. If we allow them we have two choices: we say that they are replaced by the "invalid character" byte sequence or we encode them to UTF8 anyways (https://en.wikipedia.org/wiki/UTF-8#WTF-8 notes that Rust is doing something like that).

We have a precedent for both behaviors -- string arguments of custom attributes are saved to metadata with unpaired surrogates replaced with "invalid character" while local constants are saved to PDBs in the modified UTF8 format, I believe.


In the middle of the night, @ktbartholomew stirs restlessly in his sleep.

_"They're... they're using unresponsive CSS!"_