1: D400 First line should end with '.', not 'e'
28:1: I101 Imported names are in the wrong order. Should be task_sleep_now_if_required, task_update_progress, write_message
30:1: I101 Imported names are in the wrong order. Should be Error, deserialize_via_marshal, run_sql, serialize_via_marshal
34:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.date
41:44: E251 unexpected spaces around keyword / parameter equals
41:46: E251 unexpected spaces around keyword / parameter equals
41: D401 First line should be imperative: 'Return', not 'Returns'
41: D209 Put multi-line docstring closing quotes on separate line
41: D205 Blank line missing between one-line summary and description
52:75: E502 the backslash is redundant between brackets
53:65: E502 the backslash is redundant between brackets
54:28: E225 missing whitespace around operator
65:78: E502 the backslash is redundant between brackets
66:83: E502 the backslash is redundant between brackets
67:32: E225 missing whitespace around operator
80:44: E261 at least two spaces before inline comment
80:44: E262 inline comment should start with '# '
80: D401 First line should be imperative: 'Return', not 'Returns'
80: D400 First line should end with '.', not 'm'
86:30: E261 at least two spaces before inline comment
95: D401 First line should be imperative: 'Return', not 'Returns'
95: D209 Put multi-line docstring closing quotes on separate line
95: D400 First line should end with '.', not 'n'
95: D205 Blank line missing between one-line summary and description
104: D401 First line should be imperative: 'Find', not 'Finds'
104: D400 First line should end with '.', not 'g'
104: D300 Expected """-quotes, got '''-quotes
113:66: E502 the backslash is redundant between brackets
114:46: E225 missing whitespace around operator
132:8: E114 indentation is not a multiple of four (comment)
146: D209 Put multi-line docstring closing quotes on separate line
146: D400 First line should end with '.', not 'y'
146: D205 Blank line missing between one-line summary and description
146: D300 Expected """-quotes, got '''-quotes
157:68: E502 the backslash is redundant between brackets
158:24: E225 missing whitespace around operator
169: D401 First line should be imperative: 'Return', not 'Returns'
169: D400 First line should end with '.', not 'd'
169: D300 Expected """-quotes, got '''-quotes
177:91: E502 the backslash is redundant between brackets
178:28: E225 missing whitespace around operator
181:72: E502 the backslash is redundant between brackets
182:60: E502 the backslash is redundant between brackets
183:24: E225 missing whitespace around operator
187: D209 Put multi-line docstring closing quotes on separate line
187: D400 First line should end with '.', not 'y'
187: D205 Blank line missing between one-line summary and description
187: D300 Expected """-quotes, got '''-quotes
192:73: E225 missing whitespace around operator
193:96: E502 the backslash is redundant between brackets
193:56: E231 missing whitespace after ','
193:76: E231 missing whitespace after ','
194:56: E231 missing whitespace after ','
199: D401 First line should be imperative: 'Return', not 'Returns'
199: D205 Blank line missing between one-line summary and description
209:63: E502 the backslash is redundant between brackets
210:28: E225 missing whitespace around operator
212:76: E502 the backslash is redundant between brackets
212:70: E225 missing whitespace around operator
219:58: E502 the backslash is redundant between brackets
220:28: E225 missing whitespace around operator
222:65: E225 missing whitespace around operator
233:80: E502 the backslash is redundant between brackets
234:24: E225 missing whitespace around operator
238: D400 First line should end with '.', not 'n'
238: D300 Expected """-quotes, got '''-quotes
244:34: E261 at least two spaces before inline comment
244:34: E262 inline comment should start with '# '
247:50: E225 missing whitespace around operator
253:60: E502 the backslash is redundant between brackets
254:24: E225 missing whitespace around operator
256:1: E302 expected 2 blank lines, found 1
256: D401 First line should be imperative: 'Identifie', not 'Identifies'
256: D400 First line should end with '.', not 't'
263:1: E302 expected 2 blank lines, found 1
263: D401 First line should be imperative: 'Doe', not 'Does'
263: D209 Put multi-line docstring closing quotes on separate line
263: D400 First line should end with '.', not 'e'
263: D205 Blank line missing between one-line summary and description
271:80: E502 the backslash is redundant between brackets
272:24: E225 missing whitespace around operator
278:17: E127 continuation line over-indented for visual indent
279:76: E502 the backslash is redundant between brackets
287:81: E502 the backslash is redundant between brackets
292:86: E502 the backslash is redundant between brackets
299: D400 First line should end with '.', not 'A'
311:17: E261 at least two spaces before inline comment
312:65: E502 the backslash is redundant between brackets
313:60: E225 missing whitespace around operator
320:43: E502 the backslash is redundant between brackets
321:13: E128 continuation line under-indented for visual indent
321:77: E502 the backslash is redundant between brackets
324:80: E502 the backslash is redundant between brackets
325:50: E225 missing whitespace around operator
327:75: E502 the backslash is redundant between brackets
332: D400 First line should end with '.', not 'S'
334:64: E502 the backslash is redundant between brackets
335:50: E502 the backslash is redundant between brackets
336:20: E225 missing whitespace around operator
337:60: E225 missing whitespace around operator
342:119: E502 the backslash is redundant between brackets
345:17: E261 at least two spaces before inline comment
349:62: E502 the backslash is redundant between brackets
354:43: E502 the backslash is redundant between brackets
355:13: E128 continuation line under-indented for visual indent
357:81: E502 the backslash is redundant between brackets
358:58: E225 missing whitespace around operator
360:57: E502 the backslash is redundant between brackets
361:50: E225 missing whitespace around operator
365: D209 Put multi-line docstring closing quotes on separate line
365: D205 Blank line missing between one-line summary and description
369:66: E502 the backslash is redundant between brackets
370:20: E225 missing whitespace around operator
378:42: E225 missing whitespace around operator
382:38: E225 missing whitespace around operator
387: D209 Put multi-line docstring closing quotes on separate line
387: D400 First line should end with '.', not 's'
387: D205 Blank line missing between one-line summary and description
392:54: E502 the backslash is redundant between brackets
428: D401 First line should be imperative: 'Return', not 'Returns'
428: D209 Put multi-line docstring closing quotes on separate line
428: D202 No blank lines allowed *after* function docstring, found 1
428: D400 First line should end with '.', not 'r'
428: D205 Blank line missing between one-line summary and description
434:24: E261 at least two spaces before inline comment
434:25: E262 inline comment should start with '# '
443:73: E502 the backslash is redundant between brackets
444:51: E225 missing whitespace around operator
453:63: E502 the backslash is redundant between brackets
454:49: E225 missing whitespace around operator
459: D401 First line should be imperative: 'Return', not 'Returns'
459: D400 First line should end with '.', not 's'
461:53: E502 the backslash is redundant between brackets
465: D401 First line should be imperative: 'Return', not 'Returns'
465: D209 Put multi-line docstring closing quotes on separate line
465: D400 First line should end with '.', not ':'
465: D205 Blank line missing between one-line summary and description
474:13: F841 local variable 'dummy' is assigned to but never used
479:27: E261 at least two spaces before inline comment
479:28: E262 inline comment should start with '# '
484: D401 First line should be imperative: 'Return', not 'Returns'
484: D209 Put multi-line docstring closing quotes on separate line
484: D400 First line should end with '.', not ','
484: D205 Blank line missing between one-line summary and description
496:75: E502 the backslash is redundant between brackets
497:23: E128 continuation line under-indented for visual indent
497:24: E225 missing whitespace around operator
511:72: E502 the backslash is redundant between brackets
512:69: E225 missing whitespace around operator
514:83: E502 the backslash is redundant between brackets
515:23: E128 continuation line under-indented for visual indent
515:24: E225 missing whitespace around operator
522: D401 First line should be imperative: 'Thi', not 'This'
522: D209 Put multi-line docstring closing quotes on separate line
522: D400 First line should end with '.', not 'd'
522: D205 Blank line missing between one-line summary and description
529:5: B901 blind except: statement
533:1: E302 expected 2 blank lines, found 1
533: D401 First line should be imperative: 'Thi', not 'This'
533: D209 Put multi-line docstring closing quotes on separate line
533: D400 First line should end with '.', not 'd'
533: D205 Blank line missing between one-line summary and description
548:20: E261 at least two spaces before inline comment
552:1: E302 expected 2 blank lines, found 1
552: D401 First line should be imperative: 'Thi', not 'This'
552: D209 Put multi-line docstring closing quotes on separate line
552: D400 First line should end with '.', not 'e'
552: D205 Blank line missing between one-line summary and description
562:1: E302 expected 2 blank lines, found 1
562:59: E251 unexpected spaces around keyword / parameter equals
562:61: E251 unexpected spaces around keyword / parameter equals
562: D401 First line should be imperative: 'Update', not 'Updates'
562: D209 Put multi-line docstring closing quotes on separate line
562: D202 No blank lines allowed *after* function docstring, found 1
562: D400 First line should end with '.', not 'd'
562: D205 Blank line missing between one-line summary and description
573:69: E502 the backslash is redundant between brackets
574:46: E225 missing whitespace around operator
583:75: E502 the backslash is redundant between brackets
584:24: E225 missing whitespace around operator
585:21: E261 at least two spaces before inline comment
585:22: E262 inline comment should start with '# '
589:83: E502 the backslash is redundant between brackets
590:24: E225 missing whitespace around operator
604:14: E261 at least two spaces before inline comment
618:60: E502 the backslash is redundant between brackets
619:28: E225 missing whitespace around operator
622:69: E502 the backslash is redundant between brackets
623:33: E128 continuation line under-indented for visual indent
625:60: E502 the backslash is redundant between brackets
626:28: E225 missing whitespace around operator
628:69: E502 the backslash is redundant between brackets
629:33: E128 continuation line under-indented for visual indent
631:59: E502 the backslash is redundant between brackets
632:28: E225 missing whitespace around operator
642:68: E502 the backslash is redundant between brackets
643:42: E128 continuation line under-indented for visual indent
651:85: E502 the backslash is redundant between brackets
652:28: E225 missing whitespace around operator
657:117: E251 unexpected spaces around keyword / parameter equals
657:119: E251 unexpected spaces around keyword / parameter equals
657: D401 First line should be imperative: 'Update', not 'Updates'
657: D400 First line should end with '.', not 's'
661:62: E225 missing whitespace around operator
666:64: E502 the backslash is redundant between brackets
667:24: E225 missing whitespace around operator
688:17: E125 continuation line with same indent as next logical line
704:72: E502 the backslash is redundant between brackets
716:66: E502 the backslash is redundant between brackets
717:21: E128 continuation line under-indented for visual indent
721:66: E502 the backslash is redundant between brackets
722:21: E128 continuation line under-indented for visual indent
723:64: E225 missing whitespace around operator
726: D401 First line should be imperative: 'Modifie', not 'Modifies'
726: D209 Put multi-line docstring closing quotes on separate line
726: D400 First line should end with '.', not 'n'
726: D205 Blank line missing between one-line summary and description
735: D401 First line should be imperative: 'Insert', not 'Inserts'
735: D400 First line should end with '.', not 's'
744:50: E261 at least two spaces before inline comment
744:50: E262 inline comment should start with '# '
750:34: E261 at least two spaces before inline comment
750:35: E262 inline comment should start with '# '
757:23: E261 at least two spaces before inline comment
757:24: E262 inline comment should start with '# '
764:38: E225 missing whitespace around operator
768: D400 First line should end with '.', not 's'
776:38: E225 missing whitespace around operator
780: D209 Put multi-line docstring closing quotes on separate line
780: D400 First line should end with '.', not 'w'
780: D205 Blank line missing between one-line summary and description
790: D400 First line should end with '.', not ')'
807: D401 First line should be imperative: 'Update', not 'Updates'
807: D209 Put multi-line docstring closing quotes on separate line
807: D202 No blank lines allowed *after* function docstring, found 1
807: D400 First line should end with '.', not 't'
807: D205 Blank line missing between one-line summary and description
817:66: E502 the backslash is redundant between brackets
818:59: E502 the backslash is redundant between brackets
819:20: E225 missing whitespace around operator
845:27: E261 at least two spaces before inline comment
852:61: E225 missing whitespace around operator
853:71: E225 missing whitespace around operator
856:82: E502 the backslash is redundant between brackets
857:28: E225 missing whitespace around operator
862:32: E261 at least two spaces before inline comment
866:28: E261 at least two spaces before inline comment
866:29: E262 inline comment should start with '# '
869:82: E502 the backslash is redundant between brackets
870:27: E128 continuation line under-indented for visual indent
872:84: E502 the backslash is redundant between brackets
873:19: E128 continuation line under-indented for visual indent
873:20: E225 missing whitespace around operator
878:56: E225 missing whitespace around operator
879:58: E225 missing whitespace around operator
882:87: E502 the backslash is redundant between brackets
883:27: E128 continuation line under-indented for visual indent
883:28: E225 missing whitespace around operator
888:38: E251 unexpected spaces around keyword / parameter equals
888:40: E251 unexpected spaces around keyword / parameter equals
888: D401 First line should be imperative: 'Rebalance', not 'Rebalances'
888: D400 First line should end with '.', not 't'
900:58: E225 missing whitespace around operator
902:23: E261 at least two spaces before inline comment
904:62: E502 the backslash is redundant between brackets
905:24: E225 missing whitespace around operator
911:56: E225 missing whitespace around operator
912:70: E502 the backslash is redundant between brackets
913:27: E127 continuation line over-indented for visual indent
913:28: E225 missing whitespace around operator
915:33: E128 continuation line under-indented for visual indent
916:33: E128 continuation line under-indented for visual indent
917:33: E128 continuation line under-indented for visual indent
919:72: E502 the backslash is redundant between brackets
920:28: E225 missing whitespace around operator
928: D401 First line should be imperative: 'test', not 'tests'
928: D400 First line should end with '.', not 's'


yeah, we should move the amalgamation to a script that can be run on
multiple platforms. Shall we write in JS and depend on a bootstrapped v7 ?
:-)

On Tue, Mar 24, 2015 at 5:44 PM, Alexander Alashkin <
notifications@github.com> wrote:

> I don't know why - but I don't have it.
> But I didn't try to find it - since it comples amalgamated v7.c, and if I
> try to change something in /src - .proj doesn't help.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cesanta/v7/commit/420922cee5ac2ff57fbae756d4fad88178d59b56#commitcomment-10371839>
> .
>


Still planning to add `num-round-even` for convenience of porting code that uses the IEEE default?


I don't understand what you mean. What's an explicit assertion versus an implicit one? I'm happy to fix this, just please help me better understand what you're looking for (^_^)

Also, in my email I received a separate message about adding FIXME comment, but it seems like that message has disappeared. I'm assuming it was related to this line, but I can't tell.


I must admit when I see this page, I think of AngularJS.


I guess I tend to look at examples in API docs instead of reading the text.


I think it would be better to encapsulated the behaviour here in this controller action as a method on the model, e.g. `Government#close(end_date)`? That makes it easier to change and unit test.


clang -o test_mlx test2.c -L/usr/X11/lib -lmlx -framework OpenGl -framework Cocoa -framework glut


@artemp - okay, sounds great. Will this work for the same for collections as well? That `geojson-geometry-collection` visual test mentioned at #2735 may be fixed by this. (or maybe we are just missing collections support in marker_helpers....)


The make() command for maps can take a second parameter.  I don't actually know what that second parameter does since I can't find it documented, but let's go ahead and initialize it to what the maximum size of the map could be.


in later episodes they imply that otters is not a gag manga but something serious with serious jokes


These are all private functions in anonymous namespaces, and I don't think those should go in Doxygen at all, aside from appearing in the source code pages.


Similar to: http://markdotto.com/2011/11/29/css-property-order/

Position, box model, typography, appearance.

Basically [this](https://github.com/twitter/recess/blob/master/lib/lint/strict-property-order.js#L36), but not strict.


i am trying to avoid using functions offered by GNU makefile, and to introduce less shell script here, so $(top_builddir)/man is hardcoded.


We used to use it in eagle, too


you can also do

   emptyMap := map[string]string{}

I'm not sure which is better.


cool


work in progress


@gpoirier just to be clear if this were code in one of our projects the name ``WordCount`` would've been preferred


Thanks for this!

Please look at https://github.com/kesha-antonov/jstree-rails-4/commit/272f0c2d0c9e9bc955b036e6bc1aada3087187d7#diff-2a602ad55805f6ac23f0553a79653908

Now should be fine without adding an initializer.


specify doctrees explicitly, so i can remove it without referencing the default `.doctrees` in `clean-local`.


I shall not meddle in things I don't understand.
I shall not meddle in things I don't understand.
I shall not meddle in things I don't understand.
I shall not meddle in things I don't understand.


I want to make this change because...


I don't like this.  Sometimes requests get params you don't expect.  Instead, can we make the dimensions an explicit named parameter?  What about

```/v1/collectd?dims["type"]="test"&dims["more"]="cats"```


No problem, and it sounds like you're feeling a bit more comfortable with it now.


Si totusi nu cred că e ok să dezactivăm language fallback...


Some comments


Agreed.  I've got a note on that below.
Additionally this is the recommended pattern: http://channel9.msdn.com/posts/C-and-Beyond-2012-Herb-Sutter-You-dont-know-blank-and-blank


done


As I said, this is no complete fix because the value is already used in the static constructor - which is run before you can set this property. Idea: Make the start value "false", then at the end of the static constructor set it to "true".


I don't like the idea of saving it each time the repository is visited. Couldn't we just make that directly in the template?


What about obeyFixRules and obeyWarnRules? I just think that ObeyFix is not obvious enough for the user - it does not really tell what is supposed to be put there.


there is no point to run multiple instance of sphinx-build against the same set of rst files. and `make -j8` sometimes fails sphinx-build !


Use LOG->Trace instead of printf so that it goes to the log file like everything else.


You should be able to use the [`button_to`](http://api.rubyonrails.org/classes/ActionView/Helpers/UrlHelper.html#method-i-button_to) helper here:

```ruby
<%= button_to("Yes, close this government", close_admin_government_path(@government) , method: :post, class: "btn btn-danger btn-large") %>
```


is it okay to use "It'll be" two times in a row?


ok do it


I would gladly do so if I had free time for it Justin. Sorry.


This, however, is not rendering anything as of now. I'm still looking at it.


La methode ne devrait prendre qu'un seul parametre


I've noticed you kept the `TestRunner` class name. This should be changed.


I would like that


90961636-Occurence-26---Unit-test-for-sizing-consolidation (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1165 of 1167 tests passed)

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


is the 'by' necessary?


Yeah solid point. I actually need to do some testing of this. I'll get some documentation up afterwards.


Ugh.  Sorry.


add it to included repo.sls
it's easy later to override the sls (and depend on sls but not on pkgrepo)

use-case: repoint all formulas to internal mirror


shall we use the same approach in `v7_to_double()`, and call `v7_to_double()` here?


Oh, thanks. I fixed that yesterday - I must have forgotten to push the changes.


Should be no problem in theory, however I don't own personally a board based on this chip. If somebody wants to do it, I'd be happy to merge the code :)


In general I think we should use `output.View` rather than the streams directly. I've noticed there's no `\n` appended to the value of the libexec dir, which is completely fine for scripts, but not so nice for human users.


shall we use the same approach in `v7_create_number`, and call it here?


please add in future "Sample" as middle name


https://github.com/golang/go/blob/master/src/runtime/hashmap.go ```func makemap(t *maptype, hint int64, h *hmap, bucket unsafe.Pointer) *hmap {```

Looks like it's similar to java hashmap hint and it scales correctly on load factor.


I just spoke with Dan and we'd change 'They have' for 'This user has'. Other than that it looks good.


How frequent are you sending packets? If the channel is busy, I see often
the ACK not being properly received.

On Tue, Mar 24, 2015 at 6:10 AM, vador98 <notifications@github.com> wrote:

> I'm having issues with my arduino not receiving the ACK from the raspi. Is
> this something you've seen or tested out before? It seems to be receiving
> the signal fine (RSSI of -29), but just not sending the ACK consistantly
> (Typically less than 5% of the time).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abouillot/HomeAutomation/commit/74f7165de696ce6a2f5f3bbc92fd2b9587bad586#commitcomment-10359773>
> .
>


Yes, it can be verified, but I cannot use the existing monoid laws Prop because it uses a simple equals on the operands, which wouldn't be true unless an actually class with an overridden equal is used. So it can be done, but it's a lot of work, and the author didn't even take the time to do it in its own answer folder. If there's any value in me doing it, I can take the time, but it didn't seem worth it to me.


Wow! It's sounds great! Compile itself, like any compilers!


The setting that seems to be necessary in `development.rb`:
```ruby
  ActiveSupport::Deprecation.silenced = true
```


Текст ошибки приходит с сервера. Пусть так пока. В рамках #18 аутентификация будет переделана, и окошко логина будет находиться в клиентском проекте. @Argelein закрывай задачу.


Makes sense.


Did you mean to leave these commented out lines here?


An implicit test is a test that assumes everything is copacetic just because of the existence of an object (file or object in Ruby) where an explicit test takes an adversarial approach and assumes that if I expect an object then it needs to explicitly be this kind of object.  The difference is `File.exist?` vs. `File.file?` and `File.directory?`

The FIXME comment was deleted because I decided that I would just note it (or you could) in #3543


I'm not a fan of the short names used by the author, but I usually try to keep to his style to some degree. I would never have called it WC myself.


Hello, the component is like a normal installation file joomla ??


I feel this conditional logic is a bit weird and could be handled by `_get_monkeypatch_args`.

Maybe
```perl
   my $mp_args = $self->_get_monkeypatch_args($msg);
```
And that can do the right thing, square up $msg ( seeing its not used elsewhere anyway ), and then return 'message => $msg' instead of undef.


@gpoirier nope, none. just add the comment regd. simple equals being used in Prop


think we can make it shorter?


understood, thanks for the context


There is a comma missing!


kind of -> kinda?


Please change it for now, this didn't work in 3.1.3 either.


prefer josh Pang as grey font description


I will remove it


getAcquire won't allow this method to be const.


i need my coffee :P


so You can pass `null` to the constructor or not? This `if` makes no sense, just make the argument as non-nullable and You're done


> If we don't actually do anything with value

Once you get to the schema resolution part, though, you will use the default values, and I'm thinking it will be easiest to work with the default values as parsed JSON objects.  As just one example, for a default value for a record, you have to drill down into fields — and so you need something record-like to store those fields.  Might as well just use the JSON representation that the default values have in the schema itself.  Plus then you get a lot of the `avro_default_foo` functions that you were mentioning for free!  `json_decref` frees the default value for you, for instance.

Now all of that said, the function you describe is exactly what you'd put into the public API.  And you'd parse the `field_default` string into a `json_t *`, and then stash that away as the default value.  But, when you're parsing a schema from JSON, you've already done that parsing.  So you don't want to take the parsed JSON and encode it back to a string, just so that you could pass it in to this function.


This one can be inferred by the compiler now.


here's a comment


Hm, I see, side-by-side is hard to read:
```perl
   $mp_args ? %$mp_args : (), message => $msg

   =~
   (  $mp_args ? %$mp_args : ()  ), message => $msg

So $msg trumps that from mp_args.

So what would the harm in mp_args always returning an empty list instead of undef?


here's another comment :+1:  :cake:


invert the conditional and early exit instead of indenting 30 lines


:D


This line is suspected of causing a NPE when the result of super.list is null. contents.addAll does not accept null as a parameter.


same here, early exit


Try to fill the triangle as less as you can


Above I recommend making `getAcquire const`.


Extremely infrequent.  I send them whenever a button is manually pressed.  During my testing, it's much more frequent that it will be once in place, but it's still only once every few seconds for now.


Deployd is moving fast right now. Deployd version v0.8.0-rc.2 works fine with dpd-passport, but I believe dpd-passport isn't working with the current master! (Please see https://github.com/deployd/deployd/releases... where do I file an issue?)


> and check the default value, e.g., we expect an integer

That would also be easier in the internal function that takes in a `json_t *` — if the schema is an `int`, then make sure the `json_t *` is a JSON number, and so on.


I just hadn't seen the last commit, which used Trace.  Looks like that was the only place that used printf.


This one can now be inferred by the compiler.


@johnfinley - did this change work for you?  (so I can send in a PR to master)


When updating HumHub, please provide a detailed theme change list so themes can be repaired prior to installation of new version.


Got it. I'll update the assertion to use `File.file?`.


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-MacOS-10.9-clang: OK (1667 of 1668 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)


Have not gotten to it yet.


$mp_args is used both to check truthiness and to convey a set of key/pair values.


Alright, done.


Interesting.


Just gave it a test and it works just fine. Thanks again.


Yeah this is the source of the Draft3/Draft4 weirdness...Swagger is actually inconsistent about what version of jsonschema it's pretending to be.


Where is this function defined?!
I've greped all files, but found nothing more that line.
Is this some kind of black magic?)


hello world


I have previously set

    Number of static import needed for .* (e.g. 'java.lang.Math.*')

to 0 in Organize Imports. I also have imported [eclipse.importorder](https://github.com/xwiki/xwiki-commons/blob/master/xwiki-commons-tools/xwiki-commons-tool-verification-resources/src/main/resources/eclipse.importorder) in order to follow [Imports code style](http://dev.xwiki.org/xwiki/bin/view/Community/JavaCodeStyle#HImports) (static imports last).


All of the above can be inferred by the compiler.


Line wrap at 80 columns


Thanks :)


@hemanth I don't think that's necessary -- or even valid? `$const` is a string here, right? So treating it like an array doesn't make any sense to me.

I think you need to take a step back and re-think this `handle` function. It seems to me like there's some redundant checks and logic going on.

I.E.
- `return $_` gives the user back their original query -- you should be returning `$const` right?
- why try to lookup our constant in $constants before we've verified (on the next line) that we have something to work with?
- after verifying that we have a real remainder and our hash lookup was successful we can carry on, so we don't need another `if` before our return
- again, `result` should be set to the value of `$const`

Hope this helps!

If you haven't already I'd write some tests so you know what output you're expecting. Then you'll be able to see what your code is currently giving and alter it to work as expected.


ta == nullptr


Too much primitive clutter, no?  num-round-half-to-even can be defined by user in terms of num-round(-half-to-away-from-0).


Pode ser, só colocar em hash então.


Each of these still needs a link to the issue/PR at hand


No explicit type arguments necessary here.


Can you drop a comment somewhere in here linking to the particular interface you're satisfying by returning validators.extend(...)? Just working off context, it looks like you're saying "Draft4 is good, plus know how to validate these other fields"?


If `require` is falsy it means `modil` module has not been loaded. Wouldn't it be better to say it clearly?


Please understand the exception is raised on the incorrect return of the lambda. You are in violation of the contract of select many that states the lambda should return an IEnumerable  You are returnning null, and that' s why you have an exception , not because the elements in the stream are null.

Execute this

```
 var arrray= new string[3]{"a", "b", "C"};
 array.SelectMany(n => true ? null : "x").ToList();
```
The same exception is raised and the elements are not null.

Obviously the API will raise exceptions when you use it wrong, but having null elements in the collection is not an incorrect use as the previous code shows.

until know your argument is : I like when stream removes nulls, so i remove them every where, but I do not have an alternative for when the programmer does not wish to remove then. I will force all programmers to remove nulls because I want to, even thought i'm not sure I should.
Our argument is : I don't care if the stream have nulls, the APi will work with, our without, their presence. If the programmer cares to filter them, the API enables that with WHERE , and if the programmer does like nulls, and needs them, we shall not interfere with the programmers goal/ business logic. We shall not interfere!

I have several use cases where null is a proper value and is important to business logic, so that I even produce non null view objects when a null is present. Hell I even groupby values that may be null, and null is a proper group. This stream version of "null should be auto-removed"  makes this logic not possible and so i'm forced to not use your API and use the source collection directly.  It is not a good trade off when the programmers solution is not to use your API.

You say you do not known what is the ideal behavior. The ideal behavior is the one that does not limit the programmers options. Removing nulls "silently" removes options, so it is not a correct behavior.


You guys have lots of expression I'm not used to seeing. What does 'regd' stands for? I'm not sure if you're asking me to change anything.


@jpinsonault I added a commit that tries to make AbstractConfigObject match upstream a bit more closely, and it caused the behavior to change a bit here.  I'm happy to change my code to throw a different type of error besides `ConfigBugOrBroken` if there's something that we think would make more sense, but I don't think it'll be `NoMethodError` any more since the upstream does things a little differently.  What do you think?


Ack. I just removed all of those; the khmer notes don't have the links.
(Numbers, yeah, links, no).

Alright. Adding.

On Tue, Mar 24, 2015 at 1:15 PM Michael R. Crusoe <notifications@github.com>
wrote:

> Each of these still needs a link to the issue/PR at hand
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ged-lab/screed/commit/142b6f6a6883f1fec9a4b52080a8ba58d518b4a8#commitcomment-10372538>
> .
>


"A few weeks ago"


"solves" instead of "solve"


If you use `*` for the dependencies it beats the point of the dependency badges :/

Why don't you like my approach? `npm-shrinkwrap.json` is still being used.


"applications" instead of "application"


"or tries to solve"


"applications"


Lol. Don't forget about the profile avatar in account/profile.tpl -> less/account


You might be able to do this here:

        if (packet.nodes ().size () == 0)
        {
            peer->charge (Resource::feeUnwantedData);
            return;
        }


yes I know it


Maybe explain how they are different instead of just stating thay they are different


Is there any legal reason for that? Or can I just change it to Miro Hrončok?


Since there are no screed issues right now there is no reason to specify 'khmer repo' in this document


Nice! Love to see WP developers using github to host their open source plugins :+1:


Oups, I'll fix it.


Just to be clear, "OverwriteStorage" does not support atomic file overwrites?


Upfront I am negative towards this change as it introduces extra complexity to the python code to work around a problem that would be better solved orthogonally by setting the PATH to uniquely point to the desired toolchain.


No need for `this.` prefix in each getter.


There was talk of enabling the screed issue tracker; I was attempting to
avoid people getting confused reading these release notes if the screed
tracker is brought up (though it also doesn't matter if I'm giving direct
links.)

On Tue, Mar 24, 2015 at 1:21 PM Michael R. Crusoe <notifications@github.com>
wrote:

> Since there are no screed issues right now there is no reason to specify
> 'khmer repo' in this document
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ged-lab/screed/commit/142b6f6a6883f1fec9a4b52080a8ba58d518b4a8#commitcomment-10372695>
> .
>


What steps?


"clearly"


Well this is a special module. It's used to set path:

```bash
PATH = "$(avocado exec-path):$PATH"
````
Additional `'\n'` makes a difference.


Should "...years in order that..." be "...years in order to ensure that..." or similar?


I need to look into this some more. The following fails "The site regenerator should always regenerate objects that don't respond to :path":

````ruby
    def source_modified_or_dest_missing?(source_path, dest_path)
      #source_modified = source_path ? modified?(source_path)  : true
      source_modified = (source_path and modified?(source_path)) or true
      dest_missing    = (dest_path and !File.exist?(dest_path)) or false
      return source_modified || dest_missing
    end
````

But if I swap the comments on the source_modified assignment lines, it passes. Clearly I'm not understanding something about Ruby. I'll dig into it tonight.


Suggest `EMPTY_EXTENSIONS` instead of `EMPTY_SET`.


"The data you are rendering is chaning over time"


It is recommended to use os.path.join to join paths rather than string concatenation to support multiple platforms as it will use the correct path separator. '\' on windows and '/' on linux.


@gpoirier apologies. regd = regarding.
i was requesting that you add that reason in the code comment. it's not a major point however (should you feel strongly against it)


Drop "simply"


@ghxandsky we use this in production, I think we use only the agent for KVM but still. If people run into problems we'll just have to support.


I like the idea of adding support for these as they are common. I am not sure that tgz should be application/x-gzip. Shouldn't it be application/x-tar instead?


for (auto const& p : packet.nodes ())


Apparently mame now runs for several whole seconds before it crashes.
mame really tests the limits of libco in several ways, and people who
"understand" libco are having a look at it.  What they need is
someone to sit down with them and attach a debugger.

Arm asm isn't something I understand, and I'm not really in a
position to be able to attach a debugger to this.

Joseph

On Mon, Mar 23, 2015 at 09:33:57PM -0700, eddiebrook wrote:
>I hope it is fixed, thanks
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/libretro/mame/commit/e3eeec11f7af6725c8341aaeeca08ee70991db26#commitcomment-10359505


Suggest renaming to `defaultEnabledExtensions` and `defaultEnabledExtensionsRO` for consistency.


kimjinki


I also just pushed some unit tests to look for major issues with flux normalization in COSMOSCatalog galaxies.  So that takes care of one of my two remaining questions.


Thanks for the feedback @bogdan-dumitru. I had forgotten about the interaction between the Rails thread rendering and the pool of ExecJS contexts.

I like your suggestion of each Rails rendering thread getting one ExecJS context for its entirety but I wonder if the maintainers see any issues with that, performance or otherwise. If not, I'll try that out soon.


Use os.pathsep instead of ':' see http://stackoverflow.com/questions/5226958/which-equivalent-function-in-python


I don't understand


I just remembered... the reason I didn't use percentage was that we wanted to determine the performance difference between introducing reads while controlling for the number of write threads. This is possibly different performance-wise to a workload where two worker are both doing 50/50 read and write, given the implementation of our current networking code. But I agree a percentage makes more sense for a more generic use case.... What's your take?


Any thoughts on whether to collapse `WebSocketExtensionFactory` into `WebSocketFactory`?


Add extra exportation for pricelist
Moved before csv exportation for ID_ARTICOLO key
Add extra test for delete record
Jump empty line


Is there any technical reason for this PostResolve class? I don't see why it couldn't be implemented as a normal effect.


LFTM


Does not take PATHEXT into account.


Lines 1-5 can also be removed.


Release failed as it hit github's abuse detection check. Let me get another one running to see if it works this time.


I was thinking of doing that, but it didn't seem appropriate to have the government ending appointments.


What does this magic number mean? Can it be a class static const?


probably just remove the line?


Thanks for elaborating your thought process in the readme. That was very helpful and informative. This looks very interesting. I'm looking forward to seeing what you came up with!


LGTM


I can't use `button_to` because the classes are placed on the form, not the button.


Fixed in [61f3d5ae1343836e76901a36db1c93541cd66251].


Yes, I thought I did


They're in the general, but yeah, I assumed the ministers would need associating directly to the government but they don't currently.  I can use the "current government" steps.


Er, yep.


Suggest using `Pattern` and `Matcher` instead of `split` and `trim`.


```<svg style="height:<?php echo $this->params->get('height', 350); ?>px;width=<?php $this->params->get('width', 350)?>px"></svg>```

please add echo at the width variable i.e. ```echo $this->params->get('width', 350)```


Please insert style.css in folder css/, refactor code, also create folders for images and js files too.


I locked them down in my latest push. I had opened it up to make sure I was on the latest versions but did not think about the side-effect on the dep badges. Thanks for pointing it out.


poderia corrigir a largura do iframe para o padrão de 100%?


We can avoid this on each `doReceive()`. Since we treat `VALUE` as default value we can just make that default value by default in the filed definition.
And this `Assert.notNull` should be moved to the setter.


Why not find vsimsa instead? Also why not write a general "which" function akin to the binary on linux systems with the same name that can be re-used by modelsim and riviera classes instead of copying the code which is almost the same?


My understanding is that `Document`, `Page`, and `Post` always have a `.path` and `.dest`. But for other types of objects, `source_path` needs to be able to be nil for "The site regenerator should always regenerate objects that don't respond to :path". For the destination `source_modified_or_dest_missing?` returns false if `dest_path` is nil so that Objects that don't have a destination aren't affected by the new destination check.


Done.


What I would suggest here instead is to use logger=logging.getLogger(). This will get you the default module logger, which does not have an output handler. This allows you to not have to check for not logger, but instead just call the logger as if you have an instance in all situations.


uh i don't know.


AutoDocsWithNoLTO (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1666 of 1673 tests passed)

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


I have no idea, but it sounds better in English.


Create js file in folder js/. Put your images in folder image/. And create some new pages for links in body heads menu


So which one is the test case that will lead 'W' has 10 pieces.


emplace_back instead?


This is fine, but I would suggest using self._logger in the future to make it "protected" vs public.


Well that's irritating!


here


You 'sign on' to a project. So in a contract, you have a signing on bonus which is a bonus that you get once you commit yourself by signing a the contract.
https://www.google.co.uk/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=signing-on%20bonus%20definition


There are no issues that would cause this not to work, I just had a few suggestions for possible changes in the future.


Agree


:+1: for a good build


^^ why this change.


Will do.


FlightData lock is working. Check by placing a poco sleep 10000 after setting bestld to zero inside the lock, and LK is not crashing. Do it after Unlock, crash.
So the problem is that bestld is never assigned by line 97, and remains 0.


Ok then. But maybe it's a gag manga now, but develops into a more serious one? I don't mind on this one. You can change as you see fit.


Thanks, it's fixed.


Is it possible `server` may be null here? (like we handle below)


nope, that should be vp->vp_length


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

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


mLock won't allow it.


TeamCity xBIM :: XbimWebUI :: Build [Build 40](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1195&buildTypeId=XBIM_XbimWebUI_Build) is now running


Why is the background config item of ctrl no longer needed for computeFixedMomentsFlux?
Again, be sure that the change of coordinate systems when you call calcmom is clearly commented.
Rather than have me dig into how this code works, perhaps you could explain the following code?  The units make sense, but I am unclear about the weighting by wArea.  Is this some kind of conversion from the peak intensity to the fitted Gaussian?

+ double const wArea = afw::geom::PI*std::sqrt(shape.getDeterminant());
	+
	+ result.flux = i0*2*wArea;
	+
	+ if (ImageAdaptor<ImageT>::hasVariance) {
	+ int ix = static_cast<int>(center.getX() - image.getX0());
	+ int iy = static_cast<int>(center.getY() - image.getY0());
	+ double var = ImageAdaptor<ImageT>().getVariance(image, ix, iy);
	+ double i0Err = std::sqrt(var/wArea);
	+ result.fluxSigma = i0Err*2*wArea;
	+ }


testing only


If I read the above right: this looks like on windows both `server.emit('SIGINT');` and `server.kill('SIGINT');` will be run. This is likely harmless since as you found `server.kill('SIGINT');` does nothing on windows: however it could break in the future if `server.kill('SIGINT');` starts doing something in future node versions? What do you think about not running `if (server) server.kill('SIGINT');` on windows at all?


TeamCity xBIM :: XbimWebUI :: Build [Build 40](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1195&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:37


Yeah. Can't be helped. It doesn't sound too bad...


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1134
success: http://build.servo.org/builders/linux2/builds/1134
success: http://build.servo.org/builders/android/builds/1168
success: http://build.servo.org/builders/mac1/builds/1347
success: http://build.servo.org/builders/mac2/builds/1350
success: http://build.servo.org/builders/gonk/builds/316


I don't think there's any difference between a .tgz and .tar.gz file as its just a naming convention. My specific use case here doesn't even use the .tgz extensions I just added that for kicks and you're welcome to remove that and just giz support for the .gz which covers my .tar.gz needs ;)


I'm personally fine with just raising `ConfigBugOrBroken`. Doesn't seem like it'll be too ambiguous since there will be a message to go along with the error


@springmeyer `server.on('exit', exit);` was a regression. This was originally `server.on('exit', process.exit);`.  Hitting `exit()` is already handled via `atom.on('window-all-closed', exit);` and `atom.on('will-quit', exit);`. This is also [how it is handled in studio](https://github.com/mapbox/mapbox-studio/blob/mb-pages/index-shell.js#L33).


Can you please enlighten me why you think that the Fedora-IrpMaster needs another DecodeIRCaller than non-Fedora-IrpMaster?


Yes, good catch.


@remram44 Hi remi, hopefully you've been able to look into this as well over the weekend. So as I told you last week, there's some issue with the delete later method here.
![first_increase_to_three](https://cloud.githubusercontent.com/assets/9386084/6807801/161d449e-d227-11e4-871e-698b73ec547e.jpg)

I've added a few comments to the rowSpinBoxChanged, deleteLater and the rowSpinBoxChanged method so that I can explain this better (if not then will at least save you a couple of days of tracing things back :)).
As you see on the screen shot above, this is the behavior when you first add a third cell to the sheet. it detects the cell on row 2 column 0 to be None (which is the correct behavior).
![second_decrease_to_2](https://cloud.githubusercontent.com/assets/9386084/6807873/8afbb124-d227-11e4-897d-15e8bf8cf6ba.jpg)

This shows that the delete later method is called when we reduce the counter back to 2. This should ideally make the widget (cell Widget) to be None and therefore empty but it doesn't which can be seen from the screen shot below when I increase the counter back to 3.

![third_increase_to_three](https://cloud.githubusercontent.com/assets/9386084/6807887/99ba9dba-d227-11e4-8fdf-5002d76dfbb5.jpg)

Above image Shows output from the getEmptyCells method which should have row 2, column 0 as an None type and should include it in the list it returns back but does not. Now at this stage if you close the application or click on the clear cell button on top, it pops up the error.
![fourth_crash](https://cloud.githubusercontent.com/assets/9386084/6808052/574f8994-d228-11e4-9533-873274280356.jpg)

Now, when I reduce the count back to 2, it should call the delete later on cell row 2, column 0 but it doesn't (probably because it detects as None which is the really weird part).
![fifth_decreasing_again_to_2](https://cloud.githubusercontent.com/assets/9386084/6808080/7b6aab38-d228-11e4-97ab-42d53d49ed12.jpg)

I tried using the method that Vistrails had but it also pops up the same error. So I think setting it to None manually makes sense
 Hopefully this clears some of the doubts you had last week.

Lemme know what you think.


Got it, so the checker knows about the semantics of the entity you're trying to emit, and the emitter knows what names you are allowed to generate for it.


I thought the point of it was to be too long...?

Unless you can't fit it in the timing...


fast-forwarding master to auto = 1432b630


Nah. It's good to switch it around a bit. When talking, you don't just stick to one of the variants, at least I don't :P


Tag checkout and mvn deploy?
On Mar 24, 2015 8:27 PM, "Kohsuke Kawaguchi" <notifications@github.com>
wrote:

> Release failed as it hit github's abuse detection check. Let me get
> another one running to see if it works this time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kohsuke/github-api/commit/8939179be89020bb35c4571f3ab3257db4017d06#commitcomment-10372857>
> .
>


I think this can be done with QString itself:
QString QString::arg ( const QString & a, int fieldWidth = 0, const QChar & fillChar = QLatin1Char( ' ' ) ) const

http://doc-snapshots.qt.io/4.8/qstring.html#arg


I updated my code and pushed out a new version (0.0.7) and published the npm module.


developpeur → développeur


PR to fix upstream, https://github.com/dcm-ops/chef-slack_handler/pull/6


That's part of the public API you've removed there.

The following is a supported idiom:
```
redis.pubsub.psubscribe('prefix_a:*')
redis.pubsub.psubscribe('prefix_b:*')
redis.pubsub.on(:pmessage) { |pattern, channel, message|
  # receive all messages here
}
```


saw approval from mbrubeck
at https://github.com/pcwalton/servo/commit/3d1a5eb8ab6205965b89a9890777151f635316c2


merging pcwalton/servo/dont-copy-style-guts = 3d1a5eb8 into auto


pcwalton/servo/dont-copy-style-guts = 3d1a5eb8 merged ok, testing candidate = 2a9f29f3


Shouldn't this be data.dns.DnsException?


Done. All the best, think you for the fix. - Robert

On Tue, Mar 24, 2015 at 10:38 AM, Rodney Lopes Gomes <
notifications@github.com> wrote:

> I don't think there's any difference between a .tgz and .tar.gz file as
> its just a naming convention. My specific use case here doesn't even use
> the .tgz extensions I just added that for kicks and you're welcome to
> remove that and just giz support for the .gz which covers my .tar.gz needs
> ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rlgomes/mimetype-js/commit/160a012ddee00a9c769d3dc773138890e90388f3#commitcomment-10373121>
> .
>


`v7_val_t v7_create_number(double v)` converts double to val_t, my understanding is that it could be used here.


move to `&&`


go away, mangos!


Why the full type name here given that we have an import for the package above?  Same applies later int he same file with reference to AppointmentType.


HotFixPurchasedAirUnitTest (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1674 of 1674 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)


Removendo isso não precisa mais controlar quantos elementos tem na tela, qual o id de cada um deles, etc...
Dá pra remover mais código :)


Why full type name?  Applies to reference to FileAsMapping and ContactSource,


Fui testar o código e apareceu o seguinte erro: "ioerror: [errno 2] no such file or directory: '.notes'"...


stick?


My gut reaction is to make this use the normal rules that we use for 'let' and such other constructs. I realize the drawback here is that any nested declaration called `A` will then have to be renamed, because the outer name would take precedence. Even though when we emit a reference to an export, we will rewrite it, and thus the module parameter should have the generated name. This is kind of a shame.


Maybe it should be also clarified in the README?

Let me tell you what I understaood:

The procedure when building from distributed tarball is:
 * `make`
 * `sudo make install`
 * `sudo ./install-gtex.sh system`

And when building from git:
 * `./build.sh`
 * `sudo install.sh`

Am I right?


Ciao Marco,

per vedere un log molto grossolano, in
foowd_offerte/classes/Foowd/API.php dopo di

$output=curl_exec($ch);  (riga 70)

ti suggerisco di inserire il seguente codice:

register_error($output);

in questo modo puoi visualizzare direttamente tutto il json che viene
restituito dal servizio API in api_offerte.

In ogni caso visto che in alcuni casi ho fatto dei giri piuttosto
tortuosi, chiedi pure tutto senza troppa esitazione (specialmente per la
parte concernente gli sticky_form).
Prima di iniziare a scrivere la mia parte, vorrei sapere se ti garba
come ho pensato di gestire i json di ritornati dal servizio, o se invece
hai migliorie da proporre: avendo più esperienza di me, accolgo
volentieri i tuoi suggerimenti.


Thanks for working on this. I noticed a slight issue when it finds the same height between certain columns (or at least I assume that's what is happening) where it will pick a random one to shuffle around. I illustrated this issue by re-initializing Salvattore on window resize (just to show the issue). Perhaps a workaround for this would be to fallback to the default order if two or more elements share the same height? I may not be understanding the issue correctly though.

![height-issue](https://cloud.githubusercontent.com/assets/2846662/6808747/0a74d01c-d213-11e4-8df9-b7a87d7982a8.gif)


Sorry to intrude here but consider that exposing uninitialized data violates Rust safety contract.


Ah, thanks, I missed that -- will do!


Aw what's the matter?  You talk shit on the XDA article but you can't handle it when it happens to you on github?  Poor little baby


Don't need +1


@springmeyer going to test this now.


Built on VB.net 4.5 Framework, should work on x86 and x64.  Renames files in large batches.


Can the comment be preserved?


The background control field was always a bad idea; perhaps I should have tried harder to remove it (I did try to remove it entirely at one point, but encountered a mysteriously failing test, and then forgot about it - maybe I'll give it another try).

I basically just copied this code from the old function, so I haven't followed the math exactly; I just relied on having strong unit tests here to guarantee that I didn't change the behavior.  I don't think it's a correction from peak intensity to flux, though - I think it's probably a correction from the Gaussian-weighted zeroth-moment to flux, based on what you'd have measured if you computed the Gaussian-weighted zeroth moment of the same Gaussian you're using as the weight.  But again, I'm not entirely sure.


I don't get it. Where do I expose uninitialized data ? You can create a `Combination` struct without using its `new` method. And in case its initiallization fails, it doesn't return the struct.


No +1


Where did this file come from?  I don't see a corresponding delete?


public void haha(){}


Is the comment still valid?


HotFixPurchasedAirUnitTest (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1674 of 1674 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)


at some point yeah that might be nice. low priority


Not a huge fan of PHP as a language, but this is a clean solution and you definitely accomplished the original challenge in the kata. I love how thorough your tests are.


Good catch, the assignment should be after the if.


Any thoughts about how you might eliminate those `FIXME` areas?


@wpjunior :eyes:


I like how your solution is robust enough to even include a real SQL connection. Most solutions to this problem just hard code the values. Pulling the data in from an external source forced you to abstract it and allow for it to change.


@jeremyBass . Thoughts?


coolio


Actually it should be in the else branch...


You don't want it to emit:
```
export default let _default = (function() { ... })();
```
Instead you want:
```
export default (function() { ... })();
```


@eugeneia You are right - This happened during one of the several refactorings and slipped my attention.  Do you want to implement the suggested change?  I can review if desired.


I was under the impression that Jack Square's full name was Shounen Jack Square.
I'm fine with reverting it.


in linker.js : de `thisthis` is wss omdat wanneer je in een function `this` gebruikt hij niet wijst naar het object maar naar de function waarin je zit (een voorbeeld van dit kun je zien in `DBManager.prototype.postJournalPublication`). Dus hernoem dat gewoon als representation ofzo (ik ben nog bezig met de rest te bekijken)


@FuNiMa


I dropped this in favor of a different approach in a later commit. Either way, it should be invalid to use a class in its own computed properties due to TDZ.


@Paracel


stick to the floor. That's what people here in England say...

"The snow's starting to stick! We're going to have a snow day tomorrow!"  something like that.


Merge parent


Are we using this at all ?


```($aTopic = $this->getTopicBySlug($sUrl)) and``` добавились скобки - это тонкий юмор? :smile:  такое только в коммитах на лс можно увидеть :lollipop: и только для определенного круга читателей


This needs capitalization.


The default covers Indexer/Constructor as neither of those can be decorated.


@freeman212 can you check if this has improved with the newest version?


As of right now:

From tarball:
```
configure
make
sudo ./install.sh
```

From git:
```
./build.sh
sudo ./install.sh
```

Maybe I should drop the reference to install-gtex.sh from the changelog?  It's really there more for packagers.

Would you rather have build detect if it needs to run autoreconf and/or build fonts by the existence of key files?  Then the build instructions could be the same for either tarball or git.


We currently don't capitalize by design.


@zimbatm, it's an open-source library.


should we rename this 'goodeggs-ops-sites'?


Sorry, I messed things here. Could you port this patch to the "develop" branch? I try to never do things directly on master.


@23Skidoo what is proper protocol here when I want to address your comments? Should I push commits fixing them or should I force-update my branch?


This refactor should be merged with the previous commit. Just rebase the branch and it'll be OK.


Hello Amin. Excuse me I couldn't find another way to ask you this question. I am actually using your library but it is very slow When I am lauching multiple queries in a for loop.

Is there a way to use "transactions" in your library?


Sergio has got a point.

Developing a public API is definitely harder than doing something just for
a few projects where you can evolve as you like and have no external
interference.
The compromises must be carefully taken as the personal taste sometimes
must give space to the "greater good" of the developers.

Collections is something really important and core to any project. Getting
it right is key to the success of this project.
My opinion is that instead of simply having this discussions we should use
the decisions already taken by the .NET team.

Of course some things are specific to Java and different in .NET so it's
not just a matter of copying it from LINQ.
However overall the API decisions should hold true for this project as much
as they hold for LINQ.NET.

[]s,
Ale.

On Tue, Mar 24, 2015 at 1:17 PM, sergiotaborda <notifications@github.com>
wrote:

> Please understand the exception is raised on the incorrect return of the
> lambda. You are in violation of the contract of select many that states the
> lambda should return an IEnumerable You are returnning null, and that' s
> why you have an exception , not because the elements in the stream are null.
>
> Execute this
>
>  var arrray= new string[3]{"a", "b", "C"};
>  array.SelectMany(n => true ? null : "x").ToList();
>
> The same exception is raised and the elements are not null.
>
> Obviously the API will raise exceptions when you use it wrong, but having
> null elements in the collection is not an incorrect use as the previous
> code shows.
>
> until know your argument is : I like when stream removes nulls, so i
> remove them every where, but I do not have an alternative for when the
> programmer does not wish to remove then. I will force all programmers to
> remove nulls because I want to, even thought i'm not sure I should.
>
> Our argument is : I don't care if the stream have nulls, the APi will work
> with, our without, their presence. If the programmer cares to filter them,
> the API enables that with WHERE , and if the programmer does like nulls,
> and needs them, we shall not interfere with the programmers goal/ business
> logic. We shall not interfere!
>
> I have several use cases where null is a proper value and is important to
> business logic, so that I even produce non null view objects when a null is
> present. Hell I even groupby values that may be null, and null is a proper
> group. This stream version of "null should be auto-removed" makes this
> logic not possible and so i'm forced to not use your API and use the source
> collection directly. It is not a good trade off when the programmers
> solution is not to use your API.
>
> You say you do not known what is the ideal behavior. The ideal behavior is
> the one that does not limit the programmers options. Removing nulls
> "silently" removes options, so it is not a correct behavior.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zbra-solutions/android-linq/commit/f89360972c5a1fb9233966bd47d5094d2fc99e99#commitcomment-10372589>
> .
>


The idea is to put the message in all the queue respecting the same order, otherwise if you have 2+ servers, the message order could change if it's not done inside the lock.


I'm sorry but I don't see where. Could you show exact line which requires this include please ?


OK


This is not a good idea, as batch files are typically not generated content. (e.g. ampersand-models/VOGDemo/StartCardreader.bat) The ignore excludes all new batch files from version management, which may lead to confusing situations or even loss of data (if someone thinks his batch files are in git and removes a local working copy) Why not use a more specific ignore path?


@timothy003 odd I wonder when that was broken as it use to work and use to be ().s  - Anyway, good catch and thanks for the fix.


@iKarith

for me mame 2013 work perfect without crashes in RA old version

thank you so much for MAME core 2013 its awesome!, finally i can play the gladiator,demonfront,S.V.G. - Spectral vs Generation  y some other System 16 - PGM.

the psp core will be available in near future for ios?

core config. is available for ther plataforms or only ios is unfinished?


get  the net mask from the interface, don't just assume 0xff


@edsko Whatever is your preference. Force-updating can make old review comments disappear sometimes, though.


Voici ma liste des points a discuter/valider ce soir. N'hésitez pas a
ajouter les vôtres.

Logos des corpos : a priori a faire après que les noms aient été fixés. Il
avait été envisagé une couleur aussi.
Validation de la matrice : explication de la matrice des marches de 3 à 6
et dans quelle optique elle a été conçue.
Justification rôleplay dans les règles : a priori on met de côté.
Noms des corpos a discuter
We pour les maquettes avec Rudy : on pose une date ?
Isa : good pour les retouches sur le BG des corpos par les autres ?
Peu/pas de retours sur les règles : pas assez de temps ou tout est OK ?


Are you sure this is the correct set method ?


Maar het werkt wel in de else tak van de if? Daar staat ook een this


`gsl_combination_alloc` doesn't initialize the array it allocates and its contents can later be accessed via `as_slice`. This might not be a big deal for `[u64]` but generally you're not supposed to make uninitialized memory accessible to safe code.
The `gsl_*` methods might break too, not sure if they are guaranteed to work on an uninitialized structure.


So for block scoped variables, the pattern is that you always call makeUniqueName when you process the declaration, then call getGeneratedNameForIdentifier on every reference thereafter. And that uses blockScopedVariableToGeneratedName.

But for nodeToGeneratedName, when will you ever have the opportunity to look up a name that you already generated? If it is keyed by node id, then you would have to look up the generated name for the same node more than once. Intuitively, it seems that the emitter would only visit each node once. What is an example of multiple visits to the same node?


Oh, that was an unrelated change in the commit. I just removed the unneeded 'px' from the padding-left, because my IDE was bugging me and the margin-bottom was used to have the full height of the parent, but that was technically unrelated to the Chrome-Problem. Still, it's necessary so the failure-summary-list div doesn't end 10 pixels early.


Cool.


@JefAlive Fala meu caro! É isso mesmo... Para evitar o recarregamento da página usando submit, são duas as alterativas:
- usando `addEventListener` no elemento html desejado + `evt.preventDefault()` na função chamada
- Usar `onsubmit` no form + `return false` na função chamada


Il l'est, comme tu peux le voir, mais pour moi, MainWindow ne devrait avoir aucune données membres définissant le plateau de jeu. Tout devrait être géré par une classe héritant de JPanel, de telle sorte qu'après, on ait plus qu'à l'instancier dans le Main, créer une JFrame dedans et faire un setContentPane tout con. Je vois vraiment pas l'intérêt de surcharger JFrame et pas JPanel en fait. Et pour les transitions, avec l'ActionListenerChgScr, on a juste à l'appliquer aux boutons qu'on veut et ça change le fond de la fenêtre izy.

Et aussi, c'est quoi le fihcier "Fenêtre Option" que NJ a mis, du coup ?

Sinon, go en parler demain entre midi, c'est chaud par commentaires, là.


D'acc , je ferait les modifs par la suite , la multiplication par 17 c'est ultra simple .

Quand le fade est utilisé sur RM il est seconde , mais la commande le convertie en milliseconde : ( RPG::BGM.fade(wait * 1000) )

j'ai fait en sorte de pouvoir choisir entre un fondu calculé en seconde (donc la commande de base) ou en frame (1 frame = 16,666666 milliseconde , que j'ai arrondit a 17)


90% of the issues with replication come from CREATE operation that assign an incremental RID. If operations are executed concurrently, 2 CREATE operations could be executed in different orders (see my comments above why we need to be sure messages are ordered). 2 record created with different RIDs is an error.


did you wire this up to users and authentications?


but this is `v7_create_number`, isn't it ?


Right. Force updating seems cleaner somehow, but I was worried about comments disappearing. Ok, I'll push new commits. (Currently working on PR number 3 and number 4, so there's more to come.. sorry :)


Great :) :+1:


Sorry I've been updating this haphazardly. I haven't tried to compile it, but I've only been modifying in a new folder so it shouldn't mess with whats already there. The only problem I can think of is that DELIMITER in the query objects might need to be this.DELIMITER.


Hi HolyAngel,
Can angels and demons work together ?
Can multicarrier for vzw m8's be enabled on this ?


I agree we should follow .net spec


Maybe `new` for initializing with the first combination and `with_last` with the last would be a better match to the underlying API.


Well, it's precised in the documentation of the function. Some people may want to initialize the array by themselves so I give this possibility. But maybe adding an unsafe attribute or a warning would be a good idea ?


@khiltunen   is this change correct?


Macro fixed in later commit


+1 I think this additional clarification helps.


That doesn't seem like the right description...


Make this return the AttributeHolder so we can chain setBase calls.


tedt


I'm working on that so I can finalize the oath integration. I hope to have
that completed today.

On Tuesday, March 24, 2015, Blair Anderson <notifications@github.com> wrote:

> did you wire this up to users and authentications?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/olinares/zensheets/commit/190f3ea9256c9c1c1725373fc56c00d36cc28b4b#commitcomment-10373558>
> .
>

--
Oscar Linares
Warrior Techs
2245 Railroad Ave.
Suite 100
Pittsburg, CA 94565

925-658-8054
info@warriortechs.com
www.warriortechs.com

Computer Support | Web Design | Graphic Design


Why don't using just one queue and multiple threads that work on it?


ResultsSchema (sanyalj) - x86_64-MacOS-10.9-clang: OK (1665 of 1668 tests passed)

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


you forgot to include the text folder and text file


Side note: we may want to change the names of these classes to emphasise that they are related and the one is a more fine-grained version of the other.


> I'm not sure the exposure time and effective diameter scaling quite works for the real galaxy images. Won't the noise level still be wrong? I mean, the sky level should scale the same way as the real signal, but the noise should scale like the square root of that, right? (whereas we're advising the user to scale it linearly with these things)

I'm not sure if I understand the question here.  The noise in the RealGalaxy object refers to the intrinsic noise in the image that needs to be whitened, not the noise they should add to the final image.  So that doesn't work the way you said.

If the flux of a RealGalaxy is multiplied by x, then the noise attribute needs to have its variance scaled by x^2.  But the user shouldn't have to know about this.  It gets handled automatically now, both in the RG constructor or if they rescale the returned object themselves.


isUnknownNameInEnclosingScope?


Ceci est un commentaire.


Ojo que hemos dejado de probar que nos llegan los eventos que están en fecha.


Do you think it should be 1550 for head of household in 2014? @MattHJensen


I think you forgot "docstring = " prior to the string on a bunch of the commands, unless I'm reading it wrong...


Blablabla.


bulasrburl


Looks like you got started.


No, but then it'll have a different meaning.
'Coming by' in this case works better.


Don't forget to finish your last PR on Blocitoff, though! :feet:


That might make people think it cannot change this object, which is can. I don't think it's a good idea to make any functions in the class `const` because there's the `getApp()` back door that gets you a non-const reference to this object.


Я не сталл бы мокать Crypt т.к. он вобщем и в настоящем своем исполнении не напрягает (не отправляет почту, не ходит в базу, не выполняет запросы к другим сервисам и т.п.)


i.e. generating qualified names for exports: given a node walk up till you find containing module then take generated name for this module


There is one, i have added a constructor that allows you to create an instance from an existing DB handle.
[code]
database(sqlite3* db);
[/code]

Just open youre own handle and open the transaction on it, then construct a Database instance from that handle and enjoy.


prev line needs comma


I guess we could remove it, but it would be an extra step if we want to collect statistics from this code, which we probably will.


Oh that's good news


Of course, I can just do that instead of commenting here!


すごいです！本当にありがとうございます！


AutoDocsWithNoLTO (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (653 of 655 tests passed)

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


Ok, if this isn't logically `const`, don't make it `const`.  I just didn't want the mutex to be the reason for it being non-`const`.


make the xml parser ps4 compliant


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


This one (in master) should really be called '2.1.git' or something like that to avoid confusion.


You are right. fixed in the latest commit.


At the final version we should not use this names for vars, because they are coupled with the component name (inventory) @nestorsalceda @regiluze @APA42


Triple double-quoted strings are automatically made the docstring for a function if they are the first declaration encountered.

Help is a special case, because I needed that to be created on the fly whenever it was ran, so I just made a variable called "docstring". Technically, the help function doesn't have a docstring, but that isn't an issue because if a function doesn't have a docstring, I have it fallback to the old "/<command> ?" way of calling.


Error should be put in constants near line 103: ErrConnectionClosed        = errors.New("zk: connection closed")


Oh because export default classes cannot have names?


I think we can drop it from Changelog yes...

That would be great to adapt build.sh to that yes, this would make installation instructions easy!


paper review of the entitled research paper iPOJO: an Extensible Service-Oriented Component Framework


Did you just revert the prediction update?


import numpy got duplicated here.


Для информации: можно делать несколько testsuite, добавлять в них не файлы, а директории, и запускать с параметром
```
vandor/bin/phpunit --testsuite=имя-сюиты
```


@djmitche @sa2ajj
I'd like your opinion about this before starting to fix all the unit tests!


@rearadmiral Yeah I know, that's also where my field of view ends and I was hoping to get more feedback on this from somebody who understands things better than me :)


Yeah, just return an empty list.


Generally you want to minimize the number of includes any given class or header needs to see. This helps to avoid problems that arise from things like circular dependencies. So throwing everything into the global file could possibly make things worse not better.

I'm not sure what exactly the problem was, but I would try converting all the includes in header files to forward references, if possible.


Ага, увидел комментарий. Понял зачем мокается Crypt.


Let's call it reduceLeft and reduceRight to match javascript


Awesome! Thanks!


It seems to me that the purpose of allocating an uninitialized combination is to then read it from a stream or initialize it to the last combination. It wouldn't be user-friendly to require unsafe for that but could be handled with alternative constructors.


finished demonfront in the new mame core 2013 whithout problems in RA old version stable

i think the core is fine, so the problem is RA 1.1 app something dont liked.


No, I mean the calls to `np.cast['float64]` and `float(...)` on line 441.
I think the point of the ticket was to keep `self.num_examples` a list of integers.


Great, thanks.


would it be more efficient to check length() before the getException() call, or do you have to get it first?


mempak.h and rumblepak.h are already included through game_controller.h so not needed here.

Note: My recommendation (even if not stated properly, sorry for that) is to include the header which gives the adequate level of abstraction for a given task. For example here,
the game_controller "abstract" which pak is connected to it, so that other components {pif or si_controller for instance} that don't need to care about that won't know about that. Another example, game_controller shouldn't know about the si_controller, because the PIF abstract the former from the latter.
Not sure if it makes sense, but if you draw a chart with each components and connections, you should see which component needs to know about which one.


The purpose was to let the choice to users. Only a very few people will use the `new` function but at least they can.


If it's not a default export


This is compatible with today's database dump: 240315.gz


Ok last step, move this in with all the others. This is not a special type of easing, it's just an easing function like all the others :grinning:


This kind of change is bad to review. You shouldn't change indenting while fixing a issue.


those means change all
```
    thing.master = self.master
```
by
```
   thing.setServiceParent(self.master)
```


@cyx -- the example in the readme shows the old usage.  Was a bit confusing until I tracked down this commit.


Is this in the correct plugin? RSS doesn't depend on Cheerio, it's the highlight plugin that might.


And a list of applicable `Attribute`s would be cool.

`ZOMBIE` attributes like `REINFORCEMENTS` don't make much sense on `HORSE`s, but is there anything that actually prevents that?


I don't like this for two reasons:
1) Nobody would or could ever have a const reference to this object.
2) If `const` isn't a promise not to modify the object, what purpose does it serve?
Remember, these functions call functions that get non-const references to this object through `getApp().getInboundTransactions()`.


The .desktop file in italian first  said:
GenericName[it_IT]=Esci
-Name[it_IT]=Esci dal sistema
this means "exit" and "exit system", nothing about notifications settings...


You are correct, of course!


Reverted.


Way to go!


It's the size of a nodeID and it's unchanged in this code, just moved from one place to another.


Aha! Thanks for clarifying. Scheduled for 0.4.0


Of course it is


I should have elaborated on these `const` comments:  There may be good reasons for making some (or all) of the member functions non-`const`.  But needing to lock/unlock the mutex isn't one of them.  If these functions allow logical (and observable) modifications to the object, don't make them `const`, else do make them `const`.


Can you change this so that the url is absolute instead of relative. You should use `request.is_secure` for http/https and use django Sites to get the domain name.


You can also add this image to the `hindustani` and `makam` apis


Does this cause the name to not be double bound anymore?

Also, why do you need to make a lambda closure over it, instead of just doing these operations in the outer scope? How does this prevent someone from seeing the undecorated state in the interim?


Review Complete


Not stupid at all!  cbt is still very rough.

Here's how I set my ceph.conf:

[mon]
        mon data = /tmp/cbt/ceph/mon.$id

[osd.0]
        host = <host>
        osd data = /tmp/cbt/mnt/osd-device-0-data
        osd journal = /dev/disk/by-partlabel/osd-device-0-journal

with static definitions for each, but I build my ceph.conf files automatically using this tool:

https://github.com/ceph/cbt/blob/master/tools/makecephconf.py

That tool will let you create many ceph.conf files by sweeping over parameter spaces.  An example configuration file will make several ceph.conf files:

settings:
    osd_servers: [burnupiX]
    osds_per_server: 8
    outdir: "/home/nhm/data/sharding"
    runtests_exec: "/home/nhm/src/ceph-tools/cbt/cbt.py"
    runtests_conf:
        xfs: "/home/nhm/src/ceph-tools/cbt/sharding/runtests.xfs.yaml"

default:
  global:
    osd_crush_update_on_start: "false"
    osd_crush_chooseleaf_type: "0"
    osd_pg_bits: "10"
    osd_pgp_bits: "10"
    osd_pool_default_size: "1"
    auth_cluster_required: "none"
    auth_service_required: "none"
    auth_client_required: "none"
    keyring: "/tmp/cbt/ceph/log/$name.log"
    log_to_syslog: "false"
    log_file: "/tmp/cbt/ceph/log/$name.log"
    rbd_cache: "true"
    filestore_merge_threshold: "40"
    filestore_split_threshold: "8"
    osd_op_threads: "8"
    mon_pg_warn_max_object_skew: "100000"
    mon_pg_warn_min_per_osd: "0"
    mon_pg_warn_max_per_osd: "32768"
  mon.a:
        host: "burnupiX"
        mon_addr: "192.168.10.1:6789"
        mon_data: "/tmp/cbt/ceph/mon.$id"

parametric:
  osd_op_num_shards: [1, 5, 10, 20]
  osd_op_num_threads_per_shard: [1, 2, 3, 4]


Ah I see. Is that the only example?


@ericclemmons You forgot to call `this.queue(null)` ... This is the reason for #46


Não tinha visto que já tinha uma descrição aqui. Vou dar o merge.


Sorry, I was trying to add some methods


@ST-DDT Attribute#getTargets is used to decide if an attribute can be applied.


This should say `up`, `down` or `change` methods.


Wow, that's embarrassing, will fix this shortly.

> On 25/03/2015, at 03:14, mabako <notifications@github.com> wrote:
>
> Shouldn't this have a return false somewhere?
>
> —
> Reply to this email directly or view it on GitHub.
>


This doesn't seem to work. The server is receiving  Hindi queries.


WOOOOOP


Technically this is unsafe. With constructor variants that avoid the redundant `calloc` I believe it just isn't needed.


Yes, okay, I see what you mean.


The created vote will also need to be associated with the post too. That would make line 40:

```ruby
  user.votes.create(value: 1, post: self)
```


The above should be `self.view.notify`.


Saleem, you've added so many things that most of us don't understand. We don't understand some of the functions like: `self.parse()`, `self.init_file()`, `self.write_file()`, `self.flatten()`. You're adding regular expressions which we don't know how to use in Python. So could you actually explain what you're adding and **how** it works not just what it does.


Same here.


With the following :

* the support for ```global vlan``` and actually I'd like to keep it.
* the file format is too specific

My guess is that with an id and a site we get an ip range :
So maybe something like the following is more appropriate :

```yaml
bordeaux:
  4: 10.0.0.0/18
  5: 10.0.64.0/18
[...]
nantes:
  4: 10.44.0.0/18
  5:10.44.64.0/18
[...]
```


This was not actually a bug, but intended behaviour (possibly badly designed). For proband, it always overwrote whatever is in the pedigree by what was stored in patient data. The idea was that very soon all the data would be pushed back into the patient once pedigree is saved, thus there will never be any inconsistencies, and on load need to update pedigree with the version of patient data stored on patient page, which may include not only additions but deletions as well.


Yes, this is. Your idea to to have constructors `new_init_first` and `new_init_last` is good. I'll add it when I'll work again on it. Thanks for your proposition !


With the change suggested [above](https://github.com/zein1016/bloccit/commit/c35daac8160c00557fcc43ec195fde4c29db22ea#commitcomment-10374084), then `post` on line 12 here ought to no longer be nil as reported in the error:

```
NoMethodError at /topics/3/posts
undefined method `update_rank' for nil:NilClass
```


Aside from making some of these conditional checks based on caring what OSs the thing might run on i.e. the d3dcompiler files or pdf plugin yes or no, it looks like it makes sense.


Right now, README.md directs the user to the website.  The audience for the README.md is probably too wide to describe install instructions for Linux, unless we intend to put all the installation instructions there.


@hpi1 are you interested in such a feature in general? It's only a poc at the moment.


I don't think self['from']) is always defined. The javascript version seems to do something like:
`Position from = self.from != null ? new Position.fromProxy(self.from) : new Position.fromProxy(data["from"]);`


Revert this commit and properly merge.


Shouldn't we have to also check if it's not 'both3' or 'both4'. Because in this case, MPTCP and TCP are totally differents.


@hpi1 ping


A better solution is needed.


delete


Which settings did you change? Look at these:

   - Settings > Language and Input > Language > Select Hindi
   - Settings > Language and Input > Voice input > (click the gear icon
   next to "Enhanced Google services") > Languages > Check Hindi
   - Settings > Language and Input > Text-to-speech output > (click the
   gear icon next to "Google Text-to-speech Engine") > Language > Select "Use
   System Language"

Let me know if it works. If not, try to debug and see where it's going
wrong. Basically I'm calling TranslatorUtils.delocalize(String) to
translate to English, which internally calls Locale.getDefault(), which
returns the default system locale.

On Tue, Mar 24, 2015 at 11:54 PM, Subham Gupta <notifications@github.com>
wrote:

> This doesn't seem to work. The server is receiving Hindi queries.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jarvis-app/jarvis-android/commit/17ee7826812a4761dd5eb48e796bae6b89d5c1d1#commitcomment-10374026>
> .
>

--
Vicky Chijwani
http://vickychijwani.me


Right, I'm a bit too Linux-centered... let's forget about the README.


OK.


@DennisLfromGA good link here: http://www.regular-expressions.info/posix.html

A friend of mine, Jeffrey Friedl, wrote a book on regex, (O'Reilly), but not sure how updated it has been.


Oh, fair enough.


@mafagafogigante No worries at all. :smile:


add new line


TeamCity xBIM :: XbimExchange :: Build [Build 136](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1196&buildTypeId=XBIM_XbimExchange_Build) is now running


r+


I am dmorton, you want another of your team members
On 24 Mar 2015 11:02 am, "mkiwala" <notifications@github.com> wrote:

> Don't force concurrency=1 on fork-worker testenv
>
> The original motivation for --concurrency 1 was a suspicion that logging
> was not working as expected on @mkiwala's workstation due to the default
> concurrency settings.  The suspicion did not hold after testing on
> @dmorton's workstation.  So the non-default concurrency settings should
> not be forced.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mkiwala/ptero-shell-command/commit/00e715224e6012c2e59442c94c6168afc4cc4c27
>


TeamCity xBIM :: XbimExchange :: Build [Build 136](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1196&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Compilation error: Xbim.COBieLite.SerialisationHelper\SerialisationHelper.csproj Build time: 00:00:21


Seems this is not blizzlike (wotlk) this is something for cata or mop....


Still work in progress. There may be a better solution.


Could you put the ` /` after the `if-else` block?


add new line to conform with the style guide


should there be a warning or something here? Just concerned about what happens if they make major changes and lots of these start failing.


Instead of `element` call it `container`.


XD


Is this ever not true?


Hi. Is this supposed to be new release? The version is bumped, but the new release is not tagged (therefore not picked by the F-Droid autochecker).


Yes it can, was just waiting for the source to drop, thanks for the
reminder ^^
On Mar 24, 2015 1:00 PM, "Holl0wHeart" <notifications@github.com> wrote:

> Hi HolyAngel,
> Can angels and demons work together ?
> Can multicarrier for vzw m8's be enabled on this ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/holyangel/M8-Sense_L/commit/425e678f9c7198473376fcf6be60e2d528aaa43e#commitcomment-10373572>
> .
>


LGTM. Yes, it probably makes sense to replace `InstalledPackageId` with this to cut down on the number of various identifier types.


these changes look really cool!!


It would be beneficial to have a commit message with more substance when reverting something that was merged only a day previously.  (as in - what is the rationale for this revert?)


Ah, nice!


You left a pdb here...


echt super!!!


Thanks for following up on this.


Oh, so this is an XML and won't be run through Django's template processor. Those static functions won't work.


sadly these changes have broken the mame core for android...


Seems like we're using the goodeggs- prefix to signal that it's an interal
repo, so yes?

On Tue, Mar 24, 2015 at 10:55 AM, Ben Buckman <notifications@github.com>
wrote:

> should we rename this 'goodeggs-ops-sites'?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goodeggs/ops-sites/commit/0deba0c9be233165f73427c0d5c23f3cdbce379a#commitcomment-10373487>
> .
>


There is information in cmdline that should not be world readable. this solution is not good as it is.maybe 640 permission or services runing under different users/groups


Az API-ban J, Q, K és A-kén szerepelnek a 10 feleti rankok.


i think this could be `$( document ).on( "widget-added, widget-updated" )...` which would save the duplication


It was not tagged intentionally: this version is in beta right now: I have distributed an APK for interested users, but I want to make sure that there are no bugs before branding it as the "release" 1.1 version.
I should probably have created a separate branch to make this more apparent!


Newline.


Ok. will test with these settings.


This text is not quite right for wfirst.


Просматривая предыдущий коммит как раз об этом же думал


@seanherron Says that browserconfig.xml is only used by Microsoft Surface Tablets and Windows 8 to create a tile in the interface. So, it might not be required.

If we really want to have this, we can do it directly in the template.


The use of "STATE_FLAG_CHECKED" require gtk 3.14
https://developer.gnome.org/gtkmm/unstable/since_3_14.html#_since_3_14000013


Mkay, in that case I'll do it in a separate PR.


I'd like it to be a caching property, at least -- just about everything "interesting" involves `self.master`, so making that a recursive function seems suboptimal.

I'm not sure what problem this is solving, either -- do you often run into cases where an object has no `master` attribute?


Cheers :)


@archSeer I think I should split this into 2 commands though, one for simply searching (like the google command) and one for definitions / abstract texts.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1135
success: http://build.servo.org/builders/linux2/builds/1135
success: http://build.servo.org/builders/android/builds/1169
success: http://build.servo.org/builders/mac1/builds/1348
success: http://build.servo.org/builders/mac2/builds/1351
success: http://build.servo.org/builders/gonk/builds/317


fixed in a later commit.


Makes sense...  A good question would be what the original commit was trying to "fix" - just like this one, it's pretty uninformative.

A Gerrit instance would be really useful here.  :)


exp.getException() could be null


fast-forwarding master to auto = 2a9f29f3


Note that this one was a duplicate, there is still ogre.cfg in .gitignore


the type bool in line 1997 (MPU6050.cpp) is different from uint8_y in line 611 (MPU6050.h). It seems to create an error during compilation. Could someone help me with this issue? Thanks


I think there was a problem with the CARRIER_SENSE_VALID being set to 1 even if the RSSI wasn't ready, so we needed to wait for some time before reading it out.


Good idea.


saw approval from jdm
at https://github.com/snf/servo/commit/89c208d3e41bf5fa46341bc1bf01371f66ccfa4e


merging snf/servo/storage_event = 89c208d3 into auto


snf/servo/storage_event = 89c208d3 merged ok, testing candidate = 2c51d0ef


Sounds like it would be a useful addition.


Can you use ```this.options.contentWindow``` here please? Other than that awesome job :+1:


https://docs.python.org/3/library/cgi.html#cgi.FieldStorage.getfirst - more simple


The configuration is intended to be 802.15.4g-compatible, so that's probably were this setting comes from. But I'm not 100% sure about this one.


comment is out of date!  Don't forget to remove it


Doki, azt hiszem gond van a fluxuskondenzátorral!

Ezen még csisszolunk :)

Lehetne enum a flop, turn, river-re illetve itt is elég jól el lehet lőni a rule-os mintát.


Fix in 60c1235.


if  bestld is never assigned you have invalid polar...


I think this might fix issue #14. Anyone else?


Could you perhaps explain your plans for the PQS-generation structure, i.e. the classes and functions? Should they be all in PQSLoader.cs or should there be a class for every PQSMod? :) (Or am I stuck to my "write all down" system?)


`go get` will download the package, but won't actually install the `httpdiff` binary in `$GOPATH/bin`.

How about `go install github.com/jgrahamc/httpdiff/src/httpdiff` instead?


AutoDocsWithNoLTO (kbenne) - x86_64-MacOS-10.9-clang: OK (1660 of 1667 tests passed)

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


A bet érték az eddig betett összes zseton, s nem azt adja meg, hogy ebben a körben mennyit tett be. Itt nekem az a benyomásom, hogy azt szeretted volna. De lehet tévedek.


do we really want to keep the bro? hehe


> ```c#
var rk = Registry.CurrentUser;
if (rk.OpenSubKey(TestKey) != null)
   rk.DeleteSubKeyTree(TestKey);
if (rk.GetValue(TestKey) != null)
 rk.DeleteValue(TestKey);
```
I understand this is an existing code but we already clean up things in Dispose(). If the ctor wants to check the state before running the test then I think it should fail in case it finds the key, because in that situation most likely the test is running on a system that utilizes this particular key for some reason that has nothing to do with this test and we would not want to erase that state.
However since this is an existing code, feel free to create a separate issue for this :).


Any reason not to defer to `extensionFactory.getExtensionNames()` here?


Looks like you're still using data3 inside the data4 score, which is not what you want.


miért került ide a package is, importba érdemes tenni, akkor rövidebb lesz a sor :)


I don't think that rules for 'let' can be applied here - it is ok for 'let' to be shadowed and have a different meaning in the inner scope, however for module parameter name we need it to have the same meaning in all descendant scopes.


rax retry tests


думаю, что нужно дополнить комментарий чтобы было более понятно что за "кеш орм" словом
>Сбрасываем отдельный кеш **метаданных** ORM
чтобы не было мысль что сбрасывается кеш данных из бд, полученных с помощью орм.


sure :)


Use a const for 'test1' and other similar code below.


Yes, I believe the docs imply that this field is RO (once created):
http://docs.vmd.citrix.com/XenServer/6.5.0/1.0/en_gb/api/?c=network

(Not that those docs are particularly clear!)


This breaks build.
packages/apps/Bluetooth/jni/com_android_bluetooth_btservice_AdapterService.cpp:1121:47: error: 'const struct btsock_interface_t' has no member named 'set_sock_opt'
     if ( (status = sBluetoothSocketInterface->set_sock_opt((btsock_type_t)type, channel,
                                               ^
packages/apps/Bluetooth/jni/com_android_bluetooth_btservice_AdapterService.cpp:1122:11: error: 'btsock_option_type_t' was not declared in this scope
          (btsock_option_type_t) optionName, (void *) option_val, optionLen)) !=
           ^


I know but in this moment we have to live with git only. Initial commit was triing to fix ril connected services access


Does this comment need changing now?


Yes, but that is an artifact of the fact that a reference to an export can be unqualified. The user can completely omit the name, and we have to insert it for them. That's why it has to have the same meaning in all scopes. The user can in fact shadow the module. But shadowing a module does not excuse us from ever having to refer to that module, whereas shadowing a 'let' does.


is it posible to stop using xml and migrate to new 2.0 api.?


Pour conserver la session PHP, il suffit de décommenter la ligne :
```javascript
	$httpProvider.defaults.withCredentials = true;
```
dans js/config.js
Comme quoi, c'est magique Angular


since the raspberry pi v2 is out, and it uses a armv7 cpu, we probably want to include that as well


What does the ", 10" do here? I don't see a third param to .css() in the jQuery documentation.


Sorry, I wasn't clear. I mean the XML-RPC return value - I believe it should contain a failure message (that you would find in the result object). If the credentials are invalid, it would be something like 'SESSION_AUTHENTICATION_FAILED' I believe.


elasticsearch has parameters for batch start and size. Ideally we should probably use those rather than fetching the entire resultset from elasticsearch every time. http://www.elastic.co/guide/en/elasticsearch/reference/current/search-request-from-size.html


Thanx Ted.


One drastic option could be to always rename the module parameter, in case anyone shadows it inside.


These could benefit from some docstrings.

Making sure I understand what is going on.

This updates the bardata dict with the latest `vals` of each of the `sids`?

And sid and vals are shaped so that they line up?


We should delete the value in the finally block or we might never get to it :).


@ugohil-r7 Probably, this change is just meant to unblock @rtaylor-r7 since this was patched in the internal extensions gem and not in the gem yet. We should consider doing that work on the `site-api` branch.


Erre a részre nagyon szépen alkalmazható a [Factory pattern] (http://www.tutorialspoint.com/design_pattern/factory_pattern.htm).


The reporter verified that after this change, the problem disappeared. I confirmed the same effect, too.