Are you sure removing the return value checks in ```ReplaceEntry``` is a good idea? The code will keep searching for the entry to be replaced even if it has already been found and replaced, wasting time...


(?ui) makes this regex case (i)nsensitive based on the (u)nicode standard.


Not so sure my English grammar is better but we better put a firm "."


Read the code I guess that works.


dobry pomysł


European Testing Conference PR approved.


without transition, not without rotation ofcourse


@giacombum

Why have you split Newton-Cotes formulas into multiple module? The only difference should be the weights... it should be possible to express both formulas into only one algorithm, the client will chose  the formula (say 6 or 7) by means of an `init` method. I think this makes the library more concise and clear.


makefile creado


This move goes a bit further than I would want in terms of moving functionality.

In general, the BAO should return a BAO object & it is the job of the api to convert that to an array - so there are parts there that specifically belong to the api.

From a refactoring point of view my reservation is that it is too much at once. - I would probably start by just moving lines 93-103 into the create function & expand from there.

I think the comments on the autoweight are correct & would move to eliminate those lines - but preferably with unit tests to ensure weight is only set where required (e.g a new record) & not altered ad-hoc on update.


We loose a lot of interesting things like skipping certain actions in notification. Is this a big concern? I would rather not have us pick up this change right now.


With the addition of .settings(sanomaSettings: _*) on line 34 of build.sbt, my sbt publish-local now vomits and dies because it can't find sanomaSettings:
error: not found: value sanomaSettings
  .settings(sanomaSettings: _*)

Is there some other dependency I should be including? I have sbt 0.13.8 and scala 2.10.4.


a better approach would be to just make ```epsilon``` a Function (and utilize the fact that we can pass in numbers and function expressions as well).


This line should not be indented, right?


How do I do something like request a specific article id? Does our fetch articles API always give all articles, or you can request a single article?


Ah, OK, no worries - I wanted to check I hadn't unintentionally caused this change. The lack of anything other than v0.10 on AWS Lambda seems to be enough reason for `sharp` to keep supporting it for now. Thanks!


Sprawdźmy ile czasu może nam zająć to co obecnie mamy i jak będzie miejsce to dodajmy to jako dodatkowe zadanie.


I think we have document generation which uses the `example.es6` as the example instead of having usage in the `README.md`? /cc @keithamus


Oops. I thought I removed it.


0xd9~0xda are used by current ocg cards.
Maybe Phantom knights should use other setcode.


Get read of goal. Conventionally, the property name alone is apt. When calling the property with variable name, it'd look like thisGoal.goalName, which looks weird and unnecessary.


This last paragraph is a pretty good explanation, but I misunderstood it the first time I read it.  However David gave an explanation that was immediately crystal clear to me.  You might consider plagiarizing David's explanation.  Here's what David wrote:

The race condition occurs as follows:

1. The coroutine is running.
2. The coroutine is about to suspend, but before it can do so, it must arrange for some event to wake it up.
3. The coroutine arranges for some event to wake it up.
4. Before the coroutine can suspend, that event occurs and the resumption of the coroutine is scheduled on the job queue.
5. Again, before the coroutine can suspend, the resumption of the coroutine is dispatched.
6. Again, before the coroutine can suspend, the resumption code runs the coroutine.
7. The coroutine is now running in two threads.

The lock prevents this from happening as step 6 will block until the lock is released which only happens after the coroutine suspends.


Readability over verbosity. 'Description' is the better property name.


I could be wrong, but we can put this in the devDependencies, right? (Since we do not have rendering code within this library.)


```
&:(first|last)-child {
  .page-link { ... }
}
```
You can group to `&:(first|last)-child .page-link { ... }`. In `scss/_pagination.scss` and `scss/mixins/_pagination.scss`.


Ah now I understand.
This because I want to make sure, that `rm` did work correctly, but I forgot to implement the `else` in the following branch. (candi should not automatically abort at this point).
This can happen if you do not have the rights to delete the file (maybe copied src directory from another user), but setting a `quit_if_fail` after `rm` would be better. I'll fix this.


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


Have you thought about "layout selection"?
Some people like "side", some "bottom" (which for me is a bit worse.. my handlebar hides bottom part of the screen).
Some "default" layouts (left/rigth, bottom) and position editor (with free layout) would be the best solution.


If you alter the form to call the new create then the form no longer needs to do this check. Note however, that it does need to handle the error that comes out of that duplicateField function (if any.)

We have been moving to throwing exceptions on errors - with the form generally having a try-catch & setting the session error message if it catches one.


:ok:


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


This looks good, but `etc/rc.local.linux` also needs updating


`sender()`


This line is not required because the earlier `export class RootContainer` is already exposing `RootContainer` as an key of the module.


> What happens if hubot resets in the middle of a long running operation?
It will get beck online, connect to stream and will handle the next update event as if it was originated from the same instance. The only problematic situation is when execution gets updated during hubot downtime, but we can solve that by introducing `last_id` mechanics to stream.

> We loose a lot of interesting things like skipping certain actions in notification.

We can add another option to context that would say whether this notification should be skipped. I would very much like to see some specific use cases as there's multiple solutions to this problem.

> Is this a big concern?

Depends. I personally don't like us having to make such a long loop through notification mechanics and rule engine and `hubot.post`_result as each of the steps hide more and more data from us and makes it harder and harder to trace what went wrong.

The next change I wanted to make is to implement @emedvedev solution on custom responses and it would be much easier if I would not have to fix stuff in several projects and worry about it being not compatible.

This with this change is that it old way will continue to work, no breaking changes. Maybe just a deprecation notice at some point.


See: https://github.com/danielmoll/component-react-async-container/commit/1c5d2213c85a50017897edaf6bfd0567ec879dae#commitcomment-14097767

And then, do this: `import Impart from '.';`


@danielsdeleo Does this line need to be set at info level? When using chef-provisioning against a chef-zero server I am seeing this line repeat itself 100+ times in a row but doesn't affect chef-client run at all. Can we move this to debug log level?

`INFO: Unable to access cache at /var/chef. Switching cache to /home/poliva/.chef`


Hi @rkanter, do we have a public JIRA to track this issue?


note from ned to ned: below test multiplying q by ones array (or vice versa). Does it work for floats, lists, variables, vectorvariables?


oh, oops. I was intending to use select('g.alertHolder').selectAll('rect.alertBounds') as the g.alertHolder should be unique. thanks!


Quinn is right. There should be 1 user who has many triggers. Although, Many triggers may be associated with with many journal logs. Therefore, The Trigger and Medicine should have a foreign key of user_id and there should be two pivot tables for (journal log and triggers) and (journal log and medicines).


Formatting with "x2" already produces lowercase. You would use "X2" to produce upper case.
(And ToLower() is locale sensitive unless you pass a locale!)


This check is to change value scope settings. the "is_global" can have 3 values : 0 (store view),1 (website),2 (global)
The problem is that with "select" attributes of configurables, the field itself is "global" but it can accept "per store" values.
So i think my code is having the right behaviour: forcing store scope for values for "global declared" attributes in case of configurable attributes.


Hi @drewbrokke, I still don't understand why do you add the one with primKey==0?

When we check it we never use 0, but always the primary key of entity to get the companyId.

In case, you are going to rewrite the permission check somewhere to use always primKey=0, and when we create primKey=0 for every company, the check won't work and the protection is useless.

Did I miss anything?


Nit: some wonky extra whitespace here.


Note that part of my reservation is that you want the form to also be calling this same function & you want to check that the function works for the form too - hence I would try to consolidate a small amount of functionality like the checkDuplicate thing first & test through the api & UI (Preferably adding an api unit test).


My bad... just realised it's a function returning a promise.

However, I would like `route` to receive `props` as that will make it nicer:

`route={({ articleId } => (fetch(\`http://url/to/resource/${articleId}\`)))}`


Dear Inquiring Minds,

This was an error. Fixed in a later commit. (It had serious consequences when more than one data context was present, because the slopView would end up in the prototype.)


@kelly-shen There is a bug here: if `query = None` (the default if `query` is not present in the `request.GET` dictionary) we'll get an error `AttributeError: 'NoneType' object has no attribute 'lower'`. You should change the default to be `''` so the default is an empty string and we won't get the error.


@poliva83 can you file an issue instead of committing on a comment? This commit was just a `git mv` to extract this code to a new gem, I didn't actually write most of it. And also commit comments can easily get lost in the huge amount of github email Chef maintainers get all the time.


@amyxzhang should we also check against a user's email here?


:green_heart: is no problem will do it. merge on master - is there this 9.0.4.0 release anytime soon. if not I can merge it. but first it has to be  :green_heart:


once this is only called from the create function on this class it would be good to change the visibility to protected - that tells people the function should only be used from this class (which is what we want). This function looks good.


This caused a "No registered oauth service found for:" for me....
I spent several hours on the web trying to figure out why and I ended up forking the package to add some debugging. I found that the Accounts.oauth.serviceNames() does not return anything even if the service has been registered client side. So to make it work, I had to call Accounts.oauth.registerService in both client and server. Is it normal ?


Тебя не смущает что копипаст во всех сервлетах?


Про остальные ивенты. Убрала, потому что не используются. Из личного опыта обычно события перед и во время практически никогда не понадобляются. Но если надо - я верну.


Fixes  #1928


or simply `and album.artpath:` instead of `and album.artpath is not None:`


LOL I FUCKING KNEW IT


Done


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1425 tests passed)

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


My bad.. didnt realise this will stop changes to example file from being updated in the source repo


You are right. it's mistake


It could be .. In v1 it's simple assignment like here. If you want I can change it to copy.


Do you see any value in this if-block? It seems to just be checking the id is valid to me & that is something we don't do in the api as a pattern (we decided early on that if something would cause an sql error on save we would not add additional queries to pre-check but instead attempt to add extra info when it failed. The comments block seems to concur.


This new is also not needed as `MarkdownReporter` is a case class. I will change it.


`PropTypes.oneOfType([ PropTypes.string, PropTypes.func ])` is technically more correct as components can just be `'div'` ...though in this case that would be weird.


Sorry, I thought a commit was missed.  Obviously it was just WS change, that should have just been undone.


I guess these lines update the uf-group - ok they definitely belong in the BAO! Does the form duplicate these too?


Revisions to the License section and added terms for mobile Add-ons.


I think this should stay '@'. All of the attributes should be one-way passed in and not be changed in any way by the directive. (Except for the data arrays like you pointed out before).


Duplicated `the`


As of SUPEE-6788 PATH manually escaped fields are no longer allowed (APPSEC-1063)


These line belongs to the api - although we may be able to replace it with a call to the api basic_get function once the BAO is fixed


This line should go in the BAO really


Yeah let's change collectFirst. However, let's hold off on adding option
variants of everything. It's so easy to do a pull optionally that I don't
think its worth it.
On Fri, Oct 30, 2015 at 10:50 AM Sheng Chen <notifications@github.com>
wrote:

> ok, but how about collectFirst ? Should it return I as well ? In that
> case, a collectFirstOption is needed too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scalaz/scalaz-stream/commit/0f75d83426c61ee29ae077705f673f222dac2362#commitcomment-14095420>
> .
>


arguments in the global scope is wrong, as there is no such variable defined in the global scope. It is also not needed for typing arguments inside functions, having the constructor should be sufficient. If some typing was lost in the old type checker because of this change, it's a bug in the old type checker (which we likely won't fix). The new type inference (use --new_type_inf to try it out) types arguments correctly.

http://closure-compiler-debugger.appspot.com/#input0%3D%252F**%2520%2540param%2520%257B...number%257D%2520var_args%2520*%252F%250Afunction%2520f(var_args)%2520%257B%250A%2520%2520var%2520%252F**%2520number%2520*%252F%2520n%2520%253D%2520arguments%253B%250A%257D%250A%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_TYPES_NEW_INFERENCE%3D1%26MISSING_PROPERTIES%3D1


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (838 of 838 tests passed)

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


Thanks!


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


It's just that I prefer to leave double quotes for JSON and HTML attributes.

Plus, I save a lot of keystrokes by not using the shift key :-P


Really these are the lines that are giving me the heeby-jeebies! (through to about line 92)

I'm not clear yet whether they should be shared with the form - or perhaps just live in the checkDuplicateFunction - it's my discomfort with these lines that led to my comments above mostly.


Was it causing an error in the tests? I believe we send an optimistic continue so the server can start sending the next batch while we start parsing the previous batch. Otherwise parsing has to happen first and that can take a while. I believe the server is OK with receiving extra continues for a finished cursor


You have more than deserve it! :) Thanks.


Let's add a card for supporting JS unit tests, but manually test these now. Then :shipit:


The current patch reads now as:
```
+            # Remove the file and check if that was successful
+            rm -f ${ARCHIVE_FILE}
+            if [ $? = 0 ]; then
+                cecho ${GOOD} "corrupted ${ARCHIVE_FILE} has been removed"
+            else
+                cecho ${BAD} "corrupted ${ARCHIVE_FILE} could not be removed."
+                cecho ${INFO} "Please remove the file ${DOWNLOAD_PATH}/${ARCHIVE_FILE} on your own!"
+                exit 1;
+            fi
```
Thx. for pointing that out :-)


:shipit:


Thanks, I'd like to merge it back to my own fork too and then perhaps we can re-do the formatting changes. Right?


@ssendev ah, `yay` vs `jay` :trollface:

I think there needs to be a little cleanup done on state-tracking, but otherwise thanks for the input!


`images/styles.css`? Go home @StanCallewaert, you're drunk! Did you test this? ;)


The only way would be to clone my fork and use that, sorry. :disappointed:


would it have been easier/clearer to extend File rather than have a one-to-one field? the serialisers might be simpler that way?


> Depends. I personally don't like us having to make such a long loop through notification mechanics and rule engine and hubot.post_result as each of the steps hide more and more data from us and makes it harder and harder to trace what went wrong.

We wanted to leverage the notification mechanism and provide controls so it was a conscious choice to go the circuitous route.


My bad, thanks.


Guess this should be `#if defined(__cpp_rvalue_references) && __cpp_rvalue_references >= 200610`


`route` should be `PropTypes.func.isRequired`.


Primera subida de isma


Зміни в цей бранч потрібно заливати через пул реквести і асайнити для перевірки мене або Марину


I think you are right. The default can be kept at 60 seconds and it would be great if there was a CLI option to override timeout, but it night not be necessary.

Anyway, I would like to understand it better. By default there should be no timeout on reading the stream, so it is strange that it does timeout when reading without the loop.

If I understand correctly `ReadTimeout` is checked for each `Read` call separately, so with 60 seconds timeout it should be enough to call `textReader.Read(readBuffer, 0, readBuffer.Length);` once instead of running `Read` in the loop. If you run it in a loop, then the actual timeout will be `ReadTimeout` times the loop cycles. If the loop is kept, then we can adjust the count of bytes we read on each iteration, so that most requests will have length less than this count. This way we could set a lower timeout value, so that most of responses got read in one iteration and will timeout earlier, if there is client hangs or something. But longer operations still will work, as the timeout will be basically reset on each iteration.

Does it work with a timeout set to 60 seconds and a single call to `Read` or `ReadToEnd`?


Эй! У меня от этого коммита постинг на всех бордах сломался.

Скрин из треда.
<img src="http://iichan.hk/b/src/1446222207011.jpg">


Лиса 41.


`assert " " not in item`


Some typos in the description


I reviewed PR and good with changes.


typo? MAGE_FROST***


You patch for spells working good! Thanks for work :+1:


Yeah, you're right. Per the instructions in the commit message, it should have been aligned with the start of the text for the enclosing bullet point, not with the start of the code fence for the code block. It's interesting to note that it does nevertheless render "correctly": https://github.com/gradle/gradle/blob/7bf773aacaff10a43d3fdc3176d655673bac76cc/design-docs/jdk9-support.md#story-java-library-sources-are-compiled-against-library-jar-resolved-from-maven-repository, but this is really just because neither markdown nor GFM not support indentation-based code blocks within bullet lists—which is the source of this whole formatting silliness in the first place.

So technically, yeah, it should be outdented by a couple spaces, but in practice it happens to work.


Better `#define ... 1` and later `#if` instead of `#ifdef`, as done with PICOJSON_USE_LOCALE.

Also, add `#ifndef PICOJSON_USE_RVALUE_REFERENCE` around the whole setting, then it can be overridden from outside ;)


develop (pflaumingo) - Win64-Windows-7-VisualStudio-12: OK (2325 of 2366 tests passed)

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


I merged the PR too quickly, before I realized it was being compared against the master branch. All PRs are merged first into the development branch, then merged into the master branch once everything is compiled, tested, and ready for production with the next release.


LGTM


Подозреваю, что опять из-за этого Symbol проблемы вылезли.


wth


can you pls explain what is " Slug " ?


This commit breaks the use of `@ngenerate` and `@nextract` when extending a `Base` function and is causing error in [`DataArrays`](https://github.com/JuliaStats/DataArrays.jl/pull/173#issuecomment-152512938).

Minimum repro can be found [here](https://github.com/yuyichao/explore/tree/5de306ba54673636f2ed962e8cfee9a7c1f7c60f/julia/nexpr) (run the `nexpr.jl` script with `Compat` installed, it will fail by default but will be fine with `--compilecache=no`).

```
yyc2:~/projects/julia/master
yuyichao% git bisect good
15a4a21b309d01f7bc68bf1f96b3fcea26e3fddc is the first bad commit
commit 15a4a21b309d01f7bc68bf1f96b3fcea26e3fddc
Author: Jameson Nash <vtjnash@gmail.com>
Date:   Tue Oct 27 15:13:39 2015 -0400

    avoid modifying the linfo during the add_p2c pass so that the result is indempotent from the perspective of codegen

:040000 040000 8d31e70dbb174aea1adccf56c2bce054d5c18ba42e67dd73083fe33e883005994a017bcb05e20d9f M        src
yyc2:~/projects/julia/master
yuyichao% git bisect log
git bisect start
# bad: [a36bb6ddf585f57abed6885500a1bdc25126aee4] Merge pull request #13821 from stevengj/isempty
git bisect bad a36bb6ddf585f57abed6885500a1bdc25126aee4
# good: [e5c6964a497a71fb940117530c1867ddd71f4c67] bump VERSION to 0.4.0-rc1
git bisect good e5c6964a497a71fb940117530c1867ddd71f4c67
# skip: [7aa333b749bbba1df10c0a90719b0ed3374cb1df] added `checkout` to Git REPL
git bisect skip 7aa333b749bbba1df10c0a90719b0ed3374cb1df
# good: [54f898cb1e0dc49964004e34f0eb70fa7f0d94fa] NEWS for #13387
git bisect good 54f898cb1e0dc49964004e34f0eb70fa7f0d94fa
# good: [3ca40e3e095ec7b63a2cfe716b682ca57b5af84f] fix ssh tunnel cmd and warnings in paralel test
git bisect good 3ca40e3e095ec7b63a2cfe716b682ca57b5af84f
# good: [e255b5abf8124f5001ded0f28f1e43f0e2d6b641] fix indentation of llvmcall docstring
git bisect good e255b5abf8124f5001ded0f28f1e43f0e2d6b641
# good: [d0b97f358244e278ffcaf378f9871a94522d3328] Merge pull request #13445 from JuliaLang/jrg/complextype
git bisect good d0b97f358244e278ffcaf378f9871a94522d3328
# bad: [361cf34baf707337928292f5638cd5d2d7faea43] Merge pull request #13778 from JuliaLang/anj/arpack
git bisect bad 361cf34baf707337928292f5638cd5d2d7faea43
# good: [ae903954636cc7544713d09f5af0a0b1b971ece2] block compilation of lambda objects that are not valid for compilation
git bisect good ae903954636cc7544713d09f5af0a0b1b971ece2
# bad: [15a4a21b309d01f7bc68bf1f96b3fcea26e3fddc] avoid modifying the linfo during the add_p2c pass so that the result is indempotent from the perspective of codegen
git bisect bad 15a4a21b309d01f7bc68bf1f96b3fcea26e3fddc
# good: [aee6a26310c82f1f7b3c5bdd9f96cad2a0c14645] share compiled anonymous function between specialized version of the outer function
git bisect good aee6a26310c82f1f7b3c5bdd9f96cad2a0c14645
# good: [cdfe62a65f181cc0c2e69dba75e50d09ae4bb10b] give code_lowered/code_llvm/code_native a one argument form
git bisect good cdfe62a65f181cc0c2e69dba75e50d09ae4bb10b
# first bad commit: [15a4a21b309d01f7bc68bf1f96b3fcea26e3fddc] avoid modifying the linfo during the add_p2c pass so that the result is indempotent from the perspective of codegen
```


@shaun965 please do not update Gemfile.lock. There are updates here that break the app and a bunch of extra dependencies that seem like they are unnecessary.


@vtjnash


already tried, I cannot ride the drake even if I cast 48600.


Doesn't make any difference wrt running anaconda from git b/c top_builddir isn't defined and the script assigns it the value of top_srcdir.


Radiobuttons в формі чекаються одночасно, а такого не повинно бути, бо це поведінка чекбоксів


Of course this will have to be changed in order to the "nilearn team" account on appveyor.


This looks like both access.log and error.log are going to STDOUT. I'd prefer if error.log went to STDERR.


Hey, you have to set CONFIG_CRYPTO_LZ4=y too if you want commit https://github.com/Megatron007/ghost/commit/fbd883071f98266312c7d5de01c35c660fdbbca4 to become operative.
regards


Yes I'd like to spend some time, probably after WIN, working on the doc generator, maybe getting the READMEs to be generated. For now it might be best just to say something like:

> Refer to `example.es6` on how to use this.


I think you have a very good point that the request-response loop is unnecessarily complex. I am motivated to have this conversation slightly differently where we talk first talk about the expected outcome and make sure if switching the approach proves to be the right approach.


Did you take care to only define the version-constraints we require with the lowest version which satisfies our needs (Instead of just using the latest version of everything)?


Додай ще декілька опшенів в селект #forms


@shaun965 this commit includes unnecessary local changes to database.yml. Please be careful and thoroughly check the files you add and commit.


LGTM


FYI: `super()` is sufficient in python 3, but it's hardly big deal


Hola Eduardo, acabo de usar tu clase y me he dado cuenta de una chorrada pero que a mí me ha servido, y es el que no se visualice el botón del formulario al llamar al
$pasarela->ejecutarRedireccion();
simplemente poniendo un display: none en el botón:
$btnsubmit = "< .... style="display:none;"/>";

y así no mostraría el botón en ningún momento...

Saludos y gracias por tu trabajo!


TeamCity pull requests :: netty [Build 525](https://garage.netty.io/teamcity/viewLog.html?buildId=2841&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 4704, ignored: 16; resource leak detected (new) Build time: 00:10:51
### Failed tests
```
io.netty.handler.codec.http2.Http2FrameRoundtripTest.stressTest: <no details avaliable>

```


oh i will add it defconfig :) and i forgot to remove lzo also from defconfig which is not nedded now :p


What was the intention to remove an empty line?
Looks like you haven't changed the script parser algorithm.


`jmanual.pdf` をダウンロードせずに見たいよね。


I was looking at so many tuts. Some of the code came from gugamarket framework which looks incomplete, but has a bit of structure. I am gonna finish off JenkinsCI setup so that people can see from the web if tests are passing. I don't know how that works for feature branches, but it should have options for that.


В селекті повинно бути можливість обрати декілька варінтів, а зараз тільки один


I don't think that amperage can be controlled. Only monitored. If it goes over max value, then it should HW stop or software should reboot. With higher voltage and same amperage you get higher power. Our PMIC has only fixed voltage rails. So no amperage control AFAIK. For freqs higher than stock 2.5GHz, you'll need to increase voltage exponentially due to exponential increase in power consumption. Minimum steps should be 25mV-35mV.

Transistors only need higher voltage to have more stable switching in high freqs. Higher currents can displace them physically within cores and cause CPU to die much more quickly.
I made amperage control for LP kernel for 8974 platform but it didn't change amperages. I mean, it intercepted and changed the value in kernel variable (tested), but nothing happened. It's probably only for knowing how much current is supplied to PMIC on higher voltage side - 3.3V (at least that's what my electronic class taught me). Seems like driver only loads amperage value and passes to some register on chip. But I don't know what our PMIC chip does with that value. Newer PMIC's have current monitoring capability in uA range.

Do you know if LP kernel does anything with that value? I noticed that sometimes kernel can crash if value set too low.


Thanks


Додай ще декілька опшенів в селект


Не, просто ошибка копипаста.


comment


Исправил.


```
@Documented
@Inherited
@Retention(RetentionPolicy.RUNTIME)
@Target({ ElementType.TYPE, ElementType.METHOD })
public @interface DirtiesContext {
```

`@Inherited`, hence redundant here.
Or do you observe anything wrong here?


А, posts вместо post


1. Lets pass the `props` into the route.
2. Lets grab `route` from within `this.props`.
3. What if `route` was not passed in or was not a function?
4. No need to return the promise as React probably will not care.

```
const { route } = this.props;
Promise.resolve(route(this.props)).then(...
```


does it need an assertion to ensure there is no balance sub-model?


@ted23p I use jenkins at my work and we have branchbuilds and master builds. Its pretty easy


104921238_EnergyPlus_Performance_Framework (lefticus) - x86_64-MacOS-10.9-clang: OK (1939 of 1950 tests passed)

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


тут не потірібно обертати в пустив div


700 isnt a thing.. right?


I agree. Route should get the initial props given on instantiation - from then it could also decide if it needs to make any fetch call at all.


Груба помилка - в інпутів немає атрибута name


Use more descriptive names for the succes and failure callbacks. You will find yourself needing more than one succes or failure callback in a controller and it will get messy.


@cvrebert https://github.com/twbs/bootstrap/commit/1dc8583bc5622463218b75c19c986765dce8798d#diff-4ecf4573af0e4e35dfa80e3a683bba70 this is what I meant...


`File.objects.count()`?


Overall, good job @ted23p! The models and everything else really needed to be cleaned up and separated


What's `eslint-disable-line` disabling here?


init project master


Hey Josh,

The two tests that interfered with eachother are:
```
             TestCounter++;

             {
                 //JavaQuery, range.yaml, #6
                 /* ExpectedOriginal: [0, 1, 2, 3] */
                 var expected_ = r.array(0L, 1L, 2L, 3L);

                 /* Original: r.range().limit(4) */
                 var obtained = runOrCatch( r.range().limit(4L) ,
                                            new {
                    }, conn);

                     assertEquals(expected_, obtained);
             }

             TestCounter++;

             {
                 //JavaQuery, range.yaml, #9
                 /* ExpectedOriginal: [0, 1, 2, 3] */
                 var expected_ = r.array(0L, 1L, 2L, 3L);

                 /* Original: r.range(4) */
                 var obtained = runOrCatch( r.range(4L) ,
                                            new {
                    }, conn);

                     assertEquals(expected_, obtained);
             }
```
The JSON traces for these two queries are:
```
TRACE Sending: Token: 4, JSON: [1,[71,[[173,[]],4]],{}]
TRACE JSON Received: {"t":2,"r":[0,1,2,3],"n":[]}
TRACE Sending: Token: 4, JSON: [2]
TRACE Sending: Token: 5, JSON: [1,[173,[4]],{}]
TRACE JSON Received: {"t":16,"r":["Client is buggy (failed to deserialize query)."],"n":[],"b":[]}
```
Soon as I get "Client is buggy" the server forcibly closes the connection. Almost seems like the server ***doesn't*** like that optimistic continue. :(

Do these two tests pass in the Java driver? Perhaps I'm missing something when I'm sending the `continue` to the server so the server can can be okay with the optimistic continue.

I have one more question: if we know the first response is `SUCCESS_SEQUENCE`, by definition, the query is done right? ***I think*** there's no follow up continue when the query is done and the response we get is `SUCCESS_SEQUENCE`. Why do we send `continue` (optimistically) when the query is already done? Doesn't it just make the wire more chatty when the server already said `SUCCESS_SEQUENCE` is a finished query?


Pow block also kills all OnGround NPC`s on destroy.


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


Thanks!


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


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


я обернула, бо тут видавало помилку по валідації


This should go in the ScreenShotHelper protocol, with an isVisible:Bool.

e.g. delayedScreenshot(tab, force: !viewIsVisble)
takePendingScreenshots(tabManager.tabs, force: !viewIsVisible)


http://kaskade.dwds.de/germanet/?l=Messinstrument und anderen http://kaskade.dwds.de/ -Link noch rausnehmen, ebenso Verweis auf Leitner.


@jguepin, agreed. I appreciate the feedback.


Yea, but I wasn't implemented necessary functions to apply that, therefore I left Pow block without killing onGround NPC's. Later I will add that


`view.bounds` wasn't working for OSX.


Can you create a newline after line number 200.


Il y a une faute d'épellation. Ça devrait être "Ménard" et non "Maynard"


Same as above. you can use > successCallback / failureCallback


Does this need to be settable?


Has to be pointed out here that [our `readyState` is different from Relays](https://github.com/facebook/relay/blob/cc7c0e5b16999e045937a5f75a4ef0fe05b4695e/src/tools/RelayTypes.js#L36). Will that be okay? I guess it is an implementation detail, but perhaps we should leave a comment mentioning this at least so anybody digging about realises this.


I kind of suspected that. Although, I don't have an idea of how migrations work. Like we talked about for tests, force: sync is working for the test database to drop tables before starting a test suite. There may be faster ways to clear the database though. I need a command to check what async methods are yet to finish or be cleaned up because there seems to be some delay before exiting after tests have ran. BTW, you get 4 columns for taskTags: createdAt, updatedAt, tagId, taskId.


Here's from what I understand in the RethinkDB driver docs:

> *2 SUCCESS_SEQUENCE: Either the whole query has been returned in r, or the last section of a multi-response query has been returned.


I don't think you are using is_root_pe(). Left over from development/testing?


We don't use slashes in the routes. Also when building an URL consider that, for example, in a post call we want to target a list instead of a single element so it's better if you use plurals for them.
Use this instead
```Restangular.all('classes').all('Rents').post(...```


Hi!

No, sorry, that line should not have been there. It's removed now. Please try again.


You should use 2.0.2.


Yup.  Left over from testing.


Can the redirect be a partial URL for potential local testing?


Hmm, in the runglade target only GLADE_MODULE_SEARCH uses builddir, the rest use srcdir.


Nathan


Can you redo the changes and then comment the feeds app from INSTALLED_APPS in settings.py ?


ss


opps, i will change it, thx


why add out in invalid? It seem useless. The if on valid_equivalent at the start of this method will trigger, so it don't need to be in invalid. In fact, I think it would be wrong to return None, it would be better to recompute it if it get removed from valid_equivalent.


ggg


ggd


@fchollet  Did you mean dot_axes = max(len(shape1), len(shape2)) % min(len(shape1), len(shape2)) ?

dot_axes < 0 means dot_axes = -1, which means the argument is not provided by the user. The only values that makes sense for dot_axes are positive integers and tuples.


This tuning was stalling out machines...and actually was making the website itself very slow.


Not sure what you're asking. `private(set)` means it can be only be modified privately.


Can you add the comments below the methods that you are defining so that it will be more verbose to others ?


whoops.


Showing Jagrit how comments work.


can you add the sepcific version of pyJWT that you are using ?


Можно писать без скобок:
```position, unit = ...```


More of a placeholder because we shouldn't send full errors to public, but it's kind of a thing: http://documentup.com/joho/7XX-rfc which was linked from https://github.com/for-GET/know-your-http-well/blob/master/status-codes.md#extensions


I think we'll be writing a new sample with gcloud-ruby


yeah, you're right, removing the extra ones


I have reinstated it with low throughput.


Good point. The package that should be upgraded are:
* `commitplease` (as we use it to check the commit template)
* `grunt-contrib-jshint` (to lint the source code)
* `grunt-jscs` (to check the JQuery style code)

As there is a lot of improvements and bug fixes, for others, I'm just fan of using the latest versions (even if it's not needed).


Do you need most of the directory separators that are left? PHP has standardised on "/" for all paths it uses, and will convert to whatever the OS needs when the path is used. The exception is when passing paths into a OS shell command, but that is something seldom done.


no particular reason, didn't see the need for an extra line there


Good Idea...


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


update comment to explain


@KrauseFx Oops. Really sad that I messed up again.


why assign null first instead of just final MessageDigest md = MessageDigest.getInstance("SHA-256");?


Oh: you're tightening its visibility. Good work, carry on. :)


Тут вместо `.format` написано `/format`.


This is ok but both `/dashboard/user` and `/dashboard/5874` match.
By using `url: '/dashboard/{bookId:int}' the router interprets it as an integer.


TeamCity pull requests :: netty [Build 526](https://garage.netty.io/teamcity/viewLog.html?buildId=2842&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:08


feaLib: Not quite yet, but I’ll try to finish it asap.


typo


Hey was there a reason you never got the read/in commands working on CP2103? I see those functions are still blank.


I didn't use this approach from the beginning because this `getStartupScripts()` is used for `Cluster` configuration as well as for `Session`.
See the usage of this method.

My intention of these `init scripts` was only to initialize the specific `keyspace`. And only `session` is for that.
I admit that it looks like a bug in the Spring Data Cassandra, because we are sharing the same scripts for the whole Cluster and the specific session for particular `keyspace`.
Maybe I don't know Cassandra so well, but I'm afraid that we may have some side effect to go this strange way...


101162956-IncorrectIDDError-4647 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1960 of 1961 tests passed)

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


As of beta 8, it looks like the default configuration file is appsettings.json rather than config.json from previous betas.


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


fixed in 5498570e2cb6a9b7065b7dc592c076b9b3125962 (julia-debug simply asserts on this error making it easy to track down)


WIP{'severity': 1, 'debug_msg': '', 'line_nr': 3, 'origin': 'LineLengthBear', 'file': '/tmp/tmp2clh33/setup.py', 'message': 'Line is longer than allowed. (99 > 80)'}


WIP{'severity': 1, 'debug_msg': '', 'line_nr': 16, 'origin': 'LineLengthBear', 'file': '/tmp/tmp2clh33/setup.py', 'message': 'Line is longer than allowed. (99 > 80)'}


[Fixed link](https://github.com/facebook/Relay/blob/537ccaaa92781403379ff9a16df707f2d8a4fa6e/src/container/RelayPropTypes.js#L22)


damn. fixed.


Np. Just for readability.
And yes, to be sure that parser algorithm is good :smile:


this should use reference_uid(add_str("$@arenamembers"), c)


You could do this block in one line:
```fortran
depth_weight(i,j,k) = max(G%bathyT(i,j) - CS%Z_int(k), 0.) / (CS%Z_int(k+1) - CS%Z_int(k))
```
but I also suspect you meant this:
```fortran
depth_weight(i,j,k) = max( min(G%bathyT(i,j), CS%Z_int(k+1) ) - CS%Z_int(k), 0.) / (CS%Z_int(k+1) - CS%Z_int(k))
```
which gives 1.0 in open ocean above topography.

Since `CS%Z_int(k+1) - CS%Z_int(k)` is constant over `i,j` and you are normalizing by the the global sum of `weight`, you don't need the division:
```fortran
depth_weight(i,j,k) = max( min( G%bathyT(i,j), CS%Z_int(k+1) ) - CS%Z_int(k), 0.)
```


Holy grail, that's awesome!


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

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


Thanks for the advise on this,
To be honest, my initial approach to Connect() was what you mentioned as the first option, which is

**If it is to create a connection and return it for you, then it is fairly ok, as this function is a factory function for you. This function could probably be renamed to createDbConnection() or something, to make it more understandable.
Then class name should be changed to DboFactory (database object factory). So, we would have a factory which will create a db object for us. Additionaly, functions from this factory should be probably changed to static, but it is not nessesary, as you may have this factory**

However having read both your approaches, retrospectively I think the 2nd approach would be sensible, because by that way I could just create a object of  **$databaseObject** and utilize it when necessary, as opposed to my first approach I'll have to keep calling the Connect() function when necessary...

so should I go with the first approach or second approach? which one would you advise....shall I stick to the 2nd approach is that ok proceeding with this approach?


`Relay.render` actually [seems to work slightly differently from this](https://github.com/facebook/relay/blob/d3c2ad4e4e923901b083b2f7b3aced1ee8009271/src/container/RelayRootContainer.js#L123-L138).

This will render `<div />` if nothing matches, while Relay will return `undefined`.
This will render with `renderLoading` if `readyState` is `'loading'` even if there is `data`, while Relay will not render if there was `data`, etc.

It might be nicer to change this to something more similar to their logic - though it probably only really matters if you start to implement what they did with cache and ready states. YMMV


Just drop these. ;-)


This is great! Should we merge this commit in its own PR?


В это сообщение и ниже можно писать сам `code` либо даже `http.client.responses[code]`, чтобы пользователю была понятнее ошибка.


ah, definitely makes the config clearer.  Thank you!


Nice - I need to update my image build environment anyways...


Let me test and debug it locally t obi sure what's going on.
Plus I'm sure we should have some testing against **real** Cassandra before merging to `master`.


Pour la 0.85, PHP 5.4 n'est pas obligatoire


Hmmm. I think it'd end up being more complicated. The main complexity of the serializer is really around the transaction list - the balance aspect is relatively minor. On the other hand, and correct me if I'm not seeing this correctly, making a separate sub-model for FileWithBalance (or whatevs) would result in a multiplying of serializers, views, and URLs.


My bad. Forgot about those lines.


I was seeing tabs open in background screenshotting successfully, but it failed when you bounce back and forth from the TabTray.

Also what is the performance when you:
 * restore hundreds of tabs, or
 * open up 10s of tabs at a time.


New component rather than append '_xyave' to the variable name. Needs discussion...


at some point i configured a global .gitignore for convenience.


```_.merge(storedUser, _.pick(paramUser, 'firstName', 'lastName', 'email', 'username'))``` should be enough

merge will put the values of the second param in the first param. you don't need to reassing ```storedUser```


@adrianschroeter you had me remove this when I initially introduced this feature because you feared that we would run out of sync with the backend. Now you have introduced the exact same code...


ModifyTimeBinsToUseRounding (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1955 of 1955 tests passed)

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


Bitte füge den Quellcode zu Deinem Repository hinzu und schicke uns einen neuen Pull request, die binäre Datei nützt uns nichts :sunglasses:


Ah sorry, habe eben gesehen, der Quellcode ist ja da, alles also gut :white_check_mark:


fair enough! yeah there would be separate views/urls, but i think default serialisers might be able to handle it, i.e. i think you don't need custom serialiser subclasses for simple models (not that i've tried). doesn't really matter either way :)


@lakruzz it's not about tag pages, it's about main menu. I fill it with links to all pages, that have title. It'll work correctly until you want to make a titled page without a link in main menu


**bf.jpg


You are receiving a third parameter you are not sending (The failure callback in this case)


I went.


@reefdog Thanks for understanding. lol! Nice memory...sounds like a plan.


Missing a `title` attribute like all other toolbar icons - e.g. enabling 'Show columns' has a `title` of 'Columns'


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


Correct.  And it won't be the "adams" sample that's currently being used.


Thanks.  My editor must've forgotten tabs vs spaces for Go for some reason here... :-(


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


Perhaps a better semantic name for the class would be `SonyCiCachedResponse`. The purpose of the class is to do the following:
  1. Provide access to `Rails.cache`, which afaik, is just one big key-value store for the whole app.
  1. Provide a naming convention for the cache keys that will store values from a Sony Ci response.
  1. Provide convenience methods for specific values of a Sony Ci response.
  1. Encapsulate the logic that retrieves those specific values from Sony Ci.

An instances of `SonyCiCache` acts as an accessor into `Rails.cache` for a given `ci_id`.

Class methods can be added for operations that involve multiple cached responses.


I totally forgot this was here (as evidenced by its old commit timestamp).

Either way - feel free to delete this branch if it's not useful.


Is it possible to get the swapchain from Ex, or can you get only a swapchainEx ?

If yes, then perhaps we should pass whether the device is ex or not, instead of false.


Error 404: failure callback not found :)
Use more descriptive name here too


Why `entry0`?


Had no idea. That's cool.


@lakruzz it will be shown on the blog index page. Any part of page above the tag <!--break--> , not only caption. Will that do?


The problem is in this line, it can't load the score file or can't put the scores in the arraylist scores.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


deleting


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


@lexicalbits please review.


Will this line ever be reached?


Not right now, but it would be used for any other cached Sony Ci value. And I thought `sony_ci.proxy_src` read nicely. But it can be removed if it's premature.


If i apply
 "links": {
    "Help/Support/Bugs": "https://github.com/glpi-project/doc/issues"
}
i have this error
Daux.io Config Error:
The theme you set is not not a valid option. Please use one of the following options: 0, 1, 2, 3 or learn more about how to customize the colors.


Yeah... fail.


Don't think that will work -- we need to know the updated `viewIsVisible` state *after* the delay has happened. We can't pass in `viewIsVisible` then wait, since the value can change.


This has a different capitalisation to the original. Does it work?


Npm will automatically use newer ones, we need to define the oldest ones which work in this file (since you use the `^` operator instead of fixed versions)


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

```


Just a note

When "gst_tag_list_get_string" returns False(tag is not present), value should still point to where it pointed before, i.e. not to start address of allocated memory, so long as we don't use this pointer any more we should be safe.

So in this case it shouldn't be necessary to set value to NULL, but I agree it's good approach.


no Bridge network available..
WHY..??


LineLengthBear detected a problem when runningsection pythoncheck!

Line is longer than allowed. (99 > 80), severity: 1.

(File: /tmp/tmpx7pzgg/setup.py, line: 3.


LineLengthBear detected a problem when runningsection pythoncheck!

Line is longer than allowed. (99 > 80), severity: 1.

(File: /tmp/tmpx7pzgg/setup.py, line: 16.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`vendor.css` will be concatenated with whatever is passed into `processCss`.

Generation of `vendor.css` is controlled by the `bower:css` task.


fadb21951107f562766e91d5711222d73f2b71d7
我的想法是遊戲王卡片數量龐大，難免有少數必須當作特殊的例外處理的卡片
目前的做法是把這些特例的卡片都統一記錄在constant.lua裡面，要查找也比較方便。

會想回收單一卡片佔用的flag是因為我查到讓lua能支援int64的方法都比較複雜
如果繼續使用uint32的話，flag就是比較有限的資源，似乎應該保留給較多卡片的類別


@mikepetersonccv Message is encoded in the UI, but StackTrace isn't. Does it need to be?


> Npm will automatically use newer ones, we need to define the oldest ones which work in this file (since you use the ^ operator instead of fixed versions)

I'm aware of that, I just forgot to remove the caret.

So, which packages I should upgrade and which not ?


That's a nice commit bro


LESS processing tasks now depend on `bower:css` having completed


Shouldn't we be using latches here instead of sleeping?


The links contain 'issues' in 3rd element their path (plural).


@ssakash
/home/gregory/playstation/emulateur/pcsx2_merge/pcsx2/x86/ix86-32/iR5900-32.cpp: In function ‘u32 scaleBlockCycles_helper()’:
/home/gregory/playstation/emulateur/pcsx2_merge/pcsx2/x86/ix86-32/iR5900-32.cpp:1172:2: warning: ‘scalarHigh’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  );
And potentially some coverity issues are related to it.


The ```determineBranchOffset()``` function did not deal with negative offsets. There still appears to be an encoding issue where negative offsets are encoded as 0, probably due to using ```unsigned int``` types and then converting to ```int16_t```


Separated bowerTask out into separate JS and CSS tasks, to keep things clean and extendible.


Htmlspecialchar escaping


Htmlspecialchars


void SetEquipmentSlots(bool bLoadDefault, int32 iMainHand = EQUIP_NO_CHANGE, int32 iOffHand = EQUIP_NO_CHANGE, int32 iRanged = EQUIP_NO_CHANGE);
With the current restriction on the number of fields, I can not do it.
EAI can be, because has 3 uint32 fields, db_script has only two uint32 fields.


New task - similar to `bower:js`, grabs and moves vendor css libs from `bower_components` as required


This breaks Objective-C apps such as https://github.com/layerhq/Atlas-Messenger-iOS that load Atlas via Cocoapods. Try downloading that project, pod install, and you will get the attached error.
<img width="258" alt="screen shot 2015-10-30 at 10 24 11 am" src="https://cloud.githubusercontent.com/assets/2124234/10852873/630b815a-7ef0-11e5-90b4-e117ce5b9e52.png">


Если у пользователя что-то случится с коннектом в интернеты или stepic затупит, то этот запрос кинет исключение, и скрипт грязно упадет.
Как минимум в консольных тулах надо бы без проблем доползти до вывода хелпа. Так что эти вызовы нужно вызывать только тогда, когда они действительно нужны.
Очень советую посмотреть и, возможно, заиспользовать красивую библиотечку для создания command line тулов click: http://click.pocoo.org/5/


Is this image copyrighted?


I will leave that to you. Use newer versions of those required otherwise just introduce the caret operator in the others


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


This is fine but once you start using the token for other calls you should set it in the defaults header (So it's used in the other requests) (https://github.com/mgonto/restangular#setdefaultheaders)


all of those should be fixed on my next update to the overclock function where I'm going to try using a more suitable approach instead of these scalar variables for scale blocks function result.


> I was seeing tabs open in background screenshotting successfully, but it failed when you bounce back and forth from the TabTray.

Yeah, that'll happen because of the 100ms delay. When you return to the browser, then quickly hit the tabs button again, we'll once again try to take a screenshot after the view controller is in the background. Shouldn't be a big deal though -- we'll just try again the next time you return to the browser.

> Also what is the performance when you:
>     restore hundreds of tabs, or

This PR shouldn't have any effects on session restore since this is about going into the tabs tray. But session restore shouldn't be an issue since we only restore the last active tab; all other tabs are restored lazily when they're selected.

>     open up 10s of tabs at a time.

Not sure what you mean here. How do you open "10s of tabs at a time"? Do you have any particular steps you're thinking of?


Cherry-picked into #32


我觉得你这边需要调整下，```ant-spin``` 都用 prefixCls 代替。


Раз мы ловим все эксепшны то можно не создавать эксепшн, а рейзить последний пойманый с контекстом.
```
try:
	tmp = problem_url.split("/")
	tmp = tmp[-1]
except :
	raise Exception("Doesn't a correct stepic address.")
```
см http://stackoverflow.com/a/24752607/2625715


This makes sense. See #47.


Okay, so I need to change the commit message after submitting to upstream.

On Fri, Oct 30, 2015 at 12:02 AM Patrick Ohly <notifications@github.com>
wrote:

> Yes, including the "Upstream-Status" comment in the patch sent to
> *upstream* would be weird. But this version of the patch is included in
> the *layer*, and there it makes sense. First you submit upstream, then
> you add the information where to the patch, and the you add to the layer.
> We follow the recommendations from
> http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines#Patch_Header_Recommendations
> and there it is clearly documented that "Upstream-Status" goes into the
> patch header itself.
>
> Consider scanning the layer with legal compliance tool like ProtexIP. All
> you are shown is the content of the patch. All additional information
> (commit message, pull request comments) is no longer available. Having to
> hunt for the information causes considerable extra work.
>
> A similar argument applies when going through all patches in a layer with
> a file browser or "less".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tripzero/meta-intel-iot-security/commit/0e444d40080e6dd0f3d971b0c53aa5a2a8720423#commitcomment-14086229>
> .
>


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


What is the reason for renaming the file?
singleGeneDeletion.m → singleGeneDeletion_old.m


Muito bom Alisson! Tens uma iniciativa muito boa, sempre indo bem além do que é solicitado, continue nessa pegada. Ézunmonstru na programação nêgu!
ps: na próxima poste seu trabalho na intranet como todo os outros colegas, isso facilita muito a minha vida :)


Why forcing regenerating the UI definitions before building the GUI?
Generated code should be committed to git and you should not force regenerating it before building.

For example, it will simply break using `go get` to install coy:

```
/tmp % export GOPATH=/tmp/gopath
/tmp % GTK_VERSION=$(pkg-config --modversion gtk+-3.0 | tr . _ | cut -d '_' -f 1-2)
/tmp % go get -u -tags "nocli gtk_${GTK_VERSION}" github.com/twstrike/coyim
# github.com/twstrike/coyim/gui
gopath/src/github.com/twstrike/coyim/gui/ui_reader.go:87: undefined: testDefinition
```


It also requires having `ruby` on travis to release binaries.


Why you think that ? Maybe I am wrong but according to this: http://www.contest-lighting.com/en/index.php?cat1=l10000&cat2=l18000 it make sense for me :)

Let me know your decision, I can change it if you will want to :)


Please, start leaving changes in a pull request instead of directly commiting to master


Having arrived this far you could also `yield '/usr/bin/docker:/usr/bin/docker'` so the container can use the proper docker binary version, else users have to keep them in sync and worry about it ;)


Well, I disagree on this, and actually the current code with '@' isn't working at all, if you use use-zero-min-value="false" it still uses zero min value, because '@' makes that "false" a string and thus evaluating "false" results into truthy. It can be changed to evaluate as a string instead (don't see much sense in that) but using '&' will evaluate it properly as a boolean, with no penalty, unless we wanted to update it on the directive without updating it on the parent scope, which I don't think is a requirement here.
Actually, I think this should be applied to all boolean attributes, but I'm changing them as I "touch" (and get bit by) them.


Right on. I guess I didn't quite understand what `multi_details` was supposed to be used for. If you'll look in `SonyCiCache#proxy_src` you'll see that it's still being used, since I just copied from `MediaController#show`, however it's only passing the one `ci_id`.

A class method should be added to pull back the 500 at a time.


Looks good. Have you verified if all the three components(group, name and version) are compulsory for an artifact?


@azturner yep, I think anything StackTrace is rendered in the UI it needs to be


Perfecto


Shouldn't be 'brought up'?


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


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


Why double check for null is required there ?


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


Jupp stimmt!
(Die ganze Datei ist bisher nur eine Kopier von core/form/form.php, ändert sich aber noch. Die escaping-Problematik betrifft aber somit auch core/form/form.php)


> >    open up 10s of tabs at a time.

> Not sure what you mean here. How do you open "10s of tabs at a time"? Do you have any particular steps you're thinking of?

Ok; so tens of tabs open (e.g. browsing a Twitter TL on a slow connection) would be needed then hitting the tab tray.

STR:

 0. Use a very slow connection, or the link conditioner.
 1. Go to your twitter timeline.
 2. Quickly open many links in a new tab
 3. Hit the tab tray
 4. Dismiss the tab tray
 5. Scroll through twitter timeline.

Expected: smooth scrolling
Observed: jank, because multiple screenshots are being taken.

> Don't think that will work -- we need to know the updated `viewIsVisible` state after the delay has happened. We can't pass in `viewIsVisible` then wait, since the value can change.
I guess I'm wanting a `ScreenshotHelper` to be a concrete object that should track this state, instead of the BVC (that's still the case?).


Why do we need this? `RustCompositeElement` is `Navigatable`: https://github.com/alexeykudinkin/intellij-rust/blob/master/src/org/rust/lang/core/psi/RustCompositeElement.kt#L8

Or is this a puerly stilistic matter?


Massimo, sorry for this, but I really don't know how to make right settings for this type of device. It's with 2 bulbs and each have only tilt channel and one master pan channel. And EFX without this settings I wasn't able to make it properly. Any advice ?

it's this one: http://www.jsfrance.com/fr/lyres-beam-club/16376-contest-double-lyre-evora-duo-beam-2r-lyre-beam-a-rotation-continue-neuf-2100000169238.html

http://www.jsfrance.com/fr/lyres-beam-club/16376-contest-double-lyre-evora-duo-beam-2r-lyre-beam-a-rotation-continue-neuf-2100000169238.html


Right. And it wouldn't make a difference except when using make check from a separate builddir, but it would be nice to keep that working.


No, that would make `else if` look like `else { if }`


Nope - this didn't fix the issue.


here is the Conflict

for the compile error


My fault, sorry... really dont know ... will remove it :)


It's just a matter of readability releasing you from the necessity of checking the whole inheritance chain.


Thanks for updating @bkoval :)


This kind of generated files are helpers for the package authors, not clients.

> Go generate does nothing that couldn't be done with Make or some other build mechanism, but it comes with the go tool—no extra installation required—and fits nicely into the Go ecosystem. Just keep in mind that it is for package authors, not clients, if only for the reason that the program it invokes might not be available on the target machine. Also, if the containing package is intended for import by go get, once the file is generated (and tested!) it must be checked into the source code repository to be available to clients.

https://blog.golang.org/generate

Adding them to repo is common practice. I remember using this approach back in my Eclipse MDA days (EMF, Ecore, Xtext, etc). Every change to the "template" was committed together with the modified generated files.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3045](https://teamcity.mesosphere.io/viewLog.html?buildId=66189&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3046](https://teamcity.mesosphere.io/viewLog.html?buildId=66191&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3044](https://teamcity.mesosphere.io/viewLog.html?buildId=66190&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3047](https://teamcity.mesosphere.io/viewLog.html?buildId=66192&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


Extraneous spaces.


Also, is it necessary to call wfilename()? Won't that happen automatically?


Probably should make this private.


maybe have:
```"sign_task": functools.partial(sign_task, pvt_key=self.pvt_key)```

then you can allow templates to specify valid_for


Good to always be thinking about XSS... just keep this in mind: "encoding should only happen just prior to being sent to the browser:" as per https://github.com/SparkDevNetwork/Rock/wiki/Security-Considerations#regarding-antixss-efforts


This is a different style for breaking up a long line, that is all. The && operator can be at the end or beginning of a line. It is more common in TrinityCore to find it at the end.

For example, all of these ask if condition AND condition2

    if (condition1 && condition2)

    if (condition1 &&
        condition2)

    if (condition1
        && condition2)


Impressive


> Use newer versions of those required otherwise just introduce the caret operator in the others

Make sense, I'll update the PR


That's correct. This PR moves the caching and retrieval of Sony Ci values into the `SonyCiCache` model. Now that it's in the model layer, there's no reason to involve a controller.

If we still want to provide stable URLs for Sony Ci content, we can continue to use the `MediaController`. That's why I left it here. But if we no longer need them, we can get rid of it.


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


I'd include what they return if it doesn't match as well.


参考: https://github.com/mozilla/pdf.js


Let's steal some of Relay's `README`.

```markdown
# component-react-async-container
> A Container that fulfils its Component with a Promise. In the style of Relay.

**AsyncContainer** is a React component that, given a `Component` and a `route`, attempts to fulfil the data required in order to render an instance of Component.

## Goals

- [x] Similar interface to [Relay.RootContainer](https://facebook.github.io/relay/docs/guides-root-container.html) to help with the transition to it later on.

```

See [Guide: Root Container](https://facebook.github.io/relay/docs/guides-root-container.html).


In this case, I think it was an artifact of IntelliJ, when I split it out to a variable and was doing something before with it.  It's safe to directly assign. I think I just forgot to clean this up.


back


thanks ... there seems to be some commits going in this week (not just truffle) so I thought little stuff such as these changes can get in. hold it off if you think that's best - mostly wanted to do some backports but no hurry.


There don't appear to be any errors but I haven't tested it...

On Friday, 30 October 2015, Daniel Thomas <notifications@github.com> wrote:

> This has a different capitalisation to the original. Does it work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alistairstead3408/dtg-puppet/commit/dba44d3059f12fb5020f721eb8ff2f607c27741f#commitcomment-14099138>
> .
>

--
Sent from my iPhone


Adding a macro the *prelude of the standard library* is quite a weighty operation, so I don't think it's one that should be taken lightly. I think that the use case here is legitimate, but it's important to emphasize the use case in question here along with the alternatives. Some thoughts of mine are:

* C has the default behavior of raising `SIGPIPE` (on Unix at least) when a write to a broken stdout is encountered, and most programs don't handle this signal so they're terminated. This is an option for handling this, at least.
* For handling the error, is it really worth having a whole new macro? The standard library already provides means like `write!` and `std::io::stdout` as you've found which can allow you to implement the `try_print` macro yourself or even overwrite `print`/`println` locally.
* Providing this macro would be a [violation of our (adhoc) style guide](http://doc.rust-lang.org/nightly/style/errors/signaling.html#do-not-provide-both-result-and-panic-variants) where both result and panic variants should never be provided.
* When I investigated this in the past, all other languages (other than C) raised an exception and printed an error message, matching the semantics Rust has.


seted => set


Can you fix the alphabetical order here? This should go before `_checkLayoutsLastRow`.


In the old version when the boss alien tried to shoot at the exact same moment it was killed a null pointer error occured.(this was fixed for normal aliens but not for boss aliens). but with this extra check this error doesn't occur anymore.


Adding 10 hashes to the tree for better distribution, technically probably not needed, but this should spread the hashes around better.


This doesn't belong here! Looks like you typed in something wrong to rails generate.


Use codecs?


TeamCity pull requests :: netty [Build 528](https://garage.netty.io/teamcity/viewLog.html?buildId=2844&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5240, ignored: 21 Build time: 00:15:46


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


You need to put the semicolon back otherwise the build fails :)


Testing in-line commenting


見るだけならレポジトリに置いとくだけでも


Why is this commented out?


in the old version level 16 would be created and then it would be checked if the levelnumber is greater than 15. this would lead to a crash because level 16 had null pointer errors. now the check of levelnumber is made before the level is created.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (769 of 769 tests passed)

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


@mccalluc keep in mind, I didn't change the logic here very much at all... I just moved stuff to places that seem more appropriate. Mainly, I think that retrieving data from either a cache layer, or getting them from Sony Ci is still just a data retrieval step.. which typically is the encapsulated in the model layer... so I made a model for it.

From there, I was able to avoid overwriting `CatalogController#index`, the `catalog#index` can go back to relying on `document` to have the correct values, and we may not even need `MediaController` for redirection at all.


> Expected: smooth scrolling
> Observed: jank, because multiple screenshots are being taken.

Note that multiple screenshots will be taken when returning from the tabs tray only if those tabs tried to take a screenshot while BVC was in the background. You'd likely still see the smooth scrolling jank if you didn't do steps 4/5 because we'll be taking those screenshots either way.

You could argue that the 100ms delay when returning from the tabs tray could amplify the delay if we timed it perfectly such that all of those tabs weren't done loading before we hit the tabs tray button, then we waited a sufficient amount of time for them to all try screenshotting while BVC was hidden, then we returned to the browser after that for them to retry. And that will generally happen only if the user is only a slow network as you mentioned, and if they decide to open lots of background tabs at once. That's a lot of "if"s, so I doubt is worth spending any more time optimizing unless we have a clear fix in mind.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3049](https://teamcity.mesosphere.io/viewLog.html?buildId=66198&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3050](https://teamcity.mesosphere.io/viewLog.html?buildId=66199&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3052](https://teamcity.mesosphere.io/viewLog.html?buildId=66201&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3051](https://teamcity.mesosphere.io/viewLog.html?buildId=66200&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:20


I don't think this will work. A.bind() creates a new function that is bound to the context you pass to it, so it will be different from the function you've attached when `_addFieldsChangeListener` was called. Can you add a test case that makes sure this works? It basically needs to change the layouts and then check if changes are still being caused by this event on the layouts that were removed.

The right way to do this is to store the handles returned by the calls to attach listeners by `_addFieldsChangeListener` in an array and use it for detaching, like we do on `FormBuilder`'s destructor with the line: `(new A.EventHandle(this._eventHandles)).detach();`.


In the old version pressedleft and right wouldnt be cleared between levels. this means that if you pressed left when the last alien died you would continue to move left in the next level(even if by than you stopped pressing left). Now this doesnt happen anymore.


Is it possible to pull this into 2.7? We can't upgrade at this time but would like the same functionality.


I just added it https://github.com/clappr/clappr/commit/a84eb3ebdb16371d07d68a5f77de4f73f9482400


Stimmt doch nicht, `$element['label']` ist das ganze Label, also mit Markup


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


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


Done!


I played around with this and it works very nicely. A few comments:

1. "Once the learner submits an answer n times" is a bit unclear; it could be taken to mean "the same answer n times." I suggest rephrasing as "Once the learner has made n unsuccessful attempts."

2*. Create an exploration, enable Fallbacks, set up an interaction and create a fallback with a feedback text. Click on the preview tab and click back. Change the fallback's feedback to something else and click "Save feedback". Click on the preview tab and click back. The fallback's feedback will have reverted to its initial value. A similar thing happens to the fallback's destination state (but not to the number of attempts required for it to trigger). This kind of forgetting also happens when you attempt to save.

3. If a fallback's feedback is blank then it's displayed as "<[getShortSummary()]>" in the one-line preview.

4. Arrows are not displayed on the state graph to represent paths via fallbacks. I think they should be (perhaps in a different colour). It's possible you want to require there to be a non-fallback path to the end, but at least you should not issue warnings for states that are only reachable via fallbacks (currently you do).

5**. If I delete the state to which a fallback goes, then it continues to go to that state. Also I can save the exploration like this and the backend does not object. If I then attempt to play the exploration the javascript crashes.

6. When re-ordering fallbacks, if I drag them slightly outside of the fallback region then the start appearing way above the cursor, and this persists through any subsequent attempts to drag them.


Hey! I just saw this fork from our @controlshift ruby-salsa_labs. Would definitely welcome code packaged up as a Pull Request -- it looks like we just implemented very similar code for supporter actions.

I'm also interested in doing a major v2 rewrite to fix some of the architectural issues with this gem... but can never seem to find the time to do right by it.


getIcon string:${item/getURL}/@@images/image/icon for even more clearness =)


@davywentworth you don't appear to be getting this value from the api anywhere.


@miminar what do you think?


Thx. I can confirm that indeed fixes the issue for both of my minimum repro and the original issue in `DataArrays.jl`.

.... Totally forgot about `julia-debug` .....


That is hilarious


Why you decide return back BOM bytes (EF BB BF) at begin *.p files ? I removed it before for ru lang. That bytes cause error while use gen_voice.sh for generate config.p file for not-tts voices (.ogg based):
`Interpreting the ttsconfig.p prolog file...
ERROR: d:/=programs/mingw/msys/1.0/home/egor/1/work/g_config.p:2:2: Syntax error: Operator expected`
I think that you should use "Unicode without BOM" codepage while edit *.p files.


LGTM


You should be able to tweak box padding to do this I think.


yeah, will remove it then.


@davywentworth not worth changing, but just FYI you could set the default for the `monitor` option to be `'window'` (a string) and then just use `getElement('monitor')` and it'd work without this ternary.


Thanks.  2.0.2 has @OkHttpClient for me but no longer generates the Retrofit_* classes.  I looked in the generated directory and searched for all Java files too :-(
I switched back to 2.0.0 and the Retrofit_* classes are there but it's missing LoganSquareConverter...not in retrofit.converter :-(


bad method name


do not use shortcut for variables, I fixed that


What is it? Why it is here?


:beers:


use message instead


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1376 of 1376 tests passed)

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


Typo "us" (use) introduced here?


you don't... this is why I said our own version. Because I think VS requires underscores in front of it or something.


events method should be here


Minor: "brought to you buy" makes me feel like I'm reading a commercial.  Maybe "serviced"?


empd (nmerket) - x86_64-MacOS-10.9-clang: OK (1952 of 1956 tests passed)

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


Sort files by name
Done, please review


why not just `options.affixAtElement.top.element = api.getElement('ajaxElement')`? You would need a lot less code here. Going all the way up to the top of the function, you could get rid of the options object and just do:

```javascript
var options = {
  affixAtElement: {
    top: {
      element: api.getElement('ajaxElement'),
      offset: api.getAs(Number, 'ajaxElementOffset')
    }
  }
}```


Perhaps we should capture the previous value of `autoCommit` and set it back in a finally here?


I don't understand the reason for these variables?


is this a warning? Is this not an error state?


+1 w/ comments


Maybe should check success. I'm not sure if this is strictly needed, though.


you're effectively creating this reference every time the event fires. If it'll never change I'd put it outside the function.


This needs to depends on HADOOP_VERSION instead of a hardcoded hadoop version


I don't think you want this. Behavior runs on domready (load has already occurred). I'd just invoke it; `refresh();`


probably need to `.close` the statement after this


in which case you wouldn't need this. This function (the filter itself) Is invoked when apply is first called.


@yilongli So did kompile finish normally? Did you use krun or kjrun.sh?


save_locator method name will be much better


TeamCity pull requests :: netty [Build 530](https://garage.netty.io/teamcity/viewLog.html?buildId=2846&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5240, ignored: 21 Build time: 00:15:04


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


Make it optional?


Whats the reason for removing the normalization?


Ahh yes, I should have figured that it was referring to gnu compiler collection instead of c compiler in gcc. Fixed


ㄴㄴ.... 네트워크 연결은 하나의 액티비티에서 쓰이는게 아니므로...Activity로 만드는게 아님..


Your right, reverting.


Thanks Aaron. I spent a while searching for something like this but eventually gave up and rolled my own.

I didn't know about the Concurrent library, this is excellent.


Minor: The `.bind` here isn't necessary, since you are already using the `self` variable rather than `this`.  Will change when I merge.


I would save in original method, and in this method just build_locator_by_type


This is pretty haphazard error handling, probably should be improved - but that's a nice exercise for the enthusiastic among us.


Yes I was surprised as well, eventually I got it from PPA since it was faster. Added a note about it


Ok, but you have the log level of trace below with the name logging-debug above. It looks like the previous version was called -debug, but also had the debug logging level, so it seems like if you want to keep it -debug, you should change like 246 to debug.


super bad method name


looks good


merged.


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


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


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


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


TeamCity pull requests :: netty [Build 529](https://garage.netty.io/teamcity/viewLog.html?buildId=2845&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5240, ignored: 21 Build time: 00:26:51


я про «Седона - не аттракцион для туристов, там течет своя жизнь», а не там где №1)


space after `if`

fix else format
```js
}
else {
```


I think we have some mistakes in our data. When we keep the normalisation, we get to values in some of the windows that we looked at in the range of 0.0001 - 0.0003. I'll go through all the collected data over the weekend.

We need to have sane data, and training program that works reliably on the inputs.


ну, например, через bg к `.header-image`) или через псевдоэлемент)


Does this work if the code under test throws an unexpected exception? It seems like you'd hit this code before `clojure.test` could catch it and fail the test.


If we're hooking into clojure.test for this, do they have some kind of output context we can write this to?


You guys remembered to update your databases with the new SQL?


> then we waited a sufficient amount of time for them to all try screenshotting while BVC was hidden
This tips my worry over the edge of irrelevance for me.

> That's a lot of "if"s
Agreed. :)

> Don't think that will work -- we need to know the updated `viewIsVisible` state after the delay has happened. We can't pass in `viewIsVisible` then wait, since the value can change.
Please could you file a followup bug to put `viewIsVisible` in `BrowserScreenshotHelper`, and update it from BVC?


Hey Matt,

First of all, thank you for the contribution! I think you definitely have a good idea here, I'd just like to suggest some updates:

- The above changes mean the module will only be usable in the PowerShell ISE, not regular PowerShell. That's not a huge issue right now, but eventually we want the same module to be usable in a Visual Studio version of this add on. So it would be good to remove that requirement by removing the PowerShellHostName requirement from the module manifest, and wrapping the line that adds the add on tool in a try catch, so it won't cause an issue if the module isn't run in the ISE. This should allow it to get the benefits you've provided, without forcing an ISE-only module.

- ReadMe.md should be updated to remove the uninstall/install-azureautomationiseaddon parts, since it will be no longer needed.

- Since uninstall/install-azureautomationiseaddon are no longer needed, they should be removed from AzureAutomationAuthoringToolkitInner.psm1

Let me know if you think you can get this done today -- if not I will try to make the changes. We want to publish a new version to PSGallery today and I'd like to include these changes!


Not long after this was committed, I got the following error
![fatal](https://cloud.githubusercontent.com/assets/9807615/10854272/b5ae6d24-7f08-11e5-8e52-f503126a5f25.png)


Joe,

I'll try to get it done in the next couple hours. What's your timeline on publishing?


I should think we do...


Bug: Mis-aligned custom top headers columns


Thanks, I like your suggestions. I had the ruler set to 72 chars, but I guess I should have had it set to 69, since an implicit tab of 3chars is added to everything.

Anyways, these 3 commits go together, if approved:
* (1a5c031cc4356974e74e99f0b8e85abd419f30cb) Make pseudo-code more code-like
* (f55dd67f8f203e951b2849a5f556d536ba0647fc) Addressed comments for "Make pseudo-code more code-like"
* (c6a3b1f703aedfaa26b84b001ff64aa51e2f61da) Addressed comments for "Make pseudo-code more code-like" (round 2)


I think `-cover` produces just a raw number.

IMHO, it would be better to get more in-depth data and an html report. This requires a two-pass approach for each set tests we run. For unit-tests, the command would end up looking something like:
```
godep go test ${GOTEST} --tags=unit -coverprofile=unit-coverage.dat
godep go tool cover -html=unit-coverage.dat -o unit-coverage.html
```
Then in the Jenkins job configuration we could add a link to  the HTML file.
Unfortunately, the tests require separate coverage and reports for unit vs integration

In theory we could also write something to generate build failures if code coverage fell below some value, or we could try something like https://github.com/t-yuki/gocover-cobertura to convert the data to a Java friendly format and possibly use some of the Java-centric tools to do the same


I actually didn't try to get it via package-manager but I went back and cmake doesn't seem to find it, so I'll leave a note to try to get from package manager first and if that doesn't work then compile it


Also, what functionality does the module have in the console? I didn't notice that it was capable of doing anything at the prompt.


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


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


Is there a reason you chose to use 'let' instead of 'const' for these variable declarations? Are any of them intended to be reassigned later on besides 'rgba1', 'rgba2', and 'rule' in 'setLightingStyle'?


See point below.


Changes from Littleton


Commit 17797f0 seems to have broken spacing with IE11 inputs.

Broken on IE11:
![image](https://cloud.githubusercontent.com/assets/11887843/10854301/3c5e97d8-7f11-11e5-8f9b-6e15e2075470.png)

Working on Chrome/Firefox:
![image](https://cloud.githubusercontent.com/assets/11887843/10854331/63b6c67a-7f11-11e5-9d85-092938516be3.png)


TeamCity pull requests :: netty [Build 531](https://garage.netty.io/teamcity/viewLog.html?buildId=2847&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5239, ignored: 21 Build time: 00:15:51


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


Unfortunately we cannot use constants that aren't visible inside the scope of this closure (starting at line 18) since this code basically gets `#toString()`'d and injected into the generated parser's code.

So with that said, I'll just add comments here for what 6 and 5 mean.


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


Late afternoon, PST. I can also do it over the weekend if need be.

The module is mostly meant to provide emulated versions of Get-AutomationVariable, Get-AutomationPSCredential, etc, that work in local PowerShell (ISE or not). It also wraps the ISE add on dll, but that is mostly just to make installing easier :)


We expect multiple start functions for master/node? Probably I don't understand the use case here.


TeamCity DNN JavaScript Libraries :: CI [Build 109](http://teamcity.engagesoftware.com/viewLog.html?buildId=387&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 109](http://teamcity.engagesoftware.com/viewLog.html?buildId=387&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


Sorry, my bad for not having read the added lines in detail. :p


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


Thanks for the explanation. :)


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


Thanks for the explanation, good to know. :)


hi!

are you sure you need htobs(...) there? they are simple C args that should travel in whatever is the native endinanness of the platform, no?


I think this should be:
```
attach = !wcpSettings ? true : (typeof wcpSettings.attach === 'undefined') ? true : wcpSettings.attach
```


I agree that `let open Some_monad` is viable, I think that you only need it in _large_ functions. When we have smaller functions, I think that using the specific `Lwt.bind` specifically highlights what we're using from a `let open Lwt`. It also avoids conflicts between different operators.

In general my thinking is 2 steps.
1. Avoid large functions, break things up into smaller functions (which are ideally wrapped in modules that can group functionality).
2. In a function use `let open Some_monad` for large uses of that monad, but cautiously. Use a module prefixed function when there is only one use and/or it is something that conflicts with a common application.


Hi Ken,

I think i uploaded broken packages, i've reuploaded, you can retry the
following command:

./gradlew --refresh-dependencies assembleDebug

2015-10-31 1:58 GMT+08:00 Ken Yee <notifications@github.com>:

> Thanks. 2.0.2 has @OkHttpClient for me but no longer generates the
> Retrofit_* classes. I looked in the generated directory and searched for
> all Java files too :-(
> I switched back to 2.0.0 and the Retrofit_* classes are there but it's
> missing LoganSquareConverter...not in retrofit.converter :-(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yongjhih/NotRetrofit/commit/8831958f709e69cd61b7b4da4d9aaa3ad2364b5e#commitcomment-14100315>
> .
>

--
Nicknames in internet: yongjhih, yjchen, andrew.chen, taiwandrew
Interestings: Vim, Linux, Debian, Drupal, Android.

Key fingerprint:
1024D/85EE7D38 E74A FD7A EECA 469B 5633  9523 976B 436E 85EE 7D38
4096R/01C54D62 3D32 9839 AE0E 0A9F 21CA  761C F5EE C1A8 01C5 4D62


Gotcha, I'll get it done before you need to publish. Thanks!


Thanks for the heads up! We aren't actively working on the fork, but I'd be happy to send code your way in the future.


Ignore, understood when I looked at the next commit


Can we assert the specific error here instead of asserting that the exists in the errors map?


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


Yes, that note is still there (above) for the Windows build


Separate ticket sounds good.


Move to types package?


Do we have a test for this validation?


Had the same issue, but since my app is for iPads only, picked up this plugin fix and it works now. Thanks.


the name of this doesn't make sense to me. It extracts the database name from a subname, right?


Ops, my bad. Thanks!


Also, this section violates the 72 char limit as well (it's 75 chars in the .txt version):
```
      +----------+----------------------------------------+---------------+
      | Name     | Description                            | Reference     |
      +----------+----------------------------------------+---------------+
      | br       | Brotli Compressed Data Format          | RFCXXXX       |
      +----------+----------------------------------------+---------------+
```


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

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


Can shortcut this as `(def ^:private ...)`


Since this returns a boolean, you could name it with a question mark (and probably should so it would be more clear). Also, since it is in `User`, you don't need `user` in the method name.


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


wtf is this style??


You could still use the array, just use `eq` instead of `match_array`. This way works as well though :smiley:


This introduces two problems that are still present (2015 october 30) in current master and in  https://github.com/synergy/synergy/commit/3eaefcfaf366e59bc03c0e517c7959ab90bd6eca

First problem is https://github.com/synergy/synergy/issues/5045 - multithread access to X11 clipboard introces a race condition in reading events from queue.

Second problem is more obscure: thre is a leakage of terminated threads handles that are not closed, because
thread objects created here are never deleted (next commit add waiting for thread termination but Thread is not deleted and handle is not closed).
This leads to problems described in https://github.com/synergy/synergy/issues/4876


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


:smiley:


@digitalprecision sorry, but 2.7 has been EOL for 2+ years: https://groups.google.com/forum/#!msg/puppet-users/8JEy7wY5VPs/9N07fgfU1mAJ.


I /don't/ think we should do this now, but it would be neat if we could somehow name a test db after the test it's being used in.


I think this can break the players if you have more then 1 player on a page, have you tested it with:
https://github.com/jaruba/node-vlc-multiscreen


c.Universe should let you do that.


This should be wrapped in a `describe "#video_in_user_queue"` block.