I see your perspective, I think a better way to express this would be to put it in the constructor. I think it's a little non-obvious for the getter to provide defaults, as the getter isn't really used to parse, but is just used to retrieve the value. I don't like conflating the semantic by having it provide a default. Alternatively, you could put the defaulting in `NotificationHookConfigurationParser`, if you truly think it is an aspect of parsing (vs. common to any method of constructing a configuration).


I'm wondering if it would not be better to use the file:
https://github.com/BVLC/caffe/blob/master/python/requirements.txt

It might be closer to the intent of the Caffe maintainers.
See also:
https://github.com/NVIDIA/DIGITS/blob/master/docs/BuildCaffe.md
It might help you with the list of packages (but this is for the NVIDIA/caffe fork).


I believe this is specific to our implementation given that the Java implementation from Jersey uses the dateTime() method from ISODateTimeFormat, which is yyyy-MM-dd'T'HH:mm:ss.SSSZZ.
See:
http://joda-time.sourceforge.net/apidocs/org/joda/time/format/ISODateTimeFormat.html


This should probably be inside the Dockerfile, no? With a `ENV`.


Why don't we move the npf & npcap projects into a single solution?  Is there a reason to keep them separate?


@@ -5,8 +5,9 @@ This software is licensed under the GNU GPL v2.0


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


Updated CNAME


Is this for the ipython notebook? If yes you should add a comment here.


... ... @@ -14,6 +15,6 @@ Ensure that win10_privacytool.py and hostslist.txt are both present and located


Totally forgot about it. Updating.


This doesn’t have to be a string; could be `0o755`.


Fixed #4


I don't understand which problem this patch is solving.


board::readDipSwitch always returns 11 1000 1111 0101 no matter where the switches are set, hardware is working fine


lgtm!


`mysql` is deprecated, use `mysqli`


Do you mean the resource has *not* been inserted?


update the type information here
- `NSDictionary<NSString *, NSIndexSet *> *`
- `(NSArray< _ASHierarchyItemChange *> *)changes`


This is a pile of ****


Will push the MGNet patch soon, going to try and make it optional first since it's only a partial requirement for non-GS games.

The method is Texture2D[]::.ctor, so it may very well be a 2D array then. That'd surprise me though... I wonder why it would store data that way, unless the Content/ layout prefers it.


Can also put the `self.fail` here I guess


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1535 of 1535 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)


Interesting. Didn't know you could do that. Will try this out. :smile:


`application/json` should be used when returning JSON


Newline


I am not sure if I'm the only one experiencing this issue, as I have only a few OS X machines I can test on, however `os.O_CLOEXEC` is not available on any python 3 I have installed. http://bugs.python.org/issue26343


check `isset($_REQUEST['auth'])`


@misternu Thanks.  This also made sense after tracking down & playing with your PR.


Closes #1436


Shouldn't this have been changed to `flyway.baseline()`?


:smile:


Enig, men kan ikke rigtig få det til at virke. Har tilføjet en .ignore fil. Har kørt git rm, men de er der stadig. Grrrr.


Validate $_SERVER['REMOTE_ADDR'] even though it isn't technically user input


Should probably have some way of auto-pruning the database with TTLs or max-history


OSMeasure (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


First you go to a img url. then go to [goo.gl] (goo.gl) to shorten that url. then use that given url without the https part (example if url is test.com/test.png, you shorten it and it becomes something like https://goo.gl/efhwhef then you take out the https:) You need to shorten it because the max nickname is 15 charactors. then what you do is type` [url]` in the nickname field with the `[]` encloseing the url. Press play. then you will have that skin. If you want a name, when you die after doing this, delete your name in the box and replace with a name (the skin will stick)


You didn't call the function, the printed value was its pointer. Correct form is: `board::readDipSwitch()`. Also please don't name variables `blah`, it makes it hard to guess their intended usage.


Great catch! Thanks! Fixed.


`$poll` :wink:


Neat, I wasn't aware of that!


que le hicieron a yelpcito...


test comments.


Format? Normally this defaults to JSON, why doesn't this? should we default to plaintext from now on?


Do you win the prize for tiniest change? Like 10 pixels....


Default to JSON or Plaintext. Seems like plaintext is more commonly used in my personal tooling


OSMeasure (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


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


It doesn't but I'll make it so.


how many names?
we should throw `Mohsen` and `Romeo` in there for fun! :laughing:


There's another case in #2882 where this refactor would be beneficial.


Could we just mathematically calculate it using the rpm of the motor and the gear ratio of the gearbox, taking in to consideration the connections between the motor and the shoiter?

Sent from my Verizon Wireless 4G LTE smartphone

-------- Original message --------
From: Kennedy <notifications@github.com>
Date: 02/11/2016  2:11 PM  (GMT-06:00)
To: Team5427/robotics-code-2016 <robotics-code-2016@noreply.github.com>
Subject: Re: [robotics-code-2016] Degrees turning/tilting and constant speeds (a2c825a)

is there a specific type of encoder that we'll be using (I know next to nothing about them) and how many of them will we have, and where will they be placed?

---
Reply to this email directly or view it on GitHub:
https://github.com/Team5427/robotics-code-2016/commit/a2c825ac116c476f0b30d1fb49c3af5068bb2b48#commitcomment-16043858


Yes, I think it makes sense.


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 398](http://ci.infinispan.org/viewLog.html?buildId=35265&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for c6125793f862e6297a7282bea36fbdd187c55a20
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:08:19


Neither 'numpy' not 'scipy' are used in the code. I think this is a leftover from your debugging.


Hmm. Kan se at et git pull ikke henter filerne, så tror det virker. Ellers er du velkommen til at rette op på det;-)


null loc these, qdel of mob takes a second and slows them down.


I think you did a comma once, which is smaller :)


ready_aim_fire (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


What difference is there in adding data before or after the load? Unless there is an attribute named `indexed_points`, shouldn't both cases end up with the same object?


Here's an example with a nicely resized interval for an isolated SWR (to the left, original red, new green) as well as merged SWRs with extra "non-SWR" stuff in the resizing step (right, two reds become one green). The detector is plotted in blue.
![image](https://cloud.githubusercontent.com/assets/4942743/12992847/e197187c-d0e4-11e5-88ee-cc5d98ac085b.png)


You are a puts debuggerer


This seems to having had lost `PropertyPlaceholderAutoConfiguration` when moved over from the removed class.


comma wins. I'm looking for a period

On Thu, Feb 11, 2016 at 2:28 PM, cgrandin <notifications@github.com> wrote:

> I think you did a comma once, which is smaller :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cgrandin/hake-assessment/commit/fc362175bf7e9f7f8fe5a84818cd7b64ffeb0852#commitcomment-16047188>
> .
>


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


Argh, I changed that word in at least 10 places. I kept writing pool everywhere, lol. Looks like I missed one.


sounds good. also sort of related. how can you access papplet from an intentservice?


ready_aim_fire (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (983 of 983 tests passed, 0 test warnings)

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


This whole section is not useful, aside from maybe the doctest, which should catch the bug with eigenvalues you're seeing on your system. Normal code should use '.eigenvectors_{left,right}' methods; the doctest should move into unit tests.


Around 200 so far, will add another 800 tomorrow. I will make sure to add our names as well. :smile:


le port a changé pour server.js car on utilise c9


I use a trick to shrink the G-buffer to one render target @ 32bpp and re-use the buffers between Renderers (this is kind of bad for the driver since it can't parallelize the operations), so things are not as bad as you might think, though still bad as you can tell by the framerate drops.


I don't know why this line was removed. I don't remember removing it. Let me revert it.


Fixes #871


So, the reason why this is failing is that input_adaptor doesn't do anything with plain old hierarchies. It just returns them. Maybe it should do something different (like perform selection?) but how that works depends on whether the hierarchy was constructed with PMI1 or PMI2 or whatever.  So right now `input_adaptor(hier)` returns `[hier]`


Fix #1


Actualice la libreria yelp de node y se cambio a promesas, por eso el cambio. Aproveche de corregir el lint de coffee (basicamente largo de lineas)


'assume' is never used.


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


"../commands/" looklslike magic string should be in costatnts string definition.
all your string should be like this.


You should use 'matrix_taylor0(m, x, x0, 1)' instead; it's faster.


I changed it to be consistent with what's in the table. Fine to leave though if that's how it's usually presented.


That's exactly why I've added https://github.com/zalando-stups/taupage/pull/149


we use spaces OR tabs  ?


Interesting, I like it.

You should updated the `composer.json` php version, since you've updated to have php 5.4 array syntax. Maybe even go to `5.5.9` since 5.4 has already met it's [EOL date](http://php.net/eol.php) (3 Sep 2015). Or if not then `>=5.4.0`, I'm not sure what kind of backward support you want.
```
"require" : {
     "php" : ">=5.5.9",
    "twig/twig" : "~1.2|2.0"
},
```


How were you running the command?  This works for me when it's something like:
```
bundle exec bin/ingest-etd /path/to/zipfiles/etdadmin*.zip`


Because I already have a mongo install on my workstation that I don't want to mess up. And I want to be able to run multiple dev environments in parallel.


Did you change this to make git_410 example run faster with the default seed? I'm pretty sure that's just a coincidence. For now the better option will be to add seed as a command line parameter.


I'm thinking we should pass an existing `$firstname` and `$lastname` into this function so that we can generate an email address that matches the provided names instead of random names inside the function. This is just so our data looks a little more realistic and clean.

we can add a few more `$username` patterns as well. For example pick randomly between username delimiter (`-`, `.`, `_`, `+`)


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


I did not seethe triggerInvoice (dryRun or not). Is that because we never implemented it in java?


@TuringTW   Oops!  I missed that one when I put that module together.  Can you submit a PR to the roger's repo?


R


nice!


Is this correct? Looks like it should be

COPY ./src/seed-test-data.php /usr/src/build/seed-test-data.php


Does this test actually work? I mean:

    sage: bool(1+eps < 0.5)
    False
    sage: bool(1+eps >= 0.5)
    False

I think you'll need to take limit epsilon->0 to perform a test like this.


Don't we want to also use the `toLocalDateDefaultToday ` for all the subscription endpoints?


Oooh, OK. That makes a lot of sense and is super good to know!

Well I think this project is basically done. I'd say this is really my first "real" program, so I'm very grateful to have some input on it.


The `acctsessionid` and `acctuniqueid` should be auto generated. We should add a parameter request for the NASIPAddress as well.


Some inline comments on lines :)


We need to make a distinction between what is needed on the machine of someone who is working on libsndfile itself and someone who is simply building libsndfile from source.

Someone who is working on libsndfile itself is working from the git repo and should have autotools installed.

Someone who is building libsndfile from the source does not build from code in the git repo, but rather downloads the source code tarball from the libsndfile website. The source code tarball includes the file `src/config.h.in`.

Does that make sense?


Fixed


reads good. will test next week


End with .


Can we make the "total" param more descriptive, i.e. "totalgb"


Oh scratch that, got it. We need to pass null to entitlementApi to force an event with `utcNow`... got it...


Oh, yeah


Can't we also get rid of this step since clrdbg will have a runtime next to it?


Can we also request this data from the CLI. If not specified, no quota or quota date should be recorded.


Can we just do all files instead of needing to list each type?


`anniversary_day` should be allowed to be given from CLI, and if not specified default to the day of month of the "startdate".


This is a temporary thing, right? The reducer needs to be able to figure this out on it's own.


Performing a ToLower here breaks the test case expecting to get an exact match resource for mixed case strings:

https://github.com/pweil-/kubernetes/blob/51a8f8f4798c5d37c37c04c6348e6e31a5c38678/pkg/api/meta/restmapper_test.go#L404


This commit LGTM.


As noted in the already pull request: the logic to load stub files should be changed instead of requiring copying identical stub files around.


Run as administrator. Go through the tool's dialogue to add or remove blocking entries from your hosts file.

Have fun.


Correct, I am changing to an array of objects with params 'name' and 'value'.


Thanks! Will do.


I guess the real question here is do we care?  I reset my rebase branch to run more tests and this is one of the things that was failing right off the bat


Is this a misnomer? Should be `GOOGLEAPIS_PROTOS_DIR`?


Can you modify this line to `if (packageVersion('htmltools') < '0.3.3') && getOption('htmlwidgets.copybindingdir', default = TRUE)`? This will ensure that things dont break at my end, since I am probably the only one who uses this option in production. This should not change anything for others.


Yeah but now translators will not be able to updates the messages files quickly


unfortunately there is a difference due to poor code in the customer model :disappointed:
there's an `_afterLoad()` function in the model that tries to fetch the indexed points if one isn't already set on the model. The only way to avoid re-loading the indexed points object, is to set it on the object, before the call to `load()`


Fixed


In general, I would prefer more verbose bash variables. `BIGTABLE_PROTOS_DIR` vs `BT_PROTOS_DIR`, etc. It makes it a bit easier to read for new/old eyes.


(8) comme Data Manager, je souhaite pouvoir choisir les colonnes que j'afficherai sur la page publique


Region is what A16 calls 'state or province'.


LGTM.


113360729_HumidifierOASys (Nigusse) - x86_64-MacOS-10.9-clang: OK (2082 of 2082 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


Agreed, same here. I posted about this in the other thread. IMO we should consolidate over there.

https://github.com/opentk/opentk/issues/339


Fixed throughout


Issue has been made, see #221


Maybe like this?
The property is considered specified when its value is not a {@link Nully} value.
When the property is not specified, the value of `defaultValue` is returned. The latter defaults to `undefined`.


Hello dc42 i tryed to rebuild you detector but it stopped at line 121 any suggestion how it come?


is this how we can avoid `Inactivity Milestone` triggers?  ... i forget what exactly we look for there.


Also removed check if Wire.begin() was already called, as it could potentially create incompatibilities with arduino cores that do not use that variable. Even if Wire.begin() is called multiple times in the beginning, it should not cause any errors.


Maybe like this? Note the important different in what is considered a specified property.
The property is considered specified when its value is not `undefined`.
When the property is not specified, the value of `defaultValue` is returned. The latter defaults to `undefined`.


missing `;`?


The property is considered specified when its value is not a {@link Nully} value.


Arnab we need Beehives because we need the wax for the Hardened Leather. Otherwise i have to change the recipe using Forestry wax for it.


Other changes are good i think.


All fine apart from those jsdocs corrections.


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


Thanks for the explanation. Didn't know that. :+1:


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


Do we care about the unit of measure at this point?  For the calculation we will require a unit value, but the type of unit (liters, kg, etc) is irrelevant for the calculation.


Thanks!


al references to deploymentId in this file should be updated to serviceId, right?


I am referring to the `dp0` in the service path, _NOT_ `%~dp0` in the logfile output path.
I could submit a pull-request to make it 100% clear of what I mean, if you so wish.


Unfortunately, there is a check before transfers are created. If there is no customer logged in most rules won't work at all. I am basically setting the customer session to make it look as if someone was logged in.


Commented out code (read: dead code) is bad IMO. The only exception are the places where you keep the code you would want to bring back soon as a reminder TODO, particularly if it is easy to miss without a reminder. For the places where a reminder is not needed, for example here, where not bringing back this code will cause the tests to fail we can always bring this code back from version control. After all, that's what VCS's are for and so much effort we put into keeping our software development versioned :)


Very interesting findings. I guess (2.) and (4.) are optimised away by the compiler, so no calls are actually made...


lgtm


can we just change this to `"zone.js": "~0.5.14"` then you won't have to release a new angular beta each time. also related https://github.com/angular/angular/pull/6875


return + returns


These checksums don't match the checksums of the builds of others


No, since "Horspes" has a greater priority than "Horses", it will be at the top of the list. So when poll is called, it should return "Horspes," which it does. @EricCrosson


except history command


I am not sure if @mediaquiries fall into layout or module rules


do you mathematically understand why this works? ... I can't wrap my head around it at the moment :sweat_smile:
it's the end of the day here though. Might need to look at it again after a lot of coffee tomorrow morning :smile:


Looks like you checked in Apple directory metadata crap.


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


What's the best way to edit the diagrams?


Where do you add hopses? Is my question
It's never in the queue


This section seems to search for a pair of eigenvectors that just happen to be be dual... but that's not a sufficiently good. For example, if `A` has an eigenvector `(1,1)` and `B` has `(1,1)` -- their product will be `2`, so you'll miss them, but if you'd just divide one of them by `2`, you'd get a perfectly good pair (a scaled eigenvector is still an eigenvector).

For another example let's say that `A` has an eigenspace with basis `[(1,0); (0,1)]` and `B` has an eigenvector `(1,1)` -- you'll find no dual pair with this simple search, but again, `(1/2,1/2)` and `(1,1)` will form such a pair (and `(1/2,1/2)` is an eigenvector of `A`, because it is a linear combination of it's eigenspace basis).


The comment is still out of date, however.


`eps` is unused.


added localStorage.clear() to automatically clear data.


this looks very slick!!!   ... I like your code architecture :grinning:

I want to look a bit closer at the code for your `data` and `executor` models. I'll have to resume this review tomorrow!


Missed a comma.


can't, They may use phpFastcache in middle of their project, however this one have to be on top of their setup / config files


Oh, ok. I see my failure to use git properly

Also I noticed the Travis build passed for this commit, which Was my first clue I was having a stroke


**# lag?**


Thanks @insidegui ! ;)


@ssk2 and @brndnmtthws  -- if I understood Sunil correctly, the only differences between using haproxy-marathon-bridge and marathon-lb (besides more functionality) is that marathon-lb is a Dockerized application that includes HAproxy (so you don't need to install HAProxy yourself) and you don't need to set up a cron job yourself. I've tried to make those changes on this page.


So does that mean we should make it configurable or is this whole thing only for the Jersey implementation match?


ternary operator [(allowed in JSX render functions](http://facebook.github.io/react/tips/if-else-in-JSX.html) ) allows for some clean rendering conditions


why you use the same rbac for differents commands ?
that's not how we did, pls check the wiki and learn how to code properly.


What about `SRCS`? Maybe use `ARGN` to keep the arguments similar (as in https://github.com/ament/ament_cmake/blob/master/ament_cmake_gtest/cmake/ament_add_gtest.cmake#L29-L30).

Otherwise the macro should also check for unused arguments.


I had the idea that you were going to change this to not modify the given array.


Thanks for that, @non7top ! :)


Oh yeah fixed #5


This closes #7


Fixes #1


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1535 of 1535 tests passed, 0 test warnings)

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


I Think we should be presenting the site as a product and not as a project eg
Team 1A

Also HTML  Tag is not supported in HTML5.


what ? Why you didn't use a variable declared in the correct source ?
Pls check the wiki, we are only english server


Does this mean that the earlier versions are not supported anymore?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1535 of 1535 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)


Is `mm` not equal to `m`? Why another variable?


an examle:
1.) Somebody builds a video addon for playing dash videos which are not encrypted and need no license keys at all). His addon doesn't pass any license keys and the inputstream.mpd knows from absense of the keys that no decrypting should be done. inputstream.mpd simply reads the dash file and passes packets.

2.) Sombody else writes an video addon for mpd files with a known key. He passes the license key through ListItem and inputstream.mpd addon knows that the data comes encrypted and has to be decrypted using the key passed. inputstream.mpd does the same as on 1.), but decrypts the packets befor sending to kodi.

There are plenty of different decrypting mechanism available, because of this I need a licensetype and a license key.

No keys / no type given means: do not decrypt at all.


`SearchFeature` determines whether to display results or a spinner. This way, avoid `this.state.loading` down to child components.


Can we exclude `EDHmacClient.xcodeproj/xcuserdata` from source control?


Nice spinner, @ml242 !


What do these `partial_fraction` calls do?


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 4](https://136.243.151.173:4433/viewLog.html?buildId=750&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) is now running


http://stackoverflow.com/a/5075283/421049 says you can use simply `\r?\n`. I would try it first---I only did the Google search. :wink:


Can this be a struct? Or was this done for Objective-C compatibility?


this doesn't seem to be in the API Span anymore, so remove?


Ok. I just did a test run here. Cleared all caches.

With the start/end marker Grav reads the config files correctly and compiles them correct. For example the form config generates this PHP array in the cache directory:
```
<?php
return [
    '@class' => 'Grav\\Common\\File\\CompiledYamlFile',
    'filename' => 'user/plugins/form/form.yaml',
    'modified' => 1447205012,
    'data' => [
        'enabled' => true

    ]
];

```

if I remove the start/end marker, the Yaml is not parsed correctly and Grav compiles configuration file into invalid PHP arrays like this as example:

```
<?php
return [
    '@class' => 'Grav\\Common\\File\\CompiledYamlFile',
    'filename' => 'user/plugins/form/form.yaml',
    'modified' => 1447205012,
    'data' => [
        0 => 'enabled: true
'
    ]
];

```

So Grav on my Windows machine is definitely broken when using the native extension without the start/end markers.

I have no idea if this depends on platform, line-endings or the way the PECL extension was configured for compilation.


AWESOME!

On Thu, Feb 11, 2016 at 2:12 PM, Heikki Linnakangas <
notifications@github.com> wrote:

> Yeah, should help a lot with that. On my laptop, running ANALYZE on a
> table with 500 columns and 100 rows took about 30 ms after this patch, and
> about 10 s before it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/greenplum-db/gpdb/commit/f995c59f349917a99407e3811d9b317d5d77b257#commitcomment-16046795>
> .
>


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 4](https://136.243.151.173:4433/viewLog.html?buildId=750&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:01:03


TeamCity Ppconsul :: Build (Linux) [Build 101](https://136.243.151.173:4433/viewLog.html?buildId=752&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


Yeah, actually as written it looks incorrect. Line endings are CRLF or simply LF. You're checking for CRLF or simply CR.


Hah, good catch, thanks!


@weiweizh Might be an issue with El Capitan, as the plugin works fine with Sketch **3.5.2** on Yosemite. Will look into this…


TeamCity Ppconsul :: Build (Linux) [Build 101](https://136.243.151.173:4433/viewLog.html?buildId=752&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:01:03


Will be done in `datetime-work-for-release-0.17.0` branch.


Investigate changing map to show locations shown as pins on map


5420-OSCM-Docs (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2077 of 2084 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


We do have `triggerInvoiceNotification`, but it doesn't take a date.


@chneukirchen this initializer makes it awkward to:

```ruby
Rack::Response.new *@app.call(env)
```

because the arguments would get passed into the `Response` object incorrectly. Since changing the order of these arguments would be insane, would a core rack contributor consider a patch for a method that looks something like this?

```ruby
class Rack::Response
    def self.from_app(resp)
      status, header, body = resp
      new body, status, header
    end
end
```

The middleware would look something like:

```ruby
Rack::Response.from_app @app.call
```

or I could detect if `Rack::Response` is initialized with an array with an arity of 1, I assume its from a rack request. If its an arity of 3 I'll assume its initialized in the way its currently implemented. That would look like:

```ruby
Rack::Response.new @app.call(env)
```

If there's an appetite for such a patch I'm happy to implement and open a PR.


Yup! Same with invoice (except when we need *something* for the effective date of the adjustment or credit).


We're using the format described here: https://github.com/bazaarvoice/jersey-hmac-auth/wiki/How-it-Works#clientserver-protocol .


Done please review


nice!


```$ jshint --verbose  .``` shows this errror:
```
public/javascripts/index.js: line 62, col 9, Creating global 'for' variable. Should be 'for (var item ...'. (W088)

1 error
```


You might want to make these `internal` so they can be unit-tested.


Ha ha, yes I know that feeling. :smile:

Actually, this isn't as easy to explain as I originally thought, lol. Let me give it a try. Let's say we have 4 possible actions, each one with 25% change (equal weight). Now check this map out:
```
action1 => 25,
action2 => 50,
action3 => 75,
action4 =>100
```
If I generate a number from 1 to 100 (which is the sum of all possibilities) than I can return the first action that is smaller or equal than my random number. Of course the weight can be different for each action.

I basically did the same thing, but instead of adding the weights up directly I left the map with the real percentages to make it more readable and added up the weights in the foreach loop while checking if I already reached a big enough value for the current action.


Yah i'm not seeing that with the PECL YAML parser on my Mac.  I guess we'll have to put in a check to see if the `---` is already at the top of the file.

Can you confirm if the `...` at the bottom is really needed?


This doesn't work on earlier versions of Python


r=me


Indentation more to the left.


Hain? :P


It's probably worth documenting that the API key *must* have been added first.


ahh didn't saw it, good catch


Ahh yeah, I probably can combine this abstract class and the CreateKernelSobel (will be renamed, as you already mentioned) class. This is a leftover from before the gauss-related kernel restructuring (the structure is the same).


D


Triying to compile this after this commit: Error:

CMake Error at src/CMakeLists.txt:23 (add_subdirectory):
  The source directory

    /home/ElunaTrinityWotlk/src/LuaEngine

  does not contain a CMakeLists.txt file.

-- Configuring incomplete, errors occurred!


Why not just change this to ``len(testChunk) // 6``. The ``//`` operator does integer division in both Python 2 and 3.

That way you don't have to explicitly cast it to an ``int`` in 3-4 other places.


Please [document](http://nshipster.com/swift-documentation/).


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


You can add unit tests for the scenarios that there is no content.


If we end up using cookies or query strings, this can be ignored. For now, it doesn't work as the for loop is never entered


You are pretty close.


not sure how I feel about this use of Rollback(); is the idea that it gets called in case someone forgot to call rollback explictly? And if a commit happens it's ok, because it just fails?


Doesn't this mean that the packages for Jessie will _not_ contain the sysvinit script in addition to the systemd service file?  Jessie changed the default to systemd, but sysvinit is still actively supported, so it would be neat/helpful if the package contained both to maintain compatibility. :smile:


Good points.  Possibly we could remove the cloning being done in insert() since I added it to add()?  We could get object(s) in insert, graft objects with $loki and meta in add, and depending on cloneOption store those objects or clones into collection data[].  Since the objects passed -into- add are the same ones the user passed in we can have add() return true/false and from insert() just return back the same objects they passed in.  Possibly in future 'possibly breaking' update we might even refactor insert to return true/false or some other result status even more similar to mongo.

If you would like to refactor operators, I should think that is fine.  I believe 'operators' was originally a collection level variable but making global to module proved to be significantly faster so I just left it as is otherwise.  Since that should not affect database serialization or api, go ahead.


Fixed.


this is so pervasive, might be nice to have a convenience method so that someone can just type r.Table(connectorConfigTableName) or something.


So this change reintroduces the problem I was trying to fix.  If PCMK_ipc_buffer is smaller than max, the final 'return QB_MAX(max, global_max)' will ignore the global_max set by the environment variable.


Smart man. I think it is retarded when people do this little hack lol.


I am cool with either the constructor or the parser itself. I partially put the logic in the getters to avoid changing the behavior of `toMetadata` which keeps the input values as is (if there was no input, it spits out no output), but I doubt that behavior is important or even that useful. I don't like putting it in the hook impl itself because it is cleaner IMO to let something else decide the config values and not have to check and deal with null (with the exception of the case where the entire config is null, which I would prefer to put in the parser also but it doesn't work that way unfortunately).


5420-OSCM-Docs (mitchute) - 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 know rst is traditionally for Python projects but I saw jthompson18 and the872's projects and wanted my readme to look as nice as theirs. As I am not intimately familiar with Markdown I figured it would be simpler to adapt using theirs as a template! Thus readme.rst


Good idea. I didn't know about the // operator. It's good to know :)


Personal reminder: It should use PHPUnit exception except.


Missing "err" parameter:
`db.Eviction.save(data, function(err, evictions){...}`


Line 11 to 28, it was mentioned that we might use a forEach possibly.  Googled for documentation but did not find anything.  I am hoping will get some more feedback from class or the TAs.


This commit is not needed. Just leave it here as is.


No qualms on my side, I simply wasn't familiar with what expectations might have been set the the values in that config object. From some of the code it seemed to be specific to up address, so I set this up to err on the side of preserving existing config as much as possible rather than refactoring the value to include http vs https as well as port and host.


closes #19


Why did you do this ? now it is failing with phantomjs 2.x


Reverting due to https://github.com/commercialhaskell/stack/issues/1777


Gd


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


Correct, it is there around line 176


```$ eslint --rule 'guard-for-in: 2' .``` shows this error:
```
/home/aneutrino/git/deployer/public/javascripts/index.js
  62:5  error  The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype  guard-for-in

✖ 1 problem (1 error, 0 warnings)
```


Ghostsur


*First iteration on lights
*Adjusted object transformations


Thank you for clearing this up, aggelos.

Liam Please read above and ☆Robin can set you back to your Super Admin. Aswell as that be removed from Panel and Telnet as you are currently on them, Thanks again for clearing this up aggelosQQ
Last Edit: Feb 7, 2016 at 8:09am by Sam

More can be found [http://immafreedom.boards.net/post/1464/thread](here.)


Now I see which part of code will generate the error message for Q1 when option is off. I would say it's a reasonably error message. However, I agreed with Parth that this is a UserRemoteException: SYSTEM ERROR: UnsupportedOperationException; what we want is a DrillUserException.  In other words,  the current exception is thrown in execution time, which is not ideal.

Can we check the maxdir function in UnsupportedOperatorsVisitor as well? It raise error when 1) it detects maxdir in WHERE clause, AND 2) the option is off. That way, we check and catch in planning, in stead of execution time.

Also, just curious, we do have reduction for Project [1]. I'm wondering why it will fail for the query reported in DRILL-3944 (maxdir appear in Select, which should go into a Project). In other words, seems maxdir() in where clause would be go through the same logic as the in in SELECT clause, when ReduceExpression rule is applied. But apparently, it did not work that way according to the first failed query reported in 3944.
{code}
select maxdir('vspace.wspace', 'freemat2') from vspace.wspace.`freemat2`;
{code}

[1] https://github.com/apache/drill/blob/master/exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRuleSets.java#L101-L103


I don't think we need to dispose.
That said we could change that to `Task.Run`, not sure if it makes much difference, @mikeminutillo ?


Love this if statement inside the output string. It is definitely more idiomatic Scala than the version I wrote.


What if this was an object that was embedded in all the repo structs, so that you could call
`r.executor(tx)` like before, instead of having to pass along r.dbMap each time?

eg.:

```
type Executor db.Map
func (e *Executor) func excutor( tx Transaction){
// same stuff essentially
}
```
and then when creating Repos:

```
type ClientRepo struct {
  dbMap DbMap
  Executor
}
```


Rollback is no-op if you've already called Commit. So yeah, it's to ensure one of the two is always called.


According to the [doco](https://msdn.microsoft.com/en-us/library/kbf0f1ct(v=vs.110).aspx) and the [source](http://referencesource.microsoft.com/#mscorlib/system/threading/threadpool.cs,c4540b859da63a4a,references), the return value is either true or an exception is raised.

In other words, the return falue will never return false


For now it's only used for the datastore protos. We're hoping that #1384 will be resolved in some nice way that makes this all moot.


this sucks


This seem a bit awkward that convertToJSON only prepare an object to be stringify.  I would expect this function to produce a json string that can be sent to JSON.parse(s) to get PlotState back.


-UseBattlEye
#1893 @Bletch1971


Just remove this whole function.


`$self[key]` does not work because `$self` is a pointer, I rather use the arrow notation (`$self->`) than the start (`*$self`) in this case. Returning a reference is a legitimate technique to expose internal structures ready to be changed, but these referenced objects need to live as long as the container object is alive to guarantee memory consistency. It's very convenient in the `entry` type case since together with the `operator=` set you can have `e["keyname"] = "stringvalue"`.


I have no idea why we made this a set. We only pop one value off each time anyway.


Guys, can you do a release including this fix? Thanks a lot!


Første commit


Moved down


@DaleMcGrew @nf071590 Yeah this does not have to be enforced. You can still push straight to develop branch.

This should really help you out: https://help.github.com/articles/fork-a-repo/


Probably the only truthful commit message here ;)


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


Toto bolo asi omylom revertnute, ze?


@tomgarcia This is way too complicated. Why not just use label and checkbox? http://stackoverflow.com/questions/6293588/how-to-create-an-html-checkbox-with-a-clickable-label

If you put an image in a label, it should be clickable and select the related checkbox.


Doesnt fix #888 but the raklib error doesnt halt the shutdown process and with this change I can get a cleaner shutdown. I don't even know if the latest pthreads has a kill function unless its down to my php7 build

Before
stop
[23:21:26] system> [CONSOLE: Stopping the server]
[23:21:26] emergency> Crashed while crashing, killing process
[23:21:26] emergency> Error: Call to undefined method pocketmine\network\rcon\RCONInstance::kill()

After
stop
[23:28:04] system> [CONSOLE: Stopping the server]
[23:28:04] debug> Disabling all plugins
[23:28:04] debug> Unloading all levels
[23:28:04] system> Unloading level "lobby"
[23:28:04] debug> Removing event handlers
[23:28:04] debug> Stopping all tasks
[23:28:05] debug> Saving properties
[23:28:05] debug> Closing console
[23:28:05] debug> Stopping network interfaces
[23:28:05] emergency> RakLib crashed!
[23:28:05] system> Stopping other threads
[23:28:05] debug> Stopping CommandReader thread
[23:28:05] debug> Stopping AsyncWorker thread
[23:28:05] debug> Stopping AsyncWorker thread
[23:28:05] debug> Stopping AsyncWorker thread
[23:28:05] debug> Stopping AsyncWorker thread


this might be a good time to make this and the connector config repo consistent with the other repos: namely, it never creates transactions, it expects them to be passed in.

But that might mean creating a  manager object and whatnot....


I assume these console.logs are temporarily, and that it would be removed when TODO is completed.


Was this removed on purpose? The comment on [line 13](https://github.com/clowdy/laravel-raven/commit/24bbda1d8ffa6ee9a85d1e19b88ad33764446775#diff-201ecbbaaafc94e96b2124c0c23f0960R19) still says it's attached, and explains why I did that.


This should be just "Loading data".


@stanidesis this looks good to me.


Why not skip migrations all togehter for SQL lite?


Bear with me and give me a day or so. I will investigate this today or latest tomorrow and let you know.


looks like you dont factor in only checking if: config > special > `enable_https_verification` is set to on


これ定数で良いかな？


あと配列じゃなくてハッシュでは


roger that, we'll stand by and wait for you investigations.  The new Grav 1.0.10 release is out with this change, so you might have to manually edit that line in Toolbox until we have a longterm solution that works for all situations.


The diff indeed looks strange. But the code is perfectly ok, uses early exit
for better readability.

Should have been part of my previous PR, looks like I forgot to split this
commit.

On 02/11/2016 09:29 AM, Daniel Schürmann wrote:
>
> Why was this moved below the if condition?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uklotzde/mixxx/commit/d14c9eeafdca5c1073444364a625cb36626ed682#commitcomment-16026922>.
>


None of our test cases attempted to access any invalid entries in the List, but I read somewhere that l.apply(n) will return Nil rather than an error if you try to access an invalid element. (Just an interesting thing I ran across.)


zone.Add(new HtmlString(zoneContent), Position);


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (983 of 983 tests passed, 0 test warnings)

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


Current nightly LunaLua builds are broken, giving errors whenever something tries to play a sounds.

Pretty sure the changes to MusicManager::addSound in this commit are the cause.


good catch! (it was just `go fmt` making the change)


Code is more consistent. e.g. for functional tests, you create a connection, migrate to latest, then do your tests no matter your database.


I'm confused. Isn't this always going to evaluate to `Some 0`, given the definition of `on_the_go` at line 42?


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


Haahaaa, the if else indention drives me crazy to. Its super difficult to line up your parenthesis. This function actually was much easier if you used a filter instead of if else and pattern matching though. I just used one line:

iList.filter((i: Int) => i % 2 == 1)

But its cool you figured out pattern matching this early :)


should be something like NewTranslatingExecutor, also add comment what it's for, what translate does.


"I really like the idea of adding articles, but I think it needs more than just the models before I can make a decision. After all, Rails is all about MVC working together, not just the M portion in isolation. If the views and controller are in working order, then I'll pull this into my project."


Do you want this in this PR or is that something we could do separately?


Sorry for that, because I had no time to continue works and I sent everything as-is. Anyway, Kevin fixed build by my request. Now I'm working on that progress and because I currently a bit ill, I am at home.


The build will fail with this test case, as there is an expected error on line 16.
The test case should be moved to "nullness-extras" and an expected output file should be added there.


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2117 of 2117 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


add doc string for this one like

`NewExecutor returns a new executor that wraps an exist one, using the translate function to transform the query'


yup, oops


TestCustomCommand (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2072 of 2073 tests passed, 0 test warnings)

 * 547 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.

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


It's pretty cool!


Please use sanitizeHTML for consistency with the remaining code.


does this mess up iOS?


Maybe

```go
type dbConn struct {
    *gorp.DbMap
}

func (c dbConn) executor(tx repo.Transaction) gorp.SqlExecutor {
}

func (c dbConn) quote(table string) string {
    return c.DbMap.Dialect.QuotedTableForQuery("", tableName)
}

type clientRepo struct {
    conn dbConn
}
```


do these exist yet in the skin.json?


looks like you can just use loc here?


The value of max in one call shouldn't affect the value of global_max, because that would unintentionally affect subsequent calls. global_max should simply be the configured value, with a reasonable default if not set properly, and I think the new version gets that right.

Maybe:
return max? QB_MAX(MIN_MSG_SIZE,  max) : global_max;


Please use sanitizeHTML for consistency.


I haven't read into this much, but isn't this a case for I size?


i dont like it


is this what you want?


where does this go?


Missing escapes of shell arguments.

Even if these values come from the database ... Don't trust them to be sane. Cf. Persistent XSS/RCE.


test


huh here is interesting


I did it very similarly but with an if else. I think you could get rid of the extra function definition and l::myTails(l) line if you just modified the code to:

l match {
     case Nil     => Nil
     case x :: xs =>  l :: myTails(xs)
     }

you can use " l " inside the match in your return statement. Did I understand your question correctly?


Closes #1, even though just temporary, but the better one is planned so this can be closed


Not sure if we should change this symbol to English style?

――


Can't you just download the pre-compiled Qt binaries from http://www.qt.io/? Won't they run on Wheezy?


yes, It is just the starting point for Cindy.


Sweet indeed. Thank You


Anyroad does seem to be a term but even in this dialect I don't really recognize it. May be worth changing?


closes #51


If that case is impossible, doesn't that mean that `!seen` should be included in `on_the_go`, instead?


@blankered we reserve the word VTOL for vehicles that can transition between fixed wing and multirotor. I would suggest you name your mixer vp_quad_x and/or vp_quad_h


Is `else:` missing a semicolon? :)


Hello Mark. You've got intrigued enough to respond however :-) Thanks for writing.
I've only had a few minutes to drop this comment during my working hours. And I admit - it's incomplete.

Answering your question first - it [line 543 - clean workspace] still gets executed [despite I use git-exe and not JGit]. *Still*, because of line 541 claiming it's a workaround for JGit. Git-exe I use does not complain, yet workaround is still applied.

More context and the use case.
I've stumbled upon a (potentially recurring) problem (story ahead, skip until ### if bored already):
I had an important legacy configuration script I needed to change. As it was the only existing copy lying somewhere on a dinosaur era computer, I decided to put it into git SCM. I created a dedicated git repository in central SCM server. Then I edited and tested it on a workstation, pushed changes, logged onto the target host, pulled from there and it was done.

Then to automate things (###) I wanted to establish a job on Jenkins that would get poked by a web-hook from SCM server. It should run on the target machine and update script using SCM plugin. As I tested such a job, it turned out that the first execution (when there is no repo in the workspace yet) wipes out the whole workspace before cloning. I tracked that behavior down to this pull-request. That's why I commented it here.

I know this is a hermetic case. I'm not really complaining about it. And I'm not requesting anything.
I realize nobody will bother, maybe me next time I'll run into this. There are plenty long forgotten very important legacy scripts with only a single copy existing out there in the world. Maybe I get irritated enough to make it configurable myself.

Fun fact for you: did you know, there is an option for java openjdk7 named [CompileTheWorld](http://stas-blogspot.blogspot.com/2011/07/most-complete-list-of-xx-options-for.html#CompileTheWorld)?


@oldergod background should be excluded already so checking exclude option again is not necessary.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15510](https://teamcity.mesosphere.io/viewLog.html?buildId=133494&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


Love that you got the one line version to work! (I couldn't quite get it.)


No need!!

1) center all text / images of the post

https://github.com/duongphuhiep/hcqh/wiki/AUG-Overview#l%E1%BB%B1a-ch%E1%BB%8Dn-hi%E1%BB%83n-th%E1%BB%8B-textalign-v%C3%A0-centerimages

be carefull if image has caption "Anh chụp bởi Tuyết", the caption won't be centered

2) center text / image or anything individually

http://hopcaquehuong.org/admin/#!/file//content/blog/2015-12-29 Markdown test/vi.md


++ thanks


Perhaps we should migrate in the near future (before our 1.0 release) to using names instead of ids to present them to the user? Opinions?


Great work.


and also try to make it grammatically valid sentences, i.e 'when the course user is a staff member'


to contain_exactly('active')


@Julian-B should the command class be abstract?


すみませんが、コミット文が、
「動かないので、動くようにします」
としか書いてないように読めます。
文面を書くのが難しくなっている理由として、
・ポート番号を6633で指定した
・addLink() 関数を使うようにした
の2件を1つのコミットにまとめてしまってるからかと思います。
難しいようなら、コミットを分けることをオススメします。


sorry


Ronald,

Iam using one available at python packages repo i.e 2.4.0(https://pypi.python.org/pypi/oslo.log).looks like i should use older ones.


The original ProcessConfig was a script. It was was replaced by a function with the same name, based on ProcessConfig2 with the additional spellcheck features.


Only try to split if `LANG` exists. If so, splitting and indexing at 0 must be safe - at worst the result is `''`, in which case `setDictionary` seems to go for `en_US` down the line anyway.


user.moderator!


Huh. I didn't do anything manually, but I'll re-clone the repo for good
measure. Good catch, thanks.

Matthew

On Wed, Feb 10, 2016 at 6:45 PM, Chris Kuehl <notifications@github.com>
wrote:

> fcc30d5
> <https://github.com/ocf/puppet/commit/fcc30d559909d11c42fafc1606bb80d37e291fc6>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ocf/puppet/commit/e9965a798ba11d5b5c4209aefb9f2f803df19ec7#commitcomment-16023172>
> .
>


Something you might want to think about: ids are unique (in a website, you can only use any single id on one element).  That means that if an element is going to be used more than once in the project, it should have either a class associated with it each time (which I think might work better in this case), or it should have an id specific to this instance of the button (for example, id="gohomefromresultsscreen").  Generally though, if there will be more than one, it is simpler to use a class.


@henryhadlow not quite sure what to do about the Channel Islands - any suggestions?


I preferred to let the comments because it's a bit unclear


TeamCity FubuMVC Swank :: Deploy [Build 2.0.60.0](http://build.mikeobrien.net/viewLog.html?buildId=2513&buildTypeId=fubumvcswank&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 399 Build time: 00:01:10


@scottrobertwhittaker There's another one on line #58 - "[destuctors](https://github.com/rust-lang/rust/blob/master/src/libstd/prelude/mod.rs#L58)".


One purely stylistic thing that most people are dong is putting spaces on either side of the = sign in element attributes.  Generally, those spaces are omitted to make it easier to see attributes as separate, single entities (makes it more readable).


> I see, weather you lied in your forums signature or you forgot that or you still hate us.
For what reason do you think that I hate you? So far the only thing I'd done that may be unfavourable to you is that you may affect plugin compatibility on other branches of PocketMine, leading to your users causing nuisance to some of us developers who do not wish to support these third-party builds while we do not wish to (e.g. Falkirks/Specter#26). (Yes it is an ImagicalMine issue but same appears on ClearSky)

Let's be serious about the code and put down any kind of, if any, conflicts of ego. There are three ways to be socially responsible and mitigate this problem:

* don't make any backwards-incompatible changes
* add dummy methods that do nothing in the methods you removed. Might be a bad idea though.
* bump the API major version to a large number (that official PocketMine will never be reaching, such as 1001) so that only plugins designed for ClearSky can be loaded. This can actually be practically done if you write a script that modifies the plugin.yml in phar files for your users.


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


I tried to run `DocsCronTask` but the task could not be found. The task isn't registered in any YAML file as far as I can tell. Again, I might be missing something :)


@jayphelps - tyvm, emoji is much better!!1!


Wait wait , you guys fill my mail box.
Can you move to chat group?


I would squash this commit


Should 27 be in the bottom-right corner, or is the bottom row of the right vertical intentionally flipped?


The following doc(s) have been deployed to the [rachelwhitt](http://rachelwhitt-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/environment-specific-config](http://rachelwhitt-static-docs.pantheon.io/docs/articles/wordpress/environment-specific-config)


I've just did `unsafe impl Sync` for messages: https://github.com/stepancheg/rust-protobuf/commit/c172f70cfeacd9d9da5d4a1db4ca67c1e6800fa6. It is safe, and makes &message shareable between threads.


Since we only have one button in the site that will share results, this is a great example of when it is a good idea to use an id.


Test or Rest?


@ncase Yes I saw, THANK YOU THANK YOU THANK YOU.


why delete ngx.req.discard_body()?


I found : What is Node.js for and it's url is - https://www.quora.com/What-is-Node-js-for


a Good Read


Seems you have to manual add files to the repository before you commit.
I had to change this back because we don't have chad.tmx in the assets folder yet

Select File In Assets Folder on Right > Right Click > Git > Add
then you can comit and push


Yes it would. Currently we are stuck looking at the tests to figure out
what happens.

On Mon, Feb 8, 2016, 5:52 PM Ben Lickly <notifications@github.com> wrote:

> Oh, good point. Wouldn't hurt to add another link for external users too,
> though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/closure-compiler/commit/19f3a8989ee30bf9c21aecd4956e0377560d5522#commitcomment-15965519>
> .
>


so social


Our policy should be that we ignore the request body, not throw it away. `ngx.req.discard_body()` will prevent it from being sent to the upstream entirely, which I think it too heavy-handed an approach.


dsdssss


q


This logic is much clearer, I like it


@aodzip where is it?


This commit has to be reverted.

Read this article on rebase and conflict resolution:
https://help.github.com/articles/resolving-merge-conflicts-after-a-git-rebase/


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1530 of 1530 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)


There is a typo in line.


Whether to use zero- or one-based line indexing was something that I was always unsure about. One the one hand consistency with the byte command should win out, but humans are so used to counting lines starting from '1', mainly due to text editors doing the same.

Using zero-based indexing makes less sense when arriving at dodo from a conventional editor, but it makes more sense in the end.


This will fail if you screen size is somewhat large. You need to pull down the entire ViewGroup to ensure that this tests works regardless of the screen size.


readme.md
A telegram chat group


Experimental.  Analogous to `FutureMatchable`.  If this works out, we should contribute this upstream.


Don't use replace text. You can chain multiple actions. In this case you need to clear the text and type a new string.


This is good, I have found myself neglecting the documentation in the past.


It works fine with remote sites, like ftp.gnu.org, but fails with the host address (10.0.2.2). Looks like just a limitation of QEMU networking. So I just disabled the setsockopt IP_PORTRANGE option (via ifdef), and it seems OK for now.
```
# ftp ftp.gnu.org.
Connected to ftp.gnu.org.
220 GNU FTP server ready.
Name (ftp.gnu.org.:root): ftp
230-Due to U.S. Export Regulations, all cryptographic software on this
230-site is subject to the following legal notice:
230-
230-    This site includes publicly available encryption source code
230-    which, together with object code resulting from the compiling of
230-    publicly available source code, may be exported from the United
230-    States under License Exception "TSU" pursuant to 15 C.F.R. Section
230-    740.13(e).
230-
230-This legal notice applies to cryptographic software only. Please see
230-the Bureau of Industry and Security (www.bxa.doc.gov) for more
230-information about current U.S. regulations.
230 Login successful.
Remote system type is UNIX.
Using binary mode to transfer files.
ftp> dir
227 Entering Passive Mode (208,118,235,20,81,216).
150 Here comes the directory listing.
lrwxrwxrwx    1 0        0               8 Aug 20  2004 CRYPTO.README -> .message
-rw-r--r--    1 0        0           17864 Oct 23  2003 MISSING-FILES
-rw-r--r--    2 0        0            4178 Aug 13  2003 MISSING-FILES.README
-rw-r--r--    1 0        0            1962 Nov 15  2012 README
-rw-r--r--    1 0        0          405121 Oct 23  2003 before-2003-08-01.md5sums.asc
-rw-r--r--    1 0        0          210988 Feb 10 11:25 find.txt.gz
drwxrwxr-x  306 0        3003        12288 Oct 15 00:25 gnu
drwxrwxr-x    3 0        3003         4096 Mar 10  2011 gnu+linux-distros
-rw-r--r--    1 0        0          403783 Feb 10 11:25 ls-lrRt.txt.gz
drwxr-xr-x    3 0        0            4096 Apr 20  2005 mirrors
lrwxrwxrwx    1 0        0              11 Apr 15  2004 non-gnu -> gnu/non-gnu
drwxr-xr-x   90 0        0            4096 Mar 31  2015 old-gnu
lrwxrwxrwx    1 0        0               1 Aug 05  2003 pub -> .
drwxr-xr-x    2 0        0            4096 Nov 08  2007 savannah
drwxr-xr-x    2 0        0            4096 Aug 02  2003 third-party
drwxr-xr-x    2 0        0            4096 Apr 07  2009 tmp
drwxr-xr-x    2 0        0            4096 May 07  2013 video
-rw-r--r--    1 0        0             954 Aug 13  2003 welcome.msg
226 Directory send OK.
ftp> _
```


You don't have to ignore all tests if you want to run only one of them. Right click on the method name and select run and it will only run the selected one.


As per usual, I committed this without testing it, going to give it a go in a minute.

Also, neglected to mention; this also pushes you almost straight to 'login', but forces a password change on that login.


Leave a Comment 001


applyCheck does this


This is not task 5. If you meant task 6 - don't use position to scroll to the item.


You are totally in the hook to merge the outstanding npc changes with these off-spec whitespace changes.


Everybody can say something at here!


Minor nitpick, but it would be nice if it was clearer that the 900 represented the same 15min used elsewhere. Either building it with math, or doing a TimeSpan.FromMinutes.TotalSeconds, or at least a comment.


Thanks for capturing this...I tried to figure out when I saw it in the beginning..


Hey @dduan, you actually want to do this to demarcate code that should only be compiled in DEBUG builds:

    #ifndef NDEBUG
    ...
    #endif

That is, if NDEBUG is defined, asserts are disabled. DEBUG means something else and we shouldn't be testing it. (Thanks @jckarter for pointing this out)


@aodzip I have difficulties joining it. The `Join Group` button is unresponsive.


#1 Attempting to fix error in safari.


Why do you need it for manage offer?


Ugh.


Ugly.


yeah, my concerns were around the consistency between byte and line, I dislike how this is non backwards compat, but I think this is conceptually cleaner.

I was also worried about 'absolute 1' and 'relative +1' having slightly conflicting meanings (+1 always moves, 1 never moves from start)


huh, ok making another PR.


Some feedback for HW2 -

Everything looks pretty good.  For #3, you could also use this snippet to produce a nicer looking score with less code:

```
from sklearn.cross_validation import cross_val_score
# len(y_train)
cv_sizes=[5, 10, 25, 50,100]
for i in cv_sizes:
    print 'CV size', i, 'mean score', cross_val_score(clf, X_train, y_train, cv=i).mean()
```


@neetVeritas
So Veritas do you have a roug estimate of when finis toolkit will be released?


I thought we were doing Brave (Link Bubble Browser), but I don't care a lot because it is temporary.


Thanks, I try to avoid mixing testing (if) with side effects (for loops are an exception, sometimes).


same >.>


This is off spec for the coding standards.


[got it, thanks!](https://github.com/rust-lang/rust/pull/31559)


I'll probably merge from yours again after I finish task 9


Off coding standards.


I already provide a Cargo wrapper in the `cargo` module, so there is no need to import `Command`. You can convert lines 16-24 into:

```rust
if let Some(e) = cargo::call(vec!["new", "-q", "--bin", "--vcs", "git", project_path]) {
    return Err(e);
}

// Do the other setup steps...

Ok(())
```


`DocsCronTask` may eventually contain more tasks than just `RefreshMarkdownTask` and `RebuildLuceneIndex` so I suppose it shouldn't be called like this ? Or, perhaps we should return to the name `DocsUpdateCronTask` or use `DocsRefreshCronTask` and keep this cron task solely for getting the latests md files ?


These are dangerous, as the service builder cannot handle them (see second heading in our [common pitfalls](https://github.com/CCI-MIT/XCoLab/wiki/Common-Pitfalls)).
I'm assuming this is from IntelliJ's optimize imports, you can [turn off wildcard imports in the settings](http://stackoverflow.com/a/3348855).


I would add it to `gregoriosyms.sty` as there may be instances when debugging messages are needed in `gregoriotex-symbols.tex` besides this one.


Oh boy. Why this? Have you noted comment on isRouted option?


@jasonmalinowski - is `embedInteropTypes` supported in the build task? I don't see anything in the lock file that would indicate what to do with it - do we parse it directly out of the `project.json` now?


Given the `project_path.clone()` used above, you shouldn't need this `out_path` line anymore and just use `project_path`.


This is feedback for your 2nd HW assignment.

This is the best homework I've seen.  Great use of simple, concise code.


But that can't work for transitive refs, right?


The examples in this man file both cause errors.


wait, it requires a phone number?


What if I use a different folder structure?


Looks good! Your indentation is good, you seem to have all the necessary content, and it is organized logically.  If you could make a couple of quick edits to reflect the guidelines Kincaid and I posted to Slack, that would be great.  Other than the comments above and those quick things, I think you're done.


@cruzzan fixed via 7e02e55.  Please swap our your [`qz-tray.js`](https://raw.githubusercontent.com/qzind/qz-print/2.0/js/qz-tray.js) and [`sample.html`](https://raw.githubusercontent.com/qzind/qz-print/2.0/sample.html) to see it in action.


I don't *think* this should be necessary - if this project has a P2P to the VS-Def project above, it should flow transitively.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1530 of 1530 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)


Changes from geodirectory-assets/js/geodirectory.js have moved to post.custom.js


Oh, one last thing for the night. I noticed that only 2.4, 3.0 are in archives while 3.1 and 3.2 (stable) are on display. Do you want to put 3.1 in archives as well, just as you have on the uat test server for docs ?


@zystvan I too had tested it, but on Chrome. I don't think you missed anything :)


Thanks for reviewing class :)


This is feedback for HW2:

Looks like you were able to get through and explore your dataset pretty well, but a lot of the analysis seemed challenging.  If you need some help with coding or any of the analysis for logistic regression, let me know.


request diff which includes documentation update to doc/pan.xapi.rst.


getCanCreate() will return false in a proposalRevision phase, which allows user to **edit** proposals but **not to create** new ones. I think we should allow reverting whenever we allow editing, so I could just check for getCanEdit() in this case.
One thing we might want to consider is whether any member should be able to revert or if this should be limited to the owner (we could use getCanAdminProposal() in a addition to getCanEdit() to find out). @pdeboer What do you think about the permission question?


Electric-Service-Refactor (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2082 of 2083 tests passed, 479 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.
 * 3 tests had: MTD diffs.

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


Old type doors won't work since none of the code to handle opening or closing is hooked up to them. So we want to avoid having players walk to a door and not being able to use them. In the worst case, a player walks from max ABM range to the door in use range in ~6.5 seconds or so (since the door can be used from several nodes away).

I'll raise it to 7.0 seconds, but we shouldn't go much higher than that.


ok


totally


Hilarious. I love this https://github.com/jayphelps/git-blame-someone-else/labels


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2081 of 2083 tests passed, 479 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.
 * 3 tests had: MTD diffs.

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


more work on #1


 tl;dr it's safe to remove because it was moved to the parent state.

The state config comes from [ui-router](https://github.com/angular-ui/ui-router/wiki), and [this state inherits](https://github.com/caciviclab/disclosure-frontend/commit/fe83507667790f701a6a7e7503fd7f2a1a575161#diff-397bd9abb5f4324bc4f716ede75f7621L57) from [appMain.locality](https://github.com/caciviclab/disclosure-frontend/commit/fe83507667790f701a6a7e7503fd7f2a1a575161#diff-397bd9abb5f4324bc4f716ede75f7621L12) so all those resolvers (`locality`, `ballot`, `disclosureSummary`) run as part of the parent state. Leaving a `ballot` resolver here means that endpoint would be hit twice.

Happy to explain over hangout if that wasn't clear.


TY @jasonodoom


More work on #1


It's kind of slimy to not make this public, but I know Glyph itself needs work before it can be public.


When I run the TraubMemorial.m, I got the following error

Undefined function or variable 'InitializeWorkspaceDisplay'.

I am using matlab 2015b and I guess this error is caused by a lack of some
toolbox or product which I didn't install. However I cannot figure out
which one it is. Please help me with this problem.

I commend the 'InitializeWorkspaceDisplay' line so that I can run it this
moment.

Best wishes,
Xin

On Tue, Feb 9, 2016 at 9:31 PM, fjhickernell <notifications@github.com>
wrote:

> It should be in the repository.
>
> The TeX problem could be a version issue, but the statement at the
> beginning should set the interpreter to LaTeX.
>
> Best regards,
> Fred
>
>
> Fred J. Hickernell, Professor and Chair
> Department of Applied Mathematics, Illinois Institute of Technology
> RE Bldg Rm 208, 10 West 32nd Street, Chicago, IL 60616
> hickernell@iit.edu <mailto:hickernell@iit.edu>, www.iit.edu/~hickernell <
> http://www.iit.edu/~hickernell>
> Office: 1 312 567 8983 Cell: 1 630 696 8124
>
> > On Feb 9, 2016, at 9:29 PM, YuhanDing <notifications@github.com> wrote:
> >
> > I find it in the repository. But when I run it, it gave me some warnings
> about interpreting TeX string "\(x\)". Is this because the version of
> Matlab?
> >
> > Date: Tue, 9 Feb 2016 15:25:25 -0800
> > From: notifications@github.com
> > To: LocalAdaptiveAppxOpt@noreply.github.com
> > Subject: Re: [LocalAdaptiveAppxOpt] revising (a124adb)
> >
> > Thanks, Fred. Where is TraubMemorial.m? I have not been able to locate
> it.
> >
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> >
> >
> >
> >
> >
> > —
> > Reply to this email directly or view it on GitHub <
> https://github.com/xtong5/LocalAdaptiveAppxOpt/commit/a124adbf46dd100adbb0109e616b101207cacdcc#commitcomment-15995751
> >.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xtong5/LocalAdaptiveAppxOpt/commit/a124adbf46dd100adbb0109e616b101207cacdcc#commitcomment-15995773>
> .
>


np :)


@ceyusa @dbnicholson For future reference: there's no way to safely call g_setenv in a GTK+ app (actually, it's risky in any threaded program). Some other thread calling g_getenv at the same time could cause this to crash. This has been the cause of occasional crashes in GNOME (rare crashes, you have to be unlucky), so we should never call g_setenv except at the very start of main.

For something like this, we could instead manually modify the environment list passed to the subprocess.


The code is equal. But

```java
if(stream == null)
    stream.close();
```
Potential `NulltPointerException`


This whole section is duplicated in the _AddUpdateProposalDetailsActionController_, so it would be good if you could consolidate the common elements into a method (maybe in the service layer, but perhaps you could move the handler into the _AddUpdateProposalDetailsActionController_ and turn the common elements into a private method).


Line 30 of the compiled js evaluates to false.
data = this.asciiFilter.test(data);

I believe line 21 of the coffeescript should be:
data = data.replace(@asciiFilter, "")

Line 23:
data = data.replace(@textColorFilter, "")


Sorry for the delay! It's now on Hackage:

http://hackage.haskell.org/package/dbmigrations-1.1


Why does this have an unconventional and weird name?


now there are some problems.
0.13.x is right.


Hm, I'm not sure about the `proposal.wasMovedToContest == null` part. What exactly does this do? Will this hide the revert button if the proposal was move **into** this contest? And do we want that?


No, it's not resolved in order. It's an acyclic graph :grinning:

[ui-router](http://angular-ui.github.io/ui-router/site/#/api/ui.router.util.$resolve) resolves the dependencies automatically based on the introspection of the function arguments (basically how all of angular dependency injection works). This is a double edge sword though, because if you made a mistake defining the dependencies, your page will often just hang without any error.


Erm, no... Let me completely fix the grammar. :)


@dpoeschl Will this theme appropriately, or do styles needs to be applied to the various controls below?


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


FWIW, if you didn't include the helm tables, that would probably help a lot.

The notebooks can be large if you are inlining images, which then get added as text to the XML.  Saving the images instead of inlining would save a lot on space - at least in terms of keeping the notebooks under version control.  The discussion here might be helpful: http://stackoverflow.com/questions/18734739/using-ipython-notebooks-under-version-control


ок. Осталось перенести скрытие/показ блока отзывов и будет нормально.


I missed the initial commit of just copying the first two reflections and then doing the third, so I need to go back and do them separately


Seems like there are a lot of extra newlines in some places. We generally try to keep to one empty line at a time, maybe two if we really need the separation.


@dpoeschl Is EnvironmentColors the right source for these given theming?


Overall this looks great and seems to work like a charm :) Good job @carlosbpf!


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


This doesn't specify the glyph? Or does that come later?


Ok 7s is good.


ack


Why? It's a constant across the system that never changes. We can move it somewhere else later. You'd rather it be a variable inside a function called "required_kills_to_level()"?


What coding standards are we using? I thought we were going camelCase with everything


why ARMING_CHECK_ALL is not the combination of all the others? something like

```C++
enum ArmingChecks {
    ...
    ARMING_CHECK_ALL = 0xFFFF,
};
```

Would avoid needing to check everywhere if it's "all" or the specific one that is enabled (and bugs in which it's not triggered for ARMING_CHECK_ALL.


Agree on the principle. They should be arrays but some (apparently rogue) clients will send {}. I have not calculated a rate for that.


It randomly fixed it self after xcode restart. Have no idea whats going on :grin:


why?


Never mind, off coding standards.

http://i.imgur.com/cVfL1Mn.jpg


You can add me first
@Aodzip or add +86 13298724327


is RunTurn


I have three other small nitpicks:

1. You can drop the `super::super::`, since you don't actually need it. Since this is not an external module, like `std` or `zip`, I'd also recommend dropping it to a separate line below all the other `use` statements. This is to comply with idiomatic Rust practice.
2. You used `std::fs` and `std::io`, but also `std::fs::OpenOptions` and `std::io::Write`. This is a little redundant.

```rust
use clap::ArgMatches;
use std::fs;
use std::io;
use std::path;
use zip::ZipArchive;

use cargo;
```


just Reverse()


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


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2114 of 2115 tests passed, 479 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.
 * 3 tests had: MTD diffs.

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


Idk, but if it requires my phone number, I'm not going to join :P


Are you planning to upstream this to Tassadar's repo?


Cleaned up in next commit


That is private for others
Phone number is use as user account


Можно было продолжить elif использовать


Yes, I know, but I still don't want to give it to anyone (as you might know, not even Twitter), so never mind.

P.S. I accidentally entered that phone number you gave into the web client, so sorry if I made it send an SMS to your phone.


It worries me a lot that we do this all in a single operation that we'll block the UI thread for a long time.

Have we considered timeslicing this, or allowing messages to pump between each project, or something?


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2107 of 2107 tests passed, 0 test warnings)

 * 550 tests had: AUD diffs.

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


В таких случаях лучше возвращать False


`MayBeInstalledIfWeHappenToHaveComputedThingsForThisProject`?

Not sure I like this name/implementation pairing.  People are likely to assume that it's right...


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1531 of 1531 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)


Similar concerns about reliability of this name.


catching up: what is the definition of default-free vs default-fixed? Simply whether or not a `value` was specified? Or something else?


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)


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


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


Если возвратить False заместо error то можно сделать вот такую проверку сделать `if not n`


Basically! By default, taper has a value and Lmax does not, so taper is default-fixed and Lmax is default-free...those aren't the best terms...let's say "free/fixed in the default (no arguments) case"


HBIRE_vector_take2 (amirroth) - i386-Windows-7-VisualStudio-12: OK (2074 of 2084 tests passed, 507 test warnings)

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


This isn't in a PIA somewhere?


Мог продолжить использовать  elif


How big is this `byte[]` would it be better to just expose the stream API?  I'm not sure this type holds its own.


'yet unknown'


Лучше возвращать False


fix-missed-date (jasondegraw) - 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)


Use protection


я видел инфу про `borshik`, читал, почти наизусть могу пересказать, помню о нём из предыдущих замечаний, хотел `include'ом` сделать, но понимания как это делать у меня нет, потыкался-помыкался и пока сделал так...
обещаю выучить и научиться...


In the 3 code blocks above spaces around '='.


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


lame


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15381](https://teamcity.mesosphere.io/viewLog.html?buildId=132326&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:10


In 10 years when people won't get the joke:

http://knowyourmeme.com/memes/pepe-the-frog

We past-people found this M E M E hilarious :frog:


Why is this done by reflection?  Because our build machines don't have new enough bits on them?


不应该。


Nice catch!


It seems odd to have both of these as fields.


Overall great job on the hw. One thing to note is that you should try to style html tags through classes rather than direct tag selection in css. For example instead of selecting the li tag in css directly, consider assigning it a class and selecting it that way. Otherwise keep up the great work.


@GuiltyDolphin How about [this](http://ss64.com/bash/) one ?


Looks good!


Ugh - this is ugly.  Can't we listen for solution load events instead?


using log link is more robust that inverse power link


That awkward moment when OSX corrects datalogging to cataloging...


Passing a fast default argument is not a good idea. It would be better to just have fast defined in the function itself. That way, we only need to change one bit of code, in the general_funcs file, rather than manually changing the arguments of each time we call print_line()


Overall good work on HW2.  A few comments on some of your solutions.

For #2, I you are right on.  The one thing I would ask is why is sex and age important features?
 - The model coefficients indicate that the Pclass + Sex are the most predictive feature.  Note that Pclass is negative.


This is for testing, right?  Maybe a static "CreateForTesting" and a private constructor, or at the very least a doc comment saying why this is public?


This is odd behavior for a setter - it ignores the set if the version is the same?


![untitled](https://cloud.githubusercontent.com/assets/13975637/12969351/e0c412a2-d0b1-11e5-9140-a27f3802cf92.png)

you can help me custom viewquipment hide viewsprite same pic.


Somewhat nicer to put the `OperationCanceledException` check in the filter, as that way the stack trace won't be lost, but as these are just cancels, not a big deal.


Still no battle predictions enabled? also doesnt register damage received @dragonjet


Cambiar la zona horario de esta pagina: http://php.net/manual/es/timezones.php


ok.just so so


I think there is a way to pass a pattern into EnumerateFiles, which may avoid some IO if other files somehow ended up in this directory.


Should this say:  "returns the last matching node"?
(Since container is no longer a pointer parameter)


Not wrapped in `IOUtilities`?  Is this exception safe?


empd (nmerket) - Win64-Windows-7-VisualStudio-12: OK (2076 of 2080 tests passed, 0 test warnings)

 * 550 tests had: AUD diffs.
 * 4 tests had: EIO diffs.
 * 4 tests had: ESO big diffs.
 * 2 tests had: RDD diffs.
 * 4 tests had: Table big diffs.
 * 3 tests had: MTR big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.
 * 2 tests had: ZSZ small diffs.

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


Okay to use CurrentCulture in these interpolations?


what happens if the cart is empty


All good


These names aren't really appropriate -- instead of `S_parse_node_item`, for example, we should have something like `S_parse_node_item_prefix`.  Similarly for the others.


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


I'd think the mixin itself is pretty self explanatory.


@rmacklin Fixed


IOUtilities?


Consider using a class in this case. An ID can override class styling and cause problems when web applications are in a large scale.


Fixed


Thanks for pointing out! Fixed in next patch


Consider selecting this tag through a class rather than direct tag selection.


This comment doesn't seem to be on the right method.  Those things aren't pertinent to this method.


Overall good job on the hw. Please view the comments for best practices.


Nice work refactoring out the array, and just using the value from the select field!


Looks good


Good job using `removeClass()` to clear out all previous classes, and chaining jQuery methods.


i've been playing around with google play services and i activity and context are needed for serviceintents/handlers and broadcast receivers.


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


Good job using `replaceWith()`


Good job adding `console.log` statements to test that the functions are working!


Three slashes, but not a doc comment.


Pagina de los lenguajes https://github.com/caouecs/Laravel-lang


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


Good job using `html()` to update the DOM. Your click function for `add10` and `sub1` are inside the click function for `add5`. They should be outside that. You are also missing the function that resets the score to zero. Let me know if you have any other questions.


The google fonts link tag should come before your css stylesheet. In this case it should come before your style.css


Tab vs. spaces for indents?  It looks like the other entries use spaces.


Consider using a class in this case. You can apply more than one class name to a html tag.


I think you misspelled canols... maybe meant canola?


I've managed to get around it using reflection to access the private field "activity" in papplet. Best solution I've found so far. :(


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (981 of 981 tests passed, 0 test warnings)

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


Consider selecting this tag through a class name rather than a direct tag selection.


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (973 of 973 tests passed, 0 test warnings)

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


Why is <cctype> needed?


Someday I could *definitely* imagine us wanting this in the internactive window...


Sorry to be the last to respond, but I've had some personal stuff taking up most of my time. My email is dkcarter@csustudent.net, but I think the easiest way for all of us to communicate is to create a skype chat group. That way we'll have a record of everything said, and we won't have to work around everyone's schedules to all communicate. That's assuming everyone has skype of course, but if anyone doesn't or somebody has a better idea we should decide soon.

If we agree on this, then we will need everyone's skype usernames and I'll set up the group chat.


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


Consider selecting this tag through a class name rather than a direct tag selection. This way you can avoid nested selectors.


Overall good job on the hw. Please view the comments for best practices.


Document where this came form?  I assume you didn't write it by hand :smile:


Hi,
By this commit, in All of my Contao 3.5.* that uses this package, I recieve this error:

`Warning: proc_open(): fork failed - Cannot allocate memory in phar://composer/composer.phar/vendor/symfony/process/Process.php on line 291

#0 [internal function]: __error(2, 'proc_open(): fo...', 'phar:///home/po...', 291, Array)
#1 phar://composer/composer.phar/vendor/symfony/process/Process.php(291): proc_open('rm -rf '/home/p...', Array, Array, '/home/pooyanmeh...', NULL, Array)
#2 phar://composer/composer.phar/vendor/symfony/process/Process.php(202): Symfony\Component\Process\Process->start(Array)
#3 phar://composer/composer.phar/src/Composer/Util/ProcessExecutor.php(62): Symfony\Component\Process\Process->run(Array)
#4 phar://composer/composer.phar/src/Composer/Util/Filesystem.php(119): Composer\Util\ProcessExecutor->execute('rm -rf '/home/p...', NULL)
#5 phar://composer/composer.phar/src/Composer/Downloader/FileDownloader.php(208): Composer\Util\Filesystem->removeDirectory('/home/pooyanmeh...')
#6 phar://composer/composer.phar/src/Composer/Downloader/DownloadManager.php(270): Composer\Downloader\FileDownloader->remove(Object(Composer\Package\CompletePackage), '/home/pooyanmeh...')
#7 phar://composer/composer.phar/src/Composer/Installer/LibraryInstaller.php(202): Composer\Downloader\DownloadManager->remove(Object(Composer\Package\CompletePackage), '/home/pooyanmeh...')
#8 phar://composer/composer.phar/src/Composer/Installer/LibraryInstaller.php(125): Composer\Installer\LibraryInstaller->removeCode(Object(Composer\Package\CompletePackage))
#9 phar://composer/composer.phar/src/Composer/Installer/InstallationManager.php(190): Composer\Installer\LibraryInstaller->uninstall(Object(Composer\Repository\InstalledFilesystemRepository), Object(Composer\Package\CompletePackage))
#10 phar://composer/composer.phar/src/Composer/Installer/InstallationManager.php(139): Composer\Installer\InstallationManager->uninstall(Object(Composer\Repository\InstalledFilesystemRepository), Object(Composer\DependencyResolver\Operation\UninstallOperation))
#11 phar://composer/composer.phar/src/Composer/Installer.php(604): Composer\Installer\InstallationManager->execute(Object(Composer\Repository\InstalledFilesystemRepository), Object(Composer\DependencyResolver\Operation\UninstallOperation))
#12 phar://composer/composer.phar/src/Composer/Installer.php(232): Composer\Installer->doInstall(Object(Composer\Repository\InstalledFilesystemRepository), Object(Composer\Repository\CompositeRepository), Object(Composer\Repository\PlatformRepository), Array, false)
#13 system/modules/!composer/src/Controller/UpdatePackagesController.php(139): Composer\Installer->run()
#14 system/modules/!composer/src/Controller/UpdatePackagesController.php(47): ContaoCommunityAlliance\Contao\Composer\Controller\UpdatePackagesController->runInline(Array, false)
#15 system/modules/!composer/src/ClientBackend.php(211): ContaoCommunityAlliance\Contao\Composer\Controller\UpdatePackagesController->handle(Object(Contao\Input))
#16 system/modules/core/classes/Backend.php(423): ContaoCommunityAlliance\Contao\Composer\ClientBackend->generate()
#17 system/modules/core/controllers/BackendMain.php(131): Contao\Backend->getBackendModule('composer')
#18 contao/main.php(20): Contao\BackendMain->run()
#19 {main}
`


Work needed:

- [ ] exhaustion
  - [ ] walk per meter: 0.01
  - [ ] sneak per meter: 0.005
  - [ ] swim per meter: 0.015
  - [ ] sprint per meter: 0.1
  - [ ] jump: 0.2
  - [ ] regen per halfheart | food >= 18: 4.0
- [ ] events
  - [ ] exhaust event
  - [ ] regain hunger event
  - [ ] regain saturation event


It's because we use a 1 in ARMING_CHECK to be check all values. It's a weird use of a bitmask for sure. It's a preexisting behavior.


We're not sure how we want to treat this __declspec() stuff yet, so please keep patching yourself for the time being.


OK


Consider selecting this element through a class. This will allow you to avoid nested selectors.


done.


Arming check battery runs other checks as well, so the minimum voltage to arm is tied to that in the bitmask fpr ARMING_CHECK for the ARMING_CHECK_BATTERY, but we don't run this unless the voltage is set to something. This is just to maintain the existing bitmask behavior. I'm open to other solutions.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 95](https://teamcity.labkey.org/viewLog.html?buildId=302497&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (3 new), passed: 39 Build time: 01:00:45
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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

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

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

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


Hello


Inline script tags should come before the closing of the body tag.


entries


Overall great job on the hw. Keep up the great work.