I would prefer to have the designated_x/y and the card_type/card_team fields split into separate messages, that are included in SSL_RefereeRemoteControlRequest. That way protobuf can ensure that these fields are only sent pairwise, and probably simplify some other correctness checks.


I would take this out (and insert the previous for-each-service loop-guts here).


Great :)


wow ,  good !
thanks


Iter: 1435, A: 5000, alpha 0.602000, gamma 0.101000
param: PW[1], best: 210.59, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[2], best: 203.31, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[3], best: 198.91, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[4], best: 197.94, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[5], best: 195.52, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[6], best: 187.35, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[7], best: 187.34, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[8], best: 199.58, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865


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


Same comment as above


fdgdfg


After reading [that article you provided](https://wiki.debian.org/ToolChain/DSOLinking) I'm convinced your right. That is the root cause.

Now to find out how to fix it.


Thanks for this! Two small typos:

* (one JS and one JS) should be (one CSS and one JS)
* s/containes/contains


This is problematic.  You're now going to restore one table before the other - the point of 'restore' is atomicity.  I'll give this a think, but please also think about it.


typo: "literalParamete"


Faltan las llaves


one whitespace too much...


Evita tantos .focus()


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1208.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67849&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 633, ignored: 4) Build time: 00:07:41
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

```


s


Typo


What's the rationale for reopening the file here a second time?

If we really want a second fd, wouldn't fcntl(fd, FD_DUP_CLOEXEC, 3) be the better choice?

But most importantly: wouldn't it be better to simply return -1 in *ret_data_fd if we don't distinguish compressed and non-compressed cases? Sounds much more obvious to me to return an invalidated fd, rather a duplicated one here... Of course, when using the return fd, one would have to be smart enough to handle the negative fd, and then sue the other one...


有关关键词, 虽然说比较虚, 但是宗旨的确是这样, 这个可能稀缺需要经验. 其实我不一定最开始就用3个或以上的关键词搜索, 因为有可能我都不知道我想搜的关键词对不对, 我会从最大的概念, 我知道的搜起, 然后有的可以直接auto complete了...有的 从初步搜索结果就可以知道我不清楚的那个关键字应该是怎么说....(毕竟有时是英文反应不过来)
以上~仅供参考


Looks good otherwise!


whitespace missing...


nitpick: these two lines could be combined into:

```c
fd = fd_compressed = 1;
```


Use sprout cm-12.1 libdpframework


var is cool too.


Maybe we need to change everything to return a `map[iptables.Table]map[utiliptables.Chain]string`, edit that, and then restore the whole thing.  That's probably a whole commit on its own, just laying the groundwork for REJECT.


@dogmaphobic I would suggest "3D DGPS Lock"


@dogmaphobic I would suggest "RTK GPS Lock"


Wait, what...Suspenders? What happened to Preseason?


Grr... reverted my fix for Drush alias root :|


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


char32_t is not a macro is it? I am pretty sure it's a typedef, which means you cannot test for it with #ifdef. Please instead add a configure check with `AC_CHECK_DECLS`, and then use `if HAVE_DECL_CHAR32_T`...

(see how this is done for the `key_serial_t` type, it should be very similar for char32_t, you can extend the same `AC_CHECK_DECLS` stanza in configure.ac, and just add uchar.h to the list of includes there, and the type to the arguments list...)


111411352_ATDeprecateFields (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2053 of 2054 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.95%25-yellow.svg)


Stop reviewing this commit, I uncomented code that was commented out in previous commit. See https://github.com/aspnet/Performance/compare/fcb04a9fccd594bba38ecc39ef24e6919ba8bd3e?expand=1


:smile:


Stop kissing your boss' ass!


this client.isConnected() check is needed. because of, this example don't check publish/subscribe return value in setup function. Of course some developer don't need this check but several implements will be added in this loop by developer, so this check leave clearly(just a sample).


saw approval from javajolt
at https://github.com/basho/riak-erlang-client/commit/0db46d5452a01ab9fb9523b5ec495251b27a6ef9


merging basho/riak-erlang-client/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 0db46d54 into borshop-integration-259-perf/ttb_newmsg_ts1.0_release_candidate_v2


basho/riak-erlang-client/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 0db46d54 merged ok, testing candidate = b570ef04


This is weird... the function has "int32_t" as return type, i.e. is signed, but you return a "char32_t" here, i.e. an unsigned entity... Sounds wrong...


Got it. Yeah, that makes more sense.


Otherwise looks OK to me.


We should not use android.permission.INTERNET if not really needed.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3092


One more thing.... now `encodeURI('file://' + localFileUri);` needs to be called on `serverFileUri` if source maps are being used.


extra \?


te sobra el else


This is optional and a separate kernel image is provided to boot a over-clocked kernel.


¿Qué tabulaciones usas?


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


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


TeamCity pull requests :: netty-tcnative [Build 34](https://garage.netty.io/teamcity/viewLog.html?buildId=4689&buildTypeId=pulls_netty_tcnative&guest=1) is now running


the ter


@polyrabbit moved out to

https://github.com/wechatpy/wechatpy.async.tornado

https://github.com/wechatpy/wechatpy.async.asyncio


Why send directly? Even if you want to make sure they are sent, I would use sendCommand and the flushPending.


TeamCity pull requests :: netty-tcnative [Build 34](https://garage.netty.io/teamcity/viewLog.html?buildId=4689&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:49


Congrats!


Validation API chocks on your metadata, where it's expecting it to be a list and not a string.

Obviously, we should also make validation api tolerant to malformed validations. I'll take care of this part.


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


看来是要用我司的 NLP 了😄


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


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


You are a good catch.

If you know what I mean.

On Mon, Jan 11, 2016 at 5:49 PM, Devan <notifications@github.com> wrote:

> No worrriez. 🎣 good catch 🎣
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KarrieCheng/Sign2Line/commit/f7e59e1131b38cfdfaa406edc4180d470f5af5ce#commitcomment-15384592>
> .
>


nope


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1209.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67851&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:21:18


ES6?  Are we transpiling, or only supporting new browsers, like Microsoft Edge?


actually, not sure what I meant to do here.  This sentence is trying to distinguish the literal username "`root`" from the expectation of what "root" means.  Specifically, the *..go hand in hand with being root, ..* is referring to the concept of the "superuser", which we all refer to as root given that is what the system actually calls uid, gid == 0.  Maybe I don't need to distinguish that at all anyway, but what about "go hand in hand with being the superuser"?  Or I can leave it as literal root as I assume the meaning comes across either way?


all tests pass:
success: http://buildbot.bos1/builders/erlang-client-test/builds/268


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


Referencing superuser seems fine to me?


fast-forwarding end-to-end/timeseries to borshop-integration-1323-perf/ttb_newmsg_ts1.0_release_candidate_v2 = 448e4c0f


기존 로직이 없는 경우에만 FSM을 새로 넣게 되어있는데 코드를 contains 체크 후 set으로 바꾼 다른 이유가 있나요?


it appears to be one or the other in Xcode, you can't separate simulator and arm64 build settings.


@Znuff so instead of being calm and collected about the change, you find the commit that causes it, and make snarky comment. Noice.


매번 로직에 진입할 때 마다 TSM 생성자가 불리고 있어서요.


Working now
- closed


please push tag!!


/close


Thanks, it work fine!


Maybe we should use shields.io for the badge instead?
You know, to make the design of the badges consistent.


I didn't see one for StyleCI. I emailed them to ask if they'd change the badge style.


`dd`-> `add`


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-MacOS-10.9-clang: OK (2044 of 2045 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.95%25-yellow.svg)


`manage the state of a cron expression` -> `manage the state of a cron expression,` (just add comma)

also `it's source of truth` -> `its source of truth`


I went ahead and added them into GLFW.jl:
https://github.com/JuliaGL/GLFW.jl/commit/65ae699fc45f11d981b464d131acfdefdf7e427c


fast-forwarding end-to-end/timeseries to borshop-integration-259-perf/ttb_newmsg_ts1.0_release_candidate_v2 = b570ef04


Yes in most case, you don't have to consider about the performance issue. At least it is faster than Java according to [leetcode](https://leetcode.com/)'s statistics when solving algorithm problems. If you are really care about the the speed, Python let you use module or package written by C like widely used mathematic package [numpy](http://www.numpy.org/).


I finally found some time to do this. This should fix TCP completely: https://github.com/andiwand/nagios-icinga-checks/commit/1dc22f6ee01495d0db0932549929fe7811dd8fd2


初始化


there are supposed to be comments here, but I can't see them...?


I don't see this (or the getter) being used anywhere in this PR. Is this something that slipped through the cracks with all the re-work for generalizing how type arguments are modeled in the last PR?


yeah, in docs/ and examples/ many URLs starting with an IP address, which wouldn't be a valid URL.


return entity != null ? EntityUtils.toString(entity) : null;
在EntityUtils.toString(entity) 上增加了Consts.UTF_8编码，我用的linux操作系统，如果没有编码我返回时都是乱码～


```javascript
function fancyAlert(arg) {
  if(arg) {
    $.facebox({div:'#foo'})
  }
}
```


@ezeeetm  OK, I agree your point, keep the doc clean and simple :)   The last sentence will helpful if I read carefully.


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (954 of 954 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.73%25-green.svg)


at this time, I performence this command, then my computer has recovery sounds. god.


It'd be better to inherit this from `TimeOutSkipTest`. Then the body would only be:

```
    _skip_reason = "Test skipped due a job replay filter!"
```

... well it'd be actually better to replace the `TimeOutSkipTest` to `SkipTest` with generic `Test skipped` message and then have all these Skip tests inherited from it, but that can be done in cleanup refactor. Anyway you should reuse the `TimeOutSkipTest`'s code here...


Looks like my comment is probably wrong and comes from my lack of JS knowledge.


NB: this patch is based on the old bpf_asm snapshot used by Cloudflare, not the latest Linux kernel sources.


Euh, wat moet beck voorstellen?


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


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2059 of 2062 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.85%25-yellow.svg)


Nit: typo s/suitibly/suitably/


Same here. Like this kodi will popup info "Need restart" and then it will restart.


* IMHO this patch is not as "bunch of special cases" as you say.  It is at least general for integer exponents.  It only treats "half integer" cases specially because we ALREADY special-case `sqrt()`, which was naively implemented as ** 0.5 till https://github.com/rakudo/rakudo/pull/667 )
* You can apply the same algorithm to `Num` cases like so:
````perl6
multi sub pow(Complex:D \a, Num(Real) \b) returns Complex:D {
    return Complex.new(NaN, NaN) if b.isNaN || b.abs == Inf;
    my $ib = b.Int;
    my $ic = pow(a, $ib);
    return $ic if $ib == b;
    my $u = (b - $ib).abs;
    my $t = a;
    my $r = Complex.new(1e0, 0e0);
    while $u > 0 {
        $t = $t.sqrt; $u = ($u - $u.Int) * 2;
        $r *= $t if $u >= 1;
    }
    $ic * (b < 0 ?? 1 / $r !! $r);
}
````
But that should be way slower for general cases than `(b * a.log).exp`,  calling `sqrt()` up to 52 times.


@alexkb +1


But still Error('An unknown error occured') does not look very friendly. Getting it too often here.


known problem: client-side, eXide is not processing results when "adaptive" is selected. results are displayed when the XML dropdown is selected - even if the results are identical.


why not, +1


Yes, I use regex to manipulate this path. Do you think that SimpleXML would be as fast as regex? And how would I do it?

> It's a implementation detail

But it breaks the API (`FrontendOutputPreGenerate` delegate) by changing the way it works.

Maybe it's not too important… Anyway, I wouldn't know how to find those release notes for a special version. When I updated my websites, I used the migration guides in the wiki (e.g. https://github.com/symphonycms/symphony-2/wiki/Migration-Guide-to-2.6-for-Developers).


@rihards done :dancer:


Nit: `->` on next line is the "new" style


Understood! :)


Both these changes are quite wrong, because:

- ```pan_ofs``` is ```unsigned char```, hence the ```(--pan_ofs < 0)``` check will always return false;
- ```126-pan_ofs``` could be ```-1``` in case ```pan_ofs``` is ```127```, hence it's possible to access ```dBm_pan_volume``` out of bounds.


@rihards Done :dancer:


Для себя если только


TeamCity Civic Assembly :: Build backend + frontend [Build 153](http://54.229.176.143/tc/viewLog.html?buildId=424&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 12, passed: 1520, ignored: 12 Build time: 00:19:19
### Failed tests
```
Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

User Management: Should submit a form for user creation: <no details avaliable>

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

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

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

Organization tests: Should not create new organization if cancelled: <no details avaliable>

Venue tests: Venue type: Should add new Venue: <no details avaliable>

Venue tests: Venue type: Should change Venue: <no details avaliable>

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


How do we go about merging your changes in? It looks like you've completely forked the code....


Nit: add some asserts that `impl_def_id.is_local()` (we ought to make different types for known-local DefIds, I've been too lazy to do that...)


Similarly, I'd add an assertion here about `impl_def_id`


i dieded teh overextra kode


Du coup tu l'a merge. T'a la flemme de faire de l'autre façon ? xD


@malirod , 18a37d9 , Done


Well I'd probably prefer failing the job with `raise exceptions.OptionValidationError(details)` as `--replay-test-status` is incompatible with `--multiplex-files`, so the user probably needs to decide which one he wants to remove.


# ls -la rootfs/var/log/snoopy.log
-rw-r--r-- 1 root root 162766 Jan 12 09:44 rootfs/var/log/snoopy.log

Your non-root users can not write to that file, therefore you see no output. Chmod to 666 (insecure, for testing) or route through devlog output (better).

b.


Maybe assert that return value is true?


ERROR: /home/joshndroid/temarsh/frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/AppCircleBarTile.java:64: DONT_TRACK_ME_BRO cannot be resolved or is not a field

u getting this error?


One question about the chances of performance issue due to the in-memory sorting and creation of 'PathWithStat' objects for every iterator API call. Say, if there are millions of children exists and again assume many queries hits the server at the same time. Probably we should consider the cost of this as we are exposing new API to the users.

Presently am not getting any solution to avoid the per sorting, may need to find better data structures. I'm just adding a comment here to remember this part.


`--replay-test-status was specified, but avocado failed to retrieve the source job's results. Aborting...` would be IMO more user friendly.


Hi,

the changes to `check_mariadb_slaves.py` breaks the script on Python 2.7.5. It fails with:

    check_mariadb_slaves.py: error: unrecognized arguments: /usr/lib64/nagios/plugin /check_mariadb_slaves.py

due to the wrong argument list passed to `parse_args()` when using `sys.argv`, which includes the scriptname as first element. Slicing the first element in this case should fix this issue.


Submitter as pull request #56


^^ I mean putting the argument which caused the failure is a nice thing...


These should be fixed by moving the decission to the ATT setup logic, instead of using the transport definition for it, see issue https://github.com/Particular/NServiceBus/issues/3292


Nit: I'm guessing you moved to `if let` because of rightward drift? I personally would prefer to keep the `match` but move the bodies into helper fns. But that's just my preference for short fns coming through I guess. This way is ok too.


http://hydra.nixos.org/build/29741123 - you also need to bump all tox stuff, see https://github.com/NixOS/nixpkgs/pull/12201


tested with different combinations of nameservers in resolv.conf and your new solution worked well


see rida on kuidagi ära eksinud siia


Any idea when a service pack, new update for Visual Studio or a new MS Build Tools 2015 with fixed MSBuild is going to be released? This is a big show stopper bug for us...


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


did not find any swap() method in my esp8266 code base:
```

Arduino: 1.6.7 (Linux), Board: "Generic ESP8266 Module, Serial, 80 MHz, 40MHz, DIO, 115200, 512K (64K SPIFFS), ck"

Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp: In member function 'virtual void Adafruit_8x16matrix::drawPixel(int16_t, int16_t, uint16_t)':
Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp:306:14: error: 'swap' was not declared in this scope
     swap(x, y);
              ^
Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp: In member function 'virtual void Adafruit_8x8matrix::drawPixel(int16_t, int16_t, uint16_t)':
Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp:347:14: error: 'swap' was not declared in this scope
     swap(x, y);
              ^
Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp: In member function 'virtual void Adafruit_BicolorMatrix::drawPixel(int16_t, int16_t, uint16_t)':
Arduino/libraries/Adafruit_LED_Backpack_Library/Adafruit_LEDBackpack.cpp:383:14: error: 'swap' was not declared in this scope
     swap(x, y);
              ^
Using library Wire at version 1.0 in folder: .arduino15/packages/esp8266/hardware/esp8266/2.0.0/libraries/Wire
Using library Adafruit_LED_Backpack_Library at version 1.1.0 in folder: Arduino/libraries/Adafruit_LED_Backpack_Library
Using library Adafruit_GFX_Library at version 1.1.3 in folder: Arduino/libraries/Adafruit_GFX_Library
exit status 1
Error compiling.
```

 Using Arduino IDE 1.6.7 , esp8266 2.0.0 from ESP8266 community. Commenting out the ``#ifdef`` works fine.


I will try later with mingw and See if the Problem was related to mvcs


replace this with the following in order to make this file independent of your particular setup:

    envisionDirectory = os.path.dirname( os.path.dirname( os.path.dirname(os.path.realpath(__file__))))


Quite long message. How about: `Overriding the replay multiplex with --multiplex-file`. (but I'm terrible at English, so check with @clebergnu :smile:)


my test query is as follows:

```xquery
xquery version "3.1";

attribute joe {"blow"},
1,
true(),
map{"a": map {"b" : ["c", map { "d": 1 }]} },
function ($name) { "hello" || $name },
string-join#1,
[1,2,3],
[attribute joe {"blow"}, 1, true(), map{"a": map {"b" : ["c", map { "d": 1 }]} }, string-join#1, function ($name) { "hello" || $name }],
<x/>,
<!-- comment -->,
<x joe="blow"><!-- comment --></x>
```

current, the following expected results can be seen by submitting this query with the XML dropdown selected:

```
joe="blow"
1
true()
map {"a" : map {"b" : [ "c", map {"d" : 1} ] }}
(anonymous)#1
string-join#1
[ 1, 2, 3 ]
[ joe="blow", 1, true(), map {"a" : map {"b" : [ "c", map {"d" : 1} ] }}, string-join#1, (anonymous)#1 ]
<x/>
<!-- comment -->
<x joe="blow">
<!-- comment -->
</x>
```

Screenshot showing syntax highlighting (some borrowed from `pretty:pretty-print-xml()` and others borrowed from ace editor styles for the selections:

> <img width="1209" alt="screen shot 2016-01-12 at 3 54 41 am" src="https://cloud.githubusercontent.com/assets/59118/12260793/7d3a8eec-b8eb-11e5-8391-277b52a78d30.png">


@davivel for material design purposes better use the Compat version, see https://github.com/owncloud/android/pull/1416/files#diff-4f056a984fafc9e24fd129afd3d7dd9dR143 (branch / PR I am working on polishing the share(e) views)


Please review this once again


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25250/testReport/) for this commit**


These should stay on the FactoryDefinition, inside the transport we do not have this info before initialize is called


(f1e050b) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


`val result = getTheirDetails(version, credsToConnectToThem.token, Uri(credsToConnectToThem.url))`


getattr without the default value raises exception. So you either have to specify it, or you could as well use `args.replay_ignore`.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 932](https://teamcity.labkey.org/viewLog.html?buildId=296351&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 9 (6 new), passed: 236 Build time: 00:30:46
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

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

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

org.labkey.test.tests.cas.SSOwithCASTest.testSSOwithCASFromHeaderLink: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testNewUserSSO: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOWithCASFromLoginPage: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

```


good.


I have changed it to use virtual field.


This line forces to recompute taxes and remove manually inserted taxes on the line.


Hmm, this script is a little too specific if you ask me. Why not change it so that it copies everything from the source directories except for `.h` and `.cpp` files. Then we can additionally specify explicit `.h` and `.cpp` file patterns to copy. The `glob` module might be useful here.

This should simplify the file and also make it a lot more maintainable.


https://youtu.be/EvE4Y6DjBPU
https://youtu.be/5gip4PTgSoU


Hello
Je ne vois pas de différence avec l'ancien code
Cela t'a résolu un soucis ?


just fall into this case。。。。


Wouldn't this be better as simply `self.m.borrow().parent.get().clone()`? The `return` and `;` are redundant.


Actually you can supply this function to `argparser` as `type` and it'll be done during parsing the args (and you should raise ValueError on failure)


Faq entry added:
https://github.com/a2o/snoopy/blob/master/doc/FAQ.md#8-i-do-not-see-any-non-root-snoopy-entries-in-my-logs


Ok, my mistake, thank you, through rsyslog works great :)


yep
le premier ne veut simplement pas compiler car il se retrouve avec des \n dans le fichier C


@yvesgoeleven, I have reviewed the [Azure impl](https://github.com/Particular/NServiceBus.AzureServiceBus/blob/5d9891c48b4a74fe06331f9c44c03040fbf9284c/src/Transport/Config/AzureServiceBusTransport.cs#L45-L60), and currently u guy hard code these.
Are you saying that code is going to change ?


It [was suggested by burzum](https://github.com/Codegyre/Robo/issues/233#issuecomment-157214090) to wrap webmozart inside a class.


nb: It's equal to "line.tax_id = line.product_id.taxes_id"


Hahaha, sehr sehr geil!


Anh đang hiểu sai ý của em rồi. Cái này là thông số của hình ảnh ở dạng sau
![capture](https://cloud.githubusercontent.com/assets/1934754/12261012/0ffda6d2-b951-11e5-97ae-d6d499d855ee.JPG)


Could you let both copyright ?.
Original file : https://github.com/akretion/odoo-addons-grap/commit/a4296d0cb3a4d9862b76a22c543bc0d04cd9886e#diff-4e623243841c7aa3375651cfccef2749L6
regards.


I feel uncomfortable about having a `source_substs` coming in but no inference context. At minimum, we should assert that there are no `TyInfer(_)` or `ReIner(_)` within the substitutions. e.g. trans does `assert!(!source_substs.types.needs_infer())`, which is onyl part of the strong since it doesn't cover regions.

Does this code run outside of trans? I guess i'll figure it out for myself later, but I suspect not -- in which case I think I'd prefer to move it to `librustc_trans`. We might want to make a `librustc_trans/trans/traits` module, there's a reasonably amount of trans-specific trait-resolution code by now.


Ca t'arrive à l'école ?


This is a bit of code duplication. The whole thing should be moved to a function.


change the package from org.egov.repository to org.egov.commons.repository or if not possible org.egov.commons2.repository


Hmm, never needed to use `frozenset` before. What is the benefit of using it over simple `tuple` or `list`? I can see you only check for it presence, so you don't use the extra `difference`, `intersection` ,... methods...


This is unused. You can remove it altogether.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25252/testReport/) for this commit**


You are more experienced in git magics! Your ideas would more valuable :) I'm a noob in git. I had just cloned original HDMI2USB-misoc-firmware into a directory, and pushed it into a blank repository created on Github :/

Also, We need new target file for vga. Currently I using by just modifying the `atlys_base.py`. I had also modified the `Makefile` and `Makefile.base` files. I guess I can fix these.

What do you suggest?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25253/testReport/) for this commit**


Yes, exactly like in back-end...
The constrains is only there to avoid to call manually the onchange ...


Well nothing against escaping the chars, but how about using two types of quotations instead? :smile:


For Windows, it cannot be used so the error is: [CRITICAL]: Please REMOVE xdebug in production server.
And for linux, i'm not tested the script.

Regards


Maybe you could add rsyslog config into faq or readme, like:
```
/etc/rsyslog.d/snoopy.conf
$template DYNsnoopy,"/var/log/snoopy.log"
if $programname == 'snoopy' then -?DYNsnoopy
& ~
```


hmm, the sentence does not make sense to me. By any chance didn't you wanted to use `provide` or `supply`?


LGTM


Yo Jacob :)

I was pretty sure that was write, especially with "Japanese" being in the second sentence. The thing that confused me was seeing "word" in the second part of the sentence.

Also Hello!


Missing spaces.


@Marty-D, please review this fix


In what condinition South East `y` can be smaller than North West `y`?


Should be EVPOLL I think.


Extra whitespace


You forgot to remove the script.


A spec for the user invitation service would be good..


Should I just fork again through Github, make required changes, commit+push them and send a pull request?


`__all__` is already defined a few lines above ...


Why do we need to explicitly use the Celluloid namespace inside of an actor class?


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 125](https://teamcity.labkey.org/viewLog.html?buildId=296316&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 42 Build time: 01:16:54
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <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>

```


sf


woops! now rebased, thanks.


Fixes #135


no, it's still there https://github.com/PrismarineJS/node-minecraft-protocol/blob/master/src/index.js#L12

(see https://github.com/PrismarineJS/node-minecraft-protocol/issues/258 though)


Brackets not strictly necessary of course (precedence) but I think it's clearer this way.


Yes you can drop ... as you precedary mentionned : "Beginning January 12, 2016, only the most current version of Internet Explorer available for a supported operating system will receive technical support and security updates"

On association side you have to communicate on this (which browser are supported).


Looks like this wasn't testing what it said it was testing :-)


Looks good to me. Thanks!


It's probably worth wrapping this in a `using` statement as it's a `Disposable` object.

https://msdn.microsoft.com/en-us/library/system.windows.forms.control.creategraphics%28v=vs.110%29.aspx#Anchor_1

Guidance is to call `Dispose` when your finished (`using` statement is my preference)


I don't think that this should be removed. I think that the test name is a copy/paste error and needs fixing to match the test body instead.


The reason to use a class / method and to inject it is simply to avoid tight coupling by using tons of singletons. IMHO there are already to many of them used.


Actually the whole message as a bit misleading. How about: `can't find matching job id %s in %s directory`


A test in `cluster_pod_watcher_test.rb` was failing due to `uninitialized constant` errors, complaining that it could not understand `Actor`. Could not understand why, but the error was gone when I added this. This `uninitialized constant` errors are still a mystery to me. :(


Comit message : update > Update


Btw it'd be nice to get results in case there are multiple matches and return another messages.


It's probably also worth creating some sort of `struct` to represent both values and returning them from a single call. You could of course be lazy and return a `Tuple<int, int>`.

Though from memory they're likely to be exactly the same, I don't have a source to confirm that though.


http://docs.nuget.org/consume/package-restore/migrating-to-automatic-package-restore suggests removing the whole EnsureNuGetPackageBuildImports target - any reason you didn't go for that?


Don't we have this available somewhere at result/job/test definition? We do have mapping so maybe you can reuse it here so it get's updated when we add new statuses.


Ok it works, good yob. Here's some remarks i found to improve it further:

Optimization always converges to the almost same parameters (in this case l=1.74 s=0.255), the normals are surely improved and some of them are almost identical to pcl ones (blue discs), but some of them are still incorrect and it looks random which of them are incorrect.
![screenshot from 2016-01-12 11 29 09](https://cloud.githubusercontent.com/assets/1950251/12261360/e815681a-b920-11e5-84d6-b271ba237f83.png)

Another thing i noticed, i made 10 computes or so and the FIRST normal computed is always NAN no matter what...  thus the errors towards pcl:
```
Error from pcl normal new lib: inf
Error from pcl normal new lib: 0.092285
Error from pcl normal new lib: 0.988456
Error from pcl normal new lib: 0.248141
Error from pcl normal new lib: 0.262916
Error from pcl normal new lib: 0.154093
Error from pcl normal new lib: 0.386632
Error from pcl normal new lib: 0.127535
Error from pcl normal new lib: 0.91962
Error from pcl normal new lib: 2.48641e-16
Error from pcl normal new lib: 0.696234
Error from pcl normal new lib: 0.704938
Error from pcl normal new lib: 0.410239
Error from pcl normal new lib: 0.730451
Error from pcl normal new lib: 1.07823
Error from pcl normal new lib: 0.853574
Error from pcl normal new lib: 0
Error from pcl normal new lib: 0.481036
Error from pcl normal new lib: 0.493647
Error from pcl normal new lib: 0.819825
```

As you can see some are close to zero :+1:  some other are not :-1: , plus the first one is nan.


https://github.com/ClearSkyTeam/ClearSky

Details


@ccx-testerbot runtests master


use ApplicationEnvironment.isInDevelopmentMode(), so we can 3 lines into 1 and keep the logic for checking that in 1 place


Weird. It might be because this is a class method and it's being called in a context outside Celluloid. Anyway, thanks for the explanation @sbrnunes, I'm ok with this now.


An alternative to changing all the project files would be to use src\packages for both support and generated code. Any thoughts on that?


I would like that this test case is separated.. I don't like that there is 1 huge testcase.

I suggest:
 * 1 testcase that just tests how comments are treated.
 * 1 testcase that tests locations.
 * maybe 1 testcase for different types of directives


Thank you for the request. It has been added to the queue (id: 145). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/145) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


I think sksamuel rewrote his own typeclass based on this suggestion and included it in the main distribution. you may want to check that out. :)


Thank you for the request. It has been added to the queue (id: 146). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/146) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


@lowjoel Spec for testing what ?  if `user_email.skip_confirmation!` was removed, the original spec will fail.


What is the problem here?


both fixed, thanks


"Portable" is beginning to feel like a tainted word to me. It may be still appropriate for this codebase, but I'd probably avoid using it in new code. Whether it's worth introducing here is a tricky question. (I assume here it means "PCL"?)


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2349 of 2384 tests passed, 0 test warnings)

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


Is this going to be needed on PackageEvaluator too? I see this causes Compose's tests to fail in the PackageEvaluator VM with
```
ERROR: LoadError: LoadError: Generated output differs from cached test output:
text.png

text.png:
Binary files /home/vagrant/.julia/v0.4/Compose/test/data/text.png and /home/vagrant/.julia/v0.4/Compose/test/text.png differ

 in error at ./error.jl:21
 in include at ./boot.jl:261
 in include_from_node1 at ./loading.jl:304
 in include at ./boot.jl:261
 in include_from_node1 at ./loading.jl:304
 in process_options at ./client.jl:280
 in _start at ./client.jl:378
while loading /home/vagrant/.julia/v0.4/Compose/test/compare_examples.jl, in expression starting on line 37
while loading /home/vagrant/.julia/v0.4/Compose/test/runtests.jl, in expression starting on line 18
```
though @IainNZ hasn't ran against this tagged version yet


You can receive notifications of `DPI` change by listening for the `WM_DPICHANGED` message. (The `DPI` I believe is stored in the `wParam`). So you can probably get away with calling them once and using the message to update.

> The window is created on a monitor that is not set to the default value of 100%. 100% is 96 dots per inch.

You may just be able to listen for the message and get all the info you require.

https://msdn.microsoft.com/en-us/library/windows/desktop/dn312083.aspx


Reviewed


cherry pick


@alexmorozov Thank you!
the last two lines(line 74 &75) you added should be:
profile_form.instance.user=user_form.instance
profile_form.instance.save()

after modification, it works smoothly!


perhaps the fact that the chunk will not be stopped completely, mods again to download it - partly, he will hang in the list of loaded chunks.


Ahh, more beautiful `WndProc` magic :+1:


Ahh dammit, at one point I was Disposing but it must have gotten lost in the code juggling I was doing :+1:


Ah, now I see, you have to push a tag, not master branch.

    git push origin v0.3.4


Missing semicolons and brackets.


Big +1 to this commit :)


If write needs locking, read needs that too, unless you get into RCU or
similar "lockless" techniques.

--
Bazsi

On Tue, Jan 12, 2016 at 10:35 AM, jbfuzier <notifications@github.com> wrote:

> I did not find the information in glibc doc, do we actually need to lock
> the hash table on dns cache read or is it safe not to lock? If so that
> means only write need locking, so we are indeed blocking other threads but
> locking for one insert is still quicker than doing a dns resolution. So in
> the end, other threads that got blocked for a while will benefits later on
> from using the cache instead of doing a dns resolution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbfuzier/syslog-ng/commit/6c35a9946e838ca3a5b3d0d54fd582a9dc41ff91#commitcomment-15392014>
> .
>


This needs a hook inside the endpoint to release all of its resources at once without caring about safely terminating.


Would be good to know which message we have received here.


Also here, it would be good to know which message we have received.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25260/testReport/) for this commit**


Ouch, thanks. I have forgotten to squash this fix into the PR. Sorry for that.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25261/testReport/) for this commit**


Seems like we have returned to way it was before me;)
Alright, it was inevitable.


Well...
old directory here -> old directory there
meant was here


test


visa logika iskelt, nes cia ne prie ko Termui zinoti apie nuolaidas. cia aisku tik veliau tai supranti, bet atejo laikas


You can't simply reorder existing flags. This breaks `remote` runner, which supplies `timeout` as positional argument.


Yep, gotta love it! :stuck_out_tongue:

Would be nice if they updated `WinForms` to add a few new events, I'm sure it would be easy enough.


Same as the endpoints, this needs a hook inside the socket base to release everything. The destroy function simply fails because it checks that the state is valid, which cannot happen during a fork, before terminating.


This is a slightly surprising version number for System.Runtime, but I assume it was picked automatically by VS at some point, and that it works...


test stop properly? Stopping returns `!= 0` which indicates a failure, right?


This targetFramework looks odd for a .NET 4.5 only project... is it just because that's a target that we know exists and *also* works with .NET 4.5?


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

```


:heart:


Name suggested elsewhere is googleapis-packages (lose the first hyphen). Will drop you a doc...


@huacnlee This looks like a configuration in the SMTP server gitlab.alipay-inc.com uses, which strips the custom `From` header and replaces it by `scmii <scmii@alipay.com>`. As you can see in the gitlab.com example, GitLab sets the `From` header to the user's name (`Dmitriy Zaporozhets`) and the `gitlab.email_from` address (`gitlab@mg.gitlab.com`).


Good thought. Actually error will happen on sending uninitialised message. For example, this will fail:
```
message = EmailMessage(render=True)
message.send()
```
This will throw "TemplateDoesNotExist: None". This seems loud enough. No side effects. It just will do nothing if a developer adds ``render=True`` accidentally. Same as in case he forget to remove unused variable, etc. But this adds some flexibility in rare cases. I'm that man who finds such rare cases very often :) So I'd left this implementation with late error unless there is any known kind of problem.


This could have just been `assert False, "ü"`.  Not important though


Are multiple project URLs supported these days? If so, it would be nice to link directly to Github too.


hello,angular-photo-slider is not working ,i don't know why? your test is working?


I don't think it makes sense to wrap static methods in a call just for the sake of doing so. If you follow that thought, you'd have to wrap every call to native PHP methods into classes too, which clearly doesn't make sense.

IMO you should find out whether there is any conceptual value in the extracted class. For example, you could have a `Storage` interface with a `find()` method that accepts a Glob. Maybe this makes testing easier, or maybe you actually need to support different kinds of storages. But if the answer is no, there's no point in this class for me.


Is it obvious that this means "4 and above"? I hope so, but it's worth considering.


Is it correct that we use the `JobCancellationHandler` here where we actually want to stop a job? If not, then the test case testing the GET request for stopping does not work properly to catch the wrong "stopping" and should be corrected.


JavaDocs missing


This file is not part of the commit... did you forget to add that? ;-)


原有的判断交叉的方式不全面，新加的代码的思路是：两个线段交叉时其总长度小于等于两个线段的和，结果会景点smart模式分词结果。

测试case如下 ：
输入：孙要求是正确

自定义词库：
孙要
孙要求
要求
求是
正确
———————————————————————not smart before bug fix
0 - 3 : 孙要求 | CN_WORD
0 - 2 : 孙要 | CN_WORD
0 - 1 : 孙 | CN_WORD
1 - 3 : 要求 | CN_WORD
2 - 4 : 求是 | CN_WORD
4 - 6 : 正确 | CN_WORD
————————————————————————smart before bug fix
0 - 2 : 孙要 | CN_WORD
2 - 4 : 求是 | CN_WORD
4 - 6 : 正确 | CN_WORD

———————————————————————not smart after bug fix
0 - 3 : 孙要求 | CN_WORD
0 - 2 : 孙要 | CN_WORD
0 - 1 : 孙 | CN_WORD
1 - 3 : 要求 | CN_WORD
2 - 4 : 求是 | CN_WORD
4 - 6 : 正确 | CN_WORD
————————————————————————smart after bug fix
0 - 3 : 孙要求 | CN_WORD
4 - 6 : 正确 | CN_WORD


This broke SITL


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


I'm assuming the net45 version will be in another commit.


Hi monteslu,
You've updated the version number to be equal to the packaged version available in the chrome store but there are still some discernible differences between them. Namely, in the  packaged version available in the chrome store the default javascript console.xxx functions are replaced to use the j5-chrome log function to send info messages to the ui window. Wouldn't it be better if you merged those differences first and then mark the "0.5.0" version.

I haven't done a pull request because I'm having trouble in reproducing the build and functionality exactly like it is available in the chrome store.

Probably one of the issues I'm getting is that the build is fetching different versions of the npm packages required in the package.json due to the package versions specifications and I don't know to identify the specific npm module versions from the webpacked bundle.

It would be awesome if you could provide the 'npm ls' output of the j5-chrome directory from where you built the j5-chrome packaged version available in the chrome store.

PS: Thanks a lot for j5-chrome! Controlling a robot using only the chrome browser is pure magic and it allows pretty far-out stuff! :)


> If you follow that thought, you'd have to wrap every call to native PHP methods into classes too, which clearly doesn't make sense.

Which would be the right way of doing it. The point is you can't replace a static call by overloading the method. If you want to stay flexible and allow people to change the glob implementation in their task for whatever reason you'll wrap it. And I think we agree on that the use of tons of singletons *is* bad practice?


@mikepetersonccv it's been removed. Thx.


Btw replay is actually incompatible with `--remote-hostname`. That is fine for now (I guess), on the other hand this must not break the normal `--remote-hostname` execution, which it does now. (try `avocado run passtest --remote-hostname localhost`)


Are we happy with this change? I personally am - and it may mean we're happy to start using C# 6 features within the support libraries - but I thought I'd highlight it as a change that may break some customers who are building this themselves. (Obviously it shouldn't break clients using binaries.)


Sorry, forget my comment. I didn't include your latest commit. There you fixed it properly. My bad.


I'm  basically assuming this is okay - I don't really "do" solution files.


awesome


The last two changes (lines 1112/1119 1119/1126) break the message headers when using the preview pane


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **13/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **23/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **19/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **83/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/145) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


I saw somewhere someone had actually made a Form class that wrapper a whole load of `WndProc` events, now I wish I used it.  So many magic wndproc numbers


I think that these methods should be changed to constructor parameters.
```
  def ourPartyName: String
  def ourLogo: Option[Url]
  def ourWebsite: Option[Url]
  def ourVersionsUrl: Uri
  def ourPartyId: String
  def ourCountryCode: String
```


沒了 @kpxp


Clean headers "QMApi"


But we don't want a String, but something string-y. If you don't like the rescue (I'll admit I have mixed feelings on that as well: I like how concise it is, but it is abusing exceptions a bit), then you should do `i.respond_to? :to_str` instead.


The described behavior is as it should be.

Probably this is also implemented correctly, I was confused by a different bug that I will report separately.


Test


I think that is your best bet for now.


Shouldn't we check the actual type of the error returned?


In order to support relative path, you'd have to also store `pwd` and change directory before discovering the paths. (eg. when running passtest using `examples/tests/passtest.py` and replaying it from `examples` directory raises exception as the file `examples/tests/passtest.py` does not exists.


Maybe add a comment for this? I had to check that this really means "fallback". Maybe that's just me though...


You can modify scalaz imports in the header to import scalaz.{-\/, \/-, \/} and then you could reference Success by the short name.


You can try downloading and running the "dxva checker" program. For what's it worth, I haven't seen anything that confirms or denies anything, since you've only posted logs from the wrong branches.


If we don't need these in Win81, why are they still effectively in the Nuget dependencies? Should we have another dependency group (like for net45)?


Hmmm, OK. Then that's fine.

What's the spec that will fail, by the way?


Ah, I didn't see this! Wouldn't it make more sense to enforce something like
`"{name}{length}".format(name, length*8)`
With this we could generalize our naming scheme with
```
int8
int16
int32
int64
uint8
uint16
uint32
uint64
float16
float32
float64
```
which we can directly map to [numpy](http://docs.scipy.org/doc/numpy-1.10.1/user/basics.types.html). Instead of hardcoding any legacy translation formats like `float324` we should probably drop them or provide a translator:

```
def translator(value, length):
  if (length >= 16):
    length = length/2
  name = LEGACY_LUT[name]
  return type2dtype
```


No it's fine, but could you update the header as well?


This may cause errors when compiling on a buildserver.
In case of projects that contain .xaml files the CompileTemporaryAssembly target (http://referencesource.microsoft.com/#MSBuildFiles/C/WINDOWS/Microsoft.NET/Framework64/v4.0.30319/Microsoft.Xaml.targets) is executed. This causes that $(TargetDir) contains a temporary directory which does not exist anymore when the SonarQube.Roslyn.ErrorLog.xml file should be written:
CSC : error CS0016: Could not write to output file 'C:\Builds\2\...\tqdshame\SonarQube.Roslyn.ErrorLog.json'


@Rocky640 @IIvec  I don't think SPSA works very well with larger ck values. I also got very bad results with a tuning try for the material imbalance tables, starting from zero values. CLOP can only reasonably be used with 1 or two parameters, and Texel's tuning method doesn't work for Stockfish from my experience. Too bad. :-(


Lengths of `party_id` and `country_code` are limited to 3 and 2 respectively in the spec. I think this should be validated here with a `require`.


@pranaypratyush Since we are moving to `rosparam` definitively, we can drop this.

If the `rosparam` is not available, simply throw an exception and kill the node. That's easier than using a default value that can't be tracked.

Basically,

> If the values not on the server, you can't run the module.

Okay?


cc @prudhvid @nevinvalsaraj @kalyan-kumar


@JimmyRojas is necessary change -Pstatus=isValid by -Pstatus=valid parameter in code or only documentation??


Slightly lost as to where that version number will be specified then. I'd expected us to specify it on the command line, used to build *everything* - this build file suggests that we build Google.Apis.Core.dll first with the right version number, then everything else afterwards. Am I missing something? (Seems likely.)


> So many magic wndproc numbers

Part that annoys me is that most of them are declared if you go digging through the source, they're just so very annoyingly in an `internal` class!

http://referencesource.microsoft.com/#System.Windows.Forms/winforms/Managed/System/WinForms/NativeMethods.cs,82


Just checking - Is it actually possible to be backwards compatible? If I recall, I think I remember that there is at least one breaking change in the CSV Schema 1.1 compare to 1.0.
@DavidUnderdown, do you know?


Agreed that it doesn't have any effect. However, it does improve readability somewhat IMHO, so I'll keep it in. Just as a note for others reading this: PR #225 should fix the actual bug.


@michaelkirk  https://github.com/qgis/QGIS/blob/master/python/plugins/processing/core/parameters.py#L47

That method is the one that Jurgen fixed to solve the issue with the test


Should this be added to the `values` below?


@domenkozar I don't think so. This package is not related to the tox instant messenger (libtoxcore/qtox/...). It is a generic virtualenv management and test command line tool for python projects: https://tox.readthedocs.org/en/latest/

It's a little bit confusing that both projects use the same name :smile:


Good point, I will add this note to the ticket about the error handling ;)


@pranaypratyush http://chris.beams.io/posts/git-commit/

It's a slightly long read about how to write good commit messages, but the points that the author makes are very good, and must be adhered to.

@ghostwriternr and @kalyan-kumar : You guys can go through the article as well.


:dart:


I would say that, given the same point cloud is OK that the optimisation
converges always to the same values.
I did not get if the NAN values come from pcl or from my lib. For the
sphere I don't recall seeing NAN normals.
I'm about to test my code on a real point cloud using golem. I will come
back with the results from my testing.

On 12 January 2016 at 10:42, Federico <notifications@github.com> wrote:

> Ok it works, good yob. Here's some remarks i found to improve it further:
>
> Optimization always converges to the almost same parameters (in this case
> l=1.74 s=0.255), the normals are surely improved and some of them are
> almost identical to pcl ones (blue discs), but some of them are still
> incorrect and it looks random which of them are incorrect.
> [image: screenshot from 2016-01-12 11 29 09]
> <https://cloud.githubusercontent.com/assets/1950251/12261360/e815681a-b920-11e5-84d6-b271ba237f83.png>
>
> Another thing i noticed, i made 10 computes or so and the FIRST normal
> computed is always NAN no matter what... thus the errors towards pcl:
>
> Error from pcl normal new lib: inf
> Error from pcl normal new lib: 0.092285
> Error from pcl normal new lib: 0.988456
> Error from pcl normal new lib: 0.248141
> Error from pcl normal new lib: 0.262916
> Error from pcl normal new lib: 0.154093
> Error from pcl normal new lib: 0.386632
> Error from pcl normal new lib: 0.127535
> Error from pcl normal new lib: 0.91962
> Error from pcl normal new lib: 2.48641e-16
> Error from pcl normal new lib: 0.696234
> Error from pcl normal new lib: 0.704938
> Error from pcl normal new lib: 0.410239
> Error from pcl normal new lib: 0.730451
> Error from pcl normal new lib: 1.07823
> Error from pcl normal new lib: 0.853574
> Error from pcl normal new lib: 0
> Error from pcl normal new lib: 0.481036
> Error from pcl normal new lib: 0.493647
> Error from pcl normal new lib: 0.819825
>
> As you can see some are close to zero [image: :+1:] some other are not [image:
> :-1:] , plus the first one is nan.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/db07b15c925c8b6059d397721847e3a78d00e7fd#commitcomment-15393425>
> .
>

--
Claudio

Curiously enough, the only thing that went through the mind of the bowl of
petunias as it fell was "Oh no, not again".
Many people have speculated that if we knew exactly why the bowl of
petunias had thought that we would know a lot more about the nature of the
Universe than we do now.
D. Adams - Hitchhiker's guide to the galaxy


looks like it's not used


*Begins a slow clap for Microsoft*

:clap: :clap: :clap:


build should work for each change, you are supposed to add @Ignore in this case.


I agree with you. We should make sure C++, C, Python, Fortran,... they all
understand the same types and interpret them correctly. We don't need to
support old naming schemes until we reach a stable version, and all of our
code is independent of the type name (just the elemnt size is relevant), so
if we choose some name convention we can just go ahead and impose it.

Fact is, I'm working a lot *with* serialbox lately, but I really don't have
the time to work *on* it. I'm just fixing the few things I need for myself.

On Tue, Jan 12, 2016 at 12:08 PM, Pascal Spörri <notifications@github.com>
wrote:

> Ah, I didn't see this! Wouldn't it make more sense to enforce something
> like
> "{name}{length}".format(name, length*8)
> With this we could generalize our naming scheme with
>
> int8
> int16
> int32
> int64
> uint8
> uint16
> uint32
> uint64
> float16
> float32
> float64
>
> which we can directly map to numpy
> <http://docs.scipy.org/doc/numpy-1.10.1/user/basics.types.html>. Instead
> of hardcoding any legacy translation formats like float324 we should
> probably drop them or provide a translator:
>
> def translator(value, length):
>   if (length >= 16):
>     length = length/2
>   name = LEGACY_LUT[name]
>   return type2dtype
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andyspiros/serialbox/commit/1f447fd7acb4bfbf461bb7c3ffc753b2d9175cec#commitcomment-15393902>
> .
>


gives error without a proper reason
https://forums.adafruit.com/viewtopic.php?f=50&t=87591


This looks correct to me. I would like to discuss with @michaeldrogalis whether we should default to shared mode. I believe this should be the default as it's more robust on small instances.

It'll be hard to test without simply looking at thread counts on the process.


@JustAkan UBERTC gcc-5.3.0 버젼 나왔습니다.

https://bitbucket.org/UBERTC/aarch64-linux-android-5.3-kernel/downloads


It actually should not be here at all, since this endpoint is not exposed to the external parties. Well spotted.


@locutus2 I think for tuning 2 parameters with no good guess for a starting point, CLOP might work better to find initial values. I am not sure SPSA is designed to work with bigger ck values. I may be wrong, of course!


Ridge racer 7 crash au moment d’être ingame, et le débogueur m'indique cette ligne :/


Nit: I'd prefer to see this as an iterator for walking up the parents, but this is ok too


I would not make it generic, all implementations use `Optional<String>`


> The point is you can't replace a static call by overloading the method.

Yes. But it's always important to find out whether replacing that call is useful. Did you ever want to replace a call to `glob()`? I didn't. Adding classes and method calls has a cost, and you need to find out whether it's worth to pay that cost.

Furthermore, simply wrapping static method calls in objects is a very technical approach to system design that I don't agree with. I think it's much more important to find out the semantic concepts and represent those by classes and interfaces (where appropriate).


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


injected constructors are usually package-private (with a  @VisibleForTesting annotation), see https://github.com/google/guice/wiki/KeepConstructorsHidden for rationals


@ccx-testerbot runtests master


Ogólnie gotowe wzory wszystkich licencji mamy zawsze tu:

storage:/data3/src/COPYRIGHT*

Zależnie od uzgodnień z klientem zostawiamy nasz (c), wpisujemy klienta lub
jakieś łączone warianty, to trzeba zawsze potwierdzić per projekt.

pozdr,
r

2016-01-08 15:40 GMT+01:00 semihalf-wojtas-marcin <notifications@github.com>
:

> We wszystkich nowych plikach licencje stosujmy wg wzoru z FreeBSD
> (oczywiście z nowym rokiem i Marvell zamiast Stormshield).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Semihalf/uefi/commit/007b35f2e211acd2d6fa30455b63663510c4b658#commitcomment-15340769>
> .
>


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 126](https://teamcity.labkey.org/viewLog.html?buildId=296321&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 43 Build time: 01:23:11
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <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.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

```


Thanks @stef74
@ginold Then you may drop the IE9 commit :)


Adding in the IRC channel to the website and README could be nice :D


Thank you for the request. It has been added to the queue (id: 147). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/147) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Damn, I'm used to have it at the bottom :P


Thank you for the request. It has been added to the queue (id: 148). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/148) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


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


Why did you keep this comment?


you should not commit comments


add create table


They should not break, until you're using some plugin. In that case the plugin need to be fixed.


文字化けしてるね。
UTF-8になってるかな？


Is Cargo.lock really needed? Shouldn't it be excluded from the repo?


This is incorrect. Class name here


Yeah I agree with you :+1:


Write use scenarios (we must write stuff like children so anyone can understand)


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **13/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **23/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **19/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **83/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/146) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


>  Also, I don't get the purpose of this commit.

The Elements enum was used for two different purposes. For node masks, and (for Reference, Pathgrid, Water, Fog and Terrain) to OR an icon ID that is used in pre-existing icon files. As for these icon IDs, changing an enumeration value caused the editor to break, which was not obvious from the enum itself. As for node masks, we had to shift the bits by 1 to prevent a collision with the internal 0x1 node mask used within the nif loader. That is unnecessarily error prone. By using a separate enum we can define the enumeration values more freely, so now the bit shifting is no longer needed.

> SceneToolToggle2 is meant as a general purpose tool button, that may be reused for other purposes later. We should not tie it to a specific use case.

Then the ButtonId enum should be moved to another file I suppose. Can you suggest a better location?


No particular reason ..thanks for pointing it out ..


use @ignore or remove comment


why comment if you already @ignore?


idem


gulp-livereload 是有问题的吧，之前没办法自动刷新 combo 的 css


Meh, let them figure it out. That's good for children.


見てみる


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 63](https://teamcity.labkey.org/viewLog.html?buildId=296343&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:07:55
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <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.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

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


指摘したかしなかったか忘れてしまったけど
not null でdefaultが設定されている場合
not nullの効力は、あえてnullをセットする時だけになるのかな？

「SQLで項目として指定しない」場合、NAMEは空になります。
今回の場合、必ず入れて欲しい項目だとおもうので、 not null だけで良いと思います。

どちらにしてもアプリケーション側で空かどうかの判定はするべきなので、そこでもう一度ケアしましょう。


```
try {
MailboxSession session = authMethods.stream().filter(AuthenticationStrategy::checkAuthorizationHeader).map(AuthenticationStrategy::createMailboxSession).findFirst().orElseThrow(UnauthorizedException::new);
addSessionToRequest(session);
} catch (UnauthorizedException e) {
httpResponse.sendError(HttpServletResponse.SC_UNAUTHORIZED);
             return;
}
```


The mask is used in addLight (next commit). Morrowind by default places a light in the center of the bounding box of the object, but ignores particle systems for computing this bounding box.

Now we <i>could</i> handle the mask entirely within components/, but then we'd have more work to make sure that the mask used there does not collide with masks used in OpenMW-CS or in OpenMW. I think it's better for each application to have its own 'namespace' if you will for masks, and then tell the component which mask to use.


If you remove these tests later, you can remove them now instead of ignoring them.


@pjedrzejewski the release is named incorrectly


Yes. v:count doesn't work outside mapping and command need to use <count> argument.

I'm covering these features with test to avoid future mistakes.


With java classes now I understand. With the `classified_entity` I still don't get it. Do we stop utilising the field   classifications->activity_ref->source_entities?


get() ?


createの前にdrop tableを入れてみましょう。
そうすると、テーブルを作成しなおす時に、createTable.sql だけ実行すればOKになります。
ただし、テーブルが存在する時はdrop table構文はエラーになります。これはエラーになるものだとあきらめることがあります。テーブルの存在確認が難しいので
もしかしたらナニか良い方法があるかもしれませんが、いったんあきらめる方向で


@bendavies Fixed! :)


Hoi, ik vind het een beetje jammer dat je voor Perl hebt gekozen. Dit script had ook gemakkelijk in Python geschreven kunnen worden, wat al volop door NLExtract wordt gebruikt.


Why have you reverted this? And why is it a feature in your `CHANGELOG.md`, when this commit has never applied to? Additionally it is confusing me when you are promoting this feature here [commit 21e54be](https://github.com/driftyco/ionic/commit/21e54be).


why ?


New function for switch selector not working and now all not working because after update not show level and not show select icon for switch.


Effectively the validator implements support for each version separately, so long as you declare the version correctly there shouldn't be a problem.  range is the only one that actually has a different definition in 1.1 versus 1.0, but there are new features in 1.1 of course.  This change was aimed at the situation where someone makes an incorrect version declaration (most likely a typo) eg version 1.2, before they were told that the validator only supports version 1.1, which is not true, it will still handle existing 1.0 schema as well


should be in a previous commit


idem


and @Ignore takes a value for writing your comment


@Rocky640 @joergoster : I'll mainly use ck values in the range
[max(1, 0.05 * start), max(10, 0.2 * start)], using smaller values if I'm sure that I'm close to the optimum. I think the bigger problem is number of games. We expect miracles from SPSA running it with low number of games. I already comment that on the forum, and I'll write more soon.


You should commit this as fixups and squash before PR


`/coverage` стоит добавить в .gitignore


Optional instead?


hELLO


Yeah, I had also started working on some collection groupings for phoenix/gemstone - but got pulled away and he quickly got way further than I did.  Just wondering where we were on it (since for my purposes, I don't think much would need to be done on the gemstone end if this makes it into a phoenix core.

Out of curiosity, what part of this is limiting it to v9 only (not that I care, as I see no reason for anyone to stay on v7 anymore - especially with the shiny new installer).


Nit: this comment did not suggest to me that `parameter_environment` will be completely overwritten


Was an oversight on my part, I was receiving unexpected ErrMultipleResults during tests. I misread that as the length being set as 1 not the capacity. I'll revert.


version in parent?


is it AuthenticationFilter responsibility to ignore authentication on options ?


toLowerCase needs a Locale


This is correct, but this kind of comment is used if there was a prop. Eg, canols oil instead of canola!


you may want to create the new environment by calling `old_environment.with_caller_bounds`


Mijn beheersing van Perl is vele malen beter dan Python, zodoende was dit de weg van de minste weerstand. Omdat de rest van NLExtract in Python geschreven is, heb ik er rekening mee gehouden dat een Python implementatie van het script de voorkeur had. Zodoende vind ik het geen probleem om het script opnieuw te implementeren in Python, maar dat gaat wel weer wat tijd kosten.


Hmm, no, Optional.empty if think


When is the next release, the buttons now are almost invisible, same with button bar.


well, or, hmm. I take that back. I guess the helper you want doesn't exist quite yet.


Actually, after some experiments, I suspect that the `floor` instruction is the one which does not vectorize. The following example is not vectorized on my computer:
```
function vecfun(a,b)
    N = length(a)
     @inbounds @simd for n=1:N
        b[n] = floor(Int64,a[n])
    end
end
@code_llvm vecfun(Float64[],Int64[])
```


@lowjoel  These tests failed: `expect { invite }.to change { ActionMailer::Base.deliveries.count }.by(user_attributes.length)`

Because if email is not confirmed, confirmation emails will be sent.


I think you can put it in an other file.


in particular, we don't have anything like `ParameterEnvironment::new` but which takes the free substs, rather than creating them (which is effectively what `impl1_substs` is)


What are these "." ?


What are these " " ?


use assertThatThrownBy instead.


It's simpler to declare the companion object as implementing KotlinSingleIntentionActionFactory, as many other quickfixes do.


ありがとうございます。重複を削除しました。


It'd be better to run this check after the `deadline` check.


idem


You can use Kotlin's extension function here: element.getNonStrictParentOfType<KtWhenExpression>()


On 12-1-2016 12:34, Bas Couwenberg wrote:
>
> Mijn beheersing van Perl is vele malen beter dan Python, zodoende was
> dit de weg van de minste weerstand. Omdat de rest van NLExtract in
> Python geschreven is, heb ik er rekening mee gehouden dat een Python
> implementatie van het script de voorkeur had. Zodoende vind ik het
> geen probleem om het script opnieuw te implementeren in Python, maar
> dat gaat wel weer wat tijd kosten.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sebastic/NLExtract/commit/dab9de0cb65a7358af9e4f1aad126cc221b8d104#commitcomment-15394409>.
>
Ik had al zo'n vermoeden. Herschrijven kost ook aardig wat tijd, dus wat
dat betreft laat het maar zitten. Dan reken ik erop dat jij jaarlijs de
gemeentelijst bijwerkt. Zo niet, dan is er wel iemand anders te vinden. ;)

Groeten,

Frank


tested with all plugins disabled. headers are still broken in preview with short headers on larry skin


One thing I did notice - the phoenix.view.default.moviecollections default view seems sorted differently than I can envision many wishing it done.  Level 2 is sorted by title.  This would result in (example):

Rogue One: A Star Wars Story
Star Wars
Star Wars: Clone Wars: Volume 1
Star Wars: Clone Wars: Volume 2
Star Wars: Episode I - The Phantom Menace
Star Wars: Episode II - Attack of the Clones
Star Wars: Episode III - Revenge of the Sith
Star Wars: Episode IX
Star Wars: Episode V - The Empire Strikes Back
Star Wars: Episode VI - Return of the Jedi
Star Wars: Episode VIII
Star Wars: The Force Awakens

...which I can't see anyone having this as expected behavior.  I'm thinking a better default sort order for level 2 would be release date.


@teilmeier Could you create a ticket on jira.sonarsource.com/projects/SONARMSBRU ? It will make it easier to track than a comment on a commit which is already merged into the master


Use `TODO()` as the body of the branch?


**For others**: This commit was reverted here [commit 0e0e71c](https://github.com/driftyco/ionic/commit/0e0e71cf26c07b2da1742559e59e4dd6485970d1).


Irks, jetzt warst du doch schneller. Gestern abend habe ich noch Text-Änderungen vorgenommen, habe sie jedoch noch nicht hochladen können. Ist das zu mergen mit den neuen PDF-Links? Sind die fix?


This line seems to have a typo. There is a bracket after ```$_POST['ipaddr']``` that shouldn't be there.


good


gggd


PSI operations should never create newlines from text. If correct line breaks are not generated automatically, this should be fixed elsewhere (most likely in KotlinParserDefinition.spaceExistanceTypeBetweenTokens()).


> I don't think the rounding is as described here.

Argh! Right... the endless_factor should have been applied directly to `scale` (before rounding). Silly mistake again, sorry

> Besides, is there any reason for forcing the rounding to one decimal place?

That's a good question, and the short answer is that this was introduced when they made chromium able to handle arbitrary scale factors (which were [initially not allowed](https://chromium.googlesource.com/chromium/src/+/06cda64742b45e1eb10bbbb66bca4b4da2efb3c9%5E!/)), although this was [originally rounded to 2 decimals](https://chromium.googlesource.com/chromium/src/+/b3ca230ddd7d1238ee96ed26ea23e369f10dd655%5E!/).

They [later on adjusted this to use only 1 decimal](https://chromium.googlesource.com/chromium/src/+/5cb0c01e4233ff3eb1e7f6f677b4cf9dae9ffaf2%5E!/)  since apparently using 2 caused some trouble, and that's what we have now in our code, so I think we should leave it like that.

That said, while crawling through the chromium history to find the root reason behind this I went over [chromium issue 484400](https://code.google.com/p/chromium/issues/detail?id=484400), which  reports an issue with that rounding causing fuzzy text to be displayed on displays with odd DPI values (e.g. 101), whose [fix was committed to master in September](https://chromium.googlesource.com/chromium/src/+/fa30cbdf96439ea623d47d655a10962049d83b6c%5E!/), yet not merged into chromium v45.

The reason I mention this is that **I think we should incorporate that patch too in this changeset** now and **also bump the default scaling factor for the browser so that the end result is 130%** instead of 125%. Otherwise, we'll find that the extra scaling applied to the browser will be gone as soon as we rebase against a newer version of chromium.

So, I'll prepare these changes then and update the Pull Request.


the update to usr_02.txt is missing.


**Nitpick**
Use a semicolon after every declaration for consistency and extensibility reasons.


When this isn't true, and the msgQueue is empty doesn't this cause 100% busy spinning, as the m_messageQueue.Get is called with a timeout of 0 and and so DecodeFrame returns DECODE_FLAG_TIMEOUT each time?


pure `startswith` would be probably faster here (I remember Jiří's experiments when optimizing cartesian_parser, `startswith` performed better even then compiled `re`s)


Why `editor!!`? It's in fact nullable. Use `?.`


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **12/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **18/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **82/87** |
| _DTVCC | **10/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/147) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


I'll try to update the phoenix-core sometime this week in the V9 repo, now that the newer version of SageTV jar is out there with the new repo locations.

There is nothing in the new phoenix-core that uses SageTV V9, but, in order to get collections, then the underlying TheMovieDB libraries had to be updated.  This required that the dependent http client libraries be updated, etc.  The end result is a massive amount of library updates (that we had to take ownership over), and I have a feeling that when I deploy the new phoenix-core, it will break some other plugins that depended on the other older http client.  I have no idea what those might be, so, we decided to just push ahead with a V9 release as to no upset the existing V7 users.


For historical reasons g_mpu_box_count is box_count-1 . We filed an issue https://github.com/ARMmbed/uvisor/issues/120 to fix this source of potential security bugs and make g_vmpu_box_count intuitive @avri-schneider @AlessandroA


**Nitpick**
Always use a single space between the last selector and the opening brace that begins the declaration block.


this is should be updated


Agreed with @SquidDev, I copied the table from https://github.com/oeed/CraftOS-Standards/pull/7 without much thinking. Will fix ASAP!


**Nitpick**
Always start a new line for each selector and declaration.


this is stating the obvious. The perl code is pretty readable. What about "# Our current tests rely on INSTLANG=en_US, e.g. for keyboard shortcuts, so set it by default. Also, currently installation tests are not run in other languages."


Maybe I'm missing something, but the text of the branches you insert doesn't have any curly braces? Also it's better to add the necessary PSI accessors than to use `text.indexOf()`


Okay! Looks good.


Good idea, I'll actually italicize all Standard Name occurrences.


screenshot can not work.


You don't need these two lines - you aren't (and shoudn't be) referencing anything from the .properties file.


that should be updated


AWESOME!!!! @ICKramer


As I can see we don't need to specify `groups` here. I expected that you will look at how it works and told me -- do we need to specify separate groups or we don't. You decided to specify, but now I don't see the reason to do that. AFAIR we are running check with `Default` group and after that `ReleaseDate3Checks`, so, it's ok if `@Min` will be tested first.


Eventually I guess we will generalize all this `is_from_trait` logic into partial impls...?


Ahaaa, logdir == results dir in this case, right?


Looks like you aren't shortening the full name anywhere, so the code will contain the FQ name of the class, which isn't too nice. Please add a test case covering this (class not in default package).


This `impl` smells a bit funny. Perhaps a method along the lines of `kind_match` would be better?


@exmex or is it to much to ask for?


I'm not sure we should need to explain how to get one of these characters. Linking to the [string library](http://lua-users.org/wiki/StringLibraryTutorial) should be enough. You can also write `"\30"` instead.


Ops, see now. Fixed.


この文は文法的に解析不能で意味がわからなかったところですが、石井さんの解釈が正しいと思います。
修正しました。


Good point. It is possible using property setting but I would rather do a GraphCondition that will firstly check which mode we are running in.


This should be `locus.partkey_oj` I assume?


**Nitpick**
Add new line between line 6 and 7


**Nitpick**
Add new line between line 11 and 12


Müsstest du eigentlich einfach reinmergen können. Versuch mal, das mit dem Github-Client zu machen.


Hi,
nice find! I found another build barrier in setup-makefiles.sh line 172.
cheers glad


**Nitpick**
Add new line between line 16 and 17


At this point I’m starting to wonder if it wouldn’t make sense to just set personality function during creation of basic block.


**Nitpick**
Add new line between line 21 and 22


This does not seem to be documented /cc @weaverryan

As in: Won't server need write access to this folder now?
Also, can we assume server should just simply have write access to `var` dir? Would fit eZ as we need to add folder for binary content for instance, but would like to reuse as much as possible of default doc.

ref, changing:
```bash
$ sudo chmod +a "$HTTPDUSER allow delete,write,append,file_inherit,directory_inherit" var/cache var/logs
```
to
```bash
$ sudo chmod +a "$HTTPDUSER allow delete,write,append,file_inherit,directory_inherit" var
```
 *(In all cases)*


**Nitpick**
Add new line between line 26 and 27


@itsjeyd nit/question: what's the reason behind moving and renaming all these methods?


Huh. We still need to talk about the interaction between negative impls and specialization. It seems kind of wrong though to be able to specialize a positive impl to be negative, though. One wrinkle would be cross-crate, though the orphan rules render it moot just now. Consider if this specialization appeared in some other crate, but you had code like:

```rust
fn foo<T>(t: TestType<T>) {
    let (tx, rx) = channel();
    ...
    tx.send(t);
}
```

from what we can see locally, this is perfectly legal, but in fact if `T` is `SomeDownstreamType`, it would be unsafe. Except that the orphan rules would prevent that impl, but still, something feels a bit fishy.


Fixes #10
Fixes #9


@artur-shaik  should this be removed?
`echom g:JavaComplete_PomPath`


we must add some additions here:
When you join a group you got via the Dungeon Finder you will be on Dungeon Finder cooldown, preventing you from joining another random group for the next 15 minutes (from the time you were matched with a dungeon). There are the following exceptions:
- You got kicked from the group
- The end boss (or whoever gives the reward for joining a LFD group) of the instance has been killed


This isn't correct. isalnum(), depending on locale, may accept locale-specific characters (like "ą", "Ó" or "ü"; see man 3 isalnum for further info), and we don't want such characters here.


seems like we should extend this test to check that, when you call `go`, you get the right code. or perhaps that is intended to be covered by other tests?

(in general, it can be nice to add a comment stating exactly what you were trying to test)


right, it should sleep for a short time


@arthurzurb Thanks for a progress update, nice work :+1:


**Nitpick**
Use double quotation marks for attribute selectors or property values, e.g "Lato"


never mind, I see the other tests now


Indeed..


ありがとうございます。修正しました。


I fixed it locally but it was not pushed, let me check what happened. Thanks


не делай так.
отдельно проверка на сабмит, отдельно проверка правильности.
при непрохождении валидации нужно предусмотреть отображение ошибки.


I think it'd be helpful to draw the specialization tree here in a comment so we can see more clearly what scenarios are covered vs missing. Looks to me like you got everything though.


The goal of the new structure is indeed to have the whole var folder writable (using a dedicated folder is precisely meant to simplify this). The doc may have missed this detail when updating to the new structure. Please open an issue on the doc repo.


You have a point. I've been thinking about this the fact only portions of functionality are needed by the different components of k8s. The network client is not needed by kubelet as far as I can see, neither are any of the LoadBalancer or Router configuration options.
At least for the client what about a lazy initialisation with a getter and a map of providers in the Openstack struct?
```
var SUPPORTEDCLIENTS = map[string]bool {
    "network": true,
    "compute": true,
}
type OpenStack struct {
   ...
   clients map[string]*gophercloud.ServiceClient
   ...
}
func (os *OpenStack) getClient(string) (*gophercloud.ServiceClient, error) {
	if !SUPPORTEDCLIENTS[clienttype] {
		return nil, ErrUnsupportedClient
	}

	client, ok := os.clients[clienttype]
	if !ok {
		client, err := initialiseClient(clienttype)
                if err != nil {
                        return nil, err
                }
		os.clients[clienttype] = client
	}
	return client, nil
}
```


I also had the same problem. https://github.com/roundcube/roundcubemail/commit/558a6db27ea6c8b50a113c96c65408566a626807 fixed it.


Can become `else` if you like.


Sometimes I wish we had `Either` too.


You are not 100% right -- `v:count` works for commands -- try `3:Ag ... -> :.,.+2Ag` -- then v:count will be equal 3.
But it's not obvious and truly speaking not comfortable. Therefore we return to way it was before.


Makes sense - as i said, I'd much rather everyone get 'politely nudged' into v9 anyway - saw a forum question about a bug in v6 yesterday...  gets frustrating.  I was more just curious as i couldn't imagine anything in the core needing to change to support this.  I also thought you already pretty much owned everything phoenix used top to bottom.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **12/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **18/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **83/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/148) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Mein Desk-Top Github will schon nicht publish-en. Ich arbeite gerade online ein. Und hier ist eine Idee für eine von der IT angebotenen Weiterbildung :) Vielleicht wollen es ja auch andere Clusterianer lernen.


von dir....


`for port in $(seq 9152 9189); do` would be easier to follow


statt Collectors.toCollection(ArrayList::new) -> Collectors.toList())


Let's use pluck here


kann es "null" zurückgeben?


self.package || self.project


These aren't the actual rows, but rather a result-set. It currently contains two keys:

* rows - the acutal rows to loop over
* total_rows - the amout of total rows in that view

Both should be accessible as they are today in the current solution. If possible, read-only, as it's a result, not a set that you can change (at least you can't store the changes).


ありがとうございます。「注意し、」でつなげることにしました。ついでながら、"instead"を「代わりに」と明示的に訳出しました。


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


Sorry, I'm not allowed to create issues in your JIRA system.


All 3 files syntax mistakes fixed now. Thanks!


if only we had enum hierarchies with common fields here, we could factor out this code so elegantly ;)


@lkraav Which version of nghttp2 are you using to build mod_http2.so? 1.6.0 or git head? There have been a lot of commits since the 1.6.0 release:
https://github.com/tatsuhiro-t/nghttp2/commits/master


feels like there could be a helper that factors out this and the previous instances of this message that appeared earlier, to help ensure consistency (the only difference I think is the adjective "conflicting")


WTF is ```shtruck```? more descriptive, pls.


from commit message:

> the available associated type is not "final", i.e. not marked `default`.

I think you mean *is "final"*


Ok


This should be configurable


Ask yami how she configure this one so that you can adjust


The problem here is that you may not have the controller-manager or api-server running in an OpenStack instance. Simplest example: integration testing.
The previous code assumed that all components are running in an OpenStack instance even though this code was only interesting to kubelet as far as I could see.
I was looking for a way of detecting what component the code was currently running in, so the function would only be executed when running in a kubelet, but a way to do that was not apparent to me.

But I could replace the references to localInstanceID with readInstanceID() and use the cached InstanceId in the  readInstanceID function when called multiple times, if that makes more sense to you.


Of course, I was going to suggest you that but the more interesting commit is the one introducing the matrix.stack function. I suggest to merge them all. The "stack" function was missing since a long time, other matrix-oriented application already have that is some cases even bolted into the syntax (I'm thinking to Julia).


This should use $1


I will create a pull request for this fix


Yeah, forgot clean debug code.

On 01-12 (03:53), Wang Shidong wrote:
>    [1]@artur-shaik should this be removed?
>    echom g:JavaComplete_PomPath
>
>    —
>    Reply to this email directly or [2]view it on GitHub.
>
>    Reverse link: [3]unknown
>
> References
>
>    Visible links
>    1. https://github.com/artur-shaik
>    2. https://github.com/artur-shaik/vim-javacomplete2/commit/900f63c88015c08756011459ac50f77cc2062544#commitcomment-15394832
>    3. https://github.com/artur-shaik/vim-javacomplete2/commit/900f63c88015c08756011459ac50f77cc2062544#commitcomment-15394832

--
Best regards,
Artur Shaikhullin


...are... adjacent? It seems you are checking if one line is an extension of the other (not deviating more than one line width). This comment needs to be finished and you might have to take a look whether the comment below at line 122 is still accurate, because it seems that that's what happens here.


seems like we should have a test where you have three levels of impl, each defining the same item, and the impl is default in the grandparent but not the parent.


ディレクトリをかますってどうすることですか・・・？
namespace使ってrootingわかりやすくしたらいいんですよね！


All the previous errors raised in the readInstanceID function are logged. This log was just an assumption that if readInstanceID has failed the code is not running on an instance. But if I replace references to localInstanceID with readInstanceID() this code is not needed here at all.


would be really nice to have a helper for this case I think to avoid repeating this code here and in the default-checking code


@e-kolpakov I wanted to make the naming of methods that do similar things consistent. Out of all the methods that interact with the page/exercise somehow, `send_input` was the only one whose name started with an underscore. That's why I renamed it (and moved it to be close to the other "interaction" methods).


I'll wait and rebase on that PR once it is merged.


done!


@malirod, see 4b493c6


introducing a dependency on EntityManager in pure algorithm class is annoying because:
it's limit the reuse of the function
it's not explicit that applying the function produce a side effect of cancelling the pending modification on part

Another point is that, the algorithm presume the previous revision cannot be itself checked out which is a false assumption.


> No side effects. It just will do nothing if a developer adds render=True accidentally.

Fair enough, but I doubt you'll remember this edge case for every new bit of functionality you add (making sure it's free of side-effects for instance).

What if I don't read the documentation properly, expect `render` to have an effect even if I used late template initialization, and then my application behaves differently in some rare edge case where I expected it to compile the template before serialization?


OK.


sure thing, thanks!


Um...Ok


Is it documented? Any usage examples?


Created: https://github.com/symfony/symfony-docs/issues/6132


It's also much better to compile the regex a single time. So this should be a global variable that we use every time the method is called.


thanks! :+1:


Ah okay, I think the wording confused me. The way it was written it sounded like it would just do 1.1 (because that also supports 99% or 1.0).