а) а ежли файла нету, который указали?
б) хорошо бы хоть минимальные тесты на «успешно загрузилось» (сколько там всего фильмов оказалось, среди фильмов есть фильм под названием x — хотя бы; в реальности я бы конечно загружал сам json и сравнивал, скажем, что все фильмы из json в MovieList корректно загрузились)


and for non-systemd ?


does not :warning:


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.569.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68132&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:19:41


How do you revert to the previous installation?


Ok, I'll try


Add an action to trigger a non-enabled plugin.


We should still unassign? Theory casing, but that's what I think, you are allowed to unassign so remove it and then display the error.


This commit looks good to me.


Looking great! Something we'd want to do down the line is handle errors better than just printing them. On android, I just display a common generic "Couldn't connect - please try again later!" popup whenever there's an error with a network request. (I'm considering making it an even less-obtrusive toast instead - a transient message that appears for a few seconds in a transparent bubble and disappears.
What's an iOS -idiomatic way to display connection failures or user-friendly notifications that the connection has been interrupted?


I've updated it using the latest docs.
Yes, cell classes can be added using the new version of DT which I can see in `Common.js` file. Issue arises for this case only because here we have different cell classes for same column unlike in other dataTables implemented elsewhere.

When I see the ManageGroup table, we're aware that the first column would be `title`, next will be `count` and we proceed by giving the `cell-class` attribute to column header which automatically assigns them to the `<td>` under this header.

As in this case the custom values have the classes as `crmf-custom_{$fieldId}_{$recordId}` which varies for each value in `<td>`.

What I do here is assign `className` along with each `fieldname` from the callback function and modify the cell accordingly. Eg:

The returned data from the server is in the form of

    array(
      'fieldName' => array(
         'data' => 'firstField',
         'cellClass' => 'firstFieldClass',
       ),
    );
The reason I loop `th` here is because key `fieldName` is same as attribute `'data-data'` I've assigned to each `<th>` element. So now, it becomes easy to navigate to the required class and assign it to the cell.

Also, assigning `'data-data'` to each header is also necessary for sorting functionality.

AFAIK, the limit of "cell classes can be applied automatically" is that same class will be applied to each `<td>` element under the header, which I try to fix here. Is there something I'm missing?


s/an/a


Indentação aqui.


Fixed in the latest commit.


Please open an issue if you want to start a discussion. Commit comments are ignored.


I dont know the results at the moment, so it is absolut necessary to see the possible of learning institutions.


assign?, not unassign


@michaelstingl @dragotin backposted from v2.1.1 here too...
please review.


Yes the general view on internals has changed.  TE uses them in places, but it is now generally discouraged.  In the next version of TE (called TrackableEntities.Express), we will avoid them altogether.


I was having the same issue and the above code worked for me.
My suggestion to keep the original functionality is to remove the "and gflags="R" limit 1" and also replace the "geo.placefinder" by "geo.places(1)"


Why do you use ReentrantLock here and not synchronized(readOver) {... } block?


Nice catch, fixed.


Shouldn't this be > 1 then? There's nothing to re-order until you have multiple rows.


I have added two separate functions now for validating the toleration operator and toleration value. This is checked in the latest commit.


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


Done.


Done. Added new test case for the toleration operator in the latest commit.


Could you add something here that reminds that container-based is the same `sudo: false` infrastructure? I find that sometimes people does know about `sudo: false` but not about container-based


This patch comes from issue (Baris Altuncay)  Thanks for this!.

Could not merge it automatically, so merge data is lost.


aww not again :(


The code does write to the regular log file, but it writes a much cleaner output to the secondary file.
My reason is that if I'm debugging a contract, I don't want to necessarily have to wade through the entire output_log.txt file to find the same stuff that is visible in the debug window.  This gives the option for a secondary view, in a file which hopefully won't grow too much


Please, init streetIndex just after result declaration, as I mentioned:
```c++
pair<...> result;
result.second = numeric_limits<uint32_t>::max();
...
```


Closes #127


ok, I'll delete it.  I always document my changes when I modify someone else's code.  Should I leave the plus signs or remove them as well?


> If in some cases we rewrite "method" variable and set method as "PUT" do we need any logs what happend? @polesye

it doesn't work with https://github.com/Livit/Livit.Learn.EdX/commit/888aa9ace43f8a4fd6fa89c82810f9d1ed233658#diff-c588606ec94af64ea1e13b58fa67cfdcL23


I need to know if I've reached EOF in order to quit. If EOF is reached in the middle of the stream, fgets won't tell me (without checking feof). So I just used feof.

On January 20, 2016 2:51:16 AM EST, Mike Burns <notifications@github.com> wrote:
>How about checking the return from fgets(3) instead of checking feof(3)
>manually? Like:
>
>```c
>if (fgets(buffer, sizeof(buffer), stdin) == NULL)
>  return;
>```
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/mhoran/tbd/commit/90d82fb4bd8df0fbbb89d8ce7506c0ab716fccfe#commitcomment-15550702

--
Sent from my mobile device.


@pbastowski Fixed. But I'm not sure if it won't cause any troubles.


"Theory casing" ? Can you explain ?

From my point of view we simulate as if the tag did not exist.
It's the same behavior as if an admin tries to unassign a really non-existing tag. (`tagMapper->unassignTags()` asserts tag existence)


agree. changed to a generic message.


:scissors:


See my comment above.  This writes to the standard log file, my previous write was to a seperate file which only contained the specific info related to the contract as seen internally


> Why we use dict method if DEFAULT_HEADERS type is already dict?

to avoid mutation of the original dict.


will do


You can also reorder grid editors. I just wanted to make it simple and just remove the button from the first two "initial" setup steps: set layout + set first row.


see line 35


Nit: you could adjust the commit message before merging this.


Ok senpai :full_moon_with_face:


the real fix is to remove the prefix. ref https://github.com/Pulse-Eight/platform/blob/master/CMakeLists.txt#L25


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


Probably should’ve pinged you… but didn’t you get a notification from #108?


thanks for adding this i can now test it for dual sim support


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


@den99f Why do we need to dump it twice?


yes single sim works!!!


@den99f is it a bug?


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


`csv` -> content


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


`content_type=MIME_TYPES.get(accept_format)`


ok and yes this is a todo.


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


have you added rabbit stew yet?


Ahhh, I see.

I guess there's a bug lurking in `SecureString`. I'm going to fix it.


I think I read a comment at some pvr pr that we want to keep the prefix. maybe that was @opdenkamp


@unknownbrackets @maximu I see


> content_type=MIME_TYPES.get(accept_format)

Fixed


Please remove empty functions


> csv -> content

Fixed.


Done now. Password reset uses now ReCaptcha:
https://github.com/der-dave/APF-code/commit/6a1d111b273efc9b8e47e828e3ba0b07323f9a2a


Removing the lock breaks the the whole "kill the Pharaoh to get your reward" aspect.


Ok, now I see: you wait for end of read and write in close.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.161.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68129&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:35:35


Why have two functions if they do the same thing?


@soyuka Btw, what about moving `printError` into `Common` or `Utility`? As for me, it seems to be useful for error logging, when both message and error should be printed.


then @opdenkamp needs to make up his mind ;) the build system sets the prefixed include paths and just installprefix/include. it's "pure luck" that installprefix/include is in your include paths.


I've implemented URL encoding & Decoding..
Using the built in NodeJS methods encodeURIComponent and decodeURIComponent


I am looking forward to see other solution :smile:


Hi Conejo,
I am curious about Composer, phpseclib and /vendor dir, is it used for internal testing?


Killing it.


Those are actions. Each action delivers the '@conf' instance variable to two different views.


thank you very much


@ayebare this works, however I prefer to instead structure my functions to 'early exit' or 'fail early, fail often': http://aaronjholbrook.com/fail-early-fail-often/

So this would look like after this line:

```
if ( false === $show_spop_notice ) {
  return
}
```


What is the "error rate"? We dont use that metric


Added a test case for bad key


added a test case for bad toleration operator


removed the nodestatus


Yeah, I was right. The copy constructor `SecureString(const QString &other)` contained a bug: it didn't copy at all ;-)

Fixed it locally. Will merge it into master soon.


added a test for bad taint key


patch is ok, please merge


@ayebare this helps to to reduce number of required indentations and helps to reduce mental overhead, as if this condition had not been met we've already left the function


modified the test for bad taint value instead of missing taint value


Ah.  You're right, I was testing on verbose mode.
so I'll modify the code to to the following:
1.  Only do this if the loglevel is either VERBOSE or DEBUG
2.  Make sure to strip out both the VERBOSE and DEBUG lines in the pattern2


Bad whitespace.


vendor should not have been included, sorry


Почему ты выбрал именно такое значение?
Может стоит использовать что-нибудь более простое, если значение тебе не важно, например `1ULL << 30`?


Though I'm not a member of the development project, it looks good for me.


we have it included everywhere: https://github.com/kodi-pvr/pvr.hts/pull/154/files


Why?


When making a release on github add the stuff from the pre releases into the description too.


Also, I am not sure this conclusion about "three quarters of ambiguous author names" are resolved is correct. The metrics we use relate to signatures and how they are grouped signatures. I am not sure you can draw this kind of conclusions.


if no operator is passed, default is equal. So we do no validation if any of equals, exists or "" values are passed for the operator. For all other non-empty values, we raise an error saying that the operator isn't supported.


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


thought so, ok thanks :)


then it definitely needs to set the correct non-prefixed include paths in its .pc / cmake config file.


Used validation.isValidLabelValue()  for validating the toleration value


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


good work - funny I would have used  greater than - this makes more sense ...


if the toleration operator is "Exists" , the value needs to be empty.


Used ValidateLabelName() to validate the toleration key.


NOTE: Last commit is for review only. DO NOT MERGE!
This commit addresses most of Henrik's review comments but the following has not been addressed yet:

1. Refactoring into several commits
2. Changes to documentation of the `each` method (can now be called without block)
3. More tests concerning Iterable#instance? and Iterable#assignable?

The reason for deferring that work is that we need to finish the ongoing design decision in [PUP-5648](https://tickets.puppetlabs.com/browse/PUP-5648) first.


added validation for toleration operator.


I don't really understand the code, but it looks neat.


@RandomAltThingy Got it. ;)


@soyuka Are you ready to drop 0.10 support because of `fs.access` ? :smile:


Coding style 不太對。大括號沒有對齊。請修改正確


Если это `ConfigGuard`, т.е. относится именно к `Config`, имеет смысл использовать здесь template?


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1007.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=68135&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:41


Is it possible to add some characters like + in regex here, example:
([a-zA-Z0-9_\-~\.]+) => ([a-zA-Z0-9_\-~\.\+]+)
In aeroport de lyon, we have + sign in DIMENSIONS parameter values


Thanks for the pull-request, Ryan! That is exactly what I intended to write initially, but didn't know how to do that due to lack of understanding how to work with Windows overlapped structures.


I would document here that this is the driver for the ARM VFPv3-D16 architecture.


you're welcome!


> to avoid mutation of the original dict.

How can dict mutate? Maybe we should use deep.copy than?


fixed on staging


then we need to fix up the .pc/.cmake things indeed, because in general I think it's best to be as specific as possible in code, in case that something else has a header with the same name. not that that will happen with this one ;-)


@nyalldawson test part sorted out ;)


I think `synchronized(readOver) { ... }` and `synchronized(writeOver) { ... }` would work.  `close()` would then use `synchronized` too, perhaps with empty blocks?  According to [this post](http://stackoverflow.com/questions/18056111/are-empty-synchronized-blocks-optimized-out-by-java-compiler), that should work:
```
synchronized(readOver) {}
synchronized(writeOver) {}
```


having ": /conferences/{accesscode}/participants/me" is not difficult if token mechanism is in place. I will send separate notes on One Note on how I am foreseeing tokens implementation for AWV.
Lets park : /conference/{accesscode/participants/{username_or_email} for future


No problem!


You code fast haha


Ought to solve #7.


If use 4.12 instead can use annotations as @Test


@andrey-dresvyannikov You're right `insert` is indeed deprecated, but it is still available in pymongo 3.2 (see https://api.mongodb.org/python/current/api/pymongo/collection.html#pymongo.collection.Collection.insert) so it's a short term solution

On the other hand, I've seen some others `DeprecationWarning` (about `tz_aware`, `host` instead of `address` attribute etc.) so I think I would worth it to start a branch to deal globally with those issues (by suing only pymongo 3 functions and add a compitibility layer like you did for pymongo 2.x) what do you think ?


@RoySegall Comment altered as per review.


it should be splitted


see line 35


Он относится к конкретным значениям.


@Adusei I tried to re-produce this issue, but I cannot find it. So could you please provide some screenshots or videos? It will be very helpful for me to re-produce this issue. Thanks.


Seems that you misspelled "canola"


You can *absolutely* ensure that VAL_TYPE will be <=0xff *forever* ?
If not, I think better this && stay here...


Merged.


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


this commit is ok


This is a **very** bad idea.  The whole point of libp11 is to **hide** the ugliness of the PKCS#11 interface.  Adding `PKCS11_ecdh_derive()` to exports, with `CK_*` parameters, violates the very basic principles of libp11.  `const void *` is not much better...


IPFInitializer is now exposed as an OSGi service, but ITs still fails on "missing requirement" for slf4j so I left integration test with @Ignored. MOTECH-2100 is created for this issue.


I meant Undo


Thanks


maybe remove the email adresses from authors list, can be a sensitive information and used for spam


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1203](http://teamcity.monogame.net/viewLog.html?buildId=13807&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:26:50


this breaks the projections plugin

can we do this only if opengl is off instead?


Bom dia @sotnaspina , da uma olhada nesse exemplo https://github.com/owen-it/spark-auditing, aqui eu criei uma aplicação para demonstrar o uso do laravel-auditing.

De qualquer forma,  para pegar os valores do relacionamentos tente o seguinte:

* No seu modelo
```php

    // model User.php

    /**
     * Log custom fields message
     */
    public static $logCustomFields = [
        'name'  => 'The name was defined as {new.name}',
        'group_id' => [
            'updated' => 'The group name was defined as  {||groupName} ',
            'created' => 'The group name was defined as  {||groupName}'
        ],
    ];
    /**
     * @param $log
     * @return string
     */
    public function groupName($log)
    {
        return Group::find($log->group_id)->name;
    }
    //....
```

Bom, se entendi bem o que você precisa, sua resposta poderia ser algo como isso. No entanto se mesmo assim você não tiver compreendido, me adiciona no gmail(anteriovieira@gmail.com) e te explico melhor.

Referências:
* https://github.com/owen-it/laravel-auditing#customizing-log-message
* https://github.com/owen-it/spark-auditing/blob/master/app/Team.php#L22-L28


Я о том, что у тебя есть конкретный тип `Config`, кажется, что `ConfigGuard` будет принимать только `Config &`, в этом случае не очень понятно зачем здесь использовать template.


nueva version


What's the reason for releasing `5.0` then? :)


Thx to @ZombiePower
Just added rabbit stew :)


Hey Pete,

I added these additional translations this afternoon. I had to make a
change proposal on Github in res/localization/rufus.loc because when I
copy/pasted the whole file to e-mail it to you, most of the line breaks
were gone. I hope this doesn't pose too much of a problem.

Best regards,
Marvelade

On Thu, Jan 14, 2016 at 11:38 AM, Pete Batard <notifications@github.com>
wrote:

> In that case, can you please have a look at
> https://github.com/pbatard/rufus/blob/master/res/localization/ChangeLog.txt#L18-L27
> ?
>
> These are the changes still needed to bring the Dutch translation up to
> the latest, as it is currently at version 1.0.18 and new messages have been
> introduced for 1.0.19. Please see this
> <https://github.com/pbatard/rufus/wiki/Localization#Editing_an_existing_translation>
> if you need more info about editing an existing translation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marvelade/rufus/commit/947bd451ca71a76470635aa292b5750dacf920f9#commitcomment-15446555>
> .
>


segunda version


I think it's better to catch explicit exceptions wherever this is possible.
Also; failure will not result in any message towards the user, who will assume that everything went well.


Hi,

I compiled  php 7 and tested openemr, the first error I see is in ADODB library

Adodb library uses php4 styled constructors so first we should update adodb library


What could be worthwhile would be to put an assert, like `assert(REB_MAX <= 64);` (we should set up a static assert macro).  That would encode the dependency.  But doing the mask to chop off bits of an unknown thing is just an uncertainty on top of an uncertainty... it's the wrong way to be "safe".

Here is a "static assert" we might use (costs only at compile time, not runtime).  I may have put something like it in the code already, should look:

http://stackoverflow.com/a/3385694/211160


Oh, I just see I messed up the naming 8)

```freq``` is here the center frequency of a channel and ```center_freq``` is just the sub carrier frequency.

I'll fix that later.


You should set 'supported' to true no matter what in 0.10.0, as isSupported was replaced with getSupported and the "shader" graphics feature was removed from LÖVE since 0.10.0 always supports shaders.


So it's lossy after all?


@thedrow Any idea about the next release schedule ?

btw I've seen the pypi deploy is now part of the travis config, does this mean any contributor with commit rights (like... me ^^) can/should now release version ? If so is there any rules and policy to follow ?


Fine for me! +1
Il 20/gen/2016 13:19, "Brian Dickens" <notifications@github.com> ha scritto:

> What could be worthwhile would be to put an assert, like assert(REB_MAX
> <= 64); (we should set up a static assert macro). That would encode the
> dependency. But doing the mask to chop off bits of an unknown thing is just
> an uncertainty on top of an uncertainty... it's the wrong way to be "safe".
>
> Here is a "static assert" we might use (costs only at compile time, not
> runtime). I may have put something like it in the code already, should look:
>
> http://stackoverflow.com/a/3385694/211160
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hostilefork/rebol/commit/509bc5e6ff7a5370bc52a8ed3965cde176289c1d#commitcomment-15555889>
> .
>


@topolik do we need the Integer array here? it seems like we are always creating with a single index in the initCustomMethod method: https://github.com/topolik/liferay-portal/commit/dabb72a5c1b881ff6f979a3d29addd6cd9adfc6e#diff-1139a9440ef159b426f06a2bd4d167c4R57


sas


@RandomAltThingy I code fast just because that was a stupid mistake. xD


BITTE keine natives committen! (bin/ komplett ignorieren)


ok, we will use macro instead of magic number, and also add comment for it.


If I understand that right it would need
writing the TIFF header in prepareWriteSequence
writing IDF pointer, then imagedata and at last IFD in writeToSequence
and writing EOF in endWriteSequence

Am I right that this still would work for uncompressed images only if we don't buffer the output of the compressor stream? At least I don't see a way to precalculate the length of the compression streams.


To close GH-69


We will update the number of SiP calls.


This is not needed. We embed gnatsd.


A long method which has a lot of block could be easier to understand if there would be some parts extracted to helper methods.


Other changes :
 - waypoint icon changes depending on its type
 - code correctly indented, deleted empty spaces (hopefully)
 - got rid of the buttons at the end of the list
 - changed the position of the pagination/add new (waypoint/route) buttons


Probably 1.5.3


Note that nsec could be supported in DateTime with https://github.com/JodaOrg/joda-time/issues/52#issuecomment-166028419.
It would require updating jruby if we want to keep the compatibility layer and avoid two libraries,
but computations with millis and external nanos are very error-prone at best.


Is the issue https://phabricator.babeljs.io/T7010?


We will use macro instead of magic number, and add comment for it as well.


Fixes #144


@hamitturkukaya api.bulutfon.com/docs da düzenlense iyi oliur.


Do you use this class?


saw approval from andytill
at https://github.com/basho/riak_kv/commit/add9a4c4727f7edf1c40bf0004f7e920eb175437


merging basho/riak_kv/fix/az/check-tables-activated = add9a4c4 into borshop-integration-1324-fix/az/check-tables-activated


basho/riak_kv/fix/az/check-tables-activated = add9a4c4 merged ok, testing candidate = b45e739d


this


We need to remove the `Welcome to Hello Mama!`


похоже, здесь code уже не нужен


What if you use `app.use('*', ngApp);` in that way all routes are catched by express and served with the ngApp that finds out the specific route.


@itziakos and @kitchoi ,  are the changes I did here okay?

I made these changes as I was getting errors on appveyor (https://ci.appveyor.com/project/itziakos/simphony-common/build/1.0.313).  From what I understand, appveyor has 2.7.9 installed so we can skip the boostrapping.  is that right?


We will use macro instead of magic number, and will squash this into [927a9b2](https://github.com/mtk09422/arm-trusted-firmware/commit/927a9b267fe566dda93a031ca9efdf12c82ab139)


Est-ce qu'on stocke le numéro de la saison dans le modèle, ou est-ce qu'on le stocke dans title sous la forme "Saison x - blabla" ?


You could use your own namespace.
Todo?


Just for clarity, add `// state_last_period_day`


@atfornes is this right? swellrtServerDomain might be different from the one used in waves. For example, in production we have `swellrt.teem.works` as server domain and `prototype.p2pvalue.eu` as wave domain


Will it still 404 if the route does not exisit?


Just for clarity, add `// state_baby_birth_day`


We will update the function name.


Thank you for you working!


Je propose d'utiliser un modèle intermédiaire "Subscription" pour ça pour limiter le couplage entre la feature d'inscription et le core d'ALDP.


a


We'll keep the current implementation.


Warning: must change IS_NONE to  IS_UNSET also [in Loop_Each](https://github.com/metaeducation/ren-c/blob/master/src/core/n-loop.c#L531) and [in Mold_Map](https://github.com/metaeducation/ren-c/blob/master/src/core/s-mold.c#L882)


This actually will take c.15 msec on .NET. Could this add latency where it is critical? Usually for latency critical stuff it is recommended to disable Nagle's algorithm on TCP, but here there is additional explicit buffer over TCP socket in CSNATS. Are there some settings to prefer latency over throughput?

NOP example from [here](http://stackoverflow.com/a/22223245/801189) or a spinlock could allow sleeping for less than 15 msecs


The way its currently working, the very worst case you could theoretically lose a maximum of 0.9 milli buckets, but thats only the case when you're using some very exotic floating point values in the config and under very strict conditions.
In practice, it shouldn't really happen.
There's always gonna be a trade-off between distributing the fluid as exactly as possible between the different outputs and not having any loss, I just chose to focus on distributing.


With @joaoferreira 's suggestion I now get

     Uncaught TypeError: Cannot read property 'channel' of null


Submit upstream changing the asserts to use [warnings](https://docs.python.org/3/library/warnings.html)?


Je suppose qu'on utilise ce champ pour clore une saison ?
Une saison est donc visible dès qu'elle est créée ?


What about base.sass?


Are you sure this works? I added the fonts to XCode but they have names like ```SourceSansPro-Semibold```, so either they're already supported by iOS (then we can remove them form our bundle) or you have to use the correct names.


Round ?


agree and yes we have removed this(boolean flag or point 2 above) from JoinRequest model and now for each newly created participant resource system will return the weburl by default as part of resource definition. if user is not a web user then the weburl will be null.
Now there is no WebURL endpoint in APIs.


(Just clicked a random thing)

Catching explicit exceptions is a bit hard here, due to the eval, which can run any kind of code.
I do find it odd that the whole profile setSetting value is also in this try-catch.

(And YAY for dict_of_doom reference :) )


We can probably remove get_node_disks function, this seems to be the only place it is used.


Îl faut un field author aussi, pour lier le modérateur a la manche (pour les notifications)


Roshan - pls check this and confirm


fixed #6


Set up repository for the developers academy.


Looks legit; only thing I have is that Line 270 should be an elif/else, and the else case right after can't realistically ever occur.


Use junit 4.12 will prevent you from extending this class


Je pense que tu peux utiliser deux foreign key mais vers message, pas thread.
La première foreign key est un lien vers le message d'ouverture, et la seconde vers le message de fermeture.


Ici tu peux utiliser un lien vers Message plutôt que Thread, pour les mêmes raisons.


Removal of these lines restricts us to update the testResult from within afterInvocation method or any of the TestListener methods.

I use afterInvocation to update the testResult, as i validate the softAssertion inside afterInvocation.

@Override
    public void afterInvocation(IInvokedMethod method, ITestResult testResult, ITestContext context) {
        //Reporter.setCurrentTestResult(testResult);
        if (method.isTestMethod() && null == testResult.getThrowable()) {
            SoftAssert sAssert =
                    (SoftAssert) context.getAttribute(testResult.getInstance().toString()
                            + ":sAssert");
            try {
                if (null != sAssert) {
                    sAssert.assertAll();
                }
            } catch (AssertionError ae) {
                testResult.setStatus(ITestResult.FAILURE);
                testResult.setThrowable(ae);
            }
        }
    }

The above code works fine only up to version 6.9.6


On peut ajouter un champ "author" aussi.


Wenn du schon code abschriebst, gib doch wenigstens die Quelle an:
https://gist.github.com/tiraeth/1306602


On peut ajouter un champ auteur aussi
Je ne suis pas sûr que "description" soit utile, et title peut devenir "text" vu qu'on est dans "Question".


Good catch. I expected there could be more {groupId}s in custom method signatures, but reality showed otherwise :) Feel free to to change it to Integer or change initCustomMethod signature to allow Integer[]


@property
def season(self): return self.ining.season


I would rename this function 'fault' to be more conform with the ARM FPU class.


so this is a delegate case and and will be needed only in specific cases. I dont think we need to default it to me.


Fair.


Perfect!


add
```
.idea/
*.iml
```
if you are going to work with IntelliJ


So basically we need an action like ```getPersonByID```, so maybe we have to differentiate between ```peopleNearMe``` and just ```people```, because if someone opens an old chat the person shouldn't be in ```peopleNearMe```. Even better if we have a store for random people and ```peopleNearMe``` just returns their ids.
On important thing is that we should add fetched people to the store and not just overwrite everything.
Another thing we could do is just having ```peopleNearMe``` and ```peopleWithOpenChats``` (I don't think there's a way for users to get access to anyone else).
Let me know what you think.
We should also stop using emails and use some UUIDs (that requires some server-side changes though).


minor: would prefer defining a DEFAULT_TRACE_LOG_INCLUDE also (makes it imho much clearer to understand specially if some reads this the first time)


So in the worst case it's possible to loose 90% of your fluid, that doesn't sound like fun :/
Don't forget, users are stupid, so they will definitely use such exotic values, and they will complain about loss.
I think even I created a config once with a float value (the oil one if I'm correct), so if it's possible, it should work correctly IMO.

You can still have optimal distribution AND make it work lossless. I don't have an exact algorithm in mind, but it will probably involve looping through all sides twice and remembering a min/max value for each.


saw approval from andytill
at https://github.com/basho/riak_kv/commit/add9a4c4727f7edf1c40bf0004f7e920eb175437


merging basho/riak_kv/fix/az/check-tables-activated = add9a4c4 into borshop-integration-1324-fix/az/check-tables-activated


basho/riak_kv/fix/az/check-tables-activated = add9a4c4 merged ok, testing candidate = fb04c5ae


Also this syntax error, unexpected 'new' (T_NEW)

"New objects cannot be assigned by reference"

e.g. controllers/C_Prescription.class.php on line 591


Roshan- Pls check this and confirm.


What is this comparing?


Using both versions of the same dependency might bring you problems. Just using 4.12 should be fine.


You are right, we might need change this, thanks for noticing

2016-01-20 13:28 GMT+01:00 Antonio Tapiador <notifications@github.com>:

> @atfornes <https://github.com/atfornes> is this right?
> swellrtServerDomain might be different from the one used in waves. For
> example, in production we have swellrt.teem.works as server domain and
> prototype.p2pvalue.eu as wave domain
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/P2Pvalue/teem/commit/540b86daeaa173bd215b3f7f09e9d5114d3c3b11#commitcomment-15556052>
> .
>


Why did you decide to make this a singleton.
It makes AsyncHiredisCommand unusable in several threads.

Also the logic of this class is not proper because there's no guaranties not to appear the same address of "Connection *" twice.

Despite all of the above I didn't exactly understand when the probplem occurs. Function "Disconnect()" of AsyncHiredisCommand was my mistake itself and must be deleted, it must be never used. There's no need to make redisAsyncDisconnect in disconnected callback at all, hiredis knows about this and would free connection without any additional invocations, so it seems like there is no need in class DisconnectedConnections.


scheduler and scheduler2 are not descriptive enough, call it conf_scheduler and ping_scheduler


I agree. But note that yet we have a problem with string, if we change one that is a key... :-/


"translate" should be removed here


Was spricht hier gegen eine LinkedList? Ist bei der Datenmenge signifikant effizienter (Peakdetecror nutzt ja einen Iterator).


Roshan - Pls check this and confirm.


don't you wan to set GAP_BUILD_VERSION to the git commit hash if it's undefined?


double quotes here


Hmm, raar. Je zou verwachten dat, als je op het knopje drukt, dat ie dan focus overdraagt (dat doet hij ook in mijn Chromium, Firefox, Firefox Nightly)


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


Extra space after ``<a`` :P


the first parameter for the assertFalse is meant to have the message for a failure, in this case should
```java
Assert.assertFalse("Entry n° 2 NOT deleted", flag);
```


Is really allowed to embed elements (eg. links) into buttons?


sdfsdf


I will add a constant. thanks.


Appveyor has 2.7.10 which comes with pip 7.1.2 so we should be ok. Ideally it is better to always have the latest version for pip, but I believe the problem is that the pip 8.0.0 wheel is broken in pypi.


Sinnloser null-check.


s.o.


Do you really need this link inside a link?
Do you really need Javascript to make the card clickable?


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


Eine Bemerkung zu den Testfällen: Bitte lagere die in eine eigene Klasse aus. Zudem solltest du zum Testen ein automatisches Tool verwenden. [JUnit](https://en.wikipedia.org/wiki/JUnit) bietet sich an. Ein Beispiel findest du [hier](http://www.vogella.com/tutorials/JUnit/article.html#unittesting_junitexample)


What's the connection between the elevation and and the summit icon?


I'm sorry, but I don't understand what do you mean.
Should I remove comments like this one?


Replace the lorem ipsum by the "summary" attribute.


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


You mean what era means? https://en.wikipedia.org/wiki/Calendar_era


"error rate" is used in some publications I read, Here is a recent example from a paper from Google (Quoc Le and Tomas Mikolov, 2014) and here I quote: "...For example, on sentiment analysis task, we achieve new state-of-the-art results, better than complex methods, yielding a relative improvement of more than 16% in terms of error rate.". I agree with you that we should use " relative improvement". but we can still tail it with "in term of error rate"


If you change a line like this
G_DEFINE_TYPE (MateColorSelection, mate_color_selection, GTK_TYPE_VBOX)
you neede to add a ORIENTATION in ....init function.
See https://github.com/mate-desktop/caja/commit/cf39567d3b05b69383c252812c5adf26382c866a#diff-99a15eb27b57fd6d0af1914e24425ad2R56
I found this in every example which i found.


Indeed we have a problem here. I will fix it.


It compares two `fromCal`, shouldn't the 2nd be `toCal` ?


I tought I checked that but I guess I got mixed up with https://github.com/odoo-dev/enterprise/commit/d39bbc9.

I will see to it, thanks


Vidis li problem ovdje u localhost:8080?

Tvoja virtualka ima javni IP i mi cemo se spojiti s nasih mobitela i racunala na tvoju virtualku i u RSSu cemo dobiti krivo serviranu informaciju.


jytfytrfytr tftyfyf


the first project of maven I did in console maybe because that I didn't add
the .idea, in the other project I did
El 20/01/2016 09:37, "Javier Alejandro Miño" <notifications@github.com>
escribió:

> add
>
> .idea/
> *.iml
>
> if you are going to work with IntelliJ
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mauricba/java-bootcamp-2016-Initiation/commit/0c667756c5cde6e0c1464d67e86f641bc934f34f#commitcomment-15556205>
> .
>


No we cannot. Because B3 or paired F-measures are not "error rates".


Uocavas li repetitivan dio koda?


Small question: why is the website template inside the website_mail module ?


@itziakos , thanks for the quick response.  Is it okay to merge as is or would it be better to add a comment (i.e.  something like "Not updating pip as having problems due to possibly broken pip 8.0.0 wheel")


missing @return tag


if you delete an element that is not at the end of the vector, all other elements that were added after it will have its position changed and handle ids wont be valid anymore.


Hmm, did I forget it this time...?


yes I saw, I had some problems with this yesterday, in the Skype of today I
will mention it
El 20/01/2016 09:41, "Javier Alejandro Miño" <notifications@github.com>
escribió:

> the first parameter for the assertFalse is meant to have the message for a
> failure, in this case should
>
> Assert.assertFalse("Entry n° 2 NOT deleted", flag);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mauricba/java-bootcamp-2016-Initiation/commit/b8a3e5f27de3f3ab571e680bae5d301112aa3a1c#commitcomment-15556306>
> .
>


Für updateHashInteger sollte es einen Test geben.


Ok.


What about separate fields for First Name and Last Name? - This info can then be used for the printing name tags.


Dont forget about small atomic commits :)


dosta hardkodiranih stvari. razmisli o izdvajanju u konstante.


Što se tiče repetativnog koda, to ću još izmijeniti. Išao sam prvo samo
napraviti sve da vidim jel valja i jesam dobro shvatio što je rss feed.
Napraviti ću posebnu funkciju koja generira osnovne elemente rss-a.

Nije mi skroz jasno što bih onda trebao staviti umjesto localhost:8080 jer
aplikacija nije na webu i ne postoji drugi link za doći do tweeta ili slike.

On Wed, Jan 20, 2016 at 1:47 PM, Karlo Knezevic <notifications@github.com>
wrote:

> Uocavas li repetitivan dio koda?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chiiiro/SSP-TwitterApp/commit/e7e8289076ad743f5ddd1da600d39fb6e3c55a3b#commitcomment-15556419>
> .
>


lgtm


It's necessary to use data-behavior here or we can leave as is?


Even if it is just a comment, we might want to give it a more appropriate name. I have done some thinking and it is not too easy to find a suitable name. Maybe Illegal Operations? Or if no suitable one can be found, Various?

Various isn't a very good name either, but imo is still better than derp.


you're right. Maybe I should return a pointer.


@kem008

Make sure that you are enrolled in the beta of Source SDK Base 2013 Singleplayer.

You should also verify the integrity of your game's files. This might be the cause of your crashes, as you have invalid/corrupt game files.

You can perform this check by doing the following:

- Right-click on Source SDK Base 2013 Singleplayer.
- Go to the "Local Files" tab.
- Click "Verify Integrity of Tool cache..."
- Wait until game cache inspection is completed.

If you have other questions, please let me know.


So, what about reflecting it to the number of articles instead of signature? Although I think using the number of signatures is still fine regardless of the number of clusters they can form. They somehow reflect the amount of the problem to be solve. I agree that saying "ambiguous author names" is not accurate. So, can we say "...three quarters of signatures" instead?


This is not very efficient!


Do we still need this hint now that we have Swift examples as well?


I fear that using splitInto8 in the benchmarks is not a good idea because it's slow


Ups!


Instead of trying to scan the human readable downloads page (which may undergo a very large redesign soon), I'd recommend using the appsyn feed instead at: http://wixtoolset.org/releases/feeds/wix4-0.feed. The first `as:version` element will be the latest v4.0 build.


Would move the FPU implementations to ``spec/*/fpu.cc``


"EOODF"?


:+1:

Založil jsem issue https://github.com/osmcz/osmcz/issues/66


@ztolley
reason is:
http://stackoverflow.com/questions/9867108/issue-in-double-value-comparison-with-php


Ah, the "users are stupid" argument.
Actually, the way I calculate it its not possible to lose 0.9 mb if you only provide it 1 mb - in that case it would actually output 1mb. Like I said: only in very specific edge cases you might have .x loss.
For now I'm happy with how it works, but I was planning on rewriting it anyway cause it might be doing a bit too much operations per tick.
Once I'm done implementing model rendering and textures, I'll probably revise it so don't worry about it ;)


@ceci


Don't we have a function to parse URIs? if we do, can't we use it, instead of parsing the string here? This would avoid issues with weird URIs.


Setting cache headers to public is a security risk... is this deliberate?

https://secure.php.net/manual/en/session.security.php


Spremi pivu. Nas'o sam bug.

Kad dodas prozivod u korpu, ne ispise kolicinu, ispise samo 'xx'. Isto se desi i u prihvatanju narudzbe.

Pusa <3


Would move all FPU implementations to ``spec/*/fpu.cc``


That's fine. I have now applied your changes (pbatard/rufus@3b8b7fb7beb4bcfdd1f22e9872b5a9358a63522a)

I'm a bit suprised you weren't able to send the `rufus.loc` as an attachment (instead of trying to copy/paste), but I don't have much of an issue picking changes in your repo.


> Switching off Gradle junit4 support, which can be done with the Gradle plugin of JUnit 5.

Sure.

I just focused on the simplest change possible: deleting lines was rather easy.

But I am of course fine if JUnit 5 uses JUnit 5. :wink:


Use nullptr instead of Null


I don't know, I would like to keep it readable.
Does it really worth it?


This looks like business logic which should not be part of the controller.

Please, @marcheing  and @danielkza, provide details on https://github.com/mezuro/prezento/issues/305 about what this is preventing so we can fix this in the future.


Dont use "translate" this way here since it makes ``${waypoint['waypoint_type']}`` appear in the PO files. A good way is to write:
```
<b>{{mainCtrl.translate('${waypoint['waypoint_type']}')}}</b>
```
And don't use ``<b>`` tags => styling must be done in the CSS, not with HTML tags.


Yes, but also be sure you're actually reading memory from the client. Is this trainer in .dll form? Otherwise, if it's an executable you're going to have to write to the process using the win api.


du ska inte testa spritesheet då


kako to da su modeli nezavisni otoci? sto nijedan model nema nista zajednicko?


damn, I should check for >= -FLT_MAX


So I need to use Beta version and try to verify cache also ok I will try it thanks :D


Very good that you added the benchmark for serialLargeHash. I think the reason why serialLargeHash is so slow is the MonadPut instance for LH. I added a comment there. It's probably the same reason as you pointed out with "I think that performance could be improved considerably if we add specific hu_* actions for CUChar, CUShort, CUShort and CUInt."


Sorry to be picky, but no, I am not sure you can infer a proportion of "correctly predicted signatures" given the metric we use.


Through the rethrow method, all exceptions (except Errors) are rethrown as RunTimeExceptions. If we already have rethrown an exception, do we ever reach this line? Isn't the method call terminated with throwing the exception? If not, what reason would you have to throw this (second) RunTimeException?


Store the return value and use passert with it. Assert/passert can be defined in a way that the function gets not called.


@HolgerKache  Thanks, update for the Readme file are good.


It would be great to have this in the 2.1.1 release. It is not needed for 2.1.0.


"error rate" here meant to be equals to (1.0  -  B3 F-measure). Isn't understood by readers in this form? Anyway, I will remove that expression if it causes confusion.


OOo whats going on here


Nice catch!


this refers to https://github.com/pupil-labs/pyuvc/issues/9


Yes, I deduced that from the numbers, but it does not really make sense in the context of clustering results.


Because website doesn't depend on mail, so mail.template is unknown at this time.

We can add a dependency, but it seemed wrong to me at this time.


We should remove this test as timing constraints are meaningless on different machines; and in general it is not a good practice. I used this during the development to make sure that I can reach a desired run-time limit on an average developer machine.


Okay, I will modify accordingly and push the changes.


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


actually yes, store version == crate version (libimagstore-version).


Why do you use `pkgs.postfix` instead of just calling `postalias`? Same with `postmap` and `postfix set-permissions`. We have `postfix` specified in `path` above, so this shouldn't be needed.


We also have to consider caching, so probably quite a few components will call ```getPersonByID(10)``` but only the first one (and then maybe again in like 10mins) should issue a network request.


You did not create a PR with this


Cool, I'll pull it in.


You're right I will use fabs first.


Oh, already in, never mind ;-) All good!


Raising this error will lead to a `Not Found` response which is acceptable. What do you think about responding with `Unauthorized`?


This is last commit?


A pleasure mate. Glad I could help :)


Sorry, I meant vfp.cc in the ARM case and fpu.cc in the X86 case.


Yeah. Now if only the preview looked better.


If you want to handle 404 in express, then you can't use the `app.use('*')`
But as it is now, then you can just handle 404 in the angular app


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


do not print the offending class but rather the full message, e.message


Through the rethrow method, all exceptions (except Errors) are rethrown as RunTimeExceptions. If we already have rethrown an exception, do we ever reach this line? Isn't the method call terminated with throwing the exception? If not, what reason would you have to throw this (second) RunTimeException? I am not sure whether your IDE is aware that we are actually terminating the method with the rethrow method, but if not, is there a way to do so?

I am not sure on this topic, but would a modification of the method head work?
rethrow(Exception e) throws RuntimeException

Same goes for the above methods which have a return value.


But the user already has the code but cmake crashes  because of the submodule update (which often is not necessary but is done anyway) and he/she is offline
at the end we print some info, these info tells if there the module is there or not., it should be enough for the user I think no ?


Even if it is just a comment, we might want to give it a more appropriate name. I have done some thinking and it is not too easy to find a suitable name. Maybe Illegal Operations? Or if no suitable one can be found, Various?

Various isn't a very good name either, but imo is still better than derp.


Yes,

I think that should work.

And yes, this is only for uncompressed images. It requires some code duplication as we need two parallell implementations (uncompressed/compressed).

No buffering or precalculation (I don't think it's even possible) for compressor streams. Seeking is the only way for compressed images I think,

I'm not sure if this feature is really important, but in some cases (dynamically generated content on the web) it's good to be able to stream data without the back/forth seeking (which will require buffering too). So I figured it was a good thing when I initially implemented it. If it's not too much work, it's good to keep it that way.

--
Harald K


Shouldn't the logic be as follow:
if(!children[i] || children[i].interactive)

as you'd want to set hitTest to false if children[i] is undefined or if it does have the interactive property. But this shouldn't break the test though. In theory.


Maybe I should check if the value is 0 or 1, if not I will return an error.


running this in production will not work as neither of these paths will exist


ok, thanks.


Ok, I thought mail was a dependency of website. Website_mail is auto install so it should not be an issue.

By the way, do you know where are the other templates ?


Nice find, good work


this is a comment


I do not know of a good solution to be honest. This line is indeed to stop the IDE from complaining.


looks good


Nit: newline between title and commit description (makes everything end up in %s in git log)


Use an AssertionError instead.


Agree, better for consistency.


this will alway only return true or false, it is unnecessary to make it explicit


dito


this can be done with a guard clause


No it does *not*. This version is the version of the given store representation a library chooses. It has nothing to do with `libimagestore`


Also added multiprocess compatible switch to package.json which puts it into the install.rdf after "compiling".


unnecessary explicit return for short hand if.


Same as above - would like to know how to properly revert without having to downgrade the database.

Getting the below with latest commit:

C:\SickRage\sickrage>SickBeard.py
Traceback (most recent call last):
  File "C:\SickRage\sickrage\SickBeard.py", line 23, in <module
    from __init__ import main
  File "C:\SickRage\sickrage\__init__.py", line 23, in <module>
    from sickrage.requirements import install_pip, install_reqs
ImportError: No module named sickrage.requirements


same here for short hand if


Why not on Kconfig?


Shouldn't the exec call be removed? [According to the doc, this makes it modal](http://doc.qt.io/qt-4.8/qdialog.html#exec).


'A handle of a' what? =D


@trotzig the test captures the functionality correctly. :+1:


构造函数


I intentionally left the brackets to reduce the amount of changes from P4 to P5. I'd leave it as a Plone "signature", besides it's not entirely useless after all. However if you feel like removing it just go ahead, I wouldn't oppose either.


rather use e.message then sucking the class out of the error.


Why did you switch to systemctl?  You had it correct before.


I'm sorry. I did this commit by accident. I think the modifications might be small improvements, but you can pull back commit if you want. There a bunch of small modifications, to try to make the latest version of the repository with my code. However, I gave up, and I will rewrite my code to deal in a better way with openworm.


you could have also did a cache busting technique.. append `?###` to the end of the file.  ### could be webui's processid..


can put this at the top of the class


````java
for(String param : list(var1.getInitParameterNames())
    actions.put(param, var1.getInitParameter(param));
````
Ексепшены по постройке обработчика засунь в ActionHandler.put


quick note - the "short hand if" that you refer to here is usually called a ternary statement, or just the "ternary operator".


@lijunray you need to mock the behavior of the dependencies which make it fail.  If you mean it will not compile, you need to provide an anonymous implementation of it when you construct it in the test.


SLT


explicit shorthand not needed


ok


same here


All hail our new FRC cyborg overlords.

On Tue, Jan 19, 2016, 21:44 nodcah <notifications@github.com> wrote:

> It's scary because I don't know how much you're joking haha
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRCTeam3182/FRC2016/commit/a8326bc82aef1440202637c76d338887333ee14d#commitcomment-15547097>
> .
>


You are correct. The above is for development only as per the comment.
The actual paths will be read via environment variables.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 72](https://teamcity.labkey.org/viewLog.html?buildId=298193&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17 (3 new), passed: 31 Build time: 01:10:57
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


can be done with a guard clause


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


guard clause


I tested https://dl.dolphin-emu.org/prs/pr-3364-dolphin-latest-x64.7z, and 2 bugs remains :

* When directX12 is selected and the game is started, then the emulation window isn’t closable.
* When the main video card doesn’t support ᴡᴅᴅᴍ 2.0, then the Microsoft software render doesn’t works *(which is my single ɢᴘᴜ case)*. It may also concern hardware ɢᴘᴜs supporting ᴡᴅᴅᴍ 2.0 with a ɢᴘᴜ supporting ᴡᴅᴅm 1.3 *(in a multi configuration)*.


#### Warning: There is another whoop here, it should be preR3 but N0T preR4!


s/biding/binding


Will test this as it was returning true or the dreaded nil.


Maybe we should have another pre release idk need features for a release


Change the year too?


That's a SonarQube Issue


[warning](http://eslint.org/docs/rules/camelcase})

Identifier 'helllo_this_is_a_test' is not in camel case.


[warning](http://eslint.org/docs/rules/no-undef})

"helllo_this_is_a_test" is not defined.


You're right I will remove the functions to_machine_order() and to_network_order() and leave only swap_bytes().
I will also remove the machine argument.


@RandomAltThingy We can push pre-releases as releases for expirimental purposes and it should be at least 40% stable. xD


Yes, here the ternary operator is not needed.


je viens de voir sur Naruto avec la master, j'ai plus le trai noir que j'avais avant, c'est propre maintenant :)
Il ne reste que 2 probleme je crois, la lenteur xd, l'instruction man,quante doit y etre pour quelque chose, surtout on demare a 10 fps on se retrouve a 2 quelque seconde apres.
Et ca
![capture d ecran 2016-01-20 14 23 00](https://cloud.githubusercontent.com/assets/5261759/12450023/6f351280-bf81-11e5-8c39-d588cc87072d.png)
Au dirait un probleme de profondeur, naruto est afficher comme étais derrière la porte alors qu'il est devant, je sais pas d’où ça viens ça


Actually for this one, if we simply changed raw-sha1 to only look at the first 16 bytes of the result in cmp_one (or cmp_exact?), then it would work for this format.   Using only the first 16 bytes out of 20 does not cause false positives. Dyna only uses 16 byte compare for ALL hash types.


I suppose on the module website_XXX where XXX depends directly or not on mail, like website_slides.


A simplified version of "using autotools with OpenSGX" provided by John D. Ramsdell


@qwcode a little explanation or link about why the change is needed or recommended would be great for beginners. TIA


이 역할을 해주는 함수가 <string.h> 헤더에 있습니다.
memset() 메모리공간을 초기화해주는 함수입니다.
찾아보세요 :-)


NSLog(@"SPRAWDZENIE");


ㅋㅋㅋㅋ 감사합니다


Why merge???


are these all toArray methods in planner?


These ldif + sql files are linked from the documentation.
I'm not a big fan of this move, because these files are not related to docker, they are used by docker, but not only...


I think the `Theme` should not write the settings to the filesystem by itself. Since the settings are shop-specific, maybe the `Theme` class should not access the filesystem at all, nor know about which shop it belongs to, and rather we could have the `Shop` class write / load the settings from the appropriate location on the FS and pass them to a `Theme`.


Looks good, but the "Installation" and "Usage" distinction is less useful at this point. After I merge down this PR I'm going to just give a single "Usage" heading and see if the flow of things makes more sense that way. Thanks!


THANK YOU!!! :smiley_cat:

(Just noticed the bold labels in beta2 - finally somone snuck it in :)


sidebar 里面有很多导航条目的，默认的 flex-direction 是 row ，也就是水平排列，这个肯定不行的，所以 `flex-direction: column` 很必要的，让各个条目垂直排列。


Most up to date glm function is called ds.glm.5s. ds.glm.5s calls
glmDS1.5s and glmDS2.5s (see dsBetaTest repository). It is THIS version
of ds.glm that has been modified to create ds.glm.REMA which calls
glmDS1.5s (ie same as ds.glm.5s) and glmREMADS which replaces glmDS2.5s.


Except crm and project, I don't think any other template for automatic email has been merged, have they ?


The "raising the class" short form is explicitly supported by Python and is consistent with the style of the rest of the code...


OK


Yeah, good point. Sounds good!


Maybe add a `break` here because the loop has already fulfilled its purpose.


앗! 감사합니다 ㅎ_ㅎ


This causes calculations to fail.  The correct value "appears" in the form, but the result_str (which is set to self.value in calcs.py) is the interim-field value, not the calculated result.  WINE-110 and WINE-98.  I'm changing it back to parseFloat(result.result), please check that I don't blow anything up!


Why do we need this?  @runcom PTAL


Why advance `data` twice but decrease `len` thrice?


This is definitely a bug


Can't. This will create a HTTP dependency.


All tests passing..seems ok for now.


Apparently `bundler 1.7.6 and 1.8.3` doesn't play well with `gem 2.5.1`. This should be just a temporary workaround until default versions on Travis workers become compatible


How about I make this change.
```
// SecretMount returns the Secret Mount point
func (container *Container) SecretMount() (execdriver.Mount, error) {
	var m execdriver.Mount
	return m, nil
}
```


This "if" is not necessary, isn't it? It look like the code will work as well without it. Actually it should run faster since you get rid of one RPC call.


Ah you were speaking about these templates, my bad.

Nope, only these three ones. @bst-odoo do we already have mockups ?


Apparently bundler 1.7.6 and 1.8.3 doesn't play well with gem 2.5.1. This should be just a temporary workaround until default versions on Travis workers become compatible


The API user can choose which port to use when creating the lwm2m server.
I don't think we should put this in the Kconfig.


Why does this need to exclude MULTI=1 and NOWRITEV=1 too? Those should use bundled libtom which does compile on OSX?


ok


If new_domain were None, we would have got an AttributeError long before getting this far.


要删除apktools.jar么


I think that would work better, would be easier to organise/sort the database table as well.


Typo: Hellow -> Hello


geoserver_data instead


`springVersion` and `springBootVersion` are duplicated here.


Sorry about that, and thanks for the report.
I'll try to look at it as soon as I can.


last one is for website but that's it. https://docs.google.com/document/d/1qnFAbwYt8g11NPDRow0_BhXYLq7mv1ynZGpKXK6fIXU/edit


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


/var/local/gisdata instead of /var/local/shapefile (since tiff files can aso be stored in here)


I am pretty sure this does not get ever called.


Doesn't `*cinfo` leak?


Looks good to me.


Is it possible you forgot to commit autoload/dotoo.vim with the updated use of open and close?


As I said: "Not in the most elegant way though..."
Will come up with something better shorlty.


Keyword 채널이 살아있어서 안지워도 될것같네용


Derived from https://github.com/mbutterick/pollen/pull/108


Yeah I've actually mocked this now, will be pushing today once they're all
passing again

On Wed, Jan 20, 2016 at 3:20 AM, Simon Kelly <notifications@github.com>
wrote:

> seems like this requires a running Django instance with the appropriate
> data correct?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/formsplayer/commit/69d65011ee659548ee58aaf868317c993b6500d2#commitcomment-15551121>
> .
>


봇 죽어있는데영


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


change


Take note that Tag is not equal to wallet. Tag is about the NFC service.


It’s unclear when this happens, what about prefixing the item with “When writing, …”?

I’d also s/field/fields/ or s/field/field values/.


Thanks !


Good to hear. I carved a 1.2.2 version with this change included.


Yessssss :) can't wait to test it out :D


Ok ok. @Espurna keep an eye here, please. I guess this result_str code was yours


shapefile_repo -> gisdata_repo ?


살렸슴돠


Good catch, thanks! I'll remove it.


Munge tests don't run: need `mungeuser` to chown relevant folders (see https://github.com/dun/munge/wiki/Installation-Guide )

munged: Error: Logfile is insecure: invalid ownership of "/var/log/munge"


LGTM


We have backported this to 1.13, 2.2, and it is included in 2.3. We do not intend to backport further, as there is no real need. It is pretty trivial to lock jQuery to 1.11 or 2.1, and there doesn't seem to be a major reason that I can see in the change logs that would force users to be required to upgrade jQuery further.


Sorry, I'm going to reject this PR as I have a more radical fix (which is to get rid of this -unused- struct altogether).


Correct me if I'm wrong: The point here, was so that the "calculated result" would keep the same decimal places as the "interim field"?   But this "self.value" is only the interim field that was latest edited, so maybe there are things happening here that I don't understand.


Thanks! :)


If exec_() removed than dialog does not displayed. Currently Modeller and Resuls dialogs are non-modal and don't block QGIS main window.


:+1: 6x6 matrix makes sense. Implement off, na. Or tell me, if you want me to work on it.


Hi! im here dogo amani im welcome!!!!!


Does anyone know anything more succinct than

```bash
git g -l "genotype" | xargs sed -i 's/genotype/phenotype/g'
git g -l "Genotype" | xargs sed -i 's/Genotype/Phenotype/g'
mv src/genotype.js src/phenotype.js
```


It won't the calling function will remove it from the array and delete it.


TKI-Gerüst Merge


im a new user and im so gratefull for the add.hello!!! GITHUB


Hi Rachel,

Thanks!

Here’s the extra credit :)

Best,
Douglas Walker

> On 19 Jan 2016, at 22:40, Rachel Smyth <notifications@github.com> wrote:
>
> Really great work! Very clean, very clear. I would clean up lines 114/115 and refactor some of your code even further:
>
> // you can simplify statements like this with a ternary operator
> if(computerMove === 'rock') {
>                 winner = 'Player';
>             } else {
>                 winner = 'Computer';
>             }
>
> // like so:
> winner = (computerMove === 'rock') ? 'Player' : 'Computer';
> Loved the attention to detail in line 107 with your quit functionality!
>
> I would have loved to see you try the extra credit as well.
> Overall Great work!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/douglaswissett/wdi-fundamentals-rps/commit/b9a28bade68b93dc574cfc52662d11bd40f36b69#commitcomment-15547775>.
>


Is this ok to be uploaded to SIGIR? @glouppe


@mhinz Should we use the same directory for vim and neovim ?


Well, the emunand is only selected when you enter the menu, right? If the right emuNAND offset is patched only when it's changed, you don't need to repatch the firmware on every boot.


这仅仅是一个通知程序  单加是没有任何意义的  包括powersuspend
当有驱动挂起时使用这个通知  才是有意义的


```

AS_IF([test "${GO_MAJOR}" -eq "1" -a "${GO_MINOR}" -eq "5" -a "${GO_MICRO}" -lt "3"],
      [AC_MSG_RESULT([no])
       AC_MSG_ERROR([*** go version is vulnerable to CVE-2015-8618 (${GO_VERSION})])],
      [AC_MSG_RESULT([yes])])
```


here is my vimrc
```vim
"set nobackup
set backup
set undofile
set undolevels=1000
let g:data_dir = $HOME.'/.data/'
if has('nvim')
    let g:backup_dir = g:data_dir . 'nvimbackup'
    let g:swap_dir = g:data_dir . 'nvimswap'
    let g:undo_dir = g:data_dir . 'nvimundofile'
else
    let g:backup_dir = g:data_dir . 'backup'
    let g:swap_dir = g:data_dir . 'swap'
    let g:undo_dir = g:data_dir . 'undofile'
endif
if finddir(g:data_dir) == ''
    silent call mkdir(g:data_dir)
endif
if finddir(g:backup_dir) == ''
    silent call mkdir(g:backup_dir)
endif
if finddir(g:swap_dir) == ''
    silent call mkdir(g:swap_dir)
endif
if finddir(g:undo_dir) == ''
    silent call mkdir(g:undo_dir)
endif
unlet g:backup_dir
unlet g:swap_dir
unlet g:data_dir
unlet g:undo_dir
if has('nvim')
    set undodir=$HOME/.data/nvimundofile
    set backupdir=$HOME/.data/nvimbackup
    set directory=$HOME/.data/nvimswap
else
    set undodir=$HOME/.data/undofile
    set backupdir=$HOME/.data/backup
    set directory=$HOME/.data/swap
endif

```


Just make this into a function that draws the spokes given the current spoke state, we're going to have to do it at some point or the other. Actually, i'll do it off, chill.

Also, people avoid for loops in high-level languages nowadays. We use stuff like https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map and the other array functions, because it's less code-ey and much closer to our mental model of doing things. I'll change off.


这仅仅是一个通知程序 单加是没有任何意义的 包括powersuspend

当有驱动挂起时使用这个通知 才是有意义的


To revert, I just did 'git checkout 4.2.5' or whichever commit you prefer.


`Data.LargeHashable.Tests.Class:integerUniqueness`?


cc @timse - sorry for :trollface:


I think `String` should throw its own error.


I don't think we need this.


Yeah, that was what I meant


怎么把这句也给删了？font需赋值。


Guess this was just for debugging? :)


I think this could be `try? String(bytes: replacedStringArray)`


Strange, QDialogs behave differently when I create them in a python console, but as long as it works ...


o/


No need to include `app.js` and `app.js.map` files. They get built


You have a typo above.  You changed them both to `:new_value`.  The first one should be `:old_value`.


We’ll delete it.


sdsdsd


No need to include a reference to .d.ts files in this way if they are part of the Visual Studio project


I think we can leave this and `init?(bytes: [UInt8])` as well.


It is not a big problem in my case, but renamed namepaces and assemblies could become a much more complicated things to handle under specific conditions (deserialization purpose, ...). Of course nothing is impossible, it is just a matter of difficulties and time to implement.


Hehe! Thanks, many eyes!


magic text n-cube...


Ok, I got confused. Thanks.


Ah, I didn't even think of doing it this way - very nice! LGTM


Because ```[wheel]``` was a custom behaviour in old versions of the wheel project. The standard way of setting arguments for a setuptools command is to use ```[command name]``` and the command name here is ```bdist_wheel```. Knowing about the old form isn't really relevant, as it's long out of date.

Why are you concerned about the reasons for a commit from nearly 2 years ago? Is it relevant to something you're encountering currently?


I'm seeing spaces instead of tabs. Looks like our Visual Studio formatting settings have not been imported. Believe there is a link in the dev environment instructions somewhere. Just import those settings and hit `Ctrl-K-D` -> should reformat the file correctly.


functions are lowercase


How does this look with the [wiki](https://github.com/Kayant/powerlevel9k/wiki/Install-Instructions).

@dritter, @bhilburn


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


This is not the nicest.


[warning](http://eslint.org/docs/rules/camelcase})

Identifier 'this_one_is_a_dot_es6_though' is not in camel case.


[warning](http://eslint.org/docs/rules/no-undef})

"this_one_is_a_dot_es6_though" is not defined.


[warning](http://eslint.org/docs/rules/no-undef})

"but_this_is_not_a_dot_es6_file" is not defined.


[warning](http://eslint.org/docs/rules/camelcase})

Identifier 'but_this_is_not_a_dot_es6_file' is not in camel case.


Rebuilding the image right now, will send another merge request if it's working.


would be necesary to investigate if it should remain for back compatibility => adding conditional import


Ben and I actually discussed this earlier and decided pulling source was the cleaner option - gives us more visibility into the version we're using and let's the dev environment be a lot cleaner (I have all the code in one place this way). Would you argue for continuing to use jar dependencies?


Hello, @futtta.

I think it's possible to detect Cloudflare using the [request headers they add to the request](https://support.cloudflare.com/hc/en-us/articles/200170986-How-does-CloudFlare-handle-HTTP-Request-headers-), but this does not means that we can detect if Rocket Loader (which causes the bug) is enabled or not through it.

Do you think it's important to add a condition to check if the request is passing through Cloudflare or not to add this 'data-cfasync' attribute? I am not sure if it's so important because it's just a few bytes anyway..(and it's correctly prefixed, as Cloudflare needs)

A plugin that I uses in my blog too, called Disqus Conditional Load, added an option to enable this attribute or not..It's an option too (but a bit more complex..)

Thanks.
What's your opinion about this?


@shun-s we might not need this test? as this case is already covered by quite a few tests?


Rather make this object typed : then you'll get type checking and code-completion. We reserve the square-bracket syntax for parsing and constructing JSON.


This can become `public func receiveString(bufferSize bufferSize: Int = 256, untilDelimiter delimiter: String, deadline: Deadline = noDeadline) throws -> String`


Fixed, new commit (now force push to the PR) dd2dc40


this can become `public func receiveString(bufferSize bufferSize: Int = 256, deadline: Deadline = noDeadline) throws -> String`


> I wonder why you recommend to add the cmd/ folder to the path. AFAICT cmd/contains only a redirect to mingw??/bin, right?

Not quite. It [sets a couple of environment variables first](https://github.com/git-for-windows/git/blob/78ce6bc19e6b7ce9433b889b2a67d53b8fbeb557/compat/win32/git-wrapper.c#L40-L124). And they turn out to be crucial for correct Git behavior.

> Would it hurt if I set the path directly to mingw??/bin?

Yes.


screenshot?


Sorry, I've long moved away from express-handlebars - you'd have to either edit the files yourself to use this patch (the maintainer doesn't seem to be willing to add it), or find another view engine.


Is this working?


Like I said, this is all extremely low-impact, so let's just leave things be :smiley:


we need to "return 1" in case something unexpected. so you might want to put something like:
```sh
run_mon $dir a --pid-file= || return 1
```
instead?


This is definitely not working. haha.


Nice touch to add search


Mostly just to ensure stability for today's demo.

It seems like some point between 1.9.3 and 1.11 tika-python went from returning a list of values to a single value in metadata, additionally the dyndns machine was throwing a Tika 404 which I wasn't able to replicate locally.

These are the sort of things that we'll be able to debug more effectively on dyndns when we have a staging environment, so we don't have to interrupt the SUG.


>Why does this need to exclude MULTI=1 and NOWRITEV=1 too?

You're right, there must be some shorting in my head.
I will open a pull request later.
The `NOWRITEV=1` on OSX case is going to be a little trickier, since the arguments to `sed` will be a little different.


for the range-based for-loop, you should use `for (const auto& str: mStrings) { ... }` otherwise you'll be copying each string


user must be able to delete others comment if they are in his own recipe


getBestSolution() is null. And even if would not be, the getBestSolution().getAssignment() does NOT return best known assignment. You should also check that there is a best solution saved.

How about using getBest() (or iBest) instead? Assuming that you set the best value in the bestSaved method, e.g., using:

  iBest = getValue(solution.getAssignment());


> Why did you decide to make this a singleton.
> It makes AsyncHiredisCommand unusable in several threads.
>
> Also the logic of this class is not proper because there's no guaranties not to appear the same address of "Connection *" twice.

Probably solution should be improved.

> Despite all of the above I didn't exactly understand when the probplem occurs.

The main problem is - async library doesn't handle disconnections of redis servers in cluster. Library doesn't know if some of servers were disconnected. So segfault is very possible, since hiredis context was freed for disconnected server. You can reproduce issue with async_disconnect example.

> There's no need to make redisAsyncDisconnect in disconnected callback at all, hiredis knows about this and would free connection without any additional invocations

It's true only for disconnected servers. But since we work with cluster of several servers, some of them may stay connected, so we should disconnect all of them and free contexts.
I suggest next solution:
- get disconnected callback from hiredis for some server (context)
- remember pointer to this disconnected and freed by hiredis context
- throw exception to the client
- client run cluster disconnect
- cluster run disconnect for every connection
- disconnect server and free context if it is not already disconnected and freed


Yeah I should makem all of these tests.. proper tests.. haha


@RobotXLabs This doesn't work. I don't really know why, but I am sure @thatoddmailbox can help.


yeah :(


Added comment.


Since this inherits from `SimpleAlgorithm`, and these are declared `virtual` in the parent, marking them as such here is a no-op (& isn't particularly meant to encourage subclassing). I don't think this is mentioned in our coding standards. However, it follows the convention used in [the](https://github.com/lsst/meas_base/blob/master/include/lsst/meas/base/GaussianCentroid.h#L112) [other](https://github.com/lsst/meas_base/blob/master/include/lsst/meas/base/GaussianFlux.h#L76) [measurement](https://github.com/lsst/meas_base/blob/master/include/lsst/meas/base/NaiveCentroid.h#L81) [plugins](https://github.com/lsst/meas_base/blob/master/include/lsst/meas/base/PsfFlux.h#L84), so I'm inclined to leave it as it is unless you object.

By the way, not all algorithms need inherit from `SimpleAlgorithm`: rather, we have `SingleFrameAlgorithm` and `ForcedAlgorithm` depending on the type of measurement. `SimpleAlgorithm` is a shortcut for when the algorithm is sufficiently simple that it can serve as both.


This is an excellent point, and one we don't have a good answer to at present. There are various fragmentary pieces of documentation -- in [Doxygen](https://lsst-web.ncsa.illinois.edu/doxygen/x_masterDoxyDoc/classlsst_1_1meas_1_1base_1_1_base_transform.html), in [Confluence](https://confluence.lsstcorp.org/pages/viewpage.action?pageId=20284390), in [Jim's Bootcamp talk](https://github.com/lsst-dm/Oct15_bootcamp/blob/measurement/measurement/measurement.pdf) -- but no coherent overview.

I don't think it's appropriate to try to add that to this ticket, but I'll file another requesting that we work @jonathansick to ensure it's all clearly presented in some central, maintained location.


LGTM


Why?


My initial thought was that anyone running Ruby 2.3 would want to know about future compatibility issues. Your suggestion makes sense and I will put in a PR to address it.


A better fix would be to move the following code from `simclist.h` to `simclist.c`
```
/* Be friend of both C90 and C99 compilers */
#if defined(__STDC_VERSION__) && __STDC_VERSION__ >= 199901L
    /* "inline" and "restrict" are keywords */
#else
#   define inline           /* inline */
#   define restrict         /* restrict */
#endif
```
`inline` isn't used anywhere else in `simclist.h`; it's only relevant for `simclist.c`


```html
<script async src="/client.js"></script>
```


Does it make sense to check for err ~= nil here?


srinu comma


Seems like a big update...


Why by_addr? Used to be by_name


I would call this `show`


Should be ValueUpdateType.NoUpdate as you are NOT using the ValueContext.iTotal and do not need to have it automatically updated during the search (by calling ValueContext.assigned() and ValueCotnext.unassigned()).

Or, keep it ValueUpdateType.BeforeUnassignedAfterAssigned, but do NOT override beforeUnassigned and afterAssigned on the Criterion, but override the assigned and unassigned on the Context instead.


Question: Who deletes branchJumpTableWrapper now?


Most likely won't make that change in 2.3.2 which I'm planning to get out pretty soon. Might do that with the 2.4 release.


Hmmm... if `new_client_info` fails, flow goes to `err_exit`, which does not call `client_info_del`, so I think it leaks, don't you?


I see that you changed semantics of the existing tests. Please explain.


good point


cute, we have an adventure mod.


@dontcallmedom shouldn't the above `DOMException` be replaced by `<a href="http://www.w3.org/TR/html5/infrastructure.html#domexception"><code>DOMException</code></a>` (as done on the following line).


ssdfas


nice! :)


this doesn't make sense. `autoload` only works if you remove `require_relative`


This needs to be reproduced outside the context of sonata, in our test suite.

Does this only affect 2.6.x?


Yea, there's no difference for undo/swap/backup files, so you can use the same directory. Only `'viminfo'` became `'shada'` and is put under `~/.local/share/nvim/shada/`.


wkwk


Also updated the Berksfile.lock syntax - nobody's really touched this cookbook since Berks 2.0 era.


밤늦게 고생이 많습니다. :+1:


PR updated to ensure that the 'default_source' entry is always included when a default expression is present.


`const` ?


for even more idiomatic C++11, you can use `constexpr`, which defines compile-time constants:

`constexpr auto string_size_limit = 1024 * 1024;`


Que tal voltar?


If you added a label to the searchbox that said `ID` it could prevent some confusion. I tried searching for some of the content the first time round.


references to method calls should have a weak reference to the object and therefore do not need to be scanned


I don't feel strongly one way or another; but that's out of scope for this PR which is only about making the source file valid to pass Travis validation :)


fixes #2


ok, more flans, i have yet to fiddle with futurecraft.


Because I know I'll forget: Please add a comment explaining that 65536 is the number of scaled point in a point and thus a form of "1" for the purposes of this calculation.


Why you need to keep track whether a complete solution was found? That could better done by checking the current solution (e.g., assignment.nrUnassignedVariables(getModel()) == 0) and/or the best solution (e.g., getModel().getBestUnassignedVariables() == 0). You would not need to use the SolutionListener in that case.

Also, the current implementation does not work in UniTime if a solution gets loaded in (as neither init nor solutionUpdated are called before the solver is started).


`!0 === false`, so searching for zero is impossible


Ownerpersonalid is actually  status of owner's personalid. It indicates , if owner has persoanlid in the system. It is boolean value. So we can remove from here.


git pull


Why is `.eslintrc` still in the original form as in #199, despite the discussion there?


This throws an exception when an existing solution is being loaded in UniTime.


Yes this is fine, thanks


I can search outside bounds (i.e. `<0` and `>recordCount-1`)


Shouldn’t this also be `/* eslint-env node */`?


Does it support recovery from FOTA kernel partition?


yes (those that don't rely on filling with null values at least)


Replace "Uou" with "You"


closes #11


thanks ,if so ,it I could make my vimrc shorter


this mess is no longer needed with your getter always checking for that; further, 'n-cube'.


done :+1:


Woops


The patch for `scan.c` incorrecly calls `tsd_task_reset()` instead of `tsdfx_scan_reset()`. This causes scan tasks to run continuously instead of running at the specified interval, and to fail to notice if the target disappears.


Look like invalid syntax to me.


Not sure what the purpose of this is?


i hope iggy is super configurable.


No trailing empty line please.


What about for instances where this is not 64?  Non-DM devices will return the value in ext_range.  Or should anyway.  Need to check for both here, IMHO.  First, a PED_DEVICE_DM returns 64 and then an ordinary block device returning something that is not 64.  For example, on my system here, /dev/sda has 256 in ext_range.


I think the volume_windows.go setupMounts() would end up returning a ErrorCodeVolumeNoSourceForMount error, but I haven't verified this by running it.


감사합니다 그냥 자료형을 void로 바꾸어 반환 값이 없도록 만들었습니다.


Website template is already merged in this commit ;)

I thought there were a lot of ready mockups, too bad


The access privilege allows players that have it to bypass protection on locked doors/trapdoors, chests and bones. Currently there is no way to do this without mods.


Already notices this afternoon, but thx!


What exactly is not working and why do you check it in if it is not working?


Thx, will fix!


Typo.
The previous one was correct :smile:


@Ocramius you mean doctrine? It affects all versions with this commit in it (I used `git bisect` to track it down).
I would like to help, but honestly I don't know how to build a test case with existing database entries.


Is this your proposal for the book's structure, or just a narrative about the path you've taken?


зря


This whole commit should be squashed with the first one in order to get a sane commit history.


Describe in commit message what was wrong - and why it is not wrong any more.


Sim concordo com Voltar como não tinha visto em que linhas e strings estariam essas duas opções fiquei na dúvida pois a frase podia depois não conicidir com a escolha final das duas opções


why these two files only? User can specify more of those in presto config.


I take it back, for loop was easiest there


Shouldn't be too much work, mainly moving where the IDF pointer is written.
The duplication is still there, as I thought the only special requirement for streaming is writing the IFD before image data to enable streaming, not that I can't update the pointer later on. I'll look at it :)


I think that this, and subsequent `debug` log messages indicating that the various wars are not found, should be changed to `info` level.


Now, this looks promising.


How about to count each instructor only once, like it is done in the InstructorLuchBreak.getValue(assignment, variables)? Would summing up Instructor.value work?

You can use getValue(Assignment<Lecture, Placement> assignment, Collection<Lecture> variables) here.

However, the getValue(Assignment<Lecture, Placement> assignment, Collection<Lecture> variables) needs to be overridden and changed so that getValue(assignment) == getValue(assignment, getModel().variables())

How about something like:

    @Override
    public double getValue(Assignment<Lecture, Placement> assignment, Collection<Lecture> variables) {
        InstructorFairnessContext context = (InstructorFairnessContext)getContext(assignment);
        if (assignment.nrUnassignedVariables(getModel()) == 0 || context.allInstructorsAssigned()){
            Set<InstructorConstraint> constraints = new HashSet<InstructorConstraint>();
            for (Lecture lecture : variables) {
                constraints.addAll(lecture.getInstructorConstraints());
            }
            return context.getObjectiveValue(constraints);
        }
        return 0.0;
    }

And, in the context:

        public double getObjectiveValue(Collection<InstructorConstraint> instructors) {
            double ret = 0.0;
            for (InstructorConstraint ic: instructors) {
                Instructor ins = stringInstMap.get(ic.getResourceId());
                if (ins != null)
                    ret += Math.abs(ins.getFinalValue() - instrMeanFairValue);
            }
            return ret;
        }


Yeah, I noticed!


Just one inline comment on changing some `debug` level messages to `info`.


Considering that this isn't a very expensive operation - is it worth setting this to a variable here? or is it worth being more explicit where it is being used.

If it's really thought as advantageous - could the var_name be given something more explicit, like num_of_indexes or something similar?


PKCS#11 ECDH is ugle. It has many more paramaters that need to be passed somehow.
At a minimum, the EC_POINT of the other key, which can be obtained from the certificate of the other user's certificate. OpenSSL does that and passes the EC_POINT and a KDF function to the engine.

In addition in PKCS#11 a secret key object is created which holds the derived key.
This could be on the token if the card supported it (don't know of any that do), or is maintained as a session object by the pkcs11 lib.

Then in PKCS#11 the caller would use the secret key to encrypt/decrypt, or the caller
could get the value of the secret key.

The engine  via the EC_KEY_METHOD  calls  pkcs11_ec_ckey, passing the EC_POINT  of the other key.  pkcs11_ec_ckey then calls PKCS11_ecdh_derive, and requests the value of the secret key.
OpenSSL is only using the engine for the lowest level for key derivation.

So how do we simplify this?

For now the engine could use an internal  renamed version of  PKCS11_ecdh_derive.

Then how do we expose via libp11 API the ECDH functionality without exposing:
 CKM_ECDH1_DERIVE,  CKM_ECDH1_COFACTOR_DERIVE, CKM_ECMQV_DERIVE,
 CK_ECDH1_DERIVE_PARAMS and  CK_ECMQV_DERIVE_PARAMS?

PKCS#11 2.20 only defines CKD_NULL and CKD_SHA1_KDF.
2.40 add 4 more.

My goal has been to get the engine to work with ECDH.  Exposing ECDH via libp11 API could be another project.

How do you want to proceed?


Hmm... I don't remember i've added this. Will change this


Good! I was going through the code sequentially and did not see it when I
was looking here.

sternenseemann <notifications@github.com> schrieb am Mi., 20. Jan. 2016 um
14:52 Uhr:

> Data.LargeHashable.Tests.Class:integerUniqueness?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lukasepple/large-hashable/commit/c6d65587a7a614d6804dfb8f64889aeda84b1c06#commitcomment-15557847>
> .
>


Hmm. I have a sneaky suspicion that your infinite loop is forming in lines 84-85. Do those functions need parameters?

Glad to see you worked on this. :smile: :+1:


I think these comments should be addressed by now — we're certainly not using `error` any more :-).


Don't know how useful this example is as you can use `index: logstash-*` currently.


Ok, I dug this up and it is indeed only affecting `master` (2.6.x, unreleased).

@toooni what you can do is try to track down all the interactions with the entity done by sonata (and write them down).


The `console.log` use here is correct and deliberate. It is used in a debugging function activated by setting `DEBUG` to `true` at the top of the file.


W pliku  app/models/person.rb zrobiłam scope o nazwie "current", tak jak Pan prosił. Wydaje mi się, że wszystko jest dobrze, mimo to zrzuca się wyjątek "undefined method `current' for #<Affiliation::ActiveRecord_Associations_CollectionProxy:0x007f4f1a0a0030>".


Thanks


@amaltaro is there no way to do this with a generator or return an iterable? Well I guess chain.from_iterable() does that. Do you really need a list in your final code or can you use an iterable?


Don't we have type inference.


This is crap.
1) Subclasses of higher rank will have has_register == True just because their parent has it. Removing has_register will cripple the feature of registering items to all of superclasses' registers.
2) Entities have been burdened by a part responsibility of their repositories (register and get_by_id).

The whole register thing should be moved into the scope of repositories.


Why not `>=1.1`?


looks good


The commit message doesn’t describe what the commit does but what you found. Something like “Remove duplicate case clause (found by eslint)” would be better.


I removed `String init(bytes:)` though. More consistent to go through `Data` don't you think?


s/user/use in commit message


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1204](http://teamcity.monogame.net/viewLog.html?buildId=13811&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:27:48


@amaltaro  Are you sure you can't do all of this "stuff" with the class designed to handle lumi lists? I really hate to see all these lumi list manipulations embedded in our code. It's obtuse and very easy to make off-by-one errors.


Same here `init?(bytes: [UInt8])` removed


Again, the commit message describes your progress, not what the commit does.


This commit breaks compilation of `icub-main`: https://travis-ci.org/robotology/icub-main/jobs/103605563
@randaz81


코멘트 감사합니다.
1. 지적하신 부분인 MakeEmpty() 함수는 활용의 목적이 이중적이어서 위와 같은 문제가 생기는 것 같습니다. 따로 리스트 초기화 함수를 만들었습니다.
2. 심볼릭 상수를 사용하는게 좋아보이네요.


just a note:
lines above actually a `test_setup()`

then comes the test itself, and the `unlink` actually a `test_teardown()`...


merge


merge ??


And shouldn't it be `Map<UUID, ?>`?
That way we can only take from the map, not put.


I think a better fix would be to do `var i` somewhere above both loops. The loops are similar and at the same level so it makes sense for them to use the same loop variable.


I would rather rework this method to use std::vector instead of allocate/deallocate memorry manually


Hmm, you may be right.  My reason for making this change (which doesn't necessarily justify it) is that final instance variables have to be const *if* the class contains a const constructor, e.g.:

    class C {
      final y = f(); // (a)
      static f() => null;
      const C(); // (b)
    }

We already have code to generate an error at (b) (CONST_CONSTRUCTOR_WITH_FIELD_INITIALIZED_BY_NON_CONST), but it seemed to me that it would be nice to also report errors at (a) to help the user understand *why* the analyzer considers the field to be non-const.

So IIRC, this was a not-strictly-necessary change aimed at reducing user confusion.  The fact that it contributes to #25526 means that it probably wasn't worth it.  Assuming it doesn't break any tests, feel free to revert this line.


We've already started using it on a new project we're working on, and so far, I'm really loving it! You're correct, Atelier was bulit with Flux, as was the Organize App.


Again, the commit message doesn’t describe what the commit does but what ESLint found.


You can remove that from your tree, with mine i got missing file error, so i just add it to prevent that


Why empty assignment?


This is an attempt to free memory pointed to by m_body, which was already freed on line 3651.  Can you please describe the memory leak scenario which you were trying to fix?


Same thing about the commit message.


:smile:


how about set the ip to 127.0.0.1 ?


Hey! I don't see the playToX function... did you push it up?


:joy:  :ok_hand:


The commit message is inaccurate, there is also a `;` fix.


holding this state here does not work across processes. That's why MS minidrivers are using a global caching file (on HD or on the card itself)...


looks good if reloading parent user from keytab influences all the proxy users


is the comment helpful? if you can get rid of it the copy constructor, you should probably just do it to. otherwise, the comment might "rot"


forgot to remove
akka-http-core/src/test/scala/akka/http/impl/engine/ws/WebsocketIntegrationSpec.scala
akka-http-core/src/main/scala/akka/http/javadsl/model/ws/WebsocketRequest.scala


An empty array is immutable and we can have such a constant.


Sure thing. When I see big chunks of code commented out I assume it's "work-in-progress" material. Glad to see you went the extra mile! Woot! :fireworks:


closes #3


Use AssertionError


you don't cover the test , if the comment not found.


Thank you!


Again, empty arrays are immutable.


Most common cases at the top.


I'm ok with this for now - I think this type of thing would be better in an external framework.


If you want to use puma both in development and production, you could simply go for `gem 'puma'`.


What about getFieldValue?


Sorry aboot that


Thoose are only defaults. Custom configuration resources are specified in `updater.updateConfiguration(config)` which is called below https://github.com/Teradata/presto/commit/3a83c77f94bb05938364f8921896edafa2998473#diff-34186fd35f344d0f932d659be66e11c8R53


But you've basically neutered `switch_docker_storage` and hence invalidated the tests which were using it, so maybe it would be better to just comment out the test invocations, rather than the test internals?

Another approach would be to conditionalize on `if grep Fedora /etc/os-release`.


以下是js部分


Perhaps an approach like [this](https://github.com/magnusulf/MujLib/blob/master/src/main/java/dk/muj/mujlib/util/Mujtil.java#L86) could be used instead of the last line?


It is no harm to keep it there so let leave it there...


Isn't it compared to the result of `tok` elsewhere though?


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


OK


What happens if we get a eof in this case?


Bien!


Code changes look sensible.


Well, that was stupid. Figured out the reason your code didn't work.
Need to re-patch firmware everytime EmuNAND configuration changes.
Gotta fix it.


@mauroartizzu correggi l'inglese qui.


Blog post: http://steveoscar.github.io/2015/12/27/polymorphism-hmt.html


I would write "reference" explicitely here, and maybe link to a Git doc page
https://git-scm.com/book/en/v2/Git-Internals-Git-References


`sassStream` -> `babelStream`, `jsStream`, or `stream`? If there's only one stream per file, I think the prefix is probably unnecessary.


默认显示在项目首页上的md是大写README.md，改这个不显示在首页上的小写readme.md 干啥用啊？


PR #2050 #2049 and #2044 Are all related, and have to do with removing vestigial vendor prefix for box sizing.


This will need to be redone again, gah, THIS IS WHY WE BRANCH


initial


The test is still worth something. The bind mount will make docker think that it's starting out with an empty storage. The only difference is that rather than going from devmapper to overlayfs, it's going from devmapper to devmapper. From the point of view of the `export`/`import` commands, it's still being tested to the same extent.


Looking at it again I would prefer you patch configure.ac and call eautoreconf.


test

- [x] task 1
- [ ] task 2


and apparently i still don't understand.,. I guess it just undoes the changes that were made in a commit leaving the rest of the file unchanged????? I see how that would be useful, but how ITH do I just roll back to a prior commit?


@Spoonikle Think you can get on TS?