I see the reason why you state that. Strangely the version you are using in demo website and version from bower are quite different. Bower version 2.3.5 seems to be stable where as 2.3.7 and 2.3.8 are broken.

Demo site used JS (http://www.malot.fr/bootstrap-datetimepicker/bootstrap-datetimepicker/js/bootstrap-datetimepicker.js) has working condition code:

`if (this.pickerPosition == 'top-left' || this.pickerPosition == 'top-right') { top = offset.top - this.picker.outerHeight(); } else { top = offset.top + this.height; }`

But once you download from bower 2.3.8 or have a look at latest code in grunt (https://github.com/smalot/bootstrap-datetimepicker/blob/master/js/bootstrap-datetimepicker.js), it has below code

top is undefined before this code runs, it gives out NaN.

`if (this.component) { top = top - containerOffset.top + 169; left = left - containerOffset.left + 210; } else { if (this.pickerPosition == 'top-left' || this.pickerPosition == 'top-right') { top = offset.top - this.picker.outerHeight(); } else { top = offset.top + this.height;`


04.02.2016, 20:54, Even Rouault kirjoitti:
>
> In case we will resynchronize GDAL internal shpopen.c with upstream
> shapelib, it would be perhaps good not to change the prototype of
> SHPCreateSimpleObject() to avoid compatibility issues. We might want
> to create another wrapper, or perhaps just call plain
> SHPCreateObject() from GDAL.
>

I noticed that SHPCreateSimpleObject() is only a simplifying wrapper but
was just making a first try.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajolma/GDAL-XYZM/commit/12d89e5a1055ad95a3ec604dbc3884a6830f6aee#commitcomment-15894557>.
>


Great - an easy setup that would enable the user to see pump data on NS will be a nice introductory step. The doc could then walk through the script and use it to introduce openaps use commands, reports and aliases. I would suggest including a basic cron setup example so that pump data upload can run on schedule. I do not think there is anything wrong or dangerous with just giving a crontab example at this stage.


@anssih i already apply the osx channelmap to our layout in initialize - the problem is that side and rear speakers can be swapped - dependibg the soure is using wav or usb channel order - at least that is what i read from the apple header comment here http://pastebin.com/hgPn3i9A (and i don't know when it is save to swap those channels - atm i map 1:1 from osx map to ae layout)


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


The form
```c++
this->template getTrailingObjects<MarkupASTNode *>()
```
is a bit baroque in comparison to seeing `this + 1` but I guess that's just C++ for you.


current performance: approximately 3.2MB/s


Let's change this to 2016, and I think you can add your copyright message here.


Year change and add copyright message here also


Safe place to maintain precious ML code for the course.


I like this stage. While it will eventually be possible to completely automate adjustments to basal profile, at this point it would probably be sufficient to simply instruct the user how to make use of the data available to manually make own profile adjustments if deemed necessary. Maybe DIYPS style basal reporting on NS would be introduced at this point?


I should implement releasing of reference count through the dispose method for other objects as well right; i.e. in StoragedLinuxGlusterFSVolumeObject, StoragedLinuxGlusterFSGlusterdObject too?


I already commented above that Cron (or daemon) could be introduced even earlier - with NS pump data upload; at this step the user should be able to make own modifications to crontab  to accomplish looping steps


Good idea!


test


I can, but should we? All other public API methods more or less useful for use with completer. The only obvious use for `elementSiblingIndex` would be to get selected element index. I was thinking, having a `this.selectedElement` property would be helpful.


Seems <code> preserves new lines for demo purpose, unlike other html tags.


Оправи си мейла


It doesn't hurt, and sibling index is needed often enough that libraries have utility methods for it. :)
Re:`this.selectedElement`, do we have any use cases?


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


This is a problem here. On most systems using RPM, `$(bindir)` is defined as `/usr/bin`. Ironically, this isn't really a problem on most Linux distributions, because most have completed the [UsrMerge](http://www.freedesktop.org/wiki/Software/systemd/TheCaseForTheUsrMerge/). However, the problem comes in with Debian (who is [currently working on UsrMerge](https://lwn.net/Articles/670071/), too) and non-Linux systems (who follow different conventions).

On these systems, `/bin/rpm` is a very important distinction from `/usr/bin/rpm`, because the former indicates that it's available in the early boot environment and requires privileged access (*BSD, especially FreeBSD).

I have no idea where "rpm" exists in this symlink creation call, since it's not forcing it to be `/bin/rpm`.


Do we still need this after we've dropped pe_bliss?


Now its working fine. TY mate!


hii...


This all looks good. This comment is just a side note: it would be good to extend "max iob" to include various (currently hard coded) oref0 parameters that a user may ultimately be allowed to and enabled to adjust based on personal preferences.


I think the reasoning behind this line is: the (hot) producer is generating events, but the consumer is consuming (pulling) events at its rate. The question remains when the source produces more than the consumer consume: what do you do with the additional events generated?


do you have an example on how to use this keyboard?

Thanks,
Abba


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


it should be qt:expand or qt:exptext.

> On Feb 5 2016, at 12:49 am, lohiaguitar91 <notifications@github.com>
wrote:

>

> I did set the library - however when I type qu:expand it doesn't do anything
at that point.

>

> —
Reply to this email directly or [view it on GitHub](https://github.com/raguay/
MyAlfred/commit/f270bd5ac8401ac139420caf19001ce702eb0184#commitcomment-1589316
7).![](https://github.com/notifications/beacon/ACX0xwsmzicNG2hNASRqm9u9hxWwNMT
Iks5pg4bFgaJpZM4HNnDM.gif)


@sgdc3 Thanks a bunch! :smile:


Soo, if we are getting this error, how do we fix it?


Select the keyword in the Alfred Preferences for that workflow and click the
debug button (looks like a cockroach in the upper right corner). Then run the
keyword and email the output.

> On Feb 5 2016, at 1:22 pm, Richard Guay <raguay@customct.com> wrote:
it should be qt:expand or qt:exptext.

>

>> On Feb 5 2016, at 12:49 am, lohiaguitar91 <notifications@github.com>
wrote:

>>

>> I did set the library - however when I type qu:expand it doesn't do
anything at that point.

>>

>> —
Reply to this email directly or [view it on GitHub](https://github.com/raguay/
MyAlfred/commit/f270bd5ac8401ac139420caf19001ce702eb0184#commitcomment-1589316
7).![](https://github.com/notifications/beacon/ACX0xwsmzicNG2hNASRqm9u9hxWwNMT
Iks5pg4bFgaJpZM4HNnDM.gif)


This really needs to be explained. I've been on-and-off looking at the latest meal-assist code and still can't say I fully understand what's going on or what should or should not happen in various situations. A flowchart would be nice.


I'm wondering if this part for getting the time is necessary. Couldn't you just use ``$scope.current.currently.time`` ?
The Javascript Date() class can be then used to extract the info you want from it.


A "reference" manual for openaps would fit well in the back


Seems fine, I guess, I would be compelled to reuse what LLVM offers, but it does have a higher cognitive load and it's not necessarily easier to read for someone new to this code. In particular I'm definitely not a fan of `this->template ...` and direct calls to `MarkupNodeWithChildren::getChildren()` because of the new inheritance. Having a `MarkupASTNode` and a `MarkupNodeWithChildren` makes it seem like `MarkupASTNode` doesn't have a notion of children, but it still does.

What inspired this change?


@18F/ato This is my first pass at giving the page plain-er language...is all of it still accurate?


lock


Lock


Drat. I was thinking about people who cloned my repository, rather than forked it.

Updated.


I think that $lasterr is misleading.


May be correct is `permission_exists('system_view_memcache')` not `if_group("system_view_memcache")`


What's this have to do with the "hackaround"?


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


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 141](https://teamcity.labkey.org/viewLog.html?buildId=301191&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (2 new), passed: 42 Build time: 01:22:01
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

```


If you like, the browser url will already be available in `create$html_url`.


/cc @geramirez @mzia


Hi Daniel,

I hope you are doing well,

Is it possible for you to tell which version of SES this script support?

3.x or 2.x?

Kind Regards,
Kamran


In the comment of `LifecycleHook` you specify that *only one action can be specified at a time* but I don't see this being validated.


(1) sounds great, and was definitely needed.  When noise is added, we need to make sure it uses a constant seed, but otherwise that's fine.


You can download From Google Play.
Go here, https://play.google.com/apps/testing/rs.pedjaapps.smc.android
Click "Become Tester" and then download from google play


is $state used here?


You should cal `np.random.seed(seed)` before this and have `seed=0` or something similar in the call spec


Why changing this label? Additionally I'd propose creating a constant.


tested


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


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


Nice


The name fullInstanceId is fine if it returns an Id.

However I think that it is not what the routes files need, because it would force them to do an if for emptiness to avoid the double // when it's empty, which they cannot do. So I proposed solving it by creating a new variable called fullInstanceId which would contain an / plus the fullInstanceId or an empty path if fullInstanceId is empty. In other words, move the if clause to the java class.

Makes sense?


__I love him so much


Looks good to me.


I checked and it still works with newest Xcode. [Screen](https://dl.dropboxusercontent.com/u/8868668/Screen%20Shot%202016-02-05%20at%2007.41.07.png).


that is an impressively strange issue.  Nice work tracking it down


@Azolo, I fixed it on #310


Isn't the polling thread still needed to execute `PerformDeviceScan()`?


I think you could just do `test_hdu.header = fits.Header(keylist)`, or even declare the hdu here:
`test_hdu = fits.PrimaryHDU(data=test_cube, header=fits.Header(keylist))`

Not a big deal, just more convenient / slightly faster.


Sure. I haven't finished reading the sources, yet... I'll get back to it on Monday, I gues..


add `header['RANDSEED'] = seed` too


running


Overall, this looks great.  Make it into a pull request on pyspeckit and if I have time I'll try to contribute back some edits.


I don't know how, but this merge deleted/reverted some of the coverage tests.  I will recover them this evening.


Ну, я тут кое-что почитал, и выяснил, что REST way это все-таки POST: http://stackoverflow.com/questions/5591348/how-to-implement-a-restful-resource-for-a-state-machine-or-finite-automata


@danmar Sure but noone knows in advance how `foo()` would be called.


eww


gg


now hash performance: approximately 4.25MB/s


The deal here was that I assumed that all the container tests had to pass. I should have questioned them.

After speaking to @danielmarbach we've decided some of the tests don't make sense. So all this code was an effort to get the container to pass the previous container tests.


`react-komposer` don't export `useDeps`


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


Perhaps `getListQueryBuilder()`?


あまりなさそうですが、`http` を省略したURLが渡された場合、コンパイルエラーになりそうです。
（例：`//example.com/path/to/dir`)

CDNはともかく、このクラスでスキームだけの省略をするケースは無さそうなので
このままでも問題はない気がしますので、判断は gomo さんにお任せします。


Насчет общего наименования согласен. При расширении задачи может потребоваться рефакторинг. В пределах ТЗ - нормально.
По второму пункту: обычно я так и делаю. Но опять же почитал кое-что (https://engineering.shopify.com/17488860-restful-thinking-considered-harmful), решил попробовать иначе, через второй контроллер. Есть свои плюсы ИМХО, при расширении набора состояний основной CRUD контроллер можно не трогать.


Don't worry about it, thank you.

On Fri, Feb 5, 2016 at 5:02 AM Quis <notifications@github.com> wrote:

> Sorry, too busy to do that, and thnaks for ur adjustment.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gisto/nwjs-shell-builder/commit/d4484401a9c5475e9cf9cdd36d3314e5111f5333#commitcomment-15904279>
> .
>


Extra space here.


Thanks for the tip!


This looks great Stephany! The indentations are perfect, the use of HTML tags is good, everything looks great. Seems like you've got a firm grasp on this :+1:


The only real comment that I might have is to indent everything one out of the body tag. But besides that, I can't really find any other flaw. Great work!


[![Build Status](https://travis-ci.org/nisibibu/pull-request-test.svg?branch=master)](https://travis-ci.org/nisibibu/pull-request-test)
を追加してください。


このテストみて `url.Build` 時にドメインをセットできる機能を追加してもいいかもしれない？と思いました。
ドキュメントルート以下のパスだけを使いまわしたいケースはありそうかなと。


Nice!


Просто с генератора осталось, тесты работают нормально как раз из-за инициализации модели. Но согласен, надо убрать.


We can't use such a prefix - http://www.gnu.org/software/emacs/manual/html_node/elisp/Key-Binding-Conventions.html (amusingly I've misled you yesterday - `C-c ,` wasn't reserved for the users; no idea who came up with this exception).

Btw, turns out our old prefix was actually allowed.


Поэтому везде проставляю вручную. Тоже можно убрать.


Nice method for centering :+1:


Good use of the CSS properties! And way to use the hex codes for color. :+1:


Seems it'd be best to use `C-c C-t` as the prefix.


CSS file looks great!


@AGaliuzov, have you read all comments in this PR? I've had the same discussion with @pestOO. I'll repeat. We have no strict rule to always add doxygen. At the moment we have a lot of places without the doc. And because of lack of time on win porting I think this is not the right place to **start** following best practices.


This should probably be in the test map as well.


Yes, signed is important, not just for phone. Any project that signs assemblies needs it.
PCL for mobile... Xamarin too


why is $state injected?


Может. Если включить валидацию на уникальность имени, то точно приведет.
При имеющемся наборе тестов пока не привело. Я выставил рандомный порядок.


Is there a replacement for ``nox-review revs``?


I wasn't entirely sure about this line... Maybe it should consider `serverConfigFound` to be true when there's no `loggingImpl`?


@qkmiao & @Richard-Yunchao, Please help review it, thank you.
It related with /deqp/functionals/gles3/uniformbuffers.html.
The first issue is in js to store in little- or big-endian format, may is not suit to native code.
The other issue assure that reading pixels is not out of canvas range.


Great indentation with your code Paul. Way to keep things organized and visually pleasing!


Probably shouldn't assume there will be a "#" to strip out. If this servant runs on search pages it won't match any posts because Tumblr made the "#"s there css instead of html.


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


done


You can move this CSS rule ("body { border: solid 4px; }") into the CSS file. Also, just for naming convention, try to get in the habit of not using spaces when naming files. I know it's easier to see, but I've run into issues with my computer not being able to read it at times. Just a tip to avoid any future headaches :dizzy_face:


ああ、たしかに。
まあ、でも必要になったときでもいいかな。


Seems like all the closing </li> tags are missing the letter "L"... :open_mouth:


In order to display a validation error message ng-message is used, it is more cleaner than this, please use it.
https://docs.angularjs.org/api/ngMessages/directive/ngMessages


@AGaliuzov, personally I prefer to write dtor and ctor in the beginning. E.g. virtual dtor **should** be always here. And if it in the beginning of the class it will never change it's position. And it's easy to find it's presence. It will be somewhere at the beginning.

 I agree that we can discuss this and use consistent approach.


Missing closing </ul> tag. The code might be working, but try to get in the habit of always adding a closing a tag for HTML tags that need a closing tag.


(just kidding, this is starting to look very good)


@AGaliuzov, this will help us to see was it null or not.


Good use of hexcode! :+1:


Is there no middle ground? I'd be happy if we had the spinner and hid it once we used -setHTML (or whatever the method is). Try turning off the spinner in a thread once -setHTML is sent. That may be the right amount of delay.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 242](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=751&buildTypeId=XbimEssentials_XbimEssentials) is now running


what's `Sever p`


了解です。対応させときます！

他の人が読むかもしれないので、さっき口頭で話した「なぜCDNは省略しているか？」を共有しておきます。

httpsのページにhttpの外部ファイル（JSやCSSなど）を読み込むと警告が出ます。スキームを省略すると親ページのスキームに合わせたスキームで読み込むので、CDNなどはhttps/http両方で配信しておき、省略したものをタグに書くのが一般的になっています。


@invisiblek  I met libc fault when load this lib.
I adapted CM13 for LG F460 (apq8084+mdm9x35)

rild can`t run.

02-05 15:06:06.544      615-615/? A/DEBUG﹕ Revision: '0'
02-05 15:06:06.544      615-615/? A/DEBUG﹕ ABI: 'arm'
02-05 15:06:06.544      615-615/? A/DEBUG﹕ pid: 4628, tid: 4628, name: rild  >>> /system/bin/rild <<<
02-05 15:06:06.544      615-615/? A/DEBUG﹕ signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x29b4267f
02-05 15:06:06.551    1745-2134/system_process W/NativeCrashListener﹕ Couldn't find ProcessRecord for pid 4628
02-05 15:06:06.557      615-615/? A/DEBUG﹕ r0 29b4267f  r1 29b4267f  r2 00000001  r3 00000001
02-05 15:06:06.557      615-615/? E/DEBUG﹕ AM write failed: Broken pipe
02-05 15:06:06.557      615-615/? A/DEBUG﹕ r4 ffffffff  r5 29b4267f  r6 b6ec6609  r7 fffffe88
02-05 15:06:06.557      615-615/? A/DEBUG﹕ r8 00000000  r9 b6c7e559  sl beb02dd4  fp 00000006
02-05 15:06:06.557      615-615/? A/DEBUG﹕ ip 80000000  sp beb02cc0  lr b6e91c69  pc b6e5abe6  cpsr 800e0030
02-05 15:06:06.571      615-615/? A/DEBUG﹕ backtrace:
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #00 pc 00017be6  /system/lib/libc.so (strlen+21)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #01 pc 0004ec65  /system/lib/libc.so (__strlen_chk+4)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #02 pc 00040ee1  /system/lib/libc.so (__vfprintf+6296)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #03 pc 0004503b  /system/lib/libc.so (vsnprintf+126)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #04 pc 000e2633  /system/vendor/lib/libril-qc-qmi-1.so (qcril_format_log_msg+38)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #05 pc 00098dc3  /system/vendor/lib/libril-qc-qmi-1.so (qcril_uim_remote_client_socket_init+562)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #06 pc 000b8795  /system/vendor/lib/libril-qc-qmi-1.so (qcril_init+1468)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #07 pc 000bf95b  /system/vendor/lib/libril-qc-qmi-1.so (RIL_Init+2362)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #08 pc 0000163d  /system/bin/rild
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #09 pc 000162f5  /system/lib/libc.so (__libc_init+52)
02-05 15:06:06.571      615-615/? A/DEBUG﹕ #10 pc 00000ef0  /system/bin/rild
02-05 15:06:06.711      615-615/? A/DEBUG﹕ Tombstone written to: /data/tombstones/tombstone_03


@mperry yeah, I did not check if the overloading would cause a problem in practice.
my objection to renaming ```list``` to ```reduce``` is that we 'lose' a 'convention' that sum types have an eponymous method for deconstruction of a value (aka. structural pattern pattern matching or catamorphism for non inductive types). eg. https://github.com/functionaljava/functionaljava/blob/master/core/src/main/java/fj/data/Either.java#L75 and https://github.com/functionaljava/functionaljava/blob/master/core/src/main/java/fj/data/Option.java#L124
Structural pattern matching is important so this methods should have good discoverability. But maybe ```reduce``` is good enough for inductive types...


Having `$.siblingIndex ` can be useful, it's just I think it makes more sense to have such method in Bliss, but would you add it to Bliss? If the answer is no, then probably you should be more reluctant to add it to Awesomplete.

The `this.selectedElement` would be similar to boolean `this.selected`(which returns true, if there is selected element) but will return element itself or `undefined`. Maybe we should not add it as well. It's easy to add something to API, not so easy to take it back. If done from scratch, I would probably add just selected element property, no boolean.

Even more generic property is `this._suggestions`, which contains the list of currently matched item data. Since you haven't replied to my idea having `this.suggestions` public property, I guess your answer is no :)

**Anyway, I'll change this to `$.siblingIndex` if your decision is final.** It looks like conversations stretched over time don't work very well, but I don't know how to move with the whole key/value story using small PRs and issues to discuss :)


L.536 play?


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


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 242](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=751&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:40


In sort this means, that after this commit if you want to test a npc that has a script attached to it that is in an instance.
you must then test it in the instance and not on gmisland for example.
this is only for creatures that are scripted in instances.


@AByzhynar, can you propose practical way how to do this? I don't like `check memory leaks` too. Since it's not clear how to do this during review.


You should also double all commands to have a `C-something` version (see the doc map for reference). Some prefer not having to stop pressing `C` for the duration of the entire keybinding.


Good job Paul! Overall, code looks great and it's all very organized. A few missed details, but seems like you've got a good understanding of HTML and CSS! :+1:


changed. bumped to 0.2.1 and pushed.


I'll put all my comments here, seems easiest
* I think the set and finalizer should block until the task is complete. That way we can be sure that things are setup/destroyed in the right fashion.
```
proc = subprocess.call(['tset', exec_param])
proc.wait()
```
* We should probably allow the config to be changed to give control over how much latency/packet loss, this could easily be achieved by adding these values into the ```cfme_data.yaml``` config, or even starting a simply ```tc.yaml```
* Thinking about the above comment a little more, we could always make the fixture take a tuple, so it could take the operation *and* a value.
* We are going to have the requirement to set this on the **remote** appliance, and not on the machine we are running the test on at some point, so this code is potentially going to have to move into the ```utils/appliance.py:IPAppliance``` class. Once this is done, the fixture will simply set this up for the ```current_appliance``` from the appliance stack and we'll be good.
* We should also be looking at what exactly we want to test, with respect to this performance. There is the UI aspect, but there is also the intercommunication between providers and appliances. The problem here, is that we would want a functioning UI and to nobble the connection to the providers, which is a lot trickier, needing multiple tc bins to set up.
* The other problem is, we need to be careful of harming the slave/nodes connection to the outside world.

Overall, a great start on this feature, we will get a phase one piece in place and then start to expand it.


Guess we don't have tests for this. :-)


I believe that the request mapping should be /userDetail with the method post because it refers to only one user, not all users.


Good organization of your code :1


Note to self: Use a linked list data structure where each node contains the start, end, id of event, and tail to the next node. Gaps will be determined as difference between end of A and start of B, assuming B is A's tail. Gaps can be instantiated as a linked list itself of similar structure. Insert events in the order they were updated, and detect conflicts in this way. Overwrite earlier events with later events.

TODO Times should be rounded to the minute


I think it'd be best if this were made configurable, as not every Rubyist is using Rails (otherwise this wouldn't have been suggested).


Good job using the <hr> tag!


Your branch in git was reflected as the tip in the mercurial version, so it was effectively landed by a bi-directional sync.

Please don't put experimental code in branches in the main repo, use your own fork for that and use a PR/merge when you want to land. Branches in the main repo may get landed arbitrarily and are a source of confusion for others at best.


There is similiar code in this function, inluding loadTerrains().


test


Quick tip: indenting on tab out from the OL and UL tags will be help you see this a bit more easier.


Does we may to publish secret data?


do not return null, in case of something gone wrong you should trow an exception here


Didn't we already fix this in 0.37? I remember discussing something similar with someone else...


Shouldn't the manager event names use the mapnames instead of the kRO castle names, so that this matches the other woe types?


Try to avoid inline styling if you can. It'll prevent future headaches later. You can definitely move the styling of your 'a' tag to the CSS file.


Nice usage of the hex code :+1:


Good job Khoi! Your HTML and CSS files look great. Seems like you've got a strong understanding of this. Way to go!


Ok


Why Users? I believe that this controlller is used when showing the details of a single user not all so UserDetailController will be more suitable.


Also, change end to now


@rajkumarpb Kindly change the background color to the default, and later change the opacity.
because themes have different colored background colors, for example supervan has a different color. :)


Dude jquery was already to go no setup required


@linas I mean deterministic w.r.t. to the random seed.

I'm trying to understand how the UUID is determined, but I'm puzzled, I can't see anywhere how Atom::_uuid is set to something else than Handle::INVALID_UUID...


@malirod, for me it looks like evident rule. I some one adds new api it is required to provide appropriate documentation. At least for public API. I do not believe that adding few line of doc text will take a lot of time, isn't it?


good catch !


Looks good to me :)


なくす


ok


Ah, I see now.

Agreed, 'reduce' is a bad name.  I propose I change it to `uncons`.  Overloading the type signatures does not clash.

Given `uncons` and `fromArray` clash if they are named `list`, we have to choose only one.  I think most people would choose `fromArray` to given the blessing of being called `list`, this is my slight preference.

Thoughts?


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


What is this for, can it not be auto-generated from source?


I mean the change is fine, just asking. :)


Why is it important to not create a new Fiber for every Promise callback that ever fires?


Ah yes. This should be `mantra-core`. Could you send me a PR?


Should be %%path%% as %%path%% is unique but %%name%% is not.


Please do not use same file for different services. Every service, controller or directive should be defined in it's own file. On pluralsight  in the course "angular-best-practies" there is a chapter named "Organize your code" please watch the video again ang organize the code as suggested there.


多谢！


Do we really need a sourceLoader or could the meta loader do both?
This might improve the performance.


@AGaliuzov we cannot use this macro here. We don't include `macro.h` here because of circular dependencies


You could just add another mask to map.h:

```
#define MAPID_WOETEMASK (JOBL_THIRD|MAPID_REBELLION|MAPID_KAGEROUOBORO)
```
And export it as EAJ_WOETEMASK or whatever you want to call it. Then you would have it available without having to write another function. Could even make this a configureable value instead of a define.


опечатка


Why is the user displayed like this? shouldn't be put in a <p> or something?


oh yeah, our API always sends back objects or nothing.
if you want, please create PR.


Fix typo on commit message


never mind


simply added a comment for explain what was done


You are right. With which julia version did you get the warnings? I have 0.4.2 and these warnings never showed up.


again, value should not be users because you do not update all users but only one, so userDetail should be used as value.


112796613_DesicDehum (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2060 of 2060 tests passed, 1 test warnings)

 * 7 tests had: AUD diffs.
 * 1 test had: Table small diffs.

![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)


"userDetail", you will return only one user


@Creeperface01 Why for what?


6、7、8、9、期末提交文件


With this commit you renamed the npc files from npc/re/woe_te/te_alde_cas01.txt → npc/re/woe_te/te_alde_cas1.txt but actually the name of the map is "te_aldecas1" according to what you use in the npc scripts.


Finally! Great!


Well this is duplicate. I'd prefer either implementing function to run command and call it from booth (Simple and External), or just implement the Simple runner the way we can call it from External runner using `super`.


Why do you return null? if the process fails an exception should be trown
A small example you will find here:
http://howtodoinjava.com/resteasy/exception-handling-in-jax-rs-resteasy-with-exceptionmapper/


Can we delete this branch?


minor test


À la Eric's conversation on emotive objects.


Very true and i had this in mind. I was just looking for a lesser evil than just closing the shared hibernate session, as a means of stopping these long running tasks. :)
@mseaton do you have any more ideas? Or should we revert this and wait for a smart way of dealing with this?


test test comment


Here another duplicated comment.
I think this little patch could have been read twice before submitting.


Sorry, I missed your suggestion. I do agree with `this.suggestions`, definitely better than `this._suggestions`. Please don't assume the answer is no if I don't reply to something. The volume of Github notifications I get is huge, and I often miss things.

Regarding `siblingIndex`, the question is not whether we'll add it (since you also want to add it, just have it private), but whether it will be exposed. I don't see any reason why it wouldn't be exposed. The amount of code is exactly the same. No, I wouldn't add it to Bliss, because it would add extra code which is not useful in many cases. However here we have it in the codebase already, it won't add a single byte to expose it.


Its not deterministic.   There's no random seed used.  Its first-come, first-serve. The UUID is incremented by 1 for each new atom -- see line 146 of TLB.h.   A block of UUID's can be reserved, for distributed operation (so e.g. server A issues ID's in the 10000 to 20000 range, while server B issues them in the 20000-30000 range, guaranteeing that different servers don't clobber each other.) The basic mechanics of this is in TLB.h  The multi-server coordination is in the SQL backend.


тест


@RiaanKnoetze there is a string for in the POT file, see: https://github.com/woothemes/sensei/blob/master/lang/woothemes-sensei.pot#L628

I do not yet understand why Loco translate doesn't pick it up


Nice work mister! May I propose to name the plugin Progressbar of the plugin type: progress?
I expect that will make it easier to roll out different kinds of progress indicators


ah, right


@ldoktor this has been merged, so, please send a patch! :smile:


Ah ok, then its should be just Even.


Probably because it was picked from 8960 or another device and the pick was kept intact.
No worries from me if it isn't used right now; it's not causing issues, is it?


Sorry, when I said "random" I did not mean PRNG, I meant "its not deterministic". It can't be, there is no way of globally ordering the issuance of UUID's, at least not without some kind of cryptographic hashing.  If two different threads are creating atoms, its impossible to guarantee the order of execution between the threads -- its non-deterministic because the OS scheduler might do anything at all, depending on ethernet interrupts, disk-drive interrupts, etc.


Seems like a copy/paste ;)


Посмотрел, это было заменено на `new` в итоге. Убрал вообще.


I don't don't have a clue what I'm doing @mentioned


Is this build available to download or not? And where?


Instrument :-)


I need to know what I'm doing


I think it would be best to keep the old prefix (`C-c ,`) in addition to the new. Let's minimize friction for the users.


This fixes #22


Fixed #22


Same here. Also all the of the dialogs do not work on ios9


I did :smile: (why aren't you sleeping?)


Can you link him here? If he agrees, maybe we can add some people to the contributors - and maybe admin - team to continue development and maintenance of OpenTK as a community effort.


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


@marcows Ack, agreed.  Commit a 1000 lines of perfect code to private repos at work.  Commit 14 to the public at large, and mess up 15% of it.  I have committed a fixup commit to the PR.  Thanks.


Fair enough.


TeamCity pull requests :: netty [Build 1216](https://garage.netty.io/teamcity/viewLog.html?buildId=5449&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4916, ignored: 23 Build time: 00:36:55


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


=P Autocorrect added. :+1:


@drosofff
Did you try this in a workflow or on a collection by hand?
I think I may have seen a bug when used in workflows ...


here, fetch returns a promise: Promise {[[PromiseStatus]]: "pending", [[PromiseValue]]: undefined}
after this, it goes to the .then() below it and returns another response:
 Response {}body: (...)bodyUsed: false
headers: Header
ok: true
status: 200
statusText: "OK"
type: "cors"
url: "https://kinto.dev.mozaws.net/v1/buckets/central-repository/collections/users/records/90176cbc-da9d-471b-a3b3-2c98941dc2e4"
__proto__: Response
contentScript.js:234 cs: onRequestListener(Arguments[3])

This is the correct response. How to I prevent the pending response from being returned?


1. Поскольку join будет in-memory, то выражения для ключей не обязательно должны быть Expression<Func<T, TKey>>, а просто Func<T, TKey>, ключи же по любому будут из in-memory объектов доставаться.

2. SimpleJoinStrategy.cs там есть Func<bool> next - мне кажется можно переписать чтобы был просто bool next, а то так-то всегда можно вместо var a = 0 писать Func<int> a = () => 0, но это банально не нужно.


:confused: ??


Also see [Commit d55d7fe90a80e3cd](http://github.com/weidai11/cryptopp/commit/d55d7fe90a80e3cd3202b5963e0d613e2df0824d).


ping @denisivan0v, мы тут комментим код андрея, так как pull request нет.


Agreed, changing to `this.suggestions` and exposing `$.siblingIndex` then.

By add method I meant exposing method as public API. Wasn't worried about code size, just API surface increase. New DOM related method in Bliss would make more sense to me, but I haven't looked into Bliss closely. Also we've had a discussion about adding Bliss as dependency, from which I had a feeling, that you liked an idea and we'll eventually do it sooner or later.


performance tests in development environments are a mix between impossible and pointless. What devops does is measuring production performance continuesly to catch regressions early - and fix it before users notice.

Check newrelic.com - they make good money from that need. Too bad we use about the only language not supported ;)


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


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


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


I shared the same history back with you, where i re-did 121 by hand ... it seems to do what it should.
I'll open an issue. This used to work in older versions of galaxy.


> I think it'd be best if this were made configurable

Personally, I disagree. `update` is just too generic of a name. Almost any project could possibly have a custom class with a method called `update`.

If you insist, though, I can make it configurable.


You should be able to replace that with `if(VAR)` (see `if(variable/constant)` at https://cmake.org/cmake/help/v3.0/command/if.html)


:eyes:


@malirod , this checklist for contributor, not for reviewer


> Didn't we already fix this in 0.37?

Nope.


Missing EOL


vanwaar komt smartProxyFunction?


I would never do such a thing ;)


быдло кодер рыжий


0.5.0

On 5 February 2016 at 08:28, thofma <notifications@github.com> wrote:

> You are right. With which julia version did you get the warnings? I have
> 0.4.2 and these warnings never showed up.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wbhart/Nemo.jl/commit/802841b32f29f1de8423b7b6a0ba3f56b68228fc#commitcomment-15907222>
> .
>


писька


сиська


Uhm... who works with strange locales anyway :innocent:


The two faces have identical docstrings.


Ooh yeah, I did the same almost at the same time.
I think it was also work with .name indeed, outside from a workflow
Do you think there is a chance to turn around by re-making the workflow ?

Anyway, this is all I needed to finish the metavisitor manuscript. I'll
stay focus on that

Chris

Chris

Christophe Antoniewski

ARTbio <http://artbio.fr/> - Tel +33 1 44 2*7 70 05*
Drosophila Genetics & Epigenetics <http://drosophile.org/> - Tel +33 1 44 2
*7 34 39*Mobile +33 6 68 60 51 50
https://twitter.com/ARTbio_IBPS
https://twitter.com/drosofff

2016-02-05 8:50 GMT+01:00 Marius van den Beek <notifications@github.com>:

> I shared the same history back with you, where i re-did 121 by hand ... it
> seems to do what it should.
> I'll open an issue. This used to work in older versions of galaxy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ARTbio/tools-artbio/commit/3048cdbea989bc7d28326bf9479fc3010ff8b33c#commitcomment-15907548>
> .
>


@pestOO, in this place I don't see  any problem with if\else. Here we have handling of the bytes_read. All possible cases. 1) >0 2) <0 3) ==0
I see no point extracting any branch out of the current if\else if \else flow.


Context in case one needs to debug multiple mailboxes, over multiple hosts. UID is not unique perse than.


is there a reason why we need the join ?
i removed it before because it worked perfect without but if it make sense to keep it no problem ...


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


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


@Paulomat You have to update BungeeCord...


@Paulomart update bungeecord, ci.md-5.net/job/bungeecord


it throws uncaught exception on timestamp field (i don't understand yet this why), and it throws exception when i use default strongloop user model, and the database contains invalid json data in credentials or challenges field.
i fix the exception with the extend the condition the followin code:
if(typeof val=='string' && val)
it fix the exception with date filed, and empty json field too.

please see around this bug


> Поскольку join будет in-memory, то выражения для ключей не обязательно должны быть Expression>, а просто Func, ключи же по любому будут из in-memory объектов доставаться.

Нужен именно expression, поскольку есть желание оптимизировать запрос в случае, если одно из множеств имеет сильно меньшую мощность.

> SimpleJoinStrategy.cs там есть Func next - мне кажется можно переписать чтобы был просто bool next, а то так-то всегда можно вместо var a = 0 писать Func a = () => 0, но это банально не нужно.

Принято, это один из хвостов.


@Arthur2e5 Why is that?


Thx @konqi for this.
But no need to fix the `extend` function which works for your needs. We have a default set of options which can be overloaded by client. May rename `extend` by `extendOwn` or `assign`.


Thanks for the suggestion, that would be much better. The current implementation is rather a proof of concept, it was quite tricky to find a solution...


Afaik the PCRE developers suggest using PCRE2 (pcre2-8) nowadays? I seem to vaguely remember some performance issues that were fixed in PCRE2 but declared unfixable in PCRE1...


@pestOO, same as in the posix version. I don't the point in refactoring of this if\if else\ else flow. Everithing I can imagine looks worse. I you have some idea, please give me a sketch.


Where did the 32 come from? Why not just `reason[BUFSIZE]`?


Maybe I'm wrong but the following code looks easier for understanding:
```
int bytes_read = -1;
do {
	bytes_read = recv(tcp_socket_, buffer, sizeof(buffer), MSG_DONTWAIT);
	if (bytes_read < 0) {
	  LOG4CXX_WARN(logger_,
	               "Socket " << tcp_socket_ << " closed by remote peer");
	  OnError(WSAGetLastError());
	  return;
	}
	if (bytes_read < 0) {
	  int socket_error = errno;
	  if (EAGAIN != socket_error && EWOULDBLOCK != socket_error) {
	    LOG4CXX_ERROR(logger_,
	                  "recv() failed for connection " << tcp_socket_
	                                                  << ". Error: "
	                                                  << socket_error);
	    OnError(socket_error);
	    return;
	  }
	 // continue reading
	  continue;
	}

	LOG4CXX_DEBUG(logger_,
	            "Received " << bytes_read << " bytes from socket "
	                        << tcp_socket_);
	uint8_t* casted_buffer = reinterpret_cast<uint8_t*>(buffer);
	event_handler_->OnData(casted_buffer, bytes_read);

	} while (bytes_read > 0);
```
But I'm not insist because of lac of time till porting release.


Should be `return run(t, 'lines-between-children.expected', {`, because second argument for `run()` is file name without extension.


I don't think so (but I can't remember why I removed it)


Brilliant, thanks for fixing the joins too.


@pestOO, see c5c826b


I would love it, only if i knew how. lol I don't know native code for shit. lol


> The tests do not break when I change this else block

They do break now.


I posted some changes in order to mesh with the gem I plan on using (Paperclip).


@malirod see https://github.com/malirod/sdl_core_winport/commit/cac1aeecb6ce932fe909c221a61aca3c136f3291#commitcomment-15907708


@AGaliuzov, I promise you that I'll start taking more care regarding the doc. Especially during review ;)
At the moment we have no time for this in the context of the win porting. Feel free to create a ticket.


I will test and merge.


@malirod, in reordered version become obvious that `bytes_read` checking in `while` is redundant - it could be just `true`, and `bytes_read` could be move inside the loop as a `const`.


@ldoktor thanks! Btw, why sleep? :wink:


We deal with this in our RunCi script: https://github.com/jdonaldson/haxe/blob/haxe_lua_rb_dev/tests/RunCi.hx#L854


Method name suggest that this is a boolean function. It should return `false`, not `nil`.


Same comment as https://github.com/konqi/dmak/commit/1ec468778a195afd0325baa9b737763d1542addc#commitcomment-15907680


Cane these commented out lines be removed, now?


Yeah


Nice but need to remove extra chars at end of last time by amending last commit.


done


因为：
* branch 不是固化时间点的
* 于是你不该 checksum 到每次 push 都更新的 branch zip archive 上


> Do you think there is a chance to turn around by re-making the workflow ?

This is easy to fix in galaxy, give me an hour. Since it's a bugfix this can still go into 16.01.


Please do:
add_attachment :cards, :picture


We were thinking about only having the source loader, and extracting the informations on the frontend. This would allow users to have their own information parsers etc.


I forgot executing `rake db:migrate` ...


Oops. :-)


Thank you for your help,
Regards,
Lionel.


We could hardcode magic based on e.g. `WM_CLASS`, but that only gets us more complicated code. Well-behaving clients won't mind an extra `ConfigureNotify` and these don't cause much "traffic" either.

That's why I chose to be a bit passive aggressive and just always do it like this.

TL;DR: No downsides doing it like this and it's the simplest patch possible.


這個是 release 的 URL 啊。


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 146](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=752&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


No objection, please go on.


I know that I have to update bungee, but this is a bug in Netty right? Is that warning very bad or can you ignore it?


Yes I'm sure. Locale is always of the form "ja_JP", "ja_JP" or "ja_JP_JP".


predicate conflicts with method scope variable predicate, I will use some other longer names


啊……看到没有用 github 推荐 tag 名称格式（vx.y.z），再配上 zip 而不是 targz 就傻了。


hot fix


@MBoustani My changes are in this commit- https://github.com/MBoustani/Khooshe/commit/d514523013803a28df9bea6fa29acb25d37c58ed

Checkout this code - https://github.com/MBoustani/Khooshe/blob/openlayer/Visualizer/khooshe_openlayers.html#L77


This line (112) should probably not be removed.


SUPER SPRAWA 10/10!


Rafał please create these files


Modify this fike


Bartosz please modificate these files


Wrong function


DZIENKI ZA SUPER KOD BARDZO MI POMUGŁ !


OOOOO GOOD JOB MEN


You have it, I have a business meeting until noon...

Christophe Antoniewski

ARTbio <http://artbio.fr/> - Tel +33 1 44 2*7 70 05*
Drosophila Genetics & Epigenetics <http://drosophile.org/> - Tel +33 1 44 2
*7 34 39*Mobile +33 6 68 60 51 50
https://twitter.com/ARTbio_IBPS
https://twitter.com/drosofff

2016-02-05 9:13 GMT+01:00 Marius van den Beek <notifications@github.com>:

> Do you think there is a chance to turn around by re-making the workflow ?
>
> This is easy to fix in galaxy, give me an hour. Since it's a bugfix this
> can still go into 16.01.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ARTbio/tools-artbio/commit/3048cdbea989bc7d28326bf9479fc3010ff8b33c#commitcomment-15907848>
> .
>


alright, makes sense, thx! @gdi2290  @jesperronn


The correction is not consistent with the [following commitment.](https://github.com/chocoteam/choco3/issues/345#issuecomment-157759297)
When we deprecate a method, we should follow the [following rules](https://javax0.wordpress.com/2014/02/05/use-java-annotation-deprecated-the-right-way/).


/play what


Update this!


does full refresh mean remove print statements?


I'm not sure this part should be necessary - why should starting docker imply starting sssd.service?  I guess the messy part here is that if an admin *restarts* docker, we'd need this to ensure sssd is also started?


Go check out the [list of releases](https://github.com/Elethom/contacts-phonetic/releases) yourself. I merely did the "copy and paste" procedure.


Of course there are workarounds. And your solution is perfectly valid.

Here's a second blog http://www.gravitatedesign.com/blog/seo-benefits-of-html5-and-schema/


This is already fixed in the current dev. I'm testing now if it is in 16.01, in that case we should upgrade mississippi.


@pestOO, for me the new flow is not much simpler. Plus for me it's much harder to track all those return\continue in all those branches.

A agree that `if (error_condition) { return}` looks reat. But here we have much complicated logic. Currently the logic at least indented the same way. In you variant the positive case have different indent. It's confusing.
There is another thing. For performance it's better to check more possible cases first. In your variant most possible case (normal flow) comes last.


Beau travail


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


@ReneHerthel AFAIK the I2C-alternate function (as well as other AFs) have always the same mux position. That's why you don't need to put in `GPIO_AF4` twice


The idea being that any iteration of however you want to spell True as long as it starts with t.


TeamCity pull requests :: netty [Build 1217](https://garage.netty.io/teamcity/viewLog.html?buildId=5450&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6409, ignored: 23 Build time: 00:43:57


It would be helpfull to test changes on local branches, like I'm used to work with nixpkgs.


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


test comment


I think the preferred terminology is "index" over "schema index".


You must return a bool to validate the adding of the frame in the list


@liewegas I removed fdcache.clear, instead, if the meta object doesn't exist during repair I call remove for it, which clears the fdcache for that object, subsequent touch creates it.


you killed Kenny


It is just a typo. I does not affect to the functionality, but I will correct it.


> Personally, I disagree. update is just too generic of a name. Almost any project could possibly have a custom class with a method called update.

Probably you're right. No need for further changes.


The class Frame is not entirely exposed. Setter are missing.


Ah of course, I forgot about this. I had to fix this by altering fastbuild. It turns out DLL nodes using lib.exe falsely report as being EXE_NODES. I'll flag it as a bug and add my workaround fix when I get home.

I can't share the version I'm working with at the moment since I've integrated too much company stuff, but it's a pretty simple change. In Fastbuild go to FunctionExecutable.cpp and in FunctionExecutable::DependOnNode around line 290, add the following block:
```
// an external executable ?
if (node->GetType() == Node::EXE_NODE)
{
	// depend on node - will use exe output at build time
	nodes.Append(Dependency(node));
	return true;
}
```


I think there is an error in that change.

MsUserSelect (and this only applies to microsoft prefix) should not be capitalized

msUserSelect is correct I believe.


should be msUserSelect


Go go Oat oat!


I think the ModelHandler & can be set to const here


Got your point and agree on this one.
But those helpers have been kept out of Dmak object for a good reason to keep them private and not accessible by client.

Log the point in a dedicated issue, and we need to work out a new pattern... may pass a reference to Dmak as parameter, I don't know yet.


Could you please check what will happen in your fork with this issue:
https://github.com/swagger-api/swagger-core/issues/1648

The problem is when in class annotated with @BeanParam we have methods "getXXX" which are not annotated as @XxxParam


@liewegas I also added attempt to repair PG meta on restart if it doesn't exist. Otherwise OSD wouldn't start in the case of meta corruption.


I think default constructor can be dangerous.


Docstrings would be more than welcome here...


This should be a `gpio_t` right?


One more thing. I think the new faded background used on test results is hiding the red background that is used on the word “Fail”. For instance, on the following screenshot, the word Fail used to have the same bacground as the line “2 failures” above:
![screenshot-2016-02-05--063429](https://cloud.githubusercontent.com/assets/453029/12841394/5da96f34-cbe3-11e5-8c92-37ec81aaec8c.png)


Has to be rename


Thanks, will do the edits and make a pull request.


and the AF is of type `gpio_af_t`


We generate this automatic, so the documentation should be in the javadoc, of the setter method. So move it there. Then compile the code and this file is auto updated.


Must the object be closed if you do not call the getObjectContent method? If so please add a javadoc comment that this is in purpose.


I'm going to add an additional module with PCRE2 at some point. I'd want to pass that by kaniini first anyway. Now if I had to guess, any kind of performance issue requiring a rewrite probably deals with complex expressions and cases that won't be bothersome until then. LMK if you think there's a pressing reason to switch immediately though...


Also build the code with checkstyle and fix any errors it report, see here
http://camel.apache.org/building


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


Also Included the bbcode fix from Wannabo !


Done deal !, Thank you
https://github.com/php-fusion/PHP-Fusion/commit/1966cb9dc95f3086e655e5fe9abe6d428639e8fd


@ReneHerthel did you try to find out if the toggling is needed? IIRC this was copypasta from a previous driver which at this "trick" as mentioned in it's errata sheet. MUST NOT apply to the stm32l1 MCU


Could you please file this as a regular issue? I've had one other guy reporting that favorites doesn't work for him, but haven't had time to look into it yet.


Where is `$LOG` defined?


Phải add vào branch `gh-pages` cơ ông ah.


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


so this commit incorporates also an enhancement/fix to https://github.com/windup/windup/pull/831, nice.


Velocity calls equals in practice. That's why it's working for strings among for example.


What do you think about something like this, with guards?

```
private String scoreForTie() {
  if( player1.getScore() > 2 ) {
     return "Deuce";
  }
  return nameFor(player1.getScore()) + "-All";
}
```


:fireworks:


Every Electrum component requires that its parent links it to the Electrum context.

Here, linking adds the `state` and `theme` properties so that they are available in the `<Button>` component. The common Electrum component base class requires both to be present. Without the `state`, it would not be possible to implement `onClick()` and without the `theme`, Electrum cannot compute the `styles`.


I2C needs open drain outputs. Please check #4750


Commit message: contianer -> container


@malirod, in all processor such a big magic inside instruction pipelining, that you don't need really bother about at - it predicts the most popular lines by a lot of own static and dynamic statistics.
In this case -code readability is more important.
And I depend on your choice.


Use https://www.w3.org/TR/2013/REC-page-visibility-20131029/, since the current link always points to the latest standard.


so, i search for the root of the problem and i found if the data type is object, and the database is not nullable field, and it contains empty string then the json.parse throws an exception. it should handle these cases in my opinion.


Maybe we could also move to a plugin instead to the frontend code.
This way the user has the full power of node and can load files from the disk


EditText와 Button을 제외한 객체들을 추가


Not so much no. Velocity actually calls equals, you can even compare a String to a Locale because Velocity automatically call toString() in that case. If a method takes a String the Locale will be automatically converted.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14496](https://teamcity.mesosphere.io/viewLog.html?buildId=125577&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 222 Build time: 00:10:03
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@romw
> Client just reports what it knows.  The MD5 hash was originally added for WCG and they haven’t reported any issues with it on their end.

Which version introduced this or as of which version can we expect this to work? This will allow us to decide whether we'll enforce that check in the future...


Hello, I tried this few weeks ago and I experimented some issues in some cases but I can't remember when exactly :(


Or you could check if p != null then do the split.
Or maybe the urlParam function could return an empty string like the urlHash function, couldn't it? Then split would return an array containing an empty string.


The else might mean that languageValue is null.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.discussion/26//violations
```
plone/app/discussion/contentrules.py:20:5: E731 do not assign a lambda expression, use a def
plone/app/discussion/browser/comments.py:132:1: C901 'CommentForm.handleComment' is too complex (17)
plone/app/discussion/browser/conversation.py:29:1: C901 'ConversationView._enabled_for_archetypes' is too complex (14)
plone/app/discussion/browser/conversation.py:135:12: P002 found "hasattr", consider replacing it
plone/app/discussion/browser/migration.py:34:1: C901 'View.__call__' is too complex (32)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


params.require(:event).permit(:title, :place, :content, :start_date, :end_date).merge(user_id: current_user.id)
は
 params.require(:event).permit(:title, :place, :content, :start_date, :end_date, :user_id)
じゃダメなの？


我把本地仓库的内容推送到github时，出现这个问题：
Administrator@cp MINGW32 ~ (master)
$ git push -u origin master
error: src refspec master does not match any.
error: failed to push some refs to 'git@github.com:carol-006/learngit.git'
是怎么回事？我是初学者，希望能帮忙解答一下，谢谢！


I'd say since it's reported as so widely supported (http://caniuse.com/#feat=pagevisibility) it could go directly into Document.  You could make `Document extends PageVisibilityDocument`, and still keep the spec in this file.  Best of both worlds :)

The spec seems to state `Document` rather than `HTMLDocument` gains these properties though (e.g. `SVGDocument` should in theory support this too).


достаточно @book.quotes - вернет цитаты этой книги


Maybe will increase time.
Imagine you have resources, 80s to have a super Krogoth ': X

Need have same for Bantha & Super Bantha.


Glad I could help!


Maybe we could even remove this object (apis with full support probably shouldn't need to be imported).


Зачем?


Le 2016-02-05 06:44, Loic Dachary a écrit :

> I don't think an entry with just one email is valid. Is it ?
>
> --
> Reply to this email directly or view it on GitHub [1].

Seems like it is :

man git-check-mailmap

...

In the simple form, each line in the file consists of the canonical real
name of an author, whitespace,
and an email address used in the commit (enclosed by < and >) to map to
the name. For example:

Proper Name <commit@email.xx>

....

It is valid for name normalisation, that's why I used it in the first
place, but i realized It's also valid as a catch-all for the name OR the
email, so a single entry like this exempt us to add a line in .mailmap
every time an author make a typo.

Which is great, I think.

But, in the first patch of the pull request, I wrongly oversimplified, I
thought  Proper Name <commit@email.xx> <other@ email.yy> would catch
all, and it's NOT the case. So I need to put some single Proper Name
<commit@email.xx> to complete.

I'll do this this evening, and send a mail to all maintainers to explain
this move.

Cheers,

Links:
------
[1]
https://github.com/objoo/ceph/commit/0ba39fb03c5b21b9f8c78fab2a1d602802483e33#commitcomment-15905649


Where is this variable needed?


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


@pestOO, c++ is not java. How it can predict or collect statistics? Java machine, indeed, performs runtime optimizations. But with c++, as far as I know, there is no such thing.

Not long ago I've seen some session from the cppcon, where the guy from google made such advice. I'l try to find   the proof link.


all не нужно


@Trikke76 I suspect it only matters if you are using `postgresql_unix_socket_directories` as a list or not. If you set it to a single value string, it will work fine either way. The problem is when a user relies on the old treatment and treats `postgresql_unix_socket_directories` as a list, and then the change breaks that. Obviously one could change the calling code to use the new behaviour, but this seems an unnecessary change.


если у тебя есть id цитаты можно сразу @quote = Quote.find(params[:id])


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 146](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=752&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:38:10


@h13ronim :point_up:


How would it work for relations?
Duplicate with new id? Adapt fields accordingly?


Right, how about
```
.lower() == 'true'
```
then, to make the intent more obvious?


Is `latestC` correct or should it be `latest`?


Maybe your plugins aren't load after Xcode update


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


はじめからのこり時間を60にしておこう


宣言のときにnokorijikanを60にして、この行を消す


Hey @traversaro

I looked at the thrift stuff and I think one day I will sit down and try to learn it properly I just am too lazy right now. It would definitely be useful here though.


画面が現れたときにTextViewに最初の問題を表示するようにしよう


oh ok


LGTM


It doesn't mean that. It means that the languageValue object is not of type Locale. It can be of type String for ex.

If the user passes null then I believe Velocity will pass it as an Object and thus it will reach LocaleUtils.toLocale() which knows how to handle null.

That said you're right, I broke it with languageValue.toString() which will throw an NPE. Fixing


this is just wrong.


book = Book.find(params[:book_id])
@quote = Quote.new(text: params[:quote][:text])
@quote.user = current_user if current_user
@quote.book = book if book


I can't remember exactly but it was fixed in 335db159.

2016-01-23 6:22 GMT+03:00 Konstantin Isakov <notifications@github.com>:

> Why was that commented out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zbackup/zbackup/commit/94adee16c77ab05c6f40009b885f5aa2d5fed285#commitcomment-15627726>
> .
>

--
Kind regards,
Vladimir.


зачем?


> That said you're right, I broke it with languageValue.toString() which will throw an NPE. Fixing

That was my point...


if文をつかって、10問に達するまでは問題をTextViewに表示して、indexを足す
elseのとき、タイマーを止め、次の画面に移る
というように@IBActionのコードを書き換えよう


This fails to compile on Linux:
```
unit1604.c: In function ‘unit_setup’:
unit1604.c:35:10: error: ‘SANITIZE_ERR_OK’ undeclared (first use in this function)
   return SANITIZE_ERR_OK;
          ^
unit1604.c:35:10: note: each undeclared identifier is reported only once for each function it appears in
unit1604.c:36:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
```


:+1: , just wanted to make sure you were aware of it.


will break OE


There is no model `ConflictLoop` in this commit. Is the model missing or is `ConflictLoop` just added by accident to HybTest/package.order?


As the file `/usr/lib/supportconfig/resources/rpmlist_$FRAMEWORK.rc` hasn't been created yet, this will always fail.


TeamCity pull requests :: netty [Build 1218](https://garage.netty.io/teamcity/viewLog.html?buildId=5451&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4927, ignored: 23 Build time: 00:36:10


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


For 3 properties, I'd just put them right inside `Document` if they're widely supported.


> so a single entry like this exempt us to add a line in .mailmap every time an author make a typo.

It is indeed a simplification, nice catch :-)


What to do with image upload in SCC? Do we still let WP generate thumnails and a few different sizes, so the upcoming WP responsive images can use those in WP mode?


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


.update_attributes
@quote = Quote.find(params[:id])
@quote.update_attributes(quote_params)

лучше создать отдельный метод, который возвращает только нужные параметры:
def quote_params
  params.require(:quote).permit(:text)
end
почитай про strong parameters


[Reflection.Assembly]::LoadWithPartialName('System.Speech') | Out-Null
[string[]]$phrases =@("Talk to yourself at  least  once in a Day.. Otherwise you may miss a meeting with an EXCELLENT person in this World",
"You have to grow from the inside out. None can teach you, none can make you spiritual. There is no other teacher but your own soul. ",
"Relationships are more important than life , but it is important for those relationships to have life in them…",
"The only true wisdom is knowing that you know nothing.",
"Start every day off with a smile and get it over with.",
"Failure defeats losers, failure inspires winners",
"Poor people have big TV. Rich people have big library.",
"Start every day off with a smile and get it over with. ",
"The only way to do great work is to love what you do",
"Either write something worth reading or do something worth writing",
"Success is going from failure to failure without losing your enthusiasm",
"Remember, those are not all your friends who laugh with you.",
"Build your own dreams, or someone else will hire you to build theirs. ",
"To live without an aim is to wander in a wilderness",
"We may encounter many defeats but we must not be defeated.",
"Life seems but a quick succession of busy nothings..",
"The best defense against propaganda: more propaganda..",
"Just know, when you truly want success, you’ll never give up on it. No matter how bad the situation may get..",
"Your imagination is your preview of life’s coming attractions..",
"Whoever wants to reach a distant goal must take small steps.",
"Before anything else, preparation is the key to success.",
"I am tomorrow, or some future day, what I establish today. I am today what I established yesterday or some previous day",
"Nothing important was ever achieved without someone taking a chance",
"That which is man-made can be unmade.",
"Love is energy of life.",
"The breath of an aristocrat is the death rattle of freedom.",
"To say that something is designed means it has intentions that go beyond its function. Otherwise it’s just planning.",
"If you dont have a competitive advantage, dont compete.",
"Failure will never overtake me if my determination to succeed is strong enough.",
"All of us do not have equal talent. But , all of us have an equal opportunity to develop our talents.",
"Thinking should become your capital asset, no matter whatever ups and downs you come across in your life.",
"It costs nothing to dream, but everything if you don\'t.",
"Man needs difficulties in life because they are necessary to enjoy the success")
$phrase = Get-Random $phrases
$object = New-Object System.Speech.Synthesis.SpeechSynthesizer
$object.SelectVoiceByHints('FeMale')
$object.Speak($phrase)


зачем?)


Are you sure % margin works well in mobile devices ?
I am sure there will be a better approach to this.


а если не удалится?)


см. комменты выше.


don't remove the js file, comment out the code in main.js


> here, fetch returns a promise: `Promise {[[PromiseStatus]]: "pending"`

This is expected; there's a gap in time between the moment a Promise object is created and the moment the then callback is processed and passed the result.

Basically, when using your `registerUserURL` function elsewhere in the code, you'll have to leverage the promise object you've returned and calling then on it to retrieve the asynchronously processed result:

```
var result;
var promise = registerUserURL(a, b, c, d)
  .then(data => {
    result = data;
    console.log(result) // prints the expected results, because we're later in time
  });

console.log(promise) // this is executing synchronously, prints Promise { <state>: "pending" }
console.log(result) // this is executing synchronously, prints "undefined"
```

This is how promises work; they're an abstraction over time and provide an API (`.then`) to allow defining callbacks for when things **eventually** happen. Returning a promise objet won't ever return its future result, but an abstraction of it you need to unwrap withing a callback, always.

Good links:

- [An introduction to the Promise API](http://www.html5rocks.com/en/tutorials/es6/promises/)
- [We have a problem with promises](http://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html)

Hope it helped.


В настройках поменяй rss -> live


можно эту проверку делать так:
if @quote.user == current_user


I recomend you to split this line in to 2/3 lines. It is better to prepare all complex conditions before checking statement.


My bad


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


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

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

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

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

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

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

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

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

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

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

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

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


@mwijngaard i think you probably meant to mention some other Tijs :)


Same problem, why isn't this released yet ?


Should be `this.size = tmp.intValue();`


1) loadTerrains() is just movement cost its OK.


юзай пробелы вместо табов в редакторе


Thanks Thomas. I'm glad you caught it as it would have been regression.


@zzz1999 Yes, but I won't blame you. :)


Proporrei di accorciare ulteriormente, visto che la stringa è uno slogan. Inoltre secondo me andrebbe messo in chiaro che queste azioni, organizza viaggio, pianifica acquisti ecc. sono fatte collettivamente, perché uno può organizzarsi un viaggio o pianificare gli acquisti pure per conto suo, e in questo caso non gli serve Hello.

Naviga sul Web in compagnia con Firefox Hello. Organizza un viaggio tra amici, pianifica un acquisto in comune e molto altro ancora, tutto in tempo reale.


можно даже так  new_book_quote_path(@book)


@syl20bnr I don't like `m "`.  The quotation mark needs a modifier to type which is more cumbersome than a "flat" binding.

I'd like to grab `SPC m x '`.  Currently it's taken by rubytools' string command, but the one of Emacs is somewhat better.   Why not actually replace the ruby-tools commands with the built-in function entirely on Emacs 25?

I would offer to provide a corresponding PR if that's an option…


см. выше
а еще можно вот так [@book, @quote]


参考資料：
http://www.tohoho-web.com/www.htm


Got it. Thanks.

But it's OK. At least one can still make it deterministic by running a single thread, so atom creation order is determined. That's already a useful. When debugging, you'd typically create a unit test reproducing that bug, usually you only need a single thread for that.


Much better. We could also make use of [is_wp_error](https://codex.wordpress.org/Function_Reference/is_wp_error).


это можно вынести в отдельный хелпер и передавать ему объект. метод будет возвращать эрроры объекта.


The CI build is failed after our commit, pls refer https://travis-ci.org/openmrs/openmrs-core/builds/107187130, but when we run the test at local (mvn clean install --batch-mode) it passes.


Spurious `i` at the end. Could you also center the block?


so you are releasing 0.0.29 today?


I think this is quite a bold assumption on IDs always being numbers here, don't you think? It's completely legit to have UUIDs for example. And the identifiers should always be strings as per the spec, too:

> http://jsonapi.org/format/#document-resource-object-identification
> <img width="743" alt="screen shot 2016-02-05 at 11 16 52" src="https://cloud.githubusercontent.com/assets/482561/12842229/064061f0-cbfa-11e5-9f17-85f0bb97966c.png">

Also, it's not a good idea to create functions inside a loop. Better to hoist this function out from the entire class as a helper instead of hiding it inside a `[].forEach` call.


Condividi il sito web che stai visitando in un istante.

Forse si stacca un po' dall'originale, ma qui secondo me "velocemente" sembra una questione della velocità di connessione, mentre ciò che vogliamo sottolineare è la rapidità/facilità di condivisione.


зачем ? посмотри в rake routes, они уже должны у тебя и так


Just trying to figure out what is going on, that is my build script that has updated that, will revert it in a moment


@tkonolige you should format this code by aligning the dots, makes it easier to read (or run rustfmt).


Actually, probably quite important to use `is_wp_error` because we're using `get_error_message` on line 255. Not sure there's a 3rd possibility for `$response` besides an array or error object, but may as well use that check.


I'm still worried about the commit id we use here, because it shows false information about which actual k8s we're using. See [this comment](https://github.com/openshift/origin/pull/6947#issuecomment-179100992) which still applies.


Then it is quite fitting for the legendary gnome accuracy. :)


Oh, you cold leave this in, just move it to lib/tasks. It's not dependant on capistrano.


Oh, you could leave this in, just move it to lib/tasks. It's not dependant on capistrano.


Nice :+1:


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


Done in ae37eaf


Secondo me "iniziare a navigare" è un po' obsoleto. (Vedi anche le stringhe successive)

Scarica l’ultima versione di Firefox per navigare sul Web in compagnia.
oppure addirittura
Naviga sul Web in compagnia con l'ultima versione di Firefox.
Naviga sul Web in compagnia, scarica l'ultima versione di Firefox.


Please note you need to check the user login status and also capability if any.

is_user_logged_in()
current_user_can()


How about trying to push this upstream with explanation why it's needed?


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Install.md`, line 122.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Install.md
+++ b/docs/Users/Install.md
@@ -119,7 +119,7 @@
 ## Generating Documentation

 coala documentation can be generated by fetching the test and documentation
-requirements. This can be achieved by
+requirements. This can be achieved by

 ```
 pip3 install -r test-requirements.txt
```


**Line contains following spacing inconsistencies:
- No newline at EOF.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Install.md`, line 132.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Install.md
+++ b/docs/Users/Install.md
@@ -129,4 +129,4 @@
 ```
 mkdocs build # For building the static files
 mkdocs serve # For serving the files from localhost
-```+```
```


Yep :)


hmm, it may actually be possible to monkeypatch the socket object..., looks like we just need to add two methods: `_reuse()` and `_drop()`, the latter of which will automatically close the socket if the reference counter increased by `_reuse()` falls below 0


das funktioniert doch super!


@tarcisiosilva

Please note you need to check the user login status and also capability is needed because if we dont any one can open the file and do the import

say like current_user_can('manage_options') will allow only admins to use this

is_user_logged_in()
current_user_can()


To be honest I think it's actually better to get them to just comment in the main room and say they should check back in a few hours just to ensure they have a better shot and the conversation is public.


I removed this in 130c5a4


:dancers: :dancers:


Да, я с ними ознакомился прежде, чем приступать, но потом подумал, что требуется сделать без использования гемов


The array case has to be handled correctly in order to call the `\Api\EnumType\ApiAdGroupTypesEnum::valueIsValid` method for each value of the array as in
```php
array_walk($ids, function($item) {
    if (is_int($item)) {
        throw new \InvalidArgumentException(sprintf('The Ids property can only contain items of long, "%s" given', is_object($item) ? get_class($item) : gettype($item)), __LINE__);
    }
});
```
The best solution would be to throw only one exception with all the invalid values instead of throwning an exception at the first invalid value otherwise it will be painful to launch multiple times the script to fix the invalid values one after one.


Согласен, выставляется


The array case has to be handled correctly in order to call the `\Api\EnumType\ApiStatusSelectionEnum::valueIsValid` method for each value of the array as in
```php
array_walk($ids, function($item) {
    if (is_int($item)) {
        throw new \InvalidArgumentException(sprintf('The Ids property can only contain items of long, "%s" given', is_object($item) ? get_class($item) : gettype($item)), __LINE__);
    }
});
```
The best solution would be to throw only one exception with all the invalid values instead of throwning an exception at the first invalid value otherwise it will be painful to launch multiple times the script to fix the invalid values one after one.


Ah, okay, could we discuss this on [gitter](https://gitter.im/oeed/CraftOS-Standards) please?


fjern space


Why only support laravel 5.2? 5.1 has LTS support


i have done


please do not write more than one tag per line

say like
<li .... >
   <a ..... >...</a>
</li>

this will be easily readable and will be easy to work on changes in future


Nice, it will greatly simplify our mailmap then


Errata: "Quick Sort algorithms is based on divide and conquer strategy"


errata: "two smaller lists"


LGTM


oui je vois


Thank you!!!


Più che solo "sciogliere gli ultimi dubbi", io direi "organizza una vacanza in compagnia fin nei minimi dettagli" (cioè puoi organizzare la vacanza dall'inizio alla fine con Hello, non usarlo solo per decidere le ultime cose).


the artisan start event changed in 5.2, you can still use 1.0.* for laravel 5.1, 5.2 laravel and above will use 1.1.*


@tarcisiosilva  you not supposed to edit the css files directly because this css file is compiled from theme.less
https://github.com/scconsulting/john-deere/blob/3b0fa16150985d75735072a37ed2cfc9f216585c/themes/john-deere/scit/theme/assets/css/theme.less

Less is like programming language for css - http://lesscss.org/

netbeans can be configured to use less compiler and on save of the theme.less file that file will be compiled into css files


ок, посмотрю, раньше не пользовался


@Kozoriz see also https://github.com/LuxoftSDL/sdl_core/commit/726f0b6f61fffda558a18cd97975064367c78af4


@philwebb is this expected? You did not use again application instance again.


You're right, but as it's not the same meaning I prefered a separate test.
I don't think it has a significative impact.


Already fixed in another commit.


t'as trouvé mon com compte sur github ?

Le 05/02/2016 10:32, Kanté Mahamadou a écrit :
>
> oui je vois
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nberrehouma/hello-word/commit/8ec5e49205ea3dc7ca65ea126ff5c35fee330056#commitcomment-15909069>.
>


Hm... this line breaks `%PATH%` parsing on Windows (cmd.exe) as it removes backslashes which are not normal escape sequences. For example `\M` gets replaced by `M`:

    shlex.split('C:\\Miniconda2') => ['C:Miniconda2']

which renders the full output of `set PATH` useless.


please dont comment of the previous code and write the new line
just edit the actual code . github gives us back up when needed

also commenting out and writting a new line doesnt show the change that has been made in the code


Test does'nt need model. But i let it (empty) to keep a common logic in tests writing


Ost


je veux que tu modifie le readme file de mon projet " Master-project "
qui se trouve sur  mon repository

Le 05/02/2016 10:32, Kanté Mahamadou a écrit :
>
> oui je vois
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nberrehouma/hello-word/commit/8ec5e49205ea3dc7ca65ea126ff5c35fee330056#commitcomment-15909069>.
>


I'd mark this commit as DANGER, is there post-rebase work planned for it?


Haha, wow i didn't even know Github parsed commit messages like that. Sorry
man!

On Fri, Feb 5, 2016 at 10:13 AM, Tijs Teulings <notifications@github.com>
wrote:

> @mwijngaard <https://github.com/mwijngaard> i think you probably meant to
> mention some other Tijs :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/software-engineering-amsterdam/multi-ql/commit/0398eb60f87e143da2097bf7e76038161190ee4a#commitcomment-15908695>
> .
>


I checked the [codex](https://codex.wordpress.org/Function_Reference/wp_remote_get#Return_Values) first and according to them it's either `WP_Error` or `array`.


Forse, per dare maggior rilievo al fatto che è una funzionalità aperta:

Puoi invitare tutti i tuoi amici, anche quelli che non usano Firefox


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1299](http://teamcity.monogame.net/viewLog.html?buildId=14351&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:32:02


domain need not be mentioned explicitly
relative urls are better mean  /lostpassword/  is good because we change domains from dev to production, etc
so during those times these absolute domains wont work


Pročpak? :-)

Já bych v případě že tam addr není kontatkoval nominatim, když je tak jí vypsat ve spešl formátu, což?


 Oui je l'ai trouvé, c'est ma connexion qui est un peu lente je suis la déçu

    Le Vendredi 5 février 2016 9h34, Nabil BERREHOUMA <notifications@github.com> a écrit :

 t'as trouvé mon com compte sur github ?

Le 05/02/2016 10:32, Kanté Mahamadou a écrit :
>
> oui je vois
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nberrehouma/hello-word/commit/8ec5e49205ea3dc7ca65ea126ff5c35fee330056#commitcomment-15909069>.
>

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


@tarcisiosilva  never ever edit a plugin code directly
because that will make plugin unable to upgrade it in future for new features.

there will be always a way to make it work in different way that edit it directly


@bkrausz Sure. Added here.
wonder is there is a way to validate the id as ISO country code


Fixes #6495 as well.


The code above here does not belong in the view template. It should be moved to the corresponding component.


Does the `wip` option not work for you? With --against master, it should be able to compare branches easily.

(I remember why I removed it: because nox maintains another nixpkgs repository, and the `revs` command didn't work well when using revisions that are local to the repositoru you're working on)


Thanks @living180, that sounds like a great idea to be.


Funniest commit message I've seen for a while! ;) #mademymorning


maintenant, tu doit faire un fork sur le mon projet "Master-project" ,
puis faire la modification du fichier "read-me", puis new pull request ...

Le 05/02/2016 10:40, Kanté Mahamadou a écrit :
>  Oui je l'ai trouvé, c'est ma connexion qui est un peu lente je suis
> la déçu
>
> Le Vendredi 5 février 2016 9h34, Nabil BERREHOUMA
> <notifications@github.com> a écrit :
>
>
> t'as trouvé mon com compte sur github ?
>
>
> Le 05/02/2016 10:32, Kanté Mahamadou a écrit :
> >
> > oui je vois
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/nberrehouma/hello-word/commit/8ec5e49205ea3dc7ca65ea126ff5c35fee330056#commitcomment-15909069>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nberrehouma/hello-word/commit/8ec5e49205ea3dc7ca65ea126ff5c35fee330056#commitcomment-15909228>.
>


Tohle by šlo nahradit za:

```js
general.sort(function(a,b){ return a.k.localeCompare(b.k); });
```

Ale jinak poslední commity pěkná práce - zejména jejich granularita - radost pročítat :-)


Add link github and Linked In


@msroot the entire `retrieve` method isn't needed - we define a correct one for this case on `APIResource`. You don't need to override it.

Re: validation - we tend to strongly prefer server-side validation for things like that, as it gives us much more flexibility in the future. The server will return an error on invalid codes.


say for example the  line 471 is a filter

$redirect_to = apply_filters( 'login_redirect', $redirect_to, isset( $_REQUEST['redirect_to'] ) ? $_REQUEST['redirect_to'] : '', $_REQUEST['redirect_to'] );

we can use it like in our theme class file

add_filter('login_redirect', array($this, alter_redirect_url), 10, 3)

public function alter_redirect($redirect_to, $requested_redirect, $user){
    return '/home/';
}


This looks great, it was a bugging me that I couldn't spare any time to do this for a while. I'll test it next week and merge. Thanks man!


I've used underscores to enable the [variable highlighting feature](https://twitter.com/bterlson/status/654500731857190914), but you're probably right that either all abstract operations should use underscores for arguments or alternatively none of them.


don't know what that's about!


Yes, but the processing is done immediately after a response arrives while waiting for other responses. I don't think that threading could give us more speed, since it has context switch overheads, interpreter locks and runs on a single cpu core.


注释写错了，现在的code思路不一样


112796613_DesicDehum (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2060 of 2060 tests passed, 1 test warnings)

 * 7 tests had: AUD diffs.
 * 1 test had: Table small diffs.

![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)


meh, i thought we've fixed google redirects, thanks


ok j'essaye


This update is causing problems with building VLAD (tried on two difference Macs):

```
TASK: [get current Vlad version] **********************************************
failed: [10.37.129.111 -> 127.0.0.1] => {"changed": false, "cmd": ["cat", "vlad_guts/VERSION.txt"], "delta": "0:00:00.010255", "end": "2016-02-05 09:41:45.963293", "rc": 1, "start": "2016-02-05 09:41:45.953038", "stdout_lines": [], "warnings": []}
stderr: cat: vlad_guts/VERSION.txt: No such file or directory
```

Before, the command was:

command: cat ../VERSION.txt
https://github.com/hashbangcode/vlad/blob/8989a05c9802735f99f263ae2df3309ad7e559fc/vlad_guts/playbooks/site.yml

Does it need to revert to that?


Thank you for the pull request.
You are very nice.

Only a small question. there was a var named "VeryNginxConfig" existed in many lua file, you change it to all lowercase in the filter.lua , but _M.filter_uri function is using the old var name. I think the old var will not be found.

Please revert the var name in filter.la, so i can merge the pull request.


I agree with the intent of this commit (Odoo shouldn't crash on missing xml ids). I'm not sure however that this is will have the desired effect. If you where to delete let's say both groups the POS will be pretty much broken anyway, with or without this patch:

https://github.com/odoo/odoo/blob/9.0/addons/point_of_sale/static/src/js/models.js#L250

And I feel like without this patch at least you'll get a clear traceback. With this patch you'll end up with a POS frontend (/pos/web) with no users (because ```self.config.group_pos_manager_id[0]``` will be false I suppose), that might crash as well by the way, I haven't tested it. But this situation could be perhaps a little bit more difficult to debug, instead of just a simple python exception.


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


:tada:


Love what you did here. Makes good sense to abstract property into a class. :+1:


I edited the file online on github, so there's not much I can do about that.


this will never be reached. or I have not a sample with which it will be reached


can we move this const to Const var? i dont know what means this numbers


@luiges90  ？


c'est okay


ok , je vois votre modification @niagale


Qui c'è un gioco di parole secondo me molto carino: "be on the same page" vuol dire "intendersi, essere sulla stessa lunghezza d'onda". Mi viene in mente un:
"Condividi le tue vedute con un clic, mettiti d'accordo con gli amici in tempo reale"
ma non so quanto scorra bene in italiano :/


I'm wary there is a fair bit of stuff here we do not want or need. We have no use of LESS or SCSS. Shipping this in our future releases is not desirable. Maybe there is scope for some sort of build step that strips out bloat like this.


not sure how that happened i didnt even notice til i saw your comment, i just merged the files from the decompiled factory image into the prop folder


should it be codec_id?


same here


essayons de mettre qqs chose signficative


okay


Typo "It would the force"


please do

```bash
git tag v0.2.0 e93997b61590396cac4bcfbf85b9a9a63e3a990a
git push --tags
```

to fix #44


:hibiscus:


Would this work :
return Settings().then( ... return res.locale || 'en'; ... )

Avoids the catch(reject) ugliness.


Бокс сайзинг у всех элементов по дефолту content-box, его вообще можно не указывать.


lgtm


can we move this codes to const vars, or we can find enum with this codes in impl?


the statement is correct.


@malirod, and you should not collect any statistics, but CPUs instruction pipelining [do it](https://en.wikipedia.org/wiki/Branch_predictor) own way, that's why ordering according the most common case is not really important.


In general this patches works better. with LAV patches msdk-mvc decoder has an issues with jompjomp in software mode.


и дивов не бывает дефолтных маргинов, как и у секшнов и артиклов.


Comparing to https://github.com/SixTrack/SixTrack/commit/41e8e6dd5ad15343a7952943e30f57b4fd06c238, I see that the only real change is adding the TCLX and TCLD. When merging the current master on top of this commit, the TCSP is added in both places -> OK!


@dmustieles _Interrumpido_ or _anulado_ are better translations for _aborted_.


@bkrausz `retrieve` removed


Is the signal count guaranteed to equal `ready_count`? The new implementation may skip over `0` on decrement and then return with `ready_count < 0`.


remove this comment or explain why it's there.


Чтобы всё получилось как на макете - футер должен быть абсолютно спозиционирован, с шириной 100% и прижат к низу body. Возможно ещё придётся ему бордер бокс указать.


fail test here?

(what's this "done" thing?)


You should use `protocol_handler::ServiceType`


Good.


What are you talking about? The operations tests run like s**t off a sh***l for me; it's the integration tests that are slow.


Раз делаешь резиновую вёрстку - указывай эти величины в процентах, как на макете. 6% 20% 20% _8%_ 20% 20% 6%


What are the setTimeouts for?


Removed


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-14: OK (388 of 389 tests passed, 0 test warnings)

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


This causes "Cannot refer to 'this' nor 'super' while explicitly invoking a constructor" compile error in Eclipse.


This is a very good commit, thanks for that. Only the code to carry to the next set when the first is exhausted is somehow less readable, but still, it makes the code much less cleaver and surprising without hindering the performance (I guess). Good job.


Dupe entry


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3047 of 3047 tests passed, 0 test warnings)

![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)


Weird indentation.