Add new method getFile


This variable is not available in outbound leg.
`wh_api:428 (<0.404.0>) failed to find key 'From-Network-Port' on API message`


Thanks!


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


We shouldn't have mime types in resource.format, so it'd be better to fix these at source. But this is ok for the time being. How about calling it CKAN_FORMAT_TO_DATA_PACKAGE_FORMAT?


The ```unsafe``` seems to have broken the build for the other PRs, is something missing here by any chance?


The module "compat53" does this check for you: requiring it from Lua 5.3 does nothing.


Do you still need the comments?


Do you want these to go live?


Добавь в третий патч, что после выполнения первой части дерево процессов полностью восстановлено и при старте начнет иполнять код восстановленных пользовательских процессов (throught sigreturn)


Обратите внимание, в лекциях здесь строка
`#pragma once`,
а если создавать заголовочный файл в Code::Blocks, может быть такой вариант:
```
#ifndef STRUCTURE_H  // Может быть другой идентификатор.
#define STRUCTURE_H
...
#endif  // В конце файла.
```
Эти конструкции служат для защиты от повторного включения. Например, если `structure.h` включить в `convert.h`, и оба включить в `main.cpp`, получится, что в `main.cpp` включен `structure.h` (напрямую), `structure.h` (в составе `convert.h`) и `convert.h`, то есть содержимое `structure.h` в программе будет *повторено* дважды. У вас этого нет, но случайно получить такую ситуацию очень легко. Директива `#pragma once` сообщает компилятору не включать этот файл больше одного раза в одну единицу трансляции (файл `*.cpp`), даже если программист (случайно) укажет это сделать.


Hi @timcdlucas ,

Good claiming. However, I'm having issues reading this in. I think because you have forgotten to use the '@@' rather than '@'.

Can you fix?

Tom


Alors est-ce qu'on peut le transmettre via clé USB et le copier chez chacun des membres étant donné qu'on en a besoin ?


An easier way to do this is just to bind your setTimeout functions:

```js
window.setTimeout(this.moveDown.bind(this), 500);
```

The `bind()` method returns a new function that has its `this` value permanently set. It'll save you this shim, as well as not needing to use `setTimeout.call(this)`.


egal ;-)


I know, but if you have not installed it, require fails :)

This change lets you use lua-chacha without even installing compat53.


about SDL_UpdateTexture() : _This is a fairly slow function, intended for use with static textures that do not change often._

I believe we should be using SDL_LockTexture /  SDL_UnlockTexture()


Thanks for noticing!!! :)


Don't worry, I fixed it.


You know that `keys()` takes O(n) time, right? You want to use `iterkeys()`


It seems I found a solution, which has been briefly discussed with @max-kammerer
This solution is implemented in new commit: https://github.com/JetBrains/kotlin/commit/e52c90e5d2ae131bf5fc4e42f60d7124c8909509


You can go to the extra work of making `NetworkOPsImp::states_` an array of `std::string` instead of an array of `char const *`, but at the moment it buys you nothing.  There in only one place `NetworkOPsImp::states_` used:

https://github.com/mtrippled/rippled/blob/5f342f4a568aeccaf3102b5d149d2de26635076e/src/ripple/app/misc/NetworkOPs.cpp#L720.  Here the string is returned by value, not by const reference.  So you might as well construct the string from a `char const *` as from another string.

That said, what Mickey said earlier is correct.  You can remove the `NetworkOPsImp::states_` array altogether and just use `stateNames` in its place.


I'll remove it.


still returning 0 here...

I figure this should be two lines. One that logs, and another one that does "return -EINVAL".


This look ok for me. Need to improve commit description.


same here.


Вы второй раз перехватываете это исключение, см. строку 17.


In PaaS entities, isRuning() says that an application exists and it was started. `postLaunch()` needs that the application is enabled and started. When you deploy an application, the client does not way to the entity will be running.

Probably, I could move `getEntity().waitForEntityStart();` to deploy method. However, it should be a temporal solution and I should use task (futures) for avoiding this issue asap.


4943-EMS-Trap-Unitialized-Variables (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1423 of 1424 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.39%25-green.svg)


I have looked at the code in some detail.  I understand what I am talking about, yes.

The point I was making is that the code changed here is completely orthogonal to the problem.


It is, but it working for a while now. Any suggestion?


Hi Brandon,

Thanks that helped, I think it the computer and our network that causing the slow response. Also I notice the script ignoring the CA = Canada location option because when the script gets to entering the birthday it enters it in USA format Month, Day, Year and of course Canada has to do it backwards Day, Month Year. Is there another setting in the script I might not be setting?

________________________________
From: Brandon Usher [notifications@github.com]
Sent: October-21-15 6:05 PM
To: brandonusher/Apple-ID-AppleScript
Cc: Stephen Joyce
Subject: Re: [Apple-ID-AppleScript] Added iTunes version 12.3 to compatible list (c28b68b)

Yes, there is a line that contains: property masterDelay : 1 that you can change.

Try setting that to 2 and see if that fixes it for you.

—
Reply to this email directly or view it on GitHub<https://github.com/brandonusher/Apple-ID-AppleScript/commit/c28b68bf00c78b1d965436becebdf9c6affb7482#commitcomment-13914517>.

________________________________

This message including any attachment(s) is privileged and may contain confidential information intended only for the person(s) named above. Any other distribution, copy or disclosure is strictly prohibited. If you are not the intended recipient or have received this message in error, please notify the sender immediately by reply email and permanently delete the original transmission, including attachments without making a copy.


`compact` makes a copy, and just calls `compact!`, so no.

http://ruby-doc.org/core-2.2.2/Array.html#method-i-compact-21:
```c
rb_ary_compact(VALUE ary)
{
    ary = rb_ary_dup(ary);
    rb_ary_compact_bang(ary);
    return ary;
}
```


The property `KeyboardEvent.which` is deprecated, [MDN](https://developer.mozilla.org/de/docs/Web/API/KeyboardEvent) recommends using `KeyboardEvent.key` instead. You could also use the strict comparator (and set the opening curly bracket at the end of the if statement line).


I think the width == 0.0 test was ok here - it's the value when width is not set


ㅁㄴㅇㅁㄴㅇㅁㄴㅇ


фіафіафіа


Since you can do both `.Send` and `.Publish` should we use a different name other than "Send" context?

CreateOutgoingContext or similar?


ахаха


Finally. Please add comment above here to commit message description and move restore_freezer_state to 4th patch. Don't merge this whole patch to 4th patch.


아직은 도움이 되는 스킨을 만들 수가 없기 때문에 조금만 기다려주시면 감사할 것 같습니다.


Remember to do this work on the "gh-pages" branch. Right now, you are working on the "gh-pages.io" branch.


+1 for flagging incautious use of "testing" there

Thanks again for your feedback,
Jaime


Should probably remove this line.


Or is a note?


@alwey: I'm happy to see that we came to the same conclusion, and that it seems this problem is solved, at least in theory. I'm also happy to see that currently running test is close to that theory. Of course, the difference on several thousand cores can still be dramatic.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 35](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=376&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


:+1: Awesome!


Is this even used since you just manually added the minified copy to `libs`?


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 35](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=376&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:11


Hey there! Too bad that I can't see your tattoo designs, because the link is broken ;)


tbh i have no clue what the code does


thanks for the reference, good to know. This one was actually my own fault, as I forgot to group by id before getting distinct timepoints, which left me with a single id dataset.


Note that by default serialize use "compact" which mean you will ends up with "page" if you execute that code in the wiki "wiki" and space "B" under space "A". You should pass 'default' as second parameter.


Remove these blocks and make the test (and all the rest of the test in this effort) match the existing house style.


Done (see updated PR).

On Tue, Oct 27, 2015 at 1:02 PM, JJ Allaire <jj.allaire@gmail.com> wrote:

> Okay, will do
>
> On Tue, Oct 27, 2015 at 12:55 PM, Jonathan <notifications@github.com>
> wrote:
>
>> rsconnect::accounts() returns shinyapps.io accounts, too. Since this is
>> used to light up Connect UI we should probably filter those out before
>> counting.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/rstudio/rstudio/commit/430f064a45ed9da44a823f0e1a37aa4b6e1772cb#commitcomment-14016985>
>> .
>>
>
>


No comments other than javadocs in the code.


Yep. Just verify that you either branched this manually off master or ran `git flow hotfix start ...`, then send a PR to master.


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


It's being fixed with PR #4176


`@SuppressWarnings` should be a method level annotation to match house style.  Update all other instances of this.


Please take a look at the recent changes done. Included the animation to the code. Also tell me how to achieve this with mod operator. I feel my approach is hacky and looking for even cleaner approach.


yeah. I can't even tell you how many times I've removed that and committed it. That's the issue with having so many .html files. Yifat and Patrick would end up writing from an old file that STILL had that line. It's been like 6 times.


I see, what about hosting your own custom json of all recent classified/hidden items that gets cached along with the extracted data from the manifest?

Unfortunately bungie hasn't shown any interest addressing the issues caused by these classified items in 3rd party apps.
I was disappointed how the newest manifest update did not declassify the no time to explain so this might be way more common in the future :(


Cool!

Does a BUILD_RELEASE=1 still work after we've added the --gc-sections there?


Done. Addressed it. Good to know.


End


Return should be a casted `this` to match house style.  Update all other instances.


Bring back the todo list, blank lines, and removed comments.


The problem is that I sometimes need the Go type and sometimes the C type written in Go that is the "PrimitiveType". We can do a refactoring for this but the PrimitiveType is currently all over the code.


That would be create!


Never mind, just saw the MULTIBOOT_LDFLAGS up there.


В итоге ни одной img


This belongs upwards, say below the reddit-password entry. Could be better named.


CBECC (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2323 of 2349 tests passed)

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


Вот почему это не будет работать. Метод (по вашему коду) возвращает ссылку. Ссылку *возвращать* имеет смысл только на те объекты, которые будут существовать после выхода из функции (говорят: "время жизни которых дольше, чем выполняется функция"). Конструкция `(Temperature)(*this)` создает копию `*this`, которая будет существовать всего лишь в пределах выражения после `return`. Далее для этой копии вызывается `operator+=()`, возвращающий ссылку на эту же копию. Эта ссылка возвращается, но она ссылается на объект-копию, которы уже уничтожен. Чтобы оператор заработал, возвращать нужно значение, просто `Temperature`.


@leonardodistasio oui, mais non, pas besoin d'aller si loin:

* `.env` est proper à chaque installation. Il contient, à priori, des mots de passe donc c'est mal de le versionner et il faudra le recréer sur chaque machine (plus ou moins).
* `vendor` est produit par `composer` (qui lit le fichier `composer.json`) avec la commande `$ composer install`

RTFM:
* http://laravel-recipes.com/recipes/31/managing-your-project-with-git
* https://getcomposer.org/doc/01-basic-usage.md


Так, во-первых: здесь и далее, в хорошем коде принято предпочитать цепочки методов (`результат = что-то.map{...}.map{...}`), а не последовательный вызов нескольких методов с восклицательными знаками. Смысл такого стиля — в том, что лучше читается исходное намерение, «чего мы пытались достигнуть».
То есть:
```ruby
# было
fields = File.readlines( "movies.txt" )
fields.map! { |e| e.chomp } # вот такое промежуточное состояние fields нам нужно? из кода это неясно
fields.map! { |e|  e.split("|") } # а это нужно? мы собирались что-то делать с ним отдельно?..
fields.map! do |i|  # и т.д.
  { "url" => i[0], "name" => i[1], "year" => i[2], "country" => i[3], "date" => i[4], "genre" => i[5], "duration" => i[6], "point" => i[7], "author" => i[8], "actors" => i[9] }
end

# стало: мы отчётливо сообщаем, что выполним серию преобразований с исходными данными, а потом будем работать только с тем, что получилось
fields = File.readlines( "movies.txt" ).
  map { |e| e.chomp }.
  map { |e|  e.split("|") }.
  map do |i|
    #...
  end
```


pstree_file_name :)


Perfect. Thanks!
Any guess how far away 1.5 is?


The following can be written more elegantly, but I don't mind much since it's constrained to this function.


If throw an exception, sometime render UI is block. It shouldn't affect rendering


Спасибо большое!!


Не будет работать для разных шкал у `temp[i]` и `max_t`. Воспользуйтесь оператором `<`, который написали выше.


Instead of comment, this could be a docstring.


Note, this was "words.IRC" and I've changed it to "status.IRC". I did this because "status.IRC" is what was in the tutorial I'd tested but maybe something has changed, can someone verify what the proper statement should be.


Please do it using format string that you assign to self.core_file_fmt_str


Update this test to match house style, regardless of how noisy it is.


Hi Bettina,

If you are using sonar-runner you don't need that property. You can define your source directory as usual by means of sonar.source on the properties file.

But if you were using maven then sonar.sources is set from the pom to be the sources used by maven, which by default are under src/main/java. Now lets say that coverity found a defect under src/test/java, that would not appear on sonarqube because it is not under "maven sources". That's is why we need to override that property when using our plugin, that is the purpose of the newly defined property.

If you are not seeing all defects in sonarqube, that might means that there is an issue with the path defined on your cim instance versus the one defined by your sources analyzed by sonar. It could also happens that some defects are not found because they are defined by relatively new checkers, which might not be understood by the plugin which is a little outdated in that regard.

Please let me know if you have any other question.

Best,
-Franco

________________________________
From: zabettina [notifications@github.com]
Sent: Tuesday, October 27, 2015 9:01 AM
To: coverity/coverity-sonar-plugin
Subject: Re: [coverity-sonar-plugin] Adding "sonar.coverity.sources.directory" property. (35cab26)

Hi Franco!

I need to understand the meaning of this new property.

Is it only needed for maven?
That means, if I am using the sonar runner, then I do not need it?

I tried to understand the comment of the definition. I have problems with the last sentence:
"The solution for this problem is to add a property that will tell coverity which are the sources that were analyzed by coverity, which my defer from the ones scanned by the sonar executor."
Why do you need to tell coverity about its own source path?
Or did you mean to tell sonarqube?

My current problem with the plugin is that even if the paths are correct, some issues are not imported.

Cheers
Bettina

�
Reply to this email directly or view it on GitHub<https://github.com/coverity/coverity-sonar-plugin/commit/35cab26e1e253437ffc955a2e39e90d2bf276bc4#commitcomment-14015320>.


Ditto here, changed to "status.IRC" from "words.IRC".


thanks @stephentoub


I did the same thing the first time i was trying to select until find select() method. lol, living and learn.


Good. You've fixed many things!


I'd avoid the message, or at least make it easier on the eye by not specifying the link.


В таких случаях (для обозначения названий полей) принято использовать _символы_, а не строки. Строка — это тип для изменяемых, пользовательских данных; сивол — для обозначения названий полей, опций и прочего. И пишется это короче:
```ruby
# длинная запись хеша c символьными ключами:
{ :url => i[0], :name => i[1] }
# более компактная запись, значит то же самое:
{ url: i[0], name: i[1] }

# обращение к значению хеша с символьными ключами:
i[:duration]
```


How is the foldername on disk of that given folder? (Have you domains active in cyrus?)


Items get added to it in SortedAdd, because this is the backing list for the whole collection. I'm passing it in ActionData so I can just do data.List everywhere instead of referencing the backing list directly. Makes it easier to change the backing list.


The double-spacing is intentional.


typ -> type?


If a connection is not valid anymore, is the session cleaned up after xxxx time ?


Thanks. I'll do a pull request without errors one day. Not this day, but
one day.
On 27 Oct 2015 17:08, "Dr Tom August" <notifications@github.com> wrote:

> Don't worry, I fixed it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zoonproject/modules/commit/8ac05d8f162b6e152a55482f9f19f780950c9dcf#commitcomment-14017378>
> .
>


-Wall -Werror please


I wouldn't claim this yet, especially in a warning that cannot be turned off.

I want us to land the implementation of `X <- Y` and make sure that it is is strictly better than `in PLACE BLOCK` before we start emitting warnings.


Это задание можно выполнить без ввода дополнительного массива :) /В большинстве случаев работа с массивами, когда они преобразуются в другие массивы, не требует each, потому что есть более удобные и лучше читаемые методы. `each` — наше спасение на случай, если алгоритм работы с каждым элементом действительно очень сложный; или, наоборот, если элементы надо просто перебрать (и напечатать, скажем, никуда не сохраняя)


I started with `select` but noticed that it didn't work on iOS, that's why I changed. But it ended up breaking Firefox when there's multiple elements being selected =/


Bad indentation.


seems fine


Capital "I" looks like lower case L which makes this look like Aloomalos - can we do AloomaiOS?


aaaaa


aaaaaa


И я позабыл: в пунктах 3 и 4 ведь требовалось использовать коды не `int`, а из перечисления `enum Error`.


I couldn't find a good way to fix this - I'm on it...


ZoL uses name instead of guid due to illumos/illumos-gate@57221772c3fc05faba04bf48ddff45abf2bbf2bd not having been incorporated.


Addressbar should hijack the origin the app is running on. So we only check if the url is of different origin to evaluate it being handled or not


-This is one of the most common ways of handling
-this consistent way to handle a line split
-reads from the next block


1. Вообще можно (и хорошо бы) обойтись без доп.переменной sort — просто взять да вывести результат выполнения операции
2. Когда отбираем несколько элементов с самого начала/самого конца массива, то first/last читаются лучше, чем `[0..4]` — потому что если бы было написано first, то даже краем глаза видно, что «несколько первых элементов», а когда квадратные скобки — то надо остановиться и задуматься, это ж откуда он их отобрал, и почему именно от 0 до 4, и сколько их будет, 4 или 5?..


The missing symbol is shown in the log, like this.

CANNOT LINK EXECUTABLE: cannot locate symbol "_ZN7android6Parcel13writeString16EPKtj" referenced by "/system/lib/libril.so"...

To find what it has changed to, I look in the new library with a hex editor, or just plain text editor, and search for a similar string. Nothing fancy, just plain old detective work. :)

This particular one was quite easy though. There could be other changes that are more difficult.


Terminal on any OS or is an OS implied?


@rubenv This case is duplicated and has a different implementation. See https://github.com/rubenv/angular-gettext/blame/master/src/plural.js#L27


I think a big part of the problem is that you're setting so many timeouts, but you never clear them. That means that it's easy to stack them up and have them "race" each other.

If I were writing this, I think I'd emulate game engines and only ever have a single timeout firing, probably your `checkStatus` loop. Within that function, it should be checking the elevator state at any time, and making a decision based on that, updating the state and setting the timeout for the next part of the loop. For example, we might do this:

```js
var moving = this.get("moving");
var currentFloor = this.get("currentFloor");
var selectedFloor = this.get("selectedFloor");
var doors = this.get("doors");
if (currentFloor == selectedFloor) {
  this.set("selectedFloor", null);
  this.set("direction", null);
  this.set("moving", false);
  this.set("doors", "open");
  //early exit for changing floor
  return setTimeout(this.checkStatus.bind(this), 2000);
}
if (doors == "open") {
  this.set("doors", "closed");
  //early exit while the doors close
  return setTimeout(this.checkStatus.bind(this), 500);
}
//otherwise, if there's a selected floor, let's get moving
if (selectedFloor) {
  this.set("moving", true);
  if (currentFloor < selectedFloor) {
    this.moveUp();
  } else {
    this.moveDown();
  }
}
setTimeout(this.checkStatus.bind(this), 1000);
```

Only one of these setTimeouts will run at any given time, so you don't get race conditions, but they may still pause for different amounts of time given the action that's going on, to give animations time to run and to make the user feel like the elevator is actually moving. You could break it down further, if you wanted.

Basically, what we're building is what's called a "state machine," in which the elevator shifts between a series of discrete behavioral states (moving, stopping, doors closing). State machines are a great way of dealing with asynchronous code, because they let us model behavior through a simple set of transitions, instead of having to manually nest many callbacks.


next block instead of successive block


XCode 5


But you can have multiple params of the same type or not? And if yes, how?

Since it's more complicated than I anticipated, there should probably be an example about this (how to accommodate multiple parameters, potentially of different types).


Typo at "automamtically".


maybe create a constant for "const char *"


@afeld I tried to follow the example in the syllabus but I am still struggling with QUnit testing. Can you point out / clarify what I am doing wrong? Everything looks okay to me, however the output shows "undefined" errors after everything consistently.


october?


sorry should've told you, but I intended to put the API server, Crunch dispatcher, Git server, Websockets and Workbench all on the same machine, that I called Workbench. Its in [the README](https://github.com/guillermo-carrasco/arvados-ansible#testing-arvados) and I designed it following Arvados wiki


@mflorea: If I use `/`, then they are affected by the next line of code (handling slashes written by the user).

To avoid that, I have written the following code:
```
#set ($whereSql = "${whereSql} and upper(doc.fullName) like upper(?) escape '/'")
## Note: the "_" and the "%" characters has special meanings in standard SQL servers (they behave as wildcards).
## However, it is not what the user expects (e.g. http://jira.xwiki.org/browse/XWIKI-12608).
## As a consequence, we must escape these characters.
## See http://www.jguru.com/faq/view.jsp?EID=8881 to understand why we use the special clause 'escape' above.
## We use the '/' character for the escaping because Tomcat already prevents to have slashes in page name by default.
## Note: we do not replace by '/' directly because the slashes written by the user are handled afterwards and we don't
## want our own slashes to be affected.
#set ($locationFilterValue = $filterValue.replaceAll('([%_])', '##XWIKI_ESCAPED_SLASH##$1'))
## Because the user might filter with "/" as separator we replace that by "%", because it could be "."
## (the space separator) or "/" (as it is in the document name).
## In addition, using "%" (any sequence of character) instead of "_" (any single character) gives the ability
## to filter on different levels:
##   e.g. filtering on "Blog/Intro" will return "Blog.BlogIntroduction".
##        On the other hand, it will also returns: "BlogForIntrospection", which might be unexpected.
#set ($locationFilterValue = $locationFilterValue.replaceAll('/', '%'))
## Put back the slashes introduced by the filtering of "%" and "_".
#set ($locationFilterValue = $locationFilterValue.replaceAll('##XWIKI_ESCAPED_SLASH##', '/'))
#set ($discard = $whereParams.add("%${locationFilterValue}%"))
```

Not sure if `##XWIKI_ESCAPED_SLASH##` is cleaner than `◊`, but I can commit this.


Using double tap makes it really slow. Software reaction for back key and menu is very fast, but it´s not for one tap on home, it takes a bit more.

I suggest you using Find7Parts from OmniRom, included in msm8974-common which works much better.


Width could also be calculated in `evalStyleParamWidth`, which could land in an `epsilon` ballpark.
With respect to > 0, yes we were talking about it yesterday, but decided to stick with just a within `epsilon` test.


shouldn't this be CXString?


You should mention that this depends on `<algorithm>`.


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


Ещё один — общий — комментарий ко всему заданию: очень важно обеспечить читаемость кода. У тебя с этим вообще говоря вполне неплохо, но беда с именами переменных. Например, весь скрипт мы работаем с переменной `fields` — эээ поля? Вообще поля?.. Какие поля?.. Потом ты несколько раз для разных целей записываешь данные в переменную с весьма абстрактным именем `sort`. И даже внутри маленьких блоков — лучше называть переменную, даже если одной буквой, по смыслу того, что у тебя в массиве лежит (т.е. в нашем случае, напр, m — от movies или f — от films; а когда там ещё не хеши а просто строки текста/массивы данных — ln от line, напр.)


This could go either way:
1) ...you can use a more free-form trackCustomEvent: call (call? function?) to send...
2) ...you can use the more free-form trackCustomEvent: to send...


use tkinter like in text.


matching URLs
https://112.108.40.152


Oops, there's a typo! Please, correct it:

"Počet komentátů k tomu snímku:" for "Počet komentátů k tomu**to** snímku:"


Documentation of the rest


Out of curiosity, what does this code do`?


imported by some other module or your source file.


This should be included in each file that uses `CONTAINS` since it's only used in a few places and you shouldn't bloat a basic header like this so much for one macro.  Also, see https://github.com/minetest/minetest/commit/27eed1389b8bece41950c53b47acffb3bb0c83ee#commitcomment-14018111 .


So... I guess the take home message from this is 'If you're wondering which one to use, create your own and it'll be simpler'?
If that's the case, I would probably state it clearly right after the first sentence of this section to avoid the readers thinking that the information you added is really important for them to remember and understand (which many won't, to be honest...)


Must be absolute.


In case you haven't been using Mixpanel and all you want to do is send custom JSON objects...


It would be nice to see the `.start()` being called.


@dgreisen-cfpb - did not resolve indentation for 38-2101 - must do manually.


Please fix all the close() warnings. Use C fclose() instead of POSIX close().


Also the submodules are excluded.


@otoolep I've changed this so that the init script does not rely on the knowledge of the binary. Since we're moving the location of the `DAEMON` variable, if someone upgrades while Influx is running then the script fails to recognize the currently running process.

I've tested this, but just wanted a second set of eyes to make sure this makes sense.


or the hooked module's submodules.


I'm in favour of dropping this. I think you're right that it's unlikely to be required here.


Wat?


Good call.


The example above uses `aaa.bbb.*`. This should be described here.


93653546-FuelGenShowingInABUPSwhenPV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (835 of 835 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)


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/cd4badc6f621525b15ca6d93986cd612dffdac66


merging basho/yokozuna/features+fixes/zl/handle-cleanup-allow_mult=true = cd4badc6 into borshop-integration-567-features+fixes/zl/handle-cleanup-allow_mult=true


basho/yokozuna/features+fixes/zl/handle-cleanup-allow_mult=true = cd4badc6 merged ok, testing candidate = 7c6e9e5f


Intentional or can we drop this? Looks like it was here for debugging purposes.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 93](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=377&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Add handle empty string $compName and remove redundant creating log directory code.


Now experiment zone is computed with given rods.


The AloomaiOS stores events in an internal queue...


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 93](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=377&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:41


In general, a bit of confusion here which of these are middleware, and which of these are different (like the property on the `Request` object).


4943-EMS-Trap-Unitialized-Variables (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1955 of 1960 tests passed)

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


I would write something like: "If using globs is not enough, e.g. you need to collect several, maybe nested directories, ..."


/me wonders whether we could make the satisfaction of this requirement immediately evident in the code itself if we refactored, combining `precedence` and `fixity` into a single helper method... (bleah, I tried it out, its probably not worth it.)


Thank you! Sorry, but I had forgotten about this for a while.


Crap, for some reason I thought -1 is interpreted as False. Sorry the for hassle!


smt ifdfds


True if *at least* this version.


s


Somehow related. I was trying to test the code first using nicohood's code then another but without success. Once Bridge lib (which has dot_a_linkage on master) and latest avr core will be released, I'll be able to add that test


No hosting. We need to avoid costs of hosting and dependency on brittle low
cost hosting.
On Tue, Oct 27, 2015 at 12:18 PM Vlad <notifications@github.com> wrote:

> I see, what about hosting your own custom json of all recent
> classified/hidden items that gets cached along with the extracted data from
> the manifest?
>
> Unfortunately bungie hasn't shown any interest addressing the issues
> caused by these classified items in 3rd party apps.
> I was disappointed how the newest manifest update did not declassify the
> no time to explain so this might be way more common in the future :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/v-fedorov/DIM/commit/5a8aed39de25f6a34322c5721ad29384e22ae851#commitcomment-14017662>
> .
>


Hi, react 0.14 uses ```react-dom``` for rendering dom elements.
When I use ```r-dom``` with it, it shows error ```Uncaught Error: Invariant Violation: ReactDOM.render(): Invalid component element. This may be caused by unintentionally loading two independent copies of React.```


The reason we need to do this is because the idea of `baseUrl` is that it will hijack all url changes on the origin of the app and handle them as if they were relative to the root of the app.

So: `/todomvc` is the baseUrl, and any href to `/`,  `/active` etc. will work as if it was at the root level.

If we check origin + baseUrl these hrefs will point to an invalid url... which is not good :-)

Does this make sense? Some tests are breaking now, but they are actually testing the wrong thing. When Cerebral-Router takes over with a baseUrl everything will/should work relative to that. So testing:

```js
        test.equal(routeTest.emit('/'), false);
```

Does not really make sense because those urls there are relative to the baseUrl :-)

Doing a new iteration on this now


Then just rename it with a name that is meaningful to you. :see_no_evil:


This will be 0 ELO.


Just wat


Ping @sindresorhus @vladikoff for a new release.


No longer needed... icon edited.


Fixed in 0d221d066b8bf94f8596d3f703b3272513481336


In the past when I've done configuration I've done it through `ENV` vars but this is fine too I guess (e.g. `staticAssets.acceptWebp` could also be passed in as `STATIC_ASSETS_ACCEPT_WEBP=1`).

Not sure what your thoughts on that are? It was something I started doing back when [12 Factor apps](http://12factor.net/) were a thing.


Oh, we should merge or complete #1561 very soon.


lexis - should be (2)


lexis - should be (3)


I would leave the `gt`, `lt, etc. away.


This causes word order issues.


gmaiil.con@gmail.com


What is this for?


Corrects issue #5  by fixing ==, != statements instead of ===, !=== where they were incorrectly placed. (Artifact of poor coffeescript conversion).


Use a frozenset here for VALID_CHARS, not a string
https://docs.python.org/2/library/stdtypes.html#frozenset


Use frozenset for INVALID_ELEMS
https://docs.python.org/2/library/stdtypes.html#frozenset


Corrections mineures.


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


Try to avoid moving methods (like `joinPaid` here) up-and-down in a file unnecessarily - it just makes the diffs bigger and harder to read.


There is no assertions on this test, I think you forgot to add them :)


Thank you :D


empry -> empty


this commit seems fine apart from the added warning.

(i'm pausing the review but should resume in a few hours)


Some target classes like LRGs will make an initial request for number of observations based upon the magnitude of the target.  Faint LRGs are expected to need more than one observation to get a reliable redshift, thus they include NUMOBS in the initial target selection request.  This can be later updated by observations, e.g. QSOs are (optimistically?) expected to be identifiable from a single observation, and then update their NUMOBS request based upon whether they are a z>2.15 Lyman-alpha forest QSO or not.


"annotating", not "annotation"


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 23](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=378&buildTypeId=Xbim_XbimScripting_XbimScripting) is now running


".. or if **it** has at least one ..."


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 23](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=378&buildTypeId=Xbim_XbimScripting_XbimScripting) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:10


I think this'll make it so that view recycling won't quite work properly (if a message gets bound to a view that previously contained a message from yourself, it still won't have the mute menu even if it was for someone else). Might be easier to just disable the button and then undisable it in unbind?

Not sure how disabled buttons will be styled, but that's easily fixable (and I need to fix a similar thing for the text entry anyway)


You might want to say
```
for _ in xrange(interval)
```
`_` is still a valid variable name. It's the pythonic way of saying, "I don't want to actually use that return value"


A


@ajithcj Could you schedule a test for 7 threads? I proposed a test with your changes, but the approver requests a rebase of your test to the lastest master. I think it is best that you do this. Sorry for the inconvenience.


Perhaps all of the middlewares should belong to a `middlewares.es6`?


That being said, only people who write hipster python would ever do this.


Aah - you're right !


Also, there's an extra **"a"** in the test's description. It should be: "... when the previous ...", not "... when a the previous ..."


@gravgun If you merge master into bullet regulary it will make the merging easier.


d


Strictly speaking this is a function, not a method. But maybe you want to express some "method" to handle things.


mispeled canola


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1423


This is weird. Why do you need to remove **this** but add it back again afterwards?


Do you think they used `CHECK_EQ` for no reason?


I agree with @forero's skepticism that this field really belongs to something downstream from target selection, e.g., the Merged Target List (MTL).  For example, an object may pass target selection for multiple classes, so the MTL is probably the right place for NUMOBS to be initialized depending on what logic it applies to "conflicting" targets.


As discussed, I'll take a look at whether there's a cleaner way to handle this without relying on anchor override behaviour.


manually resolve 42-3402.11


Okay, thought about this some more, we have to make a choice :-)

1. Cerebral-Router hijacks the origin and will make any hrefs relative to the baseUrl. So `baseUrl: '/todomvc'` allows you to use href: `/active` and `/` to go relative to that path

2. Cerebral-Router only hijacks the actual baseUrl, but then hrefs has to be `/todomvc/active` and `/todomvc/`

I really like the first approach there, and with the autodetect baseUrl you do not even have to configure it at all. It will just work. The limitation though is that you can not route to anything else on your origin. That may be an issue.

Hm hm, this is a difficult one.


You never `eslint-enable` again. ;)


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


As there might only be one memory disk, should this be changed to:

```
echo -n "Setting up memory disk(s)..."
```


@illus0r
Wow - you're effectively neutralizing my commit 2220796 here.

Not good!


обожаю c# 6.0


Mixing pthread mutexes and forks is undefined behavior in general.
http://stackoverflow.com/questions/14407544/mixing-threads-fork-and-mutexes-what-should-i-watch-out-for
https://www.quora.com/What-happens-when-a-mutex-owner-is-forked


So this one was very frustrating, I wasn't sure if it was the client or the server, finally a packet capture showed the HTTP was fine.. then after lots of digging I found this bug had been reported a few days before I started seeing it. :+1:

![ojibwebadmethod](https://cloud.githubusercontent.com/assets/3255752/10767181/f58ebb4a-7ca8-11e5-80a9-d80f10f2d491.png)

For anyone else experiencing this now, that wants to get their code working and not try to build the dev branch of Kestrel, here is a workaround.

``` csharp
    public class FixBadVerbMiddleware
    {
        private readonly RequestDelegate _next;

        private List<string> goodVerbs = new List<string>() { "POST", "PUT", "GET", "DELETE" };

        public FixBadVerbMiddleware(RequestDelegate next)
        {
            _next = next;
        }

        public async Task Invoke(HttpContext context)
        {
            if (!goodVerbs.Contains(context.Request.Method))
            {
                context.Request.Method = context.Request.Method.TrimStart('"');
            }

            await _next(context);
        }
    }
```

Then in startup Configure

``` csharp
app.UseMiddleware<FixBadVerbMiddleware>();
```


Please keep the file naming consistent . AzureVmBackupExtensionUtil.cs


Ups, this interface is a crap! I'll change it into `*names`

This method is not used anyway.


Awesome :+1: . Thanks!


Same here.


main.c:201:13: warning: passing argument 1 of ‘pthread_join’ makes integer from pointer without a cast [enabled by default]
             pthread_join(thread + i, NULL);


lgtm.

We are seeing this fail intermittently on AIX and giving the test a bit longer seems to resolve.    Since this is likely cause by external load or other things running on the machine the change seems better than extending the platform timeout which would make the test run longer in every case.  The change lets the test pass quickly in the normal case but gives it a bit longer when necessary.


Can you tag 0.4 now instead of 0.4-?


Looking from the outside option 2 seems less confusing to me.

Similar to how you decided to handle hash routes/links. If you want to link to a hash url, include the `#` in the link itself, don't expect the router to automatically inject `#` for you. Similarly, I'd say if you want to link to `/todomvc/active` then link to it, don't expect the router to inject `/todomvc/` where you use `/`.

Just my 2 cents! :smile:


I will manually stop the test at 20M games if this is still neutral.


@mikrobi that's strange, when I follow the link, I can see the image.


That’s completely fine with me.


...oh, probably because my Instagram account is private. I thought if you share direct links even on a private account it would show them. Hmm...


@mikrobi thanks for the heads up BTW.


93653546-FuelGenShowingInABUPSwhenPV (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1891 of 1903 tests passed)

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


I was leaning towards this too @bfitch :-) It is the most explicit way to handle this. And baseUrl just says: "I am looking for url changes on that path"


No Tests, will add
Looks good otherwise :D


Yeah seems like these links are protected...


"distributed system"?


@stevenrombauts These shouldn't be removed. The check should return TRUE if session wasn't started.


If we can add back the Toaster I would be :smile:


"searched in ~~index~~ sequence"


Unfortunately, still broken as of ccb50c9.

https://www.dropbox.com/s/ft3mqe3x5jiz9pm/issue403-vid6.mov
https://www.dropbox.com/s/art91m09vyrqx0d/issue403-vid5.mov
https://www.dropbox.com/s/6z1glr4v3dj9v5n/issue403-vid4.mov
https://www.dropbox.com/s/76wf9a5qnqw5deo/issue403-vid3.mov


Условия завершения игры должны быть тоже внутри логики


Please keep the extension naming consistent. AzureRmVmBackupExtension. Even I felt it lengthy to have AzureRmVm... but tha's the convention followed. So let us follow.


`.then(appModel`


we need to be using python 3.4


We can remove this comment


Is this typographically correct in English: Having *no* space around EM-dash?


Matrix object is not to be created by the function makeCacheMatrix, but to set and return the original and inverse matrix


Please add a comment why only one role is returned.


Great, but what are these hooks useful for?


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


@stevenrombauts I would do this in a separate issue and PR, which we can mark "security". This makes it easier to track issues that are security related and contribute them back upstream.


Yeah, they were supposed to be in the optional list. I've patched it on my working branch, just need to push it up. Thanks for the reminder.


Can drop the `return` here and in the `else` block below?


ah, yes - good point


На мой взгляд, логика разбросана по всему коду, лучше собрать ее всю в одном месте в logic


@obihann it seems it is related to https://github.com/obihann/drafter/blob/master/src/RefractAPI.cc#L265
Can you please rebase on curent master?  Maybe it is already fixed there https://github.com/apiaryio/drafter/commit/e28cbdd84a6746f5e88b1d429788aa9c28f4b23a#diff-2ab1b5da6ee8cb5b29287e4ebfd30e94R296


fast-forwarding features+fixes/zl/hotfix-2.0.6-2012.08.25-for-2.0 to borshop-integration-567-features+fixes/zl/handle-cleanup-allow_mult=true = 7c6e9e5f


'different' oil' needs to be updated to Canola oil.


This error is being caught in catch block below, so it can cause misleading diagnostics.


I had a feeling it was an accident but I commented just to be sure we were on the same page on how this should act. ;)


Fixes #60


"This is useful to get that module known to PyInstaller and avoid misleading warnings." Is this Correct?


@kyuupichan this broke the "show history of address" dialog


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


This code needs refactoring. But it is not urgent.

How it can be refactored:
read input files, convert them to ps_tree_data struct that contains clear settings for the whole process tree configuration. Pass subset of ps_tree_data to child processes.

ps_tree_data {
   ps_config root;
}

ps_data {
   pid_t pid;
   size_t num_threads;
   thread_data *thread_configs;
   size_t num_child_ps;
   ps_date *child_ps_configs;
}

thread_data {
    pid_t pid;
}

We'll discuss it in skype meeting in details.


Este cambio es para solucionar problema


Yep, I did `git flow hotfix start ...` off of master.


@johanjanssens That makes sense, but we also need to remove the $session->fork() line from [7b51eb7e7546844d3d](https://github.com/joomlatools/joomla-platform/commit/7b51eb7e7546844d3d625a60a550197d444772a4) in this branch and add it back into the new issue.


And please fix all the warnings.


What happens when I upload file "a.pdf & echo shell injection.pdf"?


I'm not very comfortable adding this constructor.  Whoever made the original version of this library knew full well about the existence of `std::string` and clearly made an explicit choice not to construct a `StaticString` from a `std::string`.

I suspect that their motivation had to do with the durability of your typical `char const *`.  A `std::string` is typically not nearly as durable as your typical `char const *`.  So, for my taste, I'd like to see this constructor go away.

If you make this change, then you'll need to turn `stateNames` back into an array of `char const *`.


Committed there anyway: https://github.com/xwiki/xwiki-platform/commit/f0f032438d031fa1fa1cbb321d99a37c7dcb54ee

We can still revert if we don't think it's good.


I'd keep this https://github.com/bbatsov/ruby-style-guide#empty-lines-between-methods


I´ve compiled FeatureCenter just fine as shown attached.

Maybe it's an environment thing. Can you post more details about the error?

[OutputFromBuild.txt](https://github.com/ErikWitkowski/eXpand/files/21900/OutputFromBuild.txt)


Suggestion, maybe as a step between 1&2, we can get preliminary approval before creating a version branch? That way we can avoid multiple branches with the same version number that have to be merged?


если хочешь просто изменить размер, то лучше написать Font(head.font.family, Font.BOLD, 17)


I assume that because we expect an exception to be thrown we don't need to close the factory or the channel. Should we anyway?


Will this suffer from the same I/O drain problem that backticks did?


manually resolve 48-1101


Логику завершения игры лучше поместить в logic


We should also close the channel, like so: "(ICommunicationObject)serviceProxy"


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


I don't like this change. For link hrefs you can use signal.getUrl() method


`getInstanceConfig` pulls in a ton of reachable methods.


If this fails...test again without the cmh/h tuned weights.


Yes, great catch!


Yes...


Can this be shortened to

```ruby
def all_can_view?
  ['normal', 'backpage'].include?(prominence)
end
```


I've thought about this some more... why did Travis need this else?.

Is ```#if TARGET_OS_IOS``` not defined on Travis's Xcode SDK...


?? I don't understand this commit. Why purge the README? I can think of a few possibilities, but none for which leaving it in the git history would be sufficient.


Same as other test, also close the channel.


This requirement is a property of parsing algorithm we use rather than this utility. Enforcing such invariants should be a job for whoever rely on them, but honestly I have a hard time coming up with a reasonable way of doing it in this case.


Yeah, sorry. I probably should have thought more about that one. The problem is running the action brings in a lot of reachable code.


g


This will need to be changed, since the Registration Form has been refactored.


очень тяжело читать и понимать, что это за тип, лучше подумать, как это можно сделать по-другому, в крайнем случае, сделать свой тип


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 24](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=379&buildTypeId=Xbim_XbimScripting_XbimScripting) is now running


Dont compila :/


Ah ah, I was just about to write...
"If this fails...test again without the pawn changes".

Pawns are very hard to tune. The current values were tuned at LTC.


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


Yeah. As it stands, this only triggers if there's a zero substitution because of issues like that.


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 24](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=379&buildTypeId=Xbim_XbimScripting_XbimScripting) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:22


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 25](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=380&buildTypeId=Xbim_XbimScripting_XbimScripting) is now running


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 25](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=380&buildTypeId=Xbim_XbimScripting_XbimScripting) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


@dang use store and not _store here


Note to self, dont use wordpad to edit/save a file. overwrites EOL to dos instead of unix.


@richardxdubois I pushed changes to filter by site correctly in full_paths(...) as well as updated the SLAC site name, which seems to have changed from "SLAC" to "slac.lca.archive" at least for the datasets that are being most recently registered in the Data Catalog by TS3.   I also implemented a member function to find the most recent folder based on the folder name and updated the constructor to do that selection by default.


I don't recall at the moment, but I think it might have been the `shutdown`. I'll have to rebuild to see.


OK.


Take also into account request someone posted to gitter today. I have promised that his issue was solved, but you have just broke it.


Thanks for the updated PR. The text here should be #1408, not #1384. Also, could you rebase if possible?


I'd keep the newline https://github.com/bbatsov/ruby-style-guide#empty-lines-between-methods


I thought we have made the decision we abort the factory in the Finally if an exception is expected?


Should this be also added to `ApptentiveConnect/resources/CocoaPodsResources/Info.plist`?


test this please


Nit: From previous team feedback we had discussed a 'commenting' convention. See the WebSocketTests as an example (...\Scenarios\Extensibility\WebSockets\WebSocketTests.cs). Don't need to block on this but at some point we would like to update all existing tests with this convention.


I'll do both then...


You know it is!

On Tue, Oct 27, 2015 at 10:21 AM, John McConnell <notifications@github.com>
wrote:

> Is quicksort next?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lawrencewalters/algorithms/commit/063d72a6e59051383a9a26ef436ed5b7a55d860f#commitcomment-14013836


Might be something nicer here? `external_url.present?` – not sure without looking


Обновы все шли и шли, а обозначения ярлыков с дока все не пропадали...


@neobuddy89 This is the correct placement, right?


Rad. Protip: `git flow feature start` and `git flow hotfix start` branch off the appropriate branch for you :guitar:


isn't this just a `compact`? `guesses.compact.uniq`


Good stuff. Wording on the info-texts need a little work. Also, I think it would be better if the grayed-out 'Unbind' buttons changed it's label to 'Logged-in' or something to that nature in addition to being grayed out.


Derp


'Reproducible Science!'


Add a comment indicating that this should be private.


Осталось Cross Cast сделать. К примеру Action Invoker cast spell on m_creature or pSender or много чего ещё.


`if hash` ?


add perl code to store files in variables, convert lower case to upper case


Hmmm, to wyglada jakbym to ja pisal, ale ni eprzypominam sobie tego :P


@ninja- Can you aware me of the 0 length packets bug? Which Commit is the latest Version which works fine with 1.7.x - 1.8.x?


@kamil-borkowski Czemu autoprefixer zamiast rozwiązania z compasa?


Ceci permet l'installation complète sans connexion à internet.


`if set_by && user`?


So I created the minified copy of it by installing the node package, navigating to the directory, and executing `make`.
It could probably be removed and if we ever needed to upgrade repeat the process.


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


`if event.described_state`?


111 минут


:scream:


Thanks for the pull request!
Can you modify the commit like this so that I can merge the pull request:
```js
start(timeout: ?number) {
    timeout = timeout ? timeout : 60;
    FBSDKGraphRequestManager.batchRequests([this], function(){}, timeout);
}
```


@klokane I was hopeful however it doesn't seem to have resolved it.


It was a note about the signature of the library to compare that the signature lined up. It could probably be removed or we could become consistent with their lib if we like the naming better..


Could this `return` be some other color than ... red?!


Correction par rapport une version béta précédente qui dédoublait certains dépôts dans le sources.list . Il en résultait un message d'erreur de Synaptic/apt/... qui bien que n'étant pas grandement problématique, mettait un doute à l'utilisateur.


@scheglov @bwilkerson Should this go into 1.13 as well?


`return revi if m.described_state`?


Ce paquet demande la validation d'un EULA lors de son installation. Il ne peut donc pas être installé automatiquement.


this means the XML file won't be imported into the image (nor container)


the file is no longer needed


блок else делать не обязательно


Yes, it would be good to include into 1.13 as well.


not needed because not run here


no need for the parens now?


I suppose the path for the js and css files would be different on the production machine. Should I change them before the merge? @zednis


midisport-firmware et madufload permettent la prise en charge de certains matériels audio et/ou midi.
Les autres paquets gèrent des puces internet et permettent donc l'utilisation de certains modèles wifi ou filaire.


* default arguments can be set as `function(a, b, c = VALUE)`
* position the create function commands in between library calls (first) and function calls (after)


You are exterminated by sysadmin :)


`base64.urlsafe_b64encode(os.urandom(24))` is faster and much secure since:
1. does not involve uuid, that is not suitable for secure token generations
2. use os random instead of python random
3. generate 32-byte length string too, but, 64**24 = 2**30 random items, while your way generates 256**16 = 2**24 variants


Está listando mas nao espera o usuário confirmar que leu. Resultado: Nao da tempo de ver a listagem e ja reescreve o menu.


13 часов?


Installe des tas de pilotes libres pour divers composants matériels.


This is why I don't share things with people ;) When I edit it in emacs it will all get set back again...


this value should be taken from configuration


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 36](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=381&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


Data you want to render into json are in `resource.actions[0].attribute`, linked commit expose them.
Now we must find reason why they are not rendered.


111 дней


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 36](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=381&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Compilation error: Xbim.Presentation\Xbim.Presentation.csproj (new); compilation error: Xbim.WindowsUI.DPoWValidation\Xbim.WindowsUI.DPoWValidation.csproj (new) Build time: 00:00:20


"surveys.csv"?


can you confirm if they're renaming/rearranging servers?


Provoque l'installation d'une palanquée de kits de batterie (et autres) pour hydrogen.


Everything looks good to me! Sorry this is so late. :+1:


Ajoute quelques kit de batterie (et autres) pour hydrogen.


ooooh


Installe un petit utilitaire en ligne de commande.


Suite au changement du nom de paquet dans le dépôt LibraZiK.


@klokane Just to clarify the commit you speak of is [this](https://github.com/apiaryio/drafter/commit/e28cbdd84a6746f5e88b1d429788aa9c28f4b23a#diff-2ab1b5da6ee8cb5b29287e4ebfd30e94R296)? If that is now exposing data I should start debugging the areas we have been already looking in, to see what has changed. Or do you have a better plan?


This will cause horrible issues for people who used the one way track in their city.

I would recommend you to use the "replace" feature to replace the one way tracks with normal tracks if the one way track mod was not found.


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


You forgot #endif here.


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


... suite à l'arrivée du logiciel dans le dépôt LibraZiK.


I'm surprised to see you using pipes in a single line of code.


Не в каждой минуте 60 секунд.
https://ru.wikipedia.org/wiki/Секунда_координации
Лучше использовать Chronounit, будет правильнее и удобнее.


@dang shouldn't this be outside the if (use.quota_enabled)?


Yes.. You can also try case... "UNBLANK" for RESUME
and "BLANK" for SUSPEND if SOD persist.


Maybe its just be, but I often find `unless ||` harder to follow.

I'd probably go with `if AlaveteliConfiguration::varnish_host.present? && id`


You can avoid all this noise in the diff by setting `__metaclass__ = type` at global scope.


Should be removed from the python3 version first.


In the past I've used a very similar signature for this.

However:

- I normally class it as an `appFactory` rather than a `serverFactory`, as that is the Express nomenclature. However, I think as we're using the term `App` within React, it's a good idea to continue calling it `server`. However, can we call this function `createServer` as I'm partial to functions being verbs and not nouns?
- I give it a `NODE_ENV` argument so as to make it as transparent as possible.

e.g. random code I did a while back when building a Koa app.

```
import mount from 'koa-mount';

import createApp from './app';
import createApi from './api';

function configure(config, NODE_ENV) {
  const app = createApp(config, NODE_ENV);

  const api = createApi(config, NODE_ENV);
  app.use(mount('/api', api));

  return app;
}

export default configure;
```

Obv `connect` is more or less the same.


Above the `sourceExpandedExpressions` are converted into a `Map<QualifiedNameReference, Expression>` and you are looking up a `Symbol` here.


Ajouts de ces paquets (légers en poids) qui sont bien utiles pour un dépannage. Ceci est surtout pensé pour le cas où l'interface graphique ne se lance pas, notamment le navigateur internet en ligne de commande : lynx.


And then try FB_NOTIFIERS 1
if issue persist. :)
Coz I had SOD issue long back due to wrong placement of hooks.


Jo bakker, container is wel basic benaming voor de grenzen van je content, maar blijft maar een benaming.


Installe (notamment) l'utiliataire en ligne de commande : lspci.


In b3c2567daa5d8f94c22148d8a6f91c7776feed53, included in the history of this change, the peerDependency to React was changed from 0.13 to 0.14.

This means a patch bump is insufficient and broke deployment for and possibly others.


Permet de tester la mémoire lors du démarrage du live.


What happens when:

```ruby
info_request = InfoRequest.new(:external_url => '')
```

Don't know if that's actually valid, but if it is then its worth a spec.

- Should use newlines after the `describe` and before the `end` of the block
- I know its consistency vs "correct", but I tend to [avoid using should](http://betterspecs.org/#should) in spec descriptions.


Okay, thanks :) I'll try dori's idea of calling state_resume and state_suspend here and adding the delay in the state notifier driver.


Permet le non-affichage des messages du noyau lors du démarrage du système.


action create_if_missing does not exist


foogly boogly


Ah ok, interesting.


rwth


bar mcnar


This is a commit-level comment.


Apologies for all of the .metadata files that eclipse decided to add


Oops.  Will fix this.


oh my. these should really use filepath.Join


Yepp try that as well... @dorimanx is on 3.4 so he has better understanding of situation.


+1 to:

```diff
-  describe InfoRequest, 'when getting similar requests' do
-
+  describe 'when getting similar requests' do
```

-1 to the removal of blank newlines. We don't do this anywhere else.


これたぶんインデント間違ってるんじゃないかな
カッコを開いたレベルで閉じるのが普通な気がする
あと38行目もなんとなく気持ち悪いような...
細いから直さなくてもいいけどね


You don't have to, but you might want to rename this to nssHandle, since it's no longer using ptm:sysm. I forgot to change it, heh.


Ajoute l'adresse du sources.list du dépôt LibraZiK pour les utilisateurs ainsi que la signature de la clef de chiffrement des paquets.


@rzarzynski what if stack is empty? should avoid accessing entry in that case


What happens when I comment here?


Suite à un bogue dans live-build.


What happens here?


done


Повторный вызов функции, выкинувшей исключение - странная обработка исключения. Если так и должно быть, то это точно требует поясняющего комментария.


Yes, just discovered this. We'll have a 2.0.5 release out shortly.


Ajoute la possibilité d'installer LibraZiK à partir du live.


これ、$coordinate を set したあとに処理しても意味なくないか？


Utilitaires avec interface graphique.


Nice to hear your plan to realease a new version :)
I have another suggestion: Maybe the timeout when connecting to the camera is a bit low. I still have to investigate this. It could be the new version of the remote control app on my NEX-6 got significantly slower. Now i get an error when connecting for the first time but after another try it works just fine.


Looks like the Java files are written in a very liberal mix of tabs and spaces… in Github’s diff view, nothing aligns


Permettent l'utilisation de certains matériels audio et/ou midi.


Mypy supports   `#type: (SimpleMapping, XK) -> XV` after the colon, should that be used? (I seems the type of `self` may be either included or omitted, should that be standardized?)

Or would it be better to bless/port `mypy.codec` (which has no deps on the rest of mypy, and really should be separately installable)?


This is an [untranslatable string](http://mysociety.github.io/internationalization.html#general-guidelines). It also misses the "A"/"An" prefix


interesting! do we have a test for this?


Ajout d'une optimisation LibraZiK permettant le fonctionnement coordonné de qjackctl et de pulse.


This is an [untranslatable string](http://mysociety.github.io/internationalization.html#general-guidelines)


nice - this is much easier to read


これでももちろん動くけど、count の返り値は int だった気がするので (int) いらないかも
詳しくは php.net/count で（おれ見てないから確認して）


Could you give more details on the reason why you remove the individual tests for the factories? Even if it reduces significantly the number of tests and allows for a generic implementation, it breaks the (hard to follow) rule of one class/one test, and may obfuscate a little bit the debugging process.


oh looks like you wrote one in a later commit - :heavy_check_mark:


Was this discussed in the meeting?

On Tue, Oct 27, 2015 at 12:39 AM, Ben Goodrich <goodrich.ben@gmail.com>
wrote:

> I can stuff the log_prob function (and possibly gradient) into the list
> returned by optimizing(). That seems to work but it is weird to call, e.g.
>
> blocker <- stan_demo("blocker", method = "optimizing", draws = 1000,
> constrained = FALSE)
> blocker$log_prob(rnorm(47), adjust_transform = TRUE, gradient = FALSE)
>
>
> On Mon, Oct 26, 2015 at 8:29 PM, Ben Goodrich <goodrich.ben@gmail.com>
> wrote:
>
>> At the moment, you would have to create a (possibly empty) stanfit object
>> to call log_prob() on. This is a weird thing about rstan.
>>
>> On Mon, Oct 26, 2015 at 8:26 PM, Jonah Gabry <notifications@github.com>
>> wrote:
>>
>>> Yes, I think the Jacobians need to be accounted for because otherwise
>>> it's
>>> actually a different unconstrained space, right? Andrew or Aki should
>>> confirm though (Andrew and I were just talking today before Aki's talk
>>> about how this gets pretty confusing).
>>>
>>> Also, is it possible to access the log_prob function after a call to
>>> optimizing? We need to compute log_p again at each iteration of the
>>> importance sampling. We're computing log_g again too but that's always
>>> just
>>> log of a MVN. For log_p we'll need the model's log_prob function.
>>>
>>> On Monday, October 26, 2015, bgoodri <notifications@github.com> wrote:
>>>
>>> > We should put the constrained = TRUE argument into optimizing() because
>>> > optimizing() does not return a stanfit object, which is needed for the
>>> only
>>> > documented way you can call constrained_pars(). Do you want to call the
>>> > log-posterior evaluated at the unconstrained draws log_p or lp__ ? And
>>> just
>>> > to confirm, that is the log-posterior with the Jacobian terms added?
>>> >
>>> > On Mon, Oct 26, 2015 at 7:39 PM, Andrew Gelman <
>>> notifications@github.com
>>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
>>> > wrote:
>>> >
>>> > > Better not to have 2 sets of draws. Better to have one set of draws
>>> and
>>> > > then user can switch to the other with an unconstrained() call or
>>> > something
>>> > > like that. lp__ will be one of the saved outputs, so that’s
>>> convenient,
>>> > we
>>> > > just give the appropriate lp__ with its set of draws, no?
>>> > > A
>>> > >
>>> > >
>>> > >
>>> > > > On Oct 26, 2015, at 7:28 PM, Jonah Gabry <notifications@github.com
>>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
>>> > > wrote:
>>> > > >
>>> > > > Hmm, that's tricky because the unconstrained stuff that's useful
>>> to us
>>> > > for
>>> > > > the importance sampling probably isn't what an rstan user would
>>> want.
>>> > > Could
>>> > > > there be an argument that toggles whether the draws are on the
>>> > > > unconstrained or constrained space?
>>> > > >
>>> > > > On Mon, Oct 26, 2015 at 7:12 PM, bgoodri <notifications@github.com
>>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
>>> > > wrote:
>>> > > >
>>> > > > > Do we want to also return the draws on the constrained space?
>>> Would
>>> > it
>>> > > be
>>> > > > > more confusing to have two sets of draws?
>>> > > > >
>>> > > > > On Mon, Oct 26, 2015 at 7:09 PM, Jonah Gabry <
>>> > notifications@github.com
>>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>
>>> > > >
>>> > > > > wrote:
>>> > > > >
>>> > > > > > Yeah I think that's right. We want to do it all on the
>>> > unconstrained
>>> > > > > space
>>> > > > > > so I think any jacobians needed to transform from constrained
>>> to
>>> > > > > > unconstrained would be needed for the log_p calculation.
>>> > > > > >
>>> > > > > > On Monday, October 26, 2015, bgoodri <notifications@github.com
>>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
>>> > > wrote:
>>> > > > > >
>>> > > > > > > @jgabry <https://github.com/jgabry> is this all you need for
>>> > > > > importance
>>> > > > > > > sampling after optimizing()? Also, is it correct that you
>>> want
>>> > the
>>> > > > > > > log-posterior evaluated with Jacobians even though the
>>> Hessian is
>>> > > > > > computed
>>> > > > > > > without them?
>>> > > > > > >
>>> > > > > > > —
>>> > > > > > > Reply to this email directly or view it on GitHub
>>> > > > > > > <
>>> > > > > >
>>> > > > >
>>> > >
>>> >
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13994021
>>> > > > > > >
>>> > > > > > > .
>>> > > > > > >
>>> > > > > >
>>> > > > > > —
>>> > > > > > Reply to this email directly or view it on GitHub
>>> > > > > > <
>>> > > > >
>>> > >
>>> >
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13998153
>>> > > > > >
>>> > > > > > .
>>> > > > >
>>> > > > > >
>>> > > > >
>>> > > > > —
>>> > > > > Reply to this email directly or view it on GitHub
>>> > > > > <
>>> > >
>>> >
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13998229
>>> > > >
>>> > > > > .
>>> > > > >
>>> > > > —
>>> > > > Reply to this email directly or view it on GitHub <
>>> > >
>>> >
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13998588
>>> > > >.
>>> > >
>>> > > >
>>> > >
>>> > > —
>>> > > Reply to this email directly or view it on GitHub
>>> > > <
>>> >
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13998766
>>> > >
>>> > > .
>>> > >
>>> >
>>> > —
>>> > Reply to this email directly or view it on GitHub
>>> > <
>>> https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13998818
>>> >
>>>
>>> > .
>>> >
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/stan-dev/rstan/commit/942aadd4ad0125659e41b8080fa46b0b27f406ba#commitcomment-13999546>
>>> .
>>>
>>
>>
>


So you think the methods are better listed under `sparse/linalg.ji`, instead of `bidiag.ji`, or `triangular.ji`?


'Reproducible Science'!


Couldn't remember, seems reasonable.


`__bytes__` doesn't exist in python2. `__unicode__` does however.


LGTM


Permet la francisation de l'installateur, de l'environnement MATE et du navigateur iceweasel.


Your take 2:  you never removed the cmh/h weights.  I think you did the opposite of what Rocky and I said...


Why the method rename?
Can't dynamically translate strings, so this won't work


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


https://github.com/healthstone/RE/commit/5f5e5325432980a9dc32dc69e2a873dfdb693007

Удалил лишний override + фикс на возможный вызов функции removefromworld под управлением кем-либо


93653546-FuelGenShowingInABUPSwhenPV (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1897 of 1909 tests passed)

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


111 ретвит


ちなみに
php ではこっちのほうが一般的かな
$item->options[] = hoge;
たぶんやってること一緒なはず


Weird. I swear I fixed these but I guess I didn't push.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 37](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=382&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


ok


Haven't expressed clearly. Phone WAS slower, less responsive and more battery saving. So revert to stock freq table


could we try to inline the PMI wrappers?


Strings need to be wrapped in `_()`


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 37](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=382&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:00:45


What if I leave an inline comment after this is merged?


What if I leave a comment after this has been merged part 2?


Not important, but I would have written (e.g.) "The first element _is_ static libs...".


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


> XWALK-5214

Please use the "BUG=XWALK-NNN" notation, both here and in the pull request message.


Looks great! I recommend to merge PR and close #3 when you're satisfied.


Given that I am not supposed to do what you and Rocky said, I just want to check exactly were things have not worked. My next step will be to remove pawn weights.


@afgane It's much more pythonic to simply return the list or raise an error that can be handled upstream, instead of returning a dict with both -- this function isn't meant to be a json-consumable web endpoint, right? Would a PR addressing this (and other places we return a dict + errors) be declined?


Are you thinking about performance?  The PMI calls only happen during wireup, so the extra function call overhead shouldn't result in a measurable overhead.


これも好みだけど手続き的に書くなら 54, 61, 53行目の順に書くのがいいかな
いまのままだとなんかいろいろ気持ち悪いｗ


Just use `basestring`?


@obihann important part is there: https://github.com/obihann/drafter/blob/master/src/Render.cc#L55
resp. on upstream it is https://github.com/apiaryio/drafter/blob/master/src/Render.cc#L66
Function `MSONToRefract()` now contains correct data. I mean with combination with  https://github.com/obihann/drafter/commit/0b1f2a644189d6a327f9c8a60431f1c12009e567#commitcomment-13952867 it should fix problem


Nice.


93653546-FuelGenShowingInABUPSwhenPV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 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.36%25-green.svg)


This update breaks python34Packages.characteristic (a dependency of nox) and possibly other packages. Is it required for nova?


`localLogDir` should use `filepath.Join`, but `dockerLogDir` should use `path.Join` instead (inside the container paths are always posix-style no matter what the host machine's filesystem uses).


Just a minor note - we may want to use logger on higher log level here or remove these System.out.println calls completely.


No. We can revert back, it's a pretty terrible change.


just a suggestion: agreed at the end of the day its not a big difference. patch looks good to me


Why is this needed when it isn't in python3?


We have separated all the sparse stuff into the `sparse` directory so I even though the sparse matrix is wrapped in the `LowerTriangular` type which is defined in the `linalg` directory, I think i belongs in `sparse`.


img/bg_main.jpg — а внизу тут не чёрный цвет должен быть? Там вообще нужен этот кусок изображения?


93653546-FuelGenShowingInABUPSwhenPV (EnergyArchmage) - 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)


The message should be to use pluginInitialize instead. initWithWebViewEngine is private API in 4.x


@beanstown106 which commit adds this lib? I can't see that anywhere.


I think this is a terrible idea!


Pushed in 6a9aeea


Вот, отлично!) Пропустил, тк `<a>` не совсем строчный элемент))


ref: https://github.com/Retsly/api/issues/1515


@regislebrun The rationale is it avoids a lot of duplicate code for classes of the same tree.
I dont want to modify each file to test a method is added/changed.


As before, add a comment mentioning that this should be private.


Ok, I'll go ahead with the merge.


Thx))


Creature Addon should not be reset here, creature addon can be reloaded once creature reaches home and that is done in reset, after reachedhome comes reset


remove spaces


Missing a `)` up there.

Learned the hard way. (I swear I tested locally, and oops – site went down royally.)


Loot recipient are only set if creature dies and respawns , not on evade,
no looting->no loot recipient ,
it should not be put there


Ups. Plural.


Отлично! Только почему не хочешь задать базовый `font-size`?


I don't think there's much point in changing this as it will all be replaced with cloudbridge once it becomes a bit more functional. Further, the endpoints will get redesigned so the app can offer a proper REST API vs. these endpoints that aren't so clear cut.


Missing a ) up there.

Learned the hard way. (I swear I tested locally, and oops – site went down royally. Granted, I shouldn't be using this in production anyway.)


せっかくリンクになったから確認したけどやっぱ int だね
object だった場合の挙動とかも書いてあるから読んでみるといいよ


primera version mas o menos completa


Yeah, programming motherfucker ;)


lgtm


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1022.bf6e6091dbbdca357422c7864dd8f2fd2cdaef9b](http://build.palaso.org/viewLog.html?buildId=64900&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 707, ignored: 6 Build time: 00:07:36


Может, пора задать значение?)


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1943 of 1954 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)


Issue #8 can now be closed :)


Seconded.


I'd take a different approach to this; Maybe:

```ruby
LAW_USED_DATA =
  { :foi => { :short => _('FOI'),
              :full => _('Freedom of Information'),
              :with_a => '',
              :act => '' },
    :eir => { :short => _('EIR'),
              :full => _('Environmental Information Regulations') } }

def law_used(modifier = :short)
  LAW_USED_DATA.fetch(modifier) { raise UnknownLawError, 'Unknown law…' }
end

def law_used_act
  warn %q([DEPRECATION] law_used_act will be replaced with #law_used(:act) in
          Alaveteli release 0.24.).squish
  law_used(:act)
end
```


а зачем это прописываем? У нас разве не по левой стороне будет ровняться?


`isContainsTab` would be better in this case.


Oye porque la borraste?


For people who might want to install a newer version of `typing` than the one that comes with their python version, treat `major > 3` the same as `major == 3`.


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


The change itself looks good to me.

I'm curious though why I've never seen this one at Delphix before. In the configuration where this problem happens, is the default log location (/var/tmp/test_results) being used, and is it part of the root filesystem?


Línea no necesaria,,, no me gustaba


I don't recognize the syntax of a string literal followed by `git d`.  What does it do?


Why not have the type name (`output`) same as the variable name (`Output`)? Not that it really matters.


this can be read from linux git address, not win?


I mean, this stuff shouldn't really be in the model. Its more of a presenter, or even a class of its own, but something like this at least extracts all the data out and leaves a simple interface.


lgtm


For `MeshPhysicalMaterial`, I proposed renaming specular map to reflectivity map, which modulates the reflectivity parameter. We can discuss it. `MeshPhysicalMaterial` does not have a specular parameter.

I made no changes to `MeshPhongMaterial`, so the specular map remains in that material.


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


On the moment this scripts are in BETA


I think a universal resource could help.


sry, the macro is #define LED_GREEN_GPIO      GPIO_PIN(PORT_A,5)


Also the nav-bar says "paper list" in the hotel dropdown while it should be "Book Room" or something similar


Hmm seems like if the eventual goal was to just have a static minified file we could have just grabbed that from their release and not compile it ourselves.


forgot to remove


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


What is this used for?


Eeek. I don't know how that got in there. This project doesn't have any tests, as I have been too lazy to create mocks for the client/server API calls. I somehow added garbage there between my last manual test and the save/push.


In general I assume that if re-ordering includes breaks the build, then there are problems with the includes I'm using.  I think you identified two problems here:

JobTypeData.h is missing an include of `beast/insight/Collector.h`
JobTypeInfo.h is missing an include of `ripple/core/Job.h`

Once those two include files were fixed then I could re-order the includes in JobQueue.h to match our standard approach and successful perform both a unity and non-unity build.


Probably some better names for everything here btw :) `LAW_USED_HUMANIZED_VALUES`? I dunno :)


Are we sure you want to use "replace = TRUE"? This gives the chance of selecting the same site twice, and since we are selecting 100 sites from 120 sites (ratio), wouldn't we just be repeating some of the randomly generated sites? Just doesn't make sense to me.


一応ちゃんと動くこと確認してるから動作的には大丈夫だと思うんだけど意味ないの？


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


should be false.


オートフォーマットかけると38行目みたいになっちゃうんだけど回避する方法ってあるかな


you've got to be careful in changing the formatting of liquibase files, it can change the md5sum and make it unable to run.


> I mean, this stuff shouldn't really be in the model. Its more of a presenter, or even a class of its own, but something like this at least extracts all the data out and leaves a simple interface.

yeah, then we have a solid starting point if we do want to extract it out into something else at a later date +1, will fix in the morning


Ahh, should have known that.


@afgane So the cloud* portion of bioblend is going to be scrapped once cloudbridge is available?


lgtm


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


How can I update this patch?


- I'd call this `search_events` since we're limiting the search to `InfoRequestEvent`
- pad the assignment: `offset = 0`


@aluitink Does the following middleware also work for you? This is closer to the fix we're using for RC1:

```CSharp
public class FixBadVerbMiddleware
{
    private readonly RequestDelegate _next;

    public FixBadVerbMiddleware(RequestDelegate next)
    {
        _next = next;
    }

    public async Task Invoke(HttpContext context)
    {
        await _next(context);
        await context.Request.Body.CopyToAsync(Stream.Null);
    }
}
```

I think this fixes the root problem which is that requests are not being consumed completely. When a request isn't fully consumed, the following request is corrupted since the end of the previous request is interpreted as the start of the next request (i.e. the verb).

This bug can cause more than just `"`'s to prefix the verb, and this modified middleware should fix the issue in any case.


Actually, I'd make all these configurable with default options:

```ruby
def self. search_events(query, opts = {})
  default_opts = # ...
  # some sort of hash merge

  ActsAsXapian::Search.new([InfoRequestEvent], query, opts)
end
```


Could do with a spec since it's a new public api method


Should that be ||   ?


Yep I can do that.


@udalov, what do you think about it?


たぶんこれでも規約に沿ってるけど気持ち悪いっていうことね


will this flags deletion fix the pre-birth animation maybe?


Thanks!
On Oct 27, 2015 20:33, "Sindre Sorhus" <notifications@github.com> wrote:

> done
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gruntjs/grunt-contrib-htmlmin/commit/89513de5fcc13803b2cddf3dbc1d621d0e4c5261#commitcomment-14019896>
> .
>


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


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


あ、そうなんだ
cake の set がどんな動きしてるか、cake のコード読んでないから分かんないけど、set したあとに変数いじるって気持ち悪くない？


Its three different sentences, each terminated with a dot. The only thing I assume from a language is that it is split in sentences, and that different sentences can share a context.


What do you mean?


> yeah, then we have a solid starting point if we do want to extract it out into something else at a later date.

Yeah that's what I was thinking. I sketched something out [here](https://github.com/mysociety/alaveteli/issues/2570#issuecomment-112890979), but I don't know if that's really the right solution for this particular code. I think we will want to move more in the direction that [we discussed](https://github.com/mysociety/alaveteli/issues/2819#issuecomment-148843349) when talking about the ICO, but lets leave that alone for now.


I simply opted to write it like this. You can snip the line before the `||` and get the "proper" SSE4.2 support.


It's currently in the private block, I think - could promote it to a public method though


I like the solution but there are several bugs, watch out wrong comparison in isObject (should be obj instead of obj1), and if you iterate with keys (for in) and the return trumps the iteration should be in a check and keys could be different (obj2 could have more)


fsvo "block"


Maybe you can combine line 243 and line 244 into one line.


very clever solution, I love it :+1:


> Note - opted not to move all the functionality of `title_formatting` to a built-in validation as the RegEx required for "does not match this pattern" is not obvious and would probably be harder to read than the current arrangement.

Should be able to set a custom message for each validation?


Nooo!! why! :(

Let me know if there was anything that wasn't working for you, or ping me -> support@heapanalytics.com


あ、どうしてほしいかというと、この処理が終わってから最後にまとめて set してほしいってことね


Invalid layout http://joxi.ru/l2ZnYMXfKR1w2J


The mod was alpha and thus users were warned. Also, I've made announcements in reddit and in mod's comments. I don't think that there are many one-way track users who don't visit those pages.


lulz, but not descriptive.


Why +1? I've also added it but I don't remember why...


:P


That also works, and it even looks slightly less hacky.. :+1:


:P :P


It should still be in the setup section. :)


:P :P :P


@halter73 That needs a try/finaly


Meestal is het wel even handig om erbij te zetten welke exceptions je wil catchen


Better to assert on the specific message that's added. I have no idea what "generic summary" means, and the message could be set as "no" and still pass this spec.


@rodrigosaling Thanks for reporting, I forgot to update the file names after renaming the bower and package names. Pushed the changes to the same tag `0.2.1` Please update your package by running the below commands.

```sh
bower cache clean
bower install spring-security-csrf-token-interceptor-extended -S
```


I think this behavior is or should be implemented in a superclass / interface, and should therefore be tested in a test suite for that class / interface instead.
<testing


lgtm


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


Assert on the actual error message. I've just noticed that the specs above don't do this, so might be worth fixing those too.


@erfannoury sorry for messing up the repo structure!


All OK then!


@est31: No, if a language uses an ordering like (`client_info, proto_mismatch, server_info`) it won't work.


These need to be in a configuration somewhere. Can't just leave them hardcoded.


I think this might add the `"Please write a summary with some text in it"` message rather than the "too long" message – good example of why I prefer checking the actual message!


Ha.


How is this even working if you never assign anything to weakSelf? Logging should also have failed.


`yam` typo?


please make task more closely to the model in future (grey background and border are abcent in requirements)


I'm going to cry.


Does logging the block help? What is the output of this?


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


RadianceMeasureMessaging (rpg777) - i386-Windows-7-VisualStudio-12: OK (2336 of 2365 tests passed)

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


105015426-DefVAVTUMinMax (wfbuhl) - x86_64-MacOS-10.9-clang: OK (1954 of 1954 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)


Please rename function not here. You should rename the function in your config file from std_config() to ext_config().


Changes looks good and works ok.  One suggestion,  in exception instead of "Error in encoder",  say "empty line".  Every message from kafka has the empty line in end,  and this message will become annoying.   Alternatively add check :  "if line : "


```ruby
[5](main)> class Blah
[5](main)*   private
[5](main)*   def self.hi
[5](main)*     'hi'
[5](main)*   end
[5](main)* end
=> nil
[6](main)> Blah.hi
=> "hi"
```


このコメントであらためて確認したら元々のコードはcountに(int)つけてたわけではなくて除算の結果を整数で出すために(int)つけてた（前回のレビューで割り算を少数に直したのが間違っていたみたい）
わざわざドキュメントまで用意してくれてありがとう


Maybe it's better to return [self.serviceData description]?


Are you checking for PII? There's a better way to do that I think.


Nah bro


lgtm


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 26](https://teamcity.labkey.org/viewLog.html?buildId=280660&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:26


Wouldn't ```#__getDisabledFields()``` return without proper quotations? As in ```[elem1, elem2]``` rather than ```["elem1", "elem2"]```.


@erfannoury the ranknet-mnist branch is deleted from the repo. I will add another branch for my refactoring.


No, my initial query was should this:
```#if __SSE4_2__ && __AVX__```
be this:
```#if __SSE4_2__ || __AVX__```


How come an error is a actually a success? Does not make sense to me.


Hello Terry,

You're welcome!


Solution suggested?


You're probably also going to want `set -o errexit`
`set -o nounset` and `set -o xtrace` are also helpful are in all the seqware scripts
http://www.davidpashley.com/articles/writing-robust-shell-scripts/


I don't think this is a good example path.  The one from your commit message would be fine.


А почему вы уверены что где то в другом месте vk не возвращает video["video_id"]?


@Tratcher _next will never throw right? :smirk:

Nice catch. At least we can await in finally blocks these days.


あーなるほど
四捨五入したいなら round がいいよ


test


//cc @max-kammerer


Its `proto_mismatch.` `server_info.` `client_info.`. They are three distinct sentences. If it is an issue, then its no word order issue, but a sentence order issue. But I highly doubt there is any language that has rules distinct sentences. You might not catch the metrum, so there won't be a translation for LATIN_ELEGIC_DYSTICHON, but I can live with that. Do you know a language where the current order imposes problems?


hi guys the above code did not seem to work,i have it  as follows.
Also i use json in return values from ajax so have changed to object
```javascipt
function goToStep(wizard, options, state, index)
{
    if (index < 0 || index >= state.stepCount)
    {
        throwError(_indexOutOfRangeErrorMessage);
    }

    if (options.forceMoveForward && index < state.currentIndex)
    {
        return;
    }

    var oldIndex = state.currentIndex;
    var response = wizard.triggerHandler("stepChanging", [state.currentIndex, index]);

    jQuery.when(response).done(function(my_status){
        /*alert(my_status)*/
        if(typeof my_status === "object"){
            //status = JSON.parse(status);
           // console.log(my_status);
            my_status = my_status.status;
        }else{
            my_status = String(my_status);
        }
        //console.log(typeof my_status);
        if (my_status ===  true || my_status ===  "true"){

             // Save new state
            state.currentIndex = index;
            saveCurrentStateToCookie(wizard, options, state);

            // Change visualisation
            refreshStepNavigation(wizard, options, state, oldIndex);
            refreshPagination(wizard, options, state);
            loadAsyncContent(wizard, options, state);
            startTransitionEffect(wizard, options, state, index, oldIndex, function()
            {
                wizard.triggerHandler("stepChanged", [index, oldIndex]);
            });
        }
       else
        {
            wizard.find(".steps li").eq(oldIndex).addClass("error");
        }
     })
     return true;

}

```


:green_heart: !! :rocket: :guitar:


Looks great, thanks!  I amended the patch to fix the example path and pushed to master:

- b96d548 busybox sys_eeprom: Support reading/writing the "eeprom" exported in sysfs file


I'm no npm expert, but I think deprecating 0.5.8 and publishing it as 0.6.0 should do the trick.

https://docs.npmjs.com/cli/deprecate


you can do this in a single line: `eventResponse.save({response: response}, {patch: true})`
the other good thing about this approach is it sends an update with only the changed values -- otherwise `save` sends all the values in the model (not the best default behavior for Bookshelf, perhaps, but ¯\_(ツ)_/¯)


#52 will change the brightness range. It will be 0 to 9 inclusive (`range(10)`)


este fue el commit que hice anoche.


This is going to be removed by #52


Looks like a rebase error. I didn't author this change.


А почему вы уверены что где то в другом месте vk не возвращает video["id"]?


I would add an SOL_ERR("A message here") before returning.
+1


It's important for the removal to happen within the same task, especially so that once we move to versioned repositories, both changes will happen in the same version. Doing the removal in the same task will also save us some overhead.

I suggest calling the unassociate manager method directly.


sir indentation asa ka hotay ? file madhe tar asa nahiye
![screenshot from 2015-10-28 00 47 48](https://cloud.githubusercontent.com/assets/13753510/10769739/9b4cd116-7d0d-11e5-80f6-99e92076066d.png)


тоже что и выше


This also takes an image (I think). It does in #52