Sean...
Yes, I thought of that overnight as a lower risk change, anyway.  I will do it.

Dave


I'm pretty sure SickRage makes a backup of your database before upgrading
in the `backup` directory. You should do the same before trying to restore
it.

Once you have reverted to whatever version or commit you want restore the
backup.

I've also found that SickRage does not like to be in a detached head state
so I have disabled checking for updates until the issues have been resolved.


Zero chance, I am on-site


Please use parameterized log4j across the file


ehh... isn't this function too long? :)

Could you send this refactor as a separate PR? Then we could merge this PR.

Thanks!


Thanks for noticing. I also now realised I should move some rst files as well, so now fixed too


Good question. The existing `plain_string` state doesn't handle EOF specially. That's why I didn't do anything special here either.


initial


@onysko А чем тебе не подошел url()->redirect?


@yoojinkimhong DWMR. You worked thru the sandbox maze on Instagram?


@mtanzi I'm guilty of a very poor commit message here and I can't remember why I changed this :\ I'll add it to the list of things to fix once @yelps gets back to me and I can start working on MarcoPolo again :)


Oh... it does. `c_eof` is included in `c_eol`.


Yes, just like before.

No change in functionality here, just the way in which it is implemented.


You just extract a couple of characters from it and check that one `== ?:` and the other `!= ?:`. I don't think the encoding should matter for that.

If it does matter, we should have a unit test which reveals the problem.


Double spaces between classes.


@ludo237 fatto!


@yoojinkimhong DWMR. More good work. You're making super use of your freeze time. Thank you.


I'm curious about this as well.


Comparison between different encodings can fail, I think.


Yes, i am debugging some trouble creating PyPI package, and found this.
So, just wanted to know what this exactly meant for.


I like that, and remove the inline from dnie code.


work good job


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.2](http://build.prxm.it:8813/viewLog.html?buildId=3396&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:03:24


@shun-s could you add a prefix for the title of your commit message? "test: " would suffice.


This if/else statement could be extracted into a helper.


This one looks much cleaner, but has the disadvantage that it doesn't work properly :-(
It allows to enable and change those dividers, but on subsequent open od the dialog it always shows the 1st choice, not what has been set in the previous round.
Help and hints needed!


not ready to merge until role delegation feature is rolled out


I'm glad you finished this! Very clean code. I wonder if there's a way you could refactor those if/else statements in your getWinner function by "switch"ing them for something else. Hmm... Overall good work! :100:


As mentioned before these classes should be named with double dash → `emoji--footprints`.

Arguments:
1) https://github.com/squerb/squerb_foundation/wiki/Code-Guidelines#componentname--modifiername
2) `.emoji-(-)…` cannot be used without `.emoji`


@yashvsingh sir, on adding to .gitignore file, after every pull we have to add the file.
So i have removed it from .gitignore file.


Remove units


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.8+8](http://build.prxm.it:8813/viewLog.html?buildId=3397&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 76 Build time: 00:02:20


Check your emails


+1 for `stream`


this is not the logic that you want.


No worries. Will chat later.


Can you explain the idea with round((int) $x)
It is a bit unclear to me. Thanks


looks good


could we put the Role method first and the User method second, to highly encourage to use roles? At some point, we'd like to deprecate user authentification


Hmm, we could also have kept this one and translate from ssl-config to it; now we are leaking ssl-config in Akka HTTP API. I think ssl-config should just be used by Akka HTTP to get good default config, and it can be used by users to get good config as well, but that should be absolutely voluntarily.


cc @camilo

I'll make a bugfix release after we have this running in production.


for the sake of compatibility we should make this work for ES5, so use ```var``` rather than ```const```


@turleypol thanks for your comments!


So we go back to 7854c6045e7e802dc28420d1dcbdb60ed76d5f7e: "nicki_defconfig: update defconfig"


OK, it was fixed.


good


Right you are:

```ruby
[2] pry(main)> ?:.encode(Encoding::UTF_32LE) == ?:.encode(Encoding::UTF_8)
=> false
```

However, if you are worried about that, you should be transcoding the lookahead to whatever the *source encoding of literal.rb* is, not to what the user's input text was encoded with.


oh, thats the one mention that has user first. will fix


it has be removed later. Unfourtunately compass i creating `emojis-footprints`.

`emojis` because of folder name where images are stored and `footprints` bcs of image name.


@whatyouhide no worries :)


@nalind right, cross compilation isn't affected, windows runtime definitely ends up with  `ErrorCodeVolumeNoSourceForMount` as outlined here: https://github.com/projectatomic/docker/blob/master/daemon/volumes_windows.go#L30


User http://compass-style.org/reference/compass/css3/transform/


Please describe me your opinion about that method. Without that method will be generate clause like that:
```
query.OrderBy("n.age", "n.name").ThenByDescending("n.number", "n.male").ThenBy("n.lastName")
RETURN n\r\nORDER BY n.age, n.name\r\n, n.number DESC, n.male DESC\r\n, n.lastName
```
I think that it's not pretty query. But without that method, i can't prepare good query. Please, describe me what do you think about that? May be you have other solution?


`0.85` is a valid value for `opacity` and is visibly different from `0.8` and `0.9`. Limiting opacity values to `/\d.\d/`, IMO, a misfeature.


http://compass-style.org/help/tutorials/spriting/

I could try to change this behavior but I don't know if it is so important.


initial


@pdoongarwal ok.


If your refactor demanded change in the features, this must be an indication that something is wrong.

But in this case I think it is not something severe.


Never mind my mental decimal to hex converter was broken ;)


Yeah, true. Shall I revive this and keep the one in ssl-config converting from one to the other or would it be better to, for now, base the conversion on raw string config...? This opens the question of binary compatibility of ssl-config too


No Problem!


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


Can we somehow automate this part? Like grep directory or maybe some rake task which creates a partial?


Just an observation. Following this detailed istructions (in method 2) I got a further error (executing classification.cpp) for 'solver_factory.hpp':

'Check failed: registry.count(type) == 0 (1 vs. 0) Solver type Nesterov already registered.'

I figured out that proceeding in the same manner as you explained above it would work. I commented lines 67-68 in 'solver_factory.hpp'  It worked.

Thanks for your work!


You've done two things into one large commit again :)


Could we also say that the ROLE is the `highly recommended best practice` according to amazon (http://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create_for-user_externalid.html), and that we recommend that too


Could we add back the link: http://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create_for-user_externalid.html


Yup, using it already. Thanks for that :)


yes, i will change that.


I merged in the proposed patch I gave above.  If that fixes the problem, then we should close this pull.


I do not think anyone uses this, I added a PR to remove the api module #53


I have the feeling that the body of this if statement is some piece of code that must appear over and over again in various places where we export comments. Is that the case? If so, then we should merge these things into a single method.


Probably want a space after the if and before the opening parenthesis i.e: `if (`


Probably want a space after the if and before the opening parenthesis i.e: if (


其实拿时间这里一层层匹配是有必要的


Remove semi-colon


Still uses facton from db table, but it compares it against dbc I'm quite positive. Also, majority of faction_A and faction_H were either equal or entirely wrong faction.


Probably you can maintain an another repo "gprMax-extras"?


Uh, what? Why would you remove instructions from the emitter?


The default simulation speed is ```1.0```.
```0.01``` means 100 times slower.


But you've got the right idea, this is one step closer to unifying the database schemas.


The goal of naming the policy was to give concrete examples. Having done the setup process many times, you can get lost by all the names. People know they can give the name they want, if they don't mind they'll pick what's in the example and it will simplify the setup process for them


There seem to be many unrelated formatting changes in this patch that make it difficult to identify the changes that are specific to the problem being solved. Also, you introduce some blank lines with spaces in them.

Please submit the formatting changes as a separate patch.


can not *
My bad.


`sns` shouldn't be here. I think we should remove this example, as it'd encourage people not to give us access to everything we's like.  `AmazonEC2ReadOnlyAccess` and `CloudWatchReadOnlyAccess` being default AWS policies, we don't need to detail this, mentioning them seems enough.


Thank you!


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


I would say it's important because of consistency with our styleguide but if we cannot change it because of compass then we have to leave it 😉


@walterbender Sure I will do that. Would you mind if I rebase and force push and ammend the commits in order to fix this?
(That would destroy this commit hash)


> facb93c : `G_DEF_CLASS` does this macro is capable to find the free function for the related `GType`?

Yes. `boxed_free()` in `rbgobj_boxed.c` does it.

> 0f90f5d : Why are you using `g_match_info_unref(_match_info);`. I though that when a function generates a `match_info` structure, this structure had one reference and if we `unref` it, the structure is freed. Is the `BOXED2RVAL` works by copying the `match_info` structure?

`BOXED2RVAL` increments reference count of `_match_info` (= 2 references). If we don't `unref` the `_match_info`, the `_match_info` is never freed.

I hope that this answers help you.


Excellent!

Factory Girl's API is not the simplest, with transient attributes, lazy blocks, evaluators etc. but it seems like you gained a good understanding of what's available.


Let me know if there is something we could do with that.


:+1: this is a good explanation


Please report this as an issue instead.


Thank you :wink:


I don't remember the issue properly, but I remember that the problem remained in the widget. When the value was something like X.0, the 0's after the point were ignored by the widget's because it rounds its value (I don't know why). But, since result_str contains the value as a string with all the decimal zeros, It seemed to me a good way to fix the issue.

So, result.result was rounding the value, and sometimes the zeros after the comma are important. I took advantage of result_string because it maintained the value as string with all its decimals and I thought it was a good way to fix the issue.


Sure it could, but in my own use case I created constraints with weight programmatically so I never bothered to change the parser. but you are right, it's more reasonable to include the weight in constraintStrings. I'll find a time to update that, if you think that it's worth to do so.


Nice catch :smile: should def be a `break` there


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


`You can also optionally add tags to all hosts and metrics`: There are other configuration options, could we add something like `and play with other options`?


Yes, you can release as long as the build passes and the changelog is up to date.
Make sure to create a github release as well because people look for those.


Gut denke dann kann man die messagebox entfernen
stört so glaub nur


owkowkkw :v:


- could we remove `You can also optionally add tags to all hosts and metrics.` from step 3
- could we rename switch step 4 and 5? (install should be the last action)
- for the current step 5, could we say that the user can set other options, like tags and filters?


why did u set Spark Home?
it could work properly without this line of code in ***local*** mode!


same here


done


Since we are on Java 8, I think we should use `LocalDateTime` instead of `Date`.


ok, have all that added based on conversations with you and with apb


to be super pedantic, i'd add a space between the `=` and `delete`


* #21


Do we need to also slightly update the `how to configure cloudtrail` section to mention to attach the policy to the previous role or user?


that is fine.

On Wed, Jan 20, 2016 at 9:59 AM, Daksh <notifications@github.com> wrote:

> @walterbender <https://github.com/walterbender> Sure I will do that.
> Would you mind if I rebase and force push and ammend the commits in order
> to fix this?
> (That would destroy this commit hash)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Daksh/musicblocks/commit/31d7328a3233ad6968ebe98b53ccee9fc1980b0e#commitcomment-15559518>
> .
>

--
Walter Bender
Sugar Labs
http://www.sugarlabs.org


Cakephp_blogの作成を行う。
スペルミスでエラーになることが多い。


This is a mac-only configuration and should be in your personal git config. e.g

```
$ cat ~/.gitconfig
[core]
  excludesfile = /Users/elliot/.gitignore
```

```
$ cat ~/.gitignore
.rvmrc
.DS_Store
.ruby-gemset
custom_plan.rb
zeus.json
.powrc
.powenv
.Trashes
.fseventsd
```


This should have been in as my commit message.


A commit message that starts with 'also' is a hint that the commit could be two at least :)


i think jar option only makes sense if there's a package management system behind it (like maven). otherwise yeah, the opaqueness of the jars now is killer. @snopoke we're thinking of having a `release` branch on those two repositories where stable code would be merged into


The problem with the above patch is that on Windows setupMounts is broken because the mountpoint given doesn't contain Source. One fix can be the one in this PR from Nalin.


This can be removed.


`values` can currently be `null`, but we should avoid that (perhaps by checking it as a precondition in the constructor).


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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1233.8338a72b500573fe43f504af8dda12a00c094628](http://build.palaso.org/viewLog.html?buildId=68148&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 840, ignored: 7 Build time: 00:07:09


Favor `—` over `-` in Javadoc.


OK, cool. Just use ```[bdist_wheel]``` everywhere and you'll be fine.


I'm a bit late to the discussion, but I agree that the shortened form from this commit should be sufficient. :+1:


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


This is not portable. On my current vendored bazel setup (which relies on manual `PATH` editing), I need to have a `linux-x86_64/bazel` and `darwin-x86_64/bazel` (the latter for OS X workstations and build slaves.)


cf commit https://github.com/humanoid-path-planner/hpp-core/commit/f1fb84095e13016c6737739cc03c8f3b5cc59fd6#diff-f830f997e1ec48498862eaaab2b9128e


TeamCity Ppconsul :: Build (Linux) [Build 76](https://78.46.71.115:4433/viewLog.html?buildId=676&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:34


How do i convert this commit into a pull request.
whenever i try other commits are also included


 how do we??

On Wed, Jan 20, 2016 at 9:41 AM, Alectowse <notifications@github.com> wrote:

> This will need to be redone again, gah, THIS IS WHY WE BRANCH
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Alectowse/HHSHealthMonitorCSp3ArmandoPeytonAlec/commit/29243152d1f2a89e2f8a6310e066c8e5883e9c10#commitcomment-15558990>
> .
>


Yikes. This shouldn't have been part of the commit.


Dockerfile for HippoCMS
java8
tomcat7
hippo 7.9.4

based on Brian Snijders blog.


:+1: :tada: :sparkles: :heart:


Pretty straightforward :+1:


Since `iojs` is [end-of-lifed](https://iojs.org/en/), maybe include a modern Node.js instead?


Remove "Air Horner"


Correct, it will sometimes coerce it to a string, but not in all cases, just like JS.  It's amazing the number of subtle design decisions where JS and Lua chose the same strange behavior.


Fixed in https://github.com/maurizi/otm-core/commit/02394a729370cd34197d171b9c559e80911bfc7b


The compromise sounds good. Flashing all slides on the presenter is never something you'd want to do anway.


The point is we don't have time to do both Python and MATLAB tools, and our focus is on the Python ones. People that want to use MATLAB can easily do so themselves.


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

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


xmpp简易聊天


This revision is clean.


Anyway this is a wrong version because it’s backward incompatible change.


Nit: You're using `charset="utf-8"`, so you should be able to safely use "°" inline in your HTML instead of using the "°" escape sequence.


Note that, if only Faure permutations are used, 0 is always permuted to 0 and this fix is redundant.


If it isn't backwards compatible i've overseen something.


Good!


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 127](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=696&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:56


We do however regularly add this to the ignore file:
https://github.com/search?utf8=%E2%9C%93&q=user%3Aalphagov+filename%3A.gitignore+DS_Store


add "Change the state to Accepted"


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


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


I wouldn't mark it as accepted just yet :)


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


Is there a reason that this is outside your IIFE defined below? If not, could you move it inside?


If you change a public API it’s a major version even if it still works (in that case there should be a warning).


If you don't mind, I'll make a diff of this and merge it upstream.


Well, i did not intent to remove it, however it may be a good idea to do so (wdyt?). Having said that I don't think we need to test that validator is working since one would need to assume that there is a test for this validator and if there are none, then it has to be added (as separate task). The same for testing validation in the scope of the processor where I would assume that such test would be added at the base class to ensure that validators (regardless which and how many) are invoked before Processor comes to life.


Looks good.


Nice! :+1:


Ok makes sense. Thank you


oh, this was actually for the 00001 one, which was already accepted and I just forgot to change the status in it.

the errata has no state I think ,which is something we can think about as well


+*.JPG	 diff=astextplain


Not necessary, handled in the HandleInvoke function.  We should know already that m_nBodySize > 0 (This check was added into the official librtmp repository - commit 07c10ae612bf5c2dbea594dcbd4da85c54dba1e4)


* Near perfectly
Close enough.


Added in content gathering even with response codes 400-599 on commit 648ea40


@borzamircea what happened to the exposed ports?


thanks


А если файл существует, но расширение у него скажем xlsx, то что произойдёт?


My memory on this is that when creating a new server the data passed back is only the server name portion, whereas for a running and already deployed server you get the full name.  Max length of that server name portion is currently 6.


Thanks, Ian!

On Tue, Jan 19, 2016 at 5:21 PM, Ian Milligan <notifications@github.com>
wrote:

> added @tapar001 to editor list
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/programminghistorian/jekyll/commit/64f4d0ee9ae678513e53246dc23c854d70780b61
>

--
Evan Taparata
Ph.D. Candidate, History Department
University of Minnesota
1100 Heller Hall
271 19th Ave S
Minneapolis, MN 55455


Predict the category of crimes that occurred in the city by the bay


@borzamircea typically we don't want to include a license file in source code (specially open source), but since it is trial, we will keep it for now...


Shouldn't this be ```object|null``` instead of ```object|false```?

because if you do

null === true

the condition should resolve to false, same for the negated version

null !== true

would also be false according to the php specs when doing strict comparisons: http://php.net/manual/de/types.comparisons.php


No thats not correct: http://semver.org/#spec-item-7


It would be good, If we avoid using function declarations and use function expressions
(as you can see in rnjn's code).
If you are not aware of it please feel free to read here.
https://javascriptweblog.wordpress.com/2010/07/06/function-declarations-vs-function-expressions/


the ```window.location``` after the reidrect_uri should be urlencoded?


Thank you. Finally works!


I think there's an edge case here where '2.rc' would end up as '2..rc'. last_was_digit can currently be true for dots.


@borzamircea add new line :)


Good question. Will recommit and squash.


So far, we've pretty much only tested it with Python 3.4. In fact, our main target is Debian Jessie, which has Python 3.4.

That doesn't mean it won't work with older Python 3 versions, but I don't know that we want to spend resources supporting them, or that we want to send the confusing message that we might.

Maybe here the doc should say "Python 3.4 and above" or something like that?


I'm unsure whether you intended to only output the x field of each vector in vu memory, because that's what you're doing here.  Printing in sets of four implies that you're trying to output vectors, but that's not what this line does.


You removed the `install` from this line. :stuck_out_tongue_winking_eye:


`if (visible)`


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1234.9fab2561d54198f2e16dcb69e44aafe384470562](http://build.palaso.org/viewLog.html?buildId=68149&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 846, ignored: 7 Build time: 00:06:48


Thanks a lot.


would be preferable to put ```client``` somewhere that's not a global


double dots are replaced with single dots below


if the comment not found the controller will raise a error.


@randaz81 This is breaking all existing controlboards, so also coman/walkman software, robotran (cc @barbalberto). For a smoother transition can we add a default implementation to the `getJointType` to `IAxisInfo`?


they are `_vecotr` in C++, so it's better to keep unchanged


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


@domenic, are these namespace URLs too? I am little lost as to how to figure out which one to change which to leave unchanged since very few have explicit indication.


what was broken about this?
it was working for non-s3 uploads for a while..

receiving reports that support for large files is no longer working.


I am using "kicked" instead because the audio from FS will say "You have been kicked...".


You probably want `textContent` here and elsewhere in your code, not `innerText`, to [ensure Firefox compatibility](https://developer.mozilla.org/en-US/docs/Web/API/Node/textContent#Differences_from_innerText).

(I think `innerText` is/will be added to Firefox, but older versions don't have it.)


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


each VMF class loads it - no need to do it in user/test code!


Why did you add `debhelper`?

It's only necessary for people building Debian packages, which is going to be rare, and it's already [documented elsewhere](https://github.com/ideascube/ideascube/blob/master/docs/install.md#installation-using-the-build-process).

But if you add it, then there are a couple more packages required... and it's documented elswhere anyway. :smiley:

So I'd just remove `debhelper` from this line.

Good catch on `git` though, there was no reason for it to be missing.


Remove "Web Starter Kit" (presumably)


It requires at least one connector as the diagram says: https://github.com/ocpi/ocpi/blob/2.0/data/locations-class-diagram.png


formatting is wrong, you lost the first column of maxReqsInFlight


I guess since we do 'continuous' deployment it makes more sense this way


Follow the coding style, use a separate condition to check value of m_nBodySize.


it's better to make names more descriptive: `tmp_rawbuffer` instead of `tmp10`


hi. what is the fix for cm profile?


Could you add this or something similiar
`/philip/long-group-name-1/long-group-name-2/long-group-name-3/long-group-name-4/long-group-name-5/long-group-name-6/ok-that-should-do` and test if it's shown, if filtering for `phil`to test the threshold?

After this I'll merge this great piece of software!!


@shminer 谢谢提醒～
我正在测试这个 commit:
https://github.com/neobuddy89/shamu/commit/1e36738425ae53bb2cbb61273cfa8413fa3f21a0
这个驱动挺有意思的，正在探究中


I agree, it makes it easier to read without the use of unless. I'm thinking I need to change the admin_user method again though


you should be able to rely that these are in the config object, you could go far as to throw an exception if they're not


Not necessary,  handled in the ServeInvoke function.  Make sure m_nBodySize > 0, check the already mentioned commit 07c10ae612bf5c2dbea594dcbd4da85c54dba1e4.


知乎观光团


Why not to do this together within the "first iteration" too? That one is called only once too.

E.g.:

        public void countInstructorFair(Assignment<Lecture, Placement> assignment){
            if (allInstructorsAssigned(assignment)){
                stringInstMap.clear();
                for (Lecture lecture: getModel().variables()) {
                    Double bestPossibleValue = null;

                    for (Placement t : lecture.values(assignment)) {
                        double f = fairnessDouble(assignment, t);
                        if (bestPossibleValue == null || f < bestPossibleValue)
                            bestPossibleValue = f;
                    }

                    Placement placement = assignment.getValue(lecture);
                    for (InstructorConstraint ic : lecture.getInstructorConstraints()) {
                        Instructor s = stringInstMap.get(ic.getResourceId());
                        if (s == null) {
                            s = new Instructor(ic.getResourceId());
                            stringInstMap.put(ic.getResourceId(), s);
                        }
                        if (bestPossibleValue != null)
                            s.addBestValue(bestPossibleValue);
                        if (placement != null) {
                            s.addValue(fairnessDouble(assignment, placement));
                            s.incNumOfClasses();
                        }
                    }
                }
                countInstrMeanFairValue();
                setFirstIterDone();
            }
        }


yes, i will update the cloudtrail docs next.

is the feature still rolling out today?


not in the global scope please


silahkan dikembangkan gan


Not necessary, handled in the ServeInvoke function. Make sure m_nBodySize > 0, check the already mentioned commit 07c10ae612bf5c2dbea594dcbd4da85c54dba1e4.


@rullzer missed this ?


Isn't this essentially a lexicographic comparison? Why not make tuples of integers, will the smaller one with 0 until they have the same size, and then use the standard tuple comparison operators?


thank you.


I have an alternative approach for this. Will push as soon as I write a unit test.


So much testing! Awesome! :)


Please revert to "Noi dung da duoc thay doi", because the file have change.


Yes, I got it. I will revert now. Thank you!


yeah i thought it was someone genuinelly interested in my project, :100:, i guess not..


I would prefer to see these methods (countInstructorFair, countInstructorUnassignedFair and countInstructorAssignedFair) done within the context. This would make it easier to override the assigned and unassigned methods (within the context), instead of using beforeUnassigned and afterAssigned (see my earlier comment).

E.g.:

        @Override
        protected void assigned(Assignment<Lecture, Placement> assignment, Placement value) {
            if (isFirstIterDone()) {
                countInstructorUnassignedFair(assignment,value);
            } else {
                countInstructorFair(assignment);
                if (isFirstIterDone())
                    countInstructorUnassignedFair(assignment,value);
            }
        }

        @Override
        protected void unassigned(Assignment<Lecture, Placement> assignment, Placement value) {
            if (isFirstIterDone()) {
                countInstructorAssignedFair(assignment,value);
            } else {
                countInstructorFair(assignment);
            }
        }


另外想请教一下，我发现 N9 充电时不能深度休眠，这个是出于什么原因？能不能修复呢


@KimDahye
아래쪽 닫는 스크립트와 </body>가 없는데 필요한거 아닐까?


It's not clear from reading the code why you're multiplying by 1000 here. Could you add in a block comment somewhere nearby that shows an example of what `data` will look like? I think that would help developers understand the context of what this function is doing.


Yeah, after some digging, I found that warning was disabled last October for clang 3.07 and beyond.

https://svnweb.freebsd.org/base/head/sys/conf/kern.mk?r1=288942&r2=288943&


Issue #34


could we point to `http://docs.aws.amazon.com/IAM/latest/UserGuide/best-practices.html#delegate-using-roles` instead? that's the relevant paragraph of this page


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


This isn't your fault, but this line is completely redundant.  It hasn't been needed since d94d415ec7f2bdd38ba1592af271d5a25750fd4d removed the last method under the `module_function` scope.  Worth removing while we're here.


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


For me it’s more like “any backwards incompatible changes are introduced to the public API” but I’m not going to discuss it here. Just leave versioning and change logs for maintainers.


Where is this message going to go? What do you think is on stdout? If this is run in a cron job then it will result in an email being sent every day. If through authorized_keys it might corrupt the backup (in a recoverable way) by causing the output to start with a log message rather than with the start of the gzip.


it's impossible to have `null` here :)


What if the countInstructorFair does not set the first iteration? If the countInstructorUnassignedFair is to be called only after the first iteration, the isFirstIterDone() needs to be checked once more (or the countInstructorFair should return a boolean).

E.g.,

    if (countInstructorFair(assignment)) countInstructorUnassignedFair(assignment,value);


@luiz-tavares , retirei a classe seguindo seu conselho.

Realmente não sei se é válido manter a possibilidade de informar outra URL para o hit. Creio que dificilmente mude, e mesmo que mude não vão tirar a URL da note pro dia. Pode ser muito preciosismo manter isso.
O que acha?


Why did you delete the color sensor? Is this just a bad merge/rebase?


See previous comment about how this code would be a lot clearer if there were an example of what `data` looks like in a comment.


Rubocop Y U NO remove the extra spaces here?


ok.


OK, that's very clear. I guess you never stop learning ;). Thanks for responding!


Doing something we shouldn't do lots of times doesn't mean we should keep doing it.


missing regex: -?[0-9]\{1,2\}\.[0-9]\{6\}


I am sorry. This might really be sonata related. I found out that it just happens if there is no `removeBookingPayment` method inside the Payment entity. So I can fix my case, but I do try to find out more because this is a pretty odd behavior.


I don't see a reason for bringing inheritance into the mix.
I think that a simple `ReceiveLogger` object which takes `@logger` as an argument is better.


maxMBInFlight  should be maxSizeInFlight

change "to fetches" to "to fetch"

Wrap the actual config variables with

I would prefer to see the description of spark.reducer.maxReqsInFlight first.  Perhaps after that put something close to what you have about the maxMbInFlight.  something like (feel free to change wording).

maxReqsInFlight limits number .....  This is sometimes needed when you have  a large number of hosts and which can lead to a very large number of in-bound connections...


printing test name is redundant


good catch for install, a bad copy/paste from my Dockerfile


ACK


Please fix indentation.


same as geolocation, missing regex: -?[0-9]\{1,2\}\.[0-9]\{6\}


We've actually been switching things alway from `object-assign` and to `object.assign`, the opposite of this change. Curious about your thoughts on these two packages…


jay!


File Formats/Images/UCG Image Format version 3.md

What are these &emsp?


@iherman I think there might be a typo here, shouldn't the short name be `pwp-ucr`, like the repo's name?


Your formatting is borked, too


Grrr. We obviously don't want them to line up anymore. :cry:


"This document blah" would feel more appropriate IMO


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1759 of 2088 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-84.24%25-red.svg)


So it's big-endian.


So x86 words.


changed api key so the one here no longer works


Oops, I will complete that. thanks.


This is moving in the right direction, and it seems we need similar entries for the all of the top-level driver names.  Please add a comment to that effect here and file a bug to that effect.


wooOh that's sweet!


Thanks & fixed now


Could you align these tables to make reading more pleasant for people reading the diff? With spaces.


v8 slack tracking lasts for 8 constructioms


What breaking changes are in Handlebars 4? Should we consider doing a major version bump of this package?

There's another open issue about moving `handlebars` to a `peerDependency`, which I think makes sense with npm v3's flatter `node_modules` structure.


I think I thought I was describing an auction. lol.


!!requestProperties equals true.


Add some spacing between these fields, too.


These are DTDs, which similarly should not be changed.

I don't want to be discouraging, but it might be best to leave this change to someone who has a full understanding of the different ways in which the spec uses URL-like things, and can distinguish URLs meant to be browsed to from DTDs and namespaces. Otherwise I feel like I'll just end up having to code review every line of the change to point out which type of thing the line is about, which is not much different from just doing the work myself.


No duh


According to the API v2 documentation at https://www.ctl.io/api-docs/v2/#servers-create-server

> Name of the server to create. Alphanumeric characters and dashes only. Must be between 1-8 characters depending on the length of the account alias. The combination of account alias and server name here must be no more than 10 characters in length. (This name will be appended with a two digit number and prepended with the datacenter code and account alias to make up the final server name.)

Now following that logic and the given examples with an account alias that is 4 characters long, then the length of <=6 would be accurate as 4 characters for account alias plus 6 characters would be the maximum 10 character length as mentioned in the documentation. However, in our case the account alias is not 4 characters but rather 3 characters so we should be able to give a server name that is 7 characters in length and in fact if we do so though the web UI rather than the API we can do so.


I don't think this is a good idea.


Are you can write how to install it?


I think a clear visual indication that your deck is currently over 30 cards would be a fine solution for now. We can just test it then, if we get the feedback that this is a cool feature: Great! And if users are complaining, we can always change it back.


@rockfruit, nothing to do with interim's decimals places. By default, the number of decimals to be applied to the result must be those defined in the "Precision" field in AS edit view or alternatively, the significant ones obtained from the uncertainty value  (if the field "Calculate the precision from uncertainties" is checked). The result_str was added as an artifact to easily obtain the "original" value, because when you convert a number like "0.00" to a float (either in python or javascript), the decimal part is removed, which is not correct if you have explicitily set a precision of 2 for that Analysis. Another case is for instance, when the result set by the analyst is 0.0005, but the precision to be applied according to an uncertainty value of 0.013 (the decimal position is given by the first number different from zero in the uncertainty, in this case 2): the result to be displayed is 0.00.


How about "displayKeys"? Since this is part of the "deployment ls" command, I think it's intuitive enough that the keys would be deployment keys.


You could replace the full path with `File.dirname(__FILE__) + '/*.rb'` so you don't have to hard-code the location of the views directory.


It's being appended to a log file. See sac92-isaac-base.pp. The job is called "isaac-backup-database".


Maybe: `By default <command>systemd-activate</command> listens on a stream socket, use <option>--datagram</option> to listen on a datagram socket instead.`


This should be "^2.7.3 || ^3.0", right?


while you're in this class, can you remove the needless parameter in the method that is already in scope? Always really annoys me


I forgot to change the phpdoc. I first had object or false but then I changed according to the getCurrentUser method and because I think null is more correct in that context than false (null is object comparable)


Maybe: `Listen on a datagram socket, instead of a stream socket.`


This is what I mean about the scaling being wrong:

![screenshot from 2016-01-20 15 40 46](https://cloud.githubusercontent.com/assets/327925/12453701/6171db78-bf8c-11e5-9e5f-fe3784560226.png)


It would be nice to put a comment here and mention that the function returns the old digest (even though it is never used in that capacity)


The object form is </A_NumberHere>
I advance 2 because I want to get rid of '</'. So this gives me object->len -= 2.
However I also want to remove the '>', So I subtract one more.


this is bringing back memories. Did I do this in a branch that I never raised a PR for?


about `debhelper`, `apt-get build-dep python3-dbus` was complaining about it for another reason. It works without it now, i'll remove it.


INTEGRATION TESTING_2016-01-20 17:43:37


INTEGRATION TESTING_2016-01-20 17:43:39


Oops, missed that. Updating.


Not sure if you intended someone to try it out right away, but second block in chapter 2 just returned:

```
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-3-65c059ae4b86> in <module>()
      1 from scipy.spatial.distance import hamming
----> 2 from skbio import DNA
      3
      4 r1 = DNA("ACCCAGGTTAACGGTGACCAGGTACCAGAAGGGTACCAGGTAGGACACACGGGGATTAA")
      5 r2 = DNA("ACCGAGGTTAACGGTGACCAGGTACCAGAAGGGTACCAGGTAGGAGACACGGCGATTAA")

ImportError: No module named skbio
```

Just FYI


This has global effects and, at the very least, shouldn't be the defaulr in private rooms.


Well if your parameter is named "user" in the .py file it should have the same name here!


TeamCity Civic Assembly :: Build backend + frontend [Build 12](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=384&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Some of this is a little non-cohesive for a single commit, sorry! Some of this was intended to help Isaiah with his feature of adding Friends functionality to our application.


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

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


does changing `def` => `val` make a difference?


Oh yeah, I got I notification. But I only noticed the additional methods before.


For consistency with the above function and to be safe, add an "if (resrc)" clause like the function above, and throw in an "if (digest)" clause for good measure.


I'll check it out!

I've been working on figuring out an optimal truncation point for the asymptotic series.  Generally somewhat more than 8 terms are needed to obtain best performance.  Here's the optimal number of terms to keep as a function of `a` and `b`, at `z = 100`:

![optimal_terms_28](https://cloud.githubusercontent.com/assets/4148295/12453688/6a7c5cfe-bf62-11e5-8ba0-ff3bccbea159.png)

In many regions of parameter space it's OK to keep summing until the terms become too small to affect the sum (to double precision).  The terms don't necessarily monotonically decrease in magnitude at first, so finding the best place to truncate in general is nontrivial.  I've been playing with the error bounds, but haven't made much progress yet.

Oh, and there is also an exponentially improved expansion that I hope to check out, but haven't coded up yet: http://dlmf.nist.gov/13.7.iii

I read a paper [(Muller 2001)](http://link.springer.com/10.1007/s002110100285) that highly recommended a rational approximation due to Yudell L. Luke, so I tried my hand at implementing it.  My implementation does not give remotely accurate results for any parameters, so either there's a bug in my code or a typo in the paper.  I got Luke's original book from the library; when I hunt down the bug I'll PR this method into `hypergeometric.py`.


Nice code comprehension and cool feature :+1:


Don't know github app did this


What's new:
- Made the entire package smaller, removed unnecessary fonts!
- Changed background to a blue-grey, lighter on the eyes :)
- Fixed "NaN" for upload ratio when download is 0KB.
- Free space information has been added to session settings.
- Fixed "Delete All Data" that is not working previously.
- Fixed broken torrent pagination in search after changing torrent filter scope.
- Changed over to CSS transition for progress bar - fixes random progress bar spikes earlier on.


I quite like having solid implementations for the collections, also this is going to cause me annoying conflicts in my branch


Why is there no check for `stderr`?


Damn.


Because those instructions are useless. The old x87 is kind of replaced by SSE. And we don't need 80 bits float operation so except super VU that still uses a couple of them. Nobody ought to use them in the future.


My bad.


let it be Anna Plyakina ! ;)


style a externaliser dans custom


The ROM, NamelessROM?


By the way, the readme file is modified.


clic et non click


The logo should be an Image instead of an URL: https://github.com/ocpi/ocpi/blob/2.0/mod_locations.md#41-businessdetails-class


Bjr,
Il manque la balise ouvrante <html> entre <!DOCTYPE html> et <head> !
le html à droite de ! DOCTYPE n'est pas l'ouvrant de </html> ;-)


look good
Ack


I prefer to make it hard to do things badly.


Ack


Fixed vector2


There's a race condition here, in which the `XMLHttpRequest` might complete before the `caches.match()` does. This would mean that the XHR data is used to update the forecast card first, followed by the (presumably staler) data from the Cache Storage API.

I'd recommend refactoring the XHR into a separate function, and then you could do something like:

```js
if ('caches' in window) {
  caches.match(url).then(function(response) {
    if (response) {
      return response.json().then(function(json) {
        json.key = key;
        json.label = label;
        app.updateForecastCard(json);
      });
    }
  }).then(function() {
    doXHR(key, label);
  });
} else {
  doXHR(key, label);
}
```

Another approach would be to set a flag once the XHR completes to indicate that the fresh results are populated, and then only call `app.updateForecastCard(json);` inside your cache promise chain if that flag is not set.


note quite sure how it will react it would need testing a long time to give my ack on this


ContainerGroup does not have a section field ?


Done


That should be in ClusterTester so other modules could benefit from it.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1235.0f2e2a179b9f62e16fc1017fb2b8682f90ab6b77](http://build.palaso.org/viewLog.html?buildId=68150&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 846, ignored: 7 Build time: 00:06:27


Don't you think this is a bit "boilerplaty"? haha


The point of the emitter is to emit x86, I don't see that removing instructions can be considered to be an improvement and we might have use for them.  Maybe.


`try String(data: Data(bytes: bytes))`


as it says Default true should be `charging_when_closed = Some(true)`


I generally think this facility should not be limited to this particular subtest suite.


Cuma bisa index aja, http://cloud.libscode.com/libscode/DateTimePicker.css < yg ini gak bisa :sake:


FWIW, tools/check-style.sh seems to disagree.

Marcs-MacBook-Pro:phantomjs-ariya marc$ tools/check-style.sh
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/cookiejar.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/env.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/filesystem.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/networkaccessmanager.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/networkreplyproxy.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/phantom.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/repl.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/webpage.cpp
Formatted  /Users/marc/Projects/NewStuff/phantomjs-ariya/src/webserver.cpp


Don't these stacks only come from BHR and are, thus, Hangs not lags?


The branch can't be updated this way, the search&replaces have to be run again.


This makes you able to install packages from the controller when the master is disconnected but does so from the internet, not from the local repositories.
That is: some updates might break something.


Why not just returning insID.hashCode()?


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


I'd just drop this task.


I'd drop this comment leftover from WSK unless you really think folks will want to do this.


Ah, good catch!


This change was breaking for me. Now the tags file begins with a !_TAG_FILE_ENCODING line, and any calls to :help cause vim to hang. It's hard for me to tell so far whose bug this is, vimoutliner or vim. Any pointers appreciated as I continue to investigate this; I will be happy to provide any additional information.


Well, i think it has better clarity, that's all. I don't mind either way.


Took me a while to grok what this about, any chance we can improve the comment about this a bit? Say something like this maybe:

"We want valgrind to print its memory usage summary before reexecuting. Valgrind won't do this is on its on exec(), but it will do it on exit(). Hence, to ensure we get a summary here, fork off a child, let it exit cleanly, so that it prints the summary, and wait for it in the parent, before proceeding into the exec()."

or something like that.

Also, I'd really like to reduce the number of #ifdefs in the main program, hence I'd like to see this moved into a function of its own, in process-util.c or so, and maybe called "show_valgrind_summary()" or "finalize_valgrind()", or "valgrind_summary_barrier()", or "valgrind_summary_hack()" or so...

But conceptionally looks good to me!


potential memory leak situation


This look sensible


这个地方咋回事儿啊，文件编码统一utf-8


I do not know what to say, after this commit I noticed that cm profile is fixed. Before no.


所有文件编码统一utf-8


in this class/case ?


Pls add my profile also
- name: Anirudh beedikar
  github: anibeedi
  image:
   facebook: anirudha.beedikar
  bio: "A Participant of GCI 2015"
![photo](https://cloud.githubusercontent.com/assets/16526381/12454181/69598824-bfbc-11e5-8897-fbebd155c57c.jpg)


Maybe the (WINE-110)[https://jira.bikalabs.com/browse/WINE-110] problem is because of:
https://demo.bikalabs.com/knowledge-centre/manual/bika-3-user-manual/setup-and-configuration/analysis-services#precision-uncertainties
Hope it helps


Seems like a typo


thanks!


Hola esto es un commit intermedio :dancer:


doh! my bad


I will agree with you if the project was to make a generic lib. But there isn't no absolutely no use case for x87 FPU. Again the only interesting capability of x87 is long double (aka 80 bits float). SSE supports double (64 bits float) which is more than enough to emulate the PS2 float. SSE unit is faster and easier to use (no need to fiddle with the stack register management).

Only svu_lower uses a couple of x87 operations otherwise all recompilers use SSE already.


thats more than just aoa


:beers:


No, this is fine. That 'short name' is the name we will use for the document in this repo. (The 'short name' in the W3C jargon is the core of the identifier of the official document that is not dated and which is always redirected to the last published version.)


@Spoonikle Also an old version of Moreplanets, good catch but intended :)


I'm sorry for the added space (ich bin in der Uni und mit dem Ändern im Github-browser ist das etwas nervig)


change looks perfect


TeamCity Civic Assembly :: Build backend + frontend [Build 12](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=384&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 70, passed: 1597, ignored: 7 Build time: 00:17:37
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

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


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - 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)


Done : https://github.com/openwide-java/owsi-core-parent/commit/5319f952ba727a944620c596b2dc0d9faa7eddaf


hmmm ... having the attribute by default is not a disaster and I don't want to make a setting out of it ...

Does cloudflare work based on a wordpress plugin, or doesn't wordpress know about cloudflare? and is rocket loader a setting in that plugin?


Thanks for fixing this, it's is bothering me as well,

any plans to merge this into master?


That's true, I've changed it back.


Also, back to the current IP from server.


This is extremely annoying when deserializing thousands of objects in a loop, especially when this code is picked up in a minor release. Instead, this should log just once, just like `deprecate` does.


delete


could you submit it as a pull request? It would fix https://github.com/tomato42/tlslite-ng/issues/85


Ah damn one `escape_path_argument` too much...


Yes, it stops it working.


unack


working branch


Hello,

effectivement, je suis allée un peu vite en besogne, oups :-)

Merci ;-)

2016-01-20 16:47 GMT+01:00 Mickey70fr <notifications@github.com>:

> Bjr,
> Il manque la balise ouvrante entre <!DOCTYPE html> et !
> le html à droite de ! DOCTYPE n'est pas l'ouvrant de ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/claireisabelle/initiation_git/commit/020b5dfb6c2963735c477a1e4bbc4723c4945664#commitcomment-15561014>
> .
>


unrelated move


That's cool I will benefit from this.

Ack.


Please describe the intention - why we need to remount all cgroups.


while this "fixed" the archiver, the unintended consequence is that the archiver now runs all the time and FILLS UP THE DISK on edex...


Here, Ashok is attacking Refactoring #8 from our Trello. He is breaking down the temporary field dependence here by replacing fields with queries.


@benoit-canet OK, let me just run some more tests before I merge this.


ok. I wasn't suggesting that it **had to** be equal to the repo's name (I'm aware of w3c's shortname concept), but it w/b easier to remember. What does `lcr` "mean" here?


You would have to tell me - The handling of these refs hammers the collections, as discussed elsewhere you could add to the filtered set immediate (the actual outcome) and save all the intermediates, would need a bit of refactoring but would reduce churn.


Great. Will merge. Please rename sac92-isaac-base.pp to something like isaac-base.pp in a separate MR.


Why is dump_resrc_state() called unconditionally?  It is an expensive and wasteful function call if the log level is not set to LOG_DEBUG.


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


yes, I'll ping you when it's queued


This was really just a quick hack to make it not quite so rough. - if you fixed it already even better.


On errors `xgettext` should return non-zero exit code. In such cases `error` object is not `null` and it's checked in the sources above. `stderr` should be used by `xgettext` only for error's description. There is no need to check `stderr` if `xgettext` exit code is `0`.


comment "why" would be useful


makes no sense?


[Nuovo Documento di testo.txt](https://github.com/davide92/progetto-esp/files/97658/Nuovo.Documento.di.testo.txt)


I didn't know that, thanks.


You'll probably want to add in a `self.skipWaiting()` as part of the promise chain: https://github.com/GoogleChrome/samples/blob/gh-pages/service-worker/immediate-control/service-worker.js#L23


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


?? I am confused - please explain


@Ocramius @guilhermeblanco I found out how to reproduce the case:
```
$payment = $em->getRepository(Payment::class)->find(71493);
$oldEntries = $payment->getBookingPayments()->toArray();

$payment->getBookingPayments()->clear(); // clear collection

$newBookingPayment = new BookingPayment();

$payment->setBookingPayments(
    array_merge(
        $oldEntries, // reusing entries from cleared collection
        [$newBookingPayment]
    )
);

$em->persist($payment);
$em->flush();
```

fyi.: `setBookingPayments` does look like this:
````
public function setBookingPayments($bookingPayments)
{
    foreach ($bookingPayments as $bookingPayment) {
        $bookingPayment->setPayment($this);
    }
    $this->bookingPayments = $bookingPayments;
}
```


You'll probably want to add in a `self.clients.claim()` as part of the promise chain: https://github.com/GoogleChrome/samples/blob/gh-pages/service-worker/immediate-control/service-worker.js#L33


Note that I moved these definitions to the top (so that they can be accessed by methods in other scripts), instead of just being nested in the "if __name__ == __main__" clause.


It seemed a little better in testing. Could keep it the old one if you'd prefer


한글이 깨졌어요!


> On 20 Jan 2016, at 17:06, Romain Deltour <notifications@github.com> wrote:
>
> ok. I wasn't suggesting that it had to be equal to the repo's name (I'm aware of w3c's shortname concept), but it w/b easier to remember. What does lcr "mean" here?
>

It is the witness of my stupidity:-) and I misunderstood your question. Of course, it should be 'ucr':-)

I will change it

Ivan


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


It seems that the LZH library depends on python-dev to build after all.


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


Offenbar wird nicht überprüft, ob für path wirklich ein Pfad übergeben wird und auch nicht, ob die Rechte zum Lesen & Schreiben für diesen Pfad gesetzt sind. Das fällt dann erst auf, wenn die entsprechenden Operationen durchgeführt werden. Ist das Absicht oder wird das ein neuer Issue?


https://github.com/GoogleChrome/airhorn/issues/10 applies here as well, about the need to cache-bust these URLs.


 > `$payment->getBookingPayments()->clear();`

That will indeed cause orphan removal of all entries: expected behavior


@malirod I'm going to don't link this library at all within my next pull request. So let's keep this line here for a while.


Nö, viel wird da nicht geprüft. Stimmt.


You'll need to be very explicit about instructing developers to bump this version string each time there's a change.


Same comment as above: call only if the log level is set to debug.


Sounds good. Thanks.


This must not be removed : style used for whole theme
![capture du 2016-01-20 17-12-21](https://cloud.githubusercontent.com/assets/1439428/12454712/fedab78e-bf98-11e5-993f-2d10a7aa9c3d.png)


I'd point out the downside of the downside of deleting the entire set of previously cached files every time `cacheName` changes. Being able to finely control what gets expired from the cache is one of the benefits of using tooling that versions each individual resource, which you get "for free" with `sw-precache`.


```
$payment->setBookingPayments(
    array_merge(
        $oldEntries, // reusing entries from cleared collection
        [$newBookingPayment]
    )
);
```
@Ocramius yes, but the old entries are "readded". The question is if it is expected behavior that this does not work.


Maybe displaying the actual type of lastSeconds could help debugging in case...


haha, ok then :smile:


This check fails on my raspberry. I do not know what you want to achieve with this, but there is no `linux` folder. The if above also fails because of this.

But this is wrong. I did build the IDE, but I did it manually because of this toolchain bug. I suggest to keep the files in it original structure.


why?


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


Die Datei, die du zum Ändern genommen hast ist nicht aktuell. Keine Ahnung was du außer alles von mir rückgangig noch geändert hast...


This solution worked for me. Thank you.


I think you mentioned that you're going to remove the `gulp` build step, so you should also make sure you're using vanilla CSS instead of SASS.


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


Hey @anibeedi , Send a PR with your details added.


see also RTF-1448


ohhhh


Does it make sense to bubble up these error descriptions?


nothing happened since 2 month...

Can we help to get the version 1.9 released?
Can I support testing the lastest version of the plugin:

https://github.com/jenkinsci/embeddable-build-status-plugin


@SubJunk Sorry but parents have similar model as Valib, with no MKV support.


taking the dog out to do its business logic


are you speaking about adminstyle.css ?
if yes, its still loaded, but with the asset manager


Looks like `QThread::terminate` will be the hidden `pthread_cancel`/`TerminateThread`.
Please, verify it in Qt source to be sure.


I introduced this field to avoid situations where the resource is on the brink of expiration: The FileSource checks for expiration and determines that it's not expired yet, then, after a few milliseconds pass, the response is passed to the requester, which checks for expiration with the current system time and now treats the resource as expired.

However, given that the file source is monitoring expiration as well, and will performance a conditional HTTP request when this happens, we *should* always get a definitive, non-expired response.


I would maybe try changing the version to one that works for you.


just simple assignment `stages = []int{2}` should be enough


With how we've switched away from using LD_LIBRARY_PATH for hooking OpenGL and we are now using LD_PRELOAD we no longer need to generate wrappers for the functions we aren't intercepting so should be ok to remove this case.


what does PR stand for?


As of M46, this polyfill shouldn't be needed for `cache.addAll()` or `caches.match()`. Unless you need to support earlier browsers, you could drop it.

Firefox should support both natively.


@AdamBunn thank you for your help


Yeah, this is an unfortunate requirement right now (from the updated version of *Getting Started* that I just merged):

Note: On opening a notebook in Binder, you'll need to select the *Kernel* menu item, then *Change kernel*, and then *Python 3*. If you don't do this, you will get errors when you try to execute the notebooks (this step won't always be required - it should be addressed as part of [Binder issue #20](https://github.com/binder-project/binder/issues/20)).


was the bug here that a `datetime` was being registered as a `date`?


Hey Justin, it looks like you pushed directly to master here.  Given the small change, it's not a huge deal, but make sure your changes are on a different branch from master if you want to make a pull request.


mark it as TODO ("removing images race")


instead of passing here we should make sure that `dm.location_id is None`


I confirm it works with single SIM config


Reviewed.


lgtm


@slentzen I just got a github email about this, did you do anything? It might be a github bug...


My bad, thanks for this fix.


What I meant with #244 was to just add `implements InsertPanel` to the class definition. The methods are already there.


@ceci


You can even give feedback on a line by line basis in the pull request as well.

Good catch on this misspelling. This happens often with me. :smile:


yeah, kind of surprising, but `datetime` is a subclass of `date` so it was impossible to reach the last check and had to flip the order


:nose: :+1:


Also, do we really need the slot.size values if they are all going to be the same?


Yeah, but I think extending to ComplexPanel has already those interface method


See new PR: https://github.com/mkj/dropbear/pull/23


Yes, this is expected as well


What guarantees that this always sorts such that index 0 is the oldest one?


MaterialWidget extends to ComplexPanel :)


@ayebare I'd actually reverse this to only continue if `true !==` but not a big deal either way


TeamCity Civic Assembly :: Build backend + frontend [Build 13](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=386&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


map should probably be a subscribe. That way you avoid having void from map and also the publish(), connect()


I don't like "sleeping" to prevent race - we need find a way to synchronize remove with end of process.


Yes, I thikn a comment is appropriate here. I would also add an issue in github so that when time permits one could update the ci setup to use the latest version when/if it is fixed


I love this patch.

Does it work when in submenus of settings?


I don't think it's a good idea to return the result of `sol_vector_append` - if vector changes, pointer may became invalid. You may want to allocate memory and store them on a `sol_ptr_vector`.


-2 for another goroutine in background - especially if the problem is with `d.cleanup() only`


Ok. I don't think this is very intuitive. But I'll move this case to sonata.


* Ederson de Souza <notifications@github.com> [2016-01-20 08:19:18 -0800]:

> lgtm
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/glima/soletta/commit/ab6699c4caf866bc555440c9800ffb9b32c126d4#commitcomment-15561987

Thanks, merging.

--
Gustavo Lima Chaves
Intel - Open Source Technology Center


ok I will check
thx


@chenji-kael I'll try to get to it again this week


Good point.


Should this be `create_slide`? Or am I misunderstanding the routes?


Why is `Revision: V.1.4.5 a` equal to CMSIS `4.3`?
Where is the correlation?


No, the ComplexPanel class implements IndexedPanel, not InsertPanel.

```java
MaterialWidget materialWidget = new MaterialWidget();
GWT.log("InsertPanel: " + (materialWidget instanceof InsertPanel));
//prints InsertPanel: false
```


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Agreed.


What is the advantage of using lower case directory names?
I am not opposed to this, just wondering, what your train of thought behind that is.


PR = Pull Request.
Are you new to Git/Github @anibeedi ?
Why don't you begin with [**FOSSASIA: Let's get you started: Learn GIT**](https://codein.withgoogle.com/tasks/5373487673769984/) task ?


Nice to see, that we got rid of a 6 year old CMSIS library.
Who is going to check, if the LPC stuff still works, after we upgraded?


`function showUndoWarning`


default to an empty string?


add git command


All of them are `_` instead of `-`


Yes. In qthread_unix implementation hided, but for windws version
```void QThread::terminate()
{
    Q_D(QThread);
    QMutexLocker locker(&d->mutex);
    if (!d->running)
        return;
    if (!d->terminationEnabled) {
        d->terminatePending = true;
        return;
    }

    // Calling ExitThread() in setTerminationEnabled is all we can do on WinRT
#ifndef Q_OS_WINRT
    TerminateThread(d->handle, 0);
#endif
    QThreadPrivate::finish(this, false);
}
```


Are these files available under a well known open source license?
Maybe from the mbed repo?
If this is not the case, we should probably warn people, that if they use `xpcc` with LPC, that their program will be bound by the proprietary NXP license.


You can use sass variables for holding your color palette.
```
color: $color-1;
background-color: $color-2;
```


I suggest to omit the "whitespace" changes for consistency with above if's.


most older code uses `e` while most newer code is using `evt` for it - do we really need a third name for the event param?

![](http://imgs.xkcd.com/comics/standards.png)


Nice to see, that they started caring about trailing white spaces!


too late :smile:


White spacing is messed up here... Appears that most of the file is using spaces... :anguished:


:haircut:


ya, im new to git


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


I had to revert in 24a70e1 because it broke at least one test, possibly more.


Don't we have a DataQuery class with constants in it?


it should! thanks :)


Is the EmiData2PolyData tool part of the PR?


Could you please either remove this, are leave it in, but not comment it out?
The 1990s called, they want their source code on a network drive with different versions commented out back.


`&` ditto


Maybe we should check how many errors are above the LIMIT and delete multiple at once instead of only always the last.
Isn't it possible to log (and store) multiple errors that are either silent or don't crash the app (and VM)?
If ```delete``` fails we rely on ```deleteOnExit``` to work, but if that doesn't work as expected, we are above the 100 entry limit forever until the whole error store is emptied.
And ```deleteOnExit``` might not be very [reliable](http://developer.android.com/reference/java/io/File.html#deleteOnExit%28%29). I'm also curious what is meant in the docs by "when the VM terminates normally".

Hm, the impact of this would be pretty minor though as eventually the error store should be emptied.


Looks like your commits are a little bit out of order, but that's not a huge deal. If I checked it out here, the date filter is missing (for the test).


This should be Optional


Let's make it clear I'm not trying to create a standard here ;)


ACP does not require cookies anyway?


nice!


Where does `'valueSelector'` come from?


This should act like ```NbtTranslator``` no?


@mic4ael could be a "barber pole".


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


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


Test


could the out of order be related to the issues I had from switching computers?


Oh, I see. Passed in via the factory, line 42. This `static $inject` is not needed then since Angular is not building this instance. `$inject` is only used to tell Angular what to inject.


It would be better to use `unique_ptr` for `result` and use `result.get()` at this place.
This would indicate that the current function is owning the `result` , and the `mesh_io` is only accessing this and should not delete the passed `result`.


I felt the same.


Would *you* let him work that close to your throat with a sharp object? :D


Works perfect!


Here is a # missing


Why do you need this?


@Minecrell

I was gonna do this but was lazy since it'd require a visitor. Why Java doesn't have one builtin for such a common task...

Thanks anyhow :).


Why empty string?

You can keep the translation `tr("Amount");` or just `"Requested amount"` without translation.


Why do you use hard coded IPs? Why not `t1.openseamap.org` instead of `195.37.132.70`?


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


Testing against a different DB name. Calling geoip_country.open() on an asnum DB should return nil.


GeoIP country DBs support the INDEX_CACHE flag.


@shun-s could you fix the indent again?


The function  only draws chessboard corners, it hasn't returened values.


Wrapping the p.module.open in a pcall so that we can (potentially) catch errors, if testing fails.


__API Description file syntax error on line 102__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


done


wrong indent.


I guess the case where the option modal is not available only happens during GUI setup. IIRC no user will see this state.


default docker jessie image doesn't include source repositories, so you can't use `build-dep`


Argument for putting the visitor in `SpongeAPI`? I mean it's possible it'd be useful to plugin authors and it's hardly implementation-specific?


and you might want to check other headers to see if we are aligning the declaration or not. or am i missing anything?


This was a holdover from my testing - I'm not sure what we want to be testing against multicast IP address ranges.


ACP will be switched over to cookie-only, but this is not part of this change and will be done at a later point in time. I've just added this comment for the future, so that everyone can understand why I explicitly opt-out the security token within rescue mode (which is now finally really working no matter how screwed your domain settings are!).


Et si on linke un thread à la saison puis les messages au manches?


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2089 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-99.95%25-yellow.svg)


It should be a list of Capabilities, here and in the Patch one


Ah non je vois maintenant.


TeamCity Civic Assembly :: Build backend + frontend [Build 13](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=386&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 69, passed: 1598, ignored: 7 Build time: 00:18:50
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

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


nice


Any idea what exact issue this fixes?


Aren't you ignoring errors now instead of catching them?


might want to move the definitions to where the defined variable are firstly used.


Agree, then uncaught errors in any of the registered render functions will also be logged to console automatically.


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


TeamCity NEventStore :: MongoDB :: Build [Build 50](http://build.prxm.it:8813/viewLog.html?buildId=3398&buildTypeId=nes_mongo_build&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:46


Yes, if we don't get a successful response from the DB ('ok'), and the result isn't nil, I'd think that we might want to dump an error, and not proceed with the test suite.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.657.37f942711143700fc2807d239dc62b71bd5e919c](http://build.palaso.org/viewLog.html?buildId=68152&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:12


please squash.


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


Probably should, this was just a quick converter class I made, I could improve it.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 128](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=697&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:25


please do not change this back until this issue is completely fixed.

This below happens when i click the drop down..

`Warning: flattenChildren(...): Encountered two children with the same key, `.$251`. Child keys must be unique; when two children share a key, only the first child will be used.
`warning.js:48 Warning: flattenChildren(...): Encountered two children with the same key, `.$264`. Child keys must be unique; when two children share a key, only the first child will be used.
^^ that happens about 1000 times and while it is happening, the user waits for the drop down.. finally the below error shows up.

` ReactElement.js:111 Uncaught RangeError: Maximum call stack size exceeded

<img width="1195" alt="screen shot 2016-01-20 at 11 41 48 am" src="https://cloud.githubusercontent.com/assets/3793595/12455784/fb054b0c-bf6a-11e5-8282-cb5d1eed1fd9.png">


I think you can simply remove the param name. I dont think you need todo this kind of trickery.
I usually do something like:
`virtual int getInteger(unsigned int /* index */) const { return 0; }`
But
`virtual int getInteger(unsigned int) const { return 0; }``
Would also work.


OK, but why did you remove the check?


Tested ACK  df6e8e1 given that

> no user will see this state


I started uploading to the new tile server before the DNS had been changed.


could you change the commit message as following?

```
test: add unitest test_pidfile.sh

Fixes: #13422
Signed-off-by: shun song <song.shun3@zte.com.cn>
```


It still is - the unified raw change bears this out.


Yes, but why did you change the existing check? Wouldn't it be better to add a new one?


use `su-exec` instead of `gosu` to save 2MB. It does same thing and is available in alpine repositories  (apk add su-exec)


What does this change mean practically for code usability / stability?


Okay, but please leave a remark about this in the comments


TeamCity NEventStore :: MongoDB :: Build [Build 51](http://build.prxm.it:8813/viewLog.html?buildId=3399&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:02:29


Semantics of the existing tests did not fit 100% with the asnum output. Instead of returning a data structure like the city/country DBs, the asnum DB returns only a string. We needed to catch some of the tests on asnum so that we weren't attempting to index a string.


Also, under elif lang == 2: there should be an idented line that says print('Hello World').  I don't see it maybe you can...


Yeah we do, I was just using the existing constant. IMO the queries shouldn't all be in one class but instead be in a class that's related to where it's actually relevant. (You won't need to prefix names e.g `WORLD_CUSTOM_SETTINGS` because the package already tells you what 'domain' it's in)


the branch? Do you mean es6 branch.. cannot really see what this comment is attached to / maybe I have deleted it..


Go ahead with the task then. Also refer [try.github.io](https://try.github.io/levels/1/challenges/1) for a quick tutorial.


-0.5 directory of files for user
-0.5 Task 3 not presented with a description explaining what is occurring (although it's self explanatory...)
-0.5 Task 4  not presented with a description explaining what is occurring (again, it's self explanatory...)

Total: -1.5 ===>   13.5/15


use `#!/bin/sh` to avoid `apk add bash` and save 800k. Script will work with minor modifications (may work as is too)


And if err is nil, but ok is false, should we continue? Why?


ok


does not work as long as plone.batching 1.1 isn't released...


@RobotXLabs Thanks!


I didn't remove the check, I instead re-enabled it. Previous behavior was to not attempt opening the country DB if the flag were INDEX_CACHE. It will now attempt to open the country DB with that flag.


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


We will need to add `covar_matrix` to the docstring for `get_draws`.


Yes, this may be an oversight on my part. We can catch them with an ok, geodb check.


Why not `this.count <= 0`?


TeamCity NEventStore :: MongoDB :: Build [Build 52](http://build.prxm.it:8813/viewLog.html?buildId=3400&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:44


You load only style/admintheme/adminstyle.css not style/adminstyle.css

Fixed in https://github.com/LimeSurvey/LimeSurvey/commit/517c9b5160f4e34ca793c7e8f12330004cb5adde

And : goes to survey show the problem in less than 1 minute


not sure if `"${1:0:1}"` is posix shell. (may be it is, i dont remember). Same thing that definitively is posix shell and slightly more readable (imho):
````sh
case "$1" in
    -*) set -- mongod "$@";;
esac
````


I was getting  errors with it when compiling. Now that I have a working
toolchain, I'll add it back in

On Wed, Jan 20, 2016, 10:33 Eric Wieser <notifications@github.com> wrote:

> Why did you delete the color sensor? Is this just a bad merge/rebase?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mitchgu/TAMProxy-Firmware/commit/613b013b9397958d34938e619bb175af178fe194#commitcomment-15560478>
> .
>


That's correct - hide the log button.


corners is rewriting, added as well as returned value


This is semantically identical to the existing error-handling logic, the only difference is that the asnum DB returns nil instead of "not found" if an entry doesn't exist.


This was a failed attempt to remove boost::asio from getting included a lot more than needed. The class is deprecated, we should just use `boost::ip::address` and take the hit on the includes. Especially since it will become part of the standard library soon-ish.


Bitte immer mit { } !


use `su-exec` on alpine. does exactly the same thing as `gosu` but with a fraction of the size. (and is available in the official repositories)


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

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


> Package IDs may not contain any spaces or characters that are invalid in an URL.

I think this is a wrong statement considering that the [Regex being used to validate this](https://github.com/NuGet/NuGet2/blob/eac0e51ceafc42cbebb0c66a64d15bf11e10516d/src/Core/Utility/PackageIdValidator.cs#L11) is the below one:

```
^\w+([_.-]\w+)*$
```

which doesn't allow some characters that are valid in a URL.


Should this say `And previous` instead of `add previous`? Also, we generally use sentence casing. So no uppercase 'P'


Does this fix the issue? I've got a fork with daigotanak's temporary fix but I'd like to stick with your official solution. Is this ready to re-download? Deleting the file at /usr/local/bin/docker-machine-nfs and rebooting should be enough prior to re-downloading the script, right?


What XXXXX means?


@neue @alex5imon Do we need to include any `license` files here or just naming them is enough?


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


This change is making it impossible to run http://try.orientation.io as an open sign in platform so there's a good chance I'll kill it because there are server-side validations happening on the User model anyway to allow or disallow signing in.


stringInstMap.put(insID, s) is not needed (it is already in the map)


Would people generally expect increment on the right and decrement on the left?


done, it was tab


I am not too happy with fetching patches that gets compiled in to source from http with no verification that it is unmodified. (think man-in-the-middle-attacks)


you add a dependency on the test, but the test isn't in this commit, its in the next


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


One question. Why?


Thanks Sidney,

Looks good, though the `*/.gitignore` needs to stay. The reason for it is that I needed to commit *empty* directories to show the structure of the project (which is not possible, so every dir also contains a .gitignore).
Additional information on the other elements of the .gitignore can also be found [here](https://github.com/dutri001/paperProject/blob/master/README.md), though your comment works well too.

> The reason for the /latex/* entry in the .gitignore is that latex generates a lot of temporary files which you absolutely do not want to commit to git. The rule is to only commit files on which you make changes and not commit any automatically generated file, and that includes the pdf of the manuscript. When you really want to add a file to the git index, it's possible, simply try to do it; it's not going to work but git will nicely tell you how to do it.

Are you considering to start a PhD ? ;)


hello


Hi Anatoly,

On 01/18/2016 11:15 PM, anatoly techtonik wrote:
> I don't have access to pull the request. Just trying to get some
> answers early.

    Cheers for that.  If you spot possible issues,  I'd want to
know about them sooner rather than later.

> Why not to rename this to wingdi then?

    I'd not have a problem with renaming it to WinGDI... obviously,
I've not been in a big rush to do so, but that's mostly just
because I thought it might cause confusion and also would require
a lot of changing 'Win32a's to 'WinGDI's. (Though I just did a
grep -ir win32a * in the PDCurses directory, and it doesn't look
as bad as I thought... it's a mere 44 lines, in fact.)

   But I'm holding off on this and a few other changes until the
current pull request goes through. I gather there is some other
forking going on, and don't want to make resynching with (for
example) Laura Michaels' SDL2 fork any more of a nuisance than
it needs to be. It's already a pretty good-sized chunk of code.
Fortunately, Mark Hessling's X11 changes were quite easy to merge
with what I had.

> And in description say that it uses most compatible Windows
> API to create windows and draws on them through GDI.

    Make that "it uses the Win32 API to create windows and draws
on them with GDI", and I'd be with you.  (Might as well be
specific about _which_ API it's using.)

-- Bill


Does this require an `{{ ansible_pkg_mgr }}` variable after the `>`?


Would it be better to use the date utility's `compareDates` function?

https://github.com/RenovoSolutions/TypeScript-Angular-Utilities/blob/master/source/services/date/date.service.ts#L128

Or better yet, `dateInRange` appears to be designed for the this! :grinning:

https://github.com/RenovoSolutions/TypeScript-Angular-Utilities/blob/master/source/services/date/date.service.ts#L134


TeamCity NEventStore :: MongoDB :: Build [Build 53](http://build.prxm.it:8813/viewLog.html?buildId=3401&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:36


TeamCity NEventStore :: MongoDB :: Build [Build 53](http://build.prxm.it:8813/viewLog.html?buildId=3401&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:37


My $0.02: I don't think our standards state whether subclass overrides of virtual member functions should be declared explicitly as virtual (as @jdswinbank says, it's a no op), but most C++ coding styles I know of suggest that they should be.  It just makes it easier for readers of the subclass to see what's going on.


Reason for removing unique constraint @rohitpaulk ?


Hm this looks weird - I don't believe ResourceData should be forcing callers to make `schema.Set`s - it should just accept `[]interface{...}` directly and convert to `schema.Set` internally.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 73](https://teamcity.labkey.org/viewLog.html?buildId=298214&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 30 (13 new), passed: 18 Build time: 00:54:48
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


I mean issue #8


2245154-NewVRFHRModel (rongpengzhang) - x86_64-MacOS-10.9-clang: Tests Failed (1663 of 1980 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-83.99%25-red.svg)


111768812_VRFNoLoadOAFlow (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 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)


A better comment would not hurt. What is the reason behind merging Libfalltergeist?


Oops... (I'll fix that in the next days.)
Btw.: You asked me to test parts of your changes to OpenOS? Where can I find them?


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


ok


Is extra \n needed ? Or can this line be simplified to :
```fmt.Fprint(self.conn, cmd)```


In the event that the expression on line 109, ```pgp == NULL``` ever evaluates to true, the value of ```dmmp_pg_count``` will be larger than the actual number of pgs  and ```dmmp_pgs``` will have a NULL field at the end, I think. Given that multipath's vector implementation allows NULL elements, so that VECTOR_SIZE really corresponds to the number of allocated slots, and that the multipath methods that yield these vectors do not seem to guarantee that all slots in the vector returned have non-NULL values, might it be better to repack multipath returned vectors before using them? Assuming that generally multipath returns full vectors, that might not be too expensive.


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


What the f*ck?! Why do you remove Lua???


In theory, ```pp``` _could_ be NULL here, unless ```pgp->paths``` is certainly full.


PlantCompBaseClassAgain (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed (1727 of 2056 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-84.0%25-red.svg)