cc @enykeev https://stackstorm.slack.com/archives/stackstorm/p1453377793029078 :)

That packaging stuff is really ... awesome. So same for me, once I dig into one thing, I need to read about another thing and then it goes to third thing I need to google/learn as fast as it possible, resulting 50+ opened tabs with no progress really.
The barrier to entry is not low if "doing things right", not doing dirty hacks here and there.

I'll probably need kind of 12 hours non-stop nirvana for several days in a row to dig into details: layer after layer, instead of trying to do something really quick (assuming several hours of time I currently have).

So hm, just :+1: to @dennybaa perseverance for learning all the packaging stuff, doing it right and understanding low-level alchemy.

-----
Working on digging into these things too, tried different ways, sharing some thoughts.

For the moment several arguable & worrying things probably we'll need to resolve:
- we shouldn't forget about `www-data:www-data` user verification and creation
- Assumption `nginx:nginx` and relying on `nginx` only is not flexible
- `touch $(DESTDIR)/var/run/st2auth.sock` and `%attr(644, nginx, nginx) %{_localstatedir}/run/st2auth.sock` shouldn't be part of packaging logic, since socket should appear only by and only when service starts. Also, `/run` and `/var/run` doesn't persist across reboots.
-  Seems relying on `/run` and `/var/run` doesn't looks ideal for non-privileged processes, since it's usually directory for root processes. So yeah, rather create socket as you already do before dropping privileges, - another point would be to save it in "app" directory. For example is `/opt/stackstorm` an option? Or maybe `/var/lib/st2` for sockets, I don't know.

There is some additional good info I found that helps in understanding the inner details:
* http://0pointer.net/blog/projects/tmp.html (totally useful thing)
* http://serverfault.com/a/727994 I remember already saw somewhere `RuntimeDirectory` mention for systemd. Also: http://uwsgi-docs.readthedocs.org/en/latest/Systemd.html#putting-sockets-in-run But that's only for `systemd`.


`z-schema` returns an error if any property has `type == 'file'`, so here we are:

* forcing the schema `type` to be `'object'`
* removing every `file` property from both the request payload and the schema for this route

We are doing this so other properties in the schema can be validated. The other possibility is to say "this has a file, don't validate this schema at all" (which may be valid option too).


Does the internal repo has a different version of this function? Otherwise you don't need to pass GlobalVariables to it, you don't use |globals| here.


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


I feel like this should have casts, `(uint64_t)data[0]`, etc.


It was longer than a couple of seconds.  Recent changes bring it down to that.  There seems to be a lower threshold caused by spawning several processes.


Fix the issues in your scripts and then the warnings will go away. When the warnings go away, the FPS problems should go away.


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


Disable flashhing of stock recovery
Thx beanstown106 I73e3fbd652189146335af0bc7fb0c6323ed3787e


htmlspecialchars should be enough.


If there is a reason you're not just using [`net/http/cookiejar`](https://golang.org/pkg/net/http/cookiejar/) (attached to the `*http.Client` you create) you should document it here.


Indeed, I had forgotten this small detail about the English and Chinese people.

Did you see the log system developped in RaidCore to save fight informations ?
I did it for persons like you or me.
It's a way to export all events and data in a JSON file. File which can be convert in CSV and so import in Microsoft excel or in a google sheet.

About encounters, some of them should be reworked a little bit.
By example:
- PhageCouncil, I reworked it recently. that should be not too bad.
- But not Ohmna haven't been rewritten since a long time.

However RaidCore is much more helpful for Datascape, than Genetic Archive.


haha


but winrenderer may not creates all required buffers if rendermanager doesn't set buffer size. does render managet set buffer size before configure renderer?


OK, still fast compared to travis setting up a test environment


You can remove all the cases where _MSC_VER < 1700. 1700 is MSVC 2013 and 1900 is MSVC 2015, and we don't support any version lower than 1700.


Check the Travis build... Failed at line 66.


But slow relative to local testing.  This PR does double testing times.  This is non-trivial.  I'm tired enough of this problem though that I'm willing to take the hit for now.


you are right! it should be moved to update, this is called right after buffersize is set


@MartinLorensen:
To be honest, we're intending to mostly use the wrapper methods now, which essentially means using it like https://github.com/graphql-dotnet/graphql-dotnet/issues/14 (those examples aren't 100% aligned with the actual implementation, and the implementation has yet to be transitioned over to the graphql-dotnet/conventions library). In other words, that would entail pulling the classes describing the mappings and the actual data-transfer-objects together.

So you could do something like:

```c#
class Query : QueryType {
    public List<Foo> Foos => new List<Foo> {
        new Foo { Bar = "abc" },
        new Foo { Bar = "def" }
    }
}

class Foo : OutputType {
    [NotNull]
    public string Bar { get; set; }

    public float? Baz => 3.14159;
}

var schema = SchemaBuilder.Instance
    .Add<Query>()
    .Generate();
```

And that would give you a type hierarchy like:

```
Query:
   Foos : [Foo]

Foo:
   Bar: String!
   Baz: Float
```

Further, that kind of removes the need for `Field`- and `ConnectionBuilder`. I will continue to support those though, so if you have any preferences, I'd like to hear them. I was planning to support both appoaches.

The `RelayNode` interface is mostly intended to be used with the wrappers demonstrated above. Would be curious to hear what you ended up doing to use it with the field builders. Also, interesting observation re `IdGraphType`; I will look into that.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 98](https://teamcity.labkey.org/viewLog.html?buildId=298540&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:27


hmm i pushed to the master branch, so i don't think i need to make a pull request. correct me if i'm wrong..


Why shift sizeof(int) here? The logic is little bit unclear. Please define another header type - ControllerToBS header that includes packet type and client id. You may refer to AthCodedHeader and see how it's used in wspace_ap_scout for reference.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 96](https://teamcity.labkey.org/viewLog.html?buildId=298542&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:07


This flag is on the edit gallery page. Can normal users even access it?


Should the [docs](https://github.com/nodejs/node/blob/0634311a9ac3dda0e3b494f7f510c23ca48acd46/doc/api/buffer.markdown#bufindexofvalue-byteoffset) be updated? It doesn't look like the encoding field is there right now.


HBIRE_vector_take2 (amirroth) - i386-Windows-7-VisualStudio-12: OK (2047 of 2057 tests passed, 506 test warnings)

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


@BHeisel you understand this PR wasn't submitted against clc/master but against the af fork, right? Just wanted to make sure.


we can do this without the extra iteration through volumes.  I would still keep the check for `!s.scc.AllowEmptyDirVolumePlugin` and `!s.scc.AllowHostDirVolumePlugin` so if there are no restrictions we don't bother iterating.


@aot221 @estevez90 Shouldn't record.filename always be present? If we are uploading a file, the filename has to be passed back, correct? So isn't this issue something else?


I've been told that `mdast` is now called `remark` so we should use that now.


Love it when code goes away! Hate that it was around, though.


Ok. What's the story? Did the project fork?


lol hi ian


@johde I understand. Can you move the 2 *if* statements starting at line 78 ending on line 83 to immediately below line 151? That should resolve the concern.


It got fixed, Thanks @twinaphex great job  =D


This is already a default on `^Aircraft`.


Please change the name of packet type. It's mentioned in my previous comment in wspace_asym_util.h. +// #define BS_STATS 8
 +// #define CONTROLLER_TO_CLIENT 9
 +// #define CLIENT_TO_CONTROLLER 10


Finished second round of reviews.


ok


I am not sure if this is a GHC bug/regression, it shows up as

```
Malformed instance: forall r m.
                    MonadReader r m => MonadReader r (GmT m)
```

under GHC-8


Nit: everywhere we say "Comparator", it should be "Comparer"


At a high level, the change makes sense to me. The printing moved out of the process (ProcessManager) and into the place where the job starts.


learning git


Nit: I find this a bit of a strange construction, with an initializer that doesn't iniitalize.


kys jose


@vonH I want to let you know that I've missed to cherry-pick this commit from my fork while we switched over to your repo :smile: Take a look at the commit message.


Line comment on a commit


Comment on a commit


since this is a pointer it should look something like the SecurityContext.Privileged conversion (same for the other conversion/deep_copy files in the other commits)

```
if in.Privileged != nil {
		out.Privileged = new(bool)
		*out.Privileged = *in.Privileged
	} else {
		out.Privileged = nil
	}
```


@stevvooe No, flush is called by Seek, so that is not the problem.


False alarm. Turns out, there was another scrobbler installed as an iTunes plugin. Sorry for the confusion.


@triblondon could you explain the purpose of this route please? I'm having a hard time thinking of a reason for it to exist in this package. Also, we should update the readme to showcase the new route. I also think this is a breaking change as it is a new route which will cause applications to crash if they don't handle the error in some form.


Could be:
```C#
Assert.Equal(index % 2 == 0, some[mask]);
```


i answer here for the question on forum .. i can launch a little script in livecd to override the xml schema and regenerate it for pantheon ? If launch in livecd and fixed in livecd when install is also installed...


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1245.7940199c7a694b1df06face91318d8fee2ceebd3](http://build.palaso.org/viewLog.html?buildId=68374&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 846, ignored: 7 Build time: 00:05:51


Remove the other declarations from arch/stages.h, please.


My attempts to handle the animations on my own haven't worked, so since you're offering I would like you to handle the animations.


I agree, but it depends on the target. On JS all these flags are `false` anyway and that's certainly the target most "looked at".

The only `pf_locals_scope` (which is the only somewhat relevant flag) targets are neko, flash (bytecode targets, var names don't matter) and cpp (for which I [recently made a change to use the original name for debugging](https://github.com/HaxeFoundation/haxe/commit/84641bf4c63064f0422b1a4dc0bb2f8bc89210a1)).


teste


Combine this with the loop above.  You can leave the checks for `!s.scc.AllowEmptyDirVolumePlugin` and `!s.scc.AllowHostDirVolumePlugin` so we don't iterate through volumes if we don't need to.


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


If I understood right on forums, you need this "fix" because some pantheon package is outdated.
I think that this fox should be applied to the proper pantheon package.


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


Do you recall why you used the latest dependencies when you ported this?


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


Didn't even remember this file was out there :)  Should be a pointer.


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


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


Please see https://bugs.gentoo.org/show_bug.cgi?id=566534 for a discussion.


Could  you add comments for why these if/elses are necessary?


Would probably make this a Warn


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


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


That's right.  The one that was important to change was `BN_sub_word64()`, since subtraction is not commutative.  I went ahead and changed `BN_add_word64()` and `BN_mul_word64()` to match so there would be fewer questions later.


What does this comment mean? Do you have some unimplemented logic?


@Yive this seems to have solved the issue. I have had no issues with it for the past few hours. PR 2?


Oh, disregard, I see that it's a non-pointer in the internal type


112158233_PVwithStorageSystem (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 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-26.85%25-green.svg)


I set the `env` to custom.


This seems not thread safe - I would not set a trans property on once-per-runtime objects like tool parameters.


I see that it's a non-pointer in the internal type - never mind


v3??? avísenme!!!!


You log these errors higher up the stack. You can remove this debug statement.


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


Outch..  I will have to check that tomorrow, Sorry about that!


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


This is a strange issue off course , i have rebuild all pkg last week , all working also gala now work perfect but this issue is not fixed .. for work gala want downgrade at r96(?) the release in alucryd repo otherwise wingpanel not working


Ça me parait indépendant des autres PR en cours. Donc directement dans master, ou dans une branche templates qui en dérive. Si c'est lié à yaml, tu peux aussi faire ça dans cette branche…


It has been too long since I looked at this code for me to know the answer to this question offhand. :-)

At least there should be a comment here giving an example of the situation this is supposed to handle


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


Thanks, you'll see the result of this once we move the package to the open (coming soon).


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


Bofan, you're right. I pulled your new commit and the bug has been fixed.
Thanks! I'll deploy it to the production server right away.

By the way, let's keep branching workflow for safe collaboration, rather
than working on master branch and directly pushing it. I want to have
master branch for just merging branches that are completely unit tested as
the app's complexity increases.

Taek

On Thu, Jan 21, 2016 at 1:31 PM, Bofan Chen <notifications@github.com>
wrote:

> hmm i pushed to the master branch, so i don't think i need to make a pull
> request. correct me if i'm wrong..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stlim0730/farmview/commit/bbe1c0083a5f4405b930d0e8e0d28882e3ebbe29#commitcomment-15598345>
> .
>


No, I suggest you apply a patch to the package that installes the schema, to add the missing key.


Seems to be doing a lot of good :)
http://pastebin.com/JB0jEQb6
However, the game does not run :'(


Russian translation adds


Probably 755 are the correct permissions for these drivers


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


oh okay! sorry about that!


Thanks for adding this README file. With this script now being in its own repository, this is very useful. It also highlights a couple of issues I'm thinking through. And pardon me if this is a little bit stream-of-consciousness:

1. The various things exported above are named inconsistently, (granted---I came up with the names of most of them---but if they're going to be exported for external use, I think it would be nice if they had a consistent name prefix like "travis_setup_" or "travis_metadata_" or similar.

2. Some of the things in the README definitely feel like they should exist only for internal use, (like configure_git_authorship). Each of the primary functions defined here will probably want to call configure_git_authorship as their first thing, but does a function like that really need to be "exported" as it were and described in the README?

3. I see you landed on the name travis-metadata-script for the script itself. But how would you actually document what that does? (That is, what does the "main" function, as it were, actually do in this script?) I think it would be hard to document since it does totally different things in various cases depending on the which repository is calling it. It's basically the conjunction of all of the travis_for_* functions described above. And that would make for a fairly messy description for a single function---which suggests something's not designed ideally here.

So here's a thought toward resolving each of the issues above:

1. Eliminate that unnamed "main" function, (if it doesn't exist, we don't have to name it---solving the difficulty of documenting it). That makes this script now do nothing more than define some functions.

2. Distinguish for each of the functions whether they are meant to be used externally, or whether they exist only as internal support for other functions. Then document in README only the public ones, and ensure that they each have a common prefix. (The internal ones should probably also get a common prefix---maybe something like an initial "_" at least, but maybe even the same prefix as the public prefix but with an '_' as well, first).

   As for the prefix for the public functions, it occurs to me that starting it with "travis_" might not be the best idea. We recently learned that the Travis folks are providing at least a function named "travis_wait" so we probably want to ensure that we aren't setting up any potential name collisions.

3. With the above idea implemented, a site using this script would simply source the script and then call into any functions desired. So, our typical Django app repositories might have the following in .travis.yaml:

before_script:
- git clone https://github.com/nimbis/travis-setup.git ./.travis-setup
script:
- . ./.travis-setup/travis-setup
- travis_setup_verify_coverage_improvement
...

So, this assumes the existing travis_for_apps function was renamed to travis_setup_verify_coverage. And this shows how the script could be sourced (to define functions) and then functions called. This would also set up the fact that the sites repository might eventually want to call "verify_coverage_improvement" itself.

4. It may be that not everything in the current travis_for_sites function, say, would need to be moved into this script. This script is for sharing, and a lot of what is in that function is very specific to the sites repository and its special configurations. (If we keep it out, that might also mean some of the dependent, internal functions don't need to be here either.) At the same time, I can see an argument for keeping all of our travis stuff in one home, even if the functions are called by multiple repositories. So I wouldn't be against it staying here either if you prefer that.

5. Finally, on naming, (always the hardest part). If we want to avoid name collisions with travis_ I start thinking about nimbis_, so nimbis_travis_, but that's already getting very unwieldy. So how about something like NImbis Functions for Travis.Yaml, with a prefix of nifty_ and maybe even a repository name of nifty? Too cutesy? Again, your call.


Electric-Service-Refactor (EnergyArchmage) - 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)


I have `use_relative_paths` set to `true` in my first config, and then in my second config I have it set to `false`, and it doesn't seem to be working.

```json
 {
   "applies_to": "spec/javascripts/**",
   "applies_from": "app/assets/javascripts/**",
   "declaration_keyword": "const",
   "import_function": "myHelperFn",
   "use_relative_paths": false,
   "strip_from_path": "app/assets/javascripts/"
 }
```

Interestingly, the `strip_from_path` seems to be working even though it ends up in the middle because of the dots and slashes.

```js
const MyThing = myHelperFn('../../../path/to/MyThing');
```


fdafffs


Correction, input shows up in jgrasp but does not show up in CMD. Previous issue caused by windows firewall. Data is sent just fine.


fdaffds


sound good .. fork pkg and install it :)


Vieille erreur crade ici


When I first tried this, I set it to `"app/assets/javascripts"` which gave me the following error:

```
Error detected while processing function importjs#ImportJSFixImports:
line    1:
ArgumentError: different prefix: "/" and "spec/javascripts/something"
```

I realized that I needed to add `/` to the end of my configuration, but this error message was pretty cryptic.


Mm, I guess not. I set up a new run (with a 256x256px version to take a look now that I thought dcgan.torch might run in constant-memory), tuned to leave ~100MB free on the GPU (balancing a dcgan that big isn't *too* hard; you just need to lower the learning rate and mini-batch size and keep the usual 2x gen:dis ratio) but no, errored out at epoch 1 with a suggestion that the RAM consumption isn't fixed:

    Epoch: [1][   70261 /    70261]  Time: 1.075  DataTime: 0.000    Err_G: 1.8536  Err_D: 0.3223
    /home/gwern/src/torch/install/bin/luajit: /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:162: $ Torch: not enough memory: you tried to reallocate 8GB. Buy new RAM! at /home/gwern/src/torch/pkg/torch/lib/TH/THGeneral.c:251
    stack traceback:
            [C]: in function 'write'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:162: in function </home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:90>
            [C]: in function 'write'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:162: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:181: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:172: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:181: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:181: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/torch/File.lua:172: in function 'writeObject'
            /home/gwern/src/torch/install/share/lua/5.1/nn/Module.lua:106: in function 'clone'
            /home/gwern/src/dcgan.torch-allbrown/util.lua:6: in function 'save'
            main-256.lua:257: in main chunk
            [C]: in function 'dofile'
            .../src/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
            [C]: at 0x00406670

I'm not sure what's going on here... I've tried out several Torch-based applications and dcgan.torch is the only one which has such enormous spikes in RAM usage at checkpoints; eg `char-rnn` and `neuraltalk2` can both be pushed to <50MB free without crashing at checkpoints.


Super nit-picky, but calling it `key` would be clearer than `uri` here.


Dont log error or debug here because you log the error higher up the stack. You log it higher up the stack


Successfully created the [circle-opti](http://circle-opti-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


Correct, `docker-build pull` might fail, but we always need to do `docker-build build`


OK


Was this supposed to be this? ![Spinner](https://raw.githubusercontent.com/kylefox/jquery-modal/858942192d0db01235a04e2ff0033c1672872b83/spinner.gif)
It looks like you curl'ed the HTML page rather than the "raw" page.

P.S. Sorry for the comment on the old commit, but this was the most recent commit that touched it (that I could comment on).


Not sure if this is related, but this happened when I was selected it as an option out of a list of options, and it appeared in the list in the same way as it was written into the file.

Expected: `path/to/MyThing`
Actual: `../../../path/to/MyThing`
Used to be: `../../../app/assets/javascripts/path/to/MyThing`


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


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


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


Good catch. I'll have to fix this for the GIT_CONFIG_EMAIL variable as well.


Ok, fine then.


Could be a BlockPos...


You got to patch it, I can build it for repo.
Just get the PKGBUILD, use diff new.schema and orig.schema, and add patch to PKGBUILD.


This code isn't really doing anything and should be deleted


please delete unused code


Does nether work?


It seems to be leaving out some overrides


TeamCity pull requests :: netty-tcnative [Build 40](https://garage.netty.io/teamcity/viewLog.html?buildId=4966&buildTypeId=pulls_netty_tcnative&guest=1) is now running


I'm pretty sure it does. If you want to generate a nether world, change your level type to nether... I think.


Oh yeah, I didn't see that!


cc @blair1618 @patriciogonzalezvivo this fixes the y coordinate off-by-one issue we observed. I'm not pleased with this solution because it fixes the symptom, not the root cause. But after struggling a bit to trace the issue back (without disturbing other functionality like back-face culling), I'm going to just move ahead with this for now. I'd like to revisit soon and step through a comparison to ES tile processing + view matrix.


the head should be indented within the html


Thank you very much for splitting these commits up when re-doing this branch. I really wanted to give this piece, (the part that computes coverage and tests that it only improves), a better review than I had been able to do before when it was in a commit mixed up with other things.

I think this looks really great. There's a tiny issue with some of the logic hiding in "make coverage", (this function makes some implicit assumptions about what "make coverage" does right now), but I think that's OK since:

1. All of our repositories actually _are_ doing that in "make coverage" already

2. We can't just inline the functionality of "make coverage" here since there is some variability to it, (the list of files to be given to the coverage program)

3. I do like that the current approach has so little impact on the app repositories, and I don't really want to lost that, (such as if we had to chang them to somehow communicate the list of files to this script).


tried this (move ManageTextures() from Configure to Update) - not much difference...


TeamCity pull requests :: netty-tcnative [Build 40](https://garage.netty.io/teamcity/viewLog.html?buildId=4966&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:59


body should be indented


css link should be in the head of the html


Argh, yes, I think I lose transitive relations.

I should really build a proper class field dependency graph, I need that for the analyzer anyway. I'll get back to you...


1. I'm not sure... it just worked easily for me , never ran into problems. Try googling that error?
2. That makes sense - an interactive rmarkdown document requires a Shiny server to host it.  It's not just an HTML document, it needs shiny's support behind the scenes. Jekyll doesn't have shiny support.  What you can do is include your document inside an `<iframe>` that points to a working interactive doc
3. As the beautiful jekyll README says, I have never tried running jekyll locally so I really can't help with that unfortunately


`options` may be a better name here. when i first read it i thought this was pulling from django settings and was curious as to why the keys were lower case


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


Confirmed, for instance in Heaps h2d.Anim overrides are eliminated with this DCE


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1246.297c8cb87e2ea4e19e87355b533cbeec5add79cf](http://build.palaso.org/viewLog.html?buildId=68375&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 846, ignored: 7 Build time: 00:05:39


dude

do u even potato


This looks pretty good, though a couple of things:
 * I'll need a signed-off-by to apply - please see CONTRIBUTING file
 * I believe shared folders will not show the 'Shared-Xyz' due to no longer exporting fullname.  "Hostname" though doesn't make a lot of sense as title for that column.  So, I'll try to figure out what the website is doing here.
 * We should be able to pull in favorites by changing ```skip(fav)``` in blob.c to an assignment to a new field in struct account.


Page looks good! Just being picky, but there are a few little items that need to be addressed. Let me know when you're done and I'll take another look at it. Nice work- I enjoyed the beard pics!


no they just wanted that name originally and finally got it.
https://github.com/wooorm/remark/issues/113

On Fri, Jan 22, 2016 at 10:34 AM, Paul Frazee <notifications@github.com>
wrote:

> Ok. What's the story? Did the project fork?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ssbc/mdmanifest/commit/94bd54d8594f69a3746800a79a5808418ec428dd#commitcomment-15598408>
> .
>


Maybe this will do instead?
https://golang.org/pkg/net/#SplitHostPort


I actually wrote it like that because in dsym headers it appears somewhat like this


numberwang*


not needed .. i can upload here http://download.tuxfamily.org/repoarch/pantheonrepo/
is my repo .. leave manjaro repo clean ;)


4.21 isn't released yet so we can keep adding bullets to 4.21


If we're validating, we're going to calculate percentages on `currentFinished + 1` inside `checkConsensusReached` which is different from our existing behavior. Is that correct?


Great test!  I did a little hunting around the web and found arguments about whether or not remainders could be negative.  I'll do some messing around with the test and see what kind of results I get.  I think the final answer is that the Quotient Remainder theorem (https://en.wikipedia.org/wiki/Euclidean_division) must be satisfied.  And the two implementations should give identical results.  I'll certainly add at least one test with a negative `BIGNUM a`.


I suppose we need something like a `LOADING` state that's set when a request is still processing, to avoid further UI input from being processed (and a check for the current state in `Receipt.payNow`, which I seemingly forgot).

I'm working on another branch to show the name of the user in the countdown, which would also suffer from races when I don't switch away from the `SALES` state immediately.


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

 * 548 tests had: AUD diffs.
 * 478 tests had: RDD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 2 tests had: Table big diffs.
 * 3 tests had: MTD diffs.

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


Ah, ok

On Thu, Jan 21, 2016 at 4:01 PM, Dominic Tarr <notifications@github.com>
wrote:

> no they just wanted that name originally and finally got it.
> https://github.com/wooorm/remark/issues/113
>
> On Fri, Jan 22, 2016 at 10:34 AM, Paul Frazee <notifications@github.com>
> wrote:
>
> > Ok. What's the story? Did the project fork?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ssbc/mdmanifest/commit/94bd54d8594f69a3746800a79a5808418ec428dd#commitcomment-15598408
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ssbc/mdmanifest/commit/94bd54d8594f69a3746800a79a5808418ec428dd#commitcomment-15599021>
> .
>


112158233_PVwithStorageSystem (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 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.69%25-green.svg)


how does this work without riak running? also could probably be `SimpleTestCase`


Should this be `mProposing` instead?


Btw, the file conflicts you get, either remove the settings-bzr package, or the files in desktop-settings. ;)


Piedra, Papel, Tijera, Lagarto, Spock


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


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


Sure, I'll do that first thing tomorrow.


Looks good


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


nit: this could benefit from a kwarg. don't know what passing `True` in does


OctreeTestEioMathDiff (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2062 of 2062 tests passed, 2 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)


you mean other issue in desktop settings ?


object.isRequired


Complete changelog:
* GamePanel is repainted when changing settings
* Enemies use old nodes when new paths cannot be found
* Added `explored` to nodes
* Added target image for goal node


I like it. If you wanted, you could have the Hint class even live inside HintsDisplay.jsx and not be exported. I don't think I have strong opinions either way.

I'm also wondering if the "ref" approach is going to become more difficult now that Hint was extracted into it's own class :(


TeamCity pull requests :: netty-tcnative [Build 41](https://garage.netty.io/teamcity/viewLog.html?buildId=4967&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty-tcnative [Build 41](https://garage.netty.io/teamcity/viewLog.html?buildId=4967&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:00


PlantCompBaseClassAgain (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)


Well,, your pantheon-settings package has the same files as pantheon-settings-bzr.

I don't know what your pantheon-settings-bzr package installs, so it perhaps better to remove conflicting flies from manjaro-pantheon-settings.


I don't recall, I was probably filling the dependencies in using VS and just chose the same version without thinking that they should use a stable version. Should I revert this down to lowest possible?


Clear is good :smile: I'll update it.


I knew that this change would break existing code (that’s why I made it version 2.0.0), but I didn’t think about your usecase. What about adding FromSQL which is From without quoting (analogous to Map and MapSQL)?


Currently we are attempting to create a GraphQL web service (for a new web-client) exposing our current .NET business objects as our existing .NET-WPF-client sees them. So we have a large codebase of C# (1+-millions lines of code) where data is obtain though some WCF remoting from a server side.

In this setup, GraphQL is just one of many component using the data classes, and decorating them with new attributes is not desirable. So for this, defining DTO contracts with ObjectGraphType-derived classes with Field<> and Connection<> with Resolve() methods making the actual mapping is a great solution. I also believe this sepeartion of the contract will make it easyer to change to server implementation without breaking the contract with the client.

In the future, we will likely move in the direction of micro services with individual caches and own data classes. When we reach that point, the markup/convention might be the better option.

Just like some prefer handwritten WSDLs when doing SOAP, having the contracts kept separate has its benefits - doing some refactoring is less likely to accidently change your contracts with the clients. And in an environment where we are not the once developing all the clients, this has a real value.


111639676-UniqueThermostats (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2350 of 2384 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-98.57%25-yellow.svg)


Can we just include the jta dependency - the narayana-jta isn't great in a maven environment as its an uber jar.


@JordanCheney Can you post-commit review?


Fair point; I'm going to start with unqualified-decl-name.


silly me - thanks!


Is this intentional?


We need to support uncompressed files.


@euharrison mal, acabei fazendo tudo no master, fiz o commit nele mesmo, pra não perder tempo. Mal mesmo


tocolor return(ed) negative values, which causes this warning to appear in masses


Can we make this logic more generic by just splitting on `.` and then taking the item at index 0?


Not yet; I'm planning to replace the since SourceLoc for names with a DeclNameLoc, which is either a single SourceLoc or a structure that contains the base name location, paren locations, and argument label locations. UnresolvedConstructorExpr had to die first, so we didn't do useless work.


Sure that's ok


Personally I would not catch the exception. I might wrap everything in a try with resources on the context?


Yeah, absolutely. BTW, check out how manage-this-node handled settings, specifically under "modal"
https://github.com/onedr0p/manage-this-node/blob/master/views/index.hbs

I've added the modal to my local copy and think it is great, though perhaps the modal could contain/load settings.php with AJAX, as to allow form submission within the modal itself.

Something like the image below, but with a column for "Landing page" checkbox and a "Dropdown menu" checkbox on each item.

I noticed in your version you made each element draggable - it'd be great if that meant we're shifting the order of the links in the menu.

![image](https://cloud.githubusercontent.com/assets/5875228/12495626/11033636-c091-11e5-80c9-e2b290c437b6.png)


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


Fixes #44


@keithkfng yes.. it should be : string
@DRedDogPE correct it


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


I accepted the pull request, but it did not help anyway.


I think that this commit could be squashed into the first one, since it revokes a change from the first commit.


This procedure implements the new rules for calculating runoff reduction.


`census['runoff-vol']` is in units of inch-pixels (a volume).


:scream:


I think that Chapel defines a use-stmt rather than a use-expr.  I know that the compiler doesn't really handle this distinction very well, and even defines stmt as a sub-type of expr, but I think it would be better to define a
UseStmt, as a sub-type of Stmt.

Given the existing short-coming with Stmt, this ought to be pretty easy to alter if we agree it is worth doing.

Mike


resolves #21


`ma*rc*os` vs. `macros` ;)


Perhaps this should be `i686`? The Pentium 4 has SSE2 instructions, but is a 32-bit machine. Same for recent Atoms.


Testing the revert now - will land it shortly.


Can't say I'm a fan of the dangling deriving here. Also, need to make sure there's no newline at the ned of the file.


I think there should be an `else()` clause here which prints an error. Not all the world is a VAX.


I think instead of setting the entire thing to nil you should set the delegates to nil. Still seems to crash occasionally with that.
<img width="1398" alt="sqmrjgt" src="https://cloud.githubusercontent.com/assets/1211049/12496392/782b1ef6-c063-11e5-945e-4ea1cb51a53c.png">


woah. Why didn't this give me some error?


Is this `unicode` flag really necessary? My code should be 7-bit ASCII clean and work on any octet stream.


`lookup_bmp_storage(...)` produces quantities in units of m^3 / m^2 so that `lookup_bmp_storage(bmp) * bmp_cell_count` is in units of m^3-pixels / m^2.


This is JavaScript :-P It's just undefined.


please no whitespace after and before a `(parameter)`


What about LLVM's `clang`?


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 96](https://teamcity.labkey.org/viewLog.html?buildId=298527&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (2 new), passed: 275, muted: 1 Build time: 01:49:58
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

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

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

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

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

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


Maybe just
```ruby
env_vars["JAVACMD"] = ENV["JAVACMD"] || Utilities.find_graal
```
?


The percentage of the original runoff volume to keep after taking BMPs into account.


`__GENERIC` is missing.


r+


s/Cluster/Status ?


Is there any risk of the ref not existing? I think the answer is no, but thought I'd ask


I think I'd prefer leaving this as a `uint` and handling the error case where `r.MaxRetries()` returns something negative.


Successfully created the [token-image](http://token-image-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/b84feef63f0430bdffd6be807256cb32b701b911


merging graydon/stellar-core/bug-931-stuck-on-bucket-repair = b84feef6 into auto


graydon/stellar-core/bug-931-stuck-on-bucket-repair = b84feef6 merged ok, testing candidate = 0497bf14


Pourquoi un appel statique ? L'accès aux posts doit passer par la classe PostRepository, il en va de la cohérence des objets affichés. Le code suivant est équivalent (à condition biensur de créer la méthode findByAuthor()

    $postRepository = $this->objectManager->get('Dawin\LgLgRbBlog\Domain\Repository\AuthorRepository');
    $posts = $postRepository->findByAuthor($author);


Given that UseExpr (or perhaps UseStmt) is new, I wonder why there's a constructor that accepts a Vec?  Is it hard to push that upstream to the parser?

In a similar way I wonder if the constructor for UseExpr/UseStmt should take a BaseAST* for the mod?  What doesn't it take an Expr* (knowing that it could be a SymExpr or a CallExpr) but avoid Symbol?


If you do end up changing these (see my [comment](https://github.com/euank/amazon-ecs-agent/commit/63298785bd81137c35a0c3c07e8395eda43a66ed#commitcomment-15599566)), you can remove the `int` casting


`</a>` shouldn't be needed here imho


Can you explain a bit about this choice?


test


once a font has been settled on, remove other googlefont imports


I don't think this does the same thing does it? I want an operation that normally sets `JAVACMD` to not do that if `--no-java-cmd` is specified. Your code always sets `JAVACMD`.

This is for the GraalVM where `bin/ruby` doesn't need `JAVACMD` to use Graal.


Hmm.  Should it really be this constructor's job to deallocate args rather than the code that allocated it?  As a general
policy allocating in one "component" and freeing in completely different one is prone to leaks or double frees.


Very, very long lines. Also, I'm not too happy about putting third-party source code into this project which is not crystal clear on license (so public domain or BSD 2-clause like the library itself). The terms themselves seem reasonable enough.


Let's remove FindStats for now.


this is a test


Can we add a nice big explanation of what's going on here, something along the lines of:

For many of our hints, we want to include a Blockly block. Doing so requires that we have a Blockly blockspace. There's no real way to do that declaratively in our render method, so instead we provide this method that allows us to inject the blockspace, along with the block contents, after our render.

I might also call this something like addBlockContentsPostRender (or something less verbose that you can come up with that makes it a little more clear that this is more of a post render step).


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 97](https://teamcity.labkey.org/viewLog.html?buildId=298564&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:16:22


kflsdfd


This includes significant changes added in 87697c4ca2d3a41c66971fe4c1cfb5b64a74f302, so the backport needs to be done carefully.


```swift
inputStream?.delegate = nil
outputStream?.delegate = nil
```

However, it still will occasionally crash at another place


not unless React does something super strange


TeamCity pull requests :: netty [Build 1084](https://garage.netty.io/teamcity/viewLog.html?buildId=4965&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6386, ignored: 23 Build time: 00:43:00


@kmroz maybe do it only if userid is not empty?


sure thing


constructor addition in StorageEngine breaks the functionality..
Now when instantiating a new Storage instance one get the `constructor(options={}) not implemented for this storage engine` Error


What does this mean for cross-compiling? I understand the use-case for detecting the architecture of the _current_ machine, but a library like this should be easy to build inside a cross-compile environment. Would it be possible to override the detected architecture when using these buildscripts?


argh :)


Kthx


@kmroz I'm not 100% sure about whether this is what we want to do. @liewegas?


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 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-26.85%25-green.svg)


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1029.689df186dfab53f8da3dea053172124365c6e3d2](http://build.palaso.org/viewLog.html?buildId=68376&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:30


This is called a "computed include" and is [implementation-defined behaviour](https://gcc.gnu.org/onlinedocs/cpp/Computed-Includes.html)...


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.655.689df186dfab53f8da3dea053172124365c6e3d2](http://build.palaso.org/viewLog.html?buildId=68377&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:05:09


this is horrible


Personally I would've added myself as a contributor rather than name myself as the author. We've merged your changes upstream and have added you as a contributor. Thanks for all the hard work!


Maybe it is better to use `defer`. It was something added on Swift 2.0 and I didn't change the current code.


:shipit:


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


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.41](https://pixelated.design/build/viewLog.html?buildId=339&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


So actually, JS allows for this since the `var


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.99](https://pixelated.design/build/viewLog.html?buildId=340&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.96](https://pixelated.design/build/viewLog.html?buildId=341&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 267 Build time: 00:00:21


Hurá! :D


I've already fixed it in a branch where I am porting the package.  No worries.  I just noticed it and wondered if there was a reason.


It is, indeed, different internally.


LGTM


Do we have a use case for this ability other than tests?


The last release of this library was 0.2.0 :)


Added option in menu to use shields on disabled allies
Added checking for creeps/heroes in way for blockable skillshots (Pudge Hook, Mirana Arrow etc)
TemplarAssassin PsionicTrap is now enabled in key combo by default
Re-enabled AbilityOverlay for OgreMagi
Reworked Omniknight Purification usage
Fixed TemplarAssassin sometimes attacking creeps from Meld
Fixed Rubick stealing spells when already stolen spell can be casted
Fixed SentryWard usage
Fixed exception coming from AllyPlayers class, when in 2nd match
Fixed hero starting to move after auto using ability
Fixed damage calculation for Wisp Overcharge
Fixed damage calculation for TemplarAssassin Refraction


Les fichiers temporaires de l'IDE sont-ils importants pour le projet ?


I think that value and cmd both need to be range checked. Otherwise this could allow for arbitrary execution of code.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.665.689df186dfab53f8da3dea053172124365c6e3d2](http://build.palaso.org/viewLog.html?buildId=68378&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:23


Uh... We haven't done anything for it so far... It was like a few months ago... Oh wait, it was last year, lelele. So uh, we will start it one day.


Sorry, both of those comments got cleaned up in later code passes


The BaseAST is still an open question, though


Maybe add JavaDocs to the overall class?


Pas sur qu'une contrainte d'égalité sur une relation multiple fonctionne correctement


Okay, fixed. This code is no longer apart of the PR.

On Thu, Jan 21, 2016 at 4:21 PM, Luke Hoersten <notifications@github.com>
wrote:

> Can't say I'm a fan of the dangling deriving here. Also, need to make sure
> there's no newline at the ned of the file.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WraithM/haskell-snippets/commit/6895eb02f3e9c02fb41ebacdd8beef1f4e7588dd#commitcomment-15599482>
> .
>


service should be range checked.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/103965221


Some minor issues:
* `tmr.c` has a couple of implicit declarations - misses to include `user_interface.h`.
* doing so triggers a `"/*" within comment [-Werror=comment]` error for `app/include/lwipopts.h` line 793 - additional start of comment needs to be removed
* `app/platform/platform.h` misses a declaration for `int platform_tmr_exists( unsigned id );`, probably on line 262

I'll follow up with some functional testing tomorrow.


Should we pass just a baseURL like you are doing other places for links?


One commit with actual functions outside of standard.
Possible bug.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 97](https://teamcity.labkey.org/viewLog.html?buildId=298532&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31 (17 new), passed: 260, muted: 1 Build time: 01:48:36
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testImmuneSpaceStudySubset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testNavigationDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testBackDoesNotRemoveFinderFilter: <no details avaliable>

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


here we should throw an exception.


here too.


:zap:


This is a direct translation of the old statement pretty much. There's no longer a default credential chain object laying around, so we have to conform to the new signature to get one.

I'm going to remove the WithRegion bit because, on closer inspection, that's unused.

The SDK didn't have a problem making this a little less simple, I think, because it's not really used in a normal case. We use it because we use credentials outside of an SDK session (acs/tcs) and otherwise the SDK would hide this change from us.

Do you have an alternative to suggest or other concerns?


It would be nice to have an ARM option here too (for instance one that fits the Raspberry PI), but I guess I can add one myself. I have the necessary test platforms.


ggc


I think you're right, I think it can be switched to an Expr*.  My original thinking when I created this code was that I'd want to just store the ModuleSymbol being referred to.  But then I thought about the case where you're accessing it through an extended list, A.B.C or whatever, and I had for some reason not leaned towards a SymExpr for the original case, though that was what I ended up using later.  I will clean that up


This change needs to be fixed up with the previous commit, `Result: Show filename of affected code` :)


unack


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND 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)


well, this is a big comit....


If we pass baseURL above, then we don't need to pass client here.


Ah I see! I saw that host and port were set at the end of the function wether or not they were declared thus becoming globally set. It probably won't cause any issues regardless. Thanks!


# JosnExample
json不同的数据解析方式。
分别使用 org.josn、com.alibaba.fastjson、com.google.gson、com.fasterxml.jackson
展示了json数据不同的解析方式，其中以阿里巴巴的fastjson和jackson性能最好，
作为android开发者学习案例分享。


nit: s/form/from/


As mentioned above, I think that the split between i386 and x86_64 is not general enough. Also, I'd do feature detection instead of platform detection, that is, checking for `__SSSE3__` and `__AVX2__`. The instruction set being 32 or 64 bits is not what's important, it's which vector extensions are available.


Could you maybe change the commit message?
You just added the `location_repr` function, in fact `Result` does nothing more now^^
I think a simple one like
```
Result: Add location_repr()

This new function prints a human-readable location for the result.
```


yes this is correct. thanks for that


А где тут bag?
Переменная ```$targetClass``` и так содержит полный путь к класс вместе с namespace илаче ошибка вылетела намного раньше.
```php
 $query = $targetClass::find();
```

Можна канечно поиграться с include и справоцировать некоректную работу валидатора но я не думаю что кто-то будет стрелять себе в ногу.


Yes, it's there to test Sentry or other error reporting services, as well as the error UI shown to the user. I realised that I had no way to test sentry in production without intentionally shipping a bug.  So I did.  Should be in the readme, though, you're right.  Will add that.


fast-forwarding master to auto = 0497bf14


Yes, but it doesn't actually set anything globally! For instance:

    function foo() {
      if (false) {
        var a, b;
      }
      a = 100; // a is still set locally because of weird JS hoisting!
    }


Considering raising >> 4 for chunks to >> 5. This would make chunk blocks the size of region files (http://minecraft.gamepedia.com/Region_file_format) at the cost of extra RAM for the null values.


lol ok just parsed this. i have no problem with this structure:
```
export_configuration = ExportConfiguration(
    tables=[
        TableConfiguration(
            table_name="My awesome form",
            columns=[
                ExportColumn(
                    item=exportable_items.items[1],
                    display="Header for Question 2 column"
                    expand_column=False
                )
            ]
        )
    ]
)
```
(the one you posted). my issue comes from not generating this on the initial request and sending it to the browser. i think we should send this structure to the browser with sensible defaults. then the page will modify the properties and we just have to call `.save`


The SDK change was made so that `-1` could be used to indicate a default value.

I'll convert back to uint and handle that appropriately.
Good call!


See: e03a064586d50725d9d2accf7c45096c7bab4d0c


James, you verified that this error message does get properly saved to the resource in cattle, right?


HBIRE_vector_take2 (amirroth) - Win64-Windows-7-VisualStudio-12: OK (2045 of 2057 tests passed, 503 test warnings)

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


its a debug gosh


saw approval from graydon
at https://github.com/raymens/stellar-core/commit/6ea25caf2e4a0b78e84fea704cc1bdddb3481f89


merging raymens/stellar-core/history-windows-fix = 6ea25caf into auto


raymens/stellar-core/history-windows-fix = 6ea25caf merged ok, testing candidate = b152f3a6


@jmaguina1202
@robertoaban
@jeanCarloRM
@carledriss

Please review.


Oops. No.


So regardless of the target architecture, AVX2 is always enabled?


Need to remove the newline at the end of the file here. Doesn't look like the diff is showing it's addition but it's showing that there's not a lack of a newline.


Should this not check `DISABLED_PROPERTY_NAME` first?


Noooooooooooo... Now I have to update a whole bunch of other stuff to eclipse that :(. Why was 4.0.11 broken?


whats this?


This commit does not take into account the stuff in `/test`, which is built with its own Makefile. That directory would also need a CMake makeover for optimal cross-platform compatibility.


you can't be hardcoding stuff like this in a public repository


Ergens bovenaan in je script:
```javascript
var API_URL = 'https://bierdopje-api.houtevelts.com/GetShowByLinkName/';
```

Dan kan je hier:

```javascript
var linkName = /shows\/(.*?)$/.exec(currentURL)[1];
$.getJSON(API_URL + linkName, function(info) {
/*
  info.name;
  info.runtime;
  info.seasons;
  info.episodes;
  info.showstatus;
  info.score;
  info.favorites;
*/
  // create tableRow
  ...
  // append tableRow to tableData
  ...
});
```


you're committing stuff with your UWaterloo git credentials, it isn't synchronized with the actual GitHub account... might wanna look into that


arg, ok I was concerned about it. I wasn't able to reproduce the crash with the last checkin (using your test project). Kinda of tricky to handle it with all the queue/context switching. I will keep looking into it.


When you reach that view, the email and name are not displayed. Is there something you know about erb that could explain why? Think about the `<%` syntax.


> I think that this commit could be squashed into the first one, since it revokes a change from the first commit.

I didn't do a sqash down, because there are quite a lot people who get upset when you rewrite public history and I didn't know whether you are one of them :wink:. I'll sqash everything down to one commit after we agreed on a final implementation, ok?


argh, a little bit too fast for my PR ;)

also: since you do not overwrite `install` a simple
```yml
cache:
  apt: true
  pip: true
```

should cut it.

https://docs.travis-ci.com/user/caching/

(P.S.: apt caching is not yet active for container builds.)


This line: I'm not sure why, but `require("../lib/transformation/plugin").default` is `undefined`? Just `require("../lib/transformation/plugin")` works as expected.


You said:

> My buttons are stacked on top of one another

but it seems you fixed that problem :+1:


We can't end up parsing a compound name as a prefix unary expression.


Should be fixed in the next 1.5.2 build.


This could probably be put into the accuracy event instead of hard coding it, right? If so, I'll do that later


@trown ^ only Ceph should be required from EPEL right?


Aww. They bumped up the version in the newest build and Coherence complains if we use an older version.


LLVM?


I think bitsPerSecond should be here too


Do you have a way to reproduce the crash with the latest code? I'm having trouble with the test project reproducing it now. I have run the project about 30 times and it worked properly each time.


dangerous... we should decode to unicode at all times for all strings, that way you are always sure you're working with unicode, even if you already know you are going to use it for utf-8 string later. If you later change the code and need the same object again you might forget that you didn't decode it to unicode. Get the point ?


And update this note


Since I'm here, I'll clean it up. We shouldn't have string literals effectively anywhere in Sema.


nit: Benamse die return-parameter am Besten (z.B. "(width int, ok bool)"). Aus der Signatur sollte, soweit das geht, klar werden, was die parameter tun. Wenn das nicht geht ohne endlos lange namen, dann ein godoc-comment.


If the original authors of these languages can come forward they can be credited. Unfortunately I do not know who authored these translations because their copies of hedit were removed from the community.


Ebenso


@liveforeverx With this commit, a Postgrex is officially out with DBConnection support and I would love to bring Mariaex. I think the best way to visualize what changed is by using something like this:

    g diff v1.1..HEAD -- lib/ecto/adapters/postgres integration_test/pg/all_test.exs integration_test/pg/test_helper.exs test/ecto/adapters/postgres_test.exs

Write the diff out and check the files that matter to you. They are not a lot of changes, most of the work is in the driver itself.


80-characters per line.


Remove `Is `.


According to the docs, that requires "language: python", which is impossible as the language is already set to "cpp" and multi-language projects are not yet supported.


this shud just be info logs. same comment for following error logs.


\o/ yay @Bytemarks!


nice


Added HostInventory.cs


Rebase all the things! :) I don't consider pull requests to be public history. I'd never rebase the master branch, but development branches are fair game. Good on you to not assume, though.


Attention, le code contenu dans les commentaires HTML est pris en compte lors de la génération du rendu des templates.


nit: "maybeTruncate"? truncate sowohl als Funktions- als auch als Parametername ist weird


Added new file HostInventory.cs to Entities namespace


You need to call `fixPkgconfig` later to fix paths in these files.


Add a comment // in milliseconds.


// comments are not C89 (explicitly set in Makefile), so this now fails to build for me.


Oh that *requires* python? Looked just like an example to me but you are probably right.


Added inventory property to Host.cs


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/103970824


@GendoIkari @vivalafiestaaa trovato.
Le loro mamme.... :facepunch:


Eh, sure.


@AndriySvyryd  - Isn't there supposed to be 4th argument for ModelCacheKeyFactory?


P.S.: for your (commented) i386 build I filed a whitelist request [here](https://github.com/travis-ci/apt-package-whitelist/issues/2284) some days ago but it usually takes a while to add it.


shit


This isn't used (which means we should probably get rid of the whole thing).  See the identifiers section on the page at https://openlibrary.org/config/edition for what needs to be edited (by an admin).


Sure, can you do that and open a new PR :smile:? I'm busy with other things right now.


I also filed one a long time ago, it's still pending!


No, I couldn't find official document for it, but it worked.


argh, probably due to the manual control that is needed with `suid`... I had filed some others last week that were rather quick and could be resolved automatically.


This looks like it reverts 17409f1193a19b7e57003542c36b6da3ad49f950 which also claims to fix CSS include paths.


But these are all semantic differences that affect the types of what we find, rather than the name we use to find them.


Yes. Could you add it?


The map_addr should used to store the address. table_hdr is meta-data.


fast-forwarding master to auto = b152f3a6


Sure. Running build with it.


For the purpose of documentation: Mike and I talked a little about this off line.  While we view 'use's as statements, the current AST version of stmt did not seem as natural to inherit from as expr, so we've agreed to leave it as UseExpr for now and to consider referring to it as UseStmt when the AST gets revamped


fixed.


OK.  Would be great if you added a note that this is relying on undocumented features.  Might be useful for someone trying to figure out compatibility issues later.


@jdalton Whoops, meant to do a pull request. PTAL


I could either have it error out, or just remove "key" on here since that is a subset of "keys"


This leaks a goroutine. You need kill this serve routine at the end.
plugin has a Close() method, and that methods causes the blocking Serve() method to return with an error.
So, you can just `defer p.Close()` and that will cause p.Server() to return an error and the goroutine to exit, BUT you hsould not log fatal. At most, log.Info()...and make the wording non-scary, like "Existing serve() for driver %s. Reason %s, driver, err"


Actually it was ngageoint/hootenanny#162 not 163.


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


Really? Are you using the included node server? It seems to happen more often if the websocket is in the process of handling messages. So the test server sends a bunch


I added a convenience method, mThread.getIdAsLong()


I don't see a newline on my local version of `module.exports`.


That would clearly be a mistake on the part of the caller, think you should throw an exception. Honestly, I'm not super excited about this method change, you are making one method do two different things. Would prefer to break it into two methods, then factor out the common parts.


Is the latest message ALWAYS the one sent to you? (eg. If you sent the last message, it'll still return the last one received?)

Also, what about threads where the only message is from you? (eg. I text you 'Hi' and you've not responded yet)


Never do this! `from X import *`. It makes life difficult for the future you, when you'll wonder where the symbols came from. It's never necessary, but it's ok in tests.

Lots of crappy languages encourage this nonsense (e.g. Ruby, which is a nice language hidden inside a crappy one). A lot of proficient coders do this, but it's very unkind to future maintainers. Google's style guide prohibits this (and the equivalent in C++, `using namespace foo`).


Whoops, I totally missed that :flushed:. Well, your last release was some time ago... Which version do you want me to put there? 0.2.0? 0.2.1? 0.3.0? What are your API stability guarantees as per [cmake docs](https://cmake.org/cmake/help/v3.0/module/CMakePackageConfigHelpers.html?highlight=COMPATIBILITY) (see section regarding COMPATIBILITY)?


there was a breaking change with the syntax


Fixed


It's worse here because `pulp` is non-standard.


I wasn't able to get the curses clear to work. Possibly because I'm not using curses for output yet?

In any case, it'll be changed once curses is fully implemented.


..or "pip-tootls", but yeah.


Successfully created the [build-speed](http://build-speed-static-docs.pantheon.io/docs) environment. New commits to this branch will automatically be deployed.


If you get this running, PR it and I'll trow it on Curse and give you a cut of the points if you want.


Are you using the included server that comes in the project? Because it still seems to be crashing fairly often for me


Ma questa è una prova che aveva fatto anche gendo. Mi pare di aver capito
che diceva che non c'era nel tuo codice, ma forse mi sbaglio.
Il 22/gen/2016 12:05 AM, "Tommaso Visconti" <notifications@github.com> ha
scritto:

> @GendoIkari <https://github.com/GendoIkari> @vivalafiestaaa
> <https://github.com/vivalafiestaaa> trovato.
> Le loro mamme.... [image: :facepunch:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GendoIkari/termodino-gui/commit/e7f4137726adaa4eaa5be6a1021bb372381b8790#commitcomment-15600387>
> .
>


So, this commit message should have been updated. Sorry it is misleading.

Just to be clear the intent changed,  we decided to keep `np.inner`'s behavior the same. Instead `np.dot` raises a `TypeError` in both cases of type mismatch.


Another idea would be to add code around the AddZCatchVictim() calls. There are two. One right below your edit and one in the game context:
```shell
> grep -nR "AddZCatchVictim" ./src/
./src/game/server/gamecontext.cpp:726: leader->AddZCatchVictim(ClientID, CPlayer::ZCATCH_CAUGHT_REASON_JOINING);
./src/game/server/gamemodes/zcatch.cpp:165: pKiller->AddZCatchVictim(victim->GetCID(), CPlayer::ZCATCH_CAUGHT_REASON_KILLED);
```


Stray typo here is causing the tests to fail


Jovem, comente mais o código, deixe ele mais explicativo sobre o objetivo principal da inserção e da busca, esta muito obscuro para usuário leigo. no mais parabéns.


.ui-content {
    margin : 5px 50px;
    text-align: center;
}

.ui-collapsible {
    margin : 5px 400px;

}


fixed


please do not push configuration files. You can add a .gitignore entries to avoid it automatically.


Should be `os.path.join(os.path.dirname(os.path.dirname(__file__)))`


should add a `defer rpcclient.Close()`


i386 can be synonymously used for the x86 resp. IA-32 instruction set see [this wikipedia article](https://en.wikipedia.org/wiki/IA-32). I didn't choose the term (as it is defined in `TargetArch.cmake`) and didn't feel the need to change it. However, as you might have already noted it means "all x86 32-bit architectures" in this context.


Pareva anche a me che l'avesse fatta, ma forse altrove, boh.
Nel mio codice non c'è infatti, ma forse le condizioni sono diverse. Ciacciando un po' di documentazione neanche loro lo mettono da tutte le parti, però a volte lo usano!
https://facebook.github.io/react/tips/communicate-between-components.html
https://facebook.github.io/react/tips/expose-component-functions.html


- use camelcase to name variables
- Use the interface Map<Item, Double> instead the Hashtable in the type of the variable.


Amigo, coloque uma breve introdução de como o mongo se comporta no fedora, e comente mais o código em suas etapas, deixe eles mais intuitivo para usuários leigos.
no final tente colocar mais umas duas distro com links para configuração se possível, no mais está de parabéns.


@vetusko Hi! My current installation of OpenEyes with oe_installer and the sample database does not have a class "OphInBiometry_Imported_Events", and hence crashes when I try to view a patient's file, eg accessing URL http://localhost:8888/patient/view/19434 . Which branch of the OphInBiometry repository are you using, which would have this class? Or do you recommend to use the branch `master` of OpenEyes, instead of the branch `IOLMaster` currently checked out by oe_installer?


Shorten `logger` to `log`, to make your life easier.

A good convention is

    log = logging.getLogger(__name__)

To get hierarchy, move your modules into a `Cogent` directory and create `Cogent/__init__.py`. Then update `setup.py` to install the package.

Any btw, you should not need `find_package()` in your `setup.py` ever. You will have a small number of packages, so just list them. All modules in the package are automatically installed.


Altough you the Post new entry was apply, the mocking, faking and stubs was not use


sip gan


Should we be using consistently using either ```pk__in``` or ```id__in``` and not both?


Commit should say: Using the injected `container` is deprecated in Ember 2.3. Changed mixin to use the `getOwner` helper instead. http://emberjs.com/deprecations/v2.x/#toc_injected-container-access


This test is correct :+1:


@orangejenny Here and below I used `col-lg-*` like the rest of the BS3 PR, but I also noticed @biyeun's comment. Just a note that you'll need to do what you did there here too (or ping me to do it).


You can use @Test at method level instead of extends TestCase


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 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.7%25-green.svg)


This commit name made my day :)


Excellent. Thank you Stephen!

Mike

On Thu, Jan 21, 2016 at 4:00 PM, Stephen Bae <notifications@github.com>
wrote:

> Hey Mike,
>
> I've already seen your stuff yesterday. All is good ;)
>
> Looked into the whole using background declaration in CSS to display
> images VS using <img> tag.
>
>
> http://stackoverflow.com/questions/492809/when-to-use-img-vs-css-background-image
> The reason I was attempting to use a <div> then hook up the image file
> using css is in that post.
>
> and the image wasn't being displayed correctly because it was missing background-size:
> 100% in the css.
>
> Stephen
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mroman09/basic_personal_website/commit/057436ecd27ce1d3f4581053950574f961f19513#commitcomment-15597583>
> .
>

--
Regards,

Michael D. Roman


it is recommended to use 4 spaces instead of tab


I think this should be `flatMap`. Then the comment would be fine.  We could write to any table with a single distinct node partitioning.


I haven't thought this all they way through, but maybe there are *some* we want to force them to implement? What do you think?


Les merges non testés, ça fait planter le code


bummer


What is this exception? Why are we swallowing it?


I see you don't have a `setup.py`. That's fine. I'd still put this into a package. Then you don't have to worry about uniqueness of filenames.

I'd also create tiny root-dir python executables which run the code within the package. Or create shell scripts which run it like `python -m Cogent.my_module $@`. The reason is to make the *contents* of your executables as testable as possible. At least you are using `if __name__ == __main__`, which is good.


I'll make it one layout for now


this differs from the `properties.example` var, and i actually think `commcarehq.host` makes more sense in this context


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


What is the behavior you would like to have? My idea for a fix would have been more in the other direction (assuming NetBSD): call lwp_create with signals blocked and before unblocking them, install a new signal stack. If I ever get around to change the kernel, that's kind of the operation I plan of providing atomically. pthread_create would get an attribute for specifying the signal stack of the new thread, _lwp_create effectively too.


@eparis Are you sure these aren't needed anymore? Bugs properly fixed in godep? @vishh still having problems with docker godeps I think.


### changes
- added some chart icons
 - uE6DB-chart-pie.svg
 - uE6DC-chart-bar.svg
 - uE6DD-chart-column.svg
 - uE6DF-chart-stacked-bar.svg
 - uE6E4-chart-pivot.svg
 - uE6E5-chart-area.svg
 - uE6E6-chart-stacked-line.svg
 - uE6E9-chart-stacked-area.svg


> So regardless of the target architecture, AVX2 is always enabled?

No, it's an user option which is only selectable if the target architecture is x86 or x86_64. Think of CMake more in terms of a scripting language like shell script than a makefile. With you runtime cpu feature detection SSE support is merely a question of executable binary size and compiler support. It is theoretically possible to detect SSE compiler support, but it is quite a lot of work and the commonly used compiler like MSVC/gcc/clang do support them. However if the compiler doesn't support the selected SSE feature level, the worst thing that will happen is that the build will fail.


Will Pride [6:31 PM]
that property is actually suppoed to point back at HQ

[6:32]
so we can get a restore


Ah ok. I don't use or have node install, but I installed it and I am now getting the issue. Thanks.


Nice!! Just gotta merge this up over in devsupport.


This check is actually not required. `QThread::start` will do the right thing.


Learn something everyday!


fixed


AFAIK the newer versions of godeps are supposed to handle this automatically. It still fails in my case though.


CB-90 #comment does this one work !!!! #In Development


Settings affected by hotkeys are stored in config.ini and loaded upon injection.


This last example's a bit of a weird one -- I wanted to demonstrate that youtube-dl includes a _ton_ of options for precisely selecting certain videos from a playlist. But maybe it's not worth including; I'm not sure.


Is this the same as `QuoteCurreny.Symbol` ?


The build script itself does support targetting the ARM platform, but since I didn't add any ARM specifc user options there currently is no elseif(). If someone chooses to add NEON support to the build script he can simply add the then required `elsif(BASE64_TARGET_ARCH MATCHES arm)` and add the user options.


The big picture works and is usable. There are no doubt details that are off..


these storage account dependsOn's can be replaced with a single "storageLoop" (it will depend on all SA's in the loop to complete); example here: https://github.com/Azure/azure-quickstart-templates/blob/master/201-vmss-linux-nat/azuredeploy.json line 206


Note `win32` in file path.


why


@rWardy  Thx for your help :D and thx for fixing my error xD


If nothing is using this code, maybe remove?


@Yomguithereal This change is a React style preference of mine that's difficult to lint for.

The general idea is that I want a `render()` method to read cleanly top-to-bottom.

Any conditional stuff gets done inside the `return`, and if it's too complicated then it gets extracted into a method which is defined after `render()` in the source order.

The idea behind this is that if you stop reading halfway, you can't have missed anything


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


Could place constant on the left to avoid assignment errors if you accidentally only use one = sign


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

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


I moved this to http://trac.buildbot.net/ticket/3416 rather than on an obscure, 18-month-old commit :)


Could we avoid conditional compilation in the test libraries and use RuntimeInformation.IsOSPlatform instead?


It seems we don't need ```all()```


I don't think this will compile


@Aaron1011 Formatting problems.


It is... rather strange. I like it though, even though it lengthens the page. I see some value in the matching/rejecting at least, though it's already specialist behavior. Not sure for now.


This change is good. Ideally this entire block of stuff should be replaced with `from_be_u32_ptr` and `bswap_u64` and friends.


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


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1734 of 2100 tests passed, 2 test warnings)

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


This is a remnant from my CMake template. If you use Win32 APIS or filesystem APIs on Windows this will mitigate some issues; I didn't check whether your code would be actually affected by one of them. On the other hand this won't have any further notable impact on anything (OK, except *you* explicitly use the UNICODE preprocessor definition), so my suggestion is to just leave it there (I'll remove it though if you insist).


@PranavMathur Can you push the changes to the website?


DOH!  missing .


Helps if remember to set my username and email... done by me.


leaving catch blocks empty is generally a bad practice


> Not all the world is a VAX.

Sorry, but I don't understand :cry:.


Thanks, I removed the check.


It pulls every ten minutes
On Jan 21, 2016 3:43 PM, "Jake Ohara" <notifications@github.com> wrote:

> @PranavMathur <https://github.com/PranavMathur> Can you push the changes
> to the website?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Thunderbots5604/Thunderbots5604.github.io/commit/99928afaa67f14cd49e9117f663e4d78761286db#commitcomment-15601031>
> .
>


No problem.  I've set a server up for my kids and they keep telling me stuff, so I've been trying to work my way round the code. Thought i'd give a fix a go :)


@chilimatic thx for your input !!!


OctreeTestEioMathDiff (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)


Thanks! keep them coming :)


I think this not enough. Reaction shoots should be blocked too.


updated readme to reflect information to be placed in this repository.


https://github.com/hammerandchisel/erlpack/blob/master/erlpack/sysdep.h#L68

pls use dis.


Good to follow the pattern, thanks for making this.

Just thinking out loud here, no need to change anything :). Maybe a quick refactoring we can do reorganize these data filters. Maybe it's best to define a `SecurityDataFilter.Null` implementation that can be the default, then we could implement other ones, like for building equity bars we filter out some non-tradeable ticks, that could be an implementation - `TradeableEquityDataFilter`.


_newSate_?
Shouldn't it be newS**t**ate? :P


avatarki? fajnie :)


FromSQL would solve this one case, but I have a niggling worry that it's a symptom that something is not quite right with the API - I'm just not able to put my finger on a specific problem though. But it seems to hint at longer-term API bloat as a possible risk. The beauty of this product is its minimalism - and my worry is that's possibly at risk.

As with From, there is a related problem with Joins - more important because table name abbreviations are common in joins.

Some other niggles include:

 * Order should be OrderBy
 * Group should be GroupBy
 * Return should be Returning
 * (there might be others I missed)

Maybe v2.0.0 should really be a more strategic change to solve these issues, without throwing away the overall simplicity.

I find the SQLite pictures helpful in piecing this stuff together (https://www.sqlite.org/lang.html). But the complexity it shows is far greater than sqlbuilder will currently manage. The challenge is to find the optimum set of features useful in most DML cases.

At its heart, a table is referred to by a name or by an abbreviation; the name might include the optional schema prefix. So modelling this using structs is a possibility - but might quickly get ugly. (https://www.sqlite.org/syntax/table-or-subquery.html). A simple way of supporting 'AS' succinctly might be a fruitful way to proceed.

I'll leave these thoughts in the air. If I were less busy right now, I'd fork it and have a go myself.


On Fri, Jan 22, 2016 at 7:17 AM, Brandon Dreager
<notifications@github.com> wrote:
>
> I wasn't able to get the curses clear to work. Possibly because I'm not using curses for output yet?

Yes.

> In any case, it'll be changed once curses is fully implemented.

You can use ANSI escape code for the time being, calling external
command has some cost to pay.


ok   $scope.rcntv = 2; this isn't going to be helpfull if DB gets reset or anyhow we change the RCN id, if this is specifically for rcn then subdomain logic should apply under global scope so it's going to be easy to put company specifics everywhere in the code.

the employeeFullName it's actually very handy we should consider moving that to global scope so we can use it in other parts when employee name's are required.


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2352 of 2389 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-98.45%25-yellow.svg)


Well be sure to remove it eventually before we release 0.29.


@ScrapCodes (can't open an issue here so I'll just comment on this commit)

I just saw the upstream fork https://github.com/sbt/sbt-pom-reader is on version 2.0, should spark upgrade its version of sbt-pom-reader?


We don't usually discuss topics on github commits, where hardly anybody sees them.  Instead we use the forums described at https://golang.org/wiki/Questions .

I think the NetBSD kernel should work the way other systems work, including OpenBSD and FreeBSD: a newly created thread should not have a signal stack.

The Go runtime does in fact do what you suggest: it calls lwp_create with signals blocked, then it installs a new signal stack.  It's just that the code that installs the new signal stack has to handle two different cases: a newly created thread, and an existing thread created by C code that is calling a Go function.  To avoid changing that function at this late stage of the Go 1.6 release cycle, this patch drops the signal stack on a newly created thread, so that the existing function sees the state that it expects for a new thread, namely, no signal stack.


Nick, should we change the comment here to read something like "//has parents therefore is matched" and "//has no parents, therefore is unmatched"


WB man! I decided to go a different way with K10
i will try to avoid caf patches (a lot of them dont help)
i will try to avoid MM related patches (weird things tends to happen)
dont add things if one part of it is broken.
keep zram the same with google ( google is updating it slowly)


you need to add `find_package(catkin REQUIRED COMPONENTS hironx_ros_bridge rostest)`


Indicate here that this last case only happens when `allow_duplicates` is true, and maybe give a hint that this happens in recurrent (or otherwise parameter-sharing) architectures to make it less abstract.


What's the thinking behind this?  I would have thought we should either we have a single `location` constraint just for the whole group, or one for each item in the group... so why the service and group, but not the VIP?


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


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND 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)


Ah, there we are.  We can get a ModuleSymbol passed to us by BlockStmt::moduleUseAdd, which gets called in a number of places in the compiler.  It seemed simpler at the time to just do the SymExpr wrapper around the module in the constructor rather than insert that wrapping at all the places where moduleUseAdd was called on a plain module symbol.  I'm willing to bite the bullet and do it if you don't think that laziness is justified.


boring words


Looks good!


These machine driver schemas should not have POST. Or is there some weird reason you need it like because you are POSTing a machine and machine has a driver config in it, the driver config needs to be POSTable?


Just for my own understanding, this assertion should be true even if you move the call to `get_application_call` outside the `with`, right? The application call was made inside the block, and its `training_mode`ness should be constant from then on.


Yes, this was the issue I explained to you before I went on FTO.  All the libraries get loaded first, then the LWRP classes are somehow emptied before being loaded.  So `Chef::Resource::PacemakerLocation#controller_only_location` only lives for a very brief time.


Maybe we'll also have to add a map from currency -> units, I imagine Ag is in troy ounces?


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


Steve,
  I definitely cut down the size of the file, some of the methods are longer but there are a lot fewer methods.  Since I have the cell information, I let the cell determine what base color to apply to a given square, this eliminated the separate `Space` and `Target` classes I defined.  I am not sure how to cut down this one case statement as I decide to color the occupant
```ruby
def render_occupant(cell, row, col)
  unless cell.not_occupied?
    case
    when cell.cart_on_target?
      render_oval(row_pos: row_pos(row), col_pos: col_pos(col), color: special_dark_grey)
    when cell.man_on_target?
      render_oval(row_pos: row_pos(row), col_pos: col_pos(col), color: special_crimson)
    when cell.space_has_cart?
      render_oval(row_pos: row_pos(row), col_pos: col_pos(col), color: @app.darkgray)
    when cell.space_has_man?
      render_oval(row_pos: row_pos(row), col_pos: col_pos(col), color: @app.crimson)
    else
    end
  end
end
```


nope, 1 issue :
pet has a spellbonus
for example  dk spell attack ~220, pet will attack ~163 (not 50%)


Do we really need all of these?  Isn't the one on the group enough?


very cool graphics


will do


No worries! The reason I changed the author here was because I was thinking about making my version a competitor to yours since most of the "core" was re-written ;) But in the end, I just rebased my changes onto your repo, it's better if we work on this together :) Adding me as a contributor and reverting this is good!


[![Lobed bowl](http://api.artsmia.org/images/122512/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122512)


[![Rest on the Flight into Egypt  and the Miraculous Field of Wheat](http://api.artsmia.org/images/50/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=50)


[![Lamentation with Saint John the Baptist and Saint Catherine of Alexandria](http://api.artsmia.org/images/547/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=547)


[![Virgin and Child in a Landscape](http://api.artsmia.org/images/4035/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4035)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (383 of 383 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)


@montellese: I'm adding a setting that looks like this:

```
Number of keyboard players: 0-4
- Configure player 1
- Configure player 2
- Configure player 3
- Configure player 4
```

where each "Configure player" action is visible if it's number is <= the number of keyboard players. Is it possible to have a "greater-than" visibility condition?