I've fixed some of the issues you mentioned but some new ones have surfaced.

The biggest one right now is how (because I couldn't use lambdas without recreating the QMenu every time) that the current tab index had to be stored as a member variable. I think that is quite "ugly" but I have no idea how to mitigate that.

Please take a look at the new code, I could really use a pair of experienced Qt user eyes because I'm not one by any means.


Copy `assertInvalidFunction` from `TestStringFunctions`. At some point we should create an `AbstractTestFunctions` class that has the common assertion methods.


8 days ago


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


I don't see that you defined '>='
Did the book introduced '>=' as an operator yet


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1950](http://teamcity.monogame.net/viewLog.html?buildId=5034&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:15:40


trailing whitespace


This works, but probably makes as much sense to quote the whole string at once.
Also, `nocase` won't be an appropriate name anymore, maybe `query_to_pattern`?


Job scala-2.11.x-validate-publish-core failed for 1724a619,  (ping @folone) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-publish-core/99/):
If you suspect the failure was spurious, comment `PLS REBUILD 1724a619f90ac4bf94168f01d6ca0f7989a6efd5` on PR 15 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-main failed for 1724a619, Took 10 s. (ping @folone) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/91/):
If you suspect the failure was spurious, comment `PLS REBUILD 1724a619f90ac4bf94168f01d6ca0f7989a6efd5` on PR 15 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


I don't believe TL here will be necessary


thx, fixed


Fixed indentation issue.


best commit yet :tada:


I'm getting Traceback Errors when trying to activate the addon after it is installed: SyntaxError: Invalid Syntax.
addon_utils.py - line 299
_init_.py - line 71
 room_maker.py - line 31
tools.py - line 4
OS Windows 7 Professional 64bit
It's a great tool and I'm hoping it can be restored. I hope this message is helpful.


Wonderful! This by itself fixes most issues with buffer command ranges, except for `:100b` throwing error `86` instead of `16`.


spaces needed between `blame`, `=`, and `(`


이건 왜죠


space needed between `blame,` and `change`


I translated "Related reading" section too, but not applied.


We could move this to FRelationCommand instead of repeating ourselves in `CmdRelation{Ally,Enemy,Neutral}`


See comment in `CmdRelationAlly`


Hi,
how to use your package with 3rdParty packeges like https://github.com/emicklei/go-restful
each time i do go get https://github.com/emicklei/go-restful during, after git push GOPATH gets changed to app-root/runtime/repo/.openshift/g


This is convenience variables to be able to check against - much like `http.StatusNotFound`, [et al](http://golang.org/pkg/net/http/#pkg-constants).

This allows the caller to compare to constants rather than random strings sprinkled throughout your code.


What is going on with these sub-project commits? Did a second git repo get initialized in a sub directory?


Didn't link to issue correctly: issue #108 now fixed


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1951](http://teamcity.monogame.net/viewLog.html?buildId=5036&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Passing `true` makes that a deep flatten. It looks like it should be shallow.


Can you explain how 'exact->inexact' works


Done


spaces needed after commas; this applies to multiple lines in this file's diff


This change is source-code incompatible, but as far as I've seen, the changes are minor.


trailing whitespace


Internet Explorer refuse de d'afficher le sélecteur de fichier si on le stylise et même s'il ne supporte pas les shadowRoots, il refuse de le divulguer correctement.
Je vais devoir ajouter une vérification supplémentaire:
*Si Chrome
*Si IE
* Si n'importe quel autre browser.


whitespace misalignment (missing a space before `return`


You could update this to `utf8Slice()`


Should use `Transformer<String, String>` and `CollectionUtils.collect()` instead of Guava types. It's more idiomatic for the Gradle codebase.


These models are unused.


next time, commit after implementing a function (making tests pass)

there should have been at least 3 commits for the mda object


trailing whitespace on multiple lines in this file's diff


Clashes with name of a model installed via `postgres_tests.migrations`


It would be great if you wanted to convert the ScalaCompileOptionsTest to Spock, too :).


I have been prompted to, and I'm going to go ahead and quote, verbatim:  "Leave a comment."


ah, thanks!  I'll change it.


You’re testing with Angular 1.3 also now? Nice.


stray comment. wrong indent level
do not commit commented out debugging lines


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2170 of 2188 tests passed)

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


Jonathan:

I'm going to need you to go ahead and make this text a little bit more interesting.  There is not enough, well, how should I go ahead and put it.. there is not enough Flavor to this revision.


magic number, name this somewhere,
ex:

````
default_array_size : 3

this.default_array_size
````


magic number. why is 0.5 significant

suggest naming that number somewhere


magic number


jonlabs:

u should get urself a clock cuz u wood no tat i hav since added flavr..............ur comment is out of date lol


Passing `false` does indeed seem to make a difference.  I'll get that committed soon along with updated results.


Btw for lodash 3.0 we're having the following methods support shortcut fusion:
_.drop, _.dropRight, _.dropRightWhile, _.dropWhile, _.filter, _.first, _.initial, _.last, _.map, _.pluck, _.reject, _.rest, _.reverse, _.slice, _.take, _.takeRight, _.takeRightWhile, _.takeWhile, & _.where

Notice the lack of `_.reduce`, `_.flatten`, & `_.zip`. A likely addition would be `_.reduce` I donno about the others at the moment.


It is. This is fixed in 7.4.546.

I'm thinking it is worth squashing all commits, since many of the issues in individual commits are later overwritten and could be fixed as a bunch (the formatting errors, for example). Only once I get rid of the logic bugs introduced by my sloppiness, though.


In reference to https://github.com/hubot-scripts/hubot-ingress/issues/21


LGTM, thanks.


If you're in to comparing shortcut fusion you should check out [Lazy.js](http://danieltao.com/lazy.js/)


> Passing false does indeed seem to make a difference. I'll get that committed soon along with updated results.

No need to pass anything, `flatten()` will do.


Awesome work :)


If an api_user param is passed into the SDK, it can safely be ignored and is not required for backwards compatibility with the REST API. Please remove this from the headers, as it is not required.


Can't hard code the table name


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1951](http://teamcity.monogame.net/viewLog.html?buildId=5036&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:16:11


test comment


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1952](http://teamcity.monogame.net/viewLog.html?buildId=5038&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Reduces the runtime of the `test_adding_field_with_default` test by ~60 seconds.


Hi there, just noticed this change when browsing the network tab. Is this ready to be merged back in? I know I've since refactored the Reader class (as well as made some additions) but I'll be more than happy to merge it in if you send me a pull request.


Hello when can we expect the B/Projection addon please

On Thu, Jan 22, 2015 at 2:24 AM, ThreeViews6 <notifications@github.com>
wrote:

> I'm getting Traceback Errors when trying to activate the addon after it is
> installed: SyntaxError: Invalid Syntax.
> addon_utils.py - line 299
> *init*.py - line 71
> room_maker.py - line 31
> tools.py - line 4
> OS Windows 7 Professional 64bit
> It's a great tool and I'm hoping it can be restored. I hope this message
> is helpful.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Antonioya/blender/commit/b9bd571e928046f02baafebf822749aa942f8948#commitcomment-9388520>
> .
>


This test is a little awkward because it depends on data for an entity with a particular ID. I'll fix this by changing the formula to "upper('a')".


Fixed


I'd as a rule blur every field after manipulating it, if running in a
js-enabled browser. Some forms have Ajax-loaded selects that don't function
without such behavior, and there are likely other quirks hiding.
On Wed, Jan 21, 2015 at 6:01 PM Jason Rosenbaum <notifications@github.com>
wrote:

> You may be right, worth trying. You'd do what in this case, click on
> another element to lose the focus?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/unitedstates/contact-congress/commit/12fe982fcdbb876ad7d5908e9a3abfc536b30b4b#commitcomment-9386996>
> .
>


Biggest pseudo-release yet. Aggregated over multiple days on local git and heroku remote.


I changed it in 771bb67


This link should probably be Course Reading list instead of 'Learning resources' right?


Yay. Ma copine était toute triste parce que le plugin ne fonctionnait plus!


Thanks!


It would be helpful to clarify the difference between these two ports some more. For example, what is a MiTM port? How about something along the lines of  "--port -- Port to use for receiving incoming traffic to be MiTM'd (via iptables redirection or other means)", "--cport -- MiTM controller port -- nogotofail clients connect here to control how their traffic is MiTM'd and to get notifications about found issues."
"?


Job pr-scala failed for a0e83673 Took 11 min. (ping @retronym) [(results)](https://scala-webapps.epfl.ch/jenkins/job/pr-scala/9564/):

To retry exactly this commit, comment `PLS REBUILD/pr-scala@a0e83673862027242ebea2d0431a9ee9c956495f` on PR 4265.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.


Which sockets are covered by "MiTM server sockets": the socket on which incoming traffic to be MiTM'd is recevied, the control socket (nogotofail clients connect there), or both?


Job scala-2.11.x-validate-publish-core failed for 190e3688, Took 4 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-publish-core/100/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-main failed for 190e3688, Took 10 min. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/92/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Hi,

Thank you for submitting this. Let me review these. It seems a fair amount of this is great. To answer some of your questions. The index page has the link to the samples. The easing graphic I need to re-create manually so this might take some time. If I use SourceTree I think I can cherry pick changes to integrate now. I will need to check. Some of your code changes I don't follow in a browser so I need to look at them in Xcode to see what the sample code is already doing.


It's a minor adjustment to the structure, but there were a lot of lines to change.


I like the fact that we're aiming for consistency, but I'm not sure about the compatibility of this change. How can we be sure this won't break builds that are currently correctly escaping their arguments for the Ant Scala compiler?

Another thing: if we decide to make this change, we should do the same for ScalaDocOptions, which can provide additional parameters to the Ant ScalaDoc task. Perhaps extracting the "EscapeAntParameterTransformer" into a separate class for use in both places.


Job scala-2.11.x-validate-test failed for 190e3688, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/22/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-test failed for 190e3688, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/23/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-test failed for 190e3688, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/24/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1952](http://teamcity.monogame.net/viewLog.html?buildId=5038&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:16:33


nice quick work mate!


@chrisdickinson @bnoordhuis It looks like this change was backed out. Am I missing anything?


Ok, I've completed the text analysis project (up to iteration v1.2) and would love to hear any overall feedback. Some specific questions:

-What is the common view on explaining methods through comments in the Ruby community? I added one line comments above each method though I wasn't sure of the best format.

-Is the amount of method decomposition right?

Thanks!


Job scala-2.11.x-validate-test failed for f725327a, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/25/):
If you suspect the failure was spurious, comment `PLS REBUILD f725327a3fdc945c0eee382bfad88ccd16b3d092` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-main failed for f725327a, Took 21 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/94/):
If you suspect the failure was spurious, comment `PLS REBUILD f725327a3fdc945c0eee382bfad88ccd16b3d092` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


It was reverted in 913addb, rationale is in the commit log.


very good idea ! thanks


`this._super.apply(this, arguments);`


@openspectrum Is "expressive" synonymous with "readable"? I see how `list.each` is easier to read than the `while` loop.


although maybe i wasn't clear, i believe my advice was for the ApplicationInstance to be initiated via the container, not to do the actually container instantiation.


I don't want to nitpick, but can we simplify this `if` slightly? I don't like `continue` but in some cases it can make the code easier to understand at a glance.

    // Suppress connection until timeout
    if (iter->second.when() > now)
        continue;
    auto const& address (iter->first.address ());
    if (squelches.find (address) != squelches.end())
        continue;
    ...


Nice, thanks!


Lol.


Vive les Yoda conditions =) !


извращенец


Is apr-util also unreliable?


If you want to submit a simplification as a FOLD, I'll add it


Please do make such changes as a PR next time...


Vive les yodas conditions =) !


Fuck trinityCore and there new ways! hate it


r+


lgtm


Have you tried parse a string WITHOUT time zone? E.g. "2011-03-13T02:23:18"?


Buh


It looks like here and in some other cases the problem is between brackets and git, because on brackets i see this and other lines well indented, while git does not. I'll check again using leafpad.


Not expecting the exception in the test, you get this more helpful stacktrace:
  jazz.py:432 in run_one
    spec(test)
  ioc_test.py:212 in it_should_raise_when_attrs_are_accessed
    InjectedFunc()
  ioc_test.py:210 in InjectedFunc
    return val.Foo()
  ioc.py:29 in __getattribute__
    raise Exception('You forgot to mark something with @Inject')


So, tried a click event on some unrelated element, to see if that helps. It did not. See https://github.com/unitedstates/contact-congress/commit/9d0539d5bee6295e66b6ddb823d20780650a6452

See the original issue here:

https://github.com/unitedstates/contact-congress/issues/1322

These forms are essentially identical. They're the same CMS (https://emmer.house.gov/contact vs https://mcsally.house.gov/contact), on a slightly different skin, but one works without double click and one needs it. They even load essentially the same files and js (though not identical). I really can't see the difference as to why one works and the other doesn't. Tried waits, tried finds, they don't help. But more ideas if you have em, let's try them for sure!


Awesome. :) :+1:


I may try this with webkit too, to see if that helps.


You might want to suggest how to use a different value for debug/release:

If you want a different key for debug and release:
1. go to `yourapp/src`
2. create `yourapp/src/debug/res/api_keys.xml`
3. create `yourapp/src/release/res/api_keys.xml`
4. place a new string entry in both files:

````xml
<resources>
    <!-- Other existing resources -->

    <!-- Add this string resource below, and change "your app key" to your app key -->
    <string name="bnc_app_key">"your app key"</string>
</resources>
````


83442720-ERVZoneMult-Issue4338 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1591 of 1592 tests passed)

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


Done.


Is this a typo btw? RE**E**FERAL_BUCKET_DEFAULT


Looks Good!


有个问题，对于MSG_TYPE是event来说是没有msgid的，event的msg微信也会重发，这个并不能去重。
比如通过点击按钮回报一个当前时间，仍然会收到多条消息。可以考虑用createtime和msg_type.加上用户在取消关注后会有unsubcribe事件，会发多个event，msg内容除了agentId都一样，最好还是带上agnetID。希望能够考虑下，谢谢罗。


Enforce unique course code (**case insensitive**, as the repository names are care insensitive).


Opinions on this? How far into the future is ok? (Time zone considerations especially) @AdrianLxM  ?


No, it's not necessary to handle the interrupted reads here and this case is extremely rare. The signals will get processed eventually elsewhere.

What were you doing to "rapidly resize" your window anyway?


@qiuzi
 1. 我不觉得全部写在一行更好，可读性和可维护性都不高
 2. 这只是一个示例文件，目的是让别人知道怎么配置


@vikramvgarg line 87 should be `[../SpalartAllmaras]` not `[./SpalartAllmaras]`, i.e. `../`. I'm surprised you weren't getting errors for rho, etc.


So, I've started diving into this in full. I'm not sure your prior test ever actually tested anything, thats why it passed. Because using a generator doesn't cause it to actually *run*, it returned immediately and the test appeared to pass. Using a list comprehension makes it actually work, and as a result the test properly fails (as it should've the entire time).

So this test never actually worked, the test was always broken. That being said, clearly it'd be nice to have a way to test this.


Was switching those environments to opcache not an option?


@Hockeyfan360 You need to be using Netbeans and commiting it directly through that, or using the git console. As you also forgot to put in the .java part to this file...


Nice spacing


Good catch


If you want this merged into ARTIQ, please integrate with the documentation in /doc instead of deploying another sphinx infrastructure. Also, all ARTIQ is Python3 - no need to specify it again in the folder names.


@mcnewton yes, you're right.

I'll update the suse script based on the RHEL one I just did. I was sort of toying with the idea of using status, and it if reported the process as running, and a control socket was configured, that instead of failing to start the server, we use raddebug to get log output from the server. What do you think?


rax retry tests


test


Just say ARTIQ.


Ummm I also [do it](https://github.com/montanaflynn/human-date) and I've seen it done by others as well


Use "==" instead of "is"


Okay - I prefer (1), though I haven’t thought hard about how to do it.

My suggestion is that since this isn’t blocking Arun’s progress, he can proceed with what he’s working on, and in the meantime I can handle this separately in the next ~week as we finalize work on this branch.=


ขอบคุณฮะ


not self.simulate_hw


Are you sure this is the place where the error is thrown?

What do you understand from the error?
From what I understand, you are using a function that initializes an object that requires 2 arguments instead of one - please have a look at http://stackoverflow.com/questions/12947599/typeerror-init-takes-at-least-2-arguments-1-given-error

From what is on this line, I don't understand where that is located. Please post the exact stack trace that you get


There is already the git commit log


hi mate.after implemented theme engine  im having surious issue with this code. if carrier label is off . having  NPE system ui fc after apply new theme and label appears on status bar again even switcher off . is that same on you ?


I forced the moto daemon to fire up, and sure enough, its a legit error. These tests were always completely broken when it used a generator because the generator caused it to not actually make the AWS calls. Currently looking into ripping out all these subprocess daemons and just mocking the heck out of it all.


Furthermore, you can find the signature of the setUp() menthod in the documentation https://docs.python.org/2/library/unittest.html#unittest.TestCase.setUp


Use underscores for method and parameter names instead of CamelCase: def debug_message(self, func_name, ...


Oh and BTW I am inclined to make the WFIRST WCS stuff work directly with a pyfits.Header() object instead of making a dict first, which also circumvents this problem.


The tag `rel_cream` is not yet defined on this branch, so use `rel_creedence` here. Would update it once it's ready.


Thanks ! But why the two dots for the SpalartAllmaras ? Is it considered a
sub-physics of INS ?

On Wed, Jan 21, 2015 at 7:35 PM, Paul T. Bauman <notifications@github.com>
wrote:

> @vikramvgarg <https://github.com/vikramvgarg> line 87 should be
> [../SpalartAllmaras] not [./SpalartAllmaras], i.e. ../. I'm surprised you
> weren't getting errors for rho, etc.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vikramvgarg/grins/commit/5587bbabb9b8a662f4f7ecde36f8d97c4df1113f#commitcomment-9389197>
> .
>

--
Vikram Garg
Postdoctoral Associate
Predictive Engineering and Computational Science (PECOS)
The University of Texas at Austin
http://web.mit.edu/vikramvg/www/

http://www.runforindia.org/runners/vikramg


I did test the clicking with build 75 and I thought it already opened the help group by clicking?... the problem is that many players don't know how to get their mouse to click the chat area.

I Agree /help <whatever> IS clunkier... I just dont know how to avoid the problems with people that dont know how to click the chat.

As far as wrapper commands.. I have observed that they only work in the console, not in the client (unless there is some permission I did not add myself to?) typing help wrapper and wrapper help (even with // did not help - "command not recognized"


If we really need such a function (I don't think so), it should be in artiq.tools and shared - not in some driver.


such plugin, many work, very useful. wow


플랫폼이 마음에 드시나봐요. ^^


Job scala-2.11.x-validate-test failed for 190e3688, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/19/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-publish-core failed for 190e3688, Took 4 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-publish-core/100/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-test failed for f725327a, Took 1 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-test/25/):
If you suspect the failure was spurious, comment `PLS REBUILD f725327a3fdc945c0eee382bfad88ccd16b3d092` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-publish-core failed for 190e3688, Took 5 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-publish-core/105/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-main failed for 190e3688, Took 10 min. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/97/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Job scala-2.11.x-validate-main failed for f725327a, Took 21 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/94/):
If you suspect the failure was spurious, comment `PLS REBUILD f725327a3fdc945c0eee382bfad88ccd16b3d092` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


I am not sure if setting a unique controller for each test is a good idea, what do you think? you are always using the default controller. I think the way it was before was correct, please check. You don't necessarily need a setUp() method - if you want one, you could just look at https://docs.python.org/2/library/unittest.html#basic-example "**When** a setUp() method is defined, the test runner will run that method prior to each test." - that means that you don't necessary need it


I followed what you did in [ReadAligner][1] (this line is not there). It was present in [LabelHash][2] and [Hashbits][3], though. (And [CountingHash][4] uses another way...)

[1]: https://github.com/ged-lab/khmer/pull/727/files#diff-bef19f7d0acfa8e850e83c3d9c50e2e4R4019
[2]: https://github.com/ged-lab/khmer/blob/master/khmer/_khmermodule.cc#L3578
[3]: https://github.com/ged-lab/khmer/blob/master/khmer/_khmermodule.cc#L4186
[4]: https://github.com/ged-lab/khmer/blob/master/khmer/_khmermodule.cc#L4173


Job scala-2.11.x-validate-main failed for 190e3688, Took 10 min. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/92/):
If you suspect the failure was spurious, comment `PLS REBUILD 190e3688a633594b18a0c2e15abeedd77690dd89` on PR 19 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


At least on my machine, Python has problems importing files that have "-" in their name, which in turn causes problems with sphinx. I suggest renaming to novatech409b_controller etc.


asfsadsadfsa


dsadsa


@SuperstarGamer I thin you probably should use a CommandManager or something similar to manage commands.


sadad


if is_continuous:


sad


about GALA


Link for this is wrong.


missing colon separator


Have you tested it? It seems that I have got opposite result...
```
>>> import re
>>> login_str = "Last Login: "
>>> regex1 = r"(?<![Ll]ast).*[Ll]ogin:\s*$"
>>> regex2 = r"(?<![Ll]ast )[Ll]ogin:\s*$"
>>> print re.match(regex1, login_str)
<_sre.SRE_Match object at 0xa03cc8>
>>> print re.match(regex2, login_str)
None
```


just fyi

```
@find({key}) _.first
```


When you get an error, the steps to debug are:
- Look at the stack trace, isolate the problem - this you have done very well: line 67 in the default controller doesn't find a dependency.
- Read the documentation of the API / dependencies you are using: in this case, a google search after "auth.requires_login web2py" will lead you to http://web2py.com/books/default/chapter/29/09/access-control; this says, in the "Authentication" section, "from gluon.tools import Auth" => you should go to the controller and add the dependency there, most probably
- Search for a solution on the internet, preferably on stackoverflow (they are quite good on specific software development questions): I will let you try this on your own.


@keradus hopefully it should be a bit more readable now. No evil continue nor overkill closures.


I would just make this work with any two letter country code so we don't have to update the code as soon as a new country is added. Then a config change would do.

We could also consider adding some config for country names, either with a `[country] NO = Norway` config section, or with something like `toplist_countries = NO:Norway,DE:Germany` etc.


where validation is implemented for $reguest


+1 as this paves the way for artist and album toplists :-)


Yay, ignore my question on IRC as you already fixed things :-)


breach a rule of naming methods


this returns status "success" even if `result` (line 185) is False.  It must only return "success" if `result` is True.


This is due the default white color thats causing the fc issue when disable.. set other color then disable!
when im home from work in a few hours. i will push a fix for this issue.


This won't work "equally well".

The node.js require is synchronous, the browser can only do async.

```clojure
(load-file "some.js")
(js/some.thing) ;; via some.js
```

This will work in node, but not in the browser since it will only be available some time after the file finished running. Not sure why you are bringing this complexity into cljs.core but we already established a difference of opinion on this subject.


TBD


qadad


This could also just jump straight to the split along the lines of what I've been doing in other backends.


No. The line by line comparison of this commit is misleading. The scribeLayer is isolated by `findPlotAxes` which is the first thing being called.


@seletskiy thx


s/type/variant/ to avoid shadowing a builtin?


No need for the initialization to an empty string. This is the default behaviour anyway.


@greg80303, if you are 100% confident that this method will not be used elsewhere, let's keep it here. Otherwise I would move it into utils, just in case.


@see postExecuteRequest


:+1: Thanks, will amend.


parse the text file in configure, make it the one authority?


Would it be beneficial here to have specializations for boolean/int/long/double ?
How would you write this node using the new DSL?


Identation (a few more further down)


s/type/variant/ again


What's going on with this change?  It doesn't look related.


Same questions as for the track version, should we avoid caching `None` values?


So nice!

Next time, I do a PR ;)

Cheers,
Thomas.


"content" or "context" ?


Thank you for pull request. Agree, but ewerywhere it should be changed to e.g. ```req.header && req.header[this._headerField] != null```  because 0 can be valid value.


afú, Maribel, qué más da... se puede cambiar, claro, pero hay que cambiar
la configuración de los repos ya descargados.

El 22 de enero de 2015, 10:06, Maribel García Arenas <
notifications@github.com> escribió:

> Y tengo otra pregunta. Por qué lo habéis llamado noisy-gecco y no
> 2015-Gecco-Noisy como hicimos con Evo* para que luego aparezcan
> ordenados que sino esto empieza a ser poco tratable.
>
> J, se puede cambiar el nombre?
>
>
> Maribel
>
> On 22/01/15 01:19, Antonio Mora wrote:
> > ¿¿¿¿¿Por qué leches habláis en inglés entre vosotros?!!?!?!?!?!?!
> >
> >
> >
> > Bueno, no estaba al tanto de que esto se iba a enviar, en cuyo caso
> > hubiese aportado algo…
> >
> > Participaré (mucho) más en la versión completa, claro. ;)
> >
> >
> >
> > De: Juan Julián Merelo Guervós [mailto:notifications@github.com]
> > Enviado el: miércoles, 21 de enero de 2015 19:36
> > Para: geneura-papers/noisy-gecco-2015
> > CC: Antonio Mora
> > Asunto: Re: [noisy-gecco-2015] First version of the abstract written
> > (9465635)
> >
> >
> >
> > just go and change it.
> >
> > 2015-01-21 19:34 GMT+01:00 Pablo García Sánchez
> > <notifications@github.com>:
> >
> > > The three problems should use the same verbal tense: "the evolution of
> > > bots for A, evolution of bots for B and finally, the design of
> > backstories
> > > for C" @jj <https://github.com/jj> @amorag <https://github.com/amorag>
> > > @deantares <https://github.com/deantares>
> > >
> > > Also, "operartors or eas" or "operators for eas" at the end?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> > <
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9381691>
>
> >
> > > .
> > >
> >
> >
> >
> > --
> > JJ
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9381709>
>
> > .
> > <
> https://github.com/notifications/beacon/AC6-WmPE2QIPiOp8I0bDzFEAvVcCu1s5ks5nj-j9gaJpZM4DVYbv.gif>
>
> >
> >
> > No se encontraron virus en este mensaje.
> > Comprobado por AVG - www.avg.com
> > Versión: 2013.0.3495 / Base de datos de virus: 4257/8970 - Fecha de
> > publicación: 01/21/15
> >
> > _____
> >
> > No se encontraron virus en este mensaje.
> > Comprobado por AVG - www.avg.com
> > Versión: 2013.0.3495 / Base de datos de virus: 4257/8953 - Fecha de
> > publicación: 01/18/15
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9388462
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9392864>
> .
>

--
JJ


Re-add this line - and set text to "default-facility"


ok ok fine!


We could also do a `variant, rest = uri.split(':', 3)[1:]` and then `self._browse_album(rest)` - but doesn't really make a big difference how this gets done.


Add a comment "Message is logged with facility = example-facility"


Hmmm..... I still need this tool to change between internal speakers/ headphones / HDMI... Can I use another tool to change default audio output?


Though the example in this case is a bit wrong :-)


Should this tuple be a constant?


Ich mag BS, allerdings ist dann der Lerneffekt bezüglich CSS und HTML nicht so groß.
Aber meinetwegen gerne....


need to specifically check for paypal-rest-future. Otherwise "any value" will select future payment which is wrong. If an invalid method is specified, it must return 400 BadRequest.

actually, this shall use use the `shop.get_payment_module` (see implementation notes in #6)


So, just need the `if pageHandler.useLocalStorage()` block uncommenting? Or is there something else?


Hmm... I would try to figure out how to use t.errors...

Now, running the tests is a very noisy experience.


Same question as for the track one, can / should this be in parallel.


この辺を変えている理由ってなんだろうか？

あんまり重要ではないけど、気になるので教えて下さい！


good job man
use http://validator.w3.org to check your html, analyze errors and explain to me what you've done wrong
keep going!


Memoized `None` value?


Even if the query works fine, I guess it is better to standardize the "FALSe" and change it to FALSE.


Can you put this code instead in communicate(), in the "if self.type == Socket.TYPE_ACL_SUB:" block?

I think that's more the right thing: in any circumstance where we might decide to recreate the ACL_SUB socket, we want to resubscribe for all existing endpoints.


Two comments for your consideration.


Thanks.


Same as the other ones again.


Looks very good!


Can you split this out over 2 lines, it makes it slightly easier to read? Something like:

```javascript
var isEditorOrManager = _.some(roles, function(role) {
    return _.contains([ContentConstants.roles.EDITOR, ContentConstants.roles.MANAGER], role);
});
return callback(null, isEditorOrManager);
```


might be more efficient to check if n > 0


@FernetMenta is that OK? I needed to populate the flags correctly for single pass deinterlacing which was not done before. That path is choosen if deinterlacing is done with 25fps instead of 50fps (VS_INTERLACEMETHOD_IMX_FASTMOTION vs. VS_INTERLACEMETHOD_IMX_FASTMOTION_DOUBLE).


この間、
if ~ then return ~ else return ~
の書き方は、
return if ~ then ~ else ~
にしてなかったっけ？

まぁ、それはどちらでも良いんだけど、ここの書き方は良くないと思う。
nullが、coffeescriptの、「最後に評価した値を返す」という機能でreturnされてるはず。


Unrelated to this PR, as I just reused what was there.


А как можно быстро сделать "upgrade"?


还是让 @Fsu0413 处理一下吧0.0 这样会在仓库里留下Initial Release里的音频图片什么的


Should this be a warning? Or are you trying to avoid red herrings?


Anywhere we break out of a/pass control to another function we tend to stick `return` in


This is moot. Request already does that: https://github.com/request/request/blob/master/lib/getProxyFromURI.js


The `scribeLayer` in HG1 is an axis of the type being built below. HG2 does not have an axis we could feed to `drawAxes` thus we create an emulation of a `scribeLayer`. I do it also for HG1 firstly because the original one has children, and `drawAxes` will start traveling down. However, what we want is to decouple annotations from `drawAxes` (since I am forced to do so for HG2).


I didn't test the regex, but thought the regex modified 'x' makes the `\s` unnecessary because white spaces are ignored (http://php.net/manual/en/reference.pcre.pattern.modifiers.php).


Fixed, thanks.


Wrap all code in the file under "main" and include the ending if-statement.


hi vivek previously practnlptools gave the correct role labels for the given input. but now when i try it its not gives correct labels. for A0 its giving A1. I cross checked with senna in c. its giving the correct one.


OK, I see.


OK, I thought it was something like that.  Personally your first suggestion (mqSmall => mqAboveSmall and mqMedium => mqAboveMedium) seems good to me.


前回、直接index.htmlを修正していたため、今回template.htmlを追従させた部分です。


I'm using `got` here.


No. PEP8 specifies that there are to be two newlines before a class definition. Just think of it as if I'd have added line 15 instead of 17 which the diff cant know.


日本語版の方も同じように修正しています。
https://github.com/nttcom/skyway-docs/commit/897d62211cc4eb7961bd41cf0d7c0b76292a57e3


Still, I don't think it's that simple. Got doesn't support proxies. I don't think you can just replace the `host`/`port`.


This entire function is completely wrong... :-1:
It violates the "DRY" programming principle. http://en.wikipedia.org/wiki/Don%27t_repeat_yourself

Please make better use of the Sencha API.
```JavaScript
renderer: function(value) {
     return Ext.util.Format.fileSize(value);
},
```


if you want to do single pass deint, you can do this in renderer without knowledge of render manager. render manager should not bother with any implementation specific methods.


note that this is probably incorrect, since now we just don't use `stride` and `blitz::neverdeleteData`. @Error323 may have a solution for this, will have a look today or tomorrow.


opened #173 , out of scope for this commit. Thanks for mentioning, I've thought about that too, it was long overdue.


see #173


I would add an `if (!plugin.path)` so people can still manually specify the dir name if needed.


I cannot test this right now, but if this code doesn't work (https://github.com/oscarotero/Gettext/blob/master/src/Translations.php#L82)
the __clone magic method could be fixed using something like this (https://github.com/oscarotero/Gettext/blob/master/src/Translations.php#L361)


`if if else if` isn't great too, but OK for me, I think we can't get any better


I would use `wstr.resize(charsWritten); return wstr;` unconditionally here.


Ah. Is this okay then:
```
        :param control_queue: queue (write). If any result gets written to the result_dict a tuple containing a
        CONTROL_ELEMENT (to indicate what kind of event happened) and either a filter name (for global results) or a
        file name to indicate the result will be put to the queue. If this BearRunner finished all its tasks it will
        put (CONTROL_ELEMENT.FINISHED, None) to the queue.
```

I'd be happy to accept any simpler proposal that expresses the same content.


It worked in #3.


Hombre pues si tu lo dices da igual, pero el desorden que se va creando
no da igual porque va en aumento.

Puedo cambiarles el nombre y podéis volver a clonarlos que no se tarda nada.

Pero dime que si o que no.

Maribel
On 22/01/15 11:09, Juan Julián Merelo Guervós wrote:
> afú, Maribel, qué más da... se puede cambiar, claro, pero hay que cambiar
> la configuración de los repos ya descargados.
>
> El 22 de enero de 2015, 10:06, Maribel García Arenas <
> notifications@github.com> escribió:
>
> > Y tengo otra pregunta. Por qué lo habéis llamado noisy-gecco y no
> > 2015-Gecco-Noisy como hicimos con Evo* para que luego aparezcan
> > ordenados que sino esto empieza a ser poco tratable.
> >
> > J, se puede cambiar el nombre?
> >
> >
> > Maribel
> >
> > On 22/01/15 01:19, Antonio Mora wrote:
> > > ¿¿¿¿¿Por qué leches habláis en inglés entre vosotros?!!?!?!?!?!?!
> > >
> > >
> > >
> > > Bueno, no estaba al tanto de que esto se iba a enviar, en cuyo caso
> > > hubiese aportado algo…
> > >
> > > Participaré (mucho) más en la versión completa, claro. ;)
> > >
> > >
> > >
> > > De: Juan Julián Merelo Guervós [mailto:notifications@github.com]
> > > Enviado el: miércoles, 21 de enero de 2015 19:36
> > > Para: geneura-papers/noisy-gecco-2015
> > > CC: Antonio Mora
> > > Asunto: Re: [noisy-gecco-2015] First version of the abstract written
> > > (9465635)
> > >
> > >
> > >
> > > just go and change it.
> > >
> > > 2015-01-21 19:34 GMT+01:00 Pablo García Sánchez
> > > <notifications@github.com>:
> > >
> > > > The three problems should use the same verbal tense: "the
> evolution of
> > > > bots for A, evolution of bots for B and finally, the design of
> > > backstories
> > > > for C" @jj <https://github.com/jj> @amorag
> <https://github.com/amorag>
> > > > @deantares <https://github.com/deantares>
> > > >
> > > > Also, "operartors or eas" or "operators for eas" at the end?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > >
> > > <
> >
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9381691>
> >
> > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > JJ
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9381709>
> >
> > > .
> > > <
> >
> https://github.com/notifications/beacon/AC6-WmPE2QIPiOp8I0bDzFEAvVcCu1s5ks5nj-j9gaJpZM4DVYbv.gif>
> >
> > >
> > >
> > > No se encontraron virus en este mensaje.
> > > Comprobado por AVG - www.avg.com
> > > Versión: 2013.0.3495 / Base de datos de virus: 4257/8970 - Fecha de
> > > publicación: 01/21/15
> > >
> > > _____
> > >
> > > No se encontraron virus en este mensaje.
> > > Comprobado por AVG - www.avg.com
> > > Versión: 2013.0.3495 / Base de datos de virus: 4257/8953 - Fecha de
> > > publicación: 01/18/15
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9388462
> > >.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9392864>
> > .
> >
>
>
>
> --
> JJ
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geneura-papers/noisy-gecco-2015/commit/946563504153291e7e4cc1ff9fc2c0a8e68fc234#commitcomment-9393760>.
>


I'm sorry, I thought that this was related to the JSONParseError, UnauthorizedError, UnknownError. But it is ofcourse compareable to the to Result.Error value.


Here has no need for additional div for article body.


TEST


Line 69 I am told is not typecasting correctly. This could be a cause of JS encoding errors. Will try recompiling with pedantic flag on gcc. h/t kratsg


`&wstr[0]` => `wstr.data()`


indention ...


yes, sure in this case I would suggest just log, because it's only the request object


How to figure out the frame field layout (top vs. bottom) in renderer? The field flag from DVDPicture is not forwarded to the renderer so I did not know how otherwise to access that information. I don't see that as specific knowledge, just forwarding known information.


Yes, I would suggest just log for same reason as the request! :)


Hi - I've been running these this morning, wouldn't it be more realistic if this example echo'ed the timestamp as well?


A Simple Khmer Clinic App by noonnightcoder team.


changing from POS to Clinic repository


Yeah, this is ![magic](http://www.reactiongifs.com/r/mgc.gif) xD


Yeah, I could check the length here. Will do.


Review this comment and the parentheses


This is currently just testing an work in progress. - its still on a snapshot and has not been released.

> On 22 Jan 2015, at 10:29, thompsri <notifications@github.com> wrote:
>
> Hi - I've been running these this morning, wouldn't it be more realistic if this example echo'ed the timestamp as well?


My suggestion was not to remove the whole section, just the leaflet code


@sguha00 take a look at this issue https://github.com/Ink/filepicker-rails/issues/112. I think if we will expose the `Policy` we need to follow this https://github.com/Ink/filepicker-rails/issues/112#issuecomment-56082262


:+1:  :+1:


These changes brake my project since facebook sdk are loaded async (FB is not defined is my Error).


Once https://github.com/caskroom/homebrew-cask/pull/9154 is merged, we can restart the failed builds of travis-ci and they should be fixed.


Ok, just wanted to make sure...


a comment


Mark thanks for the quick reply.I will check out those links on DII have read a couple of those books on the list ( you might have forgotten GOF and Fowler's books)
But I would really like to see you express how you handle Async/await and generics and parallelism  (or not ) in DI
Don't be shy Mark. Just DO IT!! 😊

Regards

Greg Frazer
0414 352 865

Date: Wed, 21 Jan 2015 23:53:52 -0800
From: notifications@github.com
To: ploeh.github.com@noreply.github.com
CC: gregfrazer@hotmail.com
Subject: Re: [ploeh.github.com] Answerer reader questions. (aefb632)

Greg, thank you for writing, and thank you for your interest in a second edition of DI in .NET.

It's true that technology moves forward, but I also did my best to write the book in such a way that the main points can stand the test of time. Some of my favourite programming books were about 10 years old (or more) when I first read them. While I don't presume that my book is comparable to those masterpieces, I hope it'll still be able to stand the test of time fairly well. Only my readers can tell.

There are certainly things I'd like to change in the book, but the question is whether it'd be worth the effort... I'm not sure about that.

Meanwhile, there's this blog :) Specifically related to ASP.NET Web API, I've already described how to approach it with Pure DI as well as with a DI Container.

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


These days we're having more failures due to network/version crap than anything else...


For the latency tests we are only dealing with one packet at a time so we don't need to include the timestamp for the test.  We could add it to the test with some random data but then you are not testing just the transfer rate any more.  The latency packets are 64 bytes (a somewhat arbitrary but realistic size) and the size of this matters more.


actually, `wstring` also maintains a `NULL`-terminated string inside, which means that you shouldn't need to allocate an additional char for a terminating `NULL` char at the end... but it also doesn't hurt that much (only in case you cannot use the small string optimization due to this additional char...)


rename it to `lazyRelationshipProxyById`?


TeamCity Ylab :: Feodorov/ylab [Build 62](http://192.168.30.225:8111/viewLog.html?buildId=611&buildTypeId=Ylab_FeodorovYlab&guest=1) is now running


TeamCity Ylab :: zmactep/ylab [Build 173](http://192.168.30.225:8111/viewLog.html?buildId=610&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


the digit passed by James is 3 digit less..some unix eporch thing...so
right way to convert it is div by 1000 then floor..or else give me the year
1970..

Lingpo Huang
Univesity of Central Oklahoma
Department Computer Science
Class of 2013| ling@lingpohuang.com

2015-01-21 17:17 GMT-06:00 smurphy8 <notifications@github.com>:

> What is the floor here about?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plow-technologies/cobalt-kiosk-data-template/commit/c3b46f0ba3765c90d6808e2861dd3156ed31ea3f#commitcomment-9387213>
> .
>


I Love : travelling:.


LGTM


サンプルとするデータを間違えただけです。(間違って、fixturesにあるデータじゃなくて、今公式サイトにあるデータを使ってました。)


I love :travelling:.


すみません。ミスですね。修正します。


aaaa


OK, done in https://github.com/AdrianVasiliu/deliteful/commit/2750e1c97b3c409c9a4f1b4bdb67cb8909938dc2.
If no other feedback, I'll merge the PR.


Revert this patch and add a `doc.txt` in `cpu/kinetis_common` instead.


This should an exceptional response (as any other ERROR)


```js
var err = {
```
?


Como veas, Maribel.


How does this look @alexanderdean ?


<> is not compatible with Java 6.


Thats great . thanks


It's not really call-by-reference, because even though you're passing a pointer it will still copy the pointer itself. (But not the value the pointer points to.)


What is there to say about `async`/`await`? Those keywords are only syntactic sugar for methods that return `Task` or `Task<T>`.

You can define interface methods that return `Task` or `Task<T>`. At that point, such an interface is like any other interface, and can be injected and consumed using the techniques described in my book.


One reason is that we have so many targets, leading to so many test dependencies...


Actually I also need to widen some columns. Is it like this: ALTER COLUMN user_ipaddress VARCHAR(45) ?


This looks good! We should add some encodings too... And a warning that this approach doesn't do things like widening the IP field...


Invalid Description


Asymmetry #1


This is useless as Kevoree-JS is already reporting **start** & **stop** actions within the **DEBUG** logs.


(24f723b) The title of the commit have a non authorized type.
 The allowed types are :
 * feat
 * fix
 * docs
 * style
 * refactor
 * test
 * chore
 * version


(4bf09a3) The title of the commit have a non authorized type.
 The allowed types are :
 * feat
 * fix
 * docs
 * style
 * refactor
 * test
 * chore
 * version


Naming conventions are awesome :+1:


@arr2036 it's a nice idea technically, but I'm not sure whether changing behaviour depending on whether FR is running is good from a user interface point of view. In fact, I started to wonder whether the "debug" option to an init script was a good idea at all, but I guess it can help (the beginner especially) if they aren't sure the right way to debug the server.

Maybe "debug" should check the server status first, then if it is not running, execute radiusd -X, otherwise if the server is running output an error something like "radiusd already running; cannot start in debug mode. See raddebug(8)."?


Wrong Permission


Renaming the ports could be great too.


You don't really know that. All you know is that nobody complained. You also don't know whether this works in all proxy scenarios. I'm just saying. This looks too easy. There's a reason Request has the code it has.


dj_level_target, dj_level_diffにしましょう。dj_level, dj_level_targetも数値で管理してしまって大丈夫ですよね？


Asymmetry. Why use the word "factions" in one location but not the other two locations?


@saily: FYI: ActionsDXTestCase is a functional testcase. Using the integration testing layer (together with transaction.commit()) leads to serious test isolation problems that are hard to track down. I will push a fix as soon as Jenkins is green again.


there is an error in line 411, you are missing a ! in front of HORIZONTAL_SCROLLING


Add "map" to the Parameter list. So the user can give the id of the Map container via Constructor. This makes it possible to add multiple instances on one Page.


Instead of the String "map" use the String of the parameter List.


Yeah, I might try this locally and see what happens.


If the connect returns a SecurityException that means that it cannot bind the service (neither now nor in the future). For this case the reconnect logic should end here.

And for the NoMultiplexerAvailableException there are two reasons:

* No package is present (then we stop here and restart reconnect when a new package is installed in the system)
* The bind operation failed (then this reconnect logic should be ok)


TeamCity Ylab :: zmactep/ylab [Build 173](http://192.168.30.225:8111/viewLog.html?buildId=610&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 126 Build time: 00:12:18
### Failed tests
```
ru.biocad.run.ytecan.TecanJobTest.TecanJob should work with data and csv: class org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1
D;Plate1;;;5;;100;;;1
W
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
W
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
W
A;src;;;4;;100;;;8
D;Plate1;;;8;;100;;;8
A;src;;;4;;100;;;8
D;Plate1;;;8;;10]0;;;8
W
" did not equal "A;src;;;1;;[30;;;1
D;Plate1;;;1;;30;;;1
W
A;src;;;2;;40;;;2
D;Plate1;;;9;;40;;;2
W
A;src;;;3;;50;;;4
D;Plate1;;;17;;50;;;4
W
A;src;;;4;;60;;;8
D;Plate1;;;25;;6]0;;;8
W
"
org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1

ru.biocad.run.ytecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in degenerate cases: class java.lang.ArithmeticException: / by zero
java.lang.ArithmeticException: / by zero
      at ru.biocad.run.ytecan.TecanMappingGenerator$.create(TecanMappingGenerator.scala:97)
      at ru.biocad.run.ytecan.TecanMappingGeneratorTest$$anonfun$1.apply$mcV$sp(TecanMappingGeneratorTest.scala:39)

```


TeamCity Ylab :: zmactep/ylab [Build 174](http://192.168.30.225:8111/viewLog.html?buildId=612&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


TeamCity Ylab :: Feodorov/ylab [Build 62](http://192.168.30.225:8111/viewLog.html?buildId=611&buildTypeId=Ylab_FeodorovYlab&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 126 Build time: 00:12:24
### Failed tests
```
ru.biocad.run.ytecan.TecanJobTest.TecanJob should work with data and csv: class org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1
D;Plate1;;;5;;100;;;1
W
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
W
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
W
A;src;;;4;;100;;;8
D;Plate1;;;8;;100;;;8
A;src;;;4;;100;;;8
D;Plate1;;;8;;10]0;;;8
W
" did not equal "A;src;;;1;;[30;;;1
D;Plate1;;;1;;30;;;1
W
A;src;;;2;;40;;;2
D;Plate1;;;9;;40;;;2
W
A;src;;;3;;50;;;4
D;Plate1;;;17;;50;;;4
W
A;src;;;4;;60;;;8
D;Plate1;;;25;;6]0;;;8
W
"
org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1

ru.biocad.run.ytecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in degenerate cases: class java.lang.ArithmeticException: / by zero
java.lang.ArithmeticException: / by zero
      at ru.biocad.run.ytecan.TecanMappingGenerator$.create(TecanMappingGenerator.scala:97)
      at ru.biocad.run.ytecan.TecanMappingGeneratorTest$$anonfun$1.apply$mcV$sp(TecanMappingGeneratorTest.scala:39)

```


TeamCity Ylab :: Feodorov/ylab [Build 63](http://192.168.30.225:8111/viewLog.html?buildId=613&buildTypeId=Ylab_FeodorovYlab&guest=1) is now running


Wow, this is actually a really good idea. This also means that we don't have to have nearly the level of bloat in the load/store routines; can't we drop QUANTIZED_REGS_TO_SAVE entirely?


1277852-CommandLineInterface (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


ok


I tested with my midlet. There are two ways to set the title of a `Displayable`:
1. by calling `Displayable.setTitle`, which calls `DisplayableLFImpl.setTitle0` ultimately
2. by the constructor of `Displayable`, which never calls  `DisplayableLFImpl.setTitle0`

So at times the title set with second way doesn't show correctly and I made this work around to fix it.


Thank you! :)


`messages.add(mflag.getStateDesc(faction.getFlag(mflag), true, true, true, true, false));`
VS
`messages.add(mflag.getStateDesc(faction.getFlag(mflag), true, true, true, false, true));`

When showing maybe we should use "specificDesc" instead of "desc"?
The non specific "desc" can be used in the list command.
Maybe explaining what the value means is better?
Just a though.


_insertElementLater seems to have been removed in Ember 1.8 (#76). How is this still supposed to be working?


Hu? Which date format case isn't handled jxmpp's XmppDataTime.parseDate()?


I'm planning a release soon with the fix included... thanks for the report!


Yes we'd need specialisations for the primitives, and then we'd just cache the metaclass of the object we get and the class we're comparing against in `@Locals` and return true or false depending on a guard that does the actual insanceof check.


The cde folder should also be hidden


@mcnewton  That's exactly what I did for RHEL script, though the man entry is a nice touch :)

The debug option isn't just useful for beginners or lazy people, it's useful where there are flags which get passed to the radiusd binary via init script includes (``/etc/sysconfig/radiusd`` or ``/etc/default/freeradius``).


@aszlig @edolstra Yes this is a bit annoying. Each ```nixos-rebuild``` takes about 30 minutes on my machine due to virtualbox. It is possible to build both versions? I think hardening is an important security feature, isn't it? If so, we should make ```enableHardening``` the default for ```services.virtualboxHost.enable```. However, when installing virtualbox interactively using ```nix-env```, the user should have the ability to pick the non hardened version. Is that possible?


yup - make sense, thanks. I'm spending a lot of time in this area at the moment so always on the look out for new approaches - appreciate that this work in progress


you can carry this info in the codec specific info CDVDVideoCodecIMXBuffer of DVDVideoPicture


![small_zombie gardeb gnomes1](https://cloud.githubusercontent.com/assets/8047888/5854420/87862e66-a225-11e4-8ed7-339b275abd8e.jpg)


なるほど、納得しました。


So some Cisco implementations of xep297 use the legacy format although xep297 explicitly mentions that xep203 should be used?


After comma comes space.


I thought we weren't happy about passing the whole Document around and only did it because it was unconvenient to have another class?

In LuceneWork:

          NOTE: we are relying on Lucene's Document to play nice unfortunately


done


explicit is better than implicit plus saves one step like u said :+1:


I'd like to change the decode default to '1', thus the x/seed will be valid by default


kannst du das noch entfernen?
Also das da nur steht, dass der Ort noch bekannt gegeben wird?
Danke.


So the error where it says "Playername must leave their current faction." is still there? You just stop executing if it happens?


Javadoc needs to be updated with new parameter


Thanks!
Oh? Was it irony?


You don't need Integer.toString() here.


OK, thanks. I wanted to avoid additional payload as much as possible but I agree this would also save us the if statement in PresentSingle.


No, it would not require nms? Just implement the CommandSender. Why extend console sender? There must be other ways to create command senders with high permission levels.


時間の取得はとても嬉しいけど、どうやって統一するかはとても悩ましいですね。

UTCが一番楽かなぁ、とも思っていたのですが、どうするのが良いと思いますか？（ぼんやり）


TeamCity Ylab :: zmactep/ylab [Build 174](http://192.168.30.225:8111/viewLog.html?buildId=612&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 126 Build time: 00:12:15
### Failed tests
```
ru.biocad.run.ytecan.TecanJobTest.TecanJob should work with data and csv: class org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1
D;Plate1;;;5;;100;;;1
W
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
W
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
W
A;src;;;4;;100;;;8
D;Plate1;;;8;;100;;;8
A;src;;;4;;100;;;8
D;Plate1;;;8;;10]0;;;8
W
" did not equal "A;src;;;1;;[30;;;1
D;Plate1;;;1;;30;;;1
W
A;src;;;2;;40;;;2
D;Plate1;;;9;;40;;;2
W
A;src;;;3;;50;;;4
D;Plate1;;;17;;50;;;4
W
A;src;;;4;;60;;;8
D;Plate1;;;25;;6]0;;;8
W
"
org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1

ru.biocad.run.ytecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in degenerate cases: class java.lang.ArithmeticException: / by zero
java.lang.ArithmeticException: / by zero
      at ru.biocad.run.ytecan.TecanMappingGenerator$.create(TecanMappingGenerator.scala:97)
      at ru.biocad.run.ytecan.TecanMappingGeneratorTest$$anonfun$1.apply$mcV$sp(TecanMappingGeneratorTest.scala:39)

```


TeamCity Ylab :: Feodorov/ylab [Build 63](http://192.168.30.225:8111/viewLog.html?buildId=613&buildTypeId=Ylab_FeodorovYlab&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 126 Build time: 00:12:16
### Failed tests
```
ru.biocad.run.ytecan.TecanJobTest.TecanJob should work with data and csv: class org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1
D;Plate1;;;5;;100;;;1
W
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
A;src;;;2;;100;;;2
D;Plate1;;;6;;100;;;2
W
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
A;src;;;3;;100;;;4
D;Plate1;;;7;;100;;;4
W
A;src;;;4;;100;;;8
D;Plate1;;;8;;100;;;8
A;src;;;4;;100;;;8
D;Plate1;;;8;;10]0;;;8
W
" did not equal "A;src;;;1;;[30;;;1
D;Plate1;;;1;;30;;;1
W
A;src;;;2;;40;;;2
D;Plate1;;;9;;40;;;2
W
A;src;;;3;;50;;;4
D;Plate1;;;17;;50;;;4
W
A;src;;;4;;60;;;8
D;Plate1;;;25;;6]0;;;8
W
"
org.scalatest.exceptions.TestFailedException: "A;src;;;1;;[100;;;1
D;Plate1;;;5;;100;;;1
A;src;;;1;;100;;;1

ru.biocad.run.ytecan.TecanMappingGeneratorTest.TecanMappingGenerator should work in degenerate cases: class java.lang.ArithmeticException: / by zero
java.lang.ArithmeticException: / by zero
      at ru.biocad.run.ytecan.TecanMappingGenerator$.create(TecanMappingGenerator.scala:97)
      at ru.biocad.run.ytecan.TecanMappingGeneratorTest$$anonfun$1.apply$mcV$sp(TecanMappingGeneratorTest.scala:39)

```


はい、OKです。


Care to elaborate what this does fix?

I see that you made a few changes to Smack. Would be great if we could discuss those and eventually get them into the official version of Smack.


修正ありがとう。


7: M102 unrecognized bullet/signature


FYI this use statement doesn't reference a class so can be safely deleted.  No need to fix it, I'll take care of it during the merge.


All core devs moved here https://github.com/markdown-it/markdown-it


i want to display one  overlay text when the video is playing and another overlay text when the video is paused in the new brightcove player by following the steps that you can find in the following link ,
http://docs.brightcove.com/en/video-cloud/smart-player-api/solutions/overlay-plugin-2.html

Here is the code that i tried ,but its not working
   var myPlayer= videojs("bc_videoplayer");

                                  myPlayer.overlay({
                                    "content": "Default overlay content",
                                        overlays: [{
                                                        content: 'playing',
                                                        start: 'play',
                                                        end: 'pause',
                                                        align: 'top-left'
                                                },{
                                                         "content" : "paused",
                                                 "start" : pause,
                                                 "end" : play,
                                                         "align" : "bottom-left"
                                                  }]
                                        });
can you please help me solve this problem?


I cant find out how `CommentBox::handleCommentSubmit` preserves `this` context when runs from inside `CommentForm::handleSubmit` (line 95).
The same goes for `CommentBox::loadCommentsFromServer` from inside `setInterval` (line 58)

Maybe it's about an interesting feature of JSX that needs to be mentioned in tutorial too.


Here is no need for additional div.


TeamCity Ylab :: Feodorov/ylab [Build 64](http://192.168.30.225:8111/viewLog.html?buildId=615&buildTypeId=Ylab_FeodorovYlab&guest=1) is now running


TeamCity Ylab :: zmactep/ylab [Build 175](http://192.168.30.225:8111/viewLog.html?buildId=614&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


I agree, but we have to make sure our naming convention is robust enough so that we do not risk to generate identifiers that clashes with each others. But I can at least remove `TestTimerJS_` (which corresponds to the name of the ThingML configuration). But in ThingML I can have two things wrapped in a single Kevoree components that declare a port having the same name (probably not recommended, but still possible). I thus need a kind of qualified name.


Hi Rob,
Isn't it better to do:
    `function cleanUp(context){
        context.activeView = null;
        context.viewElements = null;
    }`
I read somewhere that deleting javascript objects is not recommended.


Thanks Pogon. Fix committed to our main repo.


:tada: Horray! :tada:


Yup i created that file to reuse in depends configure for the opposite reason :)


Отличная новость! Огромное спасибо!


readme


Looks good


Yes, the docs says so, but as you [can see here](http://3v4l.org/2d41L) that regex doesn't work. I don't know why...


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 55839 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.62 GBytes  6.54 Gbits/sec


Oops, we don't need .orig files in the repo.


:ok_hand: With https://github.com/mlocati/Gettext/commit/0a749014c2a46b02a3ddebcff1268ef787c63ae0 the cloning works! Great!


@arr2036 OK that makes sense (and would actually apply for me as well as I leave the default config alone and run with config in a different location, so always need a -d /path/to/config option), I like that idea. I guess in the newfangled way of doing things you can then just do

    service radiusd debug

as well, which I suppose is quite neat.


I agree, this will be fixed when I implement the the wrapper generator.


why not tags.WriteString(fmt.Sprintf("host=%s server=%s script=%s", *request.Hostname, blah, blah))?


@tomgross I'm not too familiar with the code. Could you elaborate why the simple xml marshaller is not needed any longer?


Hmmm, this shouldn't be required.  There's something going on here if this fixes something.


Because, unlike rumors by @dukeboard, I code (some, I have a life too)


I use 1.6.0.11 and my Translate.php file looks different than yours?

if (isset($_MODULES[$current_key]))
				$ret = stripslashes($_MODULES[$current_key]);
			elseif (isset($_MODULES[$default_key]))
				$ret = stripslashes($_MODULES[$default_key]);
			// if translation was not found in module, look for it in AdminController or Helpers

Do I need to change it anyway? Thanks for help!


imho these belong in a global gitignore as they should be applied to every project. https://help.github.com/articles/ignoring-files/#create-a-global-gitignore


done


Ah, we can probably make a static creator function on bookmodel which would serve the purpose.  That would be better than a new class that only overrides the constructor.


I'll likely merge it in as is, then adjust from there.  Thanks.


But how does it not work? I've been using this for the last couple of days on both Windows and Linux?


Why not make a regular class for this? This would avoid the dependency and makes the code a bit cleaner, IMHO.


Ehrm same reason obviously. So depends configure could generate the in-tree addon build env.


See also https://bugs.webkit.org/show_bug.cgi?id=138562


Согласен


All hail the 80s!


Yes. Anyway, there's a very low risk of the issue.
No need to fix ASAP


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1954](http://teamcity.monogame.net/viewLog.html?buildId=5044&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


lol i'm lazy. and anyway, this was a one-off so cbb...


Guess it's faster.


Agreed, as per [this thread](http://community.sitepoint.com/t/starting-a-new-php-package-the-right-way/110669/3). Will be fixed in a future Homestead Improved release and removed from this skeleton.


ts, data - declared and unused.


why new err?


```
atlantis stephane 10:20 ceylon-dist $  ant clean publish
...
 BUILD FAILED
/home/stephane/src/java-eclipse/ceylon-dist/build.xml:601: The following error occurred while executing this line:
/home/stephane/src/java-eclipse/ceylon-dist/build.xml:622: Needs 'local' or 'siblings' target to be mentioned first
Total time: 1 minute 34 seconds
```


Das ist erstmal ein test, wie sich der Editor verhält und besser klappt.
Ist noch offen was drauß wird, vl. Lösche ich es wieder.


@hbj, does `auth.service` rely on dates being in 'YYYYMMDD'? Now it'll use the config's default ('yyyy-MM-dd').


les import_contacts n'étaient pas triés sur le nom mais sur company


:+1: didn't read the comments yet.


Why we drop table??


TeamCity Ylab :: zmactep/ylab [Build 175](http://192.168.30.225:8111/viewLog.html?buildId=614&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 128 Build time: 00:12:20


TeamCity Ylab :: Feodorov/ylab [Build 64](http://192.168.30.225:8111/viewLog.html?buildId=615&buildTypeId=Ylab_FeodorovYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 128 Build time: 00:12:21


I have almost no bash knowledge, so this is all just trial and error :)


I think identifying a the object's dynamic type based on the hash is dangerous. What if we have a collision? I think it would make sense to have two options here: either the class name or a user supplied id which is unique in the type hierarchy.


if verify returns  True it this should call process_order_confirmed


I've read again the docs. The ignored spaces are applied to the pattern, not to subject. my fault :smile:


OK for the change, but... is it right that it does ``} if`` and not ``} else if``? If it's correct, the style should be fixed, as it looks very misleading.


please cleanup unused imports


Dans l'affichage des import_contacts, les noms de société n'apparaissaient pas s'ils n'étaient pas reconnus dans la base.


This solution does not work in case if we have more than 2 level of brackets.
Example:
```
CONCAT(DATE_FORMAT(DATE_ADD(`created_at`, INTERVAL -28800 SECOND), '%Y-%m-%d %H:'), '00')
```

What about following improvement ?
```
/^([\w]*\(([^\)\(]|(?1))*\))$/
```
Looks like it works for any levels.


Caching the Inverse of a Matrix


Haha


Oh, I can't believe I didn't notice that. That's a bit weird. Looks like hr is only assigned to once, so it's probably meant to be an else if. Amending.


 if ([type isEqualToString:@"result"] && iq.childCount == 0) {

This line is not correct

in Mongoose server i get this:

<iq xmlns="jabber:client" from="JID" to="JID" id="91F0F67F-3106-4213-8592-8DA4A30EFD01" type="result">
<query xmlns="jabber:iq:register">
<remove></remove>
</query>
</iq>

So the childCount is 1 not 0, I guess that blind childCount is not what you want.
http://xmpp.org/extensions/xep-0077.html#registrar-querytypes-register Also here the child count is 1


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=b34d6cb35d844cd48c41842529d22f96
 plone-5.0-python-2.7 [SUCCESS]


why filter is point?


Overall, I think it's important to allow for user supplied identification functions. That was the purpose of the get_name function. This gets important with actions: We have the `base_action` from a `transfer_action<Action>` is derived from. The actions are serialized via `base_action`, since we can't define a generic, uniquely identifiable name for a `transfer_action`, we need support for non-intrusive naming facilities.


Easiest fix: set `may_change_files = True` for commands that need to refresh annotations.
This causes the file to be reloaded and as a consequence the event listener for annotations catches the change.


This imports seem to not be used on the file, maybe a copy&paste error?


Do we want this flag for jenkins also? For debugging or while working on fixing tests maybe that's a good idea, but for CI I kind of think it is the other way around...

@tisto any comment on it?


Can you use an `MCAutoStringRefArray` here to handle the cleanup case?


Ship it!


Why before?


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1954](http://teamcity.monogame.net/viewLog.html?buildId=5044&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:15:56


 Android based applications to access the Trusted Execution Environment
허가된 실행 환경..?
이게 정확히 무슨 드라이버 인가요?


It seems that exception handler was not properly triggered since the existing implementation is oversimplified. Can you borrow the low level parts from FreeRTOS?


This:
```
    if x == 1:
        return False
    if _primality_test(x):
        return True
    else:
        return False
```
can be replaced by:
```
    return _primality_test(x)
```


Sure, either is fine.


Would be easier to read just return False here


Well, now that I'm more used to it I don't much care. But someone else was bitching about it just yesterday. Anyway, it's ok, I think it's pretty clear here.


piojo


Did you push `bower.json`'s changes?


/cc @jofomah


spaces


changed


@tisto I most probably guess that the average developer does not have a clean system python... installing something from your package manager, trying a new cool library for a project, testing a product that maybe you want to use ...

No, I don't think we can assume people will have a clean environment.

Still, virtualenv is just a command to set up once, after running virtualenv + bootstrap + buildout you can mostly forget to do ``source bin/activate`` if you are playing only with buildout...

And on top of that virtualenv is a python good practice in itself, so something that newcomers will get benefit from even if they don't engage with Plone itself


oh! Sorry Sir... I have a other revise of that code.. Tonight I will revise
it... Thank you for your checking...

On Thu, Jan 22, 2015 at 6:21 PM, LoreZyra <notifications@github.com> wrote:

> This entire function is completely wrong... [image: :-1:]
>
> It violates the "DRY" programming principle.
> http://en.wikipedia.org/wiki/Don%27t_repeat_yourself
>
> Please make better use of the Sencha API.
>
> renderer: function(value) {
>      return Ext.util.Format.fileSize(value);
> },
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vhieragang/envitz.Xplorer/commit/4722f9facaa56c0b017f582ac6263cba8d71599c#commitcomment-9394031>
> .
>


Great!
Further improved in https://github.com/phpmyadmin/phpmyadmin/commit/d68dcef9ac3f60be7388fc37c4c0622d3dffe74a


asadaqdwerdwerwer


must have login_required set to True. Only authorized users shall be able to set a payment authorization.


I really really don't understand this, how can it fail at line 622 , when it has passed through line 601 which is in a task that has *exactly* the same test at line 577? You are able to run the task, are you? What happens if you type `ant siblings generate-tooldoc` ?


1: M111 unrecognized component name: bibtask
4: M120 missing empty line before bullet
4: M121 indentation of two spaces expected
1: M190 line is too long (70 > 50)


must have login_required set to True. Only authorized users shall be able to capture a payment.


@VincentDondain - Should we be using an **as** here? The original code did an **is** check but the new code is casting SolutionEntityItem which will fail if the currentEntry is not a SolutionEntityItem.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 134](http://192.168.30.225:8111/viewLog.html?buildId=616&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 134](http://192.168.30.225:8111/viewLog.html?buildId=616&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


TeamCity Ylab :: zmactep/ylab [Build 176](http://192.168.30.225:8111/viewLog.html?buildId=617&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


yeah i did

On Thu, Jan 22, 2015 at 12:35 PM, Tom Vincent <notifications@github.com>
wrote:

> /cc @jofomah <https://github.com/jofomah>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jofomah/direct-delivery/commit/9f437481d04162e0c9206aa943fb655ef2442ec9#commitcomment-9395136>
> .
>


no it does not.


Ta.


Yes. They are essentially class hierarchy global variables with very surprising behaviour. http://www.sitepoint.com/class-variables-a-ruby-gotcha/ explains the main problem with those. Some style guides, including the internal Github's, just recommend against using them altogether.


All methods must ask for user authentication.  Always use the `self.is_authenticated` method (in the `@action decorator`):

actions.py
```
self.method_check(request, allowed=allowed
self.is_authenticated(request)
```

this is standard tastypie


Thanks :)


Sorry if this is the wrong place for these kind of discussion. So I see that you're taking a very similar route to Nethack, or at least that's the impression I got from seeing the amulet of strangulate and now ring of conflict (which any player of Nethack will be familiar with).
I would like to say it first that I'm not particularly against the idea per se, but as you may have known nowaday not many people consider Nethack as a roguelike with good design. Don't get me wrong I like Nethack, it's a classic in it's times and still considered a major roguelike today, but perception of a good roguelike has changed since then, Nethack is often criticised as being too "linear" in the sense that late game is mostly the same for all roles, the identify minigame (potions, scrolls, rings, wands, ...), sort of spoiler-required, seemingly arbitrary rule and obscure mechanics.
A cristism against amulet of strangulate in particular is that it's a otherwise useless item that's just there to screw player up.
About the conflict, pardon me as I haven't look at all the source yet, but I think it has the potential to be a no-brainer and overpowered as making monster fight eachother on a whim is a very powerful ability. I remember the ring of conflict had made many runs in Nethack much easier for me.
So back to IA, perhaps you could consider chaging those two item to be more unique? To differentiate them from the similar items in Nethack, making it less one-dimensional and balanced?
A few random thoughts
Nerfing the ring of conflict a bit, maybe it only affect low intelligence creatures (slime, worm, wolf..), creatures with higher intelligence will simply ignore it or resist it to some degree? How about conflict making creature even more aggressive toward player and will actively seek him/her out if not already engaged?
Provide the amulet of strangulate some niches beside just a trap for player. I haven't thought about this alot as it's hard to find something beneficial to go with being strangulated. Maybe if the Spirit/Insanity is low/high enough making the player more supceptible to strangulation seeing the strangulation effect is named possessedbyZuul in source (is this correct?). This means that a player low on Siprit or near insanity will be taking a bigger risk when trying on random amulets, since I assume that most amulets will be beneficial this can make some interesting game choices.

Sorry for the long post. I kinda get drawn in :) .


Have you tried com.chrome.codereview?

https://github.com/svasilinets/codereview.chromium

https://play.google.com/store/apps/details?id=com.chrome.codereview


was this on purpose?


ack


@p4r4n01d Have you tried @svasilinets com.chrome.codereview?

https://github.com/svasilinets/codereview.chromium

https://play.google.com/store/apps/details?id=com.chrome.codereview


@gforcada yes


你们家


Nice! Please add a new SQL schema of alarm_config


BTW, the bug above was caused by this commit:
http://sourceforge.net/p/openpli/mailman/message/32623539/


@aLLeXUs вы ошибаетесь. Нет смысла выносить это в конфиг. Если в Settings.basedir указать значение иное от "AppData" то лаунчер будет крашиться т.к.  System.getenv(); возвращает значение системных переменных. Правильно было бы сделать так #7


Looks good


@icattlecoder


This can be written as:
```
sc->gic_redists.nregions = 1;
OF_getencprop(ofw_bus_get_node(dev), "#redistributor-regions",
    &redist_regions, sizeof(redist_regions));
```


TeamCity Ylab :: zmactep/ylab [Build 176](http://192.168.30.225:8111/viewLog.html?buildId=617&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 128 Build time: 00:11:48


it's done ;)

2015-01-22 12:50 GMT+01:00 Alexandr Dubovikov <notifications@github.com>:

> Nice! Please add a new SQL schema of alarm_config
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Mickaelh51/homer/commit/9f07ff7c40f6007434d992de01b72d15e52eeffc#commitcomment-9395335>
> .
>


Hello James ..I am newbie to swift n working on project and stuck in between AccordionTableView .I have to show JSON data.I was trying to run your app but end up with errors..Pods errors.help me out


thanks!


pretty cool, but how about a STR check to see if the player can maintain hold of the weapon?


Heb je hier al een PR van gemaakt? Ik heb er net ook eentje ingediend: https://github.com/KennethWilke/PingdomLib/pull/18


@iron76  You can easily disable the filtering commenting this line


Perhaps, you want a example of entry in alarm_config table ?

If yes, Where I will can write it ?

2015-01-22 12:51 GMT+01:00 Mickael Hubert <mickael@winlux.fr>:

> it's done ;)
>
> 2015-01-22 12:50 GMT+01:00 Alexandr Dubovikov <notifications@github.com>:
>
>> Nice! Please add a new SQL schema of alarm_config
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Mickaelh51/homer/commit/9f07ff7c40f6007434d992de01b72d15e52eeffc#commitcomment-9395335>
>> .
>>
>
>


Jo hab ich gemacht


otherwise it produces a 2.5M file, too big for an online demo


ack


@VincentDondain - Should we rename the methods to CreateWorkspaceItems and CreateProjects since they now return a list?

Do we need to change CreateWorkspaceItem? It always returns a single item which is either a solution or a workspace. It never returns a list of items. I think we only need the CreateProject method to change. We can handle whether a list of items or a single item is returned in the New Project Dialog.

Also not sure about changing the return type from SolutionEntityItem to IWorkspaceFileObject. Should we keep this as SolutionEntityItem?


<b>to brzydkie rozwiązanie</b> - zeby by nie było wątpliwości
nie chciałem juz sie bawić w mocki a ze względu ze stan zmiennej globalnej jest wymagany w testach views.py musiałem przywrócić poprzedni stan tej zależności


just create an example as separate file in the sql directory. I will add this to our WIKI


This still fails if `/etc/network/interfaces` still contains duplicate `#VAGRANT` comments from a previous configuration. You should be able to manually delete them inside the guest and rerun `vagrant up` without problems, though.


why does this guy needs to take a lift as parameter?


Yeah well, this part of the file is dedicated to allow user to build custom plugins by dropping a C file in the directory and call `make filename.so` (or `.dll`).  I doubt anyone actually use this, and I didn't actually test that very much (if at all?) so I didn't want to spend time trying to change those.

Ideally we probably should remove those anyway IMO, as as I already said on the PR I doubt anyone actually use this, as there are more sensible alternatives available nowadays.


This looks like this "revert" (which wasn't a true revert after all, it turns out :/) might be buggy: There are two variables saveFile, one shadowing the other. That could lead to e.g. a (minor?) leak, and perhaps even unintended behavior?


Here's the issue... Change to
```<% f.text_area :text %>
```
```

This should be
```
<% f.label :title %><br>
```


Ok, no, you are right. The property ranges should be set for each single
property.

2015-01-22 10:58 GMT+01:00 Alex <notifications@github.com>:

> @mitjanikolaus <https://github.com/mitjanikolaus> because the reasoner
> exited with an exception after my refactoring, since the "port" property of
> the dummy was not string but int. Quick fix - maybe overwriting the range
> somewhere makes more sense.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-multinet/playground-rspecs-ontology/commit/d6a83e591ffd623f5dd468d6d9111f196dd24dfc#commitcomment-9393598>
> .
>


check if you can merge following commits too:

https://github.com/Lysergic-Acid/android_frameworks_av/commit/61909fb93f956dfdca02b6d577480b95929692c8

https://github.com/Lysergic-Acid/android_frameworks_av/commit/358748ee8813e080d03cc38b82596e572614afa6


проверьте, что установлены необходимые пакеты:
https://ejudge.ru/wiki/index.php/%D0%9D%D0%B5%D0%BE%D0%B1%D1%85%D0%BE%D0%B4%D0%B8%D0%BC%D1%8B%D0%B5_%D1%81%D1%82%D0%BE%D1%80%D0%BE%D0%BD%D0%BD%D0%B8%D0%B5_%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D1%8B_%D0%B8_%D0%B1%D0%B8%D0%B1%D0%BB%D0%B8%D0%BE%D1%82%D0%B5%D0%BA%D0%B8
затем обновите систему
https://ejudge.ru/wiki/index.php/%D0%9E%D0%B1%D0%BD%D0%BE%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5

On Thu, Jan 22, 2015 at 1:16 PM, ilnur <notifications@github.com> wrote:

> А как можно быстро сделать "upgrade"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/blackav/regionals-2015-training/commit/c405fd1641e95f51dc57f35998eedc4580306ba0#commitcomment-9393918>
> .
>


you should use
```
int iHiddenChannels = g_PVRChannelGroups->Get(m_bRadio)->GetGroupAll()->GetNumHiddenChannels();
```


You're allocating 4x the amount of memory by specifying `sizeof(size_t)` instead of `sizeof(uint8_t)`


No, we don't need. It is java.awt.Font.


OK


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Edit for correct case
- [ ]
```<th>Title</th>
```


no complety sure why this end up here. ive have a new windows installation, likely the remotes went bad... ive revert the commit... pull request still to be reviewed


IMO we should do the following:

```
bool CGUIWindowPVRChannels::Update(const std::string &strDirectory, bool updateFilterPath /* = true */)
{
  CSingleLock lock(m_critSection);
  bool bReturn = CGUIWindowPVRBase::Update(strDirectory);

  int iHiddenChannels = g_PVRChannelGroups->Get(m_bRadio)->GetGroupAll()->GetNumHiddenChannels();
  if (!iHiddenChannels)
  {
    CGUIRadioButtonControl *btnShowHidden = (CGUIRadioButtonControl*) GetControl(CONTROL_BTNSHOWHIDDEN);
    if (btnShowHidden)
      btnShowHidden->SetVisible(false);
  }

  /* empty list for hidden channels */
  if (m_vecItems->Size() == 0 && m_bShowHiddenChannels)
  {
      /* show the visible channels instead */
      m_bShowHiddenChannels = false;
      lock.Leave();
      Update(GetDirectoryPath());
  }

  return bReturn;
}
```


Is there a changelog for these please?


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


This is correct solution, we don't want people to get updates on deactivated maps. This is by design, the idea of active maps that in active usage and appear in download list. You still can update, but we don't force/ask for  that update.


Middle mouse support is definitely a thing to be added to hammer.js.


I must say there was an issue and discussion regarding that fix, which I need to recall to explain everything. But I don't see big issue with this.


i think this event-listener leaks as it is not unregistered in the cleanup.

i am quite sure that we fire the event after visiting the editor and then switching to another page.

Proposal:

 - use `on` and `off`
 - namespece the event, e.g. `resize.editor`: `.on('resize.editor')`


this is removed in the second commit, but if we take garrens fix and discard the following commit, we'll have to remove it in this commit


1277852-CommandLineInterface (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Should be transformed to something like: [{string:to_upper(atom_to_list(K)), V} || {K, V} <- List].

If journal wait for strings only, it should for unknown structures using something like, as the last option: io_lib:format("~p", [V])


Another thing. Provide backwards compatibility and do not break API in master. API marked with Obsolete will be removed in major version changes.


Lately it seems to be a habit not to include change logs. Very uncomfortable. But sometimes manufacturers even don't supply one.


@dericofilho I'd update a fix for this test fail case.


I know we've had issues with all sorts of bit masks when approaching their "edges". Unit tests for such edge cases would be grand.


then i have i can add here :)


lol...typo. Thanks


So it seems Java 8 changes the interface/abstract class ideology... Not the best thing IMO, but oh well.


Seems like you forgot to pass aTHX_ to safe_av_fetch() per our discussion.


Test


pls pull src/Laravel/SortUseNamespace.php and the corresponding tests-laravel/008-inline-html-with-use.*

Please do let me know if there's other failing test case.


ok as u wish...i hope @TheRingMaster  is ok with that.


Is it really better?


sending mail


We've come across an issue with this change when targeting Windows/GL.  In particular, when attempting to exit an MG app the following exception is raised:

OpenTK.Graphics.GraphicsContextException was unhandled
  HResult=-2146233088
  Message=Failed to make context 131072 current. Error: 6
  Source=OpenTK
  StackTrace:
       at OpenTK.Platform.Windows.WinGLContext.MakeCurrent(IWindowInfo window)
       at OpenTK.Graphics.GraphicsContext.MakeCurrent(IWindowInfo window)
       at Microsoft.Xna.Framework.Threading.BlockOnUIThread(Action action)
       at Microsoft.Xna.Framework.Graphics.RenderTarget2D.Dispose(Boolean disposing)
       at Microsoft.Xna.Framework.Graphics.GraphicsResource.Finalize()
  InnerException:

Inspection of the OpenTK.Platform.Windows.WinWindowInfo instance shows that its disposed flag is true.

After reversing this change, the app successfully exits.


Stylistic suggestion: Perhaps indent this?


Add parens to `answer` just to be consistent.


PS: I don't see much of a reason to keep the scale-sep and indirect options. It should be decided automatically, and outside of the automatic decision, changing it doesn't make much sense. They are mostly useful for debugging, not as a way for the user to configure things in a meaningful way. (Also they might have been useful for testing when these things were new.) So if it simplifies the code, we can get rid of them, although the underlying mechanism is still needed. (Even so, we can probably drop using the unseparated scalers on anything but chroma.)


Как-то странно получилось. Совсем не понял о чем абзац


Ah, I see. Maybe you could fix (2) without modifying the Display class, by setting the title in the CanvasLFImpl.createNativeResource0 native.


I don't think this is needed as Ruby automatically returns the last statement so that would the result of line 54.


Scrap that, they're internal classes.


Looks good.  I'll commit after testing on musl and uclibc.  The later does have some issues with -lm but this should work there too.


@abolishme : Can you add the full size version of the Icon for the App Store and a vectorial version too? Thanks
Dimensions 1024x1024px


test comment


done


Primer fichero prueba


On 01/22/2015 05:10, Yuri Momotyuk wrote:
>
> Hmmm..... I still need this tool to change between internal speakers/
> headphones / HDMI... Can I use another tool to change default audio
> output?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pcbsd/pcbsd/commit/ac5eadcb55ab5f4543fc9aaf99146e6085987639#commitcomment-9393778>.
>

Yes, this functionality is now available from within pc-mixer (open the
mixer UI, and it is in the "configuration" tab). We simple removed
pc-soundconfig because there was no point for an entire utility just to
handle that one thing.

~~Ken Moore~~
PC-BSD/iXsystems


I will work on it and send.


Your original code seems to be using comma as the separator while cStringTokenizer by default is using white space characters for delimiters. You may need to add the , as the second parameter for the consturctor of cStringTokenizer to split the string around commas.


Can we get more specific than array here?


refegfre


7: M102 unrecognized bullet/signature


I wasn't really sure if it was being accessed by multiple threads :p


This is unfortunately a **BC break** for everyone who extended `DateTime` before. It makes it impossible to write code which will work on both PHP 5 and 7. cc @dg

~~~php
namespace Foo;

// This will not work in PHP < 7
class DateTimeA extends \DateTime
{
	// Strict Standards: Declaration of Foo\DateTimeA::createFromFormat() should be
	// compatible with DateTime::createFromFormat($format, $time, $object = NULL)
	public static function createFromFormat($format, $time, DateTimeZone $timezone = null) {}
}

// This will not work in PHP >= 7
class DateTimeB extends \DateTime
{
	// Strict Standards: Declaration of Foo\DateTimeB::createFromFormat() should be
	// compatible with DateTime::createFromFormat($format, $time, DateTimeZone $object = NULL)
    public static function createFromFormat($format, $time, $timezone = null) {}
}
~~~


OK


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1955](http://teamcity.monogame.net/viewLog.html?buildId=5046&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


large mod pack servers can take 5-8 minutes to boot after a crash


Anyway, @didlix' change is not just gender-neutral, but it's also grammatically correct. :+1:
http://en.wikipedia.org/wiki/Singular_they


@cuiguoji
@cuiguojie
test


Vil gjette at det skulle vært person.children == fatherMotherChildren.value


+/**
+ * Implements hook_schema().
+ */

Correct the above comment


 $sql = 'SELECT nid FROM {node} n WHERE n.type = :type';
+  $result = db_query($sql, array(':type' => 'products'));
+  $nids = array();
+  foreach ($result as $row) {
+    $nids[] = $row->nid;
+  }
+  // Delete all the nodes at once
+  node_delete_multiple($nids);

Enhance the above code..


1: M111 unrecognized component name: BibTask
5: M120 missing empty line before bullet
5: M121 indentation of two spaces expected


Include version in info file.


Ser bra ut!


Update this file


@eliotsykes I feel like this test is borrowing a lot from the todo_list_crud_spec, am I correct in thinking some of this, sorry, a lot of this could be dry'd up, I'm not sure where to put this code though as I don't think it's really a factory, but that's my inexperienced opinion :)


.module should never have closing ?> tag.