None of these cases do a good job at testing the *state* of the system. They're still just testing the result, and assuming that because an error wasn't thrown ant the result was a status of OK, that the method worked.

Another reminder: unit tests need to account for the *entire state of the system as a result*.

For instance, with register architecture, we need to check that the architecture is actually registered, and that the simbus is using the architecture as it is designed (and not making assumptions about it being PLP).

Hint: **Most test cases will require multiple assert statements**, even though they only test one aspect of the system.


I thought the same thing. But no....... it is necessary. The exit will wait till the subprocess is done, it's weird and maybe there is a better way to do it. Caused me a major headache.


Maybe "Only inject placeholders for multi-line arrays"


As far I can see in none of these two: 620_SLHC [1] or 73X[2].

[1] https://github.com/cms-sw/cmssw/blob/CMSSW_6_2_0_SLHC12_patch/RecoTracker/MeasurementDet/plugins/MeasurementTrackerImpl.cc#L122
[2] https://github.com/cms-sw/cmssw/blob/CMSSW_7_3_1/RecoTracker/MeasurementDet/plugins/MeasurementTrackerImpl.cc#L108


"multi lines array" isn't correct grammar


The terminal window shows
~~~
ajb@polly:/opt/dosemu2/share/dosemu/drive_z$ /opt/dosemu2/bin/dosemu -X -D+dRW -H1
ERROR: Nested thread invocation detected, please fix! (at=0)
ERROR: leavedos called recursively, forgetting the graceful exit!
~~~


not random data. it generates a range() (it fills the array with increasing integers)


不明智的添加位置，未考虑到支付宝跳转没有载入核心 :)


meh


@barbalberto notice the new constructor.


It's nicer to use [static extension](http://haxe.org/manual/lf-static-extension.html) with `using`: `_recentPositions.last()`.


why not a single if :+1:

```


总算找到原因了，就是这句话把 FontAwesome 变成方块字的 = =||（!important 去掉就正常了）


wtf is this


Another note for the near future: architecture will be a complex test suite. It's pretty involved in the backend, and there's no way to test it thoroughly without making a mock class.

Having seen the backend, I can think of at least three test cases that would (and should) fail.
**These are important**


uh


Fixed.


fixed.


Thx, I will adapt the comments tomorrow...


Shouldn't this be "ensure that this property evaluates as `false` in the last module of the reactor."? If it is set to true, no modules are deployed?


@bendyer Was the ordering affected by the version of the compiler used to compile the BL and the APP being different?

Would you have time to Sync up on skype?


:heart:


Context (from @davidaurelio):

iOS has support for “async” bridged methods since May (see https://github.com/facebook/react-native/pull/1232). Async methods return a Promise object in JavaScript rather than accepting callback functions.

The implementation adds a new interface (Promise) and exports methods that use that as type for the last parameter as async functions.


Thanks @sstucki . What about something like following?

     Γ ⊢ T caprod
    ---------------- (CP-All)
     Γ ⊢ All X.T caprod


It's actually not always a 16bit uint. I will make the declaration more explicit in the next commit


Yes, on slow VMs I observe this as did the test team.  In this particular case, something goes wrong and the destroy function is called.  I observed several cases where the code tried to remove the thinpool device but couldn't because the device hadn't become active.


I was doing that when I lost connection to my build box.


Vyskúšal som túto verziu, ide zbuildnúť+spustiť bez ďalších úprav v arucu. (Win 8.0)


93653546-FuelGenShowingInABUPSwhenPV (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1953 of 1960 tests passed)

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


It WAS the case of the wrong DIP switch settings. So apparently the board is shipped with all the dip switches ON (MCU1 address of 0) whereas the documentation and default code is written for the case where all the dip switches are OFF (MCU1 address of 30). A flip of all four switches and the Arduino connects to the board via i2c. Thanks very much, Ken.


This comment should hopefully trigger some benchmarks on my local benchmark server

`RunBenchmarks("toy", compare=>"877acff4572a4ed03b70313f6739607526e772e5")`


`render 'topics/topic', topic: topic`


unittest2 should be in requirements.txt file and pip install should use requirements.txt.


I'll try to explain how this all works and why the lock is necessary.

An RPC command is received and is handled via ServerHandler or Handler, depending on the connection type. The handler then calls the JobQueue postCoro method to create a coroutine and run it at a later point. This frees up the handler thread and allows it to continue handling other requests while the RPC command completes its work asynchronously. postCoro creates a JobCoro object. When the JobCoro ctor is called, the member coro_ (a boost coroutine pull_type), immediately calls the function param it is passed. Its important to note that construction of a boost pull_type automatically suspends execution, which we don't want at this point as we are still in the handler thread context. So in the lambda body we call yield to resume execution. postCoro then calls resume, which schedules a job on the job queue to continue execution of the coroutine in a JobQueue worker thread at some later time. When the job runs, we lock (explained why further on) on the member mutex and call coro_ which continues where we had left off. Since we the last thing we did in coro_ was call yield, the next thing to continue on is the function that was passed into JobCoro ctor. It is within this function body that the caller specifies what he would like to do while running in the coroutine and allow them to suspend and resume execution. A task that relies on other events to complete such as path finding, will yield to suspend its execution while waiting on those events to complete and continue when signaled via the resume method. There is a potential race condition that exists here where resume can get called before yield (Technically the problem only occurs if the job resume schedules is executed before yield). If the resume job is run before yield is called, undefined behavior occurs (likely an exception). The lock ensures that coro_ is not called again until we exit the coroutine. At which point a scheduled resume job waiting on the lock would simply gain entry and call coro_ doing nothing and returning as we have already completed.


I see this generally in slower, under equipped VMs.  I have never seen this on baremetal or even purposed VMs.


bad copy-paste ;-)


bad copy-paste


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


settings.history_limit is str type, is it right?


Yeah I think in a symlink too, but there's standard to create at npm postinstall? docummented support exists from WinXP through `mklink`. The other solution to use `require(riot-compiler)` is register the function to `parsers` (as custom parsers do) or `npm link` to `node_modules/riot-compiler` subdir. both requires work from the user.


OK, probably it is a useless protection.
The not very nice feature of that piece of code is that to decide if I have to call addStrip or addPixel for a given vector I have to look into the first element of that vector. Therefore I decided that I had to cover the case when the vector is empty. Probably it is not needed.

                  Andrea

On Oct 29, 2015, at 15:46 , ebrondol <notifications@github.com<mailto:notifications@github.com>>
 wrote:

As far I can see in none of these two: 620_SLHC [1] or 73X[2].

[1] https://github.com/cms-sw/cmssw/blob/CMSSW_6_2_0_SLHC12_patch/RecoTracker/MeasurementDet/plugins/MeasurementTrackerImpl.cc#L122
[2] https://github.com/cms-sw/cmssw/blob/CMSSW_7_3_1/RecoTracker/MeasurementDet/plugins/MeasurementTrackerImpl.cc#L108

—
Reply to this email directly or view it on GitHub<https://github.com/cms-sw/cmssw/commit/867561954ca2ee8c5a708cc051c4e2ec5735bf3d#commitcomment-14068940>.


`render 'posts/post', post: post`


what do you think about adding a docker-compose.yml file with rabbit in it?


thanks for this commit!


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


tracking in #456


Ah, ok. Then we just have to be sure that the middle-ware doesn't ever respond with a 403 without consulting Kinto about it. But this should be OK for both middle-ware and CDNs as long as Cliquet consistently serves `Cache-Control: no-cache` (which it will do for all resources except anonymous ones, with https://github.com/mozilla-services/cliquet/pull/522).


Why not communicate with the server directly? Opening a shell just to use netcat is extremely wasteful. If you have to use [socket](https://docs.python.org/2/library/socket.html) directly.

Also update [lintserver](https://github.com/tonyhffong/Lint.jl/blob/master/src/Lint.jl) so you can pass in the contents of a file. Making temporary files is a bad practise


g


I'm assuming you're referring to the GangaRepositoryXML?

This particular file gets a lot of errors from the XML reader we have implemented chocking badly when passed bad/empty/non-existant files and until this is all cleaned up I would prefer not to touch too much with this as it causes Ganga to not start due to uncaught exceptions for users when a database becomes corrupt.
We need to a look at how we're storing/reading data on disk imo and write some unittests around this, but this will take a back-seat while other work is going on.

As for the Local/MassStorage files. I think you're correct, this should probably be fixed in the future at some point to avoid future problems.


fix #21


> I'm curious about the use case for change_log_level? It could be inconvenient in some scenarios since it's typically a configuration option to set logging levels. For example, say I'm trying to silence requests logging output and I set the requests logging level to ERROR in my config, but any functions wrapped with this decorator would use a different, hard coded logging level and would not be silenced.

I've wanted this for one-off things like migrations or management commands - something where I DO want to capture logging output which would normally be silenced.

It's noteworthy, however, that while this will temporarily override the threshold for a particular logger, it WON'T override the handler threshold.  For instance, the `console` handler in our `settings.py` has a threshold of `INFO`, so you still won't be able to make it print out `DEBUG` logs:
```python
In  [1]: with change_log_level('testlogger', logging.DEBUG):
   ....:     logger = logging.getLogger('testlogger')
   ....:     logger.debug("debug")
   ....:     logger.info("info")
   ....:     logger.warning("warning")
2015-10-29 14:54:20,530 INFO info
2015-10-29 14:54:20,530 WARNING warning
```

I think my ideal tool here would be something instead of altering another library's log level, captures any logs from that library, and relogs them under the current context's logger.


Ok, thanks for the explanation.


Why don't you subtract the mean?


Okay. I get the point now.
I would like to add an ``__eq__`` method in the ``IndependenceAssertion``, like you suggested.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#UixOG1OWQGmqKzPQncdq8Q


I'd rather have a ``subscribe_to_tree`` and ``unsubscribe_to_tree`` (what happens if the server is slow and they select it twice?), or at least name it ``toggle_subscription_to_tree``.  It does match the existing ``subscribe`` pattern, so consistent in one way.


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


A path is missing? Does this Twilight Lord Everstop?

(no, I am not ashamed of this lame pun)


@RuhmST 哈哈，谢谢！Thks！


Gut. Ich muss noch kapieren, was diese Breaking Change mit dem #var bedeuten soll.


Need to change back to pull from master once we have tested update on dev branch


- close #21


missing space before (


space before (


hey buddy is this a git commit message or the op-ed section?


and here


Hi master，how are you？are you busy recently? when the nesbox-server is enable? I waited a year,but still not coming.


In my experience not subtracting the mean is a bad thing! Although in this case it must be the mean pixel values not the mean image.


Welcome, lets see if cm is going to bring the support back which i really doubt


I know it is a g_error so it will abort but you are leaking the memory here.


This commit also fixes the issue #3.


I have changed the type to bool instead of string.


todo o método deve usar um único linq/lambda.


same leak here


leak


@youngsterxyf 先谢谢你。线上有三个 memcache 的实例，但是不知道怎么做监控，你的代码给了我很多思路。


space


Unfortunately, still not working: the algorithm is lauched but not concluded.
Impossible to check errors on log and history because they don't report the original script sent to the FUSION command line.


i wonder if it's the right rate limit, we're already backing off a lot.


Fixed in d7e3fac0080cbc6330df595e3ffccc7b2f607315, thanks!


yeah so this confused me for a while and i was convinced we had our locking wrong. simon good call on comment, will add


probably need `--init` option too.
it will initialize any uninitialized submodules.


Note misspell in the commit message, it's a "segmentation fault" not a "syntax error".


good news is the tests definitely cover it, so we can be fairly confident in that


I must have forgotten it.
Yeah it definitely would degrade the performance.
This BGR value should be subtracted from the image: ` [103.939, 116.779, 123.68] `
Can you please update this file? Thank you :)


Refining the iOS-side of our Cordova-project, I also stumbled across these two compiler warnings. As I'm (also) always trying to pay attention even to warnings, I was tempted to implement exactly the same change.
But on the other hand, when altering foreign code, I always ask myself 'what was the intention of the original developer, when writing the code exactly this way?'. Here: could it be, that these "&"s were *not* intended?

So I went searching around on stackoverflow; finding not only the explanation for them "&"s, but also that taking them out will (theoretically) cause the App, that uses this plugin, to crash on iOS prior to 4.0 with an `EXC_BAD_ACCESS` – and finally find a way to prevent those warnings *and* keep the code working.

Don't get me wrong: I don't want to appear nagging, nor play the schoolmaster; the whole thing probably won't matter no more at all, for which developer is still using such an old version of Xcode, that allows a Deployment Target older than 6.0? And which user has devices with iOS older than that?

These two notifications were introduced with 4.0; devices still running 3.x would crash (with `EXC_BREAKPOINT`(?)), if one simply would attempt to `addObserver` with them (see [this stackoverflow](http://stackoverflow.com/questions/11212143/iphone-app-crashes-on-start-up-on-ios-3-x-uiapplicationdidenterbackgroundnot) – here's also mention of "Available in iOS 4.0 and later."); the `#ifdev` in the first code-block of course didn't work, when Deployment Target still was < 4.0; the `#ifdev` in the second code-block wouldn't be necessary, because in it our `if (nil != &UIApplicationDidEnterBackgroundNotification)` already is implemented.

[Another stackoverflow](http://stackoverflow.com/questions/3002833/weak-linking-on-iphone-refuses-to-work) did exactly code as in the change above – and did get that `EXC_BAD_ACCESS` on 3.0; and found out in the first answer that "you have to compare against the address of the symbol, not the symbol itself" … so here we are: the old version of line 121 above would have worked – but (the actual version of) Xcode complains about that construct.

So:
* we could simply accept it and live with these warnings;
* we could develop for iOS versions only newer than 3.0 and omit line 121 with that `if` (and of course line 124 :wink:);
* or take a look at another [stackoverflow](http://stackoverflow.com/questions/7396711/what-happens-when-observing-uiapplicationdidenterbackgroundnotification-on-old-i) covering the same problem, also giving the solution and explanation with "&" in the second answer, even explaining that an `#ifdev` won't solve the problem – and finally in the [third answer](http://stackoverflow.com/a/7400358) suggesting the right solution.

And with that in our fork I altered line 121 to:
```obj-c
NSString *currentSystemVersion = [[UIDevice currentDevice] systemVersion];
if ([currentSystemVersion compare:@"4.0" options:NSNumericSearch] != NSOrderedAscending) {
```

Well, sorry for way too much text and for, well, playing the schoolmaster at the end :wink:


If only AWS documented their actual rate limits anywhere!  We're left guessing.


You know that lots of people use the Parse JS framework with the native one to fill in holes in the api. It may be smarter to write an integrate function that will add proxy methods to the Parse object rather than clobbering it all together.


you are doing a dup but there is no finalize method


align properly this param


just move the methods before teh class_init so you do not need the prototype?


5 college days, perhaps? One week, and would always include a weekend to allow the Hon Sec adequate time. This would mean if we had a meeting on first Tuesday of the month, by the second Tuesday the minutes would be circulated to attendees of the meeting for comment (allow another five days) and then they can be approved and published as the first agendum of the next meeting.

If someone wants to read them prior to approval at the next meeting, they can have them three days into this commenting period as per line 135.


![300 jokes.](http://risovach.ru/upload/2014/01/mem/svoya-igra_41526448_orig_.jpeg)


no need for brackets?


* I assume, this returns just one row in the evaluation table?
* What we need is, the question set has 2 entries which each have their own questions 18 final and 15 mid i believe. So I am expecting about 35 rows in the evaluation table.
* For the one, two, three, four, five column. The sum of all the values for each row shouldn't be more than the amout of people in the class. @SkaeX


// Testing


testing


Wrong mugshot. Please use the one in the asset folder (see template and description). Thanks!


This function creates a special "matrix" object that can cache its inverse.


why you delete :cry: ?


This function returns a matrix that is the inverse of 'x'


Mosab Abumezied


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1030.4f31a0ae12adddf8c9ca96fd19b79dd7ebe10308](http://build.palaso.org/viewLog.html?buildId=65092&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 703, ignored: 6 Build time: 00:10:14


How about: Circulate to attendees for comment within 5 days. Allow 5 days for comment. Then provide to all members, whilst making it clear that they are unapproved and unpublished. Approve at next GM. Publish to public within 5 days.


There was a problem with the creation of a new \DateInterval object. It's a DST issue - more info you can find here: http://stackoverflow.com/questions/33392011/datetimediff-and-end-of-daylight-saving-time/33392122. An exeptcion is thrown for negative hour value, because then the format for DateInterval is invalid. I've changed the creation of that object in this pull request.


Bye-laws are decided by the Committee, not a GM. I'll change this to Committee Meeting on 10/11/15.


@mhoeber I need to check this further: this image and the paragraph following the image are not showing up in HTML output.


Please name this "Launch Screen Saver".


query params?


Haha, I love that this is in our git log :+1:


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1031.7e319d955c29607ee53aaa958c4320c3d2b41b05](http://build.palaso.org/viewLog.html?buildId=65093&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 716, ignored: 6 Build time: 00:09:24


其实eventHandler还可以这么写
```
widget.regEventHandler(XXXEvent.class, (w, e) -> { ... });
```
请和我念lambda大法好


Another one of Ricotez' commits, by the way


issue5221_memoryAllocation (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


As I imagined, it has a better way. I will wait other tips before commit again.


@mhoeber A minor merge conflct item that needs resolution


@rpg777 this is awesome! Check out this measure for an example for how to configure an html page with information from Ruby, start around line 57 or so: https://github.com/macumber/openstudio-vA3C/blob/master/ViewModel/measure.rb

The html template has a little code snippet `<%=os_data %>`, the variable of this name in the Ruby script gets converted to a string and replaces this marker:

https://github.com/macumber/openstudio-vA3C/blob/master/ViewModel/resources/report.html.in


### [Codacy](https://www.codacy.com/app/avnguyen/SeLion/commit?cid=18692199) detected an issue:
#### Message: `Use explicit scoping instead of the default package private level`

#### Occurred on:
+ **Commit**:  ecb85197405439f12f13d72a3829c5082dc1b0ae
+ **File**:    [server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java](https://github.com/ILikeToNguyen/SeLion/blob/ecb85197405439f12f13d72a3829c5082dc1b0ae/server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java)
+ **LineNum**: [34](https://github.com/ILikeToNguyen/SeLion/blob/ecb85197405439f12f13d72a3829c5082dc1b0ae/server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java#L34)
+ **Code**:    `    File downloadedFile = null;`

#### Currently on:
+ **Commit**:  d90c0db32ac61ba14d69235b5b19474ce05ecf20
+ **File**:    [server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java](https://github.com/ILikeToNguyen/SeLion/blob/d90c0db32ac61ba14d69235b5b19474ce05ecf20/server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java)
+ **LineNum**: [33](https://github.com/ILikeToNguyen/SeLion/blob/d90c0db32ac61ba14d69235b5b19474ce05ecf20/server/src/test/java/com/paypal/selion/grid/FileDownloaderTest.java#L33)


@lakruzz There was a possible confusion: default blog page `/blog/` dedicated to tag `featured`. I'd like to use existing `tag-page` layout for it, so as not to duplicate.
But main blog page should be named `Blog`, not `Featured`, so I made this trick: you can alter a tag-page name by setting a title. Hmmm... but it brings this page to menu as well. So we can fix it later if you need.


Perhaps it would be better to change the second append from .append(" ") to .append(", "). It also might help to surround each objects to string with quotes or singles quotes so that the end result looks like

[ "Person@59c9b9ca", "Person@be48a12ac"]

I think this delimitation is easier for the eyes to spot than spaces. However, it is possible that the toStrings of the objects contain commas or quotes and that could be an issue (maybe ` would be better than quotes)


this.sendAction('goto', this.get('iri'), 'DeleteObjectOperation', {'I-am': 'a-query-param'});

or better

routing: Ember.inject.service('rhea-routing');
...
this.get('routing').goto(this.get('iri'), 'DeleteObjectOperation', {'I-am': 'a queryParam'});


This if does not work as expected and should be if not .... or not ... or not ....


:+1: :)


I don't see concat.append(" ]") anywhere


cc @ide it'll make you happy


Done.  I made a Pay method in the Banker class.


no `malikania::` prefix needed :)


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


也可能不是，应该是微信api变动了。
1、以前有个file.api的子域名，现在都用api.了
2、'itemlist' 变成 'item' 了，'filename' 变成了'name'
还有一些其他变动，估计。暂时没发现。


Я получаю ту же ошибку , как vboby , однако , что исправление для этого? Я понимаю, что него redirect_uri не так, но где это установка ? Разве это URL переадресации в обстановке Microsoft App или что-то еще ?

https://login.live.com/err.srf?lc=1033#error=invalid_request&error_description=The+provided+value+for+the+input+parameter+'redirect_uri'+is+not+valid.+The+expected+value+is+'https://login.live.com/oauth20_desktop.srf'+or+a+URL+which+matches+the+redirect+URI+registered+for+this+client+application.


@stijnswaanen :thumbsdown:


@Rillke All good comments, thanks! Wish I had noticed them around the time when the PR was merged...


I would rename this to something like getPrintString or stringify


@lakruzz have a look! you suggested using a kramdown tag to cut first caption of a page to show on the blog-index page. I suppose, it'll be more easy and universaly to cut posts by a tag <!--break-->


This condition feels weird although I'm not sure I've deciphered it yet... shouldn't it at least be `!c->connection_closed`? I assume it's adapted more or less as such from curl though...


Should be $defined($clang_format)?

More generally: I forget the pass-invocation protocol, but isn't it important for the pass to say "try the next index" rather than falling through to the $STOP case?


Missing: "checks" + File.separator + "whitespace" ....
Input files need to be in the same package as the tests.


wow!


well i derped it, let me fix it so it boots :8ball:


shouldn't ambient be true by default?


It should be overwritten, but yeah, we don't have tests for it, will add it


**Code Quality: Nitpick**

Always use lower case for variable names. Capitalized case is used for naming classes. [See PEP-8 Style Guide](https://www.python.org/dev/peps/pep-0008/#naming-conventions)


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


develop (pflaumingo) - i386-Windows-7-VisualStudio-12: OK (2337 of 2366 tests passed)

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


This feels a bit weird - you won't run into it in most cases, but it can hit you when you least expect it. I'd rather see you  loop (perhaps move most of the code into something like tls_write_internal)


Renamed to PropertyColorGroup since that's the most "official" name I could find on http://monopoly.wikia.com/wiki/List_of_Monopoly_Properties


@lakruzz I tried to automate permalink setting up. No result, need more time for that


Its supposed to ensure that any remaining data in the encoded buffer is being processed when the connection was closed, since no new data is going to come anyway. Not sure its really needed, since worst case it would just get called again to process any remaining data I imagine.


The naming of these are a bit different than normal static functions, since they look like a win32 API function even though they aren't. For this usecase it might be ok though - is it inherited from curl?


Sure.


this whole unittest shouldn't do anything with authentication, this is copy/pasted from elsewhere ;)

Instead it should test a known verb ("PULL"), and an unknown verb (e.g. "UNKNOWN"), and expect a success in the first case and an error in the second case.


It just seems a typo, -C (capital) is the current shorthand for --color. Shall I make a new pull request?


**Code Quality: Nitpick**

This method name might not be the best. Could you use something like `add_office`? Since that's what it's doing. Adding an office to the object.


clang_format is guaranteed to be defined by the precondition check

you're right about the fallthrough, I'll fix


nevermind I see what you mean now


The one's with a space might cause a problem, you may have to use
replacetext(colour, " ", "")
Also it would need to be adult/ right?


qdel!


quick question: why this has to be restricted to FFT? Isn't `FFT` just an arbitrary interval, too? Rather `paddingIntervalCentered` should be an own `Op`, independent from FFT?


whoops :)


`configURL` sound a bit misleading. What about `cloneURL` ?


Ok... I assume it's been thought through more than I've done now at least :-P


Leave a line note


this is exactly what I was talking about


nice one!


Please only change one thing per commit. Thanks.


leave this one as the last one, for readability.
N columns of input, 1 column of expected output.


I would suggest to make that column a QString, with the actual verb in it. More readable, and more extensible.


indent here doesn't match the other lines.


go to the line between each cases. Line is too long. Same for the others case.


Please remove resContainersList because it is not used in this controller


Not precisely - I don't think it's entirely correct the way I have it now
but it was definitely wrong the way it was before. I'll have to think more
about the intended behavior of these components again but it kind of seems
like the ambient component shouldn't exist - diffuse should cover it's
purpose.
On Oct 29, 2015 11:18 AM, "Varun" <notifications@github.com> wrote:

> shouldn't ambient be true by default?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tangrams/tangram-es/commit/c9275c7615b768619a0711238c9a0ab1a681ef6f#commitcomment-14070024>
> .
>


@mcostalba: Why can't you just remove: && !((pos.nodes_searched() >> 12) % Threads.size()))

The penalty is always per thread. Every thread is going to lose a fixed amount of time per 4096 nodes. If you can get that below 0.5% of the total cost or sth like that, then there is no ELO loss...

If it works with one thread, it should work with 64. Or am I missing something?


Very nice! :tada: :tada: :balloon:


зачем?


@mlubin @jakebolewski  [Here](https://api.github.com/repos/JuliaDiff/ForwardDiff.jl/commits/180f9dfa5e4616989b67ef46047e9682914a6a84/statuses) are the result statuses posted to this commit. This benchmark "fails" for every metric, which is what I wanted to happen -  the `"toy"` benchmark on this commit is slower than the same one on the previous commit due to the chunk size change in the benchmark setting itself.

It would be cool to set up a bot that replies to a benchmark-triggering comment with the relevant status information, or at least a link to the status page.


Team Members
Sara Higginbotham


I am connecting to an amqps queue and I wish to create a receiver on a topic.  In a previous version I was able to use createReceiver('topic://topicname') but it appears after passing through this, that will no longer work.   I can pass in the topic name, but if I add "DEBUG=*" in the environment, I get an error:
Thu, 29 Oct 2015 15:18:36 GMT amqp10:client session error:  { descriptor: [Int64 value:29 octets:00 00 00 00 00 00 00 1d],
  value: undefined,
  condition: { contents: 'amqp:unauthorized-access' },
  description: 'User username is not authorized to read from: queue://topicname',
  errorInfo: undefined }

Thanks for the help!


Two different domains on single line.


Susu Yan


Site Ma


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1424 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.4%25-green.svg)


the attribute's name is *name* and not *default*, sorry


fortuna3g - XXU variant, fortunave3g - XCU variant


redirect_uri устанавливается в авторизующем приложении и должен полностью соответствовать реальному редиректу, который происходит после авторизации.


Initializing stack variables is usually always a good idea, but in this case I'd prefer if you didn't, because `rb_get_args` will unconditionally set it anyway, and I only initialize these variables when they're optional on the ruby side, with their RGSS mandated default value (in which case `rb_get_args` will leave them alone).


issue5221_memoryAllocation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1960 of 1960 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)


@DirectXMan12 Need a pull number on this.


**Code Quality: Nitpick**

This would have been a good place to make use of objects. Objects will give more robust functionality when compared to regular dictionaries. Also, you will be able to call methods on them e.g `person.has_office()`.


currently only c++ is using this, so this is the last chance to change tag number and make it
ClosedLoopParams closed_loop = 1;
closed_loop tends to be less confusing in the generated code and it is also more consistent with names of other load types.
Actually, it's fine to leave the tag number but renaming to closed_loop might be a good idea.


we cannot remove this yet, wheezy has 12.0.0


renamed to gitconfigURL...


Do you have any idea stdout messages on plugins are lost on linux? (and duplicated stderr on PCSX2 console will be nice too).


There types are wrong.  `Eq k` is a constraint, not a value.  And there shouldn't be an `m`.


Updating the project definition only has a inderect dependency with the containers. Please remove this line or give an appropriate explaination.


OK


@playachronix as an initial attempt to get it working more quickly for you, could you try passing `defaultSubjects: false` to your client's policy? I'll write up some tests quickly so we can fix this for good without the workaround


This is very nice, but if you're going to stub it out, I'd prefer if you did so in the core `Graphics` class (graphics.cpp), and then just called that function from here normally.


as we spoke, renaming this one to "led", and removing toggle would be way to go!


I think you need to add an upper bound `S` to `X` so that we can distinguish between the case where `X` is capability-producing. Then the context `Γ` should be extended with `X <: S` as usual.

Also, there should be a rule for the Top type. I guess it must be assumed to be capability producing as capabilities are subsumed by Top.


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


Neither of these work for Medusa BTW.


@KillAshley
Since Faz orbits Eve and Vix orbits Duna, I generated an Eve clone and a Duna clone for GN compatibility.

An alternative could be not using clones and give Faz the orbit Eve has in stock, and give to Vix the orbit Duna has in stock.

let me know which option you prefere ;)


the codes is ok to me.


We can sync up on material behavior in JS too soon... I recall there were a
lot of nuances around default behavior, especially when simple or no
materials are defined (the most common cases actually)... but I can't say I
now remember all the nuance off the top of my head :)

On Thu, Oct 29, 2015 at 11:26 AM, Matt Blair <notifications@github.com>
wrote:

> Not precisely - I don't think it's entirely correct the way I have it now
> but it was definitely wrong the way it was before. I'll have to think more
> about the intended behavior of these components again but it kind of seems
> like the ambient component shouldn't exist - diffuse should cover it's
> purpose.
> On Oct 29, 2015 11:18 AM, "Varun" <notifications@github.com> wrote:
>
> > shouldn't ambient be true by default?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/tangrams/tangram-es/commit/c9275c7615b768619a0711238c9a0ab1a681ef6f#commitcomment-14070024
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tangrams/tangram-es/commit/c9275c7615b768619a0711238c9a0ab1a681ef6f#commitcomment-14070286>
> .
>


Please add each command in its own line, separated by `&& \`.


GenericSnippetPage


Hi there, I want to port cm12.1 on Padfone A66 (and then use it to port SailfishOS). Is this branch fully working as the official Asus 4.1.1 Rom do? Thanks


Merged


Still slightly buggy:
1. Indexing hashmap not done yet
2. Sometimes double empty slots will appear in tasks
3. Search term doesnt display yet


You can replace strlen() with "ret" as it will be the same.


I thought about this part, wouldn't it just be enough to simply cut the buffer to the maximum size and return that from tls_write, the generic retry code would then re-send the remainder of the buffer?

At least ffurl_write and retry_transfer_wrapper seem to suggest thats how it works.


Doh! Thought I had reversed those.


missing ".checks." as compilable inputs should be located in the same folder structure as their package.


Вы имеете в виду , чтобы перенаправить по умолчанию для Joomla ? Потому что у меня , что набор для перенаправления все страницы они пытаются войдите на .

То, что я пытаюсь сделать, это после того, как пользователь уже вошел в Joomla , они идут на " Обработать социальных счетов " и нажмите на иконку Live.com связать свою реальный счет на счет Joomla.

В API У меня есть
Имя приложения устанавливается:
http://DomainName.org

Перенаправление URL набор для http://www.DomainName.org/

Целевая Домен www.DomainName.org

что не так?


Maybe rename to Event to keep to the current jargon. Check to change this later everywhere.


전에 있던 댓글은 rorlakr.github.io/rubyworld 에서는 보이나 world.rorlab.org 에서는 보이지 않습니다. 2가지 url 에서 같이 보여줄 수 있는 방법이 있을까요?


Reverted Weblate error over english language


`--no-mime-magic` is absolutely critical on POSIX generators with broken libmagic - css files for example will otherwise be uploaded with the incorrect header and be completely broken for the user without knowing why right away.

`--no-preserve` prevents leaking username and permissions, otherwise a header is passed and set on S3 that is the received by the client:

```
x-amz-meta-s3cmd-attrs: uid:1000/gname:xenith/uname:xenith/gid:1000/mode:33188/mtime:1446050473/atime:1446050473/md5:7a747784b1f67ae304c09eacc7210c2c/ctime:1446050585
```


I'm not sure about this. I think an easier way to support autofill for chrome* (*: and co). would be to add a special case in the service when the event is related to package chrome*: it would retrieve the url by locating the url bar by id and use that url as appName.


rorlakr.github.io/rubyworld 에서 보이는 댓글을 world.rorlab.org 에서도 같이 보여줄 수 있는 방법이 있을까요?


Do this cleanup when add/remove the directory. Keep it clean in the struct you use. And store namespace length as well.


If gApplication isn't a ```std::atomic``` this is still a race condition.


Renamed & correct type for the event. (https://github.com/ylatuya/longomatch/commit/cb621d936449adc7be3bebe4d343d6b24e9f9e1b and https://github.com/ylatuya/longomatch/commit/7126c9402214bc3b1a1d604d67b2f39e16a90ee2)


@mbroadst I added defaultSubjects: false to my configuration but I still get the same error message in the logs.  Let me know if I am missing something else.  If you get a patch in for this. I'll try to watch and I can make the change locally for now until it goes into a published version.  Thanks for the help!


yes you're right !


If fTypes isn't a ```std::atomic``` than this is also still a race condition.


Ok thanks. You're right, looking at it more closely it's a hashtable lookup on the sessions than the streams. We'll merge in the latest code sometime soon and I'll enter an issue if I can reproduce it. So far it's only crashed at shutdown so it may not be that serious.


@playachronix can you possibly provide me with a small code sample of what you're running. If it's converting `topic://topicname` to `queue://topicname` thats incredibly strange. Just trying to write a better test on my side


URL перенаправления неправильный, праильный посмотрите в настройках плагина авторизации.


Here you should check for at least one '/', otherwise "/namespacefile" will match "/namespace" while it shouldn't.

Or you ensure your namespace is terminated with one "/".

While at that, it would be nice to remove all double "/", not just leading. "/name//space" should be handled the same as "/name/space"


Unnecessary spacing detected.


hot fix


@GianlucaGuarini , @cognitom , forget the parsers problem, It was my error, I had installed the compiler with `npm link` in my local repo. With the normal `npm install` works fine, I'll correct it.

btw: with `require(riot-compiler)` the `make test-compiler` does not generate coverage information, so I think this tests are redundant, riot-compiler have all of this and more, and cover is 100%. We need test for `riot+compiler` only.


Aproveitando o *pull request*, creio que poderia apagar essa linha **81** :smile:


indent fail


Sounds like it could be helpful. How would you simulate errors though? For example, if you try to set a non-existent mode, the file read fails with `-EINVAL` rather than reading the file. Since python is dynamic, you could just temporarily replace a function with something that calls `raise`, but this would not work for some languages.


I think this is fine; image and following paragraph show up in my local build, no errors in build. Same with image and paragraph at line 568 below.


gem for bootstrap is included over here


I think I'm a little confused on what you've ended up with? Are you going to keep the `child_id` column on the `sports` table? Or are you going to keep the `children_sports` table to join `children` and `sports`? Could you give a brief recap on what you've settled on?


Ooops found a misstake, the mail will be sent to every player which is online. Will correct it later or tomorrow if I have time. :)


This function causes a compilation error from `scram b` because of mismatched indendation.
Shift 53... left by 4 spaces.


Этой проверкой пусть роут занимается. Не контроллер.


is TPOTM_CACHE_EXPLAIN omitted?


Whoops, sorry. Will fix that.


OK


@playachronix also, are you using ActiveMQ or one of the JMS brokers? It looks like it from what you're trying to specify as a link name. I do see that I overlooked the parsing for link names when I refactored the connect parsing to point to a policy so that needs to be fixed. AFAICT this would be specific to an ActiveMQ-esque parsing scheme


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


maybe int16_t is better here


It didn't :/ but we got it working.


@taylorotwell #10757


Hi snuk182, I want to bring sailfishos to the Padfone but first I need a working Cm Rom (cm11 or cm12, also cm10 can be used as a base). Which is the state of the porting? Do all the features already works?
Thanks in advance, BJ


> How would you simulate errors though?

No idea. But I am more interested in checking that working code is not broken by the latest commit. And the tests could be very specific, like check that motor0 is connected, that position returns what is actually in the file we created, etc.


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


Ooo, this looks great. :+1:


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


注意typdef的用法。


I noticed this once, was never able to reproduce. :+1: :100:


just drop peerDependencies?


pdb :-)


@hamnis
just out of curious, why remove joda-time? is there any issue with it? thanks


`onMoveShouldSetResponderCapture` should be included here too. :smile:


ok


i would rather prefer not to use an id that is so generic as ```error-image-loader```, just dump the id and use the following solution

it's better to use only one id in the case of the all backend log box, because at a UI level this box is also unique. The id of the backend notification log ```notifications``` so if
```
//the cpt file
<div id='notifications' class='ttc-notification'>

//the .js file
if ($('#notifications .ttc-ajax-loader-box').length == 0)       //you can check if there is an ajax loader within the notifications box.


Hi blackjack4it.
Nothing works. No ETAs of any kind :(


This API that takes a path to a file is not good. It would not work on system without FS (riot/Contiki), and doesn't allow helpful messages to be generated (such as the error url)

Make this take a callback that is feed with the failing request, make it return a response as serving regular requests.

Of course you can offer a helper that serves a file "#ifdef FILE" (or whatever is the feature macro we use). This ifdef should be based on a public symbol defined in some "-buildopts.h" and guard all APIs related to static files.


ok, in fact it was already in backend cassandra module


Hmm, that's not the right way to fix this. I think `openReadRaw()` used to already throw the proper exception itself (hence the try-catch around it), but that behavior must have gotten lost when I did the big ext supplementing rewrite.

Right now in `openReadRaw`, we just directly `PHYSFS_open` the path and assert on the handle (meaning a non-existant file would have actually aborted the program in a Debug build), but this very likely means I wrongly assumed the function was only called internally during the rewrite, with paths that were already checked beforehand to exist for sure. Since calls to `load_data` are also plumbed through it, we should do proper error checking there though. In case `PHYSFS_open` returns null, physfs should have already set an error code we can check, so we don't even have to call `exists` before opening.


@Piicksarn fixed and rebased it :)


prova 16.52


PTAL, I addressed these comments in (54d5007...), if you think that it looks good, I'll commit the following as one PR:
* (4d375a2f...) Clarify Section 7.3
* (54d5007....) Addressed comments for "Clarify Section 7.3"


prova


I reviewed and good with PR merge.


ack


теги header i footer це добре


I'm sad to see this commit but I understand why it happened :(


risky on platforms that do not have an ar with a deterministic mode :)

-u      Update files.  When used with the -r option, files in the archive will be replaced only if the disk file has a newer modification time than the file in the archive.  When used with the -x option, files in the archive will be extracted only if the archive file has a newer modification time than the file on disk.


це радше sidebar


Code looks good, but comment is incorrect - although the newer smart pointer types have an "explicit operator bool", safe_bool only gives you an "operator bool_type" (which works in practice the same way, but doesn't require the C++11 explicit operator feature).


Вот этот коммит еще примените https://github.com/Arkadiy-Sedelnikov/slogin/commit/7c3ce8aaf9a6271e2c1924a5c50c07d7cd3a1839 и смотрите в плагине какой урл перенаправления вставлять в приложение.


Ok, nothing at all? Does it boot at least?
Thanks and don't give up, I'm asking for some help on Sailfish-os
porting channel (#sailfishos-porters on IRC). Visit it too so we can
talk there and get some help :D!

Il 29/10/2015 16:51, Serhii Plyhun ha scritto:
>
> Hi blackjack4it.
> Nothing works. No ETAs of any kind :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snuk182/android_kernel_asus_A66/commit/3984314869ad057530eeea8ab110273649e94aa2#commitcomment-14070994>.
>


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


See http://php.net/manual/en/migration70.incompatible.php in  "Changes to the handling of indirect variables, properties, and methods"


wow :|


Should "NOT_MAKE_R_ONLY"  be "NOT MAKE_R_ONLY"?


I agree the documentation makes using the `require()` parameter sound OK.   But according to my tests it's problematic, albeit only for an obscure corner case.   Note that delite/handlebars has the same problem.   You can see the problem by modifying deliteful/samples/Combobox.html to have this configuration:

```js
require.config({
	baseUrl: "../..",
	map: {
		'deliteful/Combobox': {
			'requirejs-text/text': 'foo/bar'
		}
	}
});
```

That configuration is only supposed to affect deliteful/Combobox, and shouldn't break how delite/handlebars works.  Yet it does.  When you load Combobox.html in the browser, delite/handlebars ends up trying to load `foo/bar` instead of `requirejs-text/text`.


This gets handled differently in the refactoring I've done.  I basically setup to test take the number of owned railroads as an input.


This should probably just be a warning, not an error?  We don't actually require safe_bool to be safe for functionality, just for improved error checking.


Текст не по центру.


And now you know why I banged my head for 8 straight hours yesterday...

The good news is, this won't be permanent.


OK for me except one typo


inally, you get to see how the compare_cb function pointer typedef is used. I simply call test_sorting but give it the name of sorted_order, reverse_order, and strange_order as the function to use. The C compiler then finds the address of those functions, and makes it a pointer for test_sorting to use. If you look at test_sorting you'll see it then passes each of these to bubble_sort but it actually has no idea what they do, only that they match the compare_cb prototype and should work.


:+1: c863bc45840f04eb7f59c5850997dc1ba3f7b24b


ho fatto un commento a questo COMMIT


PTAL, I addressed these comments in (2107e811619c0ae675246c910d84fb6b70812666), if you think that it looks good, I'll commit the following as one PR:

(238d459ebbf73eb871d26eeb086d9665f948fc56) Clarify Section 8.
(2107e811619c0ae675246c910d84fb6b70812666) Addressed comments for "Clarify Section 8."


додаючи бордер ти збільшуєш розмір, тому сусідні блоки скачуть. можна зробити обводку справа і зліва іншим способом


Thanks a ton for the explanation, which I am now studying.  I hope you'll consider leaving this description (or something like it) as a comment in the source code somewhere.


I thought so too, but then I saw this patch that increased the size from 2M to 4M: https://github.com/raspberrypi/linux/commit/9900679044dc28a779b3afc69191322e8efa6810
This coupled with the fact that vchiq only dma_alloc_coherent(264kB), the change could not be attributed to this allocation alone.
Since this PR is so massive, I didn't want to introduce any uncertainty so I kept the 4M value.

But looking closer:

This was introduced 21 Oct 2012: https://github.com/raspberrypi/linux/commit/d38f8008db4a05e1f5c508022c08816cc0c21d21
```diff
@@ -900,7 +908,16 @@ void __init bcm2708_init_early(void)
   * context. Increase size of atomic coherent pool to make sure such
   * the allocations won't fail.
   */
+#if !defined(CONFIG_BCM_VC_CMA)
  init_dma_coherent_pool_size(SZ_2M);
+#endif
```

BCM_VC_CMA was enabled 25 Nov 2012: https://github.com/raspberrypi/linux/commit/6065063f54e9f8629ab20512e331aedf9de7f326
This would reduce the pool size to the default 256k.

11 Dec 2012 the pool size was changed from 2M to 4M I thought, but actually it was changed from 256k to 4M with the current defconfig at the time:
```diff
-#if !defined(CONFIG_BCM_VC_CMA)
- init_dma_coherent_pool_size(SZ_2M);
-#endif
+ init_dma_coherent_pool_size(SZ_4M);
```
256k was not enough to serve the 264k requirement, so maybe by mistake it was upped to 4M?
I can't find a commit that explains the 2M value.
Is vchiq the reason the pool size was changed in the first place?


*facepalm*


I guess this should be an error packet + DBG, not a WRN.


I'll look into a way to keep the current functionality


arrr, i did not think in those os. Give a path to file is much easier since it is static responses. But i see your point here.

Is much bad keep this function and add another one that uses a callback (just like we do for ordinary requests)  ?  In other os we can just set as not implemented. Or is better go with ifdef ?


D'oh! Fixed / rebased now, along with fixing tests on Django 1.7


`this.props.startDate` is not set and breaks the app.


Да , у меня есть изменить Перенаправление URL в настройки в API , что это говорится в Живой Plugin в Joomla , но она по-прежнему не работает.

Я удалил компонент slogin и заново его . Я также добавил изменения кода из двух файлов , которые были обновлены в течение последнего месяца Live.php и CallBackURL.php

Но когда я добавить эти два новых обновленные файлы , при попытке получить доступ к плагина в Joomla , я получаю эту ошибку http://puu.sh/l238H/bb5a9884ed.png

Херес образ мои настройки API внутри РС.
http://puu.sh/l23a7/8e843f034d.png


Hmm... true.  I was thinking that we do require it to at least *compile*, but this test is testing a bit more than just that.  I'll change it to `AC_MSG_WARN`.


The checkbox snippet works right now as you described.
<?= Html::checkbox('name', $checked = false, ['option' => 'value']); ?>


I would expect that if the value is changed externally and a digest runs then the inside value should update. Is that correct? If so can we add that to this test?


@playachronix also I was wrong about my initial suggestion for a "quick fix", you should instead try doing:
`createReceiver('topic://topicname', { name: 'topic://topicname' })`


PTAL, I addressed these comments in (7ed880f2c7910e6dec044e684beeda82c5bc3aed), if you think that it looks good, I'll commit the following as one PR:

(fcd05db024025fe7b9aa40b15dbb69f9108fa34b) Clarify Section 4.
(7ed880f2c7910e6dec044e684beeda82c5bc3aed) Addressed comments for "Clarify Section 4."


Super unstable. May boot, may not (annoys as hell). Kernel requires jtag debugging of init stage.


Apparently no :/ I'll see if I can fix this with an appropriate `typedef`.


You think this will cause issues?


I think it was vchiq. If you can't see a change in behaviour after removing the increase to SZ_4M I'd suggest dropping it until we have some evidence it is needed.
Not sure if @pelwell has any thoughts.


OK, but you are OK with the 'A "safe" replacement for operator bool () that behaves more like an explicit conversion operator even in C++98.' sentence though?


@cvrebert: I left grunt-contrib-uglify to the previous version due to an issue with the comments.


call this function to show ads


LGTM. However I think using shared variable for the learning rate, code can be much simplified. I will take a look into it.


I would do _MANAGER_CRED_DIR = os.path.join('/etc/cloudify/', _CRED_DIR)


And use _MANAGER_CRED_DIR here


This function is preventing installations from using advanced data types, however, changing it causes mgets and msets to fail :/


Argh, git snafu ... I made some modifications after adding it to the git index, but forgot to re-stage the change.  I've checked in the updated commit here dc8574a6076884435d26bc5ac5838ef98e7c0f0a.

The idea is that, since we are trying to infer the best schema from a sample of data, we should leave some room when choosing Int instead of Long.  In other words, if we see Int.MaxValue in the sample, we should probably choose Long for the column type even though we haven't seen anything larger.  In fact, I've set the threshold to 2/3 of Int.MaxValue.  It may be a little too loose, and we can adjust it if data shows that it needs be.


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


This does not add trailing "\0", and there is no room for it as calculate doesn't account for it.


Just deleted the integration


Glad to hear of your success Chris!  Yours would be the 4th case I've run across where wrong default DIP switch address settings caused problems.

FYI, the pertinent postings have now been removed from the letsmakerobots web site, but the board's designer had several posts there regarding the unfinished and incomplete state of the board's firmware and he was pretty negative about it ever being finished due to unacceptable current support costs.

The controller works fine for basic motor control including motor speed encoding, but more advanced features like measuring battery voltage, reporting motor current, etc. probably will never work.  Just a word of warning to prevent future frustration!

Anyway, glad to hear of your progress!

Ken

> On Oct 29, 2015, at 11:51 AM, chrisnelnine <notifications@github.com> wrote:
>
> It WAS the case of the wrong DIP switch settings. So apparently the board is shipped with all the dip switches ON (MCU1 address of 0) whereas the documentation and default code is written for the case where all the dip switches are OFF (MCU1 address of 30). A flip of all four switches and the Arduino connects to the board via i2c. Thanks very much, Ken.
>
> —
> Reply to this email directly or view it on GitHub.
>


I don't feel very strongly about hardcoding the `assets/` part, but forcing a .ttf extension feels wrong, since that is not the only type supported by SDL_ttf (and more types might be supported in the future).


@mbroadst That appeared to work (your last suggestion).  I don't really know how the broker is set up, it is run by another company, and I am reading off of their queues.  I am going to troubleshoot with them to see if I am now properly connected.


PTAL, I addressed these comments in (2c5d25c83d4786f300b38d9e9b36de4634a5e2b3), if you think that it looks good, I'll commit the following as one PR:

(1a4f884b2398bfee59df01579dd280751aed4ccb) Define the maximum number of bytes transforms may add to a word
(2c5d25c83d4786f300b38d9e9b36de4634a5e2b3) Addressed comments for "Define the maximum number of bytes transforms may add to a word"


I have a mugshot called klette.jpg in the mugshot folder. Do you need the
full path?

On Thu, Oct 29, 2015 at 4:07 PM, Alexandra Leisse <notifications@github.com>
wrote:

> Wrong mugshot. Please use the one in the asset folder (see template and
> description). Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iterate/iterate.github.com/commit/da84002fb7dff51ccbd95fa4170019f94e1a1e3e#commitcomment-14069674>
> .
>


I just back-ported the commit into 6.02 to get additional changes done by Philippe merge cleanly.

One could always extend the scope of the gROOTMutex lock to include the ifs.

In practice, do we actually expect TApplication to be created after extra threads are initialized?

About fTypes ... why can't this list just be created in the constructor?


Nope, this lie was correct.


You needed to change only this.


@playachronix okay, we've been primarily developing against SB/EH (for @noodlefrenzy) and apache qpid (for me), so there are known issues with brokers like ActiveMQ that support these types of naming schemes. I'm putting together some tests to ensure we can support the whole range through policies.


<sigh> I ran a 'make check' after building the latest and got
make[1]: *** No rule to make target `out/Debug/obj.target/libcpptest/libcpptest/googletest/googletest/src/gtest-all.o', needed by `out/Debug/obj.target/libcpptest/libcpptest.a'.  Stop.

So apparently there are some Google Test requirements that are missing from my build system.


Yes, that would be cleaner.


Have you done a `git submodule update --init` recently?


ok =)
I need to revert this?


change column width


This needs to be  also be done before `python setup.py install` as WMAKE is used in the install process.  So maybe change to something like:
<pre>
Before installing or using simphony-openfoam , make sure the OpenFOAM environment variables are set by the following command:
    source /opt/openfoam222/etc/bashrc
</pre>


Actually, the save: call is needed so that the tableview handler would update the comment view. Else, new comment won't be seen until the user refreshes the screen.


why have you checked in ngrok? it's a 12Mb in size compiled binary


Read your commit log.. I reverted these because they created checkstyle warnings.

<review, layout


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


Hi, i can't install pysmithplot. I used pip install pysmithplot and that didn't install.. Why ?


There is no problem in replace the last char with \0 here. I do this in the else (that meas the encode function worked correctly)


Adjust width in bar plot.
Keep testing tradeoffs under other countermeasures.


Could use `Enum.random` here.


I used the more repetitive form since we're going for clarity and not English prose ;)

PTAL, I addressed these comments in (ac149e19cb89ac1e6ce29d9884ab936f5de3f8bd), if you think that it looks good, I'll commit the following as one PR:

* (2459bb743cf67461549fa03937c6fc9622ff9590) Use consistent bit convention in Section 5.
* (ac149e19cb89ac1e6ce29d9884ab936f5de3f8bd) Addressed comments for "Use consistent bit convention in Section 5."


Bonjour,

Je fais suite à : Apprenez à coder avec JavaScript, chapitre 6, Comptage du nombre de voyelles.
J'ai un soucis dans mon code, si je mets :
console.log("Il contient " + nbVoyelles + " voyelle(s)"); en dehors de la fonction j'ai le message suivant :
ReferenceError: nbConsonnes is not defined

Comment puis-je obtenir le message suivant dans mon code ?
Il contient 4 voyelle(s) et 4 consonne(s). (pour le mot Anaconda)
Voici mon code actuel :

// Renvoie le nombre de voyelles d'un mot
function compterNbVoyelles(mot) {
    var nbVoyelles = 0;
    for (var i = 0; i < mot.length; i++) {
        var lettre = mot[i].toLowerCase();
        if ((lettre === "a") || (lettre === "e") || (lettre === "i") || (lettre === "o") || (lettre === "u") ||              (lettre === "y")) {
            nbVoyelles++;
        }

    }
    console.log("Il contient " + nbVoyelles + " voyelle(s)");
}

// Renvoie le nombre de consonnes d'un mot
function compterNbConsonnes(mot) {
    var nbConsonnes = 0;
    for (var i = 0; i < mot.length; i++) {
        var lettre = mot[i].toLowerCase();
        if ((lettre === "b") || (lettre === "c") || (lettre === "d") || (lettre === "f") || (lettre === "g") ||              (lettre === "h") || (lettre === "j") || (lettre === "k") || (lettre === "l") || (lettre === "m") ||              (lettre === "n") || (lettre === "p") || (lettre === "q") || (lettre === "r") || (lettre === "s") ||              (lettre === "t") || (lettre === "v") || (lettre === "w") || (lettre === "x") || (lettre === "z")) {
            nbConsonnes ++;
        }

    }
    console.log("Il contient " + nbConsonnes + " consonne(s)");
}

compterNbVoyelles("Anaconda");
compterNbConsonnes("Anaconda");


correct scroll bug


Notes for eventual PR:

* Proper strcmp() check means we now only get one guaranteed return code.
* To stop both notifications from appearing, we need a way to discern if a disk was added or removed. If we don't have a label, then it was removed. If we do have a label, it has been added.


You need to store all pending digest until they finish so you can delete on v1_close() if any were pending.


This could be `:serve_ball` rather than a tuple, because there's only one element.


This middleware should perform a check on the user session, and calls `next()` if it's valid or sends a `401` if not.
The middleware should be injected into the routes that create, modify or remove resources **only**.

Currently, this middleware will run a run
 - A regex test
 - A String function
 - Another regex test

On every resource on the server.


Интересно...Я бы не догадался наверное))0)


Ну сорян))0)


не туди задала


This die message is incorrect.  It should say "or set PTERO_WORKFLOW_SUBMIT_URL env variable."


Навіщо?


Ah. No, I haven't.
<groan> There's Yet Another Submodule? </groan>


...but that did the trick


I mean in the encode case. It doesn't add bull terminator, just the buffer version does it.


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1960 of 1960 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)


The `.authenticate` middleware should be injected into the routes that require authentication, **alone**. not the whole express instance.


@oelze-cit  @henriqueoelze  Let me know what kind of problems you are having?


Size = calculate encoded = no space for null byte.


This is a somewhat non-descriptive variable name; no reason to abbreviate "configure". I prefer more imperative/less ambiguous names like `log_to_stdout` or `use_stdout` for boolean variables. (`configure_stdout` doesn't really tell us where we're configuring stdout).


I will remove the option from files.h and the default config. However I will keep the option internally.


You broke it here.


.... here ...


Не туди задаєш


And here


Is this trace necessary to commit?


I think I have the solution to your problem with dismembering.
1. carbon/attacked_by gets overriden by carbon/human/attacked_by
2. def_zone doesn't actually seem to be the attack zone. I'm not sure if def_zone is anything, actually


buena esa, duglas


인코딩이 깨지네요. View를 클릭하시면 한글로 보실수 있습니다.


Mind doing this in a PR next time? There are other things like the `placeholder` attribute that could be removed with this.


There is a problem with your code because you used integral sign but there is no unicode defining so it raises SyntaxError of Non-ASCII character '\xe2' in file basic.py on line 60...
add this into your first line : # -*- coding: utf-8 -*-
and also you can add a bash comment of : #/usr/bin/python


this might recursive loop on itself indefinitely, you might want to handle the success case here rather than support the default - by not supporting default you also allow the compiler to warn you to add a new description for a case when you make one (just a neat little reminder for future self).


Did the linux binaries push? I wasn't able to test that in travis because the AWS keys are only exposed in `master` builds.


Need to remove the request handler from the server.


Yes.


jhj


I thought about removing it, but it is in 0.2.0 so it needs to have a deprecation warning added first, then removed in 0.4.0 at the earliest.


Unknown at the present time. This is one of those things where an issue can get hidden away and you might not understand why something broke until months down the road. PIL is well known for this behavior. Since this is in xbmc/Makefile.include, it's scope is limited to kodi main build and not depends.

Thinking on this, in the main build, ar is used to create intermediate archives like filesystem.a (see xbmc/filesystem/Makefile), so if you touch some filesystem cpp file, compile it, then ar the object into the existing filesystem.a, the old symbols are present and without the -u, they will not get replaced with the new binary code.

What is the exact compiler warning that motivated this ? Did you track back why the -u was present in the first place ? Unfortunately this means back past the infamous 45285e8a9300cd754a760560640b75b09f98035e commit.


I am new to this and am looking for help.  I have run a scan using clamtk 5.19-1 thru ubuntu 14.04.3LTS.  When the scan is done it shows 2,000+ possible threats.  Most were PUAs with Packer.  I individually clicked on delete and they went from highlighted to dull and in the action column it went from "None" to a dash.  On four items with win.trojan.virtob a screen came up and asked if I wanted to delete this.  I clicked on yes and in the action column the "None" changed to "Delete".  I then ran the scan again and all but the four win.trojan reappeared as a possible threat.  I want to copy my data files to a jump drive and then do an erase program on my hard drive.  I plan to reinstall my programs and then copy back my data files.  Will I be safe doing this or will I still have an infected computer.  Any help is appreciated.


Cool!


asdasdasdasdasd


This commit stops the project to compile using the makefile due to ifstream


Yes, you dog.


@nathanleclaire Could we skip the `Run './bin/docker-machine create --driver name' to include the create flags for that driver in the help text.` line when a driver is actually set?


comm


LGTM.


You fucktwat! Kanj ikke jør sånn! Stupphul!


Resolves issue #3 - implemented expandable and filterable object hierarchy


@playachronix actually, I've added these basic tests: https://github.com/noodlefrenzy/node-amqp10/commit/67c2903f690667b287ecbcf610becf85e59fce0b, and can verify that we were indeed parsing the addresses correctly.

The difference with the suggestion I added above is that the link _name_ (not the _target_ of the link) would be forced to be whatever you wanted (e.g. `topic://mytopic') vs using our default behavior of adding a uuid to the end of the link (e.g. `topic://mytopic_some-really-long-uuid`). This shouldn't have caused a problem.

Would you be willing to retry your test with the latest code in master, and to provide me with a trace log of a test run where it fails?


@nathanleclaire `MachineName = .*DriverName` -> me confused


@lnielsen what is the aim of this module? Will there be real example of usage for when you make the real `1.0.0`?


Error occurs with this line :

    Error: Found item String/install_scan more than one time


I'm sorry to make you repeat yourself but please bear with me; this isn't obvious to me as I've never used this software (even after rereading your comments multiple times). So, it looks like the MAlonzo backend is the default if you're not passing other options explicitly, right? In that case it seems to me (again) that there shouldn't be an option to disable it and `ghc` should be an unconditional runtime dependency.


@whenrik So it is not nullable?


Rename labels to DVD like OG Nox 5


Bad namin


is this really needed?


The point of this pull request is to remove vehicule traffic without fuel.

When you are in game for long time, sometimes vehicule traffic run out of fuel and stay on the map. The addEventHandler will detect the fuel statue change between full to empty.


Explain a bit how it's supposed to be used. I see two ports, like authenticated and token, how do they relate? Maybe authenticated should be removed and people use ERROR to notice failure?

How to know when the token expire? Can we persist the token?


We could add a `skip` property to `NodeData` to allow skipping at the beginning of an element:

```js
elementOpen('div');
  skip();
  elementVoid('inner');
elementClose('div');
```


Wow that went well!


supported ?


Zeile 8 erschließt sich mir nicht, rest geht klar. Öffne doch gerne nen Pull Request, da kann man Änderungen gut beurteilen.


What could be wrong if these are not highlighted? Is this known bug? Where too look? Also any registers names are not highlighted as well.


@mbroadst I can pull latest and give it a shot.  Perhaps connecting to topics just needs to be covered better in the documentation :)


I agree this hurts the migration story for the Ranges TS, and I don't feel it got enough discussion in LWG when this issue came up. @StephanTLavavej felt the most strongly about the issue, so maybe he'll chime in?


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


First working version of cross section functionality #2


there's a method for this in backend cassandra


Hopefully it fares well at LTC...


there's also something in backend cassandra for this


Good luck at LTC. Hopefully this wasn't just a luck run!


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


You can build one on top of the other if it's easier. If it's not, then you can keep both. It's more like an implementation detail.


@VoyagerOne : most impressive.
Besides tesing at LTC, would you give your original idea with downshift of about 4 a chance?
HistoryBonus[d] = int(-1-25 + 1.045 * pow(d + 0.49, 1.8));


Indeed, we could add some documentation about it, but it's varied depending on what broker you're using.. AMQP 1.0 itself has no concepts of exchanges, topics, queues, etc. It only deals with "links". Beyond that there are a slew of differing opinions on how to handle the broker side of the equation between the major vendors in this market, so we're sort of stuck playing catchup to their rules :)


First working implementation of #5


it doesn't look like a singleton anymore ...


@ludivinecp Je pense que la place de ce fichier est plutot dans le dossier des migrations. Il ne s'agit pas d'un model de l'application en tout cas ;)


Also, if you could send me the code you use to generate the result that would be great (sensitive information redacted of course)


Satisfactory functionality and layout - resolves #5


Are these lines still required for this test or can they be removed?


Warning was ``ar: `u' modifier ignored since `D' is the default (see `U')``. See https://github.com/xbmc/xbmc/pull/8315#issue-113996151. So basically it makes no sense if its there. From what i get is that 'u' was there because of the old default when it comes to deterministic mode of binutils.


LGTM.


Please add a space after ``{%``


exaclty, that is why i put \0 on size - 1. I just don't care with the last char.
But ok, i can alloc space for the '/0'. It's just a random string so i thought replace the last byte was good enough.


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


Is this line really necessary, @JuliusSenkus?


Here I think you should list what CUBA you support on the nodes of the JYU-LB lattice ..or at least a subset of them.

Something like
`[CUBA.VELOCITY, CUBA.RADIUS]`


No need for ``if messages`` when iterating over ``for message in messages``.


@ludivinecp je ne comprend pas pourquoi tu as une migration `CreateRoles` et une autre qui `CreateCreateRoles`. Est-ce que tu peux me guider ?


Done.  I had an irrational aversion to private helper methods in tests.


closes #8


My intention was to maintain the vagrant provisioning behavior of using xtrace, while not imposing that on dind provisioning.  While I find xtrace useful for debugging if something is wrong, it's easy to miss the forest for the trees during regular execution.


less dependencies. java.time is now in the jdk ( also known as 3.10 ). This is written by the same person as wrote joda-time.


@phamdacloc The save still happens with `[[ContextManager sharedInstance] saveContext:self.managedObjectContext]` and the comment appears without a refresh. Are you seeing something different when you test?


Shouldn't these be Predicates and not Partitions?


Please add a configuration entry for that instead of hardcoding ``False``.


Remove empty line.


okay, who committed this using the email address "root@sickkids.sickkids.ca"?


I don't know how you cloned my repo but if you did it right you would end up with an clean project. The only libs that come with the repo is my version of youtube extractor and the your frontend as submodule which have to be updated using submodule. Don't relay on my console test frontend since its kinda old. You have to do exactly what I wrote before on that other page. The packages youre talking about (I think you mean the nugets) should be restored on first build in the right version. There is no such thing as running against VS15 packages since the version is specified explicitly.


@mbroadst On latest, I can createReceiver('topic://topicname') and it works as intended. (still need the defaultSubjects:false part though).

Here is the basics.  I'm using when promise library but it works well with this other one.

```javascript
var policy = AMQP.Policy.merge({
	defaultSubjects:false,
	connect: {
		options: {
			sslOptions: {
				keyFile: './certs/client-key.pem',
				certFile: './certs/client-crt.pem',
				caFile: './certs/broker.pem',
				rejectUnauthorized:false
			}
		}
	}
});
var client = new AMQP.Client(policy);
client.connect('amqps://user:password@<ipaddress>:5671')
	.then(function() {
		console.log('Connected');
		return client.createReceiver('topic://mytopic');
	})
	.then(function(receiver) {
		console.log('Receiver created');
		receiver.on('errorReceived', function(err) {
			console.log("errorReceived: ", err);
		});
		receiver.on('message', function(message) {
			console.log("Message Received: ", message);
		});
	})
	.catch(function(err) {
		console.log("error: ", err);
	})
	.done();
```


You can't chop the last char as it would break the base16 encoding. It should be 2 chars per byte, then instead of 16 bytes of random data you end with 15 and a dangling/trailing char at the end.

Allocate size + 1


It would be good to demonstrate getting an option from the psl.properties file with a reasonable default value somewhere in here...


Same here.


I think in other parts of the code this is called `Observer`?


@gdarmont @emacarron When is the release of 1.3.0 version?


Please use ``**kwargs`` to make this more readable.


By removing the Invalidate(); the form stops flashing when the mouse is moved.


I'm already seeing deltarpm being used, indicating that it is installed in the default image.  Is your concern that alternate images should be assured of deltarpm being installed before package installation?


Can we move this comment somewhere else? It might even be worth putting the main method at the top of the file, explaining all the steps, and then defining the functions.


The comment is not seen when I tested. Only after adding the save: call back had I seen the comment. Else, I had to navigate back then reload the comments for it to show up.


I like the idea of separating defining the Predicates and defining the Rules/model. This distinguishes between data description and model description.


My commit didnt change anything in the bet sequence. I just noticed it when i was testing. I tried to figure it out, but didnt find the exact problem. What was happening was i would bet and the bot would call, but then i would get prompted for action again and this would repeat until i folded.

Sent from Yahoo Mail on Android

From:"ak212" <notifications@github.com>
Date:Thu, Oct 29, 2015 at 12:36 AM
Subject:Re: [Poker-Bot] something is wrong with the betting sequence (5b1dfbe)

What does your commit have to do with the bet sequence?

—
Reply to this email directly or view it on GitHub.￼


What are the concerns about u16 -> usize and u32 -> usize? The smallest size of usize is u32?


@yeshasimaria remove


Concernant tes 2 remarques, c'était des tests pour créer un fichier de réservation. Je ne sais pas trop pourquoi ils apparaissent quand même alors que j'étais sur une autre branche...


how to check?


This is a confusing function name. Maybe something that indicates that we're setting up for inference rather than updating the MAP state?


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


Ooops, be careful.

https://help.github.com/articles/remove-sensitive-data/


The DB population should happen in its own function, separate from inference. Maybe in the updateInference function?


@phamdacloc can you test this out on the branch `release/5.7` with the merge patch?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1425 of 1425 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)


you should use instanceof operator instead of getClass() method


Same comment - move database population to its own function or a generic setup function.


the information about hot the token expires comes in the response that is the string i put in the token port (at least for twitter)
I do not parse it because i do not know if the response changes between different services. I think it changes because twitter gives other informations like screen name.

The token string should contains at least the token and the token secret. But i do not know how they are represented in different services. I guess each service that uses oauth will have to parse string and use the information as they desire, at least this is what i'm doing in my twitter's node.

About the authenticated port, it can be replaced with the error port. I'll remove


Is this an overall thing you want for the guides? I have been using h1 `#` for the top level title of a guide. You are starting things here with h2 `##`


It's unclear why you run inference before weight learning?


check this line


For all the guides, I have been having a header. h1 `#`, to be specific at the top-level. I see you changed stuff to h2 `##`. Don't you think we should have one or the other at least?


Why you doing that? Debian has cobbler in its main repo.


@phamdacloc I can't reproduce. If you still see this behavior on the `release/5.7` branch would you mind capturing a video in the simulator and sharing it with us?


My thought here was that OSX is a bit unique since the hardware is limited. Enabled `-msse4.2` will cover all mac [CPUs back to 2010](https://en.wikipedia.org/wiki/List_of_Macintosh_models_grouped_by_CPU_type#Intel_x86). But, I agree with you that this approach is a little too generic. I'll look to see how main line 9.5 performs this check, that's a good idea. I might do a quick update for now to be specific about how we're checking for `SSE4.2` support.

Even something like `$(sysctl -n machdep.cpu.features) == *"SSE4.2"*` would be significantly better.

I'll look into it this evening.


LGTM.


Well yes it puts the item in the pot and on site but the timer isn't starting on its own... due the error.


Change `IO AriaValue` to `IO Maybe AriaValue` or `IO Either AriaValue` to use response status code (for `Nothing` or 404)


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)


check this line too


All this data-loading stuff can happen in loadData. The test set data doesn't change, so it makes sense to do it all in one function.


Скопируйте Callback URL из http://f6.s.qip.ru/SeF39Ecg.png в http://puu.sh/l23a7/8e843f034d.png в поле "Redirect URLs"


The test works both ways, but it is faster with them because the client doesn't wait as long to send the cookie body.


Well I can't see why, but the signature 0xaa55 is missing when process_master_boot_record() runs
~~~
d 0000:7dfe
0000:7dfe 06 8A 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
~~~


renderとpartialの間ってどこでしょうか？


Would be great to have something that computes model performance here using the testLabels


could you explain the rationale of removing public? thanks


Can you split these lines? Just replacing && with new lines to improve readability.


Added back in 3b451a9.


g


fixed https://github.com/ylatuya/longomatch/commit/0a2defc7033c057510b9902be95b4cadcde71b2d


we should use getters for accessing object attributes


ok, that's my fault i would not clear. The nonce does not need to be base16, i'm just using it to enforce it be a string using  chars int the set (or sub-set) of a-z0-9.


I've got ~20 biocyc databases to hand, so if there are others you need let me know, though there is a high degree of redundancy as many come from either MetaCyc or PlantCyc.

Note of caution: thought PlantCyc is a close collaborator of the PathwayTools team, their collection of InChI strings is not as good, and as such, we should always defer to the MetaCyc mappings/InChI strings.


don't forget to add proper javadoc for this interface


no `this.`


@cvrebert: I still get `min-height: .01%;` in the non minified CSS. Maybe we need to tweak the Sass options?


the only requirement is not be greater than 32


This saved me a lot of time, thank you!!


I have no idea how these stats things work but you're doing a great job @VoyagerOne


/cc @sgrif


Will the service return some error code / message here? Generally, Key-Value data stores expose just get, put and delete functions [See [levelsdb](https://github.com/google/leveldb#features)]. In that case, put will not result in error like "key already exists", overwrite if it does.
@proneetv @abhilak Views?


we should use getters for accessing object attributes (same in hashCode)
and also equals method shouldn't use id, but business key


Is this the only way to do a while(condition) loop in velocity?


Sorry for the late reply... there was a bug in 1.2.0 preventing it from working. I don't recall what it was specifically.


It's not a big deal and I am not suggesting changing it at this point but I don't see what benefit is
being derived by introducing the local var "retType".  A good fraction of the "new code" is focussed
on keeping retType and fn->retType in sync.


Looks good


Please do not include pq header in dao interface.


We could have a path for `test`


Don't forget to update the toolset page when changing Mono drops

https://github.com/dotnet/roslyn/blob/master/docs/infrastructure/mono-toolset.md


I don't understand this?


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


options should always be true, to foster trust in a thriving product


I think the import line should be `from gender import getGenders` or maybe the module should be named 'getGenders.py' instead of 'gender.py'?


I dumped the tvshow but cant manage to dump the totalCount and watchedCount but it has 17 episodes total, 14 were watched once only and 2 were never watched.

[izombie-tvshowdump.txt](https://github.com/kvirc/KVIrc/files/23548/izombie.txt)


Sorry, it's a little confusing. I added the local variable because I needed to use the type after fn->retType was changed in protoIteratorClass.


@bernd Don't blame it on me!
![WAT](https://i.imgur.com/FLwB1Cr.gif)


もうすでに.vimrcが汚い…


The license issue was in the documentation. We've fixed it already. Thanks!


whoops sorry. Thanks for the fix :)


@sirmordred i'm quite busy this week so we have to postpone the CM 12.1 release.

Also i was thinking to a Telegram  group for the whole ancora and ariesve users. If you are interested PM back where i can tell you all the details :)


unfortunately yes. keep in mind that velocity is meant to be used for templating, not programming :) ideally we should have an api that returns the correct data to be displayed instead of putting this logic here


ファイルじゃねええええええ


For sure. Just a moment.


Good call!  I didn't need Mock the way I have this set up though.


These are not always hand-defined weights. The second time this function is called, it's with learned weights.


It's best to pass event.target here, since this may have been a change on any of the layouts, not just the active one.


count will never be updated, and we'll always display "Total rows returned : 0" when the query completes


woohoo!


Thank you for fixing it! I noticed that my appointment for November 1 was shown an hour earlier. I was about to report is, but I saw version 4.7.22 on F-Droid, installed it, and the bug is fixed! OK, I had to change the time again, but it won't change back anymore.


w8 a minute, i will commit my POC version of feature (That i develop at the day I open the issue.)


Si tu as des mofifiactions en cours (non commitées) et que tu change de branche, alors les modifications se déplacent avec toi.


There's a class, AtomPrintStream, that can print formatted atoms in a canonical format. We should encourage users to use that class instead of generating their own custom output functions


buffed tank size? \*cough\*


Seems we are working on the same file...may lead to some conflicts, I will stop doing update, let's merge your difference first. After you have some features ready, please make a pull request. So I can sync with your code.


This is really really ugly. Please don't do this.


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


Practicing using namespaces. Not sure it's necessary here at all, but practice is still good.


closedTrain should include the target predicate (SamePerson)


@yeshasimaria load owner from server instead, then update global


@krisv is this commit resolves this PR https://github.com/droolsjbpm/jbpm/pull/320?


use closedTrain here.


Clearer to say if len(elem)>0  ?
Not sure it matters.


That's where the design of mocks for the CI/CT framework comes in. I tested the basics of the Driver class when I was first implementing it using the DKMS drivers for WeDo on my laptop. That worked well.
I think what we are talking about is a "device" that you can load as a driver on the test machine that creates the same entries as are on the EV3 when devices are plugged in. We can start by testing the "rosy" or good path initially, then maybe adding support to fail certain operations as needed.
Basically a DKMS driver that fakes out devices plugged in to the EV3.
This is a non-trivial effort, so I think the right thing to do is to open up an issue (@ddemidov?) against `ev3dev-lang` so we don't lose it, then continue our focus on getting the Python binding stable and more complete.


· Check if tahoe is ready to work
· And activate the tahoe ready flag (creating a file called imReady.txt)


This is not the same.  It should be `energyPositionValidator_->setUndulatorTracking(!undulatorTracking()->withinTolerance(0));` if the code you are replacing is correct.


I think "distributionDB" is confusing. Can we call this "predictionsDB" or something?


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


@playachronix that's the weird part.. I didn't actually change any code, I just added tests! Also, when writing the tests I could confirm that there wasn't a need for `defaultSubjects` (although the test I left in there uses the ActiveMQ policy which will indeed turn those off).

Perhaps there was some issue on the broker setup side of the equation. I'm glad that it seems to be working though.


Here too.


@atogle @karissademi please review. I already cleaned up the console.log and a little formatting in the following commit. Do you see any weirdness in the regex, the show/hide placement, or the URL composition?


@antonkatz did.


@sajidali Thats the first version of feature.
Its simple, i only add groups to the $scope. When I do that, combine and run my project, I received this on console:
```
TypeError: config.data_groups[j].filter is not a function
    at ChartInternal.c3_chart_internal_fn.getYDomainMin (c3.js:1344)
    at ChartInternal.c3_chart_internal_fn.getYDomain (c3.js:1404)
    at ChartInternal.c3_chart_internal_fn.initWithData (c3.js:230)
    at ChartInternal.c3_chart_internal_fn.init (c3.js:83)
    at new Chart (c3.js:34)
    at Object.c3.generate (c3.js:58)
    at e (c3-angular.js:1508)
    at Object.fn (c3-angular.js:1218)
    at Scope.$digest (angular.js:15667)
    at Scope.$apply (angular.js:15935)
```


The closed Predicates here should be closedTest (and maybe that's a bit confusing, so there might be something cleaner e.g. "closedPredictions")


That is possible. Also the coloring has issues in general. I think last time I checked it worked way better in the experimental devenv instance but not in the normal one potentially cause of other plugins that also provide syntax coloring features.


Field should be named `_provider`.


does it mean only unit tests in exec/java-exec will be less verbose ?


@alwey based from the results of the 2 SBF... it seems that slope is the important factor.


Will mis MANIFEST.MF file nümme vorhande gsi isch und de maven build nümm funktioniert hett...


it's easier to pass a path because the user does not need deal with read the file everytime or cache the data. In  the microhttpd implementation we don't even need to care with send chunks of data, we just pass a file descriptor.


Setters shouldn't listen to events, they're just supposed to normalize the data that is being set. Any other behavior needs to be a mix of initializer (or renderUI, bindUI) with change listeners.

You should instead add this to the initializer together with the call ` A.Array.invoke(this.get('layouts'), 'addTarget', this);`. Also, you need to detach listeners from the previous layouts when layouts are changed, just like we call `removeTarget` on the previous value inside `_afterLayoutsChange` and then call `addTarget` on the new ones. Otherwise we'll keep listening to events that we shouldn't anymore.


If you're feeling keen you can now remove the TODO in this file thanks to @robertknight's addition of the `USER_CHANGED` event.


Missing : here


Also, this listener is not following the pattern for listener function names. This should be: `_onFieldsChange`. Better yet, this should be calling `after` instead of `on`, and should be `_afterFieldsChange`.


Actually, both of our comments are off-base since the upstream code has a very different coding style and we should be consistent w/ _that_. Code is fine as-is.


Too bad, Can't win them all !

You boldly try new things, and you are settings up new standards to follow.
Thank you for trying this, and keep up the good work !!!


I think the titles can be bolded, but the contents shouldn't be.  Let's wait until Frank adds the contents then see how it looks.


Hi Joel,

Perhaps I'm doing something wrong, but when I make this change, I get a 422 Unprocessable Entity error as PulledPork tries to use an oinkcode of "Community" with the reg-rules tarball.  I'm guessing this is a bug in the way that PulledPork deals with multiple rule_url entries with the same base URL (www.snort.org).

My quick and dirty workaround is to change www.snort.org to snort.org in the Community rule_url entry like this:
rule_url=https://snort.org/downloads/community/|community-rules.tar.gz|Community


New version of Checkstyle claiming it's redundant.


How is it now?


@GretaCB - good catch ^  I copied this code from `https://github.com/mapbox/preprocessorcerer/blob/master/preprocessors/tif-overviews.preprocessor.disabled.js` <-- looks like it needs the same treatment


fixes https://github.com/mozilla/mozfest-schedule-app/issues/17


@stevegury cool. Glad it makes sense. I should have put more information in the javadocs!


I don't believe that Doze will work as I don't think our sensors support "SENSOR_TYPE_SIGNIFICANT_MOTION".  See https://source.android.com/devices/tech/power/mgmt.html#integrate-doze for more information.


merging this in after @jkcheong92 merges other fixes


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


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


looks good to me, someone else merge the request


develop (pflaumingo) - Win64-Windows-7-VisualStudio-12: OK (2337 of 2366 tests passed)

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


Done.


slight inconsistency in wording to above - `MdsolAttribute does not accept any child elements`
Could we create something like
```python
@property
def no_children(self):
  return "%s takes no child elements" % self.__class__.__name__
```
and have
```python
def __lshift__(self, other):
   raise ValueError(self.no_children)
```


fair enough: I agree this will break the environment-detection build; perhaps I was too presumptuous with this use-case. Thanks for the feedback.


Thanks :)


This slows down my application considerably because I use the trajectory duration scaling feature to increase the duration by a big safety margin. Otherwise the "kill lazy controller" feature will stop the execution of my trajectories midway. It is quite hard to get the trajectory time right since I use speed and acceleration overwrites on the controller (and because it tries to actually be able to stop at any point of the trajectory, which is only somewhat smoothed out).

I assume you had the following sequence in mind:
- trajectory is a "circle" ie start and goal are identical
- trajectory starts
- first position update arrives, state is still old (ie not moving)
  -> we reached the goal ( even though nothing happened yet)

My suggestion is to check whether there has been a "I'm moving" state package since starting the trajectory execution and only then finish it? Maybe make this optional if time > 0.5*total_trajectory_time