Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 11 to 11.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 70 to 70.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Good instinct that we might want to ensure that project properties besides the campaign ID bubble down to associated activities. I don't think the function signature is well-suited for that, though. I'd rather the function take an array as the argument. That way, if we decide we don't care about campaigns, or that we want to add other parameters, we don't end up with function calls that look like this:

```
self::updateAssociatedActivities(NULL, NULL, NULL, 42, NULL, FALSE, 'bananas');
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 87 to 87.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


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


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 13 to 13.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 171 to 171.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


override


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1085.1d3924fd950fdb45eb97923133361f0ca855455f](http://build.palaso.org/viewLog.html?buildId=66763&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 734, ignored: 6 Build time: 00:19:08


Idem for all other assert in this PR.


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


It's more of just me being stupid....


And the code for handling that is only in `handleIncomingRequest`, which only gets called in response to the client returning data.


Is this still compatible with serpent?


This seems like a problem we should address rather than work around, as it could have other implications.


I tested missed alerts combined with disable all alerts, with this commit one and all seems still ok.


Any minimal Solidity version expected? If so it should be checked.


Introduced the concept of temp sink replacing java temp files.


ncr-17, not ncr-14


Cache store implementations are supposed to supply their own version of temp sink. Here, we supplied one for mem.


>media-plugins/vapoursynth-plugins-meta/.list.txt.kate-swp

Junk file?


There is now!

https://github.com/djadmin/djadmin.github.io/blob/master/addon/search/goto-line.js

Demo:

http://djadmin.github.io/


Mem temp sink is based on byte array and hence penalty is almost negligible.


You have to add the length of the query to the offset. Otherwise you will get incorrect results in case the query includes a newline by itself.


Selection of temp sink is based on the cache store configured for the given cache config.


why overhead of synchronized?


Approvata


zx


Cierra issue https://github.com/InspectorIncognito/server/issues/23


@GastonTS acá para abajo


Да нет, так в принципе норм, но если что, то стоит ограничиться лишь кнопкой создания треда и оставить как было `Ответить` и `Скрыть форму` (вот они совсем не в кассу), и тем более не трогать похожие `Получить новые посты` и `Свернуть тред`. Текстовые кнопки с квадратными скобками - в общем стиле досок и не выбиваются из него (см. например на 4чане `Catalog` и `Archive`, на ичане кнопку `Назад`). Одну кнопку создания треда на доске можно и выделить таким образом, а вот остальные уже выбиваются.


@file directive missing.


Empty line.


Ouch...


Typo: Receive


i suggest something like 'maxItems < 0', this is safer than strict equality


so it does write to backend now each time some code is just referencing it?

Shouldn't we try find first and return if it already exists?


Are you sure this is right? I think you had it right the first time. We don't want to allow for campaigns to be assigned individually to activities; any campaign argument passed to this function should **always** be ignored, and the campaign of the project should **always** be applied to the activity.

In fact, it might be worth throwing in an `unset($params['campaign_id']);` before the if statement so that the param is ignored even if the project has no campaign.


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


Purpose? Just typehint the argument.
collectIdentifiers($idName, array $ids) {


Purpose? I don't see anywhere variable $ids to be used.


LinkedHashMap - is ordering of map entries important?


Qui ci vuole l'intervento di Michel


Approvata


Screenshot gentilmente della funzione?


There's no point to catch an exception to throw it again.


added bt2 mapping
added spike mapper on unmapped reads
changes while loop in after bamcleaner to sleeptime of 120s
and some other stuff i don't remember


eu-west-2, us-east-2 are added multiple times! It's a dictionary.


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


The indent is not correct.


indent is not correct


I wanted to disable this for a long time. :\ Should have been off by default.


Necessaria approvazione di Mish


Inside the `authstore/` is the docker-compose.yml for testing with vault and consul with a README. Based on feedback and performing tests the recommended configuration is a vault server and consul client together that then talks to a consul server. That has been the approach I've been using so far.


Typehint the parameter to be an object.
At least extractNetArchiveLink(stdClass $response) {


Where did you get this BRCMOMX source? @TheNikiz  you wrote it?
anyway, thanks for that.


Ну тогда несколько пожеланий от меня:
1. Убрать `border-radius: 4px;` и `background-image: linear-gradient(...`. При наведении просто делать кнопку темнее.
2. Убрать `width: 120px;` и изменить правило `padding` на `padding: 6px 15px !important;`


teste teste teste


Indeed. The STAGING_BUILD mechanism is only triggered in resin-systemd DISTRO. For these builds try to always use a local.conf similar (or exactly the same) to  https://github.com/resin-io/meta-resin/blob/master/meta-resin-fido/conf/samples/local.conf.sample


Wouldn't the name 'show-only-origin-header' better fit? IMHO it's hard to understand what include-remote-only means.


> Убрать width: 120px;

Потому и добавил, что с паддингами ширина скачет.


Yeah this works when testing zooming back and forth - i.e. when tiles of both zoom-levels are instantly available for rendering. But what happens in the case where you zoom in and it renders proxy tiles first?


Better check first whether $response->identifierInformation is isset and is traversable by foreach.


他にもWindows上でのバグをfixした


That was fixed in master. Please pull the changes.


Docblock for every method is missing.


also, removing the enire alias list from the LOG to aovid a :fire:


`mock_result` is not actually being used?


I think it would be better if this were a public non-static method. Then you could do
```
$project->updateAssociatedActivities($params);
```
instead of having to pass the ID as an argument.


Хотя... Ща еще немного поиграюсь.


Out of curiosity, how did you go about selecting convict for managing configurations?


No indent is required here.


Is it a requirement to add this new device option? Is it a boolean? Reading the whole patch, it seems that action="diag" should work without this new device option? Am I wrong?


You need to you resin-systemd DISTRO in order to have support for all the components.


????? I **completely** don't follow what you're saying, nor do I understand your concerns. I believe there is fact behind your statements, but I cannot comprehend the logic.

case: `httpRequestUrl` is called on an E2->
 - If clientside HTTP is enabled, a new netmessage is created and sent to the owner of the E2
 - The client receives the request and launches a `http.Fetch`
 - Data from the `http.Fetch` is sent back to the server at intervals of 0.5 seconds, followed by the headers of the request for termination of the request
 - The server interprets all that and calls the callback functions.

case: the client is purposefully not responding at all to any requests and is requesting as fast as possible from an E2
 - A new request/UID is set every 3 seconds (by default, according to the HTTP request limit)

nothing happens because nothing will go wrong: see-> worst case scenario
127 players can be on a sandbox server at once: say 126 of them are deliberately taking part in a slow loris attack
every `126/3 = 42` UIDs are consumed per second.

There are 255 maximum UIDs that can exist simultaneously.

The innocent player's HTTP request, which will likely finish **well** under 5 seconds will be safe from the loris attack, as it will take `255/42 = 6.1` seconds for the UID counter to cycle from 0 to 254.


Thinking about this again I would really insist on not having this line here. If there is a bug, or someone else changes the code and makes this method publicly available this might allow abusing it to gain admin access.


Purpose?


What is this commit ?

I do not get it, there are changes I did and changes you did...


This commit also corrects the crosssection given for the mA=160 GeV sync sample.


Can you expand this comment little more and mention that if docker daemon is killed, upon restart it will kill running containers and will call Put() without calling Get().


Empty line.


I would say ignore the ones who want our work to be called theirs, let them
manually change if required,  i guess majority of people will not have a
problem with this.

And if im not wrong java side still sends jhipster, let me check

On Wed, 2 Dec 2015 5:05 pm Julien Dubois <notifications@github.com> wrote:

> That was already the case, in fact I did this to correct that same
> error... Can you check this?
> Then this is something we would need to discuss, but that's a bigger issue:
>
>    - Some users didn't want to have "jhipster" written (they probably
>    work for consulting companies and sell the JHipster code as being something
>    they did manually... and bill their clients for it)
>    - If we go "micro services", then it might be a good idea to have the
>    same headers between one AngularJS front-end and several back-ends -> in
>    that case we should have a specific JHipster header
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/generator-jhipster/commit/015e7451eeb0b5c5fde0013552cb8b6825d1600f#commitcomment-14730182>
> .
>


Yes agree, will see how we can merge these

On Wed, 2 Dec 2015 5:06 pm Julien Dubois <notifications@github.com> wrote:

> OK I didn't see where they were used:
>
>    - I'm rollbacking this, of course
>    - They are the same as our validation issues on the client side, so we
>    should reuse the existing validation keys, and not create new ones -> in
>    the end those keys should be deleted
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/generator-jhipster/commit/0ad7921d79aef018b1412040a7a96d403c2ef07b#commitcomment-14730209>
> .
>


How about naming this variable as 'isMounted' instead of 'isMount'


Not actually sure why this is needed here, since Drupal forbids disable of modules that have their own fields attached anywhere.


the entire method must not be run as admin ....
k, we can add a check to each of the methods to check for admin permission first. It is just not so handy on command line then. But we may want to create a rake task instead ....


Dot.


No indent required here.


Pritaikiau pasikartojanti metoda


This is great, thanks for this :grinning:


葱哥重口味，代码都是F**kMe


This closes #2


fast-forwarding end-to-end/timeseries to borshop-integration-1212-bugfix/jrd/ddl-siblings = 5f8cb21e


Yes, it should probably have been internal_inside. However, I improved this doc yesterday, see commit c8748501f2ea96015bbc81bfb37fd10aa43a33e5.


Ehhhhh en el mar de la complejidad todo es absurdo(?).

Nah esta bien, yo en realidad haría dos sps eso del type test ahí en la base me parece medio una negrada, lo manejaría desde la app y que el typetest de ultima este en la app (caufaldo con algún elemento de interfaz o algo asi) pero tampoco esta mal esto creo. Dejalo asi si queres (saca el ambos) y concentrémonos en lo que sigue. Mientras cunmpla el requisito de ultima después refactorizamos


Typo: object


I don't thing so these two features may work together.
It was safe enough to disallow both a time.

On Wed, Dec 2, 2015 at 2:19 PM, Sean Flanigan <notifications@github.com>
wrote:

> Hi Tibor,
>
> Sorry, but I don't understand exactly what you're asking for here. But
> regardless, isn't rerunCount counting *runs* (eg run each test an extra 3
> times if it is failing), whereas failureCount is counting *tests* (eg if 10
> tests have failed all their runs - however many reruns that is - then skip
> the rest of the tests)? Unless I've misunderstood failureCount? Is
> failureCount counting runs or tests?
>
> I don't think it even makes sense to compare rerunCount with failureCount,
> because the units are not the same. Are they?
>
> On 2 December 2015 at 23:03, Tibor Digana <notifications@github.com>
> wrote:
>
> > Pls take this back and add additional check when this condition makes
> > sense && rerunCount >= failureCount.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/seanf/maven-surefire/commit/f1518a1b4f8ce87cc6f3884550af24594cd98c59#commitcomment-14734966
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seanf/maven-surefire/commit/f1518a1b4f8ce87cc6f3884550af24594cd98c59#commitcomment-14735243>
> .
>

--
Cheers
Tibor


OMG! Yay!!


Si on ne sait pas quel est le nouveau numéro, à quoi correspond celui-ci ?


alternatively, std::pair<int, int>


Why are we doing this check now? Device should always be mounted now. And if it is not, then unmount() will return an error saying it is not mounted. And anyway we are returning an error that device is not mounted. So checking mount.Mounted() seems redundant?


Because in file SnapshotManagerImpl.java in method deleteSnapshot(long snapshotId), line 424 we selecting snapshotStrategy, but only one strategy is available - XenserverSnapshotStrategy. Others not yet imlemented or XenserverSnapshotStrategy covers any situation?


Add drupal_exit(0)


@rysiekblah Tomasz, I moved your and my topics into featured topics. I hope you can talk about Reactive architecture for 10-15 minutes.


Hi @astlock. We still use Dashboard internally, but unfortunately a bunch of Etsy specific changes made their way in that we weren't able to push upstream. We're working on cleaning up our game and making sure that our repos stay in sync, but in the mean time we wanted to let folks know that we aren't actively maintaining the public project.


c'est le numéro de la chargée de com du ministère, qui parait-il reçoit plein d'appels de nos users depuis qu'on a des problèmes avec les mails d'activation.


`exec`.


That's a really clever idea. I think we could definitely move in this direction.


If this is obsolete, why this is still here?


Try to build IT with your code with more realistic counts, but the
fail-fast feature would not be coherent due to concurrency problem across
forks.

On Wed, Dec 2, 2015 at 3:03 PM, Tibor Digana <tibor.digana@googlemail.com>
wrote:

> I don't thing so these two features may work together.
> It was safe enough to disallow both a time.
>
> On Wed, Dec 2, 2015 at 2:19 PM, Sean Flanigan <notifications@github.com>
> wrote:
>
>> Hi Tibor,
>>
>> Sorry, but I don't understand exactly what you're asking for here. But
>> regardless, isn't rerunCount counting *runs* (eg run each test an extra 3
>> times if it is failing), whereas failureCount is counting *tests* (eg if
>> 10
>> tests have failed all their runs - however many reruns that is - then skip
>> the rest of the tests)? Unless I've misunderstood failureCount? Is
>> failureCount counting runs or tests?
>>
>> I don't think it even makes sense to compare rerunCount with failureCount,
>> because the units are not the same. Are they?
>>
>> On 2 December 2015 at 23:03, Tibor Digana <notifications@github.com>
>> wrote:
>>
>> > Pls take this back and add additional check when this condition makes
>> > sense && rerunCount >= failureCount.
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/seanf/maven-surefire/commit/f1518a1b4f8ce87cc6f3884550af24594cd98c59#commitcomment-14734966
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/seanf/maven-surefire/commit/f1518a1b4f8ce87cc6f3884550af24594cd98c59#commitcomment-14735243>
>> .
>>
>
>
>
> --
> Cheers
> Tibor
>

--
Cheers
Tibor


start a discussion about commits


Wrong indent.


the way i read `exists_in_auth_db` tells me that im expecting true or false..

yet, a user instance is returned .. i think this could be named better..

how about `get_user_from_auth_db`


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


Vous pouvez tout mettre dans votre _gulp_ d'un coup, d'un seul.

http://laravel.com/docs/5.0/elixir#usage


один класс - один файл, иначе сумбур


Рекомендую сделать pull :)

Алсо, расскажи пожалуйста подробнее про http://r-ambersky.livejournal.com/283212.html


один класс - один файл


VIP?


Empty line.


Is there a reason you didn't use the filter in cool?
https://github.com/Praqma/cool/blob/master/src/main/java/net/praqma/clearcase/ucm/utils/filters/AfterDate.java


Отличное решение! Можешь строчкой ниже записать пароль от Вконтактика?


<graph> TAGS.


I don't love that we have this fixture defined in three different files, can we add this to spec-helper or something and define it once.  The ideal would be a fixtures folder so that we can add fixtures for different resources over time if we need to.


fixed #88


This method doesn't really "sync" from anywhere, as all you doing is passing the data you want to save.

From reading `sync from auth db` tells me that this function will go and call the auth db.


FYI, that log also shows on iOS 8+ if `self.useAccessControl` is `NO` which is even more incorrect.

Perhaps just take out the else statement entirely?


I'd recommend something like "getID" or "getName" and return `Constants.PROPS_FILE` which will be `"qz-tray"`.  And yes, I realize "PROPS_FILE" is a bit misleading of a property name... we can change that too, but it's consistent with `build.socket.name` from ant, or in 2.0 simply `project.name`.


isn't git.CurrentRef() returns you locally checkout branch?

IMO it doesn't make sense to send name of local branch to server. Branch names on client can be named arbitrarily. Checking permissions against name of the local branch doesn't make sense.

It should be rather name of the destination branch (the one where changes are pushed to).


Also updating to match changes in iftt-norch-tools api


If the eventual purpose of this is to be primarily a client, make sure to illustrate that in the class name.


I think the following should also be inside that function for it to make sense
```
user = KagisoUser.objects.filter(id=data['id']).first()
```


Using POST makes the debugging a tedious task and there's no possibility to invoke the functionality from other places, except POST requests.


lz4 r131 compression speed looks suspicious.
Used to be 492 MB/s in previous lzbench version.
Results might be influenced by external fluctuation ?


The special case below we still need, but we can indeed get rid of this `require`.
Thanks!


So you updated vendor and device tree and will not update the kernel? You are the only person who has access to sources code of camera HAL, vendor init, audio HAL... and now you just drop support? Thanks.


This should not be harcoded - use MEDIA_ROOT and MEDIA_URL and also define the HAB settings in a settings file under nansencloud/processing_hab. I will find a way in the sar nrcs processing app...


Согласен, на футабе градиент выглядит хреново, можно и без него и с острыми углами, как у постов.


Typos: netarchive_id


hehehe


Should be `github.com/elastic/beats/%BEAT%/...`


@Zuchens - numpy można zainstalować z repozytoriów Ubuntu - chyba zainstaluje automatycznie jako zależność python-scipy


This chunk of code has changed upstream.  Please merge in what we have so you don't hit a merge conflict in the PR.

Also, minor, but if there'a way to do without all of the code formatting for this commit, it will make the change a bit cleaner.  Not mandatory though as IntelliJ has its way of doing code cleanup and making a bunch of lines as modified.


Fixed everything, please check latest commit.


Looks sane to me, although I have no idea about the Windows side.


teste aqui


Please don't add project files to your commit.  If you use the IDE to commit your changes, you can untick them.  You'll probably have to create an inverse commit prior to squashing so this one disappears from the commit history.


these properties do not belong in the command controller.

please create a separate class for it


\S ?
AFAIK local id is only numbers.


cool, [wkhtmltopdf](https://github.com/mikehaertl/phpwkhtmltopdf) est installé sur SRVZ. Il permet de généré des PDF à partir d'un fichier HTML avec plein d'option pour mettre/ôter des entêtes/pied de pages, changer le format, [etc.](http://wkhtmltopdf.org/usage/wkhtmltopdf.txt)


I think I've found a way: for all that can't be passed by value I've created temporary fields in `vehicle_priv` and pass a pointer to them when the corresponding attribute is requested. It does introduce five extra struct members but doesn't have any of the ugliness above – no memory leaks or changes to interface contracts, no passing of pointers to internal struct members (other than those created for that sole purpose), and capable of handling multiple vehicle instances.


This commit looks good!
Would you consider opening a Pull Request on the main repository, so that all the simple-nav users can use this? https://github.com/pdf/simple-navigation-bootstrap/issues/13

谢谢了！


Pass it thru code as well.


this may return different values than currently active, as it does not evaluate changes from AdditionalConfiguration.php


needs to be wrapped in escapeshellarg


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


Ah- now it makes sense!


this adds a requirement to *nix systems. we should provide at least a fallback for Windows


Ahh ok yea, will make note of that in the future.


see above. all these methods should be extracted into a different class


Nothing goes after @file directive.


I have the same issue, right now I'm building from this release, I thought my PR to official docker repo would fix this, but it has not...


AL variable?
=> Nil?


А что будет, если после этого пользователь решит обновить страницу в браузере (нажмёт F5)? После POST-а, если это не асинхронный запрос, всегда нужен редирект.


s/Writeable/Read-only/


This was supposed to be entitled `Upgrade to jQuery 1.11.3 as per Bootstrap requirements`. Oh well.


Approved!


Hello, nice work!
I can't find here the new script for Live 9.5...


Add check for close enemy that you don't waste on creeps


How will the settings.APP_NAME function look for the cms?

I assume you will be checking if its jac, ecr or the admin section, return the relevant app name, or will be only be using two app names now?


One could still run:
User.current ||= User.get_default_admin
consistency_check

be it in the consistency test or manually in the rails console


I've confirmed that mroonga 5.10 (next commit) can be built with mysql 5.7.9!!

```log
brew install mroonga --use-homebrew-mysql -v
...
==> Summary
🍺  /usr/local/Cellar/mroonga/5.10: 302 files, 4.9M, built in 13.8 minutes
```


В английском языке форма множественного числа для слов, оканчивающихся на -y, стоится с помощью добавления -ies. Specialities, faculties, etc.


+1 looks good to me


Essayez de faire remonter cette logique-là dans les niveaux au-dessus et ne gardez que l'HTML.

```php
// par exemple.
$members = extractTeamMember($teams);

foreach ($groupes as $groupe) {
    foreach ($groupes->rounds as $round) {
        // ..
    }
}
```


Good point, we can use `Object.keys(pluginSettings)` here instead


After applying these changes alone, the following test case was resulting in an unhandled rejection:
```
it('should return a promise if the endpoint supports the verb', function() {
```

```
Unhandled rejection RequestError: Error: connect ECONNREFUSED 127.0.0.1:8200
    at new RequestError (/app/node_modules/request-promise/lib/errors.js:11:15)
    at Request.RP$callback [as _callback] (/app/node_modules/request-promise/lib/rp.js:60:32)
    at self.callback (/app/node_modules/request-promise/node_modules/request/request.js:198:22)
    at emitOne (events.js:77:13)
    at Request.emit (events.js:169:7)
    at Request.onRequestError (/app/node_modules/request-promise/node_modules/request/request.js:820:8)
    at emitOne (events.js:77:13)
    at ClientRequest.emit (events.js:169:7)
    at Socket.socketErrorListener (_http_client.js:259:9)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at emitErrorNT (net.js:1253:8)
    at doNTCallback2 (node.js:441:9)
    at process._tickCallback (node.js:355:17)
```

So the request-promise was resulting in actual execution of the fake endpoint which does not exist. I made the change to handle different testing environments and then got a 404, which is expected but does validate that a promise is returned (just a rejected one).

So I updated the test as follows:

```
      // promise is returned but it is rejected since the route does not actually
      // exist within the vault server.
      endpoint.get().should.be.rejectedWith(/404 page not found/);
```


You broke the code with this change as fgets() stores the terminating newline char in the string while gets() didn't :-(


utilisez l'ORM. `Tournament::with('equipes')->find($id)`, puis ici ou dans la vue directement, `$equipes = $tournament->equipes`.


Thank you :+1:


Code style: Missing spaces


should qValues be ignored here?


Yes, let’s do that


Yes let’s do that


由于*mHighLightView*的父*View*总为*FrameLayout*，因此如果*mAnchor*不为*FrameLayout*也会进入*if*中，那么添加的*View*不能删除，内存泄漏。


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.607.348e98ea4becca63ea35c5d06f890ffda5c42e55](http://build.palaso.org/viewLog.html?buildId=66764&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:00:39


G


TeamCity KB-QS :: content-editor :: build [Build 5](http://teamcity.nice.org.uk/viewLog.html?buildId=108528&buildTypeId=KbQs_Editor_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:06:46


I feel like at point when requirements are defined name/schema/database should be taken from tableDefinition.
And TableHandle should only be used by user for table lookup. Why do we want to split TableDefinition and TableHandle at this point - it seems unnatural to me.


尝试修复了一下，请翔哥多指教


> Insert(idx, musicSymbol) ==> Insert(idx, Configuration.Settings.Tools.MusicSymbol)

Ah yeah :)


GJ


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


Good Job!


Yes. Here we simply use the first one listed in the headers, not the one with the highest q-value.
(In the probably extremely rare cases that the client lists the same range several times...)


@zloirock what is the recommended alternative for `global`?


add name


> При вводе нового элемента запрашивается у пользователя название сайта - например lenta.ru. Соответственно, в этот момент нужно добавить запись в таблицу Sites (1, "lenta.ru"), и в таблицу Pages: (1, "http://lenta.ru")
Краулер должен обрабатывать эту ссылку?


Une ligne bidon ajoutée pour voir si Patrick reçoit les notifications.
D'abord un push, et maintenant un commentaire sur le dernier commit...


Related to issue #47.


s/convex/JuMP/


Creating the variable, would be best outside of the loop, inside the loop I just need to change the value so I can register the new hook route.


A good catch. lz4, lz5 and quicklz (the fastest levels) are not right. I need to retest.


yep :-)


There is an English translation for the next line here.


Why do we need separate TableHandle and TableName class. It seems that TableHandle can be used in place where TableName is used.

Once as matcher for selecting table.
And once as key specifying actual database/schema/nameindatabase for TableInstance.


I believe this was done because it was the existing policy from the Facebook code.  This repository will be deprecated at some point in the near future and the tools we intend to keep will be put into their own repositories and we will not use the '.exe' name.

This should instead change the makefile directives.  Ex. flash_bios: -> flash_bios.exe: .  Otherwise, with the change you have in place now you've broken the makefiles such that they will always rebuild everything because there is no file called 'flash_bios' to compare timestamps against.  The '-o bin/$@' should have been left as-was.


This caused the build to fail. Fixing.


This was only partially complete, I prefer to use the pattern of ENV || CONFIG || default (3001)
This allows the decision of the port to be made by whatever is launching the app.


I do not like mixing builder and instance methods in single class.


Julien this is simply a naming issue I will rename the package.


Есть какая-то принципиальная разница между этими двумя определениями раутов? Что каждый из них должен делать?


Oh of course, the CAPS NotLikeThis


Huhu bede Cie obserwował czy sumiennie pracujesz :P


are there some extra constraints like:
- if detabase is present then schema is also present
- name is present

which should be checked here.


build.gradle


see the plugins-test.js. If an app follows the new config convention, it namespaces its own config with its own name: https://github.com/eHealthAfrica/kazana-config/blob/17-config-namespace/tests/unit/plugins-test.js#L13


closes 5


Just to keep in mind for future that building strings with "+" is a slow
operation. It is better to use lists and join the list at the end:
https://google.github.io/styleguide/pyguide.html#Strings


As argsToChild is always 0, this commit basically changes the value of `fv` from `sectionTel x - argsToUse x` to `sectionTel x - sectionTel (parent x)`.


This fixes https://github.com/cgeo/cgeo/issues/5304.


closes #5


functor should be set in the constructor


Very nice!


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


It shuld take TableName or TableHandle. Right?


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


I agree. What does for example "remote" mean when getting a HTTP resources over a HTTP proxy?


why does datasource require a name?


@zachthompson should we remove this now that we reverted to HTTP?


Also, please note that replies can be created via the NodeCommentsList endpoint.


hmm, never knew about that...


```diff
- as a fork of the [m0n0wall](http://m0n0.ch/wall/index.php m0n0wall project homepage) Project
+ as a fork of the [m0n0wall](http://m0n0.ch/wall/index.php "m0n0wall project homepage") Project
```


Did you consider using the github markdown gem? Looks like we get some extra goodies:

https://github.com/github/markup


From the [documentation](https://www.gnu.org/software/make/manual/html_node/Special-Targets.html):
> "(...) Any prerequisites on this target are ignored."

So I guess we don't actually need the `%` here.


You may also want to remove this constructor alltogether then...


vote button? is it right?


I don't understand the objective of this target here. Maybe I'm missing something?


@tridge I think the use of `.NOTPARALLEL` here is a great idea! I haven't thought about that. I'm adding this special target to my version of `Makefile.waf` :-)


Для отладочных сообщений нужно использовать Лог.Отладка()


You seem to have deleted the sample config file.


Why can't the normal malloc mutex be used for the pool?
With two mutexes we have to be careful to always takes them in the same order and then there's the overhead also.


`string` is not needed, just put ('res.partner, 'Partner')


`string` is not needed


Decrement this counter back to 0 again, otherwise consecutive tests will fail


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


cc-to-git creates one BaseLineFilter with multiple criteria, making it possible to filter on multiple things but only run the filter once.
There might be room for improvement though, but I'll get back to you with a bunch of questions on that.


But what if `Accept: audio/*;q=0.1, video/*;q=0.2` is given? Am I missing something here?

Ok, think I got it: This is only used by `qValueFor` below whose `mediaType` would match just one anyway, right?

BTW: Looks like you can drop the `x @ ` above.


Константу лучше вынести в переменную. А то одна опечатка в строке и компилятор ошибку не найдет. Я, например, вряд ли напишу слово hierarchical без ошибок 3 раза подряд :)


Oh, judging by the description in the exercise they want no bias, let me roll this back


The text should be 'should keep the counter value unchanged'


@hilld1 RE:

> I get an error with my imports:
>
> ```
> "../cluster.py", line 28, in
> from gcloud_bigtable._helpers import _require_pb_property
> ImportError: cannot import name _require_pb_property
> ```
>
> `_require_pb_property` was removed without updating `client.py`

It seems you've got out-of-date versions of some of the files (or maybe of some `.pyc` files)


Still don't get it `pluginSettings.myplugin.myplugin`?


Repos updated


Thanks, i swear i searched for it yesterday and couldn't find it.


mod_bOOking instead of mod_bOking.


The SSL certificate is for `*.pfsense.org`, it warns when pointed to https://electricsheepfencing.com

```diff
- [Electric Sheep Fencing LLC](https://electricsheepfencing.com "Electric Sheep Fencing LLC homepage")
+ [Electric Sheep Fencing LLC](http://electricsheepfencing.com "Electric Sheep Fencing LLC homepage")
```


нужно использовать Лог.Отладка()


rename `dict_odoo` to `line_vals`.


It is better to use `if c.lower() in dic:`.
Your solution works but creates temporary lists.


`malloc_add_pool` is calling `realloc`, which also locks the mutex.

We can add a new static function which would be something like `realloc_no_mutex`, and `realloc` and `mdbg_realloc` would lock the mutex and call `realloc_no_mutex`. That may be safer.


remove print


+Todo list:
+  Võimalik lisada uusi ülesandeid                           T
+  Kuvatakse failist eelnevalt salvestatud ülesanded T
+  Kui märkida ülesanne tehtuks siis see kustub
+
+Valitud kuu kuvamine:
+  Vastavalt sisestatud kuule/aastale kuvatakse kuu          T
+  Nooltega võimalik liikuda eelneva/järgneva kuu juurde     T
+  Kuud võimalik valida mitu korda
+
+Päevavaade:
+  Päevale klikkides kuvatakse päevavaade kellaaegade järgi  T
+  Võimalik sisestada uusi üritusi
+  Eelnevalt salvestatud üritused kuvatakse päevavaates


Or even more compactly:
```
return c.lower() in "aeiouy"
```


remove this empty line


Here I would build a list that only contains the "bars" as elements and join them later using `"\n".join(...)`.


might be clearer if you renamed `filter` to `filter_value`. was confused why you needed two passes at first


re~~m~~place


Thanks, fixed in cadb7ff


It might be better to use just `split()` which will split on whitespace
otherwise you may get strange effects if you have more than one space.
Then you can iterate over all words directly without the range function.


When looking at the interface it's not obvious what `check_pool_is_null` is used for, please add a comment for that.
Why `int` instead of `bool`?


private method ? should start with underscore


unnecessary comment


if the value is already in aggregation columns it just gets set twice? (i think that's probably ok, just curious)


why not

if (res < 0.f) {
       res += _ang_180; //  res = res + 360 -180;
    }
else
{
   res -= _ang_180;
}
And we gain on float operation


Please move this to xml/Menu/Volunteer.xml. I believe we created the XML manually before civix had that capability, so the file name doesn't quite match the current convention.


Why even have this function.  Why not just raise exception when building the list?
```
    def build_rpm_list(self, image_list):
        """
        Build and return a list of rpms
        :param image_list:
        :return: list
        """
        rpm_image_list = []
        for image in image_list:

             if not image.is_rpm:

                   self._cleanup(image_list)
                   raise ValueError("{0} is not RPM based.".format(image.name))
            rpm_image_list.append()
```


The default was changed from `consul` to `127.0.0.1` as recommended approach is a local consul client.


But what happens if the second character is an "x"? Then it will return `True` and that would be bad.


Sounds good, may I suggest `realloc_unlocked`?


Ah, I was imagining something happened like that. I'll try to figure out a work around.
Thanks for the reply.


I had to change this to include the `return` and `.bind(myVault)`

new line:
```
    return myVault.prepare().bind(myVault)
```


You should add skip_on_cran to these tests (or perhaps all the tests already not run on CRAN?)


I dislike Macro but in Ap_Math.h we already got
#define DEG_TO_RAD 0.017453292519943295769236907684886f
#define RAD_TO_DEG 57.295779513082320876798154814105f
Is there an advantage to redefine it in template ?


no limit is None, not an empty string (and the value should be converted to `int()` unless it's already one, but I'd do it just in case we have crappy data)


same here


Indeed, we could remove this new arg if we replace `malloc_init` (to be removed) with `malloc_add_pool`. What do you think?


Thx.


It should be:
```
if c.lower in dic:
    return dic[c]
else:
    return 'unknown'
```


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


TeamCity Test Team City :: Build Dev [Build 3](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=20&buildTypeId=TestTeamCity_BuildDev) is now running


TeamCity Test Team City :: Build Dev [Build 3](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=20&buildTypeId=TestTeamCity_BuildDev) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


why not just `return None` in the else block right before?


@pjvohra obviously the bracket use makes sense [!] Good.


aqui modifique para que se crearan nuevas listas


Group declarations:

var a = "foo",
      b = "bar";


`convert_to_unicode()` for the title


Group declarations, same as above.


Če bo kriza ti pomoje dovolimo force push :smile:


Solo escucha los eventos de guardar, y cancelar


I don’t know how this one slipped through my careful check-if-it-compiles habit...


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 63](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=553&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:05


while True is cleaner if we want to be pedantic


This produces, noise like the following:
```
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
WARNING: The HOST_IP variable is not set. Defaulting to a blank string.
docker-compose kill  0.47s user 0.05s system 97% cpu 0.528 total
```


Sounds good, please go ahead.


This needs to return an informative error message.


happy to move this to companion object or something


I don't get the meaning of available. It is set to true when the request or the response is null (which I would say is not available).


I don't think this is valid.
open( '/proc/cpuinfo', 'r' ).readlines()
this does not return a filehandle but the lines.


Cool! that's good looking! We need the input at the top too and the correct signal flow will be complete and intuitive as in analog consoles:

In->insert->pan->fader->xmit

Good job man


note that `AnyVal <: Any`, map still works


why not use an iterator?


```
return c.lower() in "aeiouy"
```


Can you add a test for the expected error message?


@VickiG DWMR. Well done here. Shipping code. :articulated_lorry: :checkered_flag: :+1:


Move the number of characters we keep from the MD5 hash, in a variable defining it.


The second return will never be executed. You can also see that in the code coverage.
First return throws you back from the function.


Since we have a factory it is better to make this class not public..


Same here. this always returns `None`.


sorry you're right @frapontillo
I have done some simple things to test the languages and the technologies to use, but now i will learn better the guidelines about the commit messages.


Why removed -Wall ?


accepted


TeamCity Test Team City :: Build Dev [Build 4](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=22&buildTypeId=TestTeamCity_BuildDev) is now running


Ah, I suspected something like that, but I'm not yaml fluent, so didn't catch it :)


TeamCity Test Team City :: Build Dev [Build 4](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=22&buildTypeId=TestTeamCity_BuildDev) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


version = max(version, soversion)


this method is awful ;-)


ffsfdfd


=) Так можно двигаться дальше?


Did not fix EOL error... :/


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


Save a filesystem later and ~3MB by running this as one command:
```
RUN curl -sL -o /tmp/envconsul.zip https://github.com/hashicorp/envconsul/releases/download/v0.6.0/envconsul_0.6.0_linux_amd64.zip && \
  unzip /tmp/envconsul.zip -d /bin && \
  chmod +x /bin/envconsul && \
  rm /tmp/envconsul.zip
```


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


Approvata


I forgot to mention that thanks @zackdreaver


You need to copy handler block, because it usually is created on stack.


Noted, I'll change it later.


TeamCity Test Team City :: Build Dev [Build 5](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=24&buildTypeId=TestTeamCity_BuildDev) is now running


TeamCity Test Team City :: Build Dev [Build 5](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=24&buildTypeId=TestTeamCity_BuildDev) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


Eigen causes a shitload of warning messages which can cause TRAVIS to crash.


1、新增标签


lines = open( fname, 'r' ).readlines()


This decrement is for visual purpose only. The original code from Juan have this too. I think this PR is good now.


For safety's sake, I would add `assert( not supports_syn_id( synid ) );` here as well.


This test's semantics are weird.  The tested endpoint is invalid regardless of method or payload.  If `/v2/nodes/$node_id/comments/$comment_id/` is being tested for 404 somewhere else, this test should be deleted.  Otherwise, the test should be renamed and the method changed to GET.


use intent instead of facebook sdk


@khancyr
I dislike AP_Math and I want to replace it in the longer run totally.
Eigen will not be an addition to AP_Math :D


open( outFile, 'w' ).write( self.toXML() )


remove this please


...and I'm an idiot.  I misread url as `/nodes/something/comments/abcde/`.  Please ignore me.


Ok ;)


This can be one line only (as many others , see below)

value = open( procFile ).read()


sweet code


wasn't this added to prevent events bubbling up on nested datatables? Does that still work correctly?


I think this can be further improved and generalised.


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


I recommend [belly-button](https://www.npmjs.com/package/belly-button). It can take care of adding all those missing semicolons into the code ;-)


not for v10?


I guess this is what Federico wanted, but I agree that then it should just be:

lines = open( '/proc/cpuinfo', 'r' ).readlines()


Thanks! My aim is to have most aircraft with nice splashes :)


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 631](http://build.prxm.it:8811/viewLog.html?buildId=5040&buildTypeId=Jarvis_DocumentStore_CI) is now running


But with the "with open().readlines() as bla" it will crash


TeamCity Test Team City :: Build Dev [Build 6](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=26&buildTypeId=TestTeamCity_BuildDev) is now running


TeamCity Test Team City :: Build Dev [Build 6](http://teamcity.skycms.mediaibox.com.br/viewLog.html?buildId=26&buildTypeId=TestTeamCity_BuildDev) outcome was **SUCCESS**
Summary: Running Build time: 00:00:09


@jensens sorry, how do we fix something on TinyMCE behavior then?


Here we should check if (gputop_gl_scissor_test_enabled) and skip chaining on to the real pfn_glScissor() while we are overriding the scissor


This works but is dangerous. Try this:
```
def dangerous(l):
    new_l = l
    new_l.reverse()

    return new_l

l = [1, 2, 3, 4, 5]
print(l)

l2 = dangerous(l)

print(l)
print(l2)
```


I see, after a rebase I think that #413 fixes the issue you mentioned.
Yep now we can simply use that to get the zooming direction, I'll make that change.


```
In [1]: with open( '/proc/loadavg' ).read() as line:
   ...:     pass
   ...:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-b2e2f48eb17b> in <module>()
----> 1 with open( '/proc/loadavg' ).read() as line:
      2     pass

AttributeError: 'str' object has no attribute '__exit__'
```


That's much better now. Thanks @thewisenerd.

P.S. I still think that AAC-Redux isn't a bad idea just because I rewrote it from scratch.


Please add tests for private_node+public_comment_level+{admin,read_contributor,logged_out_user}.  I know these should behave exactly the same as the other private_node tests, but this is the sort of weird corner cases we want to have tests for.


but you are right that the "with" statement is reserved for opening a file...
I withdraw my comments about just using open(file,'r').readlines() since the with statement includes a handling of the exception

http://www.pythonforbeginners.com/files/with-statement-in-python


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


Same thing as below about the extra ".read()" with the "with"


`reverse_string_list` is computed but never used.
`reverse_string = reverse_string+i` -> `reverse_string += i`


Menu links to individual nodes?


你改完了能测试一下吗。。。。。。。。。。


Compact solution:
```
return c.lower() in "aeiouy"
```


Скачал с оффсайта dmg-файлы и расположил SDL2.framework, SDL2_image.framework и SDL2_ttf.framework в <home directory>/Library/Frameworks, но там его не видит cmake при сборке:

bash-3.2$ cmake .

*** Building for installation into /usr/local

CMake Error at /usr/local/Cellar/cmake/3.4.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:148 (message):
  Could NOT find SDL2_image (missing: SDL2_IMAGE_INCLUDE_DIR)
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.4.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:388 (_FPHSA_FAILURE_MESSAGE)
  extlibs/libSDL2pp/cmake/FindSDL2_image.cmake:15 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  extlibs/libSDL2pp/CMakeLists.txt:35 (FIND_PACKAGE)

-- Configuring incomplete, errors occurred!
See also "/Users/randomambersky/hoverboard-sdl/CMakeFiles/CMakeOutput.log".

Пришлось поставить через brew install ( в /usr/local/ ... ) тогда ошибка исчезает.


I didn't get it. Where do you want to move it?


Likewise, please add tests for public_node+public_comment+{contributor,read_only_contributor,logged_out_user}.


Here you could reuse the vowel function from above.


I just realized the two comments above are probably out of scope for this PR.  But please consider them for your other comment work.


James, I know its just a renaming issue, but still demands attention


Seems to have fixed an EOL error somewhere in the file that Powershell ISE hasn't liked. Hopefully diffs will be correct from now on.


More compact solution:
```
histogram_list = []
for i in l:
    histogram_list.append("#"*i)
return "\n".join(histogram_list)
```


This should be X-CloudMine-UT, not HTTP_


thx !
Ok , i try to put the input on top so .
Actually learning to add pages on master control .


It would be better to just use `split()` which splits on whitespace.
Otherwise you may get strange effects if words are separated by more
than one space.


here + everywhere, maybe have a constant for the various 'cloudmine' header values?


Ca vas pas non !! t'es pas bien hein !!


Hi Frank,

I wrote it when there was a distinction in the functionality based on whether or not certain information was submitted. I'd previously only set the campaign id in the if statement inside createVolunteerActivity, and this unit test proved that it was not the right way - it failed until I moved the campaign id setting to be outside the if statement. I thought it would also be a meaningful contribution in case someone was tempted to change the logic back.

Would you like me to take it out?


Normalized line endings with Get-Content | Set-Content on PsAcano.psm1


fyi https://github.com/NixOS/nixpkgs/commit/cae5bfb991c2e0e22ab5923d968fc83d7450ff24


Makes sense :) I wasn't sure whether or not these were conditions of setting that information for other reasons.


I'll second that. Great suggestion.


An DEFINE like MAX_BOOM_TEST or so would be appricated then this test mode could easier go into master... but cannot have it all ;)  Thx for implementation anyhow to simplify testing life :)


This works but imagine you would try to find the longest word
ever published on Wikipedia. Then you would need to keep
a gigantic dictionary in memory. It might be more economic
to not keep a dictionary but just the longest word found
until now.


« D'après nos itérations (...) », est-ce qu'on pourrait avoir le détail svp ?
Cela me semble complètement irréaliste avec un seuil de 10 % et très hasardeux avec un seuil inférieur.


oops! Yes it is a typo :disappointed:


well spotted :+1:


Agreed - wanted to submit the working functionality though :) I asked on IRC if anyone else had encountered it but no response. Will debug this further.


totally agree. definitely need to refactor that into constants in the default constructor.


I think this is a 0 and the next a 1


Why not default to getMaxThreadsThreshold()


See also https://github.com/NixOS/nixpkgs/issues/8190 after I got frustrated with a similar issue.


**With or without graduations ?**
![capture-3](https://cloud.githubusercontent.com/assets/1283808/11534890/7c1afc8a-9911-11e5-9070-5c5e2fca2896.jpg)
![capture-1](https://cloud.githubusercontent.com/assets/1283808/11534904/907df092-9911-11e5-9615-f7bad48cbb72.jpg)


Done.


What is this method and why do you replace it everywhere?


The use of `.children()` appears to have the side-effect that any non-`HTMLElement` nodes are skipped and therefore dropped from the default transclusion slot. Attn: @petebacondarwin.


Yup!


More compact:
```
for i in s.lower():
    if i not in "atcg":
        return False
return True
```


Added changes for pre and post plots as discussed


it will https://github.com/runcom/docker/commit/4a7bfd9c531ca30a837b3cd3d584868c289be1e4#diff-b56258ecab11f7b797306907327cc904R303

Zero it's just a placeholder to say if the option was passed in the cli


Here one could use a dictionary, too.


"Listings" is pretty broad. How about "roster" instead? Git rebase will allow you to change this commit and run all your other ones with few or no subsequent changes.


If we default to getMaxThreadsThreshold above then this code can be removed.  And you just call

debug.SetMaxThreads(config.MaxThreads)


```
return s.upper().replace('T', 'U')
```


which just means I didn't want to hardcode a somehow *dynamic* value here


Whoops, forgot to remove that!


Here you could reuse the pair function from above.
Also check out the map function in Python.


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


what do you mean? colors?


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 631](http://build.prxm.it:8811/viewLog.html?buildId=5040&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 255 Build time: 00:12:27


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 632](http://build.prxm.it:8811/viewLog.html?buildId=5041&buildTypeId=Jarvis_DocumentStore_CI) is now running


Did you guys test this? It should almost certainly be "stack.peek().Equals(answer.peek().ToString())"


Or reusing previous code:
```
return reverse_string(get_complement(s))
```


I guess I should add some context here.

In yaml-cpp 0.5.1, the `node["missingKey"]["irrelevantKey"].as<>(defaultValue) ` construct actually worked due to an oversight in yaml-cpp; `node["missingKey"]` should have returned an invalid node, and any operation on it should have resulted in an `InvalidNode` exception. However, apparently that didn't work before an overhaul in 0.5.2 and actually resulted in memory leaks. Current git yaml-cpp (as of the time of writing) correctly throws an `InvalidNode` for missing (null/invalid) nodes.

So this is more of a preventive fix for newer yaml-cpp libraries.


check next commit


It has been fixed in devel


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


I wanted to make sure `config.MaxThreads` won't overflow the kernel setting https://github.com/runcom/docker/commit/4a7bfd9c531ca30a837b3cd3d584868c289be1e4#diff-b56258ecab11f7b797306907327cc904R296


论cpplint养成的奇怪风格= =


Hi Una I spotted second head in your site, see error below:
"Start tag head seen but an element of the same type was already open."
https://validator.w3.org/nu/?doc=http%3A%2F%2Funa.im%2Fclassy-css%2F


Yup!


Inutile, `title_activity_list_books` existe déjà !


Whenever you need the element from a list and also its index, use `enumerate()`:
```
for i, k in enumerate(kmer_list):
    ...
```


Nah .... look on the 2nd picture , the knob has graduations around , but the original ( pic 1 ) not .
It's a thing i just discovered , do you like it ? ( that could be enabled/disabled in an option panel too )


Got a bit rushed, which is why I had to fix obvious mistakes. Its also why the same git comment is showing up on several pushes.


Good advice~ , It is unnecessary code,  I will remove it.


Sure


Depends on the goal, right? A fixed-output derivation could still try to tinker with the rest of your system. It's still hindered by a minimal-power user, of course.


Thank you! It is what I was looking for


Also, since the '<' and '=' differs between each string, we would need to duplicate both the kana rule string and width rule string.  Once for '<' and once for '='.


I believe the `#define inline _inline` in this commit may be what [MPR#7071](http://caml.inria.fr/mantis/view.php?id=7071) is complaining about.


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


ok


oh...yeah....that looks good.


Possibly make countryCode() method to show that you're instantiating autocomplete just for that. Also, add a comment in the space where people would create their own instance of autocomplete going forward.


This should be `if isDefined(a[k])`. Also, I would write

```
var c = None
if isMap (b)
    c  = reduce (b, {r,_,k|if isDefined(a[k]) -> r[k] = v}, {})
elif isList (b)
    c  = reduce (b, {r,k|if isDefined(a[k]) -> r[k] = v}, {})
end
```

This will save the iteration in `keys()` if it is a `Map`


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


Pas la peine de recréer un layout. On peut utiliser `activity_book_display`.


If we really need it to avoid a circular import, put the import at the top of the function.  Having it in the middle of running code is really unusual.


no...
Thats not how the internet works. You send a redirect with the response.


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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1086.184c098ca43e7ab26b585367001c4fc5fb5bdc55](http://build.palaso.org/viewLog.html?buildId=66766&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 492, ignored: 4; exit code 1 (new)) Build time: 00:22:41


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


Done


More compact:
```
return c.lower() in "aeiouy"
```


`count=count+1` -> `count += 1`


`JSON.parse('stringValue')` will throw a `SyntaxError`, and `$scope.preselectValue` will get set to `''`. Is that what you want?


It's confusing to me man been working on it on and off for like 2 months:/
On Dec 2, 2015 2:26 AM, "Andre Caetano" <notifications@github.com> wrote:

> PHP Are cool. It's very simple.
> I begining on PHP. [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KodyBentley/MyFirstWebsite/commit/340f60b143f37bdca07aac18e89ea5e17474e0fc#commitcomment-14731840>
> .
>


Looks good, :+1:  - really needs tests but in the interests of getting this in for your deadline, will merge once you resolve conflicts. Please make sure you come back to this and write a spec!


Remember about!
ArchiDroid/android_build
ArchiDroid/android_frameworks_av
ArchiDroid/android_frameworks_native
ArchiDroid/android_hardware_libhardware
ArchiDroid/android_packages_apps_Settings :+1:


Since you don't need the indices i and j directly you can iterate over integers directly:
```
m = []
for i in l:
    m.append("#"*i)
return "\n".join(m)
```


@matheuscfernandes We now get a 25x speedup over cython on my GPU as opposed to 10x!


Done. Updated NEWEST for Java 7 compatible VMs to 6.9.9 and modified MultiVersionSpecRunner to prevent testing of duplicate versions.


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


userService.getAuthorizedUser() рулит


Done. Modified MultiVersionSpecRunner to prevent testing of duplicate versions.


> Ok, think I got it: This is only used by qValueFor below whose mediaType would match just one anyway, right?

Yes. We only need to establish an order between *overlapping* ranges. If two ranges don't overlap their order doesn't matter.

Right about the surplus `x`. Will remove.


Multiple return statements? Doesn't that mean only the first return is called, and all the others are lost?


It happens.  No big deal. :3


Good catch! Done


It is better to use just `split()` here. Compare:
```
>>> "one  two three".split(" ")
['one', '', 'two', 'three']
>>> "one  two three".split()
['one', 'two', 'three']
```


Why are you updating this file.
We don't even use it anymore so there is no need to edit grammar as it will never see the light of day


Zeby to zrobic mozesz dodac klase do tego diva `class="text-center"`, poniewaz button ma defaultowo `display: inline-block`  ktory sie zachowuje podobnie do tekstu


It is better (faster, less memory) to check `if c.lower() in dict:`.


It looks like `exitCode` is actually a field for each `Container` object within a `Task`, not the `Task` itself, according to the [DescribeTasks API](http://docs.aws.amazon.com/AmazonECS/latest/APIReference/API_DescribeTasks.html)


So the controller redirects back to the login page, that means all we need to do is link to the logout URL here, no need for a new controller.


```
return s.upper().replace('T', 'U')
```


Yes, sending only the color should be enough when it is on.

There is a test to make sure the colors show up in the state so I'm not
sure what's going on. I'll have another look

On Wed, Dec 2, 2015, 00:37 hexxter <notifications@github.com> wrote:

> yes this version works :)
> but the icon doesn't change the color to the color of the light and it
> send on at every color pic:
>
> 1449045015.7883866 stairwell/rgb1/rgb/switch b'on'
> 1449045015.7928092 stairwell/rgb1/rgb/status b'on'
> 1449045015.792987 stairwell/rgb1/rgb/color/set b'0,218,180'
> 1449045015.8329666 stairwell/rgb1/rgb/color b'0,218,180'
>
> but if the state is on it should only send stairwell/rgb1/rgb/color/set
> b'0,218,180' ?
>
> BR Hexxter
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balloob/home-assistant/commit/90eab17ea6acb26bb2824584c2d64b27343922de#commitcomment-14729693>
> .
>


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1087.184c098ca43e7ab26b585367001c4fc5fb5bdc55](http://build.palaso.org/viewLog.html?buildId=66768&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 734, ignored: 6 Build time: 00:05:51


After start server, I see the error:

 INFO [NetService] - Starting MiNET
 INFO [iNetServer] - Initializing...
 INFO [iNetServer] - Loading plugins...
 INFO [iNetServer] - Plugins loaded!
ERROR [iNetServer] - Error during startup!
System.MissingMethodException: Method 'Array.Empty' not found.
  at MiNET.Worlds.LevelInfo.LoadFromNbt (fNbt.NbtTag dataTag) [0x00000] in <filename unknown>:0
  at MiNET.Worlds.LevelInfo..ctor (fNbt.NbtTag dataTag) [0x00000] in <filename unknown>:0
  at MiNET.Worlds.AnvilWorldProvider.Initialize () [0x00000] in <filename unknown>:0
  at MiNET.Worlds.Level.Initialize () [0x00000] in <filename unknown>:0
  at MiNET.LevelManager.GetLevel (MiNET.Player player, System.String name) [0x00000] in <filename unknown>:0
  at MiNET.MiNetServer.StartServer () [0x00000] in <filename unknown>:0
 INFO [iNetServer] - Disabling plugins...
 INFO [iNetServer] - Shutting down...
MiNET runing. Press <enter> to stop service..


Here I would reuse the pair function from above.


My concern is what if the project's campaign is being unset? I.e. it used to be part of a campaign but then went on to become its own thing, or it was erroneously set to be part of a campaign. It'd be worth scripting some UTs to cover that instance IMO.


更新


Here you could reuse the `reverse_string` function.


You didn't actually need to do this, it just wraps the rand_name call in 'tempest-' (or other configured) prefix. In a real plugin we would avoid importing from tempest directly.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 633](http://build.prxm.it:8811/viewLog.html?buildId=5042&buildTypeId=Jarvis_DocumentStore_CI) is now running


this addresses #7


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 633](http://build.prxm.it:8811/viewLog.html?buildId=5042&buildTypeId=Jarvis_DocumentStore_CI) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:00


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 634](http://build.prxm.it:8811/viewLog.html?buildId=5043&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 632](http://build.prxm.it:8811/viewLog.html?buildId=5041&buildTypeId=Jarvis_DocumentStore_CI) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 221) Build time: 00:13:13
### Failed tests
```
Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_multiple_password("v1").verify_tika_job_with_password: <no details avaliable>

```


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 634](http://build.prxm.it:8811/viewLog.html?buildId=5043&buildTypeId=Jarvis_DocumentStore_CI) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:00


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 635](http://build.prxm.it:8811/viewLog.html?buildId=5044&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 635](http://build.prxm.it:8811/viewLog.html?buildId=5044&buildTypeId=Jarvis_DocumentStore_CI) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:00


Same thing here


Here I would like to see each combination of {public,private} x {comment_level_public,comment_level_private} x {admin_user,ro_contributor,non_contributor,not_logged_in} tested.  You've already got most of them.


LGTM, although you undid some of the local plugin changes I made originally. I'll see if I can figure out how to fix that before merging this


TeamCity Habitat :: Cruise [Build 314](http://build8ua1:8081/viewLog.html?buildId=788&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 56 Build time: 00:01:13


Great

Thanks @zgnachvi


You basically are just implementing <a> but without following the link, we don't really need this.


Thanks, Greg! I agree more needs to be done with handling failed containers, and I also need to add unit tests to cover this.

I think it will break as-is (see in-line comment) but looks like a good start to handling errors correctly.


Yes. Sorry for that. I have fixed the code for future courses so that this problem does not appear.


google api feed is deprecated, we could move to YQL  https://query.yahooapis.com/v1/public/yql?q= ..... &format=json&callback=


I noticed there were two builds for 2.2.3 and rails=master tested in travis ci. is this matrix include redundant at this point?


Are we releasing the stored client connections? Should we have some cache policy to close the connection after   a configured inactive time?


@gravgun Most if not all of this document should be in the specification repository, no?


Please send upstream.


But LGTM.


[MPR#7071](http://caml.inria.fr/mantis/view.php?id=7071) discusses the `#define` in this commit.


Not 100% sure if this will work on 9.1


For the tests with 40x errors, please test the error message to make sure we're being rejected at the appropriate level.


The other one would be dangerous. Try this:
```
def dangerous(l):
    l.reverse()
    reverse = l
    return reverse

l = [1, 2, 3, 4, 5]
print(l)

l2 = dangerous(l)

print(l)
print(l2)
```


These adapters are so that we can pass an model object to the db cursor as a parameter and it will know what to do with it.


More compact:
```
return c.lower() in "aeiouy"
```


Will look after that, thanks. Google api feed is deprecated, but still works ;)


This just seemed like it could be useful. Also thought of adding an 'edited_by' field


The nested for loops in this constructor do nothing.


cool


Urgs, this profile weights almost 50 MB.
I think way too big.


Imagine you would count the number of vowels in entire Wikipedia.
Then you would need an infinite list which would fill the memory.
But you only need to remember the total number.


Hvala ^^ Bom videl ce bo potrebno :)

Jure
On Dec 2, 2015 4:02 PM, "Tadej Novak" <notifications@github.com> wrote:

> Če bo kriza ti pomoje dovolimo force push [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FMF-studenti/studentski-svet/commit/bc5e855d5080b26482af2293ed7e728c9c58ecec#commitcomment-14737647>
> .
>


v10 wasn't supported before, so there is nothing to improve on ;)


@Nico87LBS please add `tests` for this feature before creating your pull request. :+1:


That was the original `isNormalCube`


Führen Sie diesen Test idealerweise im Controller durch und geben sie eine gefilterte Liste an die View.
Die Filterung im Controller können Sie z.B. mit .grep auf der Liste machen


тогда надо убрать getUserFromSecurityContext() из контроллера, да?


I would set the variable "value" to be something more meaningful like "push_id"


This is a noop operation


@alysonla  Or using `gh.js`:

```js
var GitHub = require("gh.js");

var gh = new GitHub();

gh.get("/orgs/github/members", { all: true }, function (err, members) {
   alert("GitHub has " + members.length + " public members!"); // :-)
});
```

Also, maybe [`github-org-members.js`](http://ionicabizau.github.io/github-org-members.js/github.html) could help you as well. :smile:


Should add test coverage for this new exception


@lmr thanks! I'll definitely pull your patch!


I changed `.children()` to `.contents()` instead and the non-element nodes are properly transcluded and no errors are reported. I have not run tests.


Заебался я то-то высчитывать эти сдвиги изображений. Пусть CSS занимается этой работой. Единственный недостаток: изображение будет сначала сжиматься до нулевого размера и спустя мгновение полностью разворачиваться.


I added these just so that they can't possibly get out of sync


You could get rid of code duplication here. The command line you generate is almost the same.


I believe a push can be composed of many revisions.


Sim serão Jonatas


thanks


If `isSortable` is derived from `canEdit`, then `isSortable` should be a `computed`.


This is a bit ugly but in order for this to be atomic it needs to be in a single function


Das Benutzerobjekt in der Session zu speichern ist bei Grails heikel, da dieses an eine Hibernate-Session gebunden ist. Dadurch können Zugriffsfehler auftreten. Idealerweise speichern Sie in der Session nur die Benutzer-ID, diese ist mit dem Datentyp Long unproblematisch.


Too much honour. If you want to add my name, please write Stefan Weil, UB Mannheim, DE.


Also here for an gigantic DNA string you need to keep a gigantic list and it will blow the memory.
```
for c in s.lower():
    if c not in 'actg':
        return False
return True
```


Thanks was about to test and push this. Thanks dude


Dieses Feld sollten Sie nicht ausfüllen (mit value)


`map` is a reserved name for a function so it is not a good name
(you see that the color changes).


Could add more checks here like `case_id = commcarecase.case_id`


Have you considered doing a scope import on `pom-imagej` or `pom-fiji` to avoid needing to declare all the ImageJ version numbers? http://imagej.net/BOM


More compact:
```
return s.upper().replace('T', 'U')
```


Lets add info to the gem README for all of this new event stuff and distill what we're learning about how to use it from AK support so we don't forget in a few months.


xxxx


Not sure what others think about this. It will abort the transaction.


We could see minor performance benefits by making these `pureComputeds`.


bitte entfernen Sie noch den target-Ordner vom Repository (via .gitignore)


Yes, this is where knowing the quirks of the GitLab API comes into play: the only way to access project/repo is to use a `%2F`, but the `URI.escape` escapes the percent symbol. See https://github.com/gitlabhq/gitlabhq/issues/8290 for more details.


The problem isn't `awk` itself. Its the fact `awk` is run in each `IV`-iteration.

`awk` needs longer for each separator found, or simpler the bigger the file is.
So the decryption takes even more time the more often the slow `awk` call is done...

Split the file before the loop eg. in a temp-file:

```sh
N=0
base64blocksFileName=$(mktemp)
echo -n "$encFileContentsNOHEAD" | sed -r "s/00iv00.{16}xx/\n/g" > $base64blocksFileName
for IV in $(echo -n "$encFileContentsNOHEAD" |LANG=C egrep -o '00iv00.{16}xx' |sed -r 's/^00iv00//;s/xx$//'); do
    N=$[N+1]
    #encFileContentsBASE64=$(echo -n "$encFileContentsNOHEAD" |awk -F '00iv00................xx' -v N=$N '{print $N}')
    encFileContentsBASE64=$(sed "${N}q;d" $base64blocksFileName)
   ...
done
```

Using this optimization made the 2.3M large ocManual decrypt in about 50s than one minute on our machine, compared to about 150s. As mentioned above this saving should impact even more the bigger the file is.

Thanks to @dlmueller who helped me a lot with that.

One personal thing to my mind. Error output should go to STDERR, eg.
```sh
echo "decryptFile():: File "${USER}/$encFilePath" does not exist...! Aborting." >&2
```
This would keep the output files empty on error ;-)


Bitte speichern Sie das Passwort nicht als Klartext, sondern als SHA256- oder BCrypt-Hash


:+1: This is certainly an improvement.


Maybe digging through the documentation will help https://www.mercurial-scm.org/wiki/ChangeSetID
There are various concepts involved.


This needs an addition to `config-dos.h`:
```
--- a/config-dos.h    2015-03-07 11:44:45
+++ b/config-dos.h 2015-12-02 16:51:04
@@ -42,6 +42,7 @@
 #define HAVE_ARPA_INET_H       1
 #define HAVE_ERRNO_H           1
 #define HAVE_FCNTL_H           1
+#define HAVE_FREEADDRINFO      1
 #define HAVE_GETADDRINFO       1
 #define HAVE_GETNAMEINFO       1
 #define HAVE_GETPROTOBYNAME    1
```

Any other?


You are correct. I fix.


Or more compactly:
```
return c.lower() in "aeiyuo"
```


Now Okay?


Oh wow, very nice @Jerskouille :+1:


:volcano:


I currently exclude the whole `tests/` directory in `.Rbuildignore` but I agree we should have `skip_on_cran()` anyhow in case we wanted to export some subset of tests in the future.


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


Here you could reuse the function from above.


Will do a new Build, if all is okay. I get a S3 :dango:


The impersonation client leverages the non-impersonation client's cache. I assume we have the existing code to close the non-impersonation client's cache?


Try to also look at this code and see what you can use:
https://github.com/mozilla/version-control-tools/blob/master/pylib/mozautomation/mozautomation/repository.py#L229


@jcbf - I have changed your email addresses so you have proper credit on GitHub :-)
http://stackoverflow.com/questions/2919878/git-rewrite-previous-commit-usernames-and-emails


More compact:
```
out = []
for i in l:
    out.append("#"*i)
return "\n".join(out)
```


@e-kolpakov Thanks! With this change, the parameter feels more prominent and documented.


Shouldnt this be EditFailedEx... since we are in edit operation ?


This binding handler is generic; it shouldn't know anything about contributors.


non-impersonation client is only one instance per Drillbit and we override the close method to be a no-op.


Kanger XD
Replace the Radium screenshots, etc. .


space missing


@ChaelKruip , in the latest file, is `households_space_heat_hybrid_heatpump_air_water_**electrcity**` (asterixes mine) a typo? You've used it consistently, so there probably isn't a real problem, but I might be good to correct it for administrative purposes. :wink:


Do you need to verify that `x[0] == initial[0]`?


Ok. Should I add a service class? What do you think?


More compact:
```
out = []
for word in s.split():
    out.append(len(word))
return out
```
Even more compact (list comprehension):
```
return [len(word) for word in s.split()]
```
The "pythonic" way is to avoid looping
using indices but rather iterating over the list members.


This commit is being reviewed as a part of review issue #140.


Looks Good!


Imagine you have a really long DNA string and already the second
character is wrong. Then your solution will still continue and check all
remaining ones. Function should `return False` immediately
when you hit a wrong one and stop checking.


this should be done for low deps too (as low deps might get deprecation warnings between libraries)


@jdubois you are right. I feel dumb i was referring a generated project, i should get some sleep :P


More compact:
```
return s.upper().replace('T', 'U')
```


Should we put a comment above the two "boolean" ones to say they are private hacks and shouldn't be relied upon externally?


doing now lol


can you add some more context about this change? i don't fully understand it


Here one could reuse the pair function from above.


After the full stop?


Done!


error in the last comments, the path we're checking in the code is actually:
/usr/share/kodi/addons/script.openwindow/addon.xml

disregard the previous comments about it being var/


neat


I have solved this using a dictionary where key is the 3-mer and value
is the count. Then you can convert a dictionary to a list of tuples and sort
the latter.


Ой, думаю, словосочетание оценочное множество некорректно было использовать.


There should be a static method which modifies context - so static helper "query" method can be used.


Can we just put a warning message which triggers if mountCount goes negative.

if (info.mountCount < 0) {
    logrus.Warnf("devmapper: Mount count of device went negative.")
    info.mountCount = 0
}


Lol.. i was just browsing across repoz.......


is this the function name you meant to use?


The code[1] seems to close the client.

[1] https://github.com/apache/drill/blob/master/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/schema/HiveSchemaFactory.java#L220


closes #231


Mistake will remove...


Without `"with-fortran"` the build fails, and mpif90 seems to be broken. Adding this causes OpenMPI to be rebuilt, which takes forever, but also causes the tests to pass...


is close() always called? I would rather have new connection here.


OK this exercise was not ideally documented by me but adding js and xs should not be necessary.
The idea was to get the least frequent character that is still there (that is "non-zero").


For reading files it is good to use the `with open(...) as foo:` approach.
This will make sure that files get nicely closed even if an exception happens
in the reading.


Let me fix it.


I'm not necessarily expecting other exceptions, but since throwing them kills this entire task (preventing other emails from being sent) I'd rather handle them on a per subscription basis


works


s/Wrapping/Delegating/


See NonCloseableHiveClientWithCaching.close() which is override to be a no-op.


(i guess it might be, just reads funny)


name should be handle


I think this is not the right place to set mountCount=0. First of all it tracks how many Put() have been done and resetting it should not have to be dependent on device deactivation being successful.

In normal circumstances, there is no need to reset it. Only time we need to reset it is when Put() is called without Get(). Or when mountCount() goes negative. So how about moving this code up in an if loop and do this only if mountCount is negative.


eventName


RE: #448


ha, it is, to be read "Get `bulk_get_users_by_id` `ESQuery`". Admittedly a pretty confusing name—the upside is I remove this function in a later commit :)


Why do you have this one with the tilde in the name and the correct one? I have seen that in several of your files...


It's an empty rule other targets can depend on if they need to "force run". Nobody is currently using it so could be removed.


@RubenVerborgh Excellent! :)


SignaturesToDecode


Right.


Not really fond of this one. AudioEntryPanel shouldn't be aware that there's a difference between fluidsynth and timidity; that's for the MIDI player to worry about and the MIDI player only. And getPosition() works with both players (there was an oversight before, but I fixed it).


could reduced to be `(page is None) != (offset is None)` (but is a bit clever)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1088.de1eae801537ff98fcd8006c3c157aa84a0452c4](http://build.palaso.org/viewLog.html?buildId=66769&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 492, ignored: 4; exit code 1 (new)) Build time: 00:08:33


the page/offset distinction might benefit from a doc string


use typeRef local


Compile error?


Good point, will PR this change


cygwin works with the Linux code and fails with the Windows code, so it must be removed here.


at this point we re-compress + store string. can we make this go away ?


Thanks! :)


missing argument


i.e. we can do this magic when the file is actually used?


this is nice


Oh man I can comment on commits.

Also you can use a block comment like
'''
block
comment
'''
If you want


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


This results in WinError 5 Access denied.


I'm wondering the performance for SM here.


Shouldn't a bit in `version_info` be set to reflect this (as for `USE_NGHTTP2`). I.e.
```
+ #if defined(USE_LIBPSL)
+   | CURL_VERSION_LIBPSL
+ #endif
```

And in `<curl.h>`:
```
#define CURL_VERSION_LIBPSL        (1<<20)
```


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


More compact solution:
```
    d = {}
    with open(file_name, 'r') as f:
       for c in f.read().lower():
           if c.isalpha():
               if c in d:
                   d[c] += 1
               else:
                   d[c] = 1
    # convert dictionary into a list of tuples
    l = list(d.items())
    # reverse sort list on the second items of tuples
    l = sorted(l, key=lambda x: x[1], reverse=True)
    most = l[0][0]
    least = l[-1][0]
    return (most, least)
```


Yeah good point, will PR this change.

It sounds like you got it, but the point here is that if there's any aggregation, then things also need to be split out by the filterable column as well and that column needs to be added to the data rows, so that we can do inline filtering on the mobile.


Looks awesome. Thanks Gilbert for claryifying the License.
per your comment, I want to make sure I am not forcing your hands to make it GPL. If you wish /need to release it with another license due to restrictive material use, I will just switch the repo to the non-GPL section.
Cheers :dart:
IH-COL


(That was the thing that I came to you flustered about, and we decided I didn't really have to do. It turned out to be really easy after I understood the whole data_source thing, so I implemented it anyway rather than changing the example I was using in the test.)


You want to add

\`\`\`python
*code*
\`\`\`


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1089.6e10a76136e7e09f629aa134e829a8d1506a1445](http://build.palaso.org/viewLog.html?buildId=66771&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 492, ignored: 4; exit code 1 (new)) Build time: 00:05:56


:+1: yep makes sense


And this one I think should be ## instead of #.


Wasn't too sure about that to begin with. Changed it now.


moduleHandler is not a method but an object.. You should remove the parentheses


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1091.6e10a76136e7e09f629aa134e829a8d1506a1445](http://build.palaso.org/viewLog.html?buildId=66774&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:29


Thanks for the pull request. Any chance you can provide a test for this?


I get the point. Can be done, but I'm thinking whether it's worth?

This applies only to .js files (and not even within node_modules), so should not take much time during packaging. On the other hand during package execution we would spent some CPU ticks on that. What do you think?


Should read: git pull --rebase upstream dev


Hi,

I have Swift proxy server send stats to statsd using informant middleware, and when run this from proxy-server daemon, the statsd receive information correctly, but when i move proxy-server to apache+mod_wsgi, the stastd stop to receive information.

Do you know any issue with mod_wsgi and informant?

Regards,
Carlos Rodrigues


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1090.6e10a76136e7e09f629aa134e829a8d1506a1445](http://build.palaso.org/viewLog.html?buildId=66772&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 491, ignored: 4; exit code 1 (new)) Build time: 00:03:21


These do not appear on separate lines, but should.


Haha.


I'm going to check it, but from what I remember the whole new packaging was somewhat faster on SM anyway.


The math should be done in 5c4b664


Was just trying to understand the files and I just changed those small
things as I was going through it.

On 12/2/2015 12:11 PM, gaby-baby wrote:
>
> Why are you updating this file.
> We don't even use it anymore so there is no need to edit grammar as it
> will never see the light of day
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikenpeacock/cs3716/commit/e495843f4c895d6b080d802a88cfcd1feb656494#commitcomment-14738789>.
>


i think it would be more intuitive to use sort_by as that is the param in the command


It's past Feb 2015 now. Should this begin/rescue stuff be removed?


would it be hard to forklift these functions from bolt to use here?

https://github.com/boltdb/bolt/blob/master/bolt_windows.go#L84


Sorry for the mistake. I just do not know how these conflicts came up: I did not remember there were conflicts when I committed that time. I just changed files from commit 966aab0, git add and committed. It should not bring conflicts...

Anyway, it has been resolved. Sorry again.


The reason for adding this is to avoid try/catch in plugin definitions. I check for nullptr instead of catching an exception since that would be faster and cause less overhead.


input at top , logo still at bottom , master is a fake .

![capture-2](https://cloud.githubusercontent.com/assets/1283808/11536632/9504eba4-9919-11e5-88a2-4d99da342efa.jpg)


>press press  ??


This is indirectly tested through the test-suite.


saw approval from macintux
at https://github.com/basho/riak_kv/commit/74df46b783ae51e4a50c96203908f0a871018ee5


merging basho/riak_kv/feature/at/time_query_must_be_range = 74df46b7 into borshop-integration-1271-feature/at/time_query_must_be_range


basho/riak_kv/feature/at/time_query_must_be_range = 74df46b7 merged ok, testing candidate = 31676597


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1092.6e10a76136e7e09f629aa134e829a8d1506a1445](http://build.palaso.org/viewLog.html?buildId=66776&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 734, ignored: 6 Build time: 00:05:39


First I would like to say, great job @jlindop. I would also move urlDecodeConstant() and urlEncodeConstatnt() from AssignemntData class to this helper class. What do you think about it? I would also add some tests like if string is encoded with with quotes and number without.


Yes, let's fix that


Отлично, похоже наконец-то везде всё работает просто идеально, да ещё и само!


oopss; fixing this


Uploaded!


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


@kevmoo I expected

```=> '$firstName $lastName'```

Does it matter, either way?


I like the pound sign :( It gives a nice special feel to the line.


This should never happen and if it does I think it's fair to bail out with an exception. I wouldn't add a new-test case for what is basically an assert.


Yup, that works just fine.  I'm curious as to why the stand-in structure is even needed if there's a view, but that's a separate discussion :)


I think all these change are made by me on Sunday


Is this somewhere where a template mixin could be used?


ditto


I don't know what the dmd policy is for including style edits (not that I disagree; the line is indeed too long) in arbitrary PRs, but even for the sake of later rebasing, or even merging, and other people's rebasing, I think that you should avoid it.


This is a nitpick.  Not sure that all these newlines are necessary here and below, except around the ifs.  These can probably just sit as one block.


ditto about style edits


dd


This can probably just be ||ed into the above if, as the return is the same.


I just discovered a new bug here: the stjs.Map type (which translates to a simple JS Object) also implements Iterable, but the generated JS code must be exactly the same as for Array. Right now it uses the Iterator code path, but it should use the same as Array.


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


I think it would be more efficient to change update_fee_delta to also update the descendant state.
That way resorting only happens once.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 81](https://teamcity.labkey.org/viewLog.html?buildId=288938&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14 (1 new), passed: 33 Build time: 01:10:42
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

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

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

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

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

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

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


It's unsafe when unwrap `indexPath` like that. Use this instead:
```Swift
if let indexPath = tableView.indexPathForCell(sender as! UITableViewCell) {
  navigationController.currentTweet = tweetList[indexPath.row]
}
```


It is better to change vnc_port to vm_port.


Geil :D


It is better to change wait_vnc_port_open() to wait_vm_display_port_open()


"the filenames passed to the server don't need to match the real filename on the roslyn server"

How does that work?  I can't get omnisharp-roslyn to start up (on MS runtime) with a cygpath for the solution, which is what omnisharp-emacs is going to provide afaik.

Sounds like this is more of an omnisharp-roslyn problem than a omnisharp-emacs one, but I'm just wondering if you got it working.

My only thought is to build mono on cygwin and run it with that.


Even though the name is normally at the bottom in consoles (this is to be easy to write with a pen)....It looks good at the top.


@kwalrath you're suggestion is right. We should encourage folks to use string interpolation here


need to update log as well to replace 'vnc' to 'vm display'


Can we delete the branch?


This style fix comes from #5239. Total style fix at once was rejected, so I'll incrementally fix them.


thanks! i fixed this!


Please add a blank line between 2 functions.


Hi Phil, I am new to github and recently started working on bootstrap html and conversion to WordPress.

I tried using the plugin along with the bootstrap navwalker. Interestingly the naval works on desktop but in the offcanvas mode on mobile, the navbar area is empty.

I have been trying to work it out for a few days now but to no avail. Wondering if there is a way out on this?


I don't think `skip_on_cran()` is quite right - they should be skipped whenever RStudio isn't active


Ideally we wouldn't recurse into the body if `name` is the same as `old` and error out if it is the same as `new` and the body contains a reference to `old`, but this is just a test


Newlines comes from #5239.


Although SnakeYAML emits some different output than libYAML, libYAML can still parse it as per my tests using PocketMine-provided Windows binaries.


What's wrong with this `}`? :smile:


good point, will add


ditto from #5239.


sending pr is different thing?
https://github.com/ros/rosdistro/commits/master


We can't pass Django variables inside a `{% compress js %}` block - this will break offline compression (since the `./manage.py compress` task has no way of knowing what these variables are).


Some degree of familiarity with any testing framework is to be expected, always.

Anyway, all of your points apply to both suggested versions of that line (or rather hamcrest as a whole) and as such help little in deciding for either one of them.

According to the hamcrest docs, there's another possibility:

not(equalTo(x))

Let's discuss :-)

Anyway, I don't have strong feelings about this. All I meant was that I slightly prefer avoiding the is(boolean) part because it mixes the "actual" and "matcher" arguments.


@jasononeil the deploy didn't work... any idea? https://travis-ci.org/HaxeFoundation/haxe.org/builds/94416072#L1146


Ok, changing it.


LGTM.


Fuck my bad, don't even know how that happened.


Fixes #1


fix this


Uploaded!


As I told you in last assignment, don't let a blank line after `{` and before `}`.


There's some inefficiency here. Suppose I have 10 people assigned to Need ID 44. For each one, this code does a get on Need ID 44. Let's cache the results of our api.VolunteerNeed.get calls in a class member and only fetch if we need to.