I think we can combine these two into one.

```java
K oldKeyToPreserve = this.inverse.delegate.get(oldValue);
this.inverse.delegate.removeKey(oldValue);
```

to

```java
K oldKeyToPreserve = this.inverse.delegate.removeKey(oldValue);
```

Saving one execution of `hashCode()`. After that do you think there are any more calls to `hashCode()` we can eliminate?


Great work. Clean and efficient code


I am glad this is not K&R C where that would actually compile.


@chilek: była powolność i tu się zgadam, szczerze mówiąc chyba nie co szybciej na tej wirtualce mi biega. mamy coś za coś, po prostu trzeba pamiętać o update composera i tyle :)
pozostaje jeszcze to nieszczęsne TCPDF po przenosinach


Are there any assertions to make about duplicateOfValue2? If not, you could remove it from the test.


:cry:


Yes, tested with Android 6 and so far no problems


Had a chat with Stas. One example where this might happen is when the "host IP" configured is a public IP with 1-1 NAT, like AWS instances. I'll update this after our calls.


Fixed that in a later commit :-)


:hea


Great work. Nice implementation of another service.


Are you referencing the non-compiled forms? They should be bundled (into a single file) with no need for the actual modules.


The read me ends with "Please report back to us any questions/comments/concerns" What is the preferred way to do that?


@eclarizio OK, that makes sense.
@dclarizio LGTM


thats so much easier! I can't wait to use this syntax!


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


Nein ...
SELECT entry, AIName FROM creature_template WHERE entry=12918;
"entry"	"AIName"
"12918"	"SmartAI"


Typo "invlid" --> "Invalid" (and also in the next warning message).


In the samples directory in master, there's an existing AssemblyInfo at `/src/Properties/AssemblyInfo.cs` and that's what I was matching here. I can change it back, but if I remember correctly the project didn't build with both. I'll probably need to regenerate and delete `src/Properties/AssemblyInfo.cs`.

That's why I thought it was weird, also generated for me to `src/main/csharp/IO/Swagger/Properties`. I just didn't want to be causing a regression with this.


i'd move the switch to another function, just to have this one shorter but i'm okay with leaving it this way


If you're using pre-built stuff you're gonna have a bad time. It should be avoided. If you really want to dig into its guts then you need to mimic closer redux's bundling setup.


картинки ещё и ссылки


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.500.0](http://build01:8888/viewLog.html?buildId=14160&buildTypeId=ConfigInjector_CI) is now running


This got misplace somehow.  Need to delete "AirTerminal:SingleDuct:VAV:Reheat:VariableSpeedFan"


needs the choice for firefox....
http://stackoverflow.com/questions/7435843/window-top-document-body-scrolltop-not-working-in-chrome-or-firefox


Right.
But some tests cannot work on travis.
But for others it is good approach.

2016-02-08 21:59 GMT+01:00 Marcin Kosiński <notifications@github.com>:

> To Have more checks than we Have in tests. If it causes problems on travis
> this may mean we Have not-working exampleS and we should fix them
>
> Marcin Kosinski
>
> Dnia 08.02.2016 o godz. 18:44 Przemysław Biecek <notifications@github.com>
> napisał(a):
>
> > @MarcinKosinski why have you removed the dontrun section?
> > I notice that this sometimes causes problems on travis
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pbiecek/archivist/commit/cdd89423ce3b451c57d26a2161af9a9ebb9a71f6#commitcomment-15959587>
> .
>

--
pozdrawiam serdecznie,
Przemysław Biecek


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.500.0](http://build01:8888/viewLog.html?buildId=14160&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


Hi, i changed it on my site, but the its still the same, i have one.com as provider and i know that ist can thake about 10m befor its regist by one.com. But the probleme is the same.


Good to know, thanks. FWIW I copy-pastad the pattern from `TemporaryJobs`. ;)


you're right


брать ссылку в div не надо


Here should be `5.1`
Also you need load luaciv while do testing.
You can checkout my `coveralls` branch.


I'd guess the l3_db base class uses the core plugin just as much as this class, so why not use _core_plugin here, and if caching is a win, modify that to do the caching so both layers of code benefit? But not sure if caching helps.


@sarahjean This and other changes that have been made to the site, removing or redirecting the old site content, is likely to have a negative SEO impact. It also makes it impossible for users to login and removes last years session pages and what not.

I realize I haven't been participating the last few months, so ignore me if this was discussed/considered.


Scoll down :arrow_down: for my comment.


Тебе нужна будет одна или несколько обёрток для слайдера сразу после `body`.
Если мы будем делать слайд через CSS.

Ты также можешь посмотреть скринкаст и демо: https://up.htmlacademy.ru/htmlcss/12/module/6/screencasts, https://up.htmlacademy.ru/htmlcss/12/module/6/demos


chrome is depreciating these in April 2016 for svg elements.  (We use svg a lot with popover)


// Here is some information on the Project //


+1 w/imod


Please take a look the [new commit] (https://github.com/NuGet/NuGet.Client/commit/4fb4638bfb910b91ae0e293cdf53db63aa4e6dcd).


в .slide_title не надо b
b можешь использовать в маленьких и не несущих большой смысловой нагрузки заголовках


check out my new commit. Note, the "Clone" is internal, so I made the test project a friendly assembly.


Высота сама будет задаваться контентом


10/10 commit message, would employ


@cam156 where are you seeing this error? I'm not able to reproduce it.


padding: 0 27px


:+1: Old habits.


Is this name correct since below is checking `raiseEvent`?


@sdenizot this change appears to interact with line 133 in this controller https://github.com/tranSMART-Foundation/transmart-gwas-plugin/blob/release-1.2.5-Beta/grails-app/controllers/GWASController.groovy such that the hibernate SQL therein fails. Are we missing a companion change in the controller?


done.


Отрицательный `margin` нужно использовать редко и с пониманием зачем. Здесь ты можешь обойтись без него


done


You can combine this test with the `load progress events are raised` one. Also do the same thing for `tile visible event is raised`.


I will incorporate the change in the PR for xplat cmdlets to avoid the conflictions


Never mind...


This should move down to after (perhaps) scaling by y.


@Martinsos I forgot to consider that case. The reason for change was that when pathname is `'/'` it would try to compare `http://localhost:3000` with `http://localhost:3000/`. Not equal because of the slash at the end. So I will either need to make pathname be added only if it is not '/' or dynamically constructor URL object from redirectUri and then check if paths match.


:+1: Ah, it sounds awesome! Guys from phpleague has many cool stuffs.


Not sure how wise this is. I've always avoided hardcoding Railsisms into regular cops. Some config would be better.


I'm running with devstack on stable/kilo. Even with this change, I still fail on:

> 2016-02-08 20:53:47.024 | pkg_resources.ContextualVersionConflict: (fixtures 1.2.0 (/usr/lib/python2.7/site-packages), Requirement.parse('fixtures>=1.3.0'), set(['testtools']))


You might want to remove that token and add a dummy...


I don't follow.  Any exception will be caught in libaps2 and turned into an APS2_STATUS return.


_cond, _if_branch


"size after fitting to maximum width:  "


"size after fitting to maximum height:  "


missing conditions to spell when Jovaan isn't present & old conditions are still there


izbaci `x` ispred svih imena, kako bi strana bila cela


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


The coffee to JS here makes me kinda cry with happiness


I am using CustomActions because implementing any but the most basic conditional actions and checks in wxs/MSI is quite the undebuggable nightmare.

If you can do it (easily) inside the wxs, please replace the stuff.

Some of the custom action code is nout used here (e.g. DeleteBundleItems which is used to remove orphan entries by custom bootstrappers).


It's part of the [LLVM Coding Standards](http://llvm.org/docs/CodingStandards.html#keep-internal-headers-private). Headers in include/ are meant to be used across libraries, headers in lib/ are only used within that library.


> The suggestion about using async was to include it in a way that it finishes after the assertStepEnter code so that it minimizes the need of a setTimeout in the middle of the code, by I might be saying bullshit since I haven't digged deeper into this test implementation, so I might certainly be just scratching the surface.

Ok, I think you're on the wrong track on this one.

> About the test time, could we override the CSS transition declaration so we are able to remove the animation altogether and make the tests faster? It is at least possible to assert the classes that impress adds/removes on each animation.

We could, but I'd like to run with defaults for these core tests, and let other tests change these values.

But I guess this is a bit in the wrong direction anyway: it is not a disaster if the test takes 5-10 seconds longer. The core of my argument simply that I don't think your suggestion makes the code simpler. It just splits into 2 parts what I consider very clearly is a single test.

I can try to rewrite the logic that ties together the impress:stepenter assertions vs the impress:stepleave assertions, while keeping them in the same test. Would that make you happier?


There is also an implementation of BinDecompressor within the KNIME Parso node. The -forked- parso library is available from the [update site](https://update.knime.org/analytics-platform/UpdateSite_latest31.zip) with its source (pretty big file, the source is at `org.knime.update.org/plugins/com.ggasoftware.parso.source_1.2.3.v201511241214.jar`, sorry I could not find out the direct link to that file).

I have not yet had a chance to check the changes you made with the next commit. (We had some binary files that were not parsed with the 1.2.1 version of parso, so it was easier for us to use a different implementation.)
Thanks, gabor


> The other keywords which can be used in this way are `super` without args (a
`zsuper` node), and `defined?`. But SpaceAroundKeyword doesn't check either of
those. Maybe it should?

Likely an oversight.


@cbioley I just thought it's more convenient to have all imported reducers (both app reducers and third-party reducers) in one place.


Got it, thanks.


We could go the other way around for now if you want. We should honor `@inline(never)` for both SIL and LLVM.


Great work on going for some of the stretch requirements. Nice clean and efficient code.


excellent :+1:


FYI: the original was -26...


Nice :+1: this is good enough for now


Because Reika. I have almost no real power, and it's extremely frustrating to have my decisions overruled "because" with no warning, no notification, and no discussion. The only reason I even KNOW about these is because I keep asking for the server configs so I can at least keep the damn git updated.

Reika likes them to be there and won't build the things with schematica, buildcraft, or multiMC; anything like that. He is the ONLY player that likes them, and at this point I'm going to be asking for a nearl-total map reset after he makes tweakfix able to kill the stupidity in everywhere that is NOT his territory. I'll make a few backpacks and store my whole bloody haul.


the if can be ifdefed out completely on Mono, we don't run on Win2k


`line-height` больше `font-size` – недопустимо


same here


Using a NSMutableString is probably much faster than this for very little additional cost.


Player spielt bei mir nix mehr ab :(


This is actually an xor. But as long as it works...


Hey Robert,

This revert reverted back most of the commits I made a couple of days ago (in that sloppy pull request of mine).

The functions affected are:
ft_electrodeplacement
ft_volumerealign
ft_plot_slice
ft_plot_ortho

Any idea on how best to revert this revert partially? Perhaps a revert to the commit where you did the pul request? Which should only affect the above functions. But this would have to be done before approving the pull request I just created (which removes ft_intracranialplotER, which was added in that big sloppy pull request).

Another option is for me is to redo the changes using a separate branch.


It's a good idea. What do you think about documenting it on the recipes wiki page associated with the repo? Maybe in a "How to pip install additional libs for Python 3 and 2"?

https://github.com/jupyter/docker-stacks/wiki/Docker-Recipes


margin: 0 auto 290px


margin 22px 0 0 0


In the future can you try to match the tab style? Thanks!


@arq5x no release was tagged as far as I can see: https://github.com/arq5x/gemini/releases


`malloc(dataSz * sizeof(char))` outside of the loop and using that same buffer would obviate the need of multiple memory allocations.


You are right, I mistype :D


Все супер, только можно чуть-чуть оптимизировать. В каждой функции ты выполняешь один и тот же код. Чтоб этого не было можно сделать так:

1) спанам в разных блоках дать одинаковые имена и вместо строк типа `$('.random_choice_size').append(products[random_choice].size);` написать `$('.random').find('choice_size').append(products[random_choice].size);`

2) вынести одинаковый код в функцию и передавать в эту функцию параметры: класс parent элемента и индекс массива с продуктами:
```
function renderProduct(parent_class, products_index) {
...
$('.' + parent_class).find('choice_size').append(products[products_index].size);
...
 }
```
и вызывать так: `renderProduct('random', random_choice);`


сразу после `inline-block` прописывай вертикальное выравнивание


Split these comma lists into separate lines?  That would make it easier to edit the list by simply commenting whole lines.


тут что-то намудрил
(*) в slack


I'm not sure if I missed a fix for the `env` parameter issue. If I did sorry, I'll have another look. If I didn't, would it not make sense to have `http://localhost:3000/?env=staging#/` instead? Otherwise it breaks.


Did run the tests to make sure this doesn't need a `SetLineDash(0, null, 0)` in an else case?


I am biased to running two controllers.

controller 1: manages kube objects
controller 2: manages openshift objects


At this point, two JPEG thumbnail extractors are enabled.  Problem?  Comment one?


ignore that typo pls


I don't think we need this here. How about we let Draw just draw the map, and make a Board that will have mouseListener etc. In Board we define what is happening with moving the mouse and clicking and that stuff. What do you say?


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


Cleaned up Auton Helper


Will do. My apologies for any inconvenience.

-------- Original message --------
From: Darklight-03 <notifications@github.com>
Date: 02/08/2016  3:05 PM  (GMT-06:00)
To: Team5427/robotics-code-2016 <robotics-code-2016@noreply.github.com>
Subject: Re: [robotics-code-2016] made shooter timeout (b4b881b)

Please do not push to master. use:

'git checkout dev'
once to switch your active branch to the development branch.
then whenever you want to push, use:

'git push origin dev'
after commiting.

Once it is determined that dev has necessary changes and is functional, we will merge them.

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


Maybe yes, but I think that's more a thing how you like to have your filenames. It's not really a bug.


Hi Gustav,

I remind me that this is not enough to compile the current anyfx gl shaders with opengl 4.3 hardware. Did you try this?


Running two controllers would potentially cause increased memory usage since both would be watching `ResourceQuota`.  If we don't do it right away, it would be good to at least have a comment to evaluate doing so in the future.


why creating a variable if it's only used once?


Godoc please


Godoc incorrect


LGTM


lgtm


Perhaps go ahead and convert the commented Branded Preview JPEG, ImageMagick Image Thumbnail and ImageMagick PDF Thumbnail entries to the new configuration style?


Ok, just removed it and finished to build; after flashing, SD card was shown "corrupted" and could not be accessed....


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2090 of 2090 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


Same as other comment.


Hey, I'm working on it. Keep you panties on.


@kyprifog, please be sure to PR these things so that it can be reviewed for English and correctness. This one is fine.


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


I agree.


Thank you.


commenting here so I can look at what this is... I expected a UsageFunc


I think they should return `new X509Certificate[0]`.


Do you mind submitting PR for this?


Good catch. The two are essentially the same. It seems we can drop instantiatedParents in favor of parentsWithArgs, which is simpler.


Thanks. I'll get to work on it!
On Feb 8, 2016 3:24 PM, "Marcin Lewandowski" <notifications@github.com>
wrote:

>
>    1.
>
>    See here: ayende#3248 <https://github.com/ayende/ravendb/pull/3248>
>    You can copy queryUtil to v3.5
>    2.
>
>    I wrote: What is result.length ===0 => should be what if result.length
>    === 0 (then we end up with NRE)
>    3.
>
>    I'm affraid you have to create new PR and please remember to use v3.5
>    as branch start point. Master equals to v3.0 which can contain bug fixes,
>    but this is new feature.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciel/ravendb/commit/fc85b5948c07eaa2c09d4a61aa50749461a5cc0d#commitcomment-15960714>
> .
>


5429_ConstructionInternalSource (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2060 of 2061 tests passed, 0 test warnings)

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

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


@btovar this might need backported to any new releases.


Where is "converted" defined?
I mean, it looks like converted is only defined #if DBG
(Same for the rest of the added asserts)


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


Parsing here is bad, it will prevent you from setting flag vars in packages that include log.


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


commenting here because I did not expect to see this....


the name of this just keeps changing :)


Do we want to log the error here, or do you think that's unnecessary?


missing TODO?  I would expect us to replenish here...


Great work! Code looks clean and result amazing.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.672.5eb8f4c6a37abc9278bc6430b9e11f0f3436479c](http://build.palaso.org/viewLog.html?buildId=68979&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:10:48


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1039.5eb8f4c6a37abc9278bc6430b9e11f0f3436479c](http://build.palaso.org/viewLog.html?buildId=68977&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:11:24


You don't need the AlignIO module, do you?


Can you move this to the port string commit?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.671.e1f0948e29055f6a35aa17870acb9573cf811234](http://build.palaso.org/viewLog.html?buildId=68978&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:11:26


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - 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)


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:


You might also want to handle the EILSEQ case for invalid multibyte sequences in the same manner.


are you 100% sure that is right?
I thought the course data was e.g.
1 100
2 200
3 100

rather than
1 100
1 200
1 100

??


Hi There.
 This removes tags and potential JavaScript scripting by the user. It also prevents potential remote code execution  by the user(The user being the one using the form).

A nice guide can be found here: https://www.owasp.org/index.php/XSS_%28Cross_Site_Scripting%29_Prevention_Cheat_Sheet


Leave a space between the `##` and the text of the headline. So `## Firstly, GO FOR IT. It's not as scary as you think.`


slideshow didn't build properly. fixing now.


Chris, I think that you may be modifying text that I commited a while ago, and could have a conflict. Aaron and I are qute a ways ahead of you.


I don't understand the need for this.

Can you not just run your own registry for downstream?  I don't want OpenShift objects merging their registry with what is in upstream.  For admission, OpenShift API server is only looking at openshift objects, so there is no need to blend.  If we ran two controllers to handle replenishment, there is no need to blend either.


Check out the sample entry to see how you can actually get this stuff to look like code when it's published.


If `pycouchdb` is not installed, Ansible exits with the following error:

```
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: AttributeError: 'module' object has no attribute '__version__'
fatal: [localhost]: FAILED! => {"changed": false, "failed": true, "parsed": false}
```
I think this check should be moved down to somewhere below line `169`.


godoc


can u elaborate more on why this is needed versus just UsageFunc?


Sorry for the copypasta.


Great work. Code looks clean and simple.


LGTM. Thanks!


I was using shell meta-chars in the file name, don't work since the shell is never involved with the call.


Sure, sounds good. As long as the URL is not a part of the string, I don’t mind any string!


@icehess i'm preparing a change to this


hmm, read the commit history more, and see that you want a context function to support a cache.  I am trying to think about the ramification of that.


i fixed this in the upstream PR


привет


@waylaidwanderer i think if we set cookies dir we mean that it's created and has needed rights. So script changes it to its own and we can't handle it. It's not logically.


Hi georgesequeira,

A proofset with this commit has been created for triggermail_sublimetext_plugin. Just for you!

[View Proofset](https://www.bluecore.com/admin/email_proofs/triggermail_sublimetext_plugin/proof_set/5700305828184064)


Thank you!


Yeah my code wasn't cleaned up yet :smile: you might find more of these zombie-codes in the future. Will try to clean it up at some point, but currently I'm trying out stuff too much^^.
Thanks for the review!


Thanks!


1. I'm guessing the intent is to (1) not have to reconfigure the lib on every use and (2) especially not for baking in private keys. This addresses 2, but might as well nail 1 at the same time? Namely, handle the other `register` parameters at the same time: `server`, `protocol`, `api`

2. ( + @thibaudh ) How should we document beyond sphinx docs? Maybe (a) a helpful comment in the `register` example in `readme.md`, and (2) someone else (me/you/paden?) updates the examples with the same comment?


Should be `$this->assetId` ;)


Please may I know why this revert happened?
I would have really used this feature.

Thanks in advance.
Mirek


The player shouldn't move, so it should be fine to keep it that way. Since I want infinite levels, I don't want the player or platforms to move to some engine boundary :stuck_out_tongue: fixed with 8ebc62874229d62cfdd34df3263e5a186ae1ebb6


Do we really need this Param class?  Can we use [Readable](http://hackage.haskell.org/package/readable-0.3.1/docs/Data-Readable.html) instead, or is this class intended to be specific to query string param parsing?


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:


What about Netty Acceptors not being able to connect due to already connected exceptions? would that cover it?


:vhs:


Looks like this method needs to take an argument of some individual Assignment object


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.681.e1f0948e29055f6a35aa17870acb9573cf811234](http://build.palaso.org/viewLog.html?buildId=68980&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:07:37


> Were you using -o NUL and noticed the issue after upgrading?

Yes.


The links are indeed reversed, but they have been fixed: https://github.com/restify/node-restify/commit/ebe65f241cbe54978e975f6de02e7081b527c946


This looks fragile. What if there's another element inserted in DOM between `.posts` and the posts? Why not using `.post` for this?

Also, there's no `.posts` on a single-post page, so it not going to work there.


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


Referencing: https://github.com/websharks/cometcache.com/issues/83


Looks good for the login info.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14917](https://teamcity.mesosphere.io/viewLog.html?buildId=128584&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 225 Build time: 00:18:51
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Sorry about the tabbing as I did it in visual studio and it looked fine on my side.


Yeah I did some test on iOS side and it looks like we need to do SetLineDash(0, null, 0) to reset the style to not be dash otherwise any drawing after will be dash style. Wasn't sure where the best place to do the reset and thought it was best there.


@romw This seems like an odd addition, intended?


@atalax what was your reasoning behind this line of code?  Why not just use the following: `(system_get_rtc_time() * cal) >> 12`?


Will do


Will fix to just `<` in next commit, we need to jump anyway if next position is equal to current scene duration


Nice addition of the affix. Going for those stretch requirements


Cool, let us know which one you recommend. We can maybe automate it in the future.


Sorry I missed this when it first committed.

Neither S-II or S-IVB LOX had insulation other than the bulkhead separating the two tanks. (to prevent LOX heat leakage from boiling the LH2) Which isn't really a concern since we don't model shared bulkheads.

That aside, Some of those values are a bit odd, such as wall thicknesses. 16 would be something like Inconel, which I don't see happening on a main lifting tank. That's why most of the tanks had a value of 205 there, for aluminum. (some sources indicate 250 for aluminum; I erred for the lower value)

Some of those values were even lower, like 8. You want a value that matches the material that the tank would be made of.


There is a typo in your first line of the commit message: paltform v platform


We'll have a "false positive" here if there's 90% of the viewport filled with comments of previous entry, and in the very bottom another post ("first visible post") starting. We discussed this case on Slack.


Shouldn't we add whitespaces between descriptions and values?
E.g. `BaseMatcher`:
```
public void describeMismatch(Object item, Description description) {
       description.appendText("was ").appendValue(item);
}
```


You can send them to 'dev@clover.com' for now, or, better yet, post to https://devask.clover.com


and all the beauty was gone


Great add-on to wake map on hover. This is very nice. Great work


Maybe `HamcrestMatchers`?
Although I will be fine if you don't rename it.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.682.5eb8f4c6a37abc9278bc6430b9e11f0f3436479c](http://build.palaso.org/viewLog.html?buildId=68981&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:07:00


I know, but I did not find a way to reuse it, since SolrTestUtils is built around TestUtils and the wysiwyg enterprise tests use the old selenium framework.


Otherwise LGTM. And you feel free to polish and merge.
Thank you for care of my SCS PR :smile:


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


I left these two as separate things because they're provided by RootCamera. It might make more sense to implement it in *Camera:Update(), but I thought of it as the :Update() call as modifying the internal CFrame state and then the CameraScript applies the new state.


can you PR that up?


Going for stretch requirements. Awesome!


My experience as an Open Source project maintainer is that it's incredibly valuable to have a central place for a community to form, help each other and discuss ideas and issues. So in general, I think setting up a forum is a great idea.

As to whether setting up a new one is a great idea as well heavily depends on whether @thefiddler shows up soon and allows us to restart the official one. But my impression is that we shouldn't count on this right now.

So..

> I could set up whatever forum software we think is best, and keep it free of spam and do whatever other moderation is needed.

...this sounds good to me. Just throwing some more ideas into the mix, you could host this in a distinct part of your webspace with custom FTP users, so you could easily add other admins that you deem trustworthy enough at some later point in time. Given our current situation, I think it would be wise to set up any new efforts in a person-wise redundant way and have a group of people maintain everything, rather than just a single person.

Another important step in this would be to reach out to @Frassle once again in order to update the OpenTK repository notice, but since I don't want to bother him too much with this, I think it would be wise to wait with this until a new community site and maintained GitHub fork is determined.


Plots of the residuals file:
```python
# Load the data
res_file = np.genfromtxt("tussock_full.res", names=('parameterName', 'Group', 'measured', 'modelled', 'residual', 'weight', 'wXmeasured', 'wXmodeled', 'wXres', 'meas_sd', 'nat_weight'), skip_header=1,dtype=None)

# make an array for the X axis
x = np.arange(0,len(res_file['parameterName']))

plt.plot(x, res_file['measured'])
plt.plot(x, res_file['modelled'])

plt.xticks(x, res_file['parameterName'], rotation=40, ha='right', fontsize='x-small')
plt.yscale('log')
```


Style-wise, there's a little bit of DRY violation between the password_reset_recipient() and relocate_recipients() functions. However, it was absolutely necessitated by the fact that the data structures in the array that gets passed to them are totally different. I think having a separate function is the best way to do it.


Good work adding more markers to enhanced your markers.


Wonderful!


i made the host bold to distinguish from pages that have '|' in the title


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


added node_modules.gitignore file


Sorry misread your comment. I did originally test this else clause here but I decided to move it higher up but now noticed there could be an issue here.  Any ideas what we could do so we don't have it in two places.


Wondering if this is the best place for this, or if it should be in Pkg::Util?


sorry, this was supposed to be a comment in urlBar.js, not here


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


wouldn't the decal then clip with ground?


nitpick: can the type entry align with all the types below it?


I wasn't sure, it's easy to change though.

> On 8 Feb 2016, at 21:35, AliReza Tofighi <notifications@github.com> wrote:
>
> I think it should be tab... .
>
> —
> Reply to this email directly or view it on GitHub.
>


should probably check if host is null/empty


A `kill` on Windows corresponds to `TerminateProcess` and `SIGKILL` on Unix. A ctrl-c on Unix corresponds to `kill(-pid, SIGINT)` (or maybe sigterm, not sure), which isn't `child.kill()`'s behavior, hence the difference on Unix. A ctrl-c on Windows ends up doing something that I'm not sure how to send programatically, but I think it basically ends up having the same effect as `TerminateProcess`


package ejemplo1;

import java.util.concurrent.TimeUnit;

import org.openqa.selenium.By;
import org.openqa.selenium.JavascriptExecutor;
import org.openqa.selenium.WebDriver;
import org.openqa.selenium.WebElement;
import org.openqa.selenium.ie.InternetExplorerDriver;
import org.openqa.selenium.interactions.Actions;
import org.openqa.selenium.remote.DesiredCapabilities;
import org.openqa.selenium.support.ui.ExpectedConditions;
import org.openqa.selenium.support.ui.WebDriverWait;

public class DriverExplorer {

	public static void main(String[] args) {

		 DesiredCapabilities capabilities = DesiredCapabilities.internetExplorer();
		 capabilities.setCapability(InternetExplorerDriver.INTRODUCE_FLAKINESS_BY_IGNORING_SECURITY_DOMAINS, true);
		 System.setProperty("webdriver.ie.driver", "D:\\Programacion_libre\\Librerias\\IEDriverServer.exe");
		 WebDriver driver = new InternetExplorerDriver(capabilities);
		// driver.manage().window().maximize();

		 driver.get("https://scvluxn2.cavali.com.pe:4445/Cavalipre/menusess.do?op=showMenuSession");
		 driver.navigate().to("javascript:document.getElementById('overridelink').click()");

		 driver.navigate().to("javascript:document.getElementById('btnLogin').onclick()");

		 driver.navigate().to("javascript:document.getElementById('overridelink').click()");
		 driver.switchTo().alert().accept();

		 /// Prueba de login
	     // esperar el elemento cargarse por completo
		 WebElement myDynamicElement = (new WebDriverWait(driver, 10)).until(ExpectedConditions.presenceOfElementLocated(By.id("ssousername")));

		 // iniciar proceso de logueo
		 driver.findElement(By.id("ssousername")).sendKeys("ISIS$RQC");
		 driver.findElement(By.id("txtUs1erName")).sendKeys("WARI2015");
		 driver.findElement(By.id("btnAceptar")).click();

		 /// Fin de login

		// driver.findElement(By.xpath("//input[@id='Ingresar']")).click();// - See more at: http://software-testing-tutorials-automation.blogspot.pe/2015/02/how-to-wait-for-page-to-loadready-in.html#sthash.iUqqOwVO.dpuf

		 WebElement myDynamicElement2 = (new WebDriverWait(driver, 10)).until(ExpectedConditions.presenceOfElementLocated(By.linkText("Ingresar")));
		 driver.findElement(By.linkText("Ingresar")).click();

		 //ESPERAR ELEMENTO MENU
		 WebElement myDynamicElement3 = (new WebDriverWait(driver, 10)).until(ExpectedConditions.presenceOfElementLocated(By.id("__AtJsMenu75__")));

		// driver.findElement(By.id("__AtJsMenu75__")).click();

		// driver.navigate().to("javascript:document.getElementById('btnLogin').onclick()");

		 Actions builder = new Actions(driver);
		WebElement menuLink = driver.findElement(By.id("__AtJsMenu75__"));

		 //driver.findElement(By.id("__AtJsMenu75__")).click();
		 WebElement subLink = driver.findElement(By.id("__AtJsMenu78__"));
		// WebElement subLink2 = driver.findElement(By.id("__AtJsMenu84__"));
		 builder.moveToElement(menuLink).click().moveToElement(subLink).click().build().perform();
		// builder.moveToElement(menuLink).click().moveToElement(subLink).moveToElement(subLink2).click().build().perform();

		  // driver.findElement(By.id("__AtJsMenu84__")).click();
		 //driver.findElement(By.tagName("Registrar Tipos de Cambio")).click();

		// driver.findElement(By.id("__AtJsMenu75__")).click();
		// driver.findElement(By.xpath("//input[@id='__AtJsMenu84__']")).click();
		 //__SUBMENU__
		// driver.findElement(By.id("__AtJsMenu78__")).click();
		 //
		// driver.findElement(By.id("__AtJsMenu84__")).click();

		// driver.quit();
		 }

}


but why?


Sure, will do that.


BUT WHY?


@rallytime Looks Perfect now! Thanks once again!


We all make that same mistake


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (958 of 958 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.95%25-green.svg)


i made these fixes already in my version of this commit, so no need for you to fix. i'll push it now.


TODO:
- Cleanup
- Make it better
- Make transport as small as possible
- Make encoding/decoding part of utils to have easier maintance
- ???
- Profit! :family:


Zrobiłem nowego pull requesta który poprawia TCPDF. Autoloader z Composera ma też taką zaletę że obsługuje namespace, których LMS teraz nie ma ale może w przyszłości się dorobi. Rzeczywiście trzeba będzie pamiętać podczas kodowania nowych rzeczy o przeładowywaniu autolodera, ale większość użytkowników na co dzień nie dodaje nowych klas więc ten problem nie powinien ich dotykać.


wot self doc fl;a


No z mojego puntu widzenia mówią mi jaką mają kolejność w widgetcie, choć jasne mogą mieć jakąś nazwę i na końcu numerek, ale ich zadanie się zmienia w zależności od potrzeb, dlatego też mają uniwersalne nazwy. Raz na pozycji 1 musi być ikona, a raz tekst, więc wydało mi się to naturalne, jeżeli masz jakikolwiek pomysł i potrzebę to zmienić to sporo, zaproponuj cokolwiek.

A labele czyszczę, bo dla widoczności w projektancie ui mają treść "[]", ale spoko, doda się jedną funkcję do czyszczenia, bo chciałbym zachować dla widoczności tą treść.


@mauritsvanrees Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.contentlisting/6//violations
```
plone/app/contentlisting/catalog.py:42:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:44:14: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:84:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:185:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/contentlisting.py:130:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/realobject.py:36:12: P002 found "hasattr", consider replacing it
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Config changes for compilation


Please keep pull requests limited to a single change. Also, I expect this change would have caused the corresponding unit test to break?


@mauritsvanrees Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.discussion/28//violations
```
plone/app/discussion/contentrules.py:20:5: E731 do not assign a lambda expression, use a def
plone/app/discussion/browser/comments.py:132:1: C901 'CommentForm.handleComment' is too complex (17)
plone/app/discussion/browser/conversation.py:29:1: C901 'ConversationView._enabled_for_archetypes' is too complex (14)
plone/app/discussion/browser/conversation.py:135:12: P002 found "hasattr", consider replacing it
plone/app/discussion/browser/migration.py:34:1: C901 'View.__call__' is too complex (32)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Great work. Like the way you added circle markers to the map. Nice job.


I don't know that we're using any 2.0 only features. Do you?


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [Gruntfile.js](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/Gruntfile.js)
- [app/config/sculpin_kernel.yml](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/app/config/sculpin_kernel.yml)
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/app/config/sculpin_site.yml)
- [source/_partials/author_box.twig](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_partials/author_box.twig)
- [source/_partials/footer_content.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_partials/footer_content.html)
- [source/_partials/footer_scripts.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_partials/footer_scripts.html)
- [source/_partials/landing-index.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_partials/landing-index.html)
- [source/_partials/search_box.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_partials/search_box.html)
- [source/_views/article.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_views/article.html)
- [source/_views/changelog.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_views/changelog.html)
- [source/_views/default.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_views/default.html)
- [source/_views/guide.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_views/guide.html)
- [source/_views/landing.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/_views/landing.html)
- [docs/articles/drupal](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal)
- [docs/articles/drupal/8](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/8)
- [docs/articles/drupal/8/configuration-management](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/8/configuration-management)
- [docs/articles/drupal/8/create-services-yml-file](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/8/create-services-yml-file)
- [docs/articles/drupal/8/drupal-8-performance-and-caching-settings](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/8/drupal-8-performance-and-caching-settings)
- [docs/articles/drupal/8/moving-out-of-testing](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/8/moving-out-of-testing)
- [docs/articles/drupal/drupal8](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/drupal8)
- [docs/articles/drupal/drupal8/configuration-management](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/drupal8/configuration-management)
- [docs/articles/drupal/drupal8/create-services-yml-file](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/drupal8/create-services-yml-file)
- [docs/articles/drupal/drupal8/drupal-8-performance-and-caching-settings](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/drupal8/drupal-8-performance-and-caching-settings)
- [docs/articles/drupal/drupal8/moving-out-of-testing](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal/drupal8/moving-out-of-testing)
- [docs/articles/guide-example](http://1243a11y-static-docs.pantheon.io/docs/articles/guide-example)
- [docs/articles/guide-publishing-workflow](http://1243a11y-static-docs.pantheon.io/docs/articles/guide-publishing-workflow)
- [docs/articles/local/terminus-the-pantheon-command-line-interface](http://1243a11y-static-docs.pantheon.io/docs/articles/local/terminus-the-pantheon-command-line-interface)
- [docs/articles/organizations](http://1243a11y-static-docs.pantheon.io/docs/articles/organizations)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)
- [docs/articles/sites/code/email](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/code/email)
- [docs/articles/sites/code/using-the-pantheon-workflow](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/code/using-the-pantheon-workflow)
- [docs/articles/sites/database/myisam-to-innodb](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/database/myisam-to-innodb)
- [docs/articles/sites/errors-and-server-responses](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/errors-and-server-responses)
- [docs/articles/sites/external-libraries](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/external-libraries)
- [docs/articles/sites/migrate/wordpress-site-networks](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/migrate/wordpress-site-networks)
- [docs/articles/sites/newrelic/new-relic](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/newrelic/new-relic)
- [docs/articles/sites/quicksilver](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/quicksilver)
- [docs/articles/wordpress/configuring-wp-config-php](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/configuring-wp-config-php)
- [docs/articles/wordpress/hyperdb](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/hyperdb)
- [docs/articles/wordpress/site-networks/getting-started](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/site-networks/getting-started)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/docs/assets/css/main.css)
- [source/docs/assets/images/by-sa88x31.png](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/docs/assets/images/by-sa88x31.png)
- [docs/changelog/2015-01-01-January](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-01-01-January)
- [docs/changelog/2015-03-01-March](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-03-01-March)
- [docs/changelog/2015-04-01-April](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-04-01-April)
- [docs/changelog/2015-10-01-October](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-10-01-October)
- [docs/guides/using-sendgrid-to-deliver-email-with-wordpress-and-drupal](http://1243a11y-static-docs.pantheon.io/docs/guides/using-sendgrid-to-deliver-email-with-wordpress-and-drupal)
- [source/docs/objects/index.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/docs/objects/index.html)
- [source/docs/search.html](https://github.com/pantheon-systems/documentation/commit/28b2f7b2336aadc788765ad8e9079093db4fbf6e/source/docs/search.html)


mksocfoga hostmot2 hal  files


Returning one extra variable will not noticeably increase run time. What you have is a slightly different implementation of the same concept, and arguably reduces readability


@jvdhurk Bitte löschen lassen bevor du den Router Außerhalb des Rings an einen Uplink Leitung steckst.


Compare the following to get an idea of why this wasnt quite working:

```js
const someJSON = {
  "license": "MIT",
  "files": [
    "lib",
    "README.md"
  ],
  "repository": {
    "type": "git",
    "url": "git@github.com:chrisbateman/webpack-visualizer.git"
  },
  "escapedString": "string with some \e \\ \" escaped characters"
};

const three = `console.log( ${JSON.stringify(someJSON)} );`;
const two = `console.log( JSON.parse(${JSON.stringify(someJSON)}) );`;
const one = `console.log( JSON.parse('${JSON.stringify(someJSON)}') );`;

eval(three);
/*
{ license: 'MIT',
  files: [ 'lib', 'README.md' ],
  repository:
   { type: 'git',
     url: 'git@github.com:chrisbateman/webpack-visualizer.git' },
  escapedString: 'string with some e \\ " escaped characters' }

*/

eval(two);
/*
undefined:1
[object Object]
 ^

SyntaxError: Unexpected token o
    at Object.parse (native)
    at eval ...
    ...
*/

eval(one);
/*
undefined:1
{"license":"MIT","files":["lib","README.md"],"repository":{"type":"git","url":"git@github.com:chrisbateman/webpack-visualizer.git"},"escapedString":"string with some e \ " escaped characters"}
                                                                                ^

SyntaxError: Unexpected token
    at Object.parse (native)
    at eval ...
*/


We could. We could also set _lastError I think.


Not related, but just let you know there's a segfault when using schema sample (http://datastax.github.io/php-driver/features/#schema-metadata), PHP 5.6.17
```
Program received signal SIGSEGV, Segmentation fault.
0x00007fffecda30f2 in cass::DataType::value_type (this=0x0) at /home/rav/php-driver/lib/cpp-driver/src/data_type.hpp:78
78        CassValueType value_type() const { return value_type_; }
(gdb) bt
#0  0x00007fffecda30f2 in cass::DataType::value_type (this=0x0) at /home/rav/php-driver/lib/cpp-driver/src/data_type.hpp:78
#1  0x00007fffece0178a in cass_data_type_type (data_type=0x0) at /home/rav/php-driver/lib/cpp-driver/src/data_type.cpp:127
#2  0x00007fffecd9602f in php_cassandra_value (value=0x7fffe4017ec0, data_type=0x0, out=out@entry=0x7fffffffa818) at /home/rav/php-driver/ext/util/result.c:38
#3  0x00007fffecd96b12 in php_cassandra_get_table_field (metadata=<optimized out>, field_name=field_name@entry=0x7fffece69a5c "index_interval", out=out@entry=0x7fffffffa818) at /home/rav/php-driver/ext/util/result.c:343
#4  0x00007fffecd7ea1a in zim_DefaultTable_indexInterval (ht=<optimized out>, return_value=0x7ffff7a66ca8, return_value_ptr=<optimized out>, this_ptr=0x7ffff7a663c0, return_value_used=<optimized out>)
    at /home/rav/php-driver/ext/src/Cassandra/DefaultTable.c:158
#5  0x00007ffff7e8993a in dtrace_execute_internal ()
#6  0x00007ffff7f48fe7 in zend_do_fcall_common_helper_SPEC ()
#7  0x00007ffff7ed7fe0 in execute_ex ()
#8  0x00007ffff7e897e1 in dtrace_execute_ex ()
#9  0x00007ffff7e9c2d0 in zend_execute_scripts ()
#10 0x00007ffff7e37120 in php_execute_script ()
#11 0x00007ffff7f4af42 in do_cli ()
#12 0x00007ffff7d16813 in main ()

```


@efiring you need to catch WindowsError too.


Ok, I wasn't 100% sure I was reading the bash right, so that clears things up. I slightly prefer using the content because... ah I don't really know why but what's done is done :)


Please make sure you have the following commit to *devstack*

https://github.com/openstack-dev/devstack/commit/c7656a12ec8ca6a35d4ef40c0cd62d9a2ec68079

There is a secondary issue related to virtualenv's and I'm working that.  However, a straight devstack run works for me.


These are the tests that were done at the beginning of the existing `__add__` method. Since it made sense to have the same tests (same level, index as an array) for all operators, I factored them into a separate method.

Some operators must work with Components, yet it makes no sense for `Component.ix` to return an array, hence the check on index type I think.


Ah, dang. Not sure if I want to change it again. But thanks for catching this!


I should be able to


what I mean is that I got it to run on clear by changing the ports in Network.java.


Move what exactly, just the switch? I'm not sure I see the point given that we don't reuse this code anywhere else.


These sound like good questions. Thoughts?
 @karlitschek @bboule @markviens @MTRichards @cmonteroluque @msrex


@blink1073 - any way of getting ggplot available on conda's default channel? This one is requested commonly enough.


Good catch. I failed my commits :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14923](https://teamcity.mesosphere.io/viewLog.html?buildId=128639&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 225 Build time: 00:13:03
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@indutny Sure. I'll do


This must have happened when I was searching the heading in markdown.  I also need to update one of the unit tests.  So, will do it together. @mjwitte Thanks.


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


Ok, I change it in code.


I'll have to test on my emu and on malinka's android 6


It is not needed, but I like to have a proper method that corresponds to the operator. It is easier to manipulate with functions that takes a callable as argument (functions such as `map` for instance). Also, it was consistent with the other operators that have a dedicated function.


Unfortunately `isPreCalypso` is incorrect and `isOld` is too vague. I understand it's a UI concept, but it is something that is triggered by the "age" of the account so I think it is a good description for someone to use with no context.


hmm, so we have a customized yaml for pemeetan?


Did having `<input>` instead of `<input/>` stop the sample from working? (If so... that sounds like a gotcha I should call out.)


Was this commit intended to resolve CVE-2013-6430?
http://pivotal.io/security/cve-2013-6430


This will crash the debugger since buffer_add_objid doesn't support null objects.


Fixed in 1a1df82f246547dbcc61ebb103e6353d117031cd.


I tried but I keep getting some db errors. Could I be missing something?

```
Spell 158185 listed in `spell_linked_spell` does not exist
Spell 158188 listed in `spell_linked_spell` does not exist
Spell 115191 listed in `spell_linked_spell` does not exist
```


it helps, merging

for future multiple signals single config :)


It might, if the ground isn't flat and the shift happens to be in the wrong direction. I have a slight preference for clipped decals to misaligned ones though.


Typo fixed. This commit is kaput.


Why not use what is already available? https://doc.rust-lang.org/core/macro.assert!.html


Yeah, just saw this when updated devstack. Thanks for the response.


s/the topic/the topics/


Which tests are failing?


https://quantivity.wordpress.com/2011/02/21/why-log-returns/
and
http://papers.ssrn.com/sol3/papers.cfm?abstract_id=1549328


I don't have any 4.3 at my disposal. What errors do you get?

On Feb 8, 2016 10:59 PM, MS-Astra <notifications@github.com> wrote:

Hi Gustav,

I remind me that this is not enough to compile the current anyfx gl shaders with opengl 4.3 hardware. Did you try this?

-
Reply to this email directly or view it on GitHub<https://github.com/gscept/nebula-trifid/commit/465db20cb7e8467832572d56359245d53efdb100#commitcomment-15961874>.


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


s/held its XMPP/held the XMPP/


I did not found an equivalent to `issubset` and `issuperset` in numpy. There might be more clever way of doing it, but this seemed the more readable to me. I should have the index sorted, though.


Not necessarily -- in the case of null destination module, we don't have a deadline and in the case of an error, we want the transition to happen immediately rather than when the original destination module wanted to transition.  It seemed easiest to be explicit here that the fade time comes after the original module deadline.


I am examining (1) as well.


I see. Yep. Thanks, Brian.


The translate service has two main functions - an async one that should be used when possible, and an sync version used below (`$translate.instant`). If the translation configuration hasn't yet been loaded, the sync version just returns the key. We used to load Settings to delay initialising the tour until the settings are available but I found the [onReady function](https://angular-translate.github.io/docs/#/api/pascalprecht.translate.$translate#methods_onready) which does the same thing but more reliably.


Terima kasih Om! Sangad membantu.


s/Classic MUC has first been/Classic MUC was first/
s/usage scenarios/usage scenarios:/


@bobpuffer Why is there a hard-coded value here?


I'll work at a new set of tests as soon as I have some time.

For the note on duplicates, I do have one (two actually) in all the docstrings. But I agree that they are not very clear. I will see if I can document this better.


эта картинка (и вообще любой фон - т.е. картинка, на которой есть HTML-контент) декоративная, значит, загружается через CSS


I should note that it's on my internal todo-list to have the compiler check exactly which subversion of GL is used. It's also on the very same list to switch to the Khronos reference compiler which should manage this easier than the way I'm currently doing it with a hardware context.


I don't want it to be a general method, so this is a pretty good place to keep it contained.


This will use the installed fish instead of the built fish when running tests.


It can be but I kind of hate those alignments.


The reason I didn't do `set...` is because usually that involves a boolean like `setAdminVisibility(true)`. Would it be ok to leave it as-is?


how about this?


I thin this is the wrong behavior, we want to respect exc.


Same thing, we want to respect exc


Add the following comments to a wiki page


если что-то относится по смыслу к article - оно должно быть внутри article


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


Yeah that looks pretty good :+1: http://jigsaw.tighten.co obviously doesn't exist yet but otherwise looks fine.

I think the emphasis on it using Blade is the biggest reason people would use it, maybe description should just be:

> A static site generator that uses Laravel's Blade templating language.

or

> A static site generator that uses Laravel's Blade templating language and Markdown.

...?


While I agree the way WiX/MSI handles conditionals is nowhere near optimal when you consider the effort involved in figuring out how custom actions work, tracking down the code, and then attaching a debugger the advantage in debuggability becomes tenuous at best. On top of that if you are creating registry keys outside of the MSI engine then you are now introducing the possibility of orphaned registry keys unless you properly (manually) implement rollback and uninstallation.

I already provided examples of how to do it inside of wxs: https://github.com/Corillian/dokany/commit/40261e5cfa9df95cce725220dbe1980de561d66c


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


Yes, I'll hook up the next part as soon as I have confirmation that the rest of the code is working as designed.


@yhahn just noticing this now, but you nuked the old bitcache code, right? Looks like we're still running carmen tests twice, but I don't think that env variable does anything anymore...


No -- imported bindings are always crate local and so always have spans


(surprisingly simple)


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


Investigate changing map to show locations shown as pins on map


This will blow up really bad. We should set exc.


Inputs for files are often used for things such as print Help information as well as setting certain expected paramaters.

For example our device uses -d {protocol name} -p {pools} to set the protocol type and number of workers (simultanius processes) to pull data from.

Good job keeping up on this!


![brawlboxanimation0](https://cloud.githubusercontent.com/assets/16909509/12900913/4fc9bcfa-ce7e-11e5-9c53-3ae8d8c79800.gif)
![brawlboxanimation7](https://cloud.githubusercontent.com/assets/16909509/12900918/534b31ce-ce7e-11e5-856f-e56fd6621070.gif)
Here's the updated versions.


забыд дописать название класса?


Nice improvement. Thanks!


I think 'its' works fine, but whatever. However, *enthusiasts.


With this, you will - as far as I can see - break the loading of icons for all apps which ship their icons in hicolor without having them in the hicolor-icon-theme base iconset.


@davydovanton We're simulating a file with UNIX permissions `664`. That `100664` is the string representation of file stat.


k


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1276.2e06764002524857962572b2b1dfe72ac53450fb](http://build.palaso.org/viewLog.html?buildId=68998&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 947, ignored: 12 Build time: 00:08:11


OH NO
I completely forgot that the encoding may not end correctly when it's cut to pieces, causing errors in receiving!
Oh no


I'm using webpack as a module bundler and in my code I only `import redux from 'redux'`.

Since the redux's `package.json` is like: `"main": "lib/index.js"`, which contains the transpiled unconcatenated version of the library, this is the version I'm requiring (I assume that's what you mean by "non compiled" forms).

I just ran a couple of tests (by dropping the `node_modules` and reinstalling). This is an npm problem. This isn't the first time that npm3 is having a weird behaviour and is skipping deps at `npm install` - https://github.com/topheman/d3-react-experiments/commit/28fbbc0122d5809839a7071a834bc01394bbb258

Since I need lodash v3 at the root (for some reason), I found a workaround that seem to be working:

```
cd node_modules/redux
npm install lodash@^4.1.0
```

I could add that to the `postinstall` phase so that my unit-tests pass on the latest versions, it's a lot hackish though ... (and I should make it clean to be sure that it works for every users cloning the project)


Не тот адрес)


thanks for helping! It was a talentless copy-paste error from an unfinished file (got to keep those folders properly named).


The behavior we want for those factory functions is to turn errors into an ExecutionEngineException with the original one as part of the message.


The following doc(s) have been deployed to the [terminusde](http://terminusde-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/cli](http://terminusde-static-docs.pantheon.io/docs/articles/local/cli)


*extent


You're right. I'll fix it. Give me some minutes:)


This is redundant now (I made a similar change upstream)


either 'synchronization of contact lists' or 'synchronization of the contact list' :)


Should this mention which month (for historical reasons)


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2091 of 2093 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: Table big diffs.

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


I will investigate the issue tomorrow.Thanks for notifying me.


I'm not sure if this is useful. I would rather paste a placeholder text (something the user can see) like "No posts found."


Yeah I think the else belongs here and not at the `!= null` check. Would you please move it?

The `pen == null` case doesn't need to do anything since no line will be drawn and no state will be relevant.


I think that should be "the summit's sponsors", to not confuse with the XSF's sponsors.


Hey @pdmccormick , can the design document be in markdown format?


:shipit:


Should that be "3 generations of Instant Messaging, the trough of disillusionment, and consequently the necessary cleanup yet to be accomplished"?


google-homepage


Check these 3 lines, and fix them please!


Then instead of `sleep_for`, how about calling `wait_for` and `get` on `thread_future` and if the value is not `APS2_OK`, then transforming that status value into a nice message with `get_error_msg`?


well, generally it's not good to have long methods and you wouldn't have the result variable as a simple declaration, but whathever


I'd avoid making a claim like 'most popular XMPP server' in an XSF post.


Huh, I'm surprised there aren't any existing tests for this. Nice catch.


This is different from current cfssl/log behavior (which always logs to `os.Stderr`)


Can we hit the following codition?
```exc != NULL && !mono_error_ok (&error)```

If we can, the above will leak the error.


Sounds good to me.


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I think that should be 'with what they achieved'.


Fixed #7
Whenever you need to access messages the bot can't normally see (such as those in replies), you must disable privacy mode. In the future, I may add parsing to check whether the command is directed at the bot by storing the bot name around (i.e. `/decide@glgbot` would be valid but `/decide@ijifsdjifsdijfiadiwj` would not).


We don't have access to this.


missing paren at end


ditto almost everything in this list


:custard:


Sure, but currently the way it works is that FW ask at the very beginning and exit if the permissions are not granted, then if granted, why exit?


not realy yet, but maybe you open a topic in the forum for that, so it can be discussed.


I don't know, but if we're not testing for it, we might break it (and, I'm all for requiring 2.1+)


debugging statement


Yeah, I was thinking that. The problem is that we (you) don't check this function for failure in the caller


Now they are ok,
there were falling few hours ago.

Really good job with the cleaning and shortening of examples.

2016-02-08 23:17 GMT+01:00 Marcin Kosiński <notifications@github.com>:

> Which tests are failing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pbiecek/archivist/commit/cdd89423ce3b451c57d26a2161af9a9ebb9a71f6#commitcomment-15962861>
> .
>

--
pozdrawiam serdecznie,
Przemysław Biecek


Sample has amendment to an amendment scenario


I general I'm not too happy with checking in these generated files. We have a build system for the documentation, so generating things dynamically is possible. At the moment it is difficult to see through the names and dependencies if you wanted to include the _indices attributes for example.


And also add the french missing texts, not only the quotes ;-)


Done


Yes; I already did that and re-pushed without announcing :smile:


This buffer size might too small.


nice and readable solution for such checks is lodash's `isEmpty

`if (!_.isEmpty(devices) {`

(Just an idea)


failed to save my codes


This also just doesn't really belong..


> Since the redux's package.json is like: "main": "lib/index.js", which contains the transpiled unconcatenated version of the library, this is the version I'm requiring (I assume that's what you mean by "non compiled" forms).

Yikes. Heads up @gaearon the npm package is using the non-bundled forms.


fyi - x265/package.mk won't work at all (I had no time to fix that)
x264 is not perfectly optimized either (I had no hardware to test it properly - now I have but at the moment no time to do tests)
there are reports of 20-50% in/decrease of speeds ... something isn't that right at it

If you are "able" to do some optimizations they are genial welcome :) The git is more less a test for further MAYBE implementation at OE8.


I think that's a good idea. I like the second one.

And yah, we have to figure out the domain. I figured we may end up having to wait to do this PR until we have at least *some* sort of web property to link it to?


noticed this on `master` too, should this be `didReceiveAttrs`?


:tada:


But seems like same thing in PM

> Am 08.02.2016 um 14:57 schrieb Aodzip <notifications@github.com>:
>
> Anyone want help me do the right spell?
>
> —
> Reply to this email directly or view it on GitHub.
>


should have called this eveHelper and put it in it's own file helper.service.js


I do not expect this will work correctly on Ubuntu 15.10 because we don't have a compatible .NET CLI for that distro.


Missing 'then' after 1


PHPCS: Expected 1 space after TRY keyword; newline found


PHPCS: Expected 1 space after closing brace; newline found


PHPCS: Expected 1 space after closing parenthesis; found 9


hmmm. I think the weirdness here is `parse-fn`; we're trying to provide an interface where you pass in a function to parse whatever it is you have, but we've underspecified the error behavior. What if spec that it always returns `nil` on error, or always throws IllegalArgumentException? Then we can fix the various `parse-fn`s we pass through in other parts of the code base. It looks like `Integer/parseInt` is one of these, and it looks like that will 500 in the same way.


PHPCS: Multi-line function call not indented correctly; expected 17 spaces but found 16


private function getUnitPrice() // FIXME: Convert to float


If this will encompass more eve-specific code we should consider changing the name of this library to angular-eve-tools or angular-eve-helpers or angular-eve-connector etc.


Opened PR for that. I'm sorry about this. Trying to fix the API ended up screwing things up.


Hi Yang Luo,
thanks for reply and your great project.

`win_bpf_filter.c` *is* included in Dll\Packet project, see: https://github.com/nmap/npcap/blob/2fbf96c61bcacbde0b694fe74138a469550ee602/packetWin7/Dll/Project/Packet.vcxproj#L938

`_WINDLL` macro is indeed declared only for Packet project.

Without this fix, I get compilation error about trying to resolve driver (kernel) space function (KeQuerySystemTime) during DLL build.

best regards,
Przemek


The following doc(s) have been deployed to the [rachelwhitt](http://rachelwhitt-static-docs.pantheon.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/1fd05f241a52f7baa3e1950fb84c1aabf365bd2e/app/config/sculpin_site.yml)


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


wtf


Skoro w zależności od użycia labelki mają różną rolę, to nie nazwiesz ich "uniwersalnie" - nie ma co nawet próbować (bo nazwy i tak nic by nie mówiły). Taki problem można rozwiązać na dwa sposoby:

 * utworzyć dwa osobne widgety dla list itemów (wtedy będziesz mógł nazwać konkretnie, ale ich funkcjonalność będzie się w pewnym stopniu pokrywać)
 * ustalić kolejność labelek (co jest do zrobienia) i nazwać je konkretnie, ale w zależności od użycia po prostu niektóre ukrywać (lub czyścić)


hmm... these are all things @dougwilson said he was contributing to this, that's why they're included in the list. We may need to leave these for the new TC to decide.


'enable' is not needed. 'tone_mapping' is a more common usage. So:
tone_mapping (Filmic tone mapping) bool false


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1277.5f3c27acd267f2fdac43ab881c7d17c94a686206](http://build.palaso.org/viewLog.html?buildId=69001&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 947, ignored: 12 Build time: 00:07:24


So I think we're actually going to have to interleave these reads over stdout / stderr rather than perform them sequentially as you've done here.  The pipes both need to be drained each iteration (or we risk deadlocking if we're only draining `stdout` while `stderr` fills)


Space before second D.


Oblivion doesn't have formlists, those were introduced with FO3.

I am going to assume that FO4 also has scripts that rely on them remaining in the proper order. We won't know for certain until the CK for the game is released.


sure I will do this tomorrow as it is late over here and will give it a good test before I push it in.


Should be flagged in the code as a deliberate memory leak.


Yeah, I dunno.  `.toString` makes me nervous given what we're trying to do here.

I also dislike introducing references to JRuby-specific classes here, but that kind of seems like the lesser of evils.  Walking the whole data structure twice (once in Ruby, once in Clojure) seems like enough extra overhead that I think I'd probably suck it up and just reference RubySymbol here.  And then, yeah, throw an exception if we hit some other data type that we don't recognize.

You could talk me into the approach of doing the pre-munging phase in Ruby, though, if you feel strongly that that is better than introducing the reference to RubySymbol here.


Dorina Gjoni


Nice Promise combo :)


@markelog I think we should also add tests certifying that #2828 is, indeed, fixed; otherwise we'll regress one day again.


Yes


Sure, except that this won't be synchronized across clients anymore, because clients may not all execute 'now' at the same time.


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


I do not understand your comment


Token


Corrected the typo, the unit test, the documentation and pushed the branch to remote.


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


Just when running the tests. It s fixes for this case.

Le lun. 8 févr. 2016 21:37, Jeremy Grossmann <notifications@github.com> a
écrit :

> It shouldn't happen. Did we get a crash report on this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-gui/commit/cc7bddefa1d560d1d5039789513eadfb71913bb5#commitcomment-15958751>
> .
>


ES has a bug #1. See PerformanceAnalytics package


Sure, this is not a fix. But a way to prevent the code from failing. I need to do a pull from your code first.


@chriskuehl and I explicitly removed this because it's really frustrating when things are not-quite-working


Couuuuld just setup a redirect to GitHub for now?


Should `inlineSVG` now take advantage of `svgstring`?  I would think that `svgstring` would be faster than `svglite(tempfile)`.


That's not how it worked before, I think. but sounds good. :) Will fix.


Why is this necessary -- isn't draining eviction sufficient?  Is there some case where we do splits independent of eviction, or are you thinking about some future where that might become true?


What is the purpose of that loop?


Yep I saw it, did have the courage to do a pull yet.


What do you think may be is better to add comment or replace this string to constant?


Is this the right issue number?


isn't this going to leak into other tests potentially?


This file needs some graphs


I don't think so. Just outside of the snippet here is a call to `UserSettings` which is callback based and returning a promise in a callback method wouldn't work. When we get around to promisifying UserSettings we can make this much nicer.


necessary with the environment variable above?


Fair enough, make sense. Not sure we have to explicitly have the overrides though to funnel the consumer into a specific constructor, that loses the whole point of optional parameters - makes the file look messy and aren't required IMO. We also have to remember that the users of this library will still be developers so they will have some grasp of programming.


Ideally you'd bump the podspec version too :smile_cat:


@aarondcoleman @mxa0079 glad I could help and the PR was useful :-)


Ok, thanks, will fix it.


As discussed. We use Yamload for loading the tenants config in config_loader.rb


Thanks!


no, this path only activates if the proxy is off.


@kasperkopf Ich bin nochmal eine Erinnerung


`calculateAccumulatedFrame` in SKAction produces an offset. Hence using the size + origin of the underlying frame.


True. I've left most of the validation for Yamload, however, to use Pathname I have to add a bit of basic validation so we can split up the directory and filename to pass to Yamload. Gives a nicer error message as well.


No particular reason - I want not to block for a long time, but I also prefer to let some time for the kill to operate well if possible. I guess I just like 0.7. :)


this URL is broken please update with https://github.com/Microsoft/binskim/releases/download/v1.2.4-beta/binskim1.2.4-beta.x86.zip


I don't think we need to over complicated it with additional packages. I think we either need to:

a) Add in a `IFitbitInterceptor` implementation which handles scope errors into the library but let the consumer add it into the pipeline

or

b) show it as an example in the documentation as a "you can also do this".

Lets try and keep things clean and simple :-)


:clap: I am pausing here to acknowledge the ratio of effort expended to characters changed.


The setTimeout delays execution of the digest until the service execution is complete. If I don't have them the test times out because the digest happens then something gets stuck waiting for a digest.


I'd call out the specific exception here (`socket.error`) so it's more clear what you're testing for


This is an exact replica of the ones that they have in real life.  I
thought it would be a nice edition to the new aircraft

On Monday, February 8, 2016, Jonathan Redpath <notifications@github.com>
wrote:

> McDonalds. Really.
> But seriously, nice job. However, a photo of a Big Mac or something would
> make for a nicer effect on the fuselage :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FGMEMBERS-NONGPL/MD-88/commit/1b6e68770808b2c23c083cce8a1f1eb3203b813c#commitcomment-15953355>
> .
>


@cpennington @peter-fogg This is a nit, but any objections to going back to smaller headings, e.g. h2? I find it very jarring when the headings in the cover letter use a larger and bolder font than the PR heading itself.

![image](https://cloud.githubusercontent.com/assets/5985072/12901595/8a2fa5b0-ce8a-11e5-8a02-79f2a965d66a.png)


:+1: on UserSettings...


5382_PoolMakeUpWater (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2059 of 2061 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: Table big diffs.

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


Alright, I found a way to partially revert a commit.

It is described here: http://stackoverflow.com/a/26262021

Essentially, it reverts, and then selectively commits that revert (selecting in this case only the functions in my previous message).

Pfewww, that took some figuring out :).

I made a branch in my fork and will send a pull request


Ah, good catch, you're right


This is really ugly and annoying, but I can't think of a better way...


Let's leave the constructors at optional parameters and leave it at 3 (single interceptor, multiple, custom factory) as I think it makes it cleaner


nit : I'd use a more pep008ish variable name as I'm not sure what `sai` stands for.


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


looks great! :+1: Thanks a lot


Perfect.

On Mon, Feb 8, 2016 at 5:37 PM, Adam Wathan <notifications@github.com>
wrote:

> Couuuuld just setup a redirect to GitHub for now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tightenco/staticgen/commit/37d405057528331cbc6a122967a9e0387079f0af#commitcomment-15963651>
> .
>

--

*Matt Stauffer*
*Partner + Technical Director*
Tighten Co.
352-682-2765


## Header for this comment


Let's try to identify and open issues (or come up with a list of tasks in that existing issue) related to this. I fixed the crash issue you mentioned in the Butterfly issue (and another related crash), but that's all the information I have.

Better yet, if you have a repository with your source code (pages and layout), I can take a look and fix whatever issues may be relevant there (or you can open a PR for that set of issues first).


Just had a quick play with this but I couldn't actually get to the banner to click it.


Hey,

Please refrain from using sleeps.

Also, try not to depend on other views when you need to perform a scroll. Look for a parent and perform scroll on the entire viewgroup instead.


Hm, i thought i mentioned that on the meeting, but apparently not, yeah, i was meant to do that, thanks for keeping an eye on it though


Please use string resources instead (R.string.)


I am unsure about changing the name, because the main point of this library is to work with the Eve backend. The query condition helper directly affects how the resource is used. It is not separate functionality.


Please use string resources instead (R.string.)
 Add a line note


This should take care of it:
http://currents.soest.hawaii.edu/hgstage/pycurrents/rev/f7b7fd8d92a8
I expect that OSError will also catch WindowsError.


@jezdez ok, cool.


Don't add compile dependencies please.


Great work


No objections from me.


AV. DOM JOSÉ, 1654 - CENTRO SOBRAL - CE / (88)3111-0000


You don't need to add a jawawriter dependency in your tests if it's included in the production app.


Still not fixed.


Is there a specific reason the existing Ubuntu CLI will not work on Ubuntu 15.10?


hasToString is enough in this case. Don't need two matchers.


@adamwathan done. You wanna clean up the text to be whatever you like and then we can PR it?


Ok. I made change 674b57e to add this information to the known bugs list.


should this PR be sent again v0.12.x or master?


Please use a private method instead.


oh, I should've been clearer — I meant to point out the spelling of the String. unless I'm misunderstanding you, `didReveiveAttrs` should be spelled `didReceiveAttrs`. sorry for being a pest about this!


:clap:


So you can put .java and .html and .properties related files in the same folder, not scattered


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1509 of 1509 tests passed, 0 test warnings)

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


you can catch both exceptions in 1 go if you want to do the same anyway:

except (ValueError, KeyError) as e:
    ...


You could use `ExitStack`:
```python
with ExitStack() as stack:
  tempdir = stack.enter_context(TemporaryDirectory())
  orig_file = stack.enter_context(NamedTemporaryFile(...))
  # and so on
```


Got it, I will fix this tomorrow.

On Mon, Feb 8, 2016, 4:36 PM Matt Handley <notifications@github.com> wrote:

> Sure, except that this won't be synchronized across clients anymore,
> because clients may not all execute 'now' at the same time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/chicago-brick/commit/67867a303a415caf61706aaa9101a67f59336460#commitcomment-15963597>
> .
>


To me simpler revert to tested data, than change new.
Because in testing process this data overwrite. If new and old json file the same - to eventmonkey don't send anything and I dunno it's OK or something not work.

Of course, in future they will be ignored.


That's embarassing...

I merged an old branch and I must have overseen those diffs...

I'll fix it now.  Thanks


Heh, missed that one

fixed in https://github.com/sirjuddington/SLADE/commit/52d433a58f7e7791c94a35c885c4e4aa33e9c059


Not wicket, but they're typically used by most java apps. They're complementary to spring.


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (958 of 959 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.9%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-28.69%25-green.svg)


this is not a great approach. will this snippet continue in v7 and v8?


i would prefer this broken up by version headings


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


Added a comment to clarify


better grammar: ".... distinguish between 0/0.0/L and a non-existent value" (i.e. drop the 'the' and change 'non-exist' to 'a non-existent'


Ah ... wait. How about this. Is here anything to do or a false positive?


LordPsyan you missed the custom_reforging table from combinated characters


btw, this will require a change in Attachment.rb in the else branch of `signed_location`


oops, forgot to add one more file, one sec


@ATofighi sorted


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


>  This allow NServiceBus and RavenDB to be upgraded independently.


It's alive!

Thanks for taking care of this. For your information, I started another async binding in C last year, but I have yet do add the dbus methods necessary to replace  wirefu crashy code paths. I started working on upstreaming some of my config in upstream Awesome. At some point there will be a monster code dump an my config (like a 5k line diff). I am not really happy with that, as merging your work again will be hard. I moved a lot of things around and some API are in the process of getting into Awesome 3.6. There is 2 new *useless and ugly) themes
https://bbs.archlinux.org/viewtopic.php?id=88926&p=62

And the main new feature will be a rewritten client layout system (drop-in replacement of the current one) with tabs, dynamic layouts and a lot of cool features. I am also upstreaming that. I don't push anything because it doesn't work with any Awesome versions (including git-master) and require tons of Awesome patches to start.

By the way, you might want to work with @mindeunix on the drawer code

http://i.imgur.com/OZRuu0Y.png
http://i.imgur.com/mZkKVc7.png
http://i.imgur.com/YNKuEyH.png
http://i.imgur.com/uXOvc0r.png
http://i.imgur.com/TSXAe9h.png

His version is, let say, Awesome! It isn't a fork of my config, but rather a clean room re-implementation using most of my modules and tech, very good work.


Does not look like there was any other reason than me failing to simplify the expression. Should
be fine to change it.


make indentation consistent.


@briantrice If the user sets API key in env, and uses register to change server (but not re-specify API key), won't this wipe out the existing API key?


What on earth is this?


Ok, thanks for the clarification!  @pfalcon FYI.


this was pretty difficult right at first, even harder to explain


@sonoble did you want to clean this up per my suggestion or did you want to me to merge it as people are blocking on this?


I think without comments on the rules is fine.
People that can understand the rules sufficiently will not gain anything from a comment. And people that don't understand the rules sufficiently will also not gain anything from a comment. :-)


reasons


Yeah, I know; I'll fix on the next push.


If by "wipe out", you mean "allows them to override defaults in their session", yes. It's a cascade. Any change to the API key flushes existing authentication (_is_authenticated) if done within a session.


There's a comment here that we should drop. Making a note so I don't forget it as I rebase.


https://github.com/sonoble/OpenNetworkLinux/commit/a9371065859c1a44addfb1c8363940bba6d84f32

I believe it is fixed already?


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [Gemfile](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/Gemfile)
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/circle.yml)
- [configs/wraith.yaml](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/configs/wraith.yaml)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/scripts/deploy-multidev.sh)
- [scripts/deploy-wraith-diff.sh](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/scripts/deploy-wraith-diff.sh)
- [wraith.yaml](https://github.com/pantheon-systems/documentation/commit/d248c2128296adb82b413b985a5ebef7fd8b10fb/wraith.yaml)


Heh, was just debugging this issue locally. Thanks.


Nevermind, I misread the date, This predates later changes, which I thought it was reverting.


For instance, you forget to add this new state-driving machinery to the ApplyMutation methods.


Наверняка


Thanks, Catherine!


Umm, I looked at the app using that again and it's quite hacky after all, they create an event-filter based brush that call the brush paint method (based on a QImage QPaintDevice painter) so the brush is updated just before the widget painting happen. Good enough I guess.


Why not simply make it zero?


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


A


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


I'm not sure about this one, the alpha value doesn't tell us something about whether a color is light or dark. It always depends on the background whether a color with a low alpha value should be considered light or dark. This is not something this method is aware and/or responsible of. This could lead to unexpected behavior. There should be a separate (convenience) method, which gets a background color as second parameter and considers the alpha value.


Ah! yes, that would be a typo. This string is only used for debugging so it hasn't particularly been a priority for it to be correct.


I'm still seeing the failure on both trusty and wily at this commit.


What the heck


Some of the packaged we depend on are not binary compatible across versions, so we can not load some of the native modules because the linker can't resolve the external symbols.  For 15.10 specifically, the problem is with ICU (used by System.Globalization.Native.so), ICU bakes versions numbers in to their exposed symbols across each release and 14.04 and 15.10 have different versions in their package manager.


Perhaps `server` should be an attribute of the plotter. The plotter is purely functional.


Perhaps de-emphasise jQuery so readers don't assume jQuery is necessary?


I am not sure if this works as it should.


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


I will make the change and will also add a check in build.sh for the same.


It's the actual behaviour of autostarting common event, they loop and run multiple times in each frame while their conditions are met. In RPG_RT, the number of loops depends on how many event codes it has. The more event codes it has, the less it loops. I think 500 loops is good enough.


mas vypisat:

1 2 foo 4 bar foo 7 8 foo bar 11 foo 13 14 foobar 16

teraz vypisujes:
1
2
3 foo
4
5 bar
6 foo
7
8
9 foo
10 bar
11
12 foo
13
14
15 foobar
16

cize nema tam byt 3, 5, 6, 8 10, 12, 15 :)

a taktiez, skus to vypisat bez novych riadkov

On Tue, 9 Feb 2016 at 07:28 janomuro <notifications@github.com> wrote:

> nechapem, ved za 2 mas foo a to nie je delitelne 3 atd. nerozumiem co mam
> spravit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/janomuro/Github/commit/29b447facbe0f6352f0510317f90d2b16cb23ca6#commitcomment-15958509>
> .
>


Fixed and pushed the change. @ellismg PTAL.


Sure, that would be exceedingly sensible.


Only one change? What about the rest?


"Only assignable to 'itself'" needs a test so we know if that rule is broken in the future. Not sure if that is covered in the default set of rules - It may be since there was a comment about that it must be assignable to itself.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 144](https://teamcity.labkey.org/viewLog.html?buildId=301662&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (2 new), passed: 42 Build time: 01:35:05
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

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

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

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

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

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

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

```


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


The following doc(s) have been deployed to the [1290wraith](http://1290wraith-static-docs.pantheon.io/docs) Multidev environment:
- [Gemfile](https://github.com/pantheon-systems/documentation/commit/a2353778a26828c9b5de33b2e266462fb5add949/Gemfile)
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/a2353778a26828c9b5de33b2e266462fb5add949/circle.yml)
- [configs/wraith.yaml](https://github.com/pantheon-systems/documentation/commit/a2353778a26828c9b5de33b2e266462fb5add949/configs/wraith.yaml)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/a2353778a26828c9b5de33b2e266462fb5add949/scripts/deploy-multidev.sh)
- [scripts/deploy-wraith-diff.sh](https://github.com/pantheon-systems/documentation/commit/a2353778a26828c9b5de33b2e266462fb5add949/scripts/deploy-wraith-diff.sh)


group member:YING MA, WEIYI TUO, KAIYANG LYU