Yes, for API stability. The "internal" version (what code is written against) and serialized versions (v1 and v1beta3), with defined conversions between them.


This should have been submitted as a PR rather than linking to a commit.


Using the integrated F5 approach does not save network hops - it's just that the hop is through kernel space on the ramp node, rather than through user space on the router. Also with the built-in SDN, the ramp node is a single point of failure and so two ramp nodes with IP failover are normally recommended.
Not all organizations permit wildcard SSL certs, in which case pass-through is highly recommended (at least for non-prod) so application owners can easily manage their SSL certs.


why not initialize it with that on loading constants?


this is a really ugly side effect hack


아..그렇군요?


Je pense que pour être plus clair, on peut modifier légèrement ton texte qui est plus juste que le mien :+1:

> Autre cas d'utilisation : si nous voulons certifier que nous sommes l'expéditeur d'un message, nous pouvons créer une empreinte du contenu, et la chiffrer avec notre clé privée, c'est ce qu'on appellera la signature à proprement parler. Le destinataire pourra également de son coté générer un empreinte et la comparer à celle qu'il obtient en déchiffrant la signature. Si il y a correspondance entre les deux empreintes, le destinataire pourra être sûr de deux choses grâce à cette signature numérique : c'est notre clé privée qui a permis de générer l'empreinte chiffrée du message __et__ le contenu du message est le même que celui que nous avons envoyé (personne ne l'a altéré).


ack


this only affects the logging done at AsyncPeriodicWork. Every time the task runs it logs the start and end events


`s/of failure/on failure/`


Yes. This is consistent with current behavior.


Yep nevermind


It worked!


XSS vulnerability
https://wiki.jenkins-ci.org/display/JENKINS/Jelly+and+XSS+prevention


I meant, did you remember to add the GPIO config for USB to the 407 build?

I was thinking of reasons why USB might not work for you (It's the exact
same code you wrote, just chopped up, so it should work on 407)

On 16 February 2016 at 08:30, Daniele Lacamera <notifications@github.com>
wrote:

> GPIO config is indeed still in the machine part, still I don't think that
> having 650+ lines duplicated, which are not HW specific, is a good idea.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/insane-adding-machines/frosted/commit/b1be1e181204048674e26815052d04e59412a022#commitcomment-16122745>
> .
>

--
www.nicecupoftea.com
www.biscuit.com


you need the latest coala development after merging your PR right?


@Tatsh Hey, race condition in the argument parsing. cd72e


pyprint is needed by coala, not by the bears


unack


This can be delete no?


>. still I don't think that having 650+ lines duplicated, which are not HW
specific, is a good idea.

I'm open to suggestions. Don't forget tho, we need some clean way to config
endpoints and they appear in 2 places

On 16 February 2016 at 08:30, Daniele Lacamera <notifications@github.com>
wrote:

> GPIO config is indeed still in the machine part, still I don't think that
> having 650+ lines duplicated, which are not HW specific, is a good idea.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/insane-adding-machines/frosted/commit/b1be1e181204048674e26815052d04e59412a022#commitcomment-16122745>
> .
>

--
www.nicecupoftea.com
www.biscuit.com


fwiw we do want to make each bear an own pypi package,. right? So each bear gets an own requirements.,txt and stuff?


ack for now


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


ack


I got merging rights now.

I am busy for the next 4 days(I have 2 exams), right after that I will start going over issues and PRs.

In the mean time I will remove the notice to leave.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1533 of 1533 tests passed, 0 test warnings)

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


This should be a ConfigError


by default, the exit status of the script is the one of its last command.

so presumably, this change is not necessary?


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (984 of 984 tests passed, 0 test warnings)

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


ack, maybe we could do a coala-base pypi package that has custom general stuff, also e.g. the version number incrementation routine and so on?


Yes, I’ll put together a notebook and a readme file. The slowest part is the stats.linregress for the sliding window, so unfortunately much of the slowness is unavoidable. I did fix one thing though - I guess it’s much much slower to slice a pandas Series than a list or numpy array.

Right now I’m working out a better way to find lag time and saturation time. I hope to be done by tonight with all of this!

> On Feb 16, 2016, at 11:15 AM, Bryan Weinstein <notifications@github.com> wrote:
>
> Hi Nichole, looks like you have made some very useful changes. Do you think you could upload an ipython notebook, at some point, that shows how to use this code? Or some sample scripts?
>
> Also, have you fixed that problem where it takes ~1 second to analyze each well, as opposed to 1 second for like 400 wells?
>
> Regardless, looks great, very excited to see what you changed and how the plate layout file works.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/nwespe/OD_growth_finder/commit/1d668811d5a0fb97bbb4d6d09a26d0944656197e#commitcomment-16122256>.
>


typo embedded => embeds


why would you want to test `atexit()`?


I didn't finish reviewing, so I might be missing something ahead. However, as far as I remember, read/write bit is actually for register address, no?


unack, this is not coala, it's an own package, right?


rtd handling unneeded


Made fixes (Jazzmine pulls) and made code more verbose


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (397 of 397 tests passed, 0 test warnings)

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


get rid of Jazzmine's comments


here too


Hmm... are you sure want to check for such a specific status code instead of any `2xx`?

There's one or two endpoints where I chose 200 for PUT (and DELETE) since it wasn't actually creating (or deleting) anything but just manipulating an aspect of an existing resource.

Otherwise I feel like we should have some extra tests or maybe middleware to help enforce this consistently.


One of the reasons for explicit export is so that intelliJ can track its usage better.
This is useful when you have to find usage or renaming function.
But, Tatiana is right that this can cause name clashing when importing common function names.
For this reason, I've been using this syntax for util functions:
```
import * as TblUtil from '../TableUtil.js';
TblUtil.findById()
```

findById is exported directly.  But, I can still group it at import time.  intelliJ is smart enough to track it.
I know Trey mentioned that future babel will have a harder time optimizing this sort of import.
I'll keep this in mind when we start optimizing for production deployment.


Yes, but it still won't work until 5.2.17 which is not out yet.


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


This differs from punctuated watermark where we say "larger than previous and positive".


The following doc(s) have been deployed to the [workflowau](http://workflowau-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/using-the-pantheon-workflow](http://workflowau-static-docs.pantheon.io/docs/articles/sites/code/using-the-pantheon-workflow)
- [source/docs/assets/images/desk_images/376212.png](https://github.com/pantheon-systems/documentation/commit/d085e264c126bb4d287b5f084e1f201b3dd30284/source/docs/assets/images/desk_images/376212.png)
- [source/docs/assets/images/desk_images/376217.png](https://github.com/pantheon-systems/documentation/commit/d085e264c126bb4d287b5f084e1f201b3dd30284/source/docs/assets/images/desk_images/376217.png)


here you are doing some checking on string argument. We should define methods instead, that already have well typed arguments, that do not require any checking. Maybe one for fully qualified path names as a Seq[String] (so they don't need to be parsed with a regex).


Ok cool. Maybe you should make the linear regression optional; this way things can go fast!


@travijuu https://github.com/laravel/framework/issues/12309#issuecomment-184597667


A unique_function<void()&&> is a function that we (as the people storing it) are promising we will only call once. So it makes sense as the type for something holding a callback that might might notify condition variables and trigger processing and the like.


LoadComponentRefactor (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed, 0 test warnings)

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


Congratulations!


Ah ah indeed... so many subtleties...


@ehsanotelegram


This json parsing should be done inside of a method that fully encapsulates any dynamic typing or parsing stuff, but hides it behind has a well typed signature. I don't think we need a main method here at this point, except if it is for fun. In that case the main method should turn the command line arguments into well typed values and call the method. But more important would be tests, that call the well typed methods directly.


@severin-lemaignan oops! :-) Good catch, thanks for noticing! Fixed in https://github.com/ros-infrastructure/rep/commit/fd67edbf944d61c3c4b804005e0d63663b836bc2


Should this be `./commctl`?


I think those should really be LocalDateTime and Int ;). You should be able to convert between them.


looking at this i am a bit confused. i think on the one hand, it is defined that GeometryCollection *is* a Geometry object. on the other hand, in various places the text talks about "Geometry and GeometryCollection objects". isn't that a bit conflicting? personally, i still like the second "world model" better (#148 makes that point), but whatever the top-level model is, it should be consistent.


@yhahn: this line prevents `cfn-create` from being run with both the `-c` and `-f` options. The if statements you added below would appear to cover the case of `cfn-update -f` unless I misunderstand the functionality added here. I've run through a couple of `cfn-update -f` with and without the above line, and they both behave exactly the same.


We could do make_named_thread taking a string and then the rest of the arguments to thread::thread.

If we plan to do lots of stuff we might want to consider subclassing std::thread.


Is `loc` a local variable? Should there be a `var` to declare so.


Hi,
I'd disagree if async support is needed at all making this a real limitation.

I'm referring to the fact that Erlang/Elixir as a whole is in a way asynchronous - user's call to your api will be interleaved with other operations. That's what people need "async" - to make sure their synchronous wait for reply won't starve other operations (handlers, controllers, whatever they use). Elixir (and Erlang) make it possible without inventing a separate syntax (or api) for it.

Btw thanks for making this driver for datomic.


Exercício :+1:


could also put
```c++
  struct flock l = {
    .l_type = F_WRLCK,
    .l_whence = SEEK_SET,
    .l_start = 0,
    .l_len = 0
  };
```


These changes will not go into the official PR


no spaces in  "$maxRetryCount = 4,"


Good job, @mherweg


I didn't believe we needed it as querying for the status of an operation can sometimes take a few seconds. However, after talking with Larry, I agree that we need a delay here. I would make it 5 seconds and I would also suggest we log a message after the delay so we have some telemetry around this operation.


Hi @f1ames - Thanks for looking after the fix. Do you think there is any major risk involved ? When do you think it will be delivered on the master branch?


New-Object instead of new-object, to keep the PS guidlines


Can be changed to 10_001 also


> what do you mean? the - does a weighted diff

right, but I want `3 - 2 = 1`, not `1/2 x 3 - 1/2 x 2 = 0.5`. makes sense?


Sorry


hooo i like it


add indent to the lines inside ExecRetry


Why are you not using the ability to attach user object to each ClientInputFile to save the need to do this mapping?


space between if and expression


ok. Thanks for checking.


Drop this whole block. Decoda is a lua IDE, but a largely dead one that is probably not worth giving special attention to.


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2086 of 2086 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

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


Would love to help but I don't have publish access on this one for npm. I'll ping @cowboy tomorrow (he's out today)!


The following doc(s) have been deployed to the [drupaldrus](http://drupaldrus-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/drupal-drush-command-line-utility](http://drupaldrus-static-docs.pantheon.io/docs/articles/local/drupal-drush-command-line-utility)


How long does it take to run that test?


- [ ] ページではなく、節単位のリンクは出来ないか？


create_snapshot_blob or create_blob_snapshot?


Yeah this is wrong right now. I pushed partial changes.


Moving ST_Automation to new repo and renaming TBT_OnDemand to ST_OnDemand


Missing potential in setBoundaryLine call.
Needs to be setBoundaryLine(x1,y1,x2,y2, potential);


Minor nit pick, last parens should line up with BlobSnapshot.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1300.178a1a2e469369dc7be801d849db0d4107aa1ca0](http://build.palaso.org/viewLog.html?buildId=69198&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 972, ignored: 12 Build time: 00:07:02


Perhaps `new` might be a better verb here?  It reads better to me for `restart` and `upgrade`.


also #39


he didn't have


Peer IDs can collide or be spoofed.


Are you sure you don't mean six?


WFM. I'll pull this commit out and you can address it in your PR.


### [Codacy](https://www.codacy.com/app/balazs-micskey/php/commit?cid=27558425) detected an issue:
#### Message: `hello2 accesses the super-global variable $_GET.`

#### Occurred on:
+ **Commit**:  15fb4cfb81e1fd450d94b10ea8b387c74167d903
+ **File**:    [HelloWorld.php](https://github.com/bercy/php/blob/15fb4cfb81e1fd450d94b10ea8b387c74167d903/HelloWorld.php)
+ **LineNum**: [22](https://github.com/bercy/php/blob/15fb4cfb81e1fd450d94b10ea8b387c74167d903/HelloWorld.php#L22)
+ **Code**:    `    public function hello2($what = 'World', $bool = true)`

#### Currently on:
+ **Commit**:  15fb4cfb81e1fd450d94b10ea8b387c74167d903
+ **File**:    [HelloWorld.php](https://github.com/bercy/php/blob/15fb4cfb81e1fd450d94b10ea8b387c74167d903/HelloWorld.php)
+ **LineNum**: [22](https://github.com/bercy/php/blob/15fb4cfb81e1fd450d94b10ea8b387c74167d903/HelloWorld.php#L22)


辛苦 ZZ 菊苣~

这个种子标题栏高度好像还是存在一些问题：
左边是加载完成，右边是加载中，不知为何加载完后会高一些（
![qq 20160217010750](https://cloud.githubusercontent.com/assets/1846356/13084204/7364e9be-d513-11e5-965d-048ffcc095bb.png)


Work for #72


Changed :)


[ब्याट्री] n(पिटपाट, चुटचाट, मारपिट), n(तोपको समूह), n(विद्युत्‌कोषश्रेणी, बिजुली सञ्चय गर्ने यन्त्र)


initialise


looks like your branch creation worked!


I am thinking from the point of calling this python module from command line. In stead of checking if the input is a file (```os.path.isfile()```) I can use ```os.path.exists()```.


I doubted and opted for the easiest way to test (and then forgot to change it), now it's a numeric value backed by an enum


It should be. Fixing.


renamed a bit.


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


> Maybe some checks are missing from the ESLint linter. Do we want to keep this rule?

I don't have a strong opinion on this. But I find the rule more annoying than useful.


003.1 npm - pretask posttask/after/src/vs2015-TS/package.json

Here must be: "npm run task1 && node npm run task2 && npm run task3"


good catch


Cool, it should already work with a dummy camera over IB, right? What do I need to do on the server to make it work?


Thanks! I just saw an email from him that he is in the process of moving and will have internet access shortly.


shell; [शेल] n(शंखे किरो #animal), n(छिल्का, खोल, बोक्रा)


Thinking about it. the config/interface (and above) descriptors could go in
the same file as driver code (ie cdc_ecm.c and cdc_acm.c etc etc). Then all
the endpoint refereneces would be in the same file.

The code should work across multiple CM3 HWs (assuming locm3's USB code
actually works on multiple CM3 HWs :) ). cdc_ecm.c and cdc_acm.c should
have no F4 specific code inside

On 16 February 2016 at 08:30, Daniele Lacamera <notifications@github.com>
wrote:

> GPIO config is indeed still in the machine part, still I don't think that
> having 650+ lines duplicated, which are not HW specific, is a good idea.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/insane-adding-machines/frosted/commit/b1be1e181204048674e26815052d04e59412a022#commitcomment-16122745>
> .
>

--
www.nicecupoftea.com
www.biscuit.com


It's a cool test... Another variation would be to do the same but use the gap for when the next invoice is supposed to be generated.


Da ist noch ein Rechtschreibfehler drin


here, some more :)
https://github.com/dorimanx/DORIMANX_LG_STOCK_LP_KERNEL/commit/0460f9d0f8d9042362c5c8bf68a5f90fb4fe95e5


Looks great!  I've been off in Python-land for long enough now that I don't remember how namespace imports work in Rust, but I'm sure they are expressive enough that this makes sense as they are disambiguated by the fact that they all live in the rust-osc package.


- [ ] @Cacheableは物理名なので ``インライン化`` してください。


There are two things that I think it could be different in this implementation:

First it could have zero input ports and receive 2 pin values through options and deal with 2 gpio readers internally, it would fit the most common usage way better, but I admit that would kill the usage by "soft shafts" so I think this is option.

Second, and more important,  you should keep the sampling count and output it's value corrected by the spinning direction, this is by far the most value information from the quadrature encoder since you usually knows to each direction you are commanding your motor. Ok, it can be used by shats that you don't control but track instead, but still a very valuable information in those situation. You don't need to try to convert in angles or anything because you would have to add tons of options just for this, but you can just output the raw count so the user can do that math that fits its own encoder and this node becomes more useful.


jQuery anstatt  $


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (397 of 397 tests passed, 0 test warnings)

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


put a return; there


Hat sich nur die Position des Eintrags geändert, also unrelated change


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=10)


- [ ] 直上のTipとまとめて１つのTipにしてほしいです。Tipの嵐は良くない。


Sorry, the InfiniBand parts are not yet ported to windows. So nothing to test that includes IB _yet_. @matze's uca-net basically replaces the need for TANGO. But so far, it only uses standard network sockets. As soon as Sai has ported uca-net to windows as well, we should have a workable base-line for your next beamtime. InfiniBand support for uca-net will come next.


Hrm. That's a good point. I'll generalize it for now for any ```2xx```. We can revisit later if needed.


ack


Wow, that was really a dirty check. I can see how I missed it. :)


Im concerned that the container status "Running" can change underneath us, so I rather return the status codes as per the LXD spec.


It's a single commit change so it works just like a PR. As for multiple PRs for each story, yes, we'll use that as much as possible but I'm worried about the cost of merging the changes in the parent (or previous PR) if multiple branches are used.


^ true that


I believe this used to be taken care of by commit 8e23b39221158245a85d6afd8ec45b67fd169529


Thanks, that was quick. (I didn't see the ML post until after I wrote this.)


Can we remove this line? If there's nothing that will be making requests to mapit on port 80 it's not required :)


Oh: "npm run task1 && npm run task2 && npm run task3" <== this a correct


@benhutton yeah, I agree. `ShareCounter` is a wrapper. I created the wrapper to maintain consistency with the google analytics code, and because I _think_ I was violating Demeter's 'law' previously.

I like the idea of creating one `Wakes::SocialMetricsWrapper`. Are you good with me doing that for this feature?


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=11)


I also did not verify that this is just a copy, and LGTM assuming it is.


This should be the state of the deployment. Is there a reason we are no longer checking that? The deployment can fail before the operations are even started. We can't rely just on the operations for the state of the deployment


Actually, it doesn't fully act like a PR
1. PRs result in notifications for anyone watching this repo
2. PRs add in the changes that you make in response to the comments
3. PRs add a conversation tab to more easily track the discussions about the changes.


Odd.

Message didn't pass through.

This commit message was suppose to denote that tasks now show the full stacktrace when they encounter an error (previously only showed the Lua stacktrace to the console), and that a new function was added to the scheduler metatable to allow for easy generation of tasks as opposed to constructing a task yourself. Change in code shown below.

```lua
local task = {}
task.name = "moveafrisk"
task.process = function(self, delta, input)
    -- move a frisk
end

scheduler:registerTask(task, true)
```
to

```lua
scheduler:generateTask("moveafrisk", function(self, delta, input)
end) -- automagically constructs the task and registers it, returns the id
```

Something I forgot to do was add the option of setting the active state of the task in generation; it defaults to true internally. Fixed in a later commit.


LGTM.


The changes look great to me! Nice and clean :)


Oh, this is great!


FileNotFoundException наследуется от IOException - 2 catch излишне


Yeah, that's correct. I'll comment this.


I know it might be obvious but doc string would be a good idea here?


and here


The following doc(s) have been deployed to the [drupaldrus](http://drupaldrus-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/drupal-drush-command-line-utility](http://drupaldrus-static-docs.pantheon.io/docs/articles/local/drupal-drush-command-line-utility)


Perhaps my problem was I failed to complete the edits3 changes with a comment. Here is my comment.


`var chunk = stream.read(Number.MAX_SAFE_INTEGER);`
should be:
`var chunk = stream.read(9007199254740991);`

If you want this performance fix to work for IE.


- [ ] が無いようにに、 -> が無いように、


No further comment.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=12)


In the commit message, I meant InlinePattern="" NOT InlineSetting=""

-Brian


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


I am thoroughly confused now.


loading docstrings from a separate file like you're doing here is rather unusual. There's work in progress to make it easier to go the other way around, easily autogenerate markdown files with just the docstring content.


I dont think description is required and since its an empty string best remove it?


Старайся выводить в консоль не просто stacktrace, но и какое-то сообщение.
Другие разработчики и пользователи скажут спасибо, да и при чтении кода будет понятно что случилось (по сути сообщение выступит комментарием)


Nevermind. I see it, we set this to the state of the deployment after the first pass through


This looks like it should work, let's get it imported.


ich hab mit sasa gesprochen er sagte die so.62 wurde nicht angepast sondern original und die haben wir ja nun auch im image , im azplay also hat die nichts zu suchen die libpng gibt es ja auch also beide dürfen da nicht rein, keine ahnung ob man das plugin benötigt oder nicht, ich hab keine azbox mehr die geht alles defekt, kann also selber gar nicht mehr testen


+1 on not having `libzerocash/libzerocash` anymore. I didn't carefully check that all the `#include` were corrected (if needed) to work with the new paths for all of those files, but it builds I suppose they must be.


LGTM


Быть может стоит хранить части имени отдельно в таком случае?


Ah, missed that. I'm not currently doing any character level mirroring either.


The commit message should read:
"Ignore modifier keys at menu. Improve comments"

It's more accurate.


We should probably still check the deployment state at the beginning of the loop and notify the user if the overall deployment failed.


Correct me if I'm wrong, but wouldn't this fail to add the correct line if the addpaths file already exists?


- [ ] 「文脈に依存したチェック」の意味が分かりづらい。


Yes, remove them :)


@gaganawhad that then brings about the question: is `Wakes::SocialMetricsWrapper` best, or should it be `Wakes::FacebookWrapper`? You know, SRP and all.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=13)


Well, it is to me.  `if true ? bar : foo | bar` is trivial as well as readable in my opinion.


As I say in the last pull request you've made, 0x02 is the same as IGMP_PROTOCOL_NUMBER,
so this line should not be in the if.


where is the option "CTRNAND PADGen 0x4"? I only see "CTRNAND PADGen"


The .tcc are probably needed here, otherwise it won't rebuild when they're changed, right?


this should be outside the condition too.


Ты используешь VCS - а значит всегда сможешь вернуть предыдущий код.
Не бойся удалять код, и уж тем более не стоит его комментировать.
Лучший код - удалённый код.


operation == 0x16


I mean, they either need to be in LIBZEROCASH_H or here. I'm not sure which is right.


Empty else?


else if (operation == 0x17)


Using '.' in shell variable names is not possible. We should use RAUC_* as names.


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


Yes, let's delete commented the lines


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=14)


definitely the best idea


No https for this one?


:beers:


Other than the comments I wrote, LGTM.


and so here should be an else to handle unsupported operations


Change to `_onCloseFullscreen`.  Please pay attention to my comments, it's time-consuming to have to repeat same thing three times! ;)


Okay great, comments are removed. :)


I've found this to be too long as well. Did 500ms work for you? If so, submit a pull request and I'll merge it.


Makes perfect sense, thanks for the clarification.


I think this is not the behavior we want.  We want `everything` cleaned out, then rebuilt for the new user.  So config_base should disappear too.


"o" lala ;-)


Your formula is just going to compute a number very close to 1 for every depth.
At depth = 1:  .999999993
At depth = 20  .99888


doc string?


Можно сделать наследника класса JFrame и перенести туда все внутренние элементы.
Добавить ему интерфейс для "общения" с другими формами. Тогда код отдельных форма будет в отдельных классах - проще будет ориентироваться и изменять его.


fixed


Remove cumin


You have to remove this function - because it _overrides_ the `_listen` function in Wu.Chrome.

There can NOT be a `_listen()` function in chrome.users.js.


Treat overflow (leave it unsigned might help)


this should be outside the if too.
`hdr->igmp.type = operation;`


:+1: thanks @forresst


- [ ] 「～実行時例外を発生させる設計としている。」


thanks man, works great!


Try putting slot0x11key96.bin on the root of your SD card.


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


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


LGTM


The following doc(s) have been deployed to the [drupaldrus](http://drupaldrus-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/drupal-drush-command-line-utility](http://drupaldrus-static-docs.pantheon.io/docs/articles/local/drupal-drush-command-line-utility)


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


You're welcome ^_^


well, if your logic is : action_parentobj_subobj (or something like that) I agree with you


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=15)


TeamCity Game Of Life :: Build [Build 82](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=84&buildTypeId=GameOfLife_Build) is now running


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


TeamCity Game Of Life :: Build [Build 82](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=84&buildTypeId=GameOfLife_Build) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 58, ignored: 1 Build time: 00:00:20
### Failed tests
```
com.wakaleo.gameoflife.webtests.controllers.WhenSpawningANewGeneration.theIndexUrlShouldDisplayTheHomePage: <no details avaliable>

```


I have the file in the root but nothing (new3ds , cakes, rxtool and  reinand works fine, sys 9.0, emu 10.5


You should use the placeholder, eg ${jackson2.version}


fixed


There should be no need to specify `fee(f)`, if `f` is the base fee. That value will get used by default.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=16)


accepted.


Witam
będziesz  kontynuować  prace nad ta wtyczka po porzuceniu ja przez Neopacka ??


TeamCity Game Of Life :: Build [Build 83](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=85&buildTypeId=GameOfLife_Build) is now running


Try using https://github.com/Shadowtrance/Decrypt9/releases


The fill_by_reference/fill_by_index structs need to be aware of the data they are working on, so currently a reference to the original vector is required.


The addition of "or --file" should have made it into the previous commit instead. However, I'm planning on rebasing then squashing, unless you advise otherwise, after you've reviewed this, so it won't matter if this change is in the wrong commit.


changed word wrapping.


TeamCity Game Of Life :: Build [Build 83](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=85&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:33


TeamCity Game Of Life :: Build [Build 84](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=86&buildTypeId=GameOfLife_Build) is now running


I try this app too...


Done: https://github.com/cms-sw/cmssw/pull/13306


TeamCity Game Of Life :: Build [Build 84](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=86&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:33


sudo_users =  {46621748} ,--Sudo users


The following doc(s) have been deployed to the [drupaldrus](http://drupaldrus-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/drupal-drush-command-line-utility](http://drupaldrus-static-docs.pantheon.io/docs/articles/local/drupal-drush-command-line-utility)


TeamCity Game Of Life :: Build [Build 85](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=87&buildTypeId=GameOfLife_Build) is now running


sudo_users = {46621748} ,--Sudo users


TeamCity Game Of Life :: Build [Build 85](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=87&buildTypeId=GameOfLife_Build) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 58, ignored: 1 Build time: 00:00:21
### Failed tests
```
com.wakaleo.gameoflife.webtests.controllers.WhenSpawningANewGeneration.theIndexUrlShouldDisplayTheHomePage: <no details avaliable>

```


TeamCity Game Of Life :: Build [Build 86](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=88&buildTypeId=GameOfLife_Build) is now running


It is unclear whether the first argument is a JSON file, a JSON string or both.

Also unclear what are "everything else".   Users who are unfamiliar with Vega wouldn't know where is vg2png.  (Even I do not know what are these without looking up.)  Better describe what are all the parameters here.


Danke Captain. Ich teste mal ob es ohne extra libpng funzt. Beim localen bauen hatte ich nur die so.62 rausgenommen und es gab keinen fehler, ich teste mal symlink auf die existierende version.

plugin war frueher mal dabei, aber die mrua libs hatten gefehlt (und hat dann eben gefunzt) war nuetzlich als gst nicht so richtig ging. Ich hatte das BB rezept gefixt, aber ob es aktiv genutzt wird weiss ich auch nicht -  ich poste mal im azbox forum ob es rein soll oder raus kann. die mrua libs brauchen am meisten platz, ohne waere fuer die alten boxen besser...


TeamCity Game Of Life :: Build [Build 86](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=88&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:33


TeamCity Game Of Life :: Build [Build 87](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=89&buildTypeId=GameOfLife_Build) is now running


@benjello @cbenz @fpagnoux
Je n'ai pas trouvé mieux que d'ajouter ces deux fonctions.
Elles sont utilisées là :
```python
contrat_de_travail_fin > shift_date(contrat_de_travail_debut, 6, 'month')
```
La variable contrat_de_travail_debut est un datetime64 numpy de type jour, et je dois lui ajouter 6 mois pour le comparer à contrat_de_travail_fin, ce que je n'arrive pas à faire sans les transformer en `instance` OpenFisca pour profitier de la méthode `offset`.
Si vous avez des suggestions n'hésitez pas !


TeamCity Game Of Life :: Build [Build 87](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=89&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:32


It's not specifically for WiFi, although the problem does occur there - it's to mitigate the case where the caller is blocked and unable to perform a disconnect while a previous connect is still underway. This happened when pressing the button to enter listening mode - the LED would be off until radio connect had completed. It's also happening in code when applications are trying to enter listening mode but are prevented because the caller is blocked waiting for the previous connect to complete.  The same is true for disconnect()/off() - it should be immediate rather than waiting for the connect to complete.


Uppercase here?


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


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=17)


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


Uppercase?


Lets adapt this to be "Don't mention `nodecg`. Some of the existing items probably break this but its because i copied their descriptions from the actual repo.


OK


Irrelevant here I think


ok. so this PR only miss documentation.


public class findMinNumber {

	public static void main(String[] args)
	{
		int k = 34343434, n = 53, m = 32434;
		System.out.println("Min is " + min(k, n, m));
	}
		public static int min(int a, int b, int c)
		{
			int res;
			if (a < b && a < c){
				res = a;
			}else if (c < b && b < a){
				res = c;
			}else if (b < c && b < a){
				res = b;
			}else{
				res = 0;
				System.out.println("Use diffrent number!");
			}
			return res;

			}

	}

//public class MinimThree {
//	/* Минимум трех чисел
//	 Написать функцию, которая вычисляет минимум из трёх чисел.
//	 */
//
//	 public static void main(String[] args)
//	 {
//	 int k = 2, n = 23, m = 1;
//	 System.out.println("Min is " + min(k, n, m));
//	 }
//	     public static int min(int a, int b, int c)
//	     {
//	         int res;
//	         if(a < b && a < c){
//	         res = a;
//	         }else if(c < b && c < a){
//	         res = c;
//	         }else if(b < c && b < a){
//	         res = b;
//	         }else{
//	        	 res = 0;
//	        	 System.out.println("Use different numbers!");
//	         }
//	         return res;
//	     }
//	 }


This is interesting. I will test it later, and merge it in. We should probably add it to the EE versions and do a separate PR that can be merged back up to the official git repo on this.


Whoops, how did that happen. I'm surprised it builds.


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


leaks `data->friendly_name`


I don't know.


Should we remove $ prefix from register name?


TeamCity Game Of Life :: Build [Build 88](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=90&buildTypeId=GameOfLife_Build) is now running


same.


public class findDifferBet2Numbers {

	public static void main(String[] args) {
		int k = 34, n = 53;
		System.out.println("Min is " + min(k, n));
		{
		public static int min(int a, int b,)
		{
			int res;
			if (a < b){
				res = a;}
			else if (b < a){
				res = b;
			}else{
				res = 0;
				System.out.println("Use diffrent number!");
			}
			return res;
		}
	}

	}


TeamCity Game Of Life :: Build [Build 88](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=90&buildTypeId=GameOfLife_Build) outcome was **FAILURE**
Summary: Tests passed: 52, ignored: 1; compilation error: Compiler (new) Build time: 00:00:19


Witam
Będziesz kontynuować prace nad ta wtyczka po porzuceniu ja przez Neopacka ??


- [ ] ログイン画面表示のメソッドはGETのままでよいと思いますが・・。


Too funny. I just now ran into `fatal error: 'omp.h' file not found`, and see that the most recent commit addresses my issue. Is there a new release planned with support for `clang` and Mac OS?


From Wikipedia:
```
Coding for
clockwise rotation
Phase	A	B
1	0	0
2	0	1
3	1	1
4	1	0
```
My doubt is: you are at <0, 0>, shaft is rotating clockwise, so next would be <0, 1>, then encoder_process would be called:
if (!priv->last_a && priv->input_a) would fail and you would return 0 instead of 1, isn't it?


TeamCity Game Of Life :: Build [Build 89](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=91&buildTypeId=GameOfLife_Build) is now running


TeamCity Game Of Life :: Build [Build 89](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=91&buildTypeId=GameOfLife_Build) outcome was **FAILURE**
Summary: Tests passed: 52, ignored: 1; compilation error: Compiler (new) Build time: 00:00:18


factory


Perhaps, `[GS]etEffect` could be more efficient if the `Paragraph.Effect` member would be of type `Dictionary<string, string>` ?


This scenario ended up being removed as a duplicate in a later commit, so the issue is no longer relevant.


What is this commit?


If you look at the code:
for (int i = 1; i < movesToGo; ++i)
        otherMovesImportance += move_importance(ply + 2 * i);

where movesToGo goes upto 50, we actually uses this formula up to ply + 100.

Second, currently used logistic formula is almost equal to 1 up to depth 40.

I'm drawing this in Excel, don't worry.


Lowercase here?


-1 arg not the direct object (remove the descriptor that has this keyword, not remove the keyword)


We plan to have a 1.1 release by the end of this month which will include this fix. However, building on clang will remove most of the threading in Canu as it relies on OpenMP which clang on OS X still doesn't support.


Projekt mam normalny, niedługo będę wrzucał.


I googled and apparently it was introduced in Emacs 24.4


arg not the direct object


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1536 of 1536 tests passed, 0 test warnings)

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


Didn't know i can export methods


This fixes a regression which happened while working on #207.


very interesting... the logs on my end also point to cassandra as the problem. the recurring error reads `Cassandra error: All hosts tried for query failed.` i'm assuming it's just bad config on my end. i'll let you know if i have some breakthrough that resolves it. it's great to have a fallback for some common errors, but the idea of a production kong instance throwing mysterious errors (and slowing down the response time) makes me nervous. thanks for getting back to me!


I now get:

```
src\GHC.hs:29:57:
    Ambiguous occurrence `parallel'
    It could refer to either `GHC.parallel', defined at src\GHC.hs:76:1
                          or `Base.parallel',
                             imported from `Base' at src\GHC.hs:14:1-11
                             (and originally defined in `shake-0.15.5:Development.Shake.Core')
```


The 'value()' function does not have a version that returns reference. We can keep the indexing operator.

`  return (*GetCachedParameterInfos())[type];`

I tried this commit to avoid allocation on the heap, but it crashes:

NifTK@d8982e9f384a


Well - my thinking was that at this point (if the directory exists but not the selection file) then they might still need to re-download because of updates to the parametric fits files.  So it seemed a bit more complicated than this now that there are quite a few updates.

An alternative would be to more carefully check what files exist, in what versions - do you think that's worth doing?  (selectively downloading the selection file, and/or the parametric fits file)


These should really be unarchiveObjectFrom()


Not for incremental, no, good catch; I can add one or two of those back in.


Hm, good point. I don't think it's possible :)


@BianCionaa Wil je deze vertaling zelf updaten naar 1.0.0-RC1, of ben je er toevallig al mee bezig? Anders neem ik er misschien even de tijd voor, er zijn nogal wat veranderingen zo te zien.


Sure, we can do that.


arg not direct object (self is)


Identify != deanonymize, since the fact that one unknown party has identified a person doesn't stop them being anonymous, at least in common usage.
"Tracking" was used for physically following people long before it was applied to the web.


Since CIMG is mostly used for loading saving the images, I think there is not gain to use CIMG with OpenMP support... To be confirmed...


```
if (Unit* caster = GetCaster())
   if (Creature* creature = caster->ToCreature())
   {
   }
```


To more clearly answer your question: yes, I knew I was doing that and it was intentional.

The above comment explains my reasoning (that I felt that that check did not fully address the circumstances in which a re-download might be needed, and that I thought it was sufficiently complicated that we might not want to try to carefully handle all cases).


Hi @fkoester would you like to take a look to the branch that im working now (multiple file selection) https://github.com/joni2back/angular-filemanager/tree/click-changes ? :smiley:


implied verb is 'get', which makes the direct object self


I tried getting this to work. I'll explain the issue and if you have an idea I'm happy to run with it.

Virtus, not Representable, is what we need to work with to make this happen. The calls to `attribute` at the top of the class create Virtus Attributes for the class. They also create the attribute writer methods `Story#name=` that are implemented to perform the coercion and dirty tracking. In the case of the collection attributes `attribute :labels, Array[Label]` you can implement a class like `LabelCollection` that inherits from Array and overrides the `<<` operator or wraps an internal Array and implements the `<<` operator. Either way, that is just allows you to create a Label object from a Hash or String like I do above in `add_label`. I can't get to the `attribute_dirty!` method from in the `<<` implementation.

https://github.com/solnic/virtus#important-note-about-member-coercions


Well, it does notice changes on an nfs mount then (probably contingent upon something else, but certainly not watchman).


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


Argh, I forgot that `Way.parallel` wasn't the only conflict and I don't see the warning on my machine. Sorry about that. Let me revert this.

@ndmitchell By the way, is next Shake coming soon?


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


implied verb is 'get', direct object is self


query


Yes, but not everyone sets up their projects the same and might not pay full attention.  So why not just add the ignores and same the trouble of having to weed them out by hand?


`_create_command`'s `**kwargs` are not used anywhere. Did you maybe miss to implement the `**kwargs` forwarding inside `lint()`?


Should we keep this chunk for PR?


grep output:

grep -r openmp *
Binary file binariesWin-Linux/mac_osx_fat/cmvs matches
Binary file binariesWin-Linux/mac_osx_fat/pmvs2 matches
Binary file binariesWin-Linux/Win32-VS2010/cmvs.exe matches
Binary file binariesWin-Linux/Win32-VS2010/pmvs2.exe matches
Binary file binariesWin-Linux/Win64-VS2010/cmvs.exe matches
Binary file binariesWin-Linux/Win64-VS2010/pmvs2.exe matches
program/thirdParty/nlopt-2.4.2/ltmain.sh:      |-threads|-fopenmp|-openmp|-mp|-xopenmp|-omp|-qsmp=*)
program/thirdParty/nlopt-2.4.2/ltmain.sh:        |-threads|-fopenmp|-openmp|-mp|-xopenmp|-omp|-qsmp=*)
program/thirdParty/cimg/CImg.h:// (http://www.openmp.org)
program/thirdParty/cimg/CImg.h:// Define 'cimg_use_openmp' to enable OpenMP support.
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:      std::fprintf(cimg::output(),"  > Using OpenMP:           %s%-13s%s %s('cimg_use_openmp' %s)%s\n",
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#if cimg_use_openmp
program/thirdParty/cimg/CImg.h:#if cimg_use_openmp
program/thirdParty/cimg/CImg.h:#if cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/cimg/CImg.h:#ifdef cimg_use_openmp
program/thirdParty/Eigen/CMakeLists.txt:  check_cxx_compiler_flag("-fopenmp" COMPILER_SUPPORT_OPENMP)
program/thirdParty/Eigen/CMakeLists.txt:      set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fopenmp")
program/thirdParty/Eigen/CMakeLists.txt:  check_cxx_compiler_flag("/openmp" COMPILER_SUPPORT_OPENMP)
program/thirdParty/Eigen/CMakeLists.txt:      set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /openmp")
program/thirdParty/Eigen/Eigen/src/Core/products/Parallelizer.h:  // FIXME omp_get_num_threads()>1 only works for openmp, what if the user does not use openmp?
program/thirdParty/miniBoost/boost/config/requires_threads.hpp:#   error "Compiler threading support is not turned on. Please set the correct command line options for threading: -openmp"
Readme.txt: => That's all. Openmp is not activated yet. Add openmp in the cmvs link option and define the _OPENMP cxx flags


Soonish... There's a very long list of things to do, and a very short pile of time to do it. Probably not until after we submit a paper.


![skeptical black kid](https://cloud.githubusercontent.com/assets/163977/13085496/86fa71ae-d4ac-11e5-8d59-ee36b3fc03a2.jpg)

I think I see what you're doing, I suppose there's still a lot more coming to reconcile `.data` and the data inside the search corresponding groups.

I'm also having a hard time understanding the `SearchPartitioner` and what looks like mutability and possible concurrence issues, but I trust you more than anyone with code.


This makes kick for server full not cancelable :/


TeamCity Game Of Life :: Build [Build 90](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=92&buildTypeId=GameOfLife_Build) is now running


factory


Maybe it would be good to check for Errors...

try:
    name = os.environ["PLUMA_CURRENT_DOCUMENT_NAME"]
except KeyError:
    exit(1)

f, ext = os.path.splitext(name)

if os.path.exists(name):
   if ext in ['.c', '.h']:
      subprocess.call(['/usr/bin/pluma', name])


Can we remove the app/mailers folder too then?


operator stats won't be up to date (I think more for upstream operators) here. I have noticed that this might be a bit confusing for users when they write non selective filter and they observe that the number of output rows was 0 but they see x rows in actual output.


TeamCity Game Of Life :: Build [Build 90](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=92&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:32


TeamCity Game Of Life :: Build [Build 91](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=93&buildTypeId=GameOfLife_Build) is now running


can we skip all deprecated methods?


I have an improvement for NovaGuilds. I dont know how many people are
writing you, but in my opinion a skilltree would we very nice. With some
abilities and ranks. Like an RPG plugin. I never found a good plugin that
supports guilds cities and RPG.

2016-01-26 10:25 GMT+01:00 Marcin Wieczorek <notifications@github.com>:

> Haha, I would if I were German!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MarcinWieczorek/NovaGuilds/commit/67affb869ada87cd9e7256393e1bf913b44821ee#commitcomment-15671893>
> .
>


Changed a bit this part (you will see in the next commit). And no this is not TODO, these should describe the functions of the app. That this is currently the same with TODO have the background that we are in early development phase. @phikal


Don't you maybe just return a string?


There is still a problem with this code:

simple repro:

$BlockSize = Get-Process -id $pid  | select ws
     $BlockSize -gt 0

> Cannot compare "@{WS=441434112}" to "0" because the objects are not the same type or the object "@{WS=441434112}" does not implement "IComparable".

Fix would be:

$BlockSize = Get-Process -id $pid  | foreach ws
     $BlockSize -gt 0

> True

So line 199 should be updated
from:
    $BlockSize = Get-WmiObject -Query "SELECT BlockSize from Win32_Volume WHERE DriveLetter = '$($DriveLetter):'" -ErrorAction SilentlyContinue  | select BlockSize

To:
    $BlockSize = Get-WmiObject -Query "SELECT BlockSize from Win32_Volume WHERE DriveLetter = '$($DriveLetter):'" -ErrorAction SilentlyContinue  | foreach BlockSize


TeamCity Game Of Life :: Build [Build 91](https://wipro.codefactori.com/teamcity/viewLog.html?buildId=93&buildTypeId=GameOfLife_Build) outcome was **SUCCESS**
Summary: Tests passed: 59, ignored: 1 Build time: 00:00:31


I guess I need to understand at which point the `SearchPartitioner` can be updated, and if this is something that is allowed as search results are coming in.


Thanks - the commit comment said the test was changed, but not why :smile:


Btw please return `None` or so, since an empty list could mean an empty temporary file^^


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (2092 of 2092 tests passed, 6 test warnings)

 * 3 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

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


unack


done :)


Removed


@ansis Why was there no pull request for this?

I'm still feeling pretty :-1: on the name `getSourceTileData`. We should discuss before this goes out in a release.


That was adding the new android support library


I've added more explanation, to make the issue clearer. The DatomicGenServer has a 1-1 relationship with the Datomic peer that it starts; peers are relatively expensive and the Datomic documentation does not envision creating large numbers of them. Requests single-thread through the peer and through the DatomicGenServer; calls to `handle_call` will block the caller, and responses to `handle_cast` are discarded. Thus a long-running query that does not take advantage of `transact-async` in Datomic blocks all access to the peer from all processes, and since the number of peers is limited, this could be a serious bottleneck even in an Erlang/Elixir system.


I think we should at least check if there is no selection file and redownload a new tarball.  The way you have it here, I'm pretty sure if they have the old version downloaded and unpacked already, it will imply that they don't need to re-download it.

I think the best option would be to check if the selection file exists, and if not then download both the selection file and the new parametric file (since that's probably also obsolete).  If you agree, I'd be happy to make that change.


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


I think better to use -s here and just get rid of the shorthand for save, so it would require `--save` rather than allowing `-s` as an equivalent.


OK. If the issue is not resolved, we can leave out again. No problem.


Fixed now: https://github.com/snowleopard/shaking-up-ghc/commit/89c79cdb52a9feeb949148348afffdd8cc150450


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


@bgerstle hmm… everyone uses the same IDE and gets the same warnings, and this is still the recommended practice for using primitives on iOS. So unless something weird is going on there should be a solution that follows the best practice and does not cause warnings/bugs/crashes. Really 1 of the 3 situations below should be true:

The warning is…
1. Valid and the suggested cast actually has no platform issues. So we should just use it.
2. Valid but the suggestion is incorrect and another cast/string specifier should be used.
3. Invalid and should just be suppressed using clang pop/push.

A quick look on stack overflow seems to suggest that the suggested cast is fine: http://stackoverflow.com/a/22671499/48311
http://stackoverflow.com/a/22671537/48311

But if you are skeptical, then there are also suggestions for using platform independent ones:
http://stackoverflow.com/a/26259782/48311


Noted. Thanks, Sergey.


I think you could inline `self.generate_config_file()` here, one line less^^


#James stop messing with my stuff


should be `Type[Enum[*strings]]`


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1536 of 1536 tests passed, 0 test warnings)

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


@markuz, these if branches are a bit confusing. You could achieve the same thing with something like:

```python
count = self._get_count(msg)
return minimum <= count <= (maximum or float("inf"))
```
Same thing goes for `pdf_image_count` and `pdf_pixel_coverage`


Thanks for your notes! I agree Getting Started requires more work!


Confirmed, thanks.


Can't you use the given file handler for writing to stream?


Ah. This commit, linked from your comment at https://github.com/mapbox/mapbox-gl-js/issues/2106#issuecomment-183541484, lost its association with #2052 because of a rebase. That threw me a little.


Should root_vol.iops also be vol.iops?


LoadComponentRefactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed, 0 test warnings)

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


The filename should be /etc/neutron/neutron_lbaas.conf, with a lowercase 'l'.


LoadComponentRefactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


It's nuts that Apple clang still doesn't support OpenMP. https://clang-omp.github.io/ reports that "OpenMP 3.1 is fully supported in clang/llvm 3.7", but it still hasn't made it into Apple's clang.


You don't need the `if` portion.


LoadComponentRefactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Drat. I think GitHub's commenting mechanism will get confused if I regenerate the diff; shall we just skip them?


LoadComponentRefactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Why not do
'''js
function getByGroup(brandName){
  return _groupColors([brandName] || [];
};
'''
?


Haha beautiful, you got a star


:smile_cat: You should make this more publicly visible on spacy.io


Very cool.


Melhor que one-liner é one-character! :wink: hahaha


Why not include this in the if clause just below it?


I thought that used to be taken care of. I probably lost it during the `Populator` refactor. I'll bring that back and drop this.


There's a lot of very long lines in this file. Not just from this PR. Do we have a style guide for this? It doesn't matter really, any mark-down viewer will wrap them anyway.


one a for Data (at the end)


Fake CLA user. @liach


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


Yang ini bisa ya cuma `product.variations` doang tanpa pake `all`? kalau bisa sih gak masalah.


@bgerstle hmm… yeah thats a bit confusing. nonnull is ambiguous.

Really I think the fact you are specifying the class you will accept precludes the event that you would be returned an NSNull, correct?

so maybe in this case `wmf_valueOfType:` is sufficient and removes the ambiguity.

Also its not really a type… its a class (you can't pass a protocol or a primitive)


я не совсем понял, что ты имел ввиду. Если флаг false, то отзывы дорисовываются..


Rob, I can't parse your sentence "It's because mu' with 0".


I'd... these "TODO"s will be handy during our optimization loop.


:)

`.onError()` maybe? or not necessary


There should be no need to check this beforehand


should `.lein-env.sample` be in the source?


Same here - there should be no need to check this beforehand


I guess there is a mistake in readme now. You have replaced Facade alias line with service provider class reference.


also `.onCancel()` or maybe do `onFinished(long token, byte reason)`


spec puts it as the last bit of the address... that's why most devices put it in the MSB of the register:

http://www.i2c-bus.org/addressing/


https://github.com/mojotech/modernator/commit/1621f00195da457bdb5bda5afad78010e192fc0c#diff-04c6e90faac2675aa89e2176d2eec7d8R17 will need to change too, right?


looks like there is trailing space to remove


:+1: this is perfect


The following doc(s) have been deployed to the [redisaudit](http://redisaudit-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/drupal8](http://redisaudit-static-docs.pantheon.io/docs/articles/drupal/drupal8)
- [docs/articles/getting-started](http://redisaudit-static-docs.pantheon.io/docs/articles/getting-started)
- [docs/articles/sites/code/reading-pantheon-environment-configuration](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/code/reading-pantheon-environment-configuration)
- [docs/articles/sites/redis-as-a-caching-backend](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/redis-as-a-caching-backend)
- [docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services](http://redisaudit-static-docs.pantheon.io/docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services)


I see now... forcing the cancel under all cases seems a bit harsh for times when network_disconnect() and network_off() are called by a system operation, like a network timeout that power cycles the radio.  What if we made the cancel only happen when a flag is set, and only commands that invoke network_disconnect() and network_off() asynchronously would set this flag to be true? An ex. would be if a user called Cellular.off() in multithreaded mode or from an interrupt.  Just talking this through... it could be that the cancel needs to be handled by the modem library more gracefully, although right now it works as originally intended; to cancel all AT operations immediately.  It's just that this was only originally for cases that ended up with the system shutting down or system in a dedicated Listening Mode loop.


Coding style: use camel case for function names. You could also make this a getter e.g. `get visibleApps()`


There should be a mention of "rolling out" to `Array` by using the splat operator.


Ah, it should be:

    "type": "number",
    "format": "float",

and also for `volume`.  See [here][1].

[1]: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#data-types


Shouldn't the check be `i < i2c_data.nmsgs` or something similar? I might be missing something, but it looks like you're just doing half of the number of readings you're supposed to do in this loop.

Minor: missing spaces in `i+=2`.


@hlindberg Reviewed.


Файлы с json могут иметь внутри себя разрывы строк (\n например), что сломает работу.
а) файл log.txt данный примером в задании не парсится так как там есть нечитаемые символы
б) в консоль об этом ничего не пишется (пользователь не знает, что произошла ошибка)

используй метод gson.fromJson(new FileReader(filePath), Messages[].class) вместо того чтобы самому читать :)


I didn't get why you have to multiply for dt_sec here. Shouldn't be just "last + error" for the integral part?


@bgerstle The alternative I was more alluding to was using a single WMFFetcherErrorDomain and set of shared errors that all fetchers use. I would think it is rare that a fetcher needs to convey an error that is complete unique from the other fetchers.

We have some of this already in:
`NSError+WMFExtensions.h`


Pull requests with corrections are welcome :smile:


@kbrock but yes it is possible if user's role have 'None' in 'VM & Template Access Restriction', than he can see VMs with different group.

thanks for asking


Sure.  Lets use this.  Automated tests do not test this so if you make changes you or @nazmussazib  will need to test the toolbox SINMAP function.


Always check for possible "crashes". For example when the caster suddenly disconnects, then there is noone who can CastSpell, which will result in a crash. (As you are learning a bit about core, I try to help you here and there, if you dont mind)


Is there a technically possible situation where 314:mDisposeAfterAsync = true and 876:if (mDisposeAfterAsync) are timed just right so dispose() is never called?


Probably doesn't matter because you change this later, but should have curly braces around this block.


Isn't that just a `fetchFromGitHub`? GitHub tarball URLs aren't deterministic, but their contents are.


ah yes.. I need to update the gitignore


- [ ] URLは、currentではなくスタックでサポートしているバージョン（4.2.4.RELEASE）へのリンクにしてください。


I was following the example in Auto-assign Cores-SW queue for 'osforms error'. Didn't you write that?


Change to palm tree oil


LGTM


Sorry, I added it and then remove in subsequent patch.


It's their own release (https://github.com/lwindolf/liferea/releases, see Downloads). If I understand correctly, they upload this tarball by themselves.


@phikal I swear for whatever :D


@phikal Well Google Tutorial Comments :D


Hm, now that I look at it...
```
		Engine::mixer()->removePlayHandlesOfTypes( this, remoteIPH
							? PlayHandle::TypeNotePlayHandle | PlayHandle::TypeInstrumentPlayHandle
							: PlayHandle::TypeNotePlayHandle );
```
With line splitting I can like this. Another rebase coming up!


Why not adding the already constantized form here?


Technically, it's not needed to bump versions from e.g. `4.2.4` to `4.3.0` since the abi version hasn't changed. It's just for clarity.


Coding style: should be camel case


Might be. Looks like a broken bot trying to verify stuff that is already merged.


@phikal see below at l.110


Shouldn't this be for `ttFEE`? We can do the same for `ttAMENDMENT` (and maybe we should) but your change adds the field to a `ttFEE` transaction.


I think we decided that for API functions it doesn't make sense to raise: we should cleanup the error, return NULL and make a FIXME to provide a better API in the future :grin:


Nitpick: you could save the state in a local variable.


@bgerstle I'm not understanding whats preventing you from using a block and/or method or what the trade off is in concrete terms. Can you demonstrate the un-DRY code that you are preventing so we can evaluate the trade off?


<?php
$host ="localhost";
$user = "root";
$pass = "";
$db = "system_encuestas";

					/*CONECTAR BD MSQL ANTIGUO

$conexion = mysql_connect($host, $user, $pass) OR die("No hay conexion a la base de datos");
mysql_select_db($db, $conexion) or die("No existe la base de datos");
?>
*/
				//CONECTAR BD MSQL ACTUAL
$conexion = mysqli_connect($host,$user,$pass,$db);
  mysqli_set_charset($conexion, "utf8");

?>


I'm asking to be sure, my English is not very good.
Can I use this in my own project in your library ?
No right to live license problem


Should you not set the starting state to false here?


Fixes Bug #20


Same here; API function should cleanup and return NULL.


make bind great again


What I want is different than the purpose of `NullifyBlank`. It is intended to nullify blank attributes `when coercion failed`. I want to nullify blank attributes even when the coercion is successful and results in a blank Array, for example.

I think the right thing for Virtus is to wait for the https://github.com/solnic/virtus/pull/209 to land and then use the official way to extend the attribute behavior.


You got a typo buddie


Do you have some actual knowledge how much are other non-PEP-0484 types of hinting wide-spread? It seems to me too complicated to support four types of doing the same thing (“Simple is better than complex.”) Couldn't we just support PEP-0484 fully (eventually, now only comment hints) and be done with it?


It's a void function, no need to use "return" here


thanks


Here too: cleanup and return


tt


Maybe it would be better to not use a factory for such a test. It can now easily happens that someone breaks the test by changing the factory. Or, maybe worse, hardcode the factories state to 'unconfirmed' which would make the test pointless.
Even if it's unlikely


Though it isn't caused from this PR.

Finish porting `DWORD`/u. long to `uint32_t`/u. int for a fix, except atm I'm on a break from PJ.


OK.  I am fine with that.  But (a) I will need to ask Joe to put up the new parametric fit file separately (currently it's only in the tarball) and (b) I will have to update the stand-alone selection file (currently has the old name scheme).  It could get a bit messy and confusing if we have many files listed on the website for download, but I guess we can be careful about how they are labeled and it should be OK.


The following doc(s) have been deployed to the [redisaudit](http://redisaudit-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/drupal8](http://redisaudit-static-docs.pantheon.io/docs/articles/drupal/drupal8)
- [docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services](http://redisaudit-static-docs.pantheon.io/docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services)
- [docs/articles/sites/code/reading-pantheon-environment-configuration](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/code/reading-pantheon-environment-configuration)
- [docs/articles/sites/redis-as-a-caching-backend](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/redis-as-a-caching-backend)


@phikal Google ....


Oh I see. Still seems like they're tracking the git tags on each release though. Often those github releases are just the product of someone running some script on the tagged git revision. Perhaps we could just do the same thing on the underlying source code?


One operation at a time *per bus*. If all the API capabilities are to be exposed, maybe we should have the pending handles per operation.


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


@giros
I would look into why we call updateImpl here. I think the problem is that we are not calling this in a case where we used too.


if you're going to compress them like this why add the `{ }`?


Sure thing.  If you're updating the script to do the other stuff, would you mind going ahead and doing this too?


There seems to be a [validate_uniqueness_of_matcher](https://github.com/thoughtbot/shoulda-matchers/blob/master/lib/shoulda/matchers/active_record/validate_uniqueness_of_matcher.rb). Any reason you don't use that one?


There was already a call for lua support.  But, as I'm realizing, lua is incredibly fractured.  There's big breaking changes between 5.1, 5.2, and 5.3.  Semantic versioning went out the window for some reason.  This negatively affects libs and frameworks.

Then, there's a second fracture between the jit/vanilla runtimes.  The jit efforts are separate from the core interpreter, and the changes made in lua 5.3 are not supported by the current jit.

So, we're in a situation where we can only support 5.2 vanilla, 5.2 jit, and 5.3 vanilla for Haxe.  Each will have its quirks and differences.  My gut feeling is 5.2 (jit/vanilla) will end up being popular for some time.

I have some work in progress to separate out the 5.2 jit vs. vanilla.  Then I'll figure out what needs to happen for 5.3.


@bgerstle The if / else here are exactly the same thing.
Tweaks compile down to the defaults when in release mode.


Итого:
- исправь возможность создания сообщений с одинаковым id (да и не должен пользователь id вводить руками, он ведь и накосячить может)
- поправь хэндлинг exception
- сделай хоть какой-то вывод ошибок в ui. если что-то не сработало пользователь должен об этом знать.
- поправь парсинг файла json
- убери все лишне из репзитория - он должен быть только для практики

Задание 1.1 и 1.2 принято.


That's already created in line 5, isn't it?


Agreed. "No event" is a known event.


arg is direct object


About 10s-15s.


ugly in Swift no matter what the name


I think the only thing they do is running `autoreconf`. Yeah, let's move to `fetchFromGitHub` -- if upstream did such a thing once, they would do it again ~_^. I'll submit a patch...


factory


You're welcome. Mr Saxo Beat


Or I could move the clock 1 year? I'll give it a try...


Looks great. Hopefully this didn't take too long!


Humm, do you think you could submit a request which kept compatibility for Django < 1.7? (i.e.: try to import the new from django.test.runner import DiscoverRunner and if it fails keep on with the old code).


yeah, guess i was getting a bit carried away by only parsing the "CSV" in debug mode. i can remove if it hurts clarity too much


If those lines should go away, then maybe remove them altogether instead of leaving commented out source code in there? I mean, this is git, after all. Just some minor nitpicking ;)


done


Can you use `dispatch_once` + `dispatch_sync` onto main thread to initialize these without having to explicitly call `prepare`?


@vocrsZ


What does this statement mean?


можешь мне пояснить этот момент более подробно? я не смог заметить разницы (


Add fstab lines


Edward check my branch


TODO about error handling? - Should this be fixed before PR is merged?


@phikal should this model be a class or just an array? My implementation is in my opinion better, because with a model I can't validate so easy the correct data input, or am I wrong?


\o/


My understanding is that if you want to send an error message to a client, it still has to be 200, but contain the bencoded error response.


This should be documented


@phikal Same comment as upper mentioned


@thomthom First off, I apologize for taking so long to reply.  Work has been interesting lately.

The reason I put the commit into the `fast_window_position_updates` branch is because its not really testing transparency, but rather the ability to draw in one window based on the contents of a different window.  If you build and run `examples/filters.c` in the `fast_window_position_updates` branch, and then drag the windows around on top of each other, you should see the effect (the red window will have a triangle drawn in its center, and the other windows draw triangles that exactly overlap the red windows).


Np @feliciaan


@deads2k - what did I miss here?  These third_party and libs files were required for testing but were not copied over by the godep.  Should they be added to the copy-kube-artifacts command?


@sjaakd So that's the commit which fixes the line ending style for this file. It's not clear to me how you managed to to do it, given that the repo-specific config mandates otherwise :) Can you try and squash your last commit into this one?


Does this sound better?

> Some of its output is about "things" that we do not ...?


5420-OSCM-Docs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2086 of 2086 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

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


Consider declaring inside the if statement.


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


The following doc(s) have been deployed to the [redisaudit](http://redisaudit-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/drupal8](http://redisaudit-static-docs.pantheon.io/docs/articles/drupal/drupal8)
- [docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services](http://redisaudit-static-docs.pantheon.io/docs/articles/local/ssh-tunnels-for-secure-connections-to-pantheon-services)
- [docs/articles/sites/code/reading-pantheon-environment-configuration](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/code/reading-pantheon-environment-configuration)
- [docs/articles/sites/redis-as-a-caching-backend](http://redisaudit-static-docs.pantheon.io/docs/articles/sites/redis-as-a-caching-backend)


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


@phikal Well what else I should return? The sense of this method is to give back the position. AND the BaseAdapter class forces me to implement this method.


The iterator should be enough for that. If the vector were strictly needed, then you wouldn't need the iterator --- you can just get begin/end pointers from the vector itself.


hopefully @dbnicholson can suggest a good way to deduce the URL here


Not sure if this context is needed if it's always the same. Either that or this indicates that a 'when invalid' context is missing.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (397 of 397 tests passed, 0 test warnings)

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


@phikal Why?


@jcelerier :  attention ce commit casse pleins de truc dans i-score.
est-ce que c'est possible regarder si c'est vraiment critique parce qu'au final il me semble l'utilisateur de l'API ne devrait pas pouvoir donner un temps ici car le temps est managé par la contrainte parent au process (ou par la contrainte elle-même).

à noter que j'ai rajouté une méthode offset sur une autre branche feature/offset : https://github.com/OSSIA/API/commits/feature/offset

pardon pour les changements un peu radicaux mais il le problème du goto le nécessite sinon l'API n'était pas propre à ce niveau.


Hey


Oh, I thought you had the parametric file there as a standalone, not just in the tarball.  If not, then I think it's fine to just detect that the selection file is missing and re-download the full tarball.  A slight waste of bandwidth, but probably not worth the extra machinations to avoid.


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2086 of 2086 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

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


release-5.x (lefticus) - x86_64-MacOS-10.9-clang: OK (389 of 389 tests passed, 0 test warnings)

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


looks like it might. I'll filter on the specific doc types I'm expecting in a forthcoming PR.


I need help with making 4 plugins
1.SimonSays
2.SoundEffects
3.MineKart
4.Build-Battle


It's a close call, but I think I like them both equally. It's a near zero cost call, and there's some elegance to having all the declarations in the same place. Given that there's a non-zero cost to changing it (have to push a change, retest, and so on) I'm not going to.


Agreed


just curious why we're putting this on the background queue rather than the periodic queue?  the background queue is generally for lower priority, longer running tasks, and if this runs every 5 minutes and the background queue is full these tasks are going to get backed up


@phikal Will remember this, but this class will anyway be completly new developed.


godep doesn't walk tests.  I think there's a file that picks up these packages somewhere.  @soltysh?


Maybe you should use two different maps instead of the same map.


This commit needs a proper commit message.


Good call. Fixed! In future, submitting an issue (or even better, a pull request fixing the problem) is an easier way for me to keep track of problems. Not a problem in this case since the fix required was so small :P


factory


what about normal `private`?


The blank alt tag example is broken, and really should be in lowercase.

![image](https://cloud.githubusercontent.com/assets/4443025/13086602/c8284b42-d498-11e5-80b1-0d553684f027.png)

And I suggest change out all the curly quotes to straight quotes to help with any character encoding issues.


Looks good, I think this is the best way to go. Even tough it doesn't look that pretty at first sight.


arg is not direct object


I see...

Perhaps you should put this doc block to `SimulatorInfo prepare`, and then inline this tiny method.


where is the terminal?


[pp[[pp[


- [ ] 入力チェックエラー時は「`model.addAttribute(account);`」してますが、ここでは不要ですか？改ざんしたケースなので、不正アクセスエラー扱いにしてしまってもよい気はします。


arg is direct object


Memo for tourists is prohibited :laughing:


Nice!

Rytoj bus code4vilnius meetup, tačiau aš ateiti šįkart negalėsiu.

2016-02-16 10:52 GMT+02:00 Arik100 <notifications@github.com>:

> Padaryta :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tadas-subonis/vilnius-techmap/commit/84b4dccbfbe49a43aab03a67c060ceb27506c44a#commitcomment-16111984>
> .
>

--

Kind Regards,
Tadas Šubonis


The gem is not well maintained, but again I needed something a little different that is specific to how I'm using Virtus.

Here is a diff of the changes I made to the original code. https://gist.github.com/forest/40d3244acb00bb7a0322

As you can see I use `instance_variable_get` instead of the attribute getter method to get the original value. This is because I often override the attribute getter to implement lazy loading of associated data from the API. Like `Project#epics`. Making a API request just get the original value is not what I want. I did have to be careful to make the object as clean when data is loaded from the server. Here https://github.com/dashofcode/tracker_api/commit/27599e7e2169776c32bbff8c972a31b930452879#diff-bc0aedff6e7b721b4c0483b1b1fa3bc4R34 for instance.

I decided it was easier to copy the code than fork and publish a new gem with different behavior.


not sure about this one; arg is direct object, but perhaps you could say the C string and length are peers?


I'd be happy to look at combining these if as you say there are performance issues associated.


`ng-if` does not work in this case, at least as a drop-in replacement for `ng-show`


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (389 of 389 tests passed, 0 test warnings)

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


The use of the terms "upwards" and "downwards" in this documentation seems to be flawed, or at least very confusing.  See the discussion at http://stackoverflow.com/questions/35421466/git-definition-of-upward-downward-directions-between-branches-feels-counterintu/35428018#35428018.


release-5.x (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 tests passed, 0 test warnings)

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


We should be careful with calling hashCode on JSONObject since not all JSON libraries implement hashCode correctly.  Apparently json.org doesn't: http://stackoverflow.com/questions/14129791/hashing-a-java-jsonobject


This looks pretty clever and LGTM. Thanks!


arg is not direct object


Rebase done; had to split even further than in my example to stay within the 80 char limit. Blasted long identifiers those PlayHandle types have!


No, it's not up as a standalone (and even if it had been, I'd have to replace it with the new one).

OK, let's go with the simple version that wastes a bit of bandwidth.  Since you're not putting all the checks back in, I will go ahead and change the "save" option as we discussed elsewhere on this commit.


Sure thing! Thanks for the reminder and sorry that you ran into the same issue :/


Alas, now it doesn't work on Kubuntu 12.04.


seems like this is the real fix and the rest is refactoring.


I agree that "topics" is not great, but "things" might be a bit vague... How about "properties"? Probably not "attributes" (since "file attributes" has a specific meaning).


Why not move this over to julialang.org/soc under a Juno/Tooling section? I think that will give this more visibility, and these are some potentially impactful projects


omg...


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


args are not direct objects