I have fixed by using make tools and copying haxelib manually. We need to fix the install script to it uses the correct path. Also, I'm not sure when we install we should reference the haxelib sources. Usually an install does not require the original sources to work.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


How does this checkbox look like? Could you provide a screenshot on the issue's page? Adding an extra checkbox next to any textarea might break the display/functionality of certain applications.

Remember that this applies to virtually any textarea that you attach the codemirror instance to... it can be the object editor, wiki editor, wiki macro in inline edit, etc.


This is a redundant assertion. It will always evaluate to true


The original text was very implicit about how this worked. My understanding was that only objects in the script settings stack counted as candidate entry settings objects. It seemed clearer to decouple being in the stack from being marked that way, by tracking marks explicitly.


NP, I forgot it was there as well :)


Seems to me that you got those two methods mixed up. Anyway, I don't think there should be two separate tests for this.


Yes might help
As as said, we might want to copy the haxelib client sources when doing make install, so the user can then safely remove its GIT checkout while keeping a working haxelib


tnx


I'm thinking of creating a sub directory in `repos` called `stripe


fast-forwarding end-to-end/timeseries to borshop-integration-255-bk-disable-ts-buildbot = 63decf5f


I'm a bit unclear on the design choices that got us here. It seems like `Constraint` is implementing `as_posyslt1` so that children can override it?

@bqpd can you please summarize the issue(s) at hand, and what choice(s) we need to make?


@aait If you had written `Fixes #3` in the commit message, issue #3 would resolved and closed automatically. :)


As I`ve commented on the issue:
"My suggestion is that this should be implemented as a user preference, stored in the user's profile. Makes the most sense."


Good job bro! Do you plan to add the ability to send commands to the game?
![good-job-quotes-1](https://cloud.githubusercontent.com/assets/1344659/11815400/ce05e426-a34b-11e5-89d0-fe7bb3b802ab.jpg)


Thanks and well done!


The code `JSON.parse(response.body)` was assigned to a variable twice. I wonder why that is so.


static потому, что вызывается в методе actionIndex() контроллера DefaultController


英文版是网友写的


Hi Tim, I think your idea is great, but could you please give a little detailed demo, how can user integrate docker with subuser. many thanks!


static потому, что вызывается в методе actionCheckout() контроллера SiteController


I think it's not good practice to explicitly build platform-specific paths when os.path can do that for you automatically.
I would suggest to entirely remove the win32 clause and use:
```python
default = os.path.join('$OUT', 'images', '{scene}_####.{file_type}')
```
Cheers


But it seems that it does just that: https://github.com/HaxeFoundation/haxe/blob/development/Makefile#L106


Ok, cool. Then what you need to do is to create an issue for the dependency update and make your issue about improving the keyboard shortcuts depend on this upgrade issue.


Maybe we should get it documented in Node.js docs first then.


Here is example of idiomatic list cleanup code you could find in CRIU:
while (!list_empty(&st->timers)) {
        struct proc_posix_timer *timer;
        timer = list_first_entry(&st->timers, struct proc_posix_timer, list);
        list_del(&timer->list);
        xfree(timer);
    }

Why it is not ok to use p != p->next and all the other list internal stuff? Incapsulation. You use list interface guaranteed to be stable.


ololo


No, this isn't the case. I remembered that for Z3 we do a ```strip``` on the shared-object. Doing this with the cvc4 binary it shrinks from >240MB to about 8MB.
Additionally I found some linking issues (which are only there on the computer in the university, at my laptop it works fine) where I have to look into.


I would add a reference to the commit here.


yay ABC


Yes, but the haxelib script in /usr/bin does not reference this copy it seems


Hum, this one is interesting.
It's currently unusable, because not exported.  This means that this is plain unusable with the new plugin API, because `filetypes` defined won't work there.

Also interestingly we document `filetypes_by_title()` but it's not usable either -- and even in `GEANY_PRIVATE`.


Oh sorry it does, my bad


Consider adding a flag which permits concurrent message handling.


哈哈....秋百万表示不服~


return 0;
is better because it explicitly states that everything is alright in this place (states for the reader of code)


please guard textual output with
#ifdef _SPIRV_SUPPORT_TEXT_FMT
since textual output support is only available when the macro is defined.


`can not` -> `cannot`


same as above


Hi Mark, that is a good improvement! I see a bug on page 13 of the PDF where the part paragraph:

"This anniversary was in one respect a
moment to celebrate – but also a moment
to note that torture remains rife around
the world, a reason why Amnesty
International launched its global Stop"

appears a page early, instead of at the top of page 14 where it belongs. This has pushed the paragraph which should be in this position on page 13 to the right column:

"crackdown, with use of the Mubarak-era
Law on Associations to send a strong
message that the government will not
tolerate any dissent. Leading human
rights organizations had to withdraw
from the UN Human Rights Council's
Universal Periodic Review of Egypt's
human rights record because of fears of
reprisals against them."

then the right column has whitespace.


Revert skipping R sink output (required with XCMS, CAMERA, ...).


So the gain is not only from the ifunc attribute usage but from the rework done on function pointers. Previously each of flush, pre drain fence, pcommit, fence was called by function pointer. Now there is separate function for each "HW configuration" like: clflush + sfence + pcommit + sfence, clflushopt + pcommit + sfence etc.


Хорошо, вечером займусь.


:+1:

Tu peux cherry-pick sur https://github.com/Wisembly/tapestry/pull/198 stp ?


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


Commit message and title. Again. What you do why you do it how you do it.

Example:
shmem: implement manual anon shared memory dedup
We use the same dumping and restoring mechanism for anon private and
anon shared memory. Because of this we can implement manual deduplication
of shared anon memory the same way we do it with private anonymous memory.
Also we need to rename pid parameter of cr_dedup_one_pagemap to id because
now we can pass either pid or shmid there and the actual meaning depends
on flags: PR_TASK is for pid, PR_SHMEM is for shmid.

Signed-off-by: Fyodor Bocharov <bocharovfedor@gmail.com>
Signed-off-by: Eugene Batalov <eabatalov89@gmail.com>

Read CRIU wiki 'submitting patches' for more details.


Chance to confuse was incorrectly displayed as 10% in the in-battle description.


OK, actually `filetypes_by_title` is accessible through `GeanyData`, and it's only odd because the doc is not in the right place.

Same for `filetypes_array` of course, but documentation doesn't lead here so it makes more sense.


We were supposed to inject the **Global** Thread Pool.  I don't see a point to inject the thread pool on a serverLocator as it would require a massive amount of work on the integration.

the idea is that a system could reuse the global thread pool with other pools it already has. So, this is not really the idea I think.

I think this needs a small refactoring


I forgot to add a comment for this diff, so here it goes:

I've extended the css-layout library that is used for computing layout by passing an information about height in the measure function, the same way width is being passed.
The change was done here: https://github.com/facebook/css-layout/pull/154

This diff pulls this change in RN Android-part. Since anyone can write a custom measure function for their shadow/css node this is a breaking change. The fix is really simple = adding a new (unused) parameter to all measure functions.

Similar change will be done on iOS side soon.

Motivation: Measure function was written for text nodes, which do their own layout and (usually) have constrain only on width. We want to embedd other types of nodes that can do their own layout, but they could be flexible in width and have contrain only on height. So this modification enables us to implement such nodes.


Hi,

In fact, the notice is in my editor, not in php. This is more a good practice than a notice. You can cherry-pick other ones.

Sincerely,

Daniel Berthereau
Infodoc & Knowledge management


Could this be configurable, so no warnings will be print when defaultMessage is used as the primary translation?


IMHO it is not clear what the chart presents :) I think a few words of explanation of the benchmark used here would be really helpful.


Forgot to save Protege before committing this to the repository, and so the changes to term editor did not go through. Will fix this on next commit.


FYI, on AWS forced checkpoint does not work to accomplish it.


Why not simply self.assertEqual(adapter_wwn, list_args[1][:])


this should read
```
<copyright file="UntrustedSpec.cs" company="Akka.NET Project">
```


I'm moving some of our email comments here to keep better track of the back and forth on this stuff.

---
Nov 14, 2015

It does indeed appear that the attenuation range is now properly inside the polygon, which is awesome, and it goes from minvolume at the edge to maxvolume when you get attenuation_distance away from the edge on the inside.

There is a problem when you go outside the polygon, though, which is that calculate_volume isn’t run at all so the volume stays at whatever it was previously. I realize that in most cases a user will gradually get closer to the edge of the polygon, thereby reducing volume and then when they get outside, the volume will already be at minvolume so it won’t need to recalculate, but gps jumpiness is possible/likely, so it would be best to programmatically set the speaker volume to minvolume whenever the listener is outside the polygon. Haven’t tried to fix this myself, but I imagine it’ll be pretty simple.

Another issue, which is probably easy to fix as well, is that the in_range stuff doesn’t seem to be working. I upped the distance to 100000 just to make it easier to test, and I’m still seeing that if request_stream is sent anywhere at all outside of a speaker polygon, it serves up the out of range stream rather than a real dynamic stream even if it’s within the specified range of 100000. I added some logging and my initial tests suggest that is_listener_in_range_of_stream isn’t even being called, so that function might work fine and just need to be called at the proper time. I need to try to debug more myself, but I’m guessing this is pretty easy to fix.


eta-eliminate `y`


Why do we have this: `if(percentage <100)` and the one below: `if(percentage < 10)` with the same code inside? Or I just miss something?


I suggest we test against latest patches, I think you can specify only minor like this :

```yml
rvm:
  - 2.0
  - 2.1
  - 2.2
```

But I haven't seen a lot of projet with this maybe we have to specify patch level. If it doesn't work we can specify latest patches ;

```yml
rvm:
  - 2.0.0-p647
  - 2.1.7
  - 2.2.3
```


@asbjornu I will improve, thanks.


This seems fine.  We should probably set _isReplaced : false on the prototype to be consistent with the `_isRendered` `_isDestroyed`.. also maybe this could go up near the restore/replace functions?


Didn't get anything while looking at Postman.

It's hard, because I'm not always sure whether something is either not fully specified in the CLI, not available as an easy curl request and I can't make it work, not available at all in go-ipfs (but it should be), or simply something that shouldn't be in the API (why would you want a daemon to run, for instance?). I'm still not sure how to adequately answer this question for these cases.


Yup, you can sequence with `flatMap`.
Or `map` or, most generically, with `transform`.


TeamCity pull requests :: netty [Build 878](https://garage.netty.io/teamcity/viewLog.html?buildId=4103&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 00:42:42


Fix this?


The docstring for this one isn't updated.


because of missing destroy notify?


This one too.


Also use TWTLocalizedString.


You might want to add a `suggest` so that people are aware it might be used by some of the code.


organization?


places


Thanks!


Same as above, you do not need two test cases for this.


The problem is that we don’t go back into stable area currently. But let’s wait for the two main problem to be fixed before.


from @probabble Nov 16, 2015

---
It makes sense to me that the speakers aren't turning off. Great catch. I think with a simple query on `move_listener`, we can just turn all the appropriate speakers off.

Just to make sure that I understand the expected behavior for the listener-outside-the-polygon cases:
  - if a listener is just outside any of the project speakers, rw should play (??)
  - if a listener is very far away from any speakers in the project, rw should return a default out-of-range stream

---
response from @hburgund

Just to be clear, though, with this new simple query in `move_listener`, when the listener is outside of the polygon, the speaker should not be “turned off” but rather have the volume set to the minvolume. Usually, of course, minvolume is 0, so effectively that’s the same thing, but just wanted to be sure we’re on the same page.

Regarding the “out of range” scenarios, you got it right. There is a distance threshold just outside the active speakers (collectively for a project) within which a custom dynamic stream is still generated (despite having no volume to begin with most likely) because someone might be about to walk into range. If they are more than said buffer distance from any speaker, then RW serves up the status out of range stream from the db instead of generating a dynamic one.

This is important for resource management and generally letting people know when they’re not in the proper spot for the experience. If you could set this distance to not be an arbitrary fixed number (1000 currently) but rather a multiple of the `attenuation_distance`, that would be great. Let’s put it at 2x `attenuation_distance`. If this is a pain, leave as is, but I’m guessing it’ll be about 30s of work for you when you’re already in there.


I would add a note that we are considering some mechanism which will eliminate this type-scaling phenomenon.


why does this not get passed to format_number?


Not needed.


Nope : C'est bien après la boucle


Please call this 'expected_func'


:pill:


커밋코멘트#3


I think doing

--storage-opt size:2g --storage-opt fstype:xfs

Would be better.  Then other backends could add additional storage-opts in the future.  Docker does not like adding lots of random options.

This matches up with --security-opt


Shouldn't this be `ConditionOutcome.noMatch()` ?


It does not conflict. The first one removed the spawns, the second one prevent any spawn by script to start roaming.


Why not simply import fc_struct and ctypes and use those modules directly?


@Sodman @Noah-T My conception is only the email address (and possibly logged-on state) gets saved to LocalStorage.


OK, will do that. Can you answer on the three unanswered post above? Thanks!


#4994-ControllerWaterCoil-TemperatureAndHumidityRatio-control-active-when-it-shouldn't-be (rraustad) - x86_64-MacOS-10.9-clang: OK (1959 of 1979 tests passed)

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


For bower packages you shouldn't be bundling all your projects external dependencies into a concatenated file, they should be specified in dependencies: {}

ionic.bundle.js now contains angular.js and angular is a dependency meaning that two instances of angular are brought in.

This also breaks specifying dependency versions (through overrides or otherwise), so for example I might want to use angular 1.4.8 for a new feature and know that there are no breaking changes but I can't as ionic is now hard coded to 1.4.3


add path like /capacity


whoops, forgot to account for %s -> %0, It removes the "32 -" on the result. so... let me rewrite that:
32 - __builtin_ffs(x) = (32 - __builtin_clz(x) - 1) =  __builtin_ctz(x), for 32 bit integers. Basically, we count from the lsb instead of msb.


Stools are not weapons! Last time I checked with mine it couldn't even break my balcony window.


fixed here
https://github.com/dchbx/enroll/commit/5ef52b32ae49b367a8fd6782e7eedd595b3c5733
discussed here
https://devops.dchbx.org/redmine/issues/4329


What is this line outputting?


Most likely need a newer version?


how is that needed?  Either we don't need all the `@transfer{none}` above, or this one should be `full` by default, doesn't it?


This is a bad name. We should change it to 'Converged directly' or anything that means that it converged, but not its mean.


That's it.


Cool, thx! This is the first akka-internal usage of `flatMap`. :)


It's not, you can remove it.


'kay


Why? This is the same as writting assertEqual(a + 10 - 10 + 20 - 20, b).
Just say self.assertEqual(fake_dapter_index, list_args[1].value).


Eh, english is weird. How about increasing the minimum length of generated names?


And this line?


What about “totally”?


Maybe you should also add a test for the wiki2vec_ids.txt in the if statement above


Sorry Michael, I'm really really not used to Git and I don't know if I made something wrong or not.
I made two bugfixes (merged from my own implementation of getByteBuffer(...) on the HTML platform) there : https://github.com/mbarbeaux/playn/commit/3f672d62cd213de53782aac2009255fea94f9437

I tried to make a pull request of that change but I cannot see it on Github, so I don't know if it worked or not...

On my local repo I just did :

git checkout -b upstream
git cherry-pick 3f672d62cd213de53782aac2009255fea94f9437
git push origin upstream

Is it sufficient for making a pull request from mt fork to the main PlayN repo ?
I'm very new to Git, here we're using SVN on all of our projects.


Why was this test removed?


Let's change this array so all of the elements are integers instead of strings.

That way it makes more sense to sum it, since summing just means adding together.


Yep, it's allowed, but this one was accidental and doesn't really make sense - the one at the end of the next line was intentional


@cbeust Could you make a new testng release too?


I think ideally this should do a system exit. Does returning null achieve this effect ?


:+1: about testng-team/testng-remote#2


```
100% <- 0 spaces
 10% <- 1 space
  1% <- 2 spaces
```
Code itself could be shortened.


Forced checkpoint won't help, I don't think -- I think the problem is the existing, open connection references some checkpoint at the end of the file, and it's not being discarded. If there's no open handle or the checkpoint isn't at the end of the file, (either of which may be true for MongoDB), then the file is appropriately compacted. See WT-2290 for my speculations on this -- I don't have any evidence there's anything wrong at this point, but I don't understand what is happening with these failures.


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


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1985 of 1985 tests passed)

![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)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.616.dfa1e253c526101d3095f9f095e4ec3bee95f9e0](http://build.palaso.org/viewLog.html?buildId=67294&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3323, ignored: 125 Build time: 00:07:22
### Failed tests
```
SIL.Lift.Tests.dll: SIL.Lift.Tests.Merging.SynchronicMergerTests.WorksWithTempDirectoryOnADifferentVolumne: <no details avaliable>

```


Yeah, apparently we've been using `map` and `mapWithInput` only so far...


THOSE the ones =D


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


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.617.5b493412c5237b448efaeb794997c412f37ca7c2](http://build.palaso.org/viewLog.html?buildId=67295&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3323, ignored: 125 Build time: 00:07:33
### Failed tests
```
SIL.Lift.Tests.dll: SIL.Lift.Tests.Merging.SynchronicMergerTests.WorksWithTempDirectoryOnADifferentVolumne: <no details avaliable>

```


Also, make sure to create a function instead of just writing your code inline with no function.


This should be 'predecessors'.


> should we assert that the size is still greater than fullsize here, after the reopen?

We could. In the current system, I believe that will always be the case, although it might change in the future, of course.


Given your avatar, will take this for an agreement. :p


Parameter names should definitely not consist of single character only!


Out of interest, do you still need this? I'd have hoped some work @chrisroos & I did a while ago meant you shouldn't need it.


This does not make much sense. You could check what kind of object is passed here, for example:
self.assertTrue(isinstance(list_args[2], ctypes.c_char * 256))
self.assertEqual(0, len(list_args[2].value))


I apologize for using GitHub's editor to make the change. It looks like it adds a new line character to the last line.


test test test test
#73977003816047


Great!


> object pointers default to transfer none, except if the name looks like a constructor; that is the struct name in lower case and a _new suffix. Unless the type is known to be a GtkWidget subclass, those are always transfer none (because GtkWidget is derived from GInitiallyUnowned).

so a lot of `@transfer{none}` above aren't needed


Hi @wmarquesr,

Turns out there is an extra ")" in this line as well.


Uploaded!


Which the version of Solr requires Java 8?
Whether we run it on Solr 4?


Do we need it? Description is bigger and bigger.

I don't want to add new description when new tool comes.


Perhaps we should just state that there is no processing model for other languages? Not declare it out of scope?


Doskonale, bede to sprawdzac. Dzieki.


WOOOW!! Can't wait for testing it!


@wilensky  2505a435f012be1505e05fd966adbf8072ec78f1


I thought you should not set the variables LETSENCRYPT_HOST, LETSENCRYPT_EMAIL and VIRTUAL_HOST on the docker container nginx-Proxy, but on the other docker containers. Because you want to forward the connections from the proxy to other containers and not from proxy to proxy?


これ、セキュリティチェックの絡みでどうしても欲しくなったのでつけました。
折り返しサーバで待ち受ける方式はクライアント側のFWに穴空くわトンネリング設定めんど臭いわで良いとこ無いので…
迷った挙句初期設定は機能オフにしましたが、デメリットな部分が特に浮かばないので必要ならデフォルトオンにしちゃってください


I'm not sure! I lost interest in FutureLearn: I'd rather read a book than watch a lecture.


Muy bien Sergio
El dic. 14, 2015 11:39 PM, "sergio" <notifications@github.com> escribió:

> @daton Proyecto
>
> @daton Proyecto
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sergiojoelgs9/ProyectoFinalSergio_Java/commit/d27408ba2b7c7bdefaa3f5d38ac98a22b58a2b37
>


Why do we need to re-define the DevBoxUser model? We already have it in the shared models class. One thing I also noticed, we also have it under DevBox/DevBoxUser.py - I think we can remove it from there too.


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


I thought the heart of the WIKIEDITOR-27 issue was that we can only search for the first occurence of a word before the dialog disappears, as explained in the description of the issue. This fix is based on the last comment (Make CTRL+G working and make ALT+F working).


合并分支


ack


this is because in yml format you **can't** do this:
```
key1: value1
  subkey2: value2
  subkey3: value3
```
try this:
```
key1:
  mainkey: value1
  subkey2: value2
  subkey3: value3
```


You should make shure that the target_mapping struct is updated properly using the entry_count property that is set by the called function. For example, if we pass a structure that can contain 5 elements but the caller function wants to pass us back 10 elements, it will specify this using this attribute, so we give it a larger structure.


Testing commit commenting...


are you sure the redirects are `.nq` files ?


Why you rename it to 'she'?


Cool stuff! It's very interesting how the Django DB layer will generate tables for the classes that extend from DevBoxUser (like Developer / Employer) but not for the DevBoxUser class itself.

I didn't know about this before reading https://docs.djangoproject.com/en/1.9/topics/db/models/#abstract-base-classes


TeamCity TripleA :: UnitTest [Build 541](http://home.k-moeller.dk/viewLog.html?buildId=1410&buildTypeId=TripleA_UnitTest&guest=1) is now running


ありがとうございます。

頂いたコメントを上から一つ一つ実行していきながら確認しました。

ところどころ用語の理解が曖昧だなと感じました。
fetch,fetchAllなど曖昧に使っていたのでこれからは気をつけていこうと思いました。


#65 hoàn tất


TeamCity TripleA :: UnitTest [Build 541](http://home.k-moeller.dk/viewLog.html?buildId=1410&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:47


Scratch that last comment. Prototype methods are not included in babel.


argh


you need to add snapshot cleanup on destroy goal too, current patch will break destroy.


First off, thank you for taking on the responsibility of this repo. Having this available was helpful.

This particular fix is good, but bothers me somewhat.  With this case of the FILTER_VALIDATE_IP option, this comparing the filter_var result with false is really all you need.  I don't believe there is a way to craft even an ipv6 address in a way that validates true and yet carries a payload that would be filtered out.  However I'd have felt much better if the filter were also applied on the assign:
$this->set( 'session.client.forwarded',  filter_var( $_SERVER[ 'HTTP_X_FORWARDED_FOR' ],  FILTER_VALIDATE_IP ) );

If not for clarity than just for the sake that someone could later copy this code and change it for something other than an IP filter.


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


[nit-pick] To be consistent with the below coding style e.g.

"user_name = models.CharField(...)"

it's probably better to leave spaces here as well:

abstract = True

It's OK to not leave spaces surrounding "=" if it's a method argument e.g.

CharField(max_length=50) is correct, but CharField(max_length = 50) is not.


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


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


I agree that the current solution based on mutt_append_message() seems like overkill in this case, but I have doubts this simple rename is a good idea. It would be nice to verify that all works as expected.

The mutt_append_message() does many another things, the most important seems that it to informs notmuch DB about the filaname change. If I read the code correctly than  mx_close_mailbox() is not enough for maildirs opened in M_APPEND mode (it callse calls mx_fastclose_mailbox()).


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


@makrai the diff says "207 additions and 207 deletions not shown" - did you really mean to change so many lines?


ack


It currently is between 1 and 2 syllables long, so increasing the minimum means there's no variation in length. A better solution would be to remove the single-letter syllables I think.


This should be relabelled.


@SamyPesse what's the rationale behind this change? from what I can see, it breaks Squirrel.Windows incremental updates big time


If you are making changes here... capitalize check?


it is still working, and I also took the serialgetty removal code, thanks a lot!


Wouldn't this reset the level set a line above? No log.info() for __main__?


@brianlayman This fix is back ported from Joomla 3.5.6 - please make your voice known in the main Joomla CMS Repo (https://github.com/joomla/joomla-cms) - and if/when they change it in Joomla 3.5.x then we will backport the changes.


What is the purpose of this? Shouldn’t `load` be changed to `start` in stop conditions? Currently it will never stop when “started”, so you won’t be able to restart a stp_value.

Or is that you want to run the code from n_iterations to n_iterations?


`str()` is redundant here


Why is this deprecated? Is it dangerous to wrap a context with a dict-constructor, and passing that to render(...) ?


Yes, yes, totally. Multiple remotes for the win, but you know that's what ourselves who are above average `git` users do. I agree with you, but still tonnes of people don't do that. Let's keep it like this and change it if needed in the future?


well then f*** strdup! I'll see about removing that byatch.


As a metacomment... did you run all of the tests? This commit contains spelling changes that could modify test FileCheck output.


I think I'm repeating what @m-kuhn described, but just to reiterate maybe:

`param.setValue(arg)` should do one of two things -

1.) When the arg is valid*, set the param.value, return true (this may entail mutating the arg, e.g. [extent parameter](https://github.com/qgis/QGIS/blob/master/python/plugins/processing/core/parameters.py#L191)
2.) When the arg is invalid, don't touch param.value, return false

*Validity is described by the ParameterType (e.g. a NumberParamter should get a number value) and optionality (setting None to a required type is invalid).

Equating `param.setValue(None)` to `param.resetDefaultValue` is not intuitive. The way to reset the default should be param.setValue(param.default) or we could add a param.setDefaultValue if we want to get fancy.

Thanks for putting this extensive table together @rldhont. I think executable test cases are a good form of documentation for something like this. I've spelled out some of the test cases in #2541, consider using it as part of your changes @volaya.


@rafis I'm not sure anymore. It's been a while since I worked on the project which was using this but I believe it had something to do with the keylen being doubled for some reason.


I don't think you need to use `hasattr()` here.


Once you have run tests and answered my meta comments, this commit LGTM. I may/may not look into the next one (I may leave it for someone else).


add description


Brilliant. Take care.


still continues roles ton


I know that, I wrote it :)

What I am saying is that, since you are refactoring/moving/touching it *and* since you are in a require module, there is no point in dragging along the PrototypeJS dependeny (at least in this file) just for that `document.observe` call.

It's 2 lines of code that you need to change, the require on `jquery` and `xwiki-events-bridge` plus the `$(document).on(...)` method call.


What about static imports :
import static java.lang.Boolean.TRUE;
boolean toto = TRUE;


F.ex. `em>.info


It's common to add a new-line at the end of the file (that's why the GitHub interface is complaining).


same same


why is this needed? Nothing in `bin.src` is guaranteed to be runnable so should never be added to the path. `bin` was already correct.


Just a nit-pick but in general modules are recommended to be named with underscores and lower-case letters (https://www.python.org/dev/peps/pep-0008/#package-and-module-names).

In this case, we could have a shared/models.py module which hosts both the DevBoxUser class, the DevBoxCreatedAt class, etc.


ありがとうございます。どんどんコード書いていきます!


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (910 of 910 tests passed)

![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-25.81%25-green.svg)


done in this commit 9e8e868a0fc3ecef5392d212209bf7c35a7524e4


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


Yes, i can do that with a factory or something like that.. but.. its more easy clear the conditions...

But, is just a suggestion... i'll try another approach.


I tend to think that this should be handled through semantic rather than just relying on syntax.


msgs is not the argument of this method ... it is the msgs module that contains the constants file!


TeamCity TripleA :: UnitTest [Build 542](http://home.k-moeller.dk/viewLog.html?buildId=1414&buildTypeId=TripleA_UnitTest&guest=1) is now running


Since you are changing this, could you put the session in a try  block?
A session not close add additional stack trace on the exception when a test fails


@dustinspecker Turned out it was my fault... Sorry about the noise. https://github.com/sindresorhus/xo/pull/59


I think the fallback should be to download the whole runtime from cdn.mendix.net instead of doing git fetch. Maybe that should be a second try/except.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.618.97070fbe7af68406ab00c9445e8500f094f07497](http://build.palaso.org/viewLog.html?buildId=67297&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 3323, ignored: 125 Build time: 00:04:50
### Failed tests
```
SIL.Lift.Tests.dll: SIL.Lift.Tests.Merging.SynchronicMergerTests.WorksWithTempDirectoryOnADifferentVolumne: <no details avaliable>

```


TeamCity TripleA :: UnitTest [Build 542](http://home.k-moeller.dk/viewLog.html?buildId=1414&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:43


fwiw, "killchild" name was inspired by scp code


I'm afraid I don't remember. Certainly one of the reasons was that `socket="$(mktemp -d)/weave.sock"` is slightly wrong, as it is creating the tempdir on the client (not on `$HOST1`).
Additionally, I believe I may have thought the case was covered in `test/650_proxy_env_test.sh`, however on inspection that doesn't seem true.


An excellent strategy to follow. Thank you for explaining further.


ack


beds and chairs definitely are not stools however :>


This dependency on the ptero_lsf celery_app can't be here.


You can keep it if you want, but you will be able to run the tests only from one place.

My point was that if they want to develop something they will need to change all the imports if they don't use several remotes. Example: https://github.com/milosgajdos83/servpeek/blob/master/resource/svc/svc.go#L8-L10


Not a biggy, there is another way to set these without if/else, see https://github.com/mozilla/build-cloud-tools/blob/master/cloudtools/scripts/aws_watch_pending.py#L526-L528


I hate not having core data.


Mmmh, I didn't think about it. We should add 'start' in the stop criterion as you suggetst.


Do you have a single language app, or simply not passing in `messages` to the `<IntlProvider>` for the default language?


This has been fixed.


I assumed "implented" was a technical term to describe what you and Philipp do all day ;)


```
	for objPart := range a.listObjectPartsRecursive(bucketName, objectName, uploadID) {
		if objPart.Err != nil {
			return 0, objPart.Err
		}
		// Verify if there is a hole i.e one of the parts is missing
		// Break and start uploading that part.
		if partNumber != objPart.PartNumber {
			break
		}
```

we can use a.listObjectParts() in a for loop directly here as there is no need for go routine.
if we use go routine and break here, that go routine will be left hanging in there forever as we will not be reading from the channel anymore.


To explain further - the 999 repos are NOT under development at all. They will only ever contain absolutely critical fixes to things I consider absolutely critical - like this weeks issue.

Joomla 2.5.28 is still end of life, insecure, unsupported and should never be used.  Joomla 2.5.999 gives a tiny false hope and breathing space for crazy people who could not care less about real security.


OK. So we don’t depend in arg finally?
Just run from stp_value to stp_value until n_iterations/stp_value restart?


No problem.

The themes I have installed simply ignore that setting, so some display alternating row colors with or without hints set for any treeview, and most of them don't display alternating row colors at all. Because of that and the gtk documentation I was under the assumption this setting has no effect on newer GTK versions. But if this breaks alternating row colors on some setups I will revert that of course.


interesting, exactly what I tried, among many variations that would have also matched, and it kept recursively watching that directory


Well at least it adds a focus for where to drag the window when using a Mac :-)


We need something like:

```
// UDP Port used for two way communication
unsigned int localPort = 8888;

void setup() {
  // start the UDP
  Udp.begin(localPort);
}
```

I guess i will fix the docs manually and make a new PR.


Thought: maybe we should have a method on `MWKImage` that returns the Image file name? Are we prepending "File:" anywhere else?


I'll open a pull request for reverting that later that day.


how is this different from the current solution which uses Popen with shell=True, ie. as long as the binary is somewhere in the PATH, it should work, shouldn't it?


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


This doesn't actually test the restriction works as there is only one comment created.


Would be great if this PR was accepted.


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


And probably should we allow to save multiple output file when doing so?

E.g. output_"start_iteration".dat (instead of .prev)?


what is it for?


Fixed a bug where cropping filled the background of the areas copied from ROIs.

I fumbled with code formatting which made a mess out of this commit, sorry.


Why a delete 'private'?
https://docs.oracle.com/javase/tutorial/java/javaOO/accesscontrol.html


+1 for bundling this with #2541


Hey @romulogcerqueira, why do you keep the old code commented? If this code is not necessary anymore,
why do not you remove?


Note: its still working with lava or cannonsmod.


same comment here.


Is this really a hack? Calling "did" before we actually do the download seems dubious. Actually shouldn't we wait for the images to finish downloading before reporting didFinish?


And/Or foolishly don't want to pay developers for the time it would take to upgrade their site & its custom code while it yet appears to be uncompromised.


Why do you want to do:

    render(request, dict(RequestContext(context_dict, request)))

when you could simply use:

    render(request, context_dict)

?


Ok, the problem seems to be that `Eq` can stand for "equality" or for "equation". We've previously used it to mean "equality", making it confusing here in the context of inequality constraints.


Good question, I will check that.


"simplified a bit too far, as Nearby and My could only then
in a particular cobrand potentially restrict to a particular body,
not any other criteria" in the commit message reads a little oddly to me.


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


Fair enough!


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


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1449 of 1449 tests passed)

![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.53%25-green.svg)


wasn't done that way originally, but i can bring it up w/ josh. I agree that error & progress reporting for saved pages is nowhere near comprehensive


Hi all, looking good but I'm a bit wary of this:

"This conversion process will output information indicating records that might need manual review and cleanup. Keep the output from the log to inform actions that might be required to fix problematic container data".

Log parsing such key information across many instances (in our case) sounds difficult and error prone, and may not be easy or reliable even for smaller, single instance sites as the log file is not necessarily persistent (things happen =) or for admins who are unfamiliar with the dense amount of information typically seen in the log file.

Can I suggest logging to a database table? Perhaps to the newly created system events table? I'm happy to add to the docs on how to get that information out of MySQL if you go with this (and perhaps it can even be made viewable in the frontend via a plugin -- altho' I realize that may be too much work for a one off effort).

Also could an example be added to the doc for what fixing "problematic container data" looks like?


Just a note, I don't think Myles is actually able to sign-off on this?


Im sorry I dont buy the argument - if one has a website then he has a responsibility to maintain and keep it up to date. We all service our cars and feed our kids - a website is a commitment not a one shot deal.

If "you" dont want that commitment then "one" should not commission a website :-)


t


What a nice security hole! :-D

You actually have public method that is accessible without authentication where I only need a budget's GUID to became its owner – check this link out: https://pv247expensemanager.azurewebsites.net/Account/Register?budgetId=3eabbd13-11ef-4923-b94d-d6de8e8f9772&permission=owner (after registration, guess how many owners will Spain Holiday budget have?)

Remember, when sending an email to a customer, always consider it a postcard equivalent (or at least an open envelope). Only thing that I needed to do was to change "read" at the very end of the link to "write" or "owner" which allows me to manipulate with the budget I do not have access to.

This is usually solved by sending a hash where there is everything encoded in along with the data. Also, this method should not contain any parameters. There should only be returnUrl parameter with address to method that requires authentication and that will be able to care about properties correctness. The link would look something like this: https://pv247expensemanager.azurewebsites.net/Account/Register?returnUrl=Budget/Share?budgetId=<guid>&permission=<permission>&hash=<hash-of-email-guid-permission>. Post Register method should then follow the returnUrl if one is given to it (provided it is local URL, …)

This also discovered 2 bugs you have in the project:

First is that you allow multiple owners per budget that does not seem to cope with your initial design.

Second is that even registered user can register a new one – once a user is authenticated, they should not be allowed to register again (this opens a possible solution to send a link to every user registered would be forced to log-in, new would be forced to register and logged-in would simply proceed – i.e. if you send https://pv247expensemanager.azurewebsites.net/Budget/Share?budgetId=<guid>&permission=<permission>&hash=<hash-of-email-guid-permission> link, MVC and Owin will do just that for you (including the returnUrl parameter that you only have to handle in your post actions).

Please do solve both the bugs. Do not hesitate to create separate issues for them though, so others can help you. And do not forget to write tests for at least first case.


Move this to function to a separate file, like `css.js` or something.


Qué?


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


As this site is defunct, I couldn’t be bothered to check it worked!


Lines 45-49 will be packaged into a function for use when `includeExterior` is `false`.


Well, this has the side effect of a violent boost of the simulation start…


You can remove the note on line 84/85, too.


I'm not sure this is the right exception :)


it would be nice to keep this function in code for non-linux systems


@SamyPesse it looks like Squirrel.Windows expects a full RELEASES file: https://github.com/Squirrel/Squirrel.Windows/issues/517


This is to remove duplicate y values.  The assumption here is that there is enough precision so that y-values are the same if and only if the scanline passes through a corner or intersection.  Later on in the code for `includeExterior=true`, I will need to assume that the polygon does not self-cross.


What's wrong?


What do you mean?


TeamCity TripleA :: UnitTest [Build 543](http://home.k-moeller.dk/viewLog.html?buildId=1418&buildTypeId=TripleA_UnitTest&guest=1) is now running


```
 dt_T_init, dt_nu_init:   1.7481022948067006E-002   1291.7974030753817
 W: T is taking time to converge, already          20 iterations
 W: T is taking time to converge, already          40 iterations
 W: T is taking time to converge, already          60 iterations
 W: T is taking time to converge, already          80 iterations
 W: T is taking time to converge, already         100 iterations
 W: T is taking time to converge, already         120 iterations
 W: T is taking time to converge, already         140 iterations
 W: T is taking time to converge, already          20 iterations
 Converged in mean!   9.9619647902481730E-009   9.9619647902481735E-007
 Mdot kick! YOLOOOOO   1.9999999999999980
 W: T is taking time to converge, already          20 iterations
 W: T is taking time to converge, already          20 iterations
 Converged in mean!   9.9846816650166104E-009   9.9846816650166096E-007
 Mdot kick! YOLOOOOO   3.9999999999999938
 W: T is taking time to converge, already          20 iterations
 W: T is taking time to converge, already          20 iterations
 Converged in mean!   9.9929096456710717E-009   9.9929096456710714E-007
 Mdot kick! YOLOOOOO   7.9999999999999982
 Converged in mean!   9.9929524835385966E-009   9.9929524835385969E-007
 Mdot kick! YOLOOOOO   15.999999999999988
 snapshot        8000   2310230.8663645373      S   1296.6227098121324        3.6951774801822830        1.0000000000000000
 Converged in mean!   9.9990183642992255E-009   9.9990183642992263E-007
 Mdot kick! YOLOOOOO   31.999999999999936
 Converged in mean!   9.9870998533382461E-009   9.9870998533382455E-007
 Mdot kick! YOLOOOOO   63.999999999999901
 snapshot       16000   5592862.2262420626      S   806.88841805874017        11.368013232584413        1.0000000000000000
 Converged in mean!   9.9970964807805721E-009   9.9970964807805725E-007
 Mdot kick! YOLOOOOO   127.99999999999960
 Converged in mean!   9.9898041547722432E-009   9.9898041547722438E-007
 Mdot kick! YOLOOOOO   255.99999999999943
 snapshot       24000   6636911.2178468993      S   215.15738463172551        9.1536754309192947        1.0000000000000000
 Converged in mean!   9.9995351924766692E-009   9.9995351924766694E-007
 Mdot kick! YOLOOOOO   511.99999999999937
 snapshot       32000   7392339.1472657584      S   131.50736143714033        11.343740772588310        1.0000000000000000
 Switched to explicit mode!
```


TeamCity TripleA :: UnitTest [Build 543](http://home.k-moeller.dk/viewLog.html?buildId=1418&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


Of course...  ;)   Good catch on spotting that!  :)


I'm not sure this is the right place to do this redirect. I would've thought that the router/panopticon might be more appropriate. If you haven't already, it's definitely worth checking with someone who knows how that stuff is supposed to work.


Oops: I intended to do that but must've forgotten. I even [wrote a category](https://github.com/wikimedia/wikipedia-ios/pull/325/files#diff-73801ce15533dfd176188cc4958337d4R13) and imported it, but must've gotten sidetracked by something


修正ErrorPage標題


await inside a string? OMG :-D
You can leave it as it is, but creating a local variable would be much more readable in this case ;-)


@Sodman @Noah-T ... email address ... which is also basic user data. Got it. In both cases the question is what we are using as the user identity token, and we may consolidate.


Please, add also fallback for systems where is no /dev/urandom, just use random or jrand48 etc.


Right, but this could not happen because if someone is logged in they are automatically redirected to the app. The button does not exist.


Yep, that's why I waited for a TSC member to sign off also. Wanted to at least note it tho.


I guess it checks it does return a result ;) Ta, will add another comment.


Didn't clue in to the fact this was in the TSC repository. Should I refrain from reviewing TSC related PR's in the future?


TeamCity pull requests :: netty [Build 879](https://garage.netty.io/teamcity/viewLog.html?buildId=4104&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6230, ignored: 21 Build time: 01:21:43


This is in grid coordinates, so I am assuming that this enough of a buffer on the left and right.  Unfortunately, using infinity does not seem to work.


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


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


Is this error condition supposed to not return FALSE?


Do we really need to remove BOUNDARYLEN and introduce MUTT_RANDTAG_LEN on another place if all is necessary only in mutt_generate_boundary()

. I think char rs[BOUNDARYLEN + 1] is just fine.


It is ok so do push request if you want to be contributor :-)


This is adapted from TokuTek's work.   You should give credit.


So if the currency was switched to usd, this would pass, right?

I'm going to give it a spin in a sec, and I'll try that.


Per the 2.7 changelog:
"added support for larger DH keys + compile-time parameter for DH bits (workaround for OpenSSL limitation)"

You would enable this with --with-dh on configure.

This means [upstream] (http://www.apsis.ch/pound/) chose another route to do this - and made it a compile time option, not a config time option.

You have the option to include this patch yourself, or to use the stage_for_upstream/v2.8a branch, where I put all my patches that upstream hasn't accepted.  Unfortunately, I have no control over what upstream accepts - I announce the patches I write on the pound list, and others provide patches there as well... I merely host them here on github.


@mhoemmen
This is the test that's failing on the dashboard, and it's failing here, when I initialize this matrix.  It only does so when OpenMP is enabled and displays the message

 p=0: *** Caught standard std::exception of type 'std::runtime_error' :

  Kokkos::OpenMP parallel_for ERROR: not initialized
  Traceback functionality not available


Agreed.


Note that I like your mutt_rand32() and mutt_rand64(), it's elegant, but it would be nice to keep the code more portable.


ok- I have to ask; is this for a College class?

That's awesome if it is.


`$('input').attr('name', 'authenticity_token').val()` が `✓` 帰ってきてます。

`$('input[name=authenticity_token]').val()` としましょう


Configurations are ordered alphabetically.  So this config should be put above


というかこれ、元のコードもバグってるけどhtmlで最初のinput要素が 'authenticity_token' だったので動いてたのでは


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


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


@kalashnikovisme  просто если вызвать эту функцию с другим массивом users на той же странице, то покажутся пользователи, которые закешились при первом вызове


@TaiseAssis precisamos, por enquanto, deixar o nome desse campo como 'support_line' ainda. Outros lugares o utilizam. ;)


https://github.com/cheeriojs/cheerio#attr-name-value-

`attr(name, value)` はsetterですね


Referencing websharks/zencache#624


@JonathonReinhart sorry for being late. But this commit is not included in version 2.0.0. This is because it adds new gem dependencies, and at this point there was a dependency freeze. Sorry for the inconvenience.


You shouldn't need to add the syntax plugin if you are already using the equivalent transform since it automatically gets added - [example](https://github.com/babel/babel/blob/59f5e7f218ffa788e6e3f320a3e7e0cbe6d50894/packages/babel-plugin-transform-class-properties/package.json#L12) for class-properties


I don't get that error. Yes, you need the 23 version. Try updating first.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.619.f0c9169e1717ea9ae45c8ba7d894b3fa73895663](http://build.palaso.org/viewLog.html?buildId=67301&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 3323, ignored: 125 Build time: 00:04:39
### Failed tests
```
SIL.Lift.Tests.dll: SIL.Lift.Tests.Merging.SynchronicMergerTests.WorksWithTempDirectoryOnADifferentVolumne: <no details avaliable>

```


New Map File.  Wider Streets, Smaller rooms


4959-plant-HX-CoolDifferential-stuck-Off (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1968 of 1978 tests passed)

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


herooo :runner: :muscle: :smile:  I would prefer to use the name 'category-header' instead of 'category-hero'


Yep that looks good to me :)


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


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1985 of 1985 tests passed)

![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)


TeamCity TripleA :: UnitTest [Build 544](http://home.k-moeller.dk/viewLog.html?buildId=1422&buildTypeId=TripleA_UnitTest&guest=1) is now running


I think this needs a bit more research.  If themes are able to do this, in 3.14 onwards, without that particular statement then there is no need to revert.

My own testing showed that without rules hints, no theme would or could display alternating colours.  The GTK devs, Emmanuele Bassi to be specific, apparently agreed with this, although looking closely he still never quite came out and said it.  However, without the hints there is no odd/even information for a theme to base its decisions on so its hard to see how it could happen.

Do you have an example of a theme that shows alternating colours even without the rules hint?  Is Ubuntu doing something funky to make this happen?


109806134-AbsChlrBlankWaterFlow (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (0 of 0 tests passed)

![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)


TeamCity TripleA :: UnitTest [Build 544](http://home.k-moeller.dk/viewLog.html?buildId=1422&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


nit: small typo in pytest.skip() statement


More testing notes on speaker volume attenuation behavior, specifically when listener is outside of a speaker. This is not investigating the out-of-range situation, but rather when the listener is in-range, but not inside any or all active speakers.

* speaker volume is currently only calculated for speakers within which the listener is currently; all other speakers are simply ignored, it seems.
* this means that unless a speaker *fully* fades out before a listener leaves its range - ie there is a `move_listener` call made exactly at the boundary of the speaker shape - there will be "residual" volume from the speaker that will persist ad infinitum until the listener re-enters said speaker and the volume is adjusted accordingly.
* I think the solution, or at least the first thing to try, would be to have the volume for all active speakers calculated no matter where the listener is located in relation to the speaker and if it is outside of the speaker range, set the volume to minvolume (as previously discussed). Obviously, this is only in situations where the listener is in-range when the session begins.
* If there is a significant cpu hit for doing this in cases where there are hundreds of speaker, for example, then we can consider doing some intelligent filtering so that only the speakers that are remotely close to the listener are even considered/calculated and all of the others are turned off i.e. volume=0 and no calculations done. We would need to figure out how/when to recalculate which speakers are close enough intelligently; maybe it would be easier and a reasonable approximation to use "recent" speakers rather than "close" speakers but then I suppose we'd have an issue when new speakers are introduced/entered into range. This is probably more complicated than we need to get at this point, but something to keep in mind.

@probabble I think this is stuff we've previously discussed and are on the same page about, but I'm outlining it here again in tandem with my additional testing just to document and be as clear as possible.


Cool, just wanted to add this :)


also email, please.


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


Well, the problem is that it iterates over an array in each step; it’s not optimized by JS interpreter or Ramda… FFFFuuuu! This problem can be solved by using transducers (see [this article](http://simplectic.com/blog/2015/ramda-transducers-logs/) and [#865](https://github.com/ramda/ramda/pull/865) and [#878](https://github.com/ramda/ramda/pull/878)), but it seems that the current Ramda’s implementation is inefficient, so it didn’t help in our case.

So I ended up with Marian’s hodgepodge of functional and imperative approach and just improved it. It’s more readable and even slightly efficient now.

For education purposes, there are some variants I’ve been playing with this night:

```js
export function mapPropertiesToClass (properties, elementClass) {
  return R.pipe(
    R.keys,
    R.filter(key => properties[key]),
    R.filter(startsWith('is')),
    R.map(key => ` ${elementClass}--${key.substr(2).toLowerCase()}`),
    R.reduce(R.concat, elementClass)
  )(properties)
}

export function mapPropertiesToClass (properties, elementClass) {
  const transducer = R.compose(
    R.filter(key => !!properties[key]),
    R.filter(startsWith('is')),
    R.map(key => ` ${elementClass}--${key.substr(2).toLowerCase()}`)
  )
  return R.into(elementClass, transducer, R.keys(properties))
}

export function mapPropertiesToClass (properties, elementClass) {
  const transducer = R.compose(
    R.filter(R.allPass([key => !!properties[key], startsWith('is')])),
    R.map(key => ` ${elementClass}--${key.substr(2).toLowerCase()}`)
  )
  return R.into(elementClass, transducer, R.keys(properties))
}

export function mapPropertiesToClass (properties, elementClass) {
  const isFlag = R.allPass([
    key => !!properties[key],
    startsWith('is'),
  ])
  const className = (key) => `${elementClass}--${key.substr(2).toLowerCase()}`

  return R.reduce((acc, key) => {
    return isFlag(key) ? `${acc} ${className(key)}` : acc
  }, elementClass, R.keys(properties))
}

export function mapPropertiesToClass (properties, elementClass) {
  const className = (key) => `${elementClass}--${key.substr(2).toLowerCase()}`

  return R.reduce((acc, key) => {
    if (key.startsWith('is') && !!properties[key]) {
      return `${acc} ${className(key)}`
    }
    return acc
  }, elementClass, R.keys(properties))
}


If you are going to fix grammar here, how about this:

"Sometimes one may want to know if an optimization inserts, removes, or moves a specific instruction"?


@figment  Mentioning so you can see.  The Unk2, the 18 bytes is at least partially float/hfloat.  Treating as all float didn't give me any NaN but who knows.  I haven't bothered trying to decode it.


class instead of classs ;-)


@nicolas-grekas this is a BC break on `2.8` too and if I'm right should also be set to `false` over there.

The custom choices on my `TimezoneType` were flipped around because it's default value is `true`.


(ah, wait.  let me keep reading.  perhaps you just get it from somewhere else?)


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


There's a better fix here: https://github.com/Moini/NiceCharts/tree/patch-1
(will work in Inkscape 0.48 and 0.91 this way. There are quite a few people out there using 0.48 still....)


That doesn't really matter actually cause the last entry will be taken into consideration. But sure - it will look nicer :)


StartTCPServer の修正について全体を見てないので的外れかもしれませんが、ポートを変更して再接続したときにポートが変わらない気がするのですが、勘違いでしょうか？
穴が開くから意図的に変えなくしてるのかなとも思えますが、ユーザーの意図と反する気も？
Listenモードいらない子… ではあるのですが。


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


@githubteacher please review and approve for merge to master


You feel strongly about it? Tbh. I made it as simple as possible on purpose for the sake of making it simple to grasp. It's not the style of coding I'd do in a production environment, but for the tests I am not overly concerned about stale data in case of test failures and the like.


Looking at the code prior my change, i can see that `specified_type` [has a default value of `java`](https://github.com/killbill/killbill-cloud/blob/v0.1.0/kpm/lib/kpm/base_installer.rb#L62), then we lookup the entry but the type is computed [by first looking at  `specified_type`](https://github.com/killbill/killbill-cloud/blob/v0.1.0/kpm/lib/kpm/base_installer.rb#L75) (which is always set because of default value). So as far as i can see previous version did not use the returned type from the lookup.

So am i missing something? Or was there already an issue before (in which case i am happy to change the code) to match your observation.


This line needs to be fixed. It should read: `%{!?_licensedir:%global license %doc}`

See [this from Fedora as reference](https://fedoraproject.org/wiki/EPEL:Packaging#The_.25license_tag).


`after` –> `for`?


TeamCity TripleA :: UnitTest [Build 545](http://home.k-moeller.dk/viewLog.html?buildId=1426&buildTypeId=TripleA_UnitTest&guest=1) is now running


커밋코멘트#11: 기본 브랜치가 아닌 경우는 어떨까요?


TeamCity TripleA :: UnitTest [Build 545](http://home.k-moeller.dk/viewLog.html?buildId=1426&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:44


It seems better to forbid them than to say "you can support them, but they must not have a processing model."


Yep, I can add a "success" response here if you'd like


@rvanlaak I don't understand what you mean. In 3.0, choices are always set as value, and the ChoiceType already defaults it to ``true``. So overwriting the default in TimezoneType is now useless


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


Github:
![GitHub tinting](https://cloud.githubusercontent.com/assets/6959245/11817604/ef588368-a355-11e5-848c-c274faaf8879.png)

Reddit:
![Reddit tinting](https://cloud.githubusercontent.com/assets/6959245/11817603/ef568cca-a355-11e5-8b21-33b44dd5ccdc.png)

You see, the overflow is tinted.


changed my mind: i think this table should contain all the data from the form except for the password.  even if it's stored elsewhere.


I think it is a good idea to remove the inline style if value is zero


This final implementation is 3afbec274bbfa808e62b616e1c142055cb105f97.


The code does not correspond to the error message in multiple places.


Seems like you deleted a comment from another patch but shouldn't be an issue.


Looking upstream into rack-livereload, it looks like they ignore by URL, not on-disk path. Would that effect your ignore, @ELouisYoung ?


Confused about the comment. `specified_packaging`, `specified_classifier ` are explicitly [initialized with `nil`](https://github.com/killbill/killbill-cloud/blob/v0.1.0/kpm/lib/kpm/base_installer.rb#L62), so why can't they never be null?


why do you use the transaction here instead of the action?


I used "After" because the command is gone... but I think for is better?


ignore.  sorry.


Any suggestions for name? `OneLineConstraint`?


`we can only search for the first occurence of a word before the dialog disappears`
You are half right. See more details in the comments section, specifically http://jira.xwiki.org/browse/WIKIEDITOR-27?focusedCommentId=88354&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-88354
The issue's description talks about the browser's default search function, not about the codemirror search function.

What you have tackled is basically point 2).

Now that I am looking again into the real problem, I notice the `disable_in_input` that you have used and I believe that the real fix of the `CTRL+G` binding is to fix the `jumpToPage.js` file in XWiki to pass the `disable_in_input=true` option when registering its keyboard shortcut. That should be fixed in platform.

The solution you have applied for the `CTRL+G` issue could be kept in the SyntaxHighlighting application for now, but as soon as it`s fixed in platform, it should be either removed or guarded by a check that makes sure it applies only on the older XWiki versions (the ones that do not contain the platform fix).

Regarding point 1) is not the default behavior of CodeMirror for scalability reasons and now that the find-next functionality should be improved by your changes, it should be enough for WIKIEDITOR-27. Additionally, a new issue needs to be created to handle the remaining UI improvements that make the search function hard to use in the SyntaxHighlighting app, like Vincent mentioned in his original issue.


I know that the gtk docs says not to use this function, since they (gtk people) are of the opinion that every instance of an application should have the same class. But the case of setting the window class covers exactly my use case where I use separate terminal windows for separate "applications" (e.g. one for email, one for shells, etc.). And follows the behavior of every other X terminal emulator I've ever used. So please don't change the behavior of this feature - consider adding a separate option for it instead.


1. static
2. capital letters for constants
3. private


added region column to schema*


Ok, the simpler the better. Are we actually returning blank strings from the lookup?


Why read in the data a second time?? In particular the "sfpd_districts.geojson" file, which you do not use at all?
Why not put ```$("#killBrowser").click(function(){L.geoJson(sf_c).addTo(map);});``` right after ```startVisualization(sfpd_d, sf_c);``` in line 52 (along with ```drawMap()```)??


@itsjeyd Because the existing code has a bad habit of representing integer IDs as strings, and `1` and `"1"` will only compare as true with `===` if I coerce the string value to an integer. The `+` prefix here is the unary plus operator, which coerces to a number.


Closes #993 .


@itsjeyd Yep, on purpose :)


Think so, there are not that much occurrences of "exit status after command" in search results. :bowtie:


It's pretty simple: we want `posys_lt1` to be able to return some kind of "Nope" object, which signifies to the solver that  something went wrong and an either an approximation should be used (SP) or an error should be thrown.

Because `posys_lt1` returns a list, and python is good at distinguishing falsy/truthy things, I propose using falsiness as the sign of a "Nope". NotImplemented is another alternative: it would be easier to read in the constraint's code, but require in the solver code several additional list comprehensions replacing `all`s (and would in general make it more difficult to tell if all your constraints were good from inside a Constraint).


Sorry to bother you, @mariofusco : just a quick double check:

Are you absolutely sure this should be added to `KieContainer` (and not to `org.drools.compiler.kie.builder.impl.InternalKieContainer`)?

Thanks.


That's fine, but you need to add it to your gemspec, otherwise people get missing dependency errors.


I changed this test result because there are multiple signatures for [strtr](http://php.net/manual/en/function.strtr.php) and its random which one we consider to be the canonical signature, but it seems reasonable to treat the first one as canonical.


Hi, after playing with 1.3.0.RELEASE version, my custom modules are not loaded by the auto-configured Jackson2ObjectMapperBuilder.
My modules beans are created after the auto-configured Jackson2ObjectMapperBuilder thus are not available when Jackson2ObjectMapperBuilder is created.
Before the registration happened in a @PostConstruct method so they were already created.


oops?


Nope because the default value we provide are `nil` (see `KILLBILL_JAVA_PLUGIN_CLASSIFIER` and `KILLBILL_RUBY_PLUGIN_CLASSIFIER`) so that seems like an acceptable value.


Yes that makes total sense. Will do.


`\` should be replaced with `/`.


Does it make most sense to move towards using "latest" as the dependency version instead of the specific version we are on? With this set up we would be having to do pull requests / updates every time a new version of electron is done but if we use latest like on PR #23 it will be a permanent fix with the latest stable version of electron always being used.


Will this cause issues if the sort order of the file changes? Refs https://github.com/JabRef/jabref/issues/520


That's okay. I've simply hacked the CSS file to exclude those resource URLs. It looks like hell (the switches are invisible), but it's good enough to get by until this commit is included :-)


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


@wachterjohannes @danrot ???


Then why not make, as described in this document?


That was copied from scp as well


`specified_type` has a default value only if nothing is passed (short form of the constructor used). In kpm-plugin, we can pass `nil` [though](https://github.com/killbill/killbill-kpm-plugin/blob/9ddf384cd2849fc8c4426b0ce95675883c2b8310/lib/kpm/plugins_installer.rb#L26).


@sleepinyourhat Where are the `g` and `b` of the BatchNorm op being updated?
These should slowly track the empirical mean and variance, right?


``acts_as_list`` uses callbacks, which are triggered multiple times and therefore break the ordering.
You can take a look at [the source](https://github.com/swanandp/acts_as_list/blob/master/lib/acts_as_list/active_record/acts/list.rb#L432)

The callbacks are not executed with ``update_columns``.

I haven't checked yet if following 0e8f1d5c4564b8f187ff468c6c2ac474c6930dc8 is sufficient.


`this.queries` contains the spec, `queries` contains the request queries.


See [line 74](https://github.com/killbill/killbill-cloud/commit/6cf89be8936f8bfed93be5247c19007553e854a3#diff-fce29b6c3d488a98988c065471e51b7fR74) above.


Thought that, too. http://php.net/exception.getcode tells something different. But the constructor contains the `int` type declaration. https://3v4l.org/q8VN0

It's PHP. ¯\_(ツ)_/¯

Will change.


Extra empty line detected at block body end.


It's a semantics issue here. The more i think about it the more i think we should throw an error:
1. User first install plugin `FOO` version `1.0` using a `pluginKey` = `FOO_KEY`, and the mapping gets created
2. User then install plugin `FOO` version `2.0` using a  pluginKey` = `BAR_KEY`, we should raise an exception.

(Noe of course, things could be reversed meaning the first key was `BAR_KEY` (user made an error). Tough luck? or do we need to provide a way or override the key?


No, but the user can (in `specified_arg`).


Ok will remove.


I didn't modify - a lot - this part yet since it will be moved into somewhere else - either *domain_cert* action or in a new *certificate* category. I will see that at this time, thanks for the idea.


Let;'s first agree on the semantics, then i'll add the check.


This is really a bad way, I am suggesting https://github.com/evonove/django-oauth-toolkit/issues/338 to get rid of it


This is really a bad way, I am suggesting #338 to get rid of it


It's unclear what can cause "decryption to fail" and so it's unclear whether that makes all the MediaKeySessions fundamentally unusable as "unavailability of the CDM" clearly does.

For example, perhaps "decryption failure" can occur because something it wrong with the media stream ? (subsample encryption map in CENC is malformed). Perhaps some of the MediaKeySessions associated with this MediaKeys are in use by other media elements sharing the same MediaKeys (if that is possible).

I think "decryption failure" should be handled just like media corruption and leave the state of the MediaKeySessions alone.


Oh that's so confusing!


Needs [bugfix](https://github.com/apmuthu/frontac24/commit/9162f914dba3be2197ab1566a62d674b3297e6c9)


![fa 2 4 dashboard link on default theme](https://cloud.githubusercontent.com/assets/2995044/11817975/a6d24e58-a37d-11e5-8bec-bacfdb1818aa.png)
![fa 2 4 canvas theme](https://cloud.githubusercontent.com/assets/2995044/11817978/a742ccdc-a37d-11e5-89a4-ef3ec2a070aa.png)
![fa 2 4 dropdown theme](https://cloud.githubusercontent.com/assets/2995044/11817977/a7411b6c-a37d-11e5-8d16-8d99f9f9c0dc.png)


yes, this commit changes probably every occurrence of after and before (the direction of the -0-> changes), so 207 is OK

makrai@makrai-imac:~/repo/4lang$ egrep -w after 4lang -c
158
makrai@makrai-imac:~/repo/4lang$ egrep -w before 4lang -c
83


Note: IMO that's old school. SNMP reports CPU usage in similar way - maximum is number of CPU*100. Zabbix concept is that CPU max is always 100% (because it's percentage). Then it's also easier to compare CPU usage of different machines with different number of online CPUs. :-)


Yes; sorry; I forgot that I extracted that interface from abstract class.


@Tetradeus Please open an issue if you believe you've found a problem. A comment on a commit is likely to be forgotten. If you do open an issue, please provide a sample that illustrates your problem as, going on the description above, I'm not sure how the problem can occur. As long as the _definition_ for your custom module exists before the `Jackson2ObjectMapperBuilder` is created, the beans for your custom modules will be created as needed and injected into the object mapper.


Can you change on merge?


I'm not sure I like that datasources have to be aware of concrete HTTP requests (even though it's definitely an improvement already). There should still be a separate component responsible for translating an HTTP request into a request for a fragment. [Example](https://github.com/LinkedDataFragments/Server.js/tree/develop/lib/routers) in the JavaScript code.


Noooo -- why would you do this??!
The whole point of interaction and animation is that you can see it!
The animations are to guide the eye to the changes resulting from your selection. If it is out of view and you have to scroll up and down you completely loose the intuitive understanding and comparison of the distributions.
I know my initial layout was far from perfect, but at least it was on the screen.


Too bad your comment will disappear in history once you squash this commit. Better put it up on Gist.


I'm confused - why is it treated differently than `packaging` for example?

In general, I find this piece of code quite difficult to read / review :crying_cat_face:

* Let's figure out what kind of validation we need (above comment)
* If we do, do we need a proc? Could we use a method instead?
* Maybe the whole merge / lookup code could be extracted / unit tested separately?


I'm using the simpler version from the BatchNorm paper: I use the empirical means/variances from each batch directly without any tracking between batches. g and b are the learned mu and sigma parameters that are applied to the activations *after* they are normalized.

I've seen that fancier moving average version around in a few places. Have you seen any evidence that it's much better than the original?

> On Dec 15, 2015, at 9:15 AM, Jon Gauthier <notifications@github.com> wrote:
>
> @sleepinyourhat <https://github.com/sleepinyourhat> Where are the g and b of the BatchNorm op being updated?
> These should slowly track the empirical mean and variance, right?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hans/rembed/commit/50f940dfbebde8bd416c56f1d13a9b44eb726a1e#commitcomment-14991820>.
>


We cannot do that as `shipmments_attrs` can be `nil`.


Forgot to remove debug code


The [upgrade](https://github.com/symfony/symfony/blob/2.8/UPGRADE-2.7.md#form) says that `choices_as_values => true`  should be added when you use `ChoiceType`, but this applies the other way around when you use the `TimezoneType`:

Before:
```
$formBuilder->add('timezone', 'timezone', array(
            'choices'        => array(
               'value' => 'label',
               'value' => 'label',
               'value' => 'label',
            ),
            // ...
        ));
```
After:
```
$formBuilder->add('timezone', 'timezone', array(
            'choices'        => array(
               'value' => 'label',
               'value' => 'label',
               'value' => 'label',
            ),
            'choices_as_values' => false,
            // ...
        ));
```
or
```
$formBuilder->add('timezone', 'timezone', array(
            'choices'        => array(
               'label' => 'value',
               'label' => 'value',
               'label' => 'value',
            ),
            // ...
        ));
```

`ChoiceType` has `false` as default value for `choices_as_values`, which is okay. The types that extend it have `true` as default. This leads to unexpected behavior if the `choices` option is used on the `TimezoneType`.


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


Change jQuery script links method.
"PHP - include_once()" -> "<script>...</script>"


EconomizerIntegrationNew (lgu1234) - x86_64-MacOS-10.9-clang: OK (1956 of 1974 tests passed)

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


we can probably remove the `channelName`


Pedantic comment but the unit is a "jansky" and not a "Jansky" (like a "watt" or a "tesla").


Thanks!


here too


Shouldn't the mapping be removed on `uninstall`?

Interesting test:

* Install `stripe` with `me:analytics` key
* Oops: uninstall `me:analytics`
* Install `analytics` with `me:analytics` key


Thanks for this fix.


In following statement you have about 16 Qt functions. They will be less in case of arg().


You could just use `self.topLayoutGuide` instead of the magic number. `100` seems excessive though. The navigation bar is typically just 42 pixels high.


Uhm from the code it looks like you didn't initialize Kokkos? I mean
thats also what the message says. The serial backend doesn't complain,
since it does not have any internal state, but every other backend will
complain.

Christian

On 12/15/2015 09:52 AM, amklinv wrote:
> @mhoemmen
> This is the test that's failing on the dashboard, and it's failing here, when I initialize this matrix.  It only does so when OpenMP is enabled and displays the message
>
>   p=0: *** Caught standard std::exception of type 'std::runtime_error' :
>
>    Kokkos::OpenMP parallel_for ERROR: not initialized
>    Traceback functionality not available
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/trilinos/Trilinos/commit/0d587efe1875d935e7db100011d7c948dff33000#commitcomment-14991241


Thinking about it once again, maybe it would be simpler to add only <liferay-ui:error /> and set only "warning" in the request ... https://github.com/liferay/liferay-portal/blob/master/portal-web/docroot/html/taglib/ui/error/end.jsp#L34


a


I send an email as well but here it is: shouldn't there be a Kokkos initialize somewhere in the code?


@asier-gutierrez  He seguido las instrucciones que subistes al drive, y me dice que:"no hay nada que cambiar". No se si es cosa mía (al no tener permisos) o es que directamente no funciona o que no lo he hecho bien.


Welcome to Ruby :-)


this is for 3.1 only, no relation to 2.8


`@link` is considered to be [deprecated under the draft PSR-5](https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#710-link-deprecated)


TeamCity TripleA :: UnitTest [Build 546](http://home.k-moeller.dk/viewLog.html?buildId=1430&buildTypeId=TripleA_UnitTest&guest=1) is now running


Sure.


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


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


TeamCity TripleA :: UnitTest [Build 546](http://home.k-moeller.dk/viewLog.html?buildId=1430&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


Already fixed by a subsequent commit


Got it, thanks. But one more question, it seems that initially mgs is `"% " + msg` -> 1 space
in the next 2 ifs 1 or 2 spaces are added. What do you mean by 0 spaces? That 0 spaces is added to with these ifs if its 100% right?


pleprince,

Good point. I forgot all about that. It also looks like I missed some path definitions also.

Thank you.


TeamCity TripleA :: UnitTest [Build 547](http://home.k-moeller.dk/viewLog.html?buildId=1434&buildTypeId=TripleA_UnitTest&guest=1) is now running


use placeholders instead of ng-value, remove getStandardStartHour function, from view and controller.


as above


@gottesmm did we really want to remove the "FIXME" part of this?  (This change is based on your comment in #501).


I'm going to try importing the project from scratch to see if I get the same errors.


TeamCity TripleA :: UnitTest [Build 547](http://home.k-moeller.dk/viewLog.html?buildId=1434&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


TeamCity TripleA :: UnitTest [Build 548](http://home.k-moeller.dk/viewLog.html?buildId=1436&buildTypeId=TripleA_UnitTest&guest=1) is now running


nice ascii


Why?
You are introducing magic constants and making the function calls below hard to understand.
What do you gain??


TeamCity TripleA :: UnitTest [Build 548](http://home.k-moeller.dk/viewLog.html?buildId=1436&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


Right, of course, `vs.add_param` makes things automatically trainable. I got mixed up for a moment.

> I've seen that fancier moving average version around in a few places. Have you seen any evidence that it's much better than the original?

I was under the impression that this "fancier" version was the standard—it's what I've seen implemented in a few NN frameworks. These moving averages are used only at test time, in place of an empirical mean+variance of the test batch.

But I have little-to-no experience with BN, so I shouldn't vote in either direction.. :)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1445 of 1445 tests passed)

![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)


@fasterthanlime Please post an issue that describe the problem (Commit's comments are not easily browsable and searchable).


I believe the `tsconfig.json` info appendix is also slated to be moved elsewhere, when a good place for it is found.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1447 of 1448 tests passed)

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


I realized later on what you meant. You are right if I remove the warning and reraise the exception it should stop.


TeamCity Civic Assembly :: Build backend + frontend [Build 94](http://54.229.176.143/tc/viewLog.html?buildId=219&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


what happens with n-ary functions?


I'm not sure about removing `ignorenonframetext`.  That would break lots of current slide shows, I'm guessing -- people may be counting on nonframe text being ignored.  (Unless I misunderstand what this option does.)


>Then why not make, as described in this document?

I thought it could break the backtrace function. I should try it out & rethink about it.


And, would you please rebase them?


@cpuguy83 where is this handled in the new code?


I added filter_var as an optional setting in 5e650f464d4831988a2d09d93579ce48c3aacbcd, thank you for the suggestion.


CGI processing script for this example:

#!/usr/bin/perl

use CGI qw(cgi-lib);
use Data::Dumper;

my $form = new CGI();
my %in   = $form->Vars();

open (OUT,">data");

print OUT "Temp Files:\n";

my %files = %{ $form->{'.tmpfiles'} };
for my $x ( keys(%files) ) {
	my $F = $files{$x};
	my $name = $F->{name};

	print OUT "- File [$name]\n";
	}

print OUT "\nInput:\n";
for $x ( sort keys %in ) {
	print OUT "- [$x] = [$in{$x}]\n";
	}

print OUT "\nEnvironment:\n";
for $x ( sort keys %ENV ) {
	print OUT "- [$x] = [$ENV{$x}]\n";
	}

print OUT "\nRAW File Handles:\n";
my $d = new Data::Dumper( [ \%files ], [ "files" ] );
$d->Indent(1)->Purity(1);
print OUT $d->Dump();

print OUT "\nCGI upload:\n";
my @H = $form->upload("file_names");
my $n = 0;
for my $h (@H) {
	++$n;
	print OUT " - Found File Handle #$n\n";
	}

close(OUT);


XZ is preferable (ef43)


@DDoS Seems OK to me


I am also thinking about hacking on these new first-class entities as of Docker 1.9 next week. Are you following up on that branch? Interested to mutualize the development on a PR or discuss the API?


Discrete Gauss Transform.
Compute density and spatial gradient via GMM


Ah, yes. I should do something about test time, since neither dataset will have the same distribution at test time: SST will have longer sentences at test time, and SNLI isn't shuffled, so there'll be correlations within minibatches.

> On Dec 15, 2015, at 9:36 AM, Jon Gauthier <notifications@github.com> wrote:
>
> Right, of course, vs.add_param makes things automatically trainable. I got mixed up for a moment.
>
> I've seen that fancier moving average version around in a few places. Have you seen any evidence that it's much better than the original?
>
> I was under the impression that this "fancier" version was the standard—it's what I've seen implemented in a few NN frameworks. These moving averages are used only at test time, in place of an empirical mean+variance of the test batch.
>
> But I have little-to-no experience with BN, so I shouldn't vote in either direction.. :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hans/rembed/commit/50f940dfbebde8bd416c56f1d13a9b44eb726a1e#commitcomment-14992211>.
>


With CGI v3.15 I get both images uploaded from iOS.  Here is the data:

Temp Files:
- File [CGITempFile=SCALAR(0xa02792c)]
- File [CGITempFile=SCALAR(0xa0277d0)]

Input:
- [file_names] = [image.jpegimage.jpeg]

Environment:
- [CONTENT_LENGTH] = [642572]
- [CONTENT_TYPE] = [multipart/form-data; boundary=----WebKitFormBoundaryobnvj6Ah60LDEgi2]
- [DOCUMENT_ROOT] = [/xd1/homes/500/kangas/erikkangas.com/www]
- [GATEWAY_INTERFACE] = [CGI/1.1]
- [HTTP_ACCEPT] = [text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8]
- [HTTP_ACCEPT_ENCODING] = [gzip, deflate]
- [HTTP_ACCEPT_LANGUAGE] = [en-us]
- [HTTP_CACHE_CONTROL] = [max-age=0]
- [HTTP_CONNECTION] = [keep-alive]
- [HTTP_HOST] = [erikkangas.com]
- [HTTP_ORIGIN] = [http://erikkangas.com]
- [HTTP_REFERER] = [http://erikkangas.com/test/c.html]
- [HTTP_USER_AGENT] = [Mozilla/5.0 (iPhone; CPU iPhone OS 9_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13C75 Safari/601.1]
- [PATH] = [/usr/local/bin:/usr/bin:/bin]
- [QUERY_STRING] = []
- [REMOTE_ADDR] = [71.174.64.40]
- [REMOTE_PORT] = [59077]
- [REQUEST_METHOD] = [POST]
- [REQUEST_URI] = [/test/process.pl]
- [SCRIPT_FILENAME] = [/xd1/homes/500/kangas/erikkangas.com/www/test/process.pl]
- [SCRIPT_NAME] = [/test/process.pl]
- [SERVER_ADDR] = [172.16.29.14]
- [SERVER_ADMIN] = [[no address given]]
- [SERVER_NAME] = [erikkangas.com]
- [SERVER_PORT] = [80]
- [SERVER_PROTOCOL] = [HTTP/1.1]
- [SERVER_SIGNATURE] = []
- [SERVER_SOFTWARE] = [Apache]
- [SMTPAUTH] = [1053:500:500:250:500:$1$dFvhibUp$0A08TgJhC5DzXdDJQ5sVh/]
- [TZ] = [GMT]

RAW File Handles:
$files = {
  '*Fh::fh00001image.jpeg' => {
    'info' => {
      'Content-Type' => 'image/jpeg',
      'Content-Disposition' => 'form-data; name="file_names"; filename="image.jpeg"'
    },
    'name' => bless( do{\(my $o = '/usr/tmp/CGItemp58446')}, 'CGITempFile' ),
    'hndl' => bless( \*{'Fh::fh00001image.jpeg'}, 'Fh' )
  },
  '*Fh::fh00002image.jpeg' => {
    'info' => {
      'Content-Type' => 'image/jpeg',
      'Content-Disposition' => 'form-data; name="file_names"; filename="image.jpeg"'
    },
    'name' => bless( do{\(my $o = '/usr/tmp/CGItemp58447')}, 'CGITempFile' ),
    'hndl' => bless( \*{'Fh::fh00002image.jpeg'}, 'Fh' )
  }
};

CGI upload:
 - Found File Handle #1
 - Found File Handle #2


Thanks again. Will have a look at it later this evening.

Joe

Sent from my Ipad

> 15 dec 2015 kl. 18:18 skrev Ap.Muthu <notifications@github.com>:
>
> Needs bugfix
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity Civic Assembly :: Build backend + frontend [Build 94](http://54.229.176.143/tc/viewLog.html?buildId=219&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:08:56


You are welcome.
I am a big fan of os.path for multi-platform projects. There are probably already more than enough special cases for win32 in the code... :-)


So primarily this change is to allow grandchild resources, correct?


Yes, fixed.


I'll bring this up to date and add the defect file (wherever that goes).


x_position is telling us which row and y_position is telling us which column, right? Even though in a mathematical x y axis, it would be the opposite, x establishing the horizontal dimension while y establishes the vertical dimension.


Nits:
1. OSS Header on top
2. Sort, remove usings (system should be on top).


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


This change is to make parent singletons handle urls like normal parents do. But yes, the reason why this change was needed is that the grandchild resources of serviceEvent were broken. (Time entry and vendor sub-contract are children of performedBy, which is a child of the serviceEvent)


If start and max_return don't actually reduce the amount we get from the mon, might be best to just skip them for now.  We can add a variant with paging later.


Upgrading to v4.22 and running the same test, I am a little surprised. Based on the data below, the "upload" command does indeed return an array of the file handles.  However, the ".tmpfiles" structure only includes ONE of the files.  Perhaps the "bug" is that this hash no longer works a a way to get at the raw uploaded temp files on v4.x when you have multiple files with the same name.

Is there another way to get all of the temp files ... similar to how you can in v3.x?

~~~

Temp Files:
- File [/tmp/B4bCnyXClI]

Input:
- [file_names] = [image.jpegimage.jpeg]

Environment:
- [CONTENT_LENGTH] = [642572]
- [CONTENT_TYPE] = [multipart/form-data; boundary=----WebKitFormBoundaryobnvj6Ah60LDEgi2]
- [DOCUMENT_ROOT] = [/xd1/homes/500/kangas/erikkangas.com/www]
- [GATEWAY_INTERFACE] = [CGI/1.1]
- [HTTP_ACCEPT] = [text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8]
- [HTTP_ACCEPT_ENCODING] = [gzip, deflate]
- [HTTP_ACCEPT_LANGUAGE] = [en-us]
- [HTTP_CACHE_CONTROL] = [max-age=0]
- [HTTP_CONNECTION] = [keep-alive]
- [HTTP_HOST] = [erikkangas.com]
- [HTTP_ORIGIN] = [http://erikkangas.com]
- [HTTP_REFERER] = [http://erikkangas.com/test/c.html]
- [HTTP_USER_AGENT] = [Mozilla/5.0 (iPhone; CPU iPhone OS 9_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13C75 Safari/601.1]
- [PATH] = [/usr/local/bin:/usr/bin:/bin]
- [QUERY_STRING] = []
- [REMOTE_ADDR] = [71.174.64.40]
- [REMOTE_PORT] = [59081]
- [REQUEST_METHOD] = [POST]
- [REQUEST_URI] = [/test/process.pl]
- [SCRIPT_FILENAME] = [/xd1/homes/500/kangas/erikkangas.com/www/test/process.pl]
- [SCRIPT_NAME] = [/test/process.pl]
- [SERVER_ADDR] = [172.16.29.14]
- [SERVER_ADMIN] = [[no address given]]
- [SERVER_NAME] = [erikkangas.com]
- [SERVER_PORT] = [80]
- [SERVER_PROTOCOL] = [HTTP/1.1]
- [SERVER_SIGNATURE] = []
- [SERVER_SOFTWARE] = [Apache]
- [SMTPAUTH] = [1053:500:500:250:500:$1$1c2507fi$klJgPGnmq0axY/0bxmxWd/]
- [TZ] = [GMT]

RAW File Handles:
$files = {
  '*File::Temp::$fhimage.jpeg' => {
    'info' => {
      'Content-Type' => 'image/jpeg',
      'Content-Disposition' => 'form-data; name="file_names"; filename="image.jpeg"'
    },
    'name' => '/tmp/B4bCnyXClI',
    'hndl' => bless( \*{'File::Temp::$fh'}, 'CGI::File::Temp' )
  }
};
*{'File::Temp::$fh'} = \'/tmp/B4bCnyXClI';
*{'File::Temp::$fh'} = {
  'DIR' => undef,
  'UNLINK' => 1
};

CGI upload:
 - Found File Handle #1
 - Found File Handle #2


The sample is for 1.6 -- the README just needed to be updated.


IRQ5 is not compatible with programs expecting interrupts from a MPU-401, which is hard-wired to IRQ2.