@chriscool OK, thanks for reviewing. Here's the new outline:

* Intro
* Discussions
  * General
    * Some discussions (unless otherwise specified, these are from *our* mailing list)
  * Reviews
    * Same pattern here
  * Support
    * Same pattern here
* Developer spotlight interview
* Releases/Announcements
  * Git release
  * Tig release
  * JGit release
* Other news
  * Various links go here
* Credits

I am a bit tempted to suggest moving the short news sections near the top of the mail, as this will increase the digest-nature of the news letter. If not, perhaps we should have some table-of-contents near the top that says that news and links will be found further down below.


I would assume so, thanks, I'll fix that.


especially this method really needs a good doc comment


why should it? let's say your timestep is 0.1 and the annual cost is a constant 1e6. then:
plotdata['annual']['total']['total']['ylinedata'] = [1e6, 1e6, 1e6, ....]
plotdata['cumulative']['total']['total']['ylinedata'] = [1e5, 2e5, 3e5 ...]
and the 0th element of each isn't equal...?


While I applaud the push to add Serializable to Factorie's classes (just had a pull request committed for serializing classifiers), I just want to point out that we might need to be cautious in making things like document annotators serializable, because this might result in large resources (e.g., lexicons) getting serialized across the wire or to disk.  Transitive closure can be a real PITA.

There are likely cases where you will need to override the default deserialization code to handle cases where resources need to be re-created on the far end.  Note that when objects are deserialized, the usual constructor is not called (this includes any code at the class level). This has come up in our serialization of Factorie objects, and I had to do:

```scala
/**
 * Deserializes a model from an object stream, setting the transient variables
 * to something sensible.
 */
@throws(classOf[IOException])
private def readObject(in: ObjectInputStream): Unit = {
    in.defaultReadObject();
[class specific setup stuff here]
}
```
So, in summary: Serialization is awesome, but watch out for the weird cases.


just run this:
```python
import numpy
a = numpy.arange(10)
a[1:] = a[:-1]
```

and

```python
import numpy
a = numpy.arange(10)
for i in range(9):
    a[i+1] = a[i]
```

the answer will appear :-) this should remind you of some teachings I gave you ;-)


You may want to name those macros with `FULL_UPPERCASE`, add a prefix (like `PRETTY_PRINT`), to their names and generalize them to take `ostream`s as arguments, but if they aren't just for testing, you should just remove them and put them in testing files, where appropriate (those names will screw a LOT of codebases up).


They had a major redesign a few months ago. Right now the CocoaPods repo keeps processed pod descriptions in a json form, and you don't submit updates to it via pull request, but rather using a command-line tool that contacts their server that handles this automatically. So you keep a podspec file in your repo, and then when you want to make a release, you add a tag, update the version number in the podspec, and call something like "pod push foo.podspec" (I don't remember the exact syntax).


@waster using native HLS or contrib-hls? Would you mind filing an issue so we can track it better?


Ups, I thought this is a Singleton in EventDispatcher... thats wrong...


@brummett taught me that.  `!!` is good if you are ok with `1` and `undef` being returned.  If you want `1` and `0` you have to use ternary or whatever.

```
$ perl -Mv5.10 -e 'my $f = sub { !!"foo"}; say $f->();'
1
$ perl -Mv5.10 -e 'my $f = sub { !!""}; say $f->();'

```


Возможно стоит сделать публичным OnPropertyChanged, но в моем случае необходимо вызывать метод не указывая свойство, т.к. не известно, изменилось ли что-нибудь, а обработчик события вызвать все равно надо.


Congratulations!!


tested successfully


Corrigé dans f81f773ee1fd9eef758d87adae2385ad2d77f1b9


Please don't make change without issuing a PR (even for something very simple like this)...


What do you think of "graph-draggable-tools"?


Agree. Do you want to say something or this is a day of sharing obvious facts?


Ok about the new outline and the table of contents near the top.


cool :shipit:


Yes, you are right, this (creation of the file etc) gets handled when I instantiate HTMLWriter, so this part is redundant and could be removed.
Is removing (scraping it entirely) ```__del__``` advisable , given that all our files have it, or do we just want a ```return``` when its invoked?


Mind adding a comment in the source linking to the issue, just so future readers can be in-the-know?


Good idea!


1: D400 First line should end with a period (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 in imperative mood ('Return', not 'Returns')
41: D209 Multi-line docstring closing quotes should be on a separate line
41: D205 1 blank line required between summary line and description (found 0)
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 in imperative mood ('Return', not 'Returns')
80: D400 First line should end with a period (not 'm')
86:30: E261 at least two spaces before inline comment
95: D401 First line should be in imperative mood ('Return', not 'Returns')
95: D209 Multi-line docstring closing quotes should be on a separate line
95: D400 First line should end with a period (not 'n')
95: D205 1 blank line required between summary line and description (found 0)
104: D401 First line should be in imperative mood ('Find', not 'Finds')
104: D400 First line should end with a period (not 'g')
104: D300 Use """triple double quotes""" (found '''-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 Multi-line docstring closing quotes should be on a separate line
146: D400 First line should end with a period (not 'y')
146: D205 1 blank line required between summary line and description (found 0)
146: D300 Use """triple double quotes""" (found '''-quotes)
157:68: E502 the backslash is redundant between brackets
158:24: E225 missing whitespace around operator
169: D401 First line should be in imperative mood ('Return', not 'Returns')
169: D400 First line should end with a period (not 'd')
169: D300 Use """triple double quotes""" (found '''-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 Multi-line docstring closing quotes should be on a separate line
187: D400 First line should end with a period (not 'y')
187: D205 1 blank line required between summary line and description (found 0)
187: D300 Use """triple double quotes""" (found '''-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 in imperative mood ('Return', not 'Returns')
199: D205 1 blank line required between summary line and description (found 0)
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 a period (not 'n')
238: D300 Use """triple double quotes""" (found '''-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 in imperative mood ('Identifie', not 'Identifies')
256: D400 First line should end with a period (not 't')
263:1: E302 expected 2 blank lines, found 1
263: D401 First line should be in imperative mood ('Doe', not 'Does')
263: D209 Multi-line docstring closing quotes should be on a separate line
263: D400 First line should end with a period (not 'e')
263: D205 1 blank line required between summary line and description (found 0)
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 a period (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 a period (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 Multi-line docstring closing quotes should be on a separate line
365: D205 1 blank line required between summary line and description (found 0)
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 Multi-line docstring closing quotes should be on a separate line
387: D400 First line should end with a period (not 's')
387: D205 1 blank line required between summary line and description (found 0)
392:54: E502 the backslash is redundant between brackets
407:66: F821 undefined name 'six'
427: D401 First line should be in imperative mood ('Return', not 'Returns')
427: D209 Multi-line docstring closing quotes should be on a separate line
427: D202 No blank lines allowed after function docstring (found 1)
427: D400 First line should end with a period (not 'r')
427: D205 1 blank line required between summary line and description (found 0)
433:24: E261 at least two spaces before inline comment
433:25: E262 inline comment should start with '# '
442:73: E502 the backslash is redundant between brackets
443:51: E225 missing whitespace around operator
452:63: E502 the backslash is redundant between brackets
453:49: E225 missing whitespace around operator
458: D401 First line should be in imperative mood ('Return', not 'Returns')
458: D400 First line should end with a period (not 's')
460:53: E502 the backslash is redundant between brackets
464: D401 First line should be in imperative mood ('Return', not 'Returns')
464: D209 Multi-line docstring closing quotes should be on a separate line
464: D400 First line should end with a period (not ':')
464: D205 1 blank line required between summary line and description (found 0)
473:13: F841 local variable 'dummy' is assigned to but never used
478:27: E261 at least two spaces before inline comment
478:28: E262 inline comment should start with '# '
483: D401 First line should be in imperative mood ('Return', not 'Returns')
483: D209 Multi-line docstring closing quotes should be on a separate line
483: D400 First line should end with a period (not ',')
483: D205 1 blank line required between summary line and description (found 0)
495:75: E502 the backslash is redundant between brackets
496:23: E128 continuation line under-indented for visual indent
496:24: E225 missing whitespace around operator
510:72: E502 the backslash is redundant between brackets
511:69: E225 missing whitespace around operator
513:83: E502 the backslash is redundant between brackets
514:23: E128 continuation line under-indented for visual indent
514:24: E225 missing whitespace around operator
521: D401 First line should be in imperative mood ('Thi', not 'This')
521: D209 Multi-line docstring closing quotes should be on a separate line
521: D400 First line should end with a period (not 'd')
521: D205 1 blank line required between summary line and description (found 0)
528:5: B901 blind except: statement
532:1: E302 expected 2 blank lines, found 1
532: D401 First line should be in imperative mood ('Thi', not 'This')
532: D209 Multi-line docstring closing quotes should be on a separate line
532: D400 First line should end with a period (not 'd')
532: D205 1 blank line required between summary line and description (found 0)
547:20: E261 at least two spaces before inline comment
551:1: E302 expected 2 blank lines, found 1
551: D401 First line should be in imperative mood ('Thi', not 'This')
551: D209 Multi-line docstring closing quotes should be on a separate line
551: D400 First line should end with a period (not 'e')
551: D205 1 blank line required between summary line and description (found 0)
561:1: E302 expected 2 blank lines, found 1
561:59: E251 unexpected spaces around keyword / parameter equals
561:61: E251 unexpected spaces around keyword / parameter equals
561: D401 First line should be in imperative mood ('Update', not 'Updates')
561: D209 Multi-line docstring closing quotes should be on a separate line
561: D202 No blank lines allowed after function docstring (found 1)
561: D400 First line should end with a period (not 'd')
561: D205 1 blank line required between summary line and description (found 0)
572:69: E502 the backslash is redundant between brackets
573:46: E225 missing whitespace around operator
582:75: E502 the backslash is redundant between brackets
583:24: E225 missing whitespace around operator
584:21: E261 at least two spaces before inline comment
584:22: E262 inline comment should start with '# '
588:83: E502 the backslash is redundant between brackets
589:24: E225 missing whitespace around operator
603:14: E261 at least two spaces before inline comment
617:60: E502 the backslash is redundant between brackets
618:28: E225 missing whitespace around operator
621:69: E502 the backslash is redundant between brackets
622:33: E128 continuation line under-indented for visual indent
624:60: E502 the backslash is redundant between brackets
625:28: E225 missing whitespace around operator
627:69: E502 the backslash is redundant between brackets
628:33: E128 continuation line under-indented for visual indent
630:59: E502 the backslash is redundant between brackets
631:28: E225 missing whitespace around operator
641:68: E502 the backslash is redundant between brackets
642:42: E128 continuation line under-indented for visual indent
650:85: E502 the backslash is redundant between brackets
651:28: E225 missing whitespace around operator
656:117: E251 unexpected spaces around keyword / parameter equals
656:119: E251 unexpected spaces around keyword / parameter equals
656: D401 First line should be in imperative mood ('Update', not 'Updates')
656: D400 First line should end with a period (not 's')
660:62: E225 missing whitespace around operator
665:64: E502 the backslash is redundant between brackets
666:24: E225 missing whitespace around operator
687:17: E125 continuation line with same indent as next logical line
703:72: E502 the backslash is redundant between brackets
715:66: E502 the backslash is redundant between brackets
716:21: E128 continuation line under-indented for visual indent
720:66: E502 the backslash is redundant between brackets
721:21: E128 continuation line under-indented for visual indent
722:64: E225 missing whitespace around operator
725: D401 First line should be in imperative mood ('Modifie', not 'Modifies')
725: D209 Multi-line docstring closing quotes should be on a separate line
725: D400 First line should end with a period (not 'n')
725: D205 1 blank line required between summary line and description (found 0)
734: D401 First line should be in imperative mood ('Insert', not 'Inserts')
734: D400 First line should end with a period (not 's')
743:50: E261 at least two spaces before inline comment
743:50: E262 inline comment should start with '# '
749:34: E261 at least two spaces before inline comment
749:35: E262 inline comment should start with '# '
756:23: E261 at least two spaces before inline comment
756:24: E262 inline comment should start with '# '
763:38: E225 missing whitespace around operator
767: D400 First line should end with a period (not 's')
775:38: E225 missing whitespace around operator
779: D209 Multi-line docstring closing quotes should be on a separate line
779: D400 First line should end with a period (not 'w')
779: D205 1 blank line required between summary line and description (found 0)
789: D400 First line should end with a period (not ')')
806: D401 First line should be in imperative mood ('Update', not 'Updates')
806: D209 Multi-line docstring closing quotes should be on a separate line
806: D202 No blank lines allowed after function docstring (found 1)
806: D400 First line should end with a period (not 't')
806: D205 1 blank line required between summary line and description (found 0)
816:66: E502 the backslash is redundant between brackets
817:59: E502 the backslash is redundant between brackets
818:20: E225 missing whitespace around operator
844:27: E261 at least two spaces before inline comment
851:61: E225 missing whitespace around operator
852:71: E225 missing whitespace around operator
855:82: E502 the backslash is redundant between brackets
856:28: E225 missing whitespace around operator
861:32: E261 at least two spaces before inline comment
865:28: E261 at least two spaces before inline comment
865:29: E262 inline comment should start with '# '
868:82: E502 the backslash is redundant between brackets
869:27: E128 continuation line under-indented for visual indent
871:84: E502 the backslash is redundant between brackets
872:19: E128 continuation line under-indented for visual indent
872:20: E225 missing whitespace around operator
877:56: E225 missing whitespace around operator
878:58: E225 missing whitespace around operator
881:87: E502 the backslash is redundant between brackets
882:27: E128 continuation line under-indented for visual indent
882:28: E225 missing whitespace around operator
887:38: E251 unexpected spaces around keyword / parameter equals
887:40: E251 unexpected spaces around keyword / parameter equals
887: D401 First line should be in imperative mood ('Rebalance', not 'Rebalances')
887: D400 First line should end with a period (not 't')
899:58: E225 missing whitespace around operator
901:23: E261 at least two spaces before inline comment
903:62: E502 the backslash is redundant between brackets
904:24: E225 missing whitespace around operator
910:56: E225 missing whitespace around operator
911:70: E502 the backslash is redundant between brackets
912:27: E127 continuation line over-indented for visual indent
912:28: E225 missing whitespace around operator
914:33: E128 continuation line under-indented for visual indent
915:33: E128 continuation line under-indented for visual indent
916:33: E128 continuation line under-indented for visual indent
918:72: E502 the backslash is redundant between brackets
919:28: E225 missing whitespace around operator
927: D401 First line should be in imperative mood ('test', not 'tests')
927: D400 First line should end with a period (not 's')


I'd Like the complete message to be colored, I am not sure how to resolve this, its an aesthetic choice :)


Corrigé dans  9d214a9


That could be reverted though. Do you need help getting it out of there?

Oh, 3am pushing code? I guess that explains it. :)

On Fri, Mar 27, 2015 at 2:59 AM, Tres Finocchiaro <notifications@github.com>
wrote:

> @robertcasto <https://github.com/robertcasto>, et.al , I hit a merge
> conflict (my own fauls) and some jetbrains stuff got pushed. Sorry about
> that.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/qzind/qz-print/commit/2ba0832f012b9e084b11466678654fe17088c1ef#commitcomment-10426700>
> .
>

--
Robert Casto
*Owner | SellersToolbox*
p: 1-513-847-4942
c: 1-513-886-1550
e: casto.robert@gmail.com
www.sellerstoolbox.com
Tools for Amazon Sellers


I don't mind ;)


Nice example, but I think the arguments to extend should be reversed here, as the later takes predecence on duplicate keys. Not that it matters here, but that way the prototype might contain default values that might be overriden by the specific instance.


I'm quite shure TC has something for this, but I prefer investigation instead of blindly copy paste... I have no time to check the function in TC (ingame).
TBH it is blizzlike enough for me. I don't care about 2-3 feet more east or west... I'm quite shure no one will recognize it. (3 hours work on an issue... you need 6 minutes for this quest.... relation?)


@cliffckerr: so should we assume they spend their money at the start of the year or the end? if it's the start then
plotdata['cumulative']['total']['ylinedata'][0] = plotdata['annual']['total']['total']['ylinedata'][0].
but your way assumes the end, so that
plotdata['cumulative']['total']['ylinedata'][9] = plotdata['annual']['total']['total']['ylinedata'][0]
I thought the first way would be better because if they export the data into excel they'll see the values corresponding to the start of the year. What do you think?


we may get rid of `encodeVisitor` here too, why leaving one _visitor_ if we kill other ones?


I'm a little confused as to why we clear readers & writers before adding. Why is that necessary?


Because performing conversion at the JAX-RS level (where it really belongs IMO) leads to a cleaner code and better documentation.

* Conversion issues
When the string passed in the URI (path segment or query param) has an invalid format, `TagsConverter` throws an `IllegalArgumentException`. RestEasy then invokes our `BadRequestExceptionMapper` which is the central point to convert a `BadRequestException` into a JAX-RS response with the format we want (`ApiError`).

* Cleaner code
In `MetricHandler` we can assume that, if the parameter is not null, it has a valid format.

* Documentation
Having a dedicated class for such parameters allows to write Swagger extensions files:
```json
{
  "id": "Tags",
  "properties": {
    "value": {
      "required": false,
      "description": "A list of tags, comma separated.",
      "notes": "A tag has the form name:value. Name and values cannot contain commas or colons.",
      "type": "string",
      "format": "tag-list"
    }
  }
}
```
With such an extension, now there's no need to document the tag list format everywhere it's used. Swagger will generate a piece of documentation for tag lists once and for all.


Commander subplugin support for #19


con custom attributes esto funciona?


In this case, singleton would not exactly mean global. What I mean to say here is, that say I call ```ColorPrinter.print_colored``` , herein I would need to input the following params: "output" string which is same for all, so not a problem, "color" again same not a problem. Arbitary additional args suggesting "how" I want my stuff printed, here I'd need to essentially check the type of logging I want done, which will then suggest if I need an HTMLWriter to print this colored output, a consolePrinter.
The amount of redirects here kill the ideal decoupling and cohesion one would need in a design. Too much decoupling IMHO leads to a spiderweb of requests jumping from one module to another without a mediator to direct them. I find it more elegant to limit the print stuff for HTML in its own class and reduce unnecessary dependencies.


acá nunca llegan los meta attributes tipo local_status_for_belgrano ?


Would be adding more tests to this, to boost coverage.


I mean, you can handle down arrow presses in a `KSPasswordField` just like you'd handle down arrow presses in a normal `NSTextField` without this method existing there at all. I've tried this now, I have a delegate attached to a `KSPasswordField`, I commented out this whole method, and the delegate is receiving callbacks to `control:textView:doCommandBySelector:` with selector == `moveDown:` when I press the down key. So this method actually prevents this from working as it would normally.


Very nice @tsykora


Isn't it just a second child or descendant?


variable declaration outside the if


we still use `toSVGString` to initialize Path2D - if import would convert binary paths to SVG syntax - then yes, absolutely.


I would get rid of RF24_SPI_TRANSACTIONS altogether.  Less code that way =
better.
On Mar 27, 2015 3:46 AM, "TMRh20" <notifications@github.com> wrote:

> Use SPI_HAS_TRANSACTION to detect transaction API
>
> - Per suggestion by @nerdralph use SPI_HAS_TRANSACTION instead of
> Arduino version for detection of transaction support
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TMRh20/RF24/commit/28c363f4965ad4059e108a0a4921a64e4e737cac
>


Here's how tag lists are declared before:
https://github.com/tsegismont/hawkular-metrics/commit/acd8cd1c1f4f5fb0a74b8d70f89ae7784d47b6ec#diff-b1b8e2573d72cf8ece42cb68f5a8603cL678

and after:
https://github.com/tsegismont/hawkular-metrics/commit/acd8cd1c1f4f5fb0a74b8d70f89ae7784d47b6ec#diff-b1b8e2573d72cf8ece42cb68f5a8603cR717


I've updated the example, but we can't simply reverse because it would affect the "class". It should be `extend(extend({}, s:Shell), {'name', a:name})`


i was thinking that they spend their money continuously throughout the year, not at the start or the end -- so if the annual spending for the first year is X, then, if you consider a timestep ε, they've spent 0 at t=0, εX at t=1, and X by the end of that year...


C'est la bonne façon de faire :)


I have edited the readme file for Linux users


What I mean is use SPI_HAS_TRANSACTION in RF24.h
On Mar 27, 2015 3:46 AM, "TMRh20" <notifications@github.com> wrote:

> Use SPI_HAS_TRANSACTION to detect transaction API
>
> - Per suggestion by @nerdralph use SPI_HAS_TRANSACTION instead of
> Arduino version for detection of transaction support
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TMRh20/RF24/commit/28c363f4965ad4059e108a0a4921a64e4e737cac
>


I have edited the readme file for the linux users


This won't compile if the array is of size 0.
Try:
```c++
int main() {
  int arr2[] = {0, 1};
  printarray(arr2);

  int arr0[] = {};
  printarray(arr0);
}
```


super commit Adrian


30414/Huruiala V. Vasile Alexandru/assignments/codeblocks/assignment3/operations.c

I guess not, some of these were done only to follow the main from instructions.


What would happen if the next year rates will decrease? Will we need to change this method?


Creo que debería tener otro nombre mas apropiado, ya que este error se muestra si hubo algún problema en el login (signInErrorInvalidLogin?)


Corresponding implementation in LibisIn worker is: https://github.com/libis/libisinworker/commit/0d2e88ce8e1325e796d5d24e2b6e6f22982c663f


Hi,

thanks for looking into my changes :)
I'm pretty new to Python so I wasn't sure what the best practice for
python2 vs python3 is.
Do you want to have checks for the python version, so that __qualname__ is
only used for python3?

On Fri, Mar 27, 2015 at 5:40 AM, Maxime Petazzoni <notifications@github.com>
wrote:

> Unfortunately __qualname__ is Python >= 3.3 only. Do you think you could
> maintain Python 2.x compatibility? Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zeitlinger/pyformance/commit/6481573daa457f589da6316e54ce079bd9720f77#commitcomment-10425832>
> .
>

--
Gregor Zeitlinger
gregor@zeitlinger.de


The testing framework is low on tests...
Even though it failed at STC, I feel this may do very well at LTC.
If you don't mind...please resubmit this test at LTC.

Pass or Fail this data will provide good insight.


There is a ";" missing in line 2919 !


So, the real interesting question is how to load the style sheets. In terms of timing, This is probably not the best place to put this. CustomElement has a load method which any resource can implement to allow async loading/processing of dependencies of that resource. So, I think, after the the view factory is returned, if it is not null, we would grab the stylesheets our of the template and the load them there. We would then change your inlineLInkedCSS method to use the pre-loaded style content and inject that into the shadow root.

The real trick here is how to load the style sheets. Right now we have everything going through the ES6 loader, so I would like to keep it that way. It may be that we add a new method on the loader abstration itself for loading styles and then let the implementation use a native loader plugin. You might have a look at how the css plugin works for system.js and see if that can be used or our implementation can be based on that. We can inline the css loader plugin inside our default loader, which is what we do for view loading now. If we have a standard way to handle this, then we can also teach our bundler to do the right thing too.


я честно никаких книг еще нормально не прочитал, поэтому не могу комментировать. пусть так


That looks to be the third descendant. I'd suggest either to group test by depth, or make a unified method which can test any depth and test all them in one test method calling sequentially like that:
testStandaloneElement("company.name", "testing first child");


the `max` key is a date (a bit confusing when referring to `max` when sorting, I know), so it's going to increase next year unless someone invents a time machine.


I think the normal RE style lookes better :D


Done in commit 9fb44a2d94b749d772d806a10c544538ed0182c6


After a JSCSD file loaded, how to add a new entity in the space?


I contemplated this, thanks.


`copy()` might not be `std::copy()`! Funnily enough, GCC 4.9.2 accepts it...
This applies to all `copy()`s below.


much bettter


a method with just a return isn't really helping us, just scrape it ;)


Values for the field types are back to being case insensitive, and the boolean changes were reverted as well.

I wasn't exactly sure what you were asking for in terms of getting the test cases in the csv reader to piggy back on the string case.  What I've added is a more exhaustive test, and I'm no longer putting data into the structures that are used in the other cases.  That should be sufficient.


plus its unneeded code. Since each code has to be maintained and tested less code is usually better.


oh, you are right. I thought it was looking for the highest rate. :+1:


I'm worried about readability then!


just my opinion, bu could this not be a little bit clearer this way:

```ruby
def parse(resource_link)
  parse_string_object(resource_link) || parse_resource(resource_link)
end
```


it's funny how consistently you see this pattern in apps, where the config initialization is the first thing you do


I was wondering about the ability to not show it in the lists... maybe it is far fetched - or can be fixed later


nice


Travis alone might not be that much useful but if we would integrate also coverity scan this might make more sense.


thanks!


Yeh but it colors the tooltip right ?


Seems like it could definitely break stuff. Gonna accept it for now should test around it later but won't hurt anything as of currently


There is no v30a kernel from what I do know, only v20a.

Also I use j1vd defconfig for e975 (geehrc)


@DamianMac Agreed. However, I didn't want to break the existing contract. Granted, the interface is internal in this case, so its probably okay. To make it cleaner, I would return a boolean instead of throwing an exception so the result code would be something along the lines of:

    public bool IsValidMessageType(Type messageType)
    {
        return _validRequestTypes.Value.Contains(messageType);
    }

If we wanted, we then add in the original `AssertValidMessageType` that throws the exception, thereby giving the caller the option of having an exception or the boolean for flow control. How does that sound?


What is the GID check doing here?


I finally added it to gitignore.  I forgot to pull before coding and shot myself in the foot.  Worse yet, GitHub was doing maintenance, so I got all sorts of issues.  That gitignore push took me 20 minutes. :)

In terms of getting it out of there... I don't know if we need (or should have) any of the `.idea` stuff in there... You likely would be a better judge than me... :)


I think d25a9b0e4191e10385aa4243f6be288a06f78432 fixes the issue.


No, it colors the hovertooltip div element.


kk, this change was 3 days old or so, good to know that all done:)


I  believe this can be rewritten as `eq([{index: match_array(cities)}])`


@jhpoelen `interactionTypes` worked without it, though. strange.


+1 on 2.9.0-dev, though it's going to break the pyroma test. I've special cased in the -rc suffix there to allow our tests to pass in a pre-release sort of manner.

In general, I'd rather that our releases be the only ones with the exact version that looks like the release tag. Anything that goes closer to that is good.


The correct speed is 10.89 block/second (http://minecraft.gamepedia.com/The_Player#Movement)
so that's 10.89/20 = 0.5445 block every 20ms
So your value is almost correct. This will use values from minecraft-data anyway. (see https://github.com/PrismarineJS/minecraft-data/issues/6)


I've been working on a new release for the module but see you've got a bunch of changes. Let me know if you want to merge changes I'm happy to rebase at this point.

You've dropped topdir management - did it create an issue for you?


Thanks for the pointers!


Sorry, I missinterpreted the `extend()` docs, yes you're right it both modifies its first argument and also returns it, but it could then be written as `extend(copy(s:Shell), {'name', a:name})`


Nice. Love seeing some programmer intent up in these function names


<Insert generic Windows hate here>


HoverTooltip is the container ?


:+1: Didn't knew about copy. Will update the example


These say different things; when extracting a property value, indeed, metadata will already have been merged. In any case, _has been mereged_ would be correct and _a metadata file_. I think the original describes my intent.


Awesome :-)


"No newline at end of file"


@mogoodrich  and @djazayeri these additional CoreConstructors could go in metadatadeploy, along with all of the Descriptor interfaces and abstract classes?  Alternatively we put them in pihcore


how can i use this in my eclipse?


np! Will clean it up.

On Fri, Mar 27, 2015 at 7:29 AM, Joshua Blum <notifications@github.com>
wrote:

> in the future can you wrap this in a helper function? its silly to put it
> a few times around the code base since its so specific. also, we always
> have a baseUrl variable laying around we should never have to hardcode
> http://eyebrowse.csail.mit.edu
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/haystack/eyebrowse-chrome-ext/commit/6d2a78dc1999d9aa3326c6278dc0a1bcd7f66dce#commitcomment-10429534>
> .
>

--
Amy X. Zhang | Ph.D. student at MIT CSAIL | http://people.csail.mit.edu/axz
| @amyxzh


There is this problem in every xml file.


Ich versteh nich ganz was ` _vpn blocking start_after_last_client` tut.


I'm not sure that this code does what it _intends_ to do?

For instance, in the case of
```
myclafer
   yourclafer -> int
```
`clafer.getType()` on _yourclafer_ will return AstConcreteClafer and not AstPrimClafer. so the isPrimitive() check _always_ returns false!

I would get that the intended code would rather look like
```
[kgad@lt00583 ~/git/gsdlab/ClaferChocoIG]$
 git diff src/main/java/Utils.java
diff --git a/src/main/java/Utils.java b/src/main/java/Utils.java
index 5350a14..58b1ccc 100644
--- a/src/main/java/Utils.java
+++ b/src/main/java/Utils.java
@@ -198,7 +198,7 @@ public static InstanceClafer getInstanceValueByName(InstanceClafer[] topClafers,
         }

         if(clafer.hasRef()) {
-            out.append("  =  ").append(clafer.getType().isPrimitive()
+            out.append("  =  ").append(clafer.getRef().getClass().isPrimitiveOrWrapper()
                     ? clafer.getRef().toString()
                     : clafer.getType().getName() + "$" + clafer.getRef());
         }
```
where the _isPrimitiveOrWrapper()_ call is necessary due to auto-boxing (http://stackoverflow.com/questions/709961/determining-if-an-object-is-of-primitive-type)

(but then again, my java knowledge is close to non-existing)


Looks good.


Good catch!  I didn't realize that `isRegisteredForRemoteNotifications ` was iOS 8 and above.


How is the `\IteratorAggregate` causing issues?


Le document que j'ai téléchargé ne correspond pas à celui qui apparait
quand on active le lien de votre message. En faisant un copié collé de
la page web, j'ai reconstitué un fichier qui a installé le client sur
scribe.
En revanche, je ne parviens pas à me connecter. Je vois une liste
d'utilisateurs (tous ceux du collège) et quand je saisis le mot de
passe, la machine tourne dans le vide.
Je ne suis donc pas en mesure de paramétrer la machine en suivant les
commentaires du script et je ne parviens pas à me connecter sur le poste
en local.

Cordialement

D. Benveniste

Le 27/03/2015 13:04, dane-lyon a écrit :
> Bonjour,
>
> Nous avons un script d'intégration pour le client Linux Mint 17/17.1 et
> il fonctionne (
> https://raw.githubusercontent.com/dane-lyon/clients-linux-scribe/master/client_scribe_linuxmint_17.X.sh
>
> ) mais il faut bien lire ce qui est en commentaire dans le script au
> début.
>
> En faite, toutes les dérivés d'Ubuntu fonctionnent avec le script de la
> 14.04 sauf la dérivé avec KDE (Kubuntu) qui n'est pas compatible.
>
> Voici ce qui fonctionne :
>
> - Ubuntu 14.04
> - Xubuntu 14.04
> - Lubuntu 14.04
> - Ubuntu Gnome 14.04
> - Ubuntu Mate 14.04
> - Linux Mint 17 Cinnamon/Mate
> - Linux Mint 17.1 Cinnamon/Mate
>
> A noté que Debian n'est pas compatible a notre script mais il y a le
> script crée par quelqu'un d'autre pour ça.
>
>
> Cordialement;
>
>
>
> Le 27/03/2015 12:14, David Benveniste a écrit :
> >
> > Bonjour,
> >
> > j'ai testé avec succès le script sur une distribution Edubuntu 12.04.
> > En revanche, je ne parviens pas à cloner le poste avec Oscar.
> > Je viens d'essayer d'utiliser le script client avec une distribution
> > Mint 17.1 et la console me retourne le message suivant : Syntax error
> > : newline unexpected
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10429363>.
> >
>
> --
> Simon Bernard
> académie de Lyon - Délégation Académique au Numérique Éducatif
> <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Simon Bernard
> Rectorat de Lyon
> DANE - Délégation Académique au Numérique Éducatif
> Assistance de niveau 2 aux utilisateurs du réseau pédagogique
> * Téléphone : 04 72 80 64 88
>
> Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon Cedex 07
> Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
> *http://www.ac-lyon.fr*
> <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10430053>.
>


Any chance we could put this logic in `.travis.yml` instead? This is going to repeat itself in every test otherwise, and I'm not sure whether we should litter the filesystem of a user who runs the tests with converted `.hdf5` files... I would just skip if not available. Converting in `.travis.yml` has the added advantage of testing the command line script.


How would you name it?

Anyway it's out of the scope of this PR.


Thinking about it, why do we even need to check to see if its a valid type? We could just change the `IsValidCommandType` to this:

        private void IsValidCommandType(Type commandType)
        {
            if (!_allowsSendingOfUnregistedTypes)
            {
                _knownMessageTypeVerifier.AssertValidMessageType(commandType);
            }
        }

Thereby allowing it to pass through without any exception handling. Don't know why I didn't think of that originally. Definitely a Friday!!


I understood W3C style guidelines to encourage the use of an Oxford style comma; I don't normally use it otherwise, but this is why I've tried to use it consistently here.


Corresponding changes in CAG Collective Access are: https://github.com/libis/cagsandbox/commit/09c0746b5a28a88a36de46a019ddf544bb02abf9


This leaves me with an error on starting attack:
context = ssl._create_unverified_context()
AttributeError: 'module' object has no attribute '_create_unverified_context'


For now I try to avoid implementation details. And concentrate just on the interfaces between objects. No synchronization issues addressed before first phase either. Just some data structures and possible operations on them. By no means is it complete or final.


"Redistributions in binary form must reproduce the above copyright notice,"

You are restricting my freedoms


I'm torn, and this is a great suggestion, but I think I'm going to leave it as is.

The skip list implementation adds overhead when adding and removing requests, whereas snapshot is ugly but it only adds overhead when the activity or fragment goes through a significant lifecycle event (comparatively infrequent). The skip list also would require us to implement some kind of comparator (just hashcode?) and maintain an ordering we don't really need. Although it's not the end of the world, using the skip list would also mean some lifecycle observers would be notified of onStart twice in a row which could be unexpected.

I'll keep thinking about it though, thanks for looking in to this in so much detail!


```
<HoverTooltip
    onMouseEnter={this._handleMouseEnter}
    onMouseLeave={this._handleMouseLeave}
    hasTooltip={hasTooltip}
    renderTooltip={this._renderTooltip}
>
  <div className="HorizontalGauge"
          style={{ backgroundColor: all.color }}
 ```


LGTM


Looks good, but in a future change, we need to uncomment out the use of the connection policy in db init, and also parameterize it via config, whether to use gateway/http vs direct/tcp.


No se si sea responsabilidad de este servicio el manejar lo de la cookie, o si solamente debería manejar la logica de consumir los endpoints y hacer la acción acorde al success / error.


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


@chyld Thank you for your comments on my assignment3.
They helped a lot.  I developed a solution for getting 100% coverage without exporting
stuff from lib/index.js just to pass tests.  I think this solution will work for your too.
1.) In your test/index.js file require your version plugin.
var Version = require('../lib/version');
2.) Then write tests for coverage of your init() method in test/index.js.
@TheAlphaNerd has written good tests to provide coverage.  However, he relies
on data exported just to get tests to pass.

To fix this just change his:
lib.Version.register to Version.register as I did after adding the
Version = require('../lib/version'); described above .
Then, you can get 100% test coverage without exporting data just to get your tests to pass.
Please let me know what you think.


Merci du feedback !


See http://unix.stackexchange.com/questions/18743/whats-the-point-in-adding-a-new-line-to-the-end-of-a-file of an explanation on why a text file should end with a newline character.


You will need Ruby


To note: this is not master but our tackbp2015 branch which is currently more experimental, in particular with respect to serialization / making Factorie Spark-friendly, so we hope to get such kinks worked out before we merge these changes into master.


In the `examples.js` file I have a "lopped animation", would you not rather have control like that?


For blocks like these ReST has 'admonitions' - http://docutils.sourceforge.net/docs/ref/rst/directives.html#attention


Maybe addons_separated? Having "added" as the name doesn't seem ideal, since one of two uses is where it's actually being added.

And yeah, I can elaborate more in the comment.


Froger sounds like Kroger for Frogs


Isn't this supposed to be a fix for Dual Sim support on sony nicki? So why is it still not working? Is there an underlining baseband conflict users should know about?

Thanks for the reponse....


Les nested routers te permettre d'avoir des composants découplés. Et puis il y a un gain de performance car ils permettent d'avoir un routage en arbre.


Did you mean to leave this as a comment in the pull request? A `CONTRIBUTING.md` file would be the customary way of adding contribution guidelines, that way it'll show up as a handy little link for anyone who files a pull request. I have patch files for most of X that I can send your way, hopefully I'm not duplicating work with someone.


I haven't tried it yet, but looks good.

On Fri, Mar 27, 2015 at 3:24 PM, hristoterezov <notifications@github.com>
wrote:

> I think d25a9b0
> <https://github.com/jitsi/jitsi-meet/commit/d25a9b0e4191e10385aa4243f6be288a06f78432>
> fixes the issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jitsi/jitsi-meet/commit/dbcfc92dc4744e3540fd90be5f98700c442df49c#commitcomment-10432596>
> .
>


like it! :)


@morrisonlevi `IteratorAggregate` is already part of `Enumerable`, so you're implementing the interface twice. If `Enumerable` has a more precise implementation with an added return type, that will cause an LSP error - the error itself is a bug on PHP's side, but implementing the interface twice is an error in its own right, which PHP happens to not throw in this particular implementation order. See SO chat for more info ^^


:facepalm: damn. just didn't see that!

please delete this PR


I feel like changing 0 and 1 to symbolic names would just make the expressions longer without helping much. Maybe a couple of new methods on Payload? environmentAddonsSpecific and environmentAddonsGeneric or something like that.


Right, unless it came from upstream.


Talvez em "where you don't need to add"  o termo  "where" não faça muito sentido em inglês, mas só desconfio.


Few weeks ago


No prob, you have been doing such an amazing work with this project than I just gave you full access to mess around with it :)


The serial number is the number of commits that modified the macro. Therefore, the serial number for this macro is now 27, not 26. You can squash the two commits into one, of course, to avoid this issue.

Running `make maintainer-all all` locally before pushing allows you to test for that kind of issue.


It seems nice


Helps in printing HTML Log files.
What would you rather have here ?


ack


Do not prefix reader method names with `get_`.


Unnecessary spacing detected.


Align the parameters of a method call if they span more than one line.


Line is too long. [108/100]


Another obvious fact: why are MPC devs so rude?


I can put a line for that in the `script` section.


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


I'm really impressed with the amount of changes introduced with this update.


Branch testing..


`@kasper93 Yes but I don't have permission to do that. You might want to take over this commit.


Bonjour,

Il faut désactiver l'userlist avec le compte local (vous pouvez le faire
avant de lancer le script) et ensuite vous pourrez vous connecter en
tapant le login et le mdp comme sous Windows, vous pourrez vous
connecter a la fois avec un compte du domaine (compte admin du domaine,
compte prof, compte élève) mais aussi avec le compte locale.

A noté que Mint est assez exigeant en ressource, si le pc n'est pas très
performant, il vaut mieux utiliser Xubuntu voir Lubuntu.

cordialement;

Le 27/03/2015 15:32, David Benveniste a écrit :
> Le document que j'ai téléchargé ne correspond pas à celui qui apparait
> quand on active le lien de votre message. En faisant un copié collé de
> la page web, j'ai reconstitué un fichier qui a installé le client sur
> scribe.
> En revanche, je ne parviens pas à me connecter. Je vois une liste
> d'utilisateurs (tous ceux du collège) et quand je saisis le mot de
> passe, la machine tourne dans le vide.
> Je ne suis donc pas en mesure de paramétrer la machine en suivant les
> commentaires du script et je ne parviens pas à me connecter sur le poste
> en local.
>
> Cordialement
>
> D. Benveniste
>
> Le 27/03/2015 13:04, dane-lyon a écrit :
> > Bonjour,
> >
> > Nous avons un script d'intégration pour le client Linux Mint 17/17.1 et
> > il fonctionne (
> >
> https://raw.githubusercontent.com/dane-lyon/clients-linux-scribe/master/client_scribe_linuxmint_17.X.sh
>
> >
> > ) mais il faut bien lire ce qui est en commentaire dans le script au
> > début.
> >
> > En faite, toutes les dérivés d'Ubuntu fonctionnent avec le script de la
> > 14.04 sauf la dérivé avec KDE (Kubuntu) qui n'est pas compatible.
> >
> > Voici ce qui fonctionne :
> >
> > - Ubuntu 14.04
> > - Xubuntu 14.04
> > - Lubuntu 14.04
> > - Ubuntu Gnome 14.04
> > - Ubuntu Mate 14.04
> > - Linux Mint 17 Cinnamon/Mate
> > - Linux Mint 17.1 Cinnamon/Mate
> >
> > A noté que Debian n'est pas compatible a notre script mais il y a le
> > script crée par quelqu'un d'autre pour ça.
> >
> >
> > Cordialement;
> >
> >
> >
> > Le 27/03/2015 12:14, David Benveniste a écrit :
> > >
> > > Bonjour,
> > >
> > > j'ai testé avec succès le script sur une distribution Edubuntu 12.04.
> > > En revanche, je ne parviens pas à cloner le poste avec Oscar.
> > > Je viens d'essayer d'utiliser le script client avec une distribution
> > > Mint 17.1 et la console me retourne le message suivant : Syntax error
> > > : newline unexpected
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10429363>.
> > >
> >
> > --
> > Simon Bernard
> > académie de Lyon - Délégation Académique au Numérique Éducatif
> > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> *Simon Bernard
> > Rectorat de Lyon
> > DANE - Délégation Académique au Numérique Éducatif
> > Assistance de niveau 2 aux utilisateurs du réseau pédagogique
> > * Téléphone : 04 72 80 64 88
> >
> > Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon Cedex 07
> > Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
> > *http://www.ac-lyon.fr*
> > <http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10430053>.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dane-lyon/clients-linux-scribe/commit/b547803a04abbbe6be07813ba9933f0bb9f425c3#commitcomment-10432751>.
>

--
Simon Bernard
académie de Lyon - Délégation Académique au Numérique Éducatif
<http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/> 	*Simon Bernard
Rectorat de Lyon
DANE - Délégation Académique au Numérique Éducatif
Assistance de niveau 2 aux utilisateurs du réseau pédagogique
* Téléphone : 04 72 80 64 88

Adresse postale : 92 rue de Marseille - BP 7227 - 69354 Lyon Cedex 07
Locaux site Croix-Rousse (47 rue Philippe de Lassalle 69004 Lyon)
*http://www.ac-lyon.fr*
<http://www2.ac-lyon.fr/serv_ress/mission_tice/wiki/>


This is really useful, thanks!


Is there a reason for not using placeholders here?


The HTMLWriter is written because we need it for the HTMLPrinter. However it *is* a general utility, I'd describe it as `Helper utility to generate HTML files.` without reference to HTMLPrinter since there may be arbitrary other users of this class in the future.


yep, adding them, the language I use might need some nitpicking again ;)


So now that you have released 1.11.0, can you revert that commit and create
a ticket for it so we don't forget to address the root cause?

Wyclif

On Thu, Mar 26, 2015 at 2:51 PM, k-joseph <notifications@github.com> wrote:

> I understand @wluyima <https://github.com/wluyima>, however we failed to
> establish the reason why the changeset was not executed, so @dkayiwa
> <https://github.com/dkayiwa>'s commit helped alot since we are almost
> running our of time before 1.11.1
>
> We can monitor this to see why, by the way, it looks like this was
> happening when checked out the tag, the standalone built well on
> 1.11.2-SNAPSHOT
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-standalone/commit/e4e7d927239a45294672bf8a9f4d5e8d30d7cb0c#commitcomment-10419096>
> .
>

--
Wyclif Luyima
Regenstrief Institute Inc.

Confidentiality Notice: The contents of this message and any files
transmitted with it may contain confidential and/or privileged information
and are intended solely for the use of the named addressee(s).
Additionally, the information contained herein may have been disclosed to
you from medical records with confidentiality protected by federal and
state laws. Federal regulations and State laws prohibit you from making
further disclosure of such information without the specific written consent
of the person to whom the information pertains or as otherwise permitted by
such regulations. A general authorization for the release of medical or
other information is not sufficient for this purpose.

If you have received this message in error, please notify the sender by
return e-mail and delete the original message. Any retention, disclosure,
copying, distribution or use of this information by anyone other than the
intended recipient is strictly prohibited.


Maybe the middle tick should be a setting, like the thresholds.  Then you could set it to 100, 83, 5.5 or whatever your magic value is


`private static final DIR_EXTENTION`
и все константы так же


cool, thanks :)

2015-03-27 15:48 GMT+01:00 Shivani Poddar <notifications@github.com>:

> yep, adding them, the language I use might need some nitpicking again ;)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/shivanipoddariiith/coala/commit/5a048c2427d321587c3ed9c7b84b49050a02b5a1#commitcomment-10433009>
> .
>


D'accord je l'avais plutôt fait pour le découplage au départ mais si ça permet en plus d'augmenter les perf' tant mieux (bien que je pense qu'à cette échelle ça soit totalement négligeable)


> On 27 Mar 2015, at 15:32 , Gregg Kellogg <notifications@github.com> wrote:
>
> I understood W3C style guidelines to encourage the use of an Oxford style comma;

Indeed.

> I don't normally use it otherwise, but this is why I've tried to use it consistently here.

The problem is: lists without the oxford comma is just about everywhere in the document. I did not even try to change those. I think the primary thing is to be consistent...

Ivan

----
Ivan Herman, W3C
Digital Publishing Activity Lead
Home: http://www.w3.org/People/Ivan/
mobile: +31-641044153
ORCID ID: http://orcid.org/0000-0003-0782-2704


赞磊磊O(∩_∩)O


Done in eee8696c3a886213b00e29f4539296ae010e4a39.


Can I just give here a big :+1: Much, much, clearer now. Thanks, Ben!


Не сокращайте имена переменных. dBt -> dataBaseTable.


1: M111 unrecognized component name: docker


29:1: I101 Imported names are in the wrong order. Should be BlueprintAutoDiscoveryRegistry, ConfigurationRegistry, ExtensionRegistry, PackageRegistry, Registry
32:1: I201 Missing newline before sections or imports.
33:1: I201 Missing newline before sections or imports.
35:1: I101 Imported names are in the wrong order. Should be unicodifier, with_app_context
284:9: B901 blind except: statement


@mflorea added in eee8696c3a886213b00e29f4539296ae010e4a39.


That works but creates an additional div.


But it clearer


They are supposed to look like this:
![screenshot 2015-03-27 07 52 21](https://cloud.githubusercontent.com/assets/99704/6869953/3e7b4342-d456-11e4-936d-0362f45494e2.png)


А вот это уже не библиотечный класс, а ваше приложение, поэтому вместо того, чтобы бросать исключение, вы должны рассказать пользователю о том, что он должен выставить переменную fizteh.db.dir


Ah, yes. Withdraw my comment (I am not sure how you do this interactively on the github site, though...)


By "manifest" I meant letting the user specify "xwalk_apk_url" in `manifest.json`, not the Android manifest :-)


Ah, I do change it to placeholders in the next commit, this commit was just rewriting the nested if logic.


There are certain files we want there, and certain ones we don't. Brett can
speak to this better so I am copying him on this message so he can tell you
which files specifically should not be checked in.

Robert

On Fri, Mar 27, 2015 at 10:24 AM, Tres Finocchiaro <notifications@github.com
> wrote:

> I finally added it to gitignore. I forgot to pull before coding and shot
> myself in the foot. Worse yet, GitHub was doing maintenance, so I got all
> sorts of issues. That gitignore push took me 20 minutes. :)
>
> In terms of getting it out of there... I don't know if we need (or should
> have) any of the .idea stuff in there... You likely would be a better
> judge than me... :)
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/qzind/qz-print/commit/2ba0832f012b9e084b11466678654fe17088c1ef#commitcomment-10432595>
> .
>

--
Robert Casto
*Owner | SellersToolbox*
p: 1-513-847-4942
c: 1-513-886-1550
e: casto.robert@gmail.com
www.sellerstoolbox.com
Tools for Amazon Sellers


+1, I really really need this PR too, we do not use ngMin on our bower deps so it breaks my build too :(


Ah, yes. It's unused now. Missed it in the visual review.

Could probably just remove it entirely.


À cette échelle c’est négligeable oui. Mais comme Express parcoure les routes de manières séquentielles, c’est l’aider de grouper les routes par grappes.

—
Aymeric Beaumet

On Friday, Mar 27, 2015 at 3:49 pm, Axel <notifications@github.com>, wrote:

D'accord je l'avais plutôt fait pour le découplage au départ mais si ça permet en plus d'augmenter les perf' tant mieux (bien que je pense qu'à cette échelle ça soit totalement négligeable)

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


Interesting - I have never used reorder before. You taught me something!


Just as a cool FYI, you can also use back ticks to include spaces in the function name.


smart-cast (participle II of "to cast" is "cast")


@taylordowns2000 there's a error here - the path to the template is ```the_bridge_templates/product/facebook_plugin.html```


This is still one string, so you need to have a period at the end and a space, otherwise it will be shown as "The download URL of Crosswalk runtime library APKDefault updater [...]". Plus you're missing a "the" ("download URL of **the** Crosswalk runtime library APK").


@javisantana I guess we want something more elegant than this?


essentially every function has a task that is small but limited.

This is the description for the ColorPrinter:

1. `print` takes care of the `delimiter` and `end` argument and joins everything to a simple `output` string
2. `_print` takes care that if possible the output is printed colored, else uncolored. If people want to use that they essentially should expect a variable that derives from `ColorPrinter` (ckeckable via isinstance which will also match derivatives) not a `Printer`.
3. `_print_uncolored` and `_print_colored` are obviously only responsible for *just* printing the content. They don't need to care about anything else.

Essentially one could eliminate the second point but that would make another method more complex. That's three functions that will be called for a print at max and makes every single one of them very simple and straightforward.

The `LogPrinter` uses the `print` function to print its things which fits nicely because it takes some more abstract content (`LogMessage` instances), changes them into strings and then lets the rest of the architecture care about actually printing it.

IMO that "web" makes sense and I don't see another possibility without duplicating code. I really hate duplicate code because usually this means having

 * more code to maintain and test
 * more options for bugs sneaking in
 * complexity within the function is usually higher, which is a killer


i think we might want to keep this one


fixed, thanks!


This sentence is still confusing and gramatically incorrect. What's a "default updater"? Do you mean something like "By default, the Android download manager will be used to fetch the URL"?


and this one


Not sure that I used the schedules Runnable in the right way. Could you please check that?


and possibly this one


Also looks good.

On 27 March 2015 at 04:03, Fabian Greffrath <notifications@github.com>
wrote:

> Wait a minute, we can have this easier, no need to reimplement strncpy().
> Are you fine with this?
>
> --- a/src/m_misc.c
> +++ b/src/m_misc.c
> @@ -372,12 +372,15 @@ char *M_StringReplace(const char *haystack, const char *needle,
>
>  boolean M_StringCopy(char *dest, const char *src, size_t dest_size)
>  {
> +    size_t len;
> +
>      if (dest_size >= 1)
>      {
>          dest[dest_size - 1] = '\0';
>          strncpy(dest, src, dest_size - 1);
>      }
> -    return strlen(dest) == strlen(src);
> +    len = strlen(dest);
> +    return src[len] == '\0';
>  }
>
>  // Safe string concat function that works like OpenBSD's strlcat().
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chocolate-doom/chocolate-doom/commit/62b5c602821cee8ac703ebe6362f3e1fc6d9ad3c#commitcomment-10427220>
> .
>

--
Simon Howard
https://soulsphere.org/


I think I worked through all of you comments. Hopefully everything is correct now.


how come you are bumping this?


Does this support where index is an array? i.e. download multiple pads?


That is very cool! I'll try and remember that


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


We need a status getter...


Using builderTask.outputFile doesn't automatically create a dependency on builderTask.
You can see this in action when building, the copyBinaryToLibs task runs first, then the compiling and linking tasks of the natives run, which means the copied natives are always going to be outdated vis-a-vis the latest build.


I'm actually using external flashhls plugin for videojs and see that canplay event didn't fire , I'll check with contrib-hls too.


Ok, solved.
I thought it was enough to run:
```
cd ~/CoolProp && git pull origin master && cd build && cmake --build . && cd ~ && sudo cp ~/CoolProp/build/libCoolProp.so /usr/lib/x86_64-linux-gnu/julia/CoolProp.so
```
to update my build


Compared with JSON parser - is this correct even for exception throwing?


It will look like full dump on every query. I can limit it to fetched urls.
I don't think that client api should log anything to console or service messages
(running it under TeamCity or as command line utility do not cover all cases of using TeamCity client)


> I'm note sure if the CREATE TEMPORARY TABLE syntax is interrpeted by all SQLite, PostgreSQL and MySQL?

I did test it with sqlite only, but the documentation of the two says yes, TEMPORARY is implemented and working in all of them.

I will try the way with the custom mirror.


You can add it to gitignore. If there is not configure.out file, a default configuration will be used.


Uhm, thanks and all, but this isn't finished - it was only an example.

It's kinda messed up what I was working on :cry:


Viet TV Channels


Thank you for your help with this!


TeamCity xBIM :: XbimExchange :: Build [Build 161](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1235&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:03:22


Hi Adib - you need to create a "Pull Request" to ask me to merge this back in. The github website should have a "Pull Request" button somewhere here. Thanks


Yes, I skipped editing it out, this was before the whole ```__del__``` discussion. Fixing it , thanks :)


Agreed. Fixing it.


Ok, I'm convinced - `insertIntoImage` will go back in.  I think it does belong on `SpanRegion`, not `Footprint`, though, and I think it should take the ID as an argument, rather than use some internal ID - you'll note that I've removed `Footprint`'s internal ID on this proposal, as I think the Citizen and Source IDs should be enough, and one more is confusing.


We do not need to have the .idea folder on git. It is only so that anyone working on the project can have the same Intellij setup (ex. Formatting style, project structure)
There is a [support](https://intellij-support.jetbrains.com/entries/23393067) page that tells you what to include for the .idea project structure


Hi - I think you might need to create a "Pull Request" if you want to merge this edit back in. I suspect github didn't make this as obvious as it might have done!


@TIarkRompf -- the link to your thesis is now a 403 Forbidden on the LAMP server. Fix, please? :)


there are a ton of merge conflicts in this


Good, but I'd put the IOException back.


Your code is quite good, almost every line is necessary. But i think a, b, c and input_a , input_c, input_c can be replaced by two array to make it succinct.


1) can we move this into an issue if this is an actual issue rather than a commit comment?
2) what version of contrib-ads are you using?


My editor does auto add a newline. This file, however, is 100% written by `VersionManager`. I forgot to add a `\n` to the write method.


Good stuff


Note to self: add `\n` here.


Cool :+1:  Just noticed it now also resets auto-increment counts since it executes a TRUNCATE query in SQL.


Strong powerful rays


Yep.


Shouldn't there be some indication of the current location in here, as well?  Ie, most of the fields on `BreakPoint`


With that we verify that stupidity can reduce performance in 1.000%


```
var delta = (gaBrowserSniffer.mobile) ? 0 :1;
if (btnStatus < maxNumStatus) {
              btnStatus++;
            } else {
              btnStatus = 0;
           }
```


I don't have it anymore. Try [searching it](https://www.google.com/search?q=xato%2010k%20passwords).


It's better to chain the filters and force the evaluation once.


s/start/end


r+


Alternatives? Inheritance? I don't like project() method there than


It's the circle of life!

![circle_of_life_simba](https://cloud.githubusercontent.com/assets/380809/6870453/a2718ebc-d472-11e4-9bbe-4657c5f95715.jpg)


Oh okay you can not do anything?

Am Freitag, 27. März 2015 schrieb Alexander01998 :

> I don't have it anymore. Try searching it
> <https://www.google.com/search?q=xato%2010k%20passwords>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/6f6ca05a4d670a7ae2537ce96e2e22f11da2cc09#commitcomment-10433605>
> .
>


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1163 of 1165 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)


Thank you very, very much for this fantastic timezone finder. You certainly put a lot of work into it. Thank you


oops, merge error...


this can be simplified to `my $stops = Load(scalar share('stops.yml')->slurp`


Typo: `desfined`


This could be just my Emacs, but the string is not sent until the highlighting is finished. Commenting the pulse highlight out makes sending instantaneous.


@kevmoo :) Thanks


This looks like a great start! I am glad that someone was able to figure all of this out (you even found the right namespace!).

A few things to note:
- You used tabs for indenting in some cases. Visual Studio's default is spaces, so I am not quite sure how  you were able to insert them, but it would be great if we can avoid tabs from the start. Mainly, we just want consistency. On the Web, tabs are interpreted as 8 spaces instead of 4, which makes the code fairly unreadable. To fix this, click `EDIT > ADVANCED > Format document`, select all, and then click `EDIT > ADVANCED > Untabify selected lines`.
- You added your code to the `HuntTheWumpus.SharedCode.Map` namespace. This is fine, but if you think you might want to create more than one file, you may want to consider creating your own `HuntTheWumpus.SharedCode.Player` namespace.
- To make it a bit clearer, you may want to put all the private variable declarations together at the top
- Instead of a variable name like `Turns`, it would make more sense to call it something like `NumTurns` or `NumTurnsRemaining`, to make it obvious what it does.
- If we don't know what a method does yet, just don't put code in it; we don't need to fill every method immediately ;)
- Add summaries for each item; there's no reason to have a documentation comment if it just says "Filler code is cool"

Sorry for the long list!


New functions in v0.2
- 'Always on top' switch
- Improved hit rate (png, gif pictures)
- (almost) no duplicates at youtube links
![al](https://cloud.githubusercontent.com/assets/3893603/6870516/8b2985da-d494-11e4-80c1-d4b6f694982e.jpg)


We can probably get rid of this and update our `if` below to:

```perl
if( defined $stops->{$orig} && defined $stops->{$dest}) { ... };
```


What does it mean for a browsing context to be "open"?


@sweetpi
I have two rules for switches but the framework tried to create a ActionParser with the buttonsParser

```
16:08:33.101 [pimatic] Could not force add rule, because error has no rule attribute.
16:08:33.168 [pimatic] TypeError: Cannot read property '1' of null
16:08:33.168 [pimatic]>    at getAssertionExpression (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/cassert/index.js:54:43)
16:08:33.168 [pimatic]>    at assert (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/cassert/index.js:27:18)
16:08:33.168 [pimatic]>    at new ButtonActionHandler (/home/ronny/actions.coffee:415:7)
16:08:33.168 [pimatic]>    at ButtonActionProvider.parseAction (/home/ronny/actions.coffee:407:30)
16:08:33.168 [pimatic]>    at ButtonActionProvider.parseAction (/home/ronny/actions.coffee:1:1)
16:08:33.168 [pimatic]>    at RuleManager._parseAction (/home/ronny/rules.coffee:486:35)
16:08:33.168 [pimatic]>    at RuleManager._parseAction (/home/ronny/rules.coffee:1:1)
16:08:33.168 [pimatic]>    at RuleManager._parseRuleActions (/home/ronny/rules.coffee:432:41)
16:08:33.168 [pimatic]>    at /home/ronny/rules.coffee:229:21
16:08:33.168 [pimatic]>    at tryCatcher (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/util.js:24:31)
16:08:33.168 [pimatic]>    at Function.Promise.attempt.Promise.try (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/method.js:31:24)
16:08:33.168 [pimatic]>    at RuleManager._parseRuleString (/home/ronny/rules.coffee:194:14)
16:08:33.168 [pimatic]>    at RuleManager.addRuleByString (/home/ronny/rules.coffee:629:15)
16:08:33.168 [pimatic]>    at /home/ronny/framework.coffee:744:26
16:08:33.168 [pimatic]>    at Framework.<anonymous> (/home/ronny/framework.coffee:731:11)
16:08:33.168 [pimatic]>    at /home/ronny/framework.coffee:730:28
16:08:33.168 [pimatic]>    at tryCatcher (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/util.js:24:31)
16:08:33.168 [pimatic]>    at Promise._settlePromiseFromHandler (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:454:31)
16:08:33.168 [pimatic]>    at Promise._settlePromiseAt (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:530:18)
16:08:33.168 [pimatic]>    at Promise._settlePromises (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:646:14)
16:08:33.168 [pimatic]>    at Async._drainQueue (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:79:16)
16:08:33.168 [pimatic]>    at Async._drainQueues (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:89:10)
16:08:33.168 [pimatic]>    at Async.drainQueues (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:14:14)
16:08:33.168 [pimatic]>    at process._tickCallback (node.js:415:13)

16:08:33.174 [pimatic] Could not parse rule "if dummydimmerswitch is turned off then switch Licht2 off": Cannot read property '1' of null
16:08:33.174 [pimatic] TypeError: Cannot read property '1' of null
16:08:33.174 [pimatic]>    at getAssertionExpression (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/cassert/index.js:54:43)
16:08:33.174 [pimatic]>    at assert (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/cassert/index.js:27:18)
16:08:33.174 [pimatic]>    at new ButtonActionHandler (/home/ronny/actions.coffee:415:7)
16:08:33.174 [pimatic]>    at ButtonActionProvider.parseAction (/home/ronny/actions.coffee:407:30)
16:08:33.174 [pimatic]>    at ButtonActionProvider.parseAction (/home/ronny/actions.coffee:1:1)
16:08:33.174 [pimatic]>    at RuleManager._parseAction (/home/ronny/rules.coffee:486:35)
16:08:33.174 [pimatic]>    at RuleManager._parseAction (/home/ronny/rules.coffee:1:1)
16:08:33.174 [pimatic]>    at RuleManager._parseRuleActions (/home/ronny/rules.coffee:432:41)
16:08:33.174 [pimatic]>    at /home/ronny/rules.coffee:229:21
16:08:33.174 [pimatic]>    at tryCatcher (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/util.js:24:31)
16:08:33.174 [pimatic]>    at Function.Promise.attempt.Promise.try (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/method.js:31:24)
16:08:33.174 [pimatic]>    at RuleManager._parseRuleString (/home/ronny/rules.coffee:194:14)
16:08:33.174 [pimatic]>    at RuleManager.addRuleByString (/home/ronny/rules.coffee:629:15)
16:08:33.174 [pimatic]>    at /home/ronny/framework.coffee:744:26
16:08:33.174 [pimatic]>    at Framework.<anonymous> (/home/ronny/framework.coffee:731:11)
16:08:33.174 [pimatic]>    at /home/ronny/framework.coffee:730:28
16:08:33.174 [pimatic]>    at tryCatcher (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/util.js:24:31)
16:08:33.174 [pimatic]>    at Promise._settlePromiseFromHandler (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:454:31)
16:08:33.174 [pimatic]>    at Promise._settlePromiseAt (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:530:18)
16:08:33.174 [pimatic]>    at Promise._settlePromises (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/promise.js:646:14)
16:08:33.174 [pimatic]>    at Async._drainQueue (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:79:16)
16:08:33.174 [pimatic]>    at Async._drainQueues (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:89:10)
16:08:33.174 [pimatic]>    at Async.drainQueues (/home/ronny/pimatic-dev/node_modules/pimatic/node_modules/bluebird/js/main/async.js:14:14)
16:08:33.174 [pimatic]>    at process._tickCallback (node.js:415:13)

```


I like this idea of adding fields to the schema but only running the algorithm which is actually being tested.  It bothered me to run 2 or 3 algorithms when only one was being tested.


This will be changed shortly.


There's a 3rd party implementation in the `thirdparty` package, though I never really tried all the edge cases - not sure if you had a look?
Are you planning to move this into a more generic place (so that we can use it for ESXi tasks too)?


> Does this only happen when built from source using git on Windows, or is there any chance it could happen when run as an installed game on Windows?

It can only happen in a Git checkout, because the broken symlinks are created by `git clone`. If in turn you then create an installer from that, then there’s be broken files in the installer, but that’s a logical consequence. No need to test something here.

Vale,
Quintus


Reminder: must be translatable.


Should be "NoPermissionException".


Indeed it is. Thanks for pointing this out, @phatmann.


Looks great. Makes a whole lotta sense.

So just the one question: if my `attributes` and `attributesToIndex` are the exact same, do I need to include both?

Oh, and the second clarification: `attributes` accepts a list of Symbols where `attributesToIndex` accepts an array of Strings? I can't pass in an array of Symbols for `attributesToIndex`?

Thanks!


Nice. Thanks for cleaning the stuff up from long ago!


Good question, I had assumed that it was the window. I had just copied it from https://github.com/w3c/webdriver/blob/master/05_navigation.html#L56 and when I landed that I just made the assumption it was a known thing.


This currently also has issue of freenode disabling Tor because of abuse at the moment.


any chance this can be pushed to the master build, if this is the fix for search on main screen


It's not fixing the issue for me on chrome 41


If `version` itself is null, this now throws an error and is causing a regression in my code.  I know... garbage in garbage out... fixing on my side anyway just wanted to point it out for ya.


What document?

Also you probably need to link to http://tools.ietf.org/html/rfc6265 somewhere to define the properties of a cookie.


<!DOCTYPE html>

<meta charset="utf-8">
<title>Edycja produktu</title>

<legend>
Edycja produktu
</legend>
<form action="editform" method="post">

Wpisz nazwę produktu:
<textarea id="text" name="text" rows="1" cols="30">
<?php htmlout($text); ?></textarea>

<form action="editform" method="post">

Obecna ilość produktu: <?php htmlout($quantity); ?>

<form method="post">
Podaj aktualną ilość: <input type="text" name="quantity">

	Producent:
	<select name="author" id="author">
		<option value="">Wybierz producenta</option>
		<?php foreach ($authors as $author): ?>
		<option value="<?php htmlout($author['id']); ?>"<?php
		 if ($author['id'] == $authorid)
		{
			echo ' selected';
		}
		?>><?php htmlout($author['name']); ?></option>
		<?php endforeach; ?>
	</select>

<fieldset>
	<legend>Kategorie:</legend>
	<?php foreach ($categories as $category): ?>

<label for="category<?php htmlout($category['id']); ?>">
		<input type="checkbox" name="categories[]"
		id="category<?php htmlout($category['id']); ?>"
		value="<?php htmlout($category['id']); ?>"<?php
		if ($category['selected'])
		{
			echo ' checked';
		}
		?>><?php htmlout($category['name']); ?>

	<?php endforeach; ?>
</fieldset>

	<input type="hidden" name="id" value="<?php htmlout($id); ?>">
	<input type="submit" value="<?php htmlout($button); ?>">

</form>


Oh wow, I hadn't realized they'd done that. Yeesh.


I don't think you need to link to WebIDL just to write a loop.


nur zum Testen


s/Initialize [...] to/Let [...] be/


Hey Megan, you should set your git email so you can get credit for these great patches.

`git config --global user.email "your_email@example.com"` as from [here](https://help.github.com/articles/setting-your-email-in-git/)


Yes you need to include both; otherwise all fields are indexed a the same "level of importance", and by "chance" a word before another will be considered more important -> that explains why your ordering was impacting the search.

You, you can pass un array of symbols. As was using an array of string because of the last `unordered(description)` one; but totally OK otherwise!


Arg, on it. thx!


Should this be a JSON list rather than a sequence?


zookeeper config is not needed.


The point of these descriptor classes is that these are what gets generated by the maven build? If you're confident in this approach, it seems fine to include it in metadatadeploy.


I don't really know where these properties come from.


@DimaSoroka


Well, you're right, it is not a hard rule, and the best thing is to be consistent. I can take a pass through to fix the remainder. Should be something like `/[^,]\s+(or|and)/m` to find them.


A couple bugs in here, should be:
```python
log("Attempting git pull for %s..." % repo_dir)
output = run_git(['pull'], git_directory=repo_dir)
```


@frrrances can you please review this PR. Thanks


Right on, thanks for the clarification!


Looks great! Yup, this is what I meant by the optimization.


Better to factor out the repetition.


good find :+1:


Given your explanation, how about extending claim-timeout to "1h"?


Я не согласен с логикой `add(doc)` только потому, что логично будет достроить объект, а только потом добавлять, чем добавлять и строить


No. This has nothing to do with maven building classes.  These are just representations of our metadata.  There would be one of these that corresponds to each openmrs metadata class


I started already yesterday. Will try and complete by today


Done in 907b88b0bfacbbd52a6a895e9467d7370b641b26.


Ok I have teh project you have make a lot of work I check that.

2015-03-27 15:33 GMT+01:00 GatisBee <notifications@github.com>:

> For now I try to avoid implementation details. And concentrate just on the
> interfaces between objects. No synchronization issues addressed before
> first phase either. Just some data structures and possible operations on
> them. By no means is it complete or final.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GatisBee/LTSProject/commit/0175aa15af872f0d40e1068862beb7afe07c6ea6#commitcomment-10432780>
> .
>


hardcoded credentials!


記得follow Tony的最愛：[馬英九](https://github.com/mayingjeou)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/13296/testReport/) for this commit**


nice work with the resolution changes, I've been thinking about that for a while :+1:


ext4, btrfs, xfs, f2fs 暂时这些


path = {{ SERVER_ROOT }}/plugins


@serkan-ozal, what's the reason of adding `notifyAll()` here? I don't get the logic behind calling `notify()` after `wait()`...


we usually do raises below the param descriptions and align it with them


@atsnyder Don't worry about this commit.  There are no bugs in it.


I think this can be replaced with `$members = $q->data;`


SuppressWarnings + double cast with generics and wildcard : I would need some justification and context here...


main description missing?


:bouquet::cry:


include "{{ SERVER_ROOT }}/secondary.conf"


@aashish24 Alright, I `None`d the renderer. Anything else?


Thank you! :)


develop (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


I did leave it as a comment in the pull request, however github was slow and I had to leave home, so things probably got lost in cyberspace.  I will add soon contribution guidelines, and admittedly am also self-debating as to where discussion of patches and changes should take place. I have set up bugzilla on midipix.org (bugs.midipix.org), and as it is normally desirable to have all of the information in a single place, my general preference goes towards having that place midipix.org rather than github. I still think that having the repos _also_ on github is very useful, and am now searching for a way to make the best of both worlds. One suggestion that came up on #midipix was to disable the 'issues'; for pull requests, my idea is to have the request as the last step in the discussion of a change, rather the step that initiates the discussion. Would love to hear your thoughts about this.PS. nobody has been working on X yet; what is being worked on right now (other than my own work on the core components) is the set of winapi headers. I assume your patches for X already take into account the presence of a posix libc?

-------- Original Message --------
Subject: Re: [cbb-gcc-4.6.4] gcc/config.gcc: enable_secureplt: set
default to 'yes'. (71f7603)
From: Tony Kelman <notifications@github.com>
Date: Fri, March 27, 2015 7:36 am
To: "midipix-ports/cbb-gcc-4.6.4" <cbb-gcc-4.6.4@noreply.github.com>
Cc: midipix <writeonce@midipix.org>

Did you mean to leave this as a comment in the pull request? A CONTRIBUTING.md file would be the customary way of adding contribution guidelines, that way it'll show up as a handy little link for anyone who files a pull request. I have patch files for most of X that I can send your way, hopefully I'm not duplicating work with someone. —Reply to this email directly or view it on GitHub.


Is there a need to set the random seed here?


Here is a followup / proper fix for this TypeError, caused by imports: https://github.com/Shougo/neovim/pull/3


I know this tool is not supposed to have a state, but I prefer to keep the unit tests isolated just in case.

    private EscapeTool tool = new EscapeTool();


I agree that it's clearer, but I still think it's a bit of a waste. Updated.


If the seed needs to be set in a particular way, you should probably note that.


Il faudrait essayer de mettre des titres de commit plus évocateurs de leur contenu...


I think it should be "pixel sizes" that have units instead of "image sizes" since that's what is actually in the Model.


good commit ;)


Can we ignore `bottle_specification.compatible_cellar?` if there's a `--force-bottle` flag? In my dev repo(whose cellar is not in `/usr/local`), `brew pull --force-bottle` is broken.


Ah, gotcha. So far the patches to at least get things to build are just config.sub and libtool, for the most part. Things may or may not function at all, we'll see.


Is there something more to do here?
On 26 Mar 2015 22:59, "Ruben Dijkstra" <notifications@github.com> wrote:

> You could relax the type argument of the Class<T> parameter to Class<?
> extends T>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/querydsl/querydsl/commit/2e84800b2ce227fc0452cf5ba357dd3fa39245da#commitcomment-10421230>
> .
>


`volatile` has to do with access from other threads in parallel programming. Using that modifier here seems missplaced.


On a side note I don't think you should cache this at all.


up to 7a279875c


Missing last `"`


Really?! Yay, I feel special! :D Seemed to be the best way to override the
default scope from what I read; how would you have done it?

On Fri, Mar 27, 2015 at 7:56 AM, Christopher Beck <notifications@github.com>
wrote:

> Interesting - I have never used reorder before. You taught me something!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rachelcolby11/Bloccit/commit/12669967b8c2e72b15e69b65fe34c7b798b6520b#commitcomment-10433185>
> .
>


Ah, yeh, good point. :+1: on that.


I've sent PR with full fix of this issue.


Initial task was to identify doctests that where not executing in sphinx


Big like for this, thanks Zsolt ;-)


[Eggert, Marek](http://marek4f.github.io/). [notacje](https://github.com/Marek4f/techniki) [tv]()http://marek4f.github.io/tv.html


MDR


Thank you for your comment.
We are planning release as early as within the next month.
Otherwise, SNAPSHOT version is available now.

```scala
libraryDependencies ++= Seq(
  "com.github.aselab" %% "scala-activerecord" % "0.3.2-SNAPSHOT"
)

resolvers += Resolver.sonatypeRepo("snapshots")
```


LGTM thanks


Good find! Think the oldVersion should include the newVersion though? e.g. 0.0.0.0-1.2.4.0


Thanks for your advice! I'll refine this commit and add manifest support.


Oops made a mistake, yeah should be 1.2.4 in the new version


I'm talking about "GAME BEING RANKED":

![sem titulo](https://cloud.githubusercontent.com/assets/6980488/6871227/11bb8fe0-d499-11e4-86af-5ab0d6e59e35.png)


should read: remove absolute paths


Are these multipliers tuned to the particular test, or are they consistent across multiple realizations?


I just noticed we should cast this to type string.


Maybe we could use "Built-in updater"? The updater is integrated into XWalkActivity and will be activated by default on every xwalk app with shared mode


I guess you would need something like that:

```python
def get_name(obj):
    if sys.version_info[0] > 2:
        return obj.__qualname__
    return obj.__name__
```

And then use `get_name(self)` wherever you need it.


Can you do that also in:
theano/sandbox/gpuarray/tests/test_scan.py


Again, is this line necessary to the test?


Afaik this is a bug because using `is` requires task.info['state'] to be **the same memory address as*** the `'success'` string on the right hand side


Did you try it before the memory fix to make sure it crash?

Can you check the compiled function to assert that the scan have the right number of outputs (or something else to detect that the optimization was done)


``const std::string&``, no?


What about the built-in default style? It should be used if resource files are unavailable.


I like that idea, but `student_view = mentoring_view` would copy the`__doc__` string for the method, though, right? If so, it may be better to leave it as is?


Maybe waiting for the task to end could be the default, what do you think?


Solide le WordPress :wink:


Is there any change to switch to JSON btw? How hard is it?


@speedywizard Just FYI: I have moved this to Assembla’s GitHub account.


В этом тесте явно не хватает проверок:
 - сейчас, если контекст не установлен, то `ПолучитьКонтекст` вызывает исключение. Соответственно, нужна проверка, что метод выполнился без ошибок;
 - нужно проверить, что контекст вернулся верный, то есть `Структура`;
 - и что в структуре есть свойство Ключ

Только после этого проверять значение ключа.


@jaredpar Don't use full path; that's often going to be long, ugly, and unhelpful in the places where `Display` is used.

I think we should replace this whole method with something simpler, anyway, since we only care about the name and not the version, public key, etc., and the assembly name and file name are generally the same thing:

``` C#
return Path.GetFileNameWithoutExtension(this.FullPath);
```

In cases where the assembly exists, this should yield the same result. In cases where the assembly *doesn't* exist we used to return the full path and now we'll just return the name. I think that's fine, though, since the full path is surfaced in other ways in that case.


These look OK to me.


For now I think we can use what we have. But for sure that is a great suggestion.


@mdogan I had suspected from a starvation between a thread that sets response over `public void set(Object offeredResponse)` and response poller thread. I had thought that response setter thread couldn't get the lock of object to get into `synchronized` block. I thought that after response poller thread finished its work, it got object lock again before response setter thread to join into `synchronized` block. Since I am not sure and couldn't reproduce the case, I called it `attempt`.


I know. I considered doing that pointless and a static approach would be faster anyway.

However, if you insist, I can make it test-local inside the @Before method to have a fresh instance for each test.


mb ``size_t`` would be more logical?


The citizen ID would be enough, but given our use of smart pointers we haven't seen any memory problems and we might want to disable Citizen at least for production builds.  The Source ID isn't available to the Footprint, so I think I'd rather keep an ID in the Footprint.  IDs are useful for debugging, and cheap.

						R


Hi, @kozec ! Is this already available on version 0.6.3? I'm asking because I can't find it anywhere.

Thanks!


solved with this https://github.com/pimatic/pimatic/commit/b768f6cca90d19727d1001a4bc37d9a1704aa16f


This is printing an error con cart.


BTW, if `notifyAll`wakes up threads only wait with `wait`method, this fix attempt will be invalid.


We should test that the `_name` gets removed properly when `_removeChildApp` is called.


But why your instead of you?


Isn't there a specific measurement error for this fault?  And can it be tested for?


Thanks!

PS
@kavinsky @jonpas check BIS forum\skype


It will be pushed on master Sunday.

@fernandog


Thanks for you meticulous review :)


I think it will be better to return the ref of the immutable data structure since it can not be change outside we will gain the performance avoiding unnecessary component rendering.
But using .toJS() method will not give us any performance profits, because we will get the new ref and component will be re-rendered all the time.

Here is draft of possible solution:

```` javascript
getState: {
	value: function getState() {
		if (typeof this[STATE_CONTAINER].toJS === "function") {
			return this[STATE_CONTAINER];
		} else {
			var state = this[STATE_CONTAINER];
			return Object.keys(state).reduce(function (obj, key) {
				obj[key] = state[key];
				return obj;
			}, {});
		}
	}
}

````

Moreover, basing on [this](https://github.com/facebook/immutable-js/wiki/Immutable-as-React-state), will be needed to create new mixing for using ````replaceState```` instead of ````setState````.

If you want I can provide some drafts via PR.


Now string is included only if `RAPIDJSON_HAS_STDSTRING` was not defined - probably not what was intended.


OK, agreed.


1: M111 unrecognized component name: Websession


add a comment on what this is doing


@jonpas sorry,  I mean @JonBons :(


I'm sorry this change to order class shouldn't be here. I'm new to github and don't know how to remove it without removing the rest.


Do you want to not bother doing this if they're not an authenticated user? Don't suppose it matters much.


Could be a good idea to raise an exception instead?
`raise Timeout()`

I think it should not be possible to "accidentally" ignore the fact that there was a timeout, so

```python
wait_for_task_to_complete(task)
do_stuff_that_requires_task_to_have_succeeded()
```
should blow up if there's a timeout, so that I don't run the following function.


Thanks to you man! :)


This is really a general comment about setting the tolerances.  I've noticed in my own work on these tests that they had to change as the algorithms changed, but I was troubled at not knowing at times what a reasonable tolerance should be.  In the absence of noise, is this a reasonable tolerance?


Add a comment for this whole section describing what is going on.  You can just copy over the text from the PR description.


OK.


Also add that information to the commit message.


Yes, you're right. This is already fixed by a later commit


Nice to get rid of some of that ancient hash_map code :+1:


I would suggest to change a few var names to something more descriptive. For example I am not really sure what the var in this line stands for.


This is also already set to True in a later commit. Maybe we should remove the flag, if you don't see any needs?!


Do not hard code names!
Instead just use `__name__`.


Given we have this in two places possibly worth commenting them both to say where the other is in case we ever have to add more cases.


This is notably higher tolerance than the untransformed case.  Was this necessary.  Perhaps you should note this with the test.


If I put # in from of all the puts, ruby will not count on those codes.


C'est mieux :+1:


Create a few entries for the next quarter too.


Strange thing, are you looking at the latest version. This is also already a:
raise Exception("Timeout ...")


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2104 of 2196 tests passed)

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


Ah! That is a regression, sorry.  I'll have a new version today. Thanks for
letting me know.

On Friday, March 27, 2015, Corey Frang <notifications@github.com> wrote:

> If version itself is null, this now throws an error and is causing a
> regression in my code. I know... garbage in garbage out... fixing on my
> side anyway just wanted to point it out for ya.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/npm/node-semver/commit/c80180d8341a8ada0236815c29a2be59864afd70#commitcomment-10433768>
> .
>


Can you add a test to AbstractTestOrcReader that fails without this fix?  If you can't figure out that code, let me know and I'll try writing one.


this should use a placeholder.


well earned, Wonderful work, please keep it up!

On Fri, Mar 27, 2015 at 12:04 PM, Michail Nikolaev <notifications@github.com
> wrote:

> @jonpas <https://github.com/jonpas> sorry, I mean @JonBons
> <https://github.com/JonBons> :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/michail-nikolaev/task-force-arma-3-radio/commit/a43b214704a9c667f5002dda31e5a44dbe0be2ff#commitcomment-10434716>
> .
>


This line seems fragile.  Can't is be set based on the len of catalog?


Closes #14.


I wasn't sure if data was a "private" variable or not ;) I see calendar uses it, but nothing else; really it'd be nice to replace it all with something that didn't have to have `$q->field()`s all over the place...


ULTIMATE_QUADS :metal: :imp: :metal:


OK, might as well avoid the extra query for anonymous users, I suppose.


Yeah, thanks, will do.


This try is probably not needed. This is the only time we check before accessing arguments values.