1: D100 Missing docstring in public module
28:1: I100 Imports statements are in the wrong order. import re should be before import time
29:1: I100 Imports statements are in the wrong order. import datetime should be before import re
31:1: I100 Imports statements are in the wrong order. import getopt should be before import marshal
35:1: I100 Imports statements are in the wrong order. import signal should be before from subprocess
37:1: I101 Imported names are in the wrong order. Should be CFG_BIBTASK_FIXEDTIMETASKS, CFG_BIBTASK_MONOTASKS, CFG_BIBTASK_VALID_TASKS
41:1: I101 Imported names are in the wrong order. Should be CFG_BIBSCHED_GC_TASKS_OLDER_THAN, CFG_BIBSCHED_GC_TASKS_TO_ARCHIVE, CFG_BIBSCHED_GC_TASKS_TO_REMOVE, CFG_BIBSCHED_INCOMPATIBLE_TASKS, CFG_BIBSCHED_MAX_NUMBER_CONCURRENT_TASKS, CFG_BIBSCHED_NEVER_STOPS, CFG_BIBSCHED_NODE_TASKS, CFG_BIBSCHED_NON_CONCURRENT_TASKS, CFG_BIBSCHED_REFRESHTIME, CFG_BINDIR, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_PREFIX, CFG_RUNDIR, CFG_SITE_URL, CFG_TMPSHAREDDIR, CFG_VERSION
41:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask_config
41:1: F401 'CFG_INSPIRE_SITE' imported but unused
59:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.config
60:1: I101 Imported names are in the wrong order. Should be real_escape_string, run_sql
61:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.dbquery
77: D400 First line should end with a period (not 'h')
77: D205 1 blank line required between summary line and description (found 0)
98: D400 First line should end with a period (not 'm')
98: D205 1 blank line required between summary line and description (found 0)
133: D200 One-line docstring should fit on one line with quotes (found 3)
148: D200 One-line docstring should fit on one line with quotes (found 3)
165: D101 Missing docstring in public class
169: D401 First line should be in imperative mood ('Return', not 'Returns')
169: D209 Multi-line docstring closing quotes should be on a separate line
169: D205 1 blank line required between summary line and description (found 0)
169: D208 Docstring is over-indented
191: D103 Missing docstring in public function
204: D400 First line should end with a period (not 'd')
219:1: E302 expected 2 blank lines, found 1
223:1: E302 expected 2 blank lines, found 1
223: D400 First line should end with a period (not ')')
230: D401 First line should be in imperative mood ('Return', not 'Returns')
239: D401 First line should be in imperative mood ('Return', not 'Returns')
248:68: E261 at least two spaces before inline comment
268:45: E127 continuation line over-indented for visual indent
279:45: E127 continuation line over-indented for visual indent
282: D400 First line should end with a period (not 't')
282: D205 1 blank line required between summary line and description (found 0)
298: D400 First line should end with a period (not 'k')
336:1: E302 expected 2 blank lines, found 1
340:1: E302 expected 2 blank lines, found 1
364: D103 Missing docstring in public function
368:1: E302 expected 2 blank lines, found 1
368: D103 Missing docstring in public function
373: D103 Missing docstring in public function
378: D103 Missing docstring in public function
389: D101 Missing docstring in public class
390: D102 Missing docstring in public method
400: D102 Missing docstring in public method
403: D102 Missing docstring in public method
406: D102 Missing docstring in public method
409: D102 Missing docstring in public method
415: D102 Missing docstring in public method
418: D102 Missing docstring in public method
422: D101 Missing docstring in public class
423: D102 Missing docstring in public method
428:9: E266 too many leading '#' for block comment
433:9: E266 too many leading '#' for block comment
441: D401 First line should be in imperative mood ('Set', not 'Sets')
441: D400 First line should end with a period (not 't')
441: D205 1 blank line required between summary line and description (found 0)
448:45: E127 continuation line over-indented for visual indent
451: D210 No whitespaces allowed surrounding docstring text
451: D401 First line should be in imperative mood ('Remove', not 'Removes')
451: D202 No blank lines allowed after function docstring (found 1)
451: D400 First line should end with a period (not 'e')
451: D200 One-line docstring should fit on one line with quotes (found 2)
467: D209 Multi-line docstring closing quotes should be on a separate line
467: D400 First line should end with a period (not 'n')
467: D205 1 blank line required between summary line and description (found 0)
482: D102 Missing docstring in public method
485:41: W503 line break before binary operator
485:41: E127 continuation line over-indented for visual indent
487:41: W503 line break before binary operator
487:41: E127 continuation line over-indented for visual indent
491: D400 First line should end with a period (not 't')
491: D205 1 blank line required between summary line and description (found 0)
498:13: E266 too many leading '#' for block comment
501:22: E126 continuation line over-indented for hanging indent
524:9: E303 too many blank lines (2)
534: D209 Multi-line docstring closing quotes should be on a separate line
534: D400 First line should end with a period (not 'd')
534: D205 1 blank line required between summary line and description (found 0)
548: D209 Multi-line docstring closing quotes should be on a separate line
548: D205 1 blank line required between summary line and description (found 0)
592:21: E266 too many leading '#' for block comment
593:21: E266 too many leading '#' for block comment
594:21: E266 too many leading '#' for block comment
595:21: E266 too many leading '#' for block comment
596:21: E266 too many leading '#' for block comment
601:17: E266 too many leading '#' for block comment
602:17: E266 too many leading '#' for block comment
617:21: E266 too many leading '#' for block comment
620:17: E266 too many leading '#' for block comment
621:17: E266 too many leading '#' for block comment
634:21: E266 too many leading '#' for block comment
642:25: E266 too many leading '#' for block comment
646:17: E266 too many leading '#' for block comment
650:13: E266 too many leading '#' for block comment
656:13: E266 too many leading '#' for block comment
657:13: E266 too many leading '#' for block comment
663:17: E266 too many leading '#' for block comment
664:17: E266 too many leading '#' for block comment
696:21: E266 too many leading '#' for block comment
698:25: E266 too many leading '#' for block comment
699:25: E266 too many leading '#' for block comment
713:21: E266 too many leading '#' for block comment
716:21: E266 too many leading '#' for block comment
719:25: E266 too many leading '#' for block comment
725:29: E266 too many leading '#' for block comment
726:29: E266 too many leading '#' for block comment
737:17: E266 too many leading '#' for block comment
738:17: E266 too many leading '#' for block comment
739:17: E266 too many leading '#' for block comment
758: D102 Missing docstring in public method
818:9: E266 too many leading '#' for block comment
836:13: E128 continuation line under-indented for visual indent
837:57: E126 continuation line over-indented for hanging indent
838:9: E266 too many leading '#' for block comment
848: D400 First line should end with a period (not 'e')
874: D102 Missing docstring in public method
882:34: E127 continuation line over-indented for visual indent
885: D102 Missing docstring in public method
894: D102 Missing docstring in public method
907:58: E127 continuation line over-indented for visual indent
921:21: E266 too many leading '#' for block comment
922:21: E266 too many leading '#' for block comment
924:25: E266 too many leading '#' for block comment
927:21: E266 too many leading '#' for block comment
932: D102 Missing docstring in public method
933:9: E266 too many leading '#' for block comment
934:9: E266 too many leading '#' for block comment
949:29: E266 too many leading '#' for block comment
950:29: E266 too many leading '#' for block comment
952:33: E266 too many leading '#' for block comment
955:29: E266 too many leading '#' for block comment
969: D103 Missing docstring in public function
979: D401 First line should be in imperative mood ('Thi', not 'This')
979: D300 Use """triple double quotes""" (found '-quotes)
988: D103 Missing docstring in public function
993: D401 First line should be in imperative mood ('Print', not 'Prints')
1036: D103 Missing docstring in public function
1041: D103 Missing docstring in public function
1045: D103 Missing docstring in public function
1063:12: E713 test for membership should be 'not in'
1070: D103 Missing docstring in public function
1074: D210 No whitespaces allowed surrounding docstring text
1074: D209 Multi-line docstring closing quotes should be on a separate line
1074: D202 No blank lines allowed after function docstring (found 1)
1074: D400 First line should end with a period (not 'g')
1074: D205 1 blank line required between summary line and description (found 0)
1128:49: E261 at least two spaces before inline comment
1128: D103 Missing docstring in public function
1149:48: E261 at least two spaces before inline comment
1149: D209 Multi-line docstring closing quotes should be on a separate line
1149: D205 1 blank line required between summary line and description (found 0)
1157:17: E128 continuation line under-indented for visual indent
1167:76: E261 at least two spaces before inline comment
1167: D202 No blank lines allowed after function docstring (found 1)
1167: D200 One-line docstring should fit on one line with quotes (found 3)
1172: D200 One-line docstring should fit on one line with quotes (found 3)
1194:29: E126 continuation line over-indented for hanging indent
1195:42: E203 whitespace before ':'
1196:21: E128 continuation line under-indented for visual indent
1227: D103 Missing docstring in public function
1232: D400 First line should end with a period (not 'd')
1232: D205 1 blank line required between summary line and description (found 0)
1276: D103 Missing docstring in public function
1327:22: E203 whitespace before ':'
1328:15: E127 continuation line over-indented for visual indent
1328:22: E203 whitespace before ':'
1331:13: E128 continuation line under-indented for visual indent
1332:13: E128 continuation line under-indented for visual indent
1333:13: E128 continuation line under-indented for visual indent
1334:13: E128 continuation line under-indented for visual indent


1: D209 Multi-line docstring closing quotes should be on a separate line
1: D400 First line should end with a period (not 'l')
1: D205 1 blank line required between summary line and description (found 0)
25:1: E402 module level import not at top of file
29: D103 Missing docstring in public function


28:1: I100 Imports statements are in the wrong order. from invenio.ext.cache should be before from werkzeug.datastructures


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
147:16: E126 continuation line over-indented for hanging indent
153:58: E203 whitespace before ','
213:15: E126 continuation line over-indented for hanging indent
216:1: E402 module level import not at top of file
250:17: E203 whitespace before ':'
251:22: E203 whitespace before ':'
252:20: E203 whitespace before ':'
253:18: E203 whitespace before ':'
254:24: E203 whitespace before ':'
255:22: E203 whitespace before ':'
256:19: E203 whitespace before ':'
257:20: E203 whitespace before ':'
258:23: E203 whitespace before ':'
259:18: E203 whitespace before ':'
260:17: E203 whitespace before ':'
261:19: E203 whitespace before ':'
262:20: E203 whitespace before ':'
263:21: E203 whitespace before ':'
264:19: E203 whitespace before ':'
265:19: E203 whitespace before ':'
266:20: E203 whitespace before ':'
267:17: E203 whitespace before ':'
268:24: E203 whitespace before ':'
269:23: E203 whitespace before ':'
270:21: E203 whitespace before ':'
271:37: E203 whitespace before ':'
285:4: E131 continuation line unaligned for hanging indent
314:1: W391 blank line at end of file


37:1: I201 Missing newline before sections or imports.
145: D102 Missing docstring in public method
154: D400 First line should end with a period (not 'n')
154: D205 1 blank line required between summary line and description (found 0)
243: D102 Missing docstring in public method
283: D400 First line should end with a period (not 'w')
283: D205 1 blank line required between summary line and description (found 0)
296: D400 First line should end with a period (not 'e')
296: D205 1 blank line required between summary line and description (found 0)
311: D400 First line should end with a period (not 'e')
311: D205 1 blank line required between summary line and description (found 0)
319: D400 First line should end with a period (not 'e')
319: D205 1 blank line required between summary line and description (found 0)
330: D400 First line should end with a period (not 'd')
330: D205 1 blank line required between summary line and description (found 0)
352: D102 Missing docstring in public method
356: D102 Missing docstring in public method
368: D400 First line should end with a period (not 'd')
391: D400 First line should end with a period (not 'n')
391: D200 One-line docstring should fit on one line with quotes (found 3)
398: D400 First line should end with a period (not 't')
398: D200 One-line docstring should fit on one line with quotes (found 3)
405: D400 First line should end with a period (not 'e')
405: D200 One-line docstring should fit on one line with quotes (found 3)
412: D102 Missing docstring in public method
434: D102 Missing docstring in public method
442: D102 Missing docstring in public method
446: D210 No whitespaces allowed surrounding docstring text
446: D400 First line should end with a period (not ')')
451: D210 No whitespaces allowed surrounding docstring text
451: D400 First line should end with a period (not 'd')
458: D210 No whitespaces allowed surrounding docstring text
458: D400 First line should end with a period (not 't')
474:17: W503 line break before binary operator
488: D102 Missing docstring in public method
502: D102 Missing docstring in public method
523: D210 No whitespaces allowed surrounding docstring text
523: D400 First line should end with a period (not 's')
529: D102 Missing docstring in public method
536: D210 No whitespaces allowed surrounding docstring text
536: D400 First line should end with a period (not 's')
541: D210 No whitespaces allowed surrounding docstring text
541: D400 First line should end with a period (not 's')
546: D210 No whitespaces allowed surrounding docstring text
546: D400 First line should end with a period (not 's')
551: D210 No whitespaces allowed surrounding docstring text
551: D400 First line should end with a period (not 's')
556: D400 First line should end with a period (not 'r')
556: D205 1 blank line required between summary line and description (found 0)
569: D400 First line should end with a period (not 'e')
614: D102 Missing docstring in public method
619: D102 Missing docstring in public method
630: D102 Missing docstring in public method
637: D102 Missing docstring in public method
643: D102 Missing docstring in public method
649: D102 Missing docstring in public method
654: D102 Missing docstring in public method
661: D210 No whitespaces allowed surrounding docstring text
661: D400 First line should end with a period (not 'e')
665: D210 No whitespaces allowed surrounding docstring text
665: D400 First line should end with a period (not 'e')
669: D210 No whitespaces allowed surrounding docstring text
669: D400 First line should end with a period (not 'e')
673: D210 No whitespaces allowed surrounding docstring text
673: D400 First line should end with a period (not 'e')
678: D203 1 blank line required before class docstring (found 0)
678: D204 1 blank line required after class docstring (found 0)
678: D400 First line should end with a period (not 'r')
678: D205 1 blank line required between summary line and description (found 0)
684: D102 Missing docstring in public method
689: D200 One-line docstring should fit on one line with quotes (found 3)
705: D102 Missing docstring in public method
710: D210 No whitespaces allowed surrounding docstring text
710: D400 First line should end with a period (not 'n')
718: D210 No whitespaces allowed surrounding docstring text
718: D400 First line should end with a period (not 'n')
722: D210 No whitespaces allowed surrounding docstring text
722: D400 First line should end with a period (not 'n')
728: D200 One-line docstring should fit on one line with quotes (found 3)
734: D400 First line should end with a period (not 's')
734: D200 One-line docstring should fit on one line with quotes (found 3)
747: D203 1 blank line required before class docstring (found 0)
747: D204 1 blank line required after class docstring (found 0)
747: D400 First line should end with a period (not 'm')
747: D200 One-line docstring should fit on one line with quotes (found 3)
751: D102 Missing docstring in public method
762: D102 Missing docstring in public method
770: D102 Missing docstring in public method
781: D102 Missing docstring in public method
784: D102 Missing docstring in public method
787: D102 Missing docstring in public method
794: D200 One-line docstring should fit on one line with quotes (found 3)
800: D200 One-line docstring should fit on one line with quotes (found 3)
810: D200 One-line docstring should fit on one line with quotes (found 3)
873:13: W503 line break before binary operator
943: D400 First line should end with a period (not 's')
951:9: E731 do not assign a lambda expression, use a def
966: D203 1 blank line required before class docstring (found 0)
966: D204 1 blank line required after class docstring (found 0)
966: D400 First line should end with a period (not 't')
973: D102 Missing docstring in public method
995: D102 Missing docstring in public method
999: D102 Missing docstring in public method
1003: D102 Missing docstring in public method
1008: D102 Missing docstring in public method
1012: D102 Missing docstring in public method
1016: D102 Missing docstring in public method
1023: D400 First line should end with a period (not 'd')
1043: D400 First line should end with a period (not 't')
1043: D200 One-line docstring should fit on one line with quotes (found 3)
1066: D400 First line should end with a period (not 't')
1066: D200 One-line docstring should fit on one line with quotes (found 3)
1093: D200 One-line docstring should fit on one line with quotes (found 3)
1107: D400 First line should end with a period (not 't')
1107: D200 One-line docstring should fit on one line with quotes (found 3)
1124: D400 First line should end with a period (not 'n')
1124: D200 One-line docstring should fit on one line with quotes (found 3)
1147: D400 First line should end with a period (not 'w')
1175: D400 First line should end with a period (not ')')
1175: D200 One-line docstring should fit on one line with quotes (found 3)
1188: D200 One-line docstring should fit on one line with quotes (found 3)
1201: D400 First line should end with a period (not 'e')
1201: D205 1 blank line required between summary line and description (found 0)
1208: D400 First line should end with a period (not 'p')
1208: D200 One-line docstring should fit on one line with quotes (found 3)
1215: D400 First line should end with a period (not 'r')
1215: D200 One-line docstring should fit on one line with quotes (found 3)
1232: D400 First line should end with a period (not 't')
1232: D200 One-line docstring should fit on one line with quotes (found 3)
1240: D400 First line should end with a period (not 'd')
1240: D200 One-line docstring should fit on one line with quotes (found 3)
1258: D200 One-line docstring should fit on one line with quotes (found 3)
1267: D400 First line should end with a period (not 'e')
1284: D400 First line should end with a period (not 'm')
1284: D205 1 blank line required between summary line and description (found 0)
1300: D200 One-line docstring should fit on one line with quotes (found 3)
1311: D102 Missing docstring in public method
1317: D102 Missing docstring in public method
1323: D102 Missing docstring in public method
1337: D102 Missing docstring in public method
1350: D200 One-line docstring should fit on one line with quotes (found 3)
1377: D102 Missing docstring in public method
1436: D102 Missing docstring in public method
1481: D102 Missing docstring in public method
1489: D102 Missing docstring in public method
1499: D102 Missing docstring in public method
1508: D102 Missing docstring in public method
1511: D102 Missing docstring in public method
1515: D102 Missing docstring in public method
1519: D102 Missing docstring in public method
1539: D102 Missing docstring in public method
1547: D102 Missing docstring in public method
1555: D102 Missing docstring in public method
1561: D102 Missing docstring in public method


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: Tests Failed

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


Yes it can, added a test case for that.


This breaks some things i'm working on, and while raising errors is normally good, this isn't actually an error because volumes are allowed to have no SDC's mapped, which is a valid volume and not an error.


Nope, removed it


@scottgonzalez see #784


hard to read


UNREADABLE!!


unreadable


The user's UDID is in $_SERVER["HTTP_X_UNIQUE_ID"]. However, this header isn't passed when a user is viewing a depiction page, only when the repo refreshes.

Also that error probably means it's permission issues, make sure you set everything up correctly as per the instructions (basically, make all json files 777). It might be something else, but try that first. :)


tls interface multiplatform surport


Oh and also, see https://github.com/SpongePowered/SpongeAPI/issues/448


@oneam hi i took your webgl canvas and used it for Broadway.js
thank you!
do you want to be mentioned in the credits?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Why is this now an atomic reference?
Its only modified on initialization from one thread?


LGTM


@PaulStoffregen
`build.elfpatch` may be replaced with another objdump recipe?

something like:

```
[...]
recipe.objcopy._elf_patch.pattern=...your elf-patch recipe...
recipe.objcopy.bin.pattern=...your bin objcopy recipe...
[...]
```

As explained here: https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5---3rd-party-Hardware-specification#recipers-for-extraction-of-executable-files-and-other-binary-data
the various objcopy recipes are sorted and executed in order (and just after your additional `recipe.elfpatch.pattern`). Naming it with the underscore prefix `_elf_patch` guarantees to be executed before the usual objcopy.


unnecessary "this"; nbd. Mainly just want to see what github comments look like


Should call out that this is "Unit Testing AngularJS Isolate Directives"


Great work!


Why?


Awesome job Kyla, +1 for revising! :-) These personas are now better suited as potential employers/clients.

There is a stray "/" in your <img> tag that needs to be removed.

Looking forward to seeing your Milestone 3!


typo expexted  -> expected


So we are also assuming that we can only cancel from the thread that originated the task, right? We should probably put some assertion somewhere to prevent a race here in case someone inadvertently does that in the future.


Piston build probably should be optional, and there are a lot of stinky code parts here... but it works :).

ps: sudo apt-get install libfreetype6-dev


hard to read


isn't `global.zone` and `zone` the same thing here?


unreadable


This is great! It's much more farther than I went. Your code is readable which is good


thank you!

I find code with blank lines to separate logical chunks of code easier to parse and navigate


Actualmente este metodo cuando lo vaya a invocar no va a recibir parametro porque no tiene el @PathParam entonces, corregir ese error


from the description of this commit it's not clear what you are fixing here.


I didn't see any reference to uBLAS updates in the release notes for Boost 1.58.  And the releases page on GitHub suggest that is has been a while (2013) since there was an update.  What's the plan going forward?


This functionality should be merged everywhere if it works well. It's critical.


I use it to override `com.googlecode.wicket.kendo.ui.widget.menu.ContextMenuBehavior.OnOpenJQueryBehavior#getCallbackParameters()`.
This way I have just one instance of the menu for the whole page that uses a `target`: `options.set("target", Options.asString("li[data-thingy]"));`. With a custom callback parameter I extract the data attribute and later I know on which component the context menu has been opened.
JavaScript event delegation.
Something like the way you detect which IMenuItem has been clicked but one layer up in the component hierarchy.


We have to use `global.zone` for `node`.
I agree this code is not supposed to run in a node env... but consistency


Mismo error que en el recurso administrador @luislenes


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Tests Failed

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


Corregir aquí también.


Corregir!


zone -> global.zone


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


may be I can change fix for style ?


no tests?


You are re-defining ``object`` I would suggest to use ``obj`` instead.


Hi Stefan,
In the latest whl file jsonpatch-1.10-py2-none-any.whl available in pypi, the file jsonpatch-1.10.data/scripts/jsonpatch has the following
#!/home/stefan/work/python-json-patch/venv/bin/python
Can you fix this so that it has the correct path?


Eclipse...


It is a dependency already and got skipped if you check some of my old build logs.


ok, great, was just wondering
thanks!


Ah, I see what you mean now. Not a blocker, but I'd definitely suggest the albeit more verbose, "If either of the other arguments is null" or just naming them explicitly.


damam garm ;)


It should be `.*.sw[ponmlkjihgfedcba]` because when Vim is started and swp file exists, it creates swo, and when even swo exists, it creates swn. This requires killing Vim multiple times, tho, so it is not very common to see these files.


Yes looks good to me.


sub 12 has unreadable lines


Was my bad - have lots of edits pending and one got missed


You absolutely right. I forgot about strategy helper. I thought about moving that method to spec_helper or coupon_helper. But strategy - best way. Thanks, I'll clean up that code.


The thing is...this is not being tested against the noclear patch, as it is now being cleared after each game.  Anglerfish would have easily passed if it was just against the noclear patch.

The Restore deterministic search state IS a functional change.


Awesome! Yeah, I should have noticed it as well, but I wasn't paying attention :(.


アプトプット?
apt-put?
apt-get?
output?


You are right, it will show the value as provided in the xml. I am not sure if that is a problem. In either case the value will not be used. We should make sure that the re-extension is *really* required since this parameter is a hack. Ideally we keep its complexity as low as possible and then replace it with a more viable solution in the future.


Quibble: you don't really need this variable any more!

You can simply change line 65 to be a return statement, and then change line 78 to be `return {};` (we know this form of uniform initialization works even on Windows...)


that's something we can discuss :)

You have added a test for onclick, with the `supportsOnClick`. I think this is related ?


On this path, we now return null where we used to return a non-empty Ledger pointer.  Is this correct?


Hi Edward,

Thank you for catching that!

On Thu, May 7, 2015 at 9:25 PM, Edward Anderson <notifications@github.com>
wrote:

> @chrisfoster22 <https://github.com/chrisfoster22> @Rocky-R
> <https://github.com/Rocky-R>
> There's a typo in this SQL, a missing space: ANDcontent IS NULL
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LessonTree/lessontree/commit/116ff2c4738b9d30c24f7957e927751bfcb0d46c#commitcomment-11099548>
> .
>

--
Five minutes of art a day keeps you headed in a good way.


Why create ```redef``` production when you can directly put the token ```kwredef?```  I think it'll be easier in the interpreter to work with tokens than productions or alternatives.


miko lesz kövi fogadóóra? Megint szarúl lettél?

2015-05-07 14:30 GMT+03:00 fogasl <notifications@github.com>:

> Ezt a mezőt érdemes lenne átnevezni, mert a size védett szó
> JavaScript-ben! (kerékátmérő, ha jól gondolom...)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MarkCsosz/webapp-lessons/commit/82df0051d384fd07486dd98fbf142382b00ed899#commitcomment-11084743>
> .
>


Many lines here are over 80 columns, and this one is so long it doesn't even fit into my github diff.  Can I ask you to fix these?


This saves the case when receiving a fatal TLS alert where
https://github.com/iojs/io.js/blob/master/deps/openssl/openssl/ssl/s3_pkt.c#L1456-L1466

SSL_read(3SSL) says that
>  RETURN VALUES
>  0   The read operation was not successful. The reason may either be a
> clean shutdown due to a "close notify" alert sent by the peer (in which case
> the SSL_RECEIVED_SHUTDOWN flag in the ssl shutdown state is set (see
> SSL_shutdown(3), SSL_set_shutdown(3)). It is also possible, that the peer
> simply shut down the underlying  transport and the shutdown is incomplete. Call SSL_get_error() with the return value ret to find out, whether an error occurred or the connection was shut down cleanly (SSL_ERROR_ZERO_RETURN)

We should check SSL_get_error() when read == 0.


ta bien lo acepto por esta vez.


Yep we can actually remove `supportsOnClick` now. I think in both cases.
I'll update the tests in the PR.


done.


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


> Was my bad - have lots of edits pending and one got missed

If you IDE is setup correctly that should not happen. Please check the following screen in preferences...

![screenshot00141](https://cloud.githubusercontent.com/assets/2443519/7539389/96fdd476-f59e-11e4-8e19-7f20eb9162a1.png)

... and make sure _Continue launch if project contains errors_ is set to __prompt__. This should ensure when you build and there are errors you get ...

![screenshot00140](https://cloud.githubusercontent.com/assets/2443519/7539448/facc6940-f59e-11e4-8b80-ee23b7dae12d.png)

Also switch off _Project menu -> Build automatically_.


Oh. Good :)


Also check the _Problems_ tab...
![screenshot00143](https://cloud.githubusercontent.com/assets/2443519/7539509/6637fb18-f59f-11e4-8c40-a74e5871d8a7.png)


Working for me, thanks!


Oh I've seen this happening before but couldn't reproduce.

Thanks


I'm not using eclipse to push/pull/merge, using SourceTree - My build was ok, with no problems, just didn't push all my edits with SourceTree


s/Gocko/Gecko


Should set lenient(false)


Apparently I failed to catch the fact that you put this in the wrong cohort @jorgehjr84. I'm going to edit you into the right place.


Ping @rfellows, @scottyaslan. Do you have any ideas on some weakness of the original unit tests that may be causing these random failures? Do you have any possibility of giving me a hand on this one? Thanks :-)


This is definitely a bug. Fixed.


Sorry, that makes no sense. :confused:


Thank you


This is really helping me to better understand while and until loops.


transient bruchts nur in domain klasse wo direkt id DB gmappt sind. da chamers entferne


Would make sense (at least in the longer-term), to generalize all of this so that it could be run against any endpoint, table, or date range?


Thanks Marc


Just wanted to touch back and mention that I was able to get alt working with the LMenu and RMenu enumerations instead of the Keys.Menu, also escape was working as well from getting the state with GetAsyncKeystate.  I know it would be possible to include decision logic within the switch case to check for specific conditions, but do you think there is a more elegant solution to this?  I'm afraid this can appear to be a sort of "hack-type" solution


@jsquyres Looks OK to me.  The only other alternative that comes to mind would be to disable the print by default and then have an environment variable that can be set to re-enable the print when you are debugging.


Was actually a typo followed by consequent copy and paste. Fixed.


Block indentation goes a little wonky right here.


e methode setti immer es return statement ha, i dem fall hets nur im else fall eis. d'exception bricht dmethode scho vorher ab. ich wür eifach s'else wegla und nachem if direkt return user schriebe


Capitalize "dispatch"


da genauso


Eclipse is set up ok + has no problems. In amongst my pending edits was one file I missed
![sourcetree](https://cloud.githubusercontent.com/assets/2363180/7539653/49709dd6-f5a0-11e4-9c5b-5751dc81f4bd.png)


Thanks. And thanks for the great help! :)

> Op 8 mei 2015 om 11:22 heeft JC Brand <notifications@github.com> het volgende geschreven:
>
> Restore the markup that @cornae added to download.htlm
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Patternslib/Patterns-site/commit/71484b6995f5006ecf14543597035df040586b7d


Also, is this boolean literal being used here to disable this code?


It's only this usage, that makes the intent behind `genStaticsES6Class` clear. Maybe add a short comment that says it exists to emit less code if not necessary.


... but of course I would be happy to learn how to do this better.


with this path, the library compiles. The example from the haddock page works in a compiled program, but not in ghci. ghci crashes when calling solve. Maybe the ghci issue is unrelated?


Also decimal floating-point :-)


Indentation is a little off here, as well.


momentan erbet die exception ja direkt vo runtimeexception, ich wür denn im 2te schritt eifach na bar exception dezwüsche hänke, wie z.b. businessexception, userexception etc. denn chönti mer die au direkt abfange ohni zfest is detail zga


adoption


niubi!


Maybe we should move this into the Emitter after this PR. It doesn't really make sense here anymore.


the `username` is only used in the message when a user doesn't exist, a few lines down. the variable could be moved down a bit, but I don't feel like it's worth a PR


There was no specific reason. It just happened as I actually rewrote the whole dialog from scratch. Now inline (again).


Same here.


I'm not sure key/value would solve my problems, since fundamentally, I need to be able to pass around an ordered array. The string Split that was added to the template in recent patches may work as a hack, but would be uglier to make work in a world of arbitrary data. Is there a way to get the template to loop over a set of numerically suffixed keys? (e.g. path_1, path_2, path_3?) If that's possible, would be a good way to take advantage of new key/value support.

The conversation also makes me think I need to look into the rules for properly escaping all the special characters inside an haproxy string value. We've got a shared self-service environment, so I can't afford to have bad data from one user take the whole system down.


Aaaaand here


I generally have quite a few unstaged files; as I was working with images I had them sorted by file path instead of my normal 'status' - the edited file got looked over.


Addresses requested feature in #4.


OK


@ddcesare @vramosp if you want to proof read it :)


@jeffposnick merge me! :)


Do not know =) When I was trying to to fix the error and I used grep the similar type in the current redmine instance directory in db/migrate folder and got that
````./db/migrate/087_change_projects_description_to_text.rb:    change_column :projects, :description, :text, :null => true, :default => nil````
So, why there is ````:null => true```` Does it matter?


Excellent job!

I saw you've convinced LeaVerou to go with Jasmine. That's huge!
I've also just noticed you've started to add more tests here.

You might be interested in https://github.com/LeaVerou/awesomplete/pull/16734
With Karma test runner it would be easier to add tests.

One thing I wanted but haven't changed yet on this PR is folder structure for tests. The default convention in Karma is to ```test``` directory (singular). Also, it's unlikely that there would be tests other than in Jasmine. So why use ```jasmine/spec``` as well? What's your take on this? How about changing ```tests/jasmine/spec``` into just ```test```?

And I'll probably have some free time soon and can add some tests as well. Don't wan't to overlap though. It's better to join our efforts. What do you think?


Yes. Or if you prefer, I can first squeeze a commit that moves it for ES51Isolated and ES6 to Emitter, and then this commit would just modify Emitter.
?


debugger !


actually you're correct, it's not used in that method anymore. I was thinking of the wrong class without looking at the context of your comment.

So yeah, that `username` is no longer used and can be removed. I'll let R# clean it up in my next PR


Why don't we emit this directly?


Thanks for the cleanup :). `mapcat` looks tasty.

One comment on Mother Goddess that you can decide how to resolve: I noticed that the `:trash` event happens before the card is moved to the trash, so the Mother Goddess subtype update will "see" the ICE that is being trashed and use that ICE's subtypes, even though the target is now going away. I had to add an extra filter to rule out `target` in `:trash`; by using the same function for all three events, that filter is now lost.

The net effect is that M.G. will retain the subtypes of the ICE that was trashed. This has no effect on the NEXT Gold interaction; as long as Gold is still active, M.G. will keep the NEXT subtype. But it would break attempts at automating other cards like Knifed/Spooned/Forked, or Quetzal's ability, because in a few minor cases M.G. would retain the subtypes of a trashed ICE. I don't see any problems for now, it would only affect future attempts at automation.

To resolve, we would either need to reinstate the separate routine for `:trash`, or add a new post-trash event to listen to instead of the current "pre-trash".

Let me know if I'm misinterpreting anything :).


Thx a lot for this patch... that the solution. :) Tested it with 1.5.0 and works fine!


Of course. Oops, I didn't mean to break compatibility. I'll fix this right away.


I think it would be better to add a ## Troubleshooting section


lines below should go away


I agree that it should. Retracing through the code here I only use the fields proxier.listenIP and proxier.hostIP so we can just pass in those in a new struct instead of Proxier. I'll go do that.


I hope that now is good.


Hi mate, your API demo is brilliant!
I just wanted to ask if you could help me with the function to call the "featured image" from a wordpress blog to be displayed in the post list view?

thanks..


:+1: - thanks for testing !! :raising_hand:


Not sure what I am supposed to be looking for but anyhow it seems to me you have a slightly dodgy process in how you use git. Before publishing you should make sure there are no uncommitted changes and the staging area is clean. You can use stash to temporarily save work if you need to. Check out ``master`` (if required) and do a complete build. If everything works then you can safely publish your local ``master`` branch.


From what I recall, I think it refers to GDC, which has a better at constant folding capabilities, and removes all tests here as nop's during compilation.


this is still not right. AIOs deployers still have wasted steps copying and modifying files that are wiped out


this runs the setup-logging plays a second time, unnecessarily


Accessing the TnE Policy:  https://drive.google.com/file/d/0B-HZkymIAir_SFhMYVRfdTdocS1xblpic1EycGtqOWJnQVRv/view?usp=sharing


I see, thanks for the explanation!
So I will keep it, but I will rename class OnOpenAjaxBehavior and method #newOnOpenAjaxBehavior if you don't mind....


@michaelcahill, I was debugging some test breakage when I realized we don't validate the extractor when creating an index. I think this change is correct, but would you please confirm?

It's on a separate commit so it's easy to revert, but there's a test in test_bug012.py, `test_illegal_extractor`, that would have to be removed as well.


Why can't we use `makeIsArrayOfPrimitive` here?


Same here


OK... I don't know how relevant this is but `isNaN` is the first thing that breaks in the case of [issue 5227](https://issues.dlang.org/show_bug.cgi?id=5227).


This is actually not correct, I'm going to revert it in favor of how I fixed it.


lösche... ich glaub das crasht swagger


Can you use wait for status function to wait for complete start?  That
should remove the ability to the user to stop the package until it's fully
started.


Done, added it to the current head of RB-1.5.
Sorry about that.


大大


May I remove `if (event instanceof SelectEvent)`? (the condition is already in `super`)


get_reference_genome is also in intSiteRetriever, but it doesn't make much sense to require intSiteRetriever as a dependency for intSiteCaller, so I duplicated the function instead.  Perhaps this would be a good function to add to hiAnnotator?  It's probably ok to keep hiAnnotator as a dependency for intSite Caller.


May I ask you to put a small comment about SSL_ERROR_ZERO_RETURN and read == 0 before the condition?

Thanks for the explanation!


Yeah - something I've just got used to, and normally is ok. Just got hit with a double-whammy of Eclipse not building cleanly from ant + me missing a pending edit.


Effectivement :v


Is there some missing code from the commit? SH2Disasm uses 4 args, not 5


Any reason why 0xFFFFFFFF cannot be used as the end address for the hex editor?


no


@dleacock You should checkout the latest codes and look for the usage of `createTopFrameSqueezeContent`. Replace `createTopFrameSqueezeContent` with your function and then remove `createTopFrameSqueezeContent`.


Of course this should read 0.1.3


Mmm.  Interesting idea.  Do you have a gut feeling for which way would be more paletable to the libibverbs community?  I note that Sean recently removed the corresponding printfs in librdmacm: http://git.openfabrics.org/?p=~shefty/librdmacm.git;a=commit;h=2b2aad809afc56fa3157f5cf99036f92b9c90f16


Writing to private text is not working, suspended until Keen fixes.


tinydocfixes (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1643 of 1643 tests passed)

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


```lua
-- pick 3rd part;delete 1st line;delete lines started with '[' and '{';delete basic command using extend regexp;pick the last part divided by '/';sort the list
-- make sure your sed command has '-r' or '-E' option
local pscmd="ps | awk '{print $3}' | sed '1d' | sed '/^\\[/d' | sed '/^{/d' | sed -E '/(-ash|sh|sed|awk|hostapd|pppd|mwan3|sleep|sort|uniq|ps|insmod|rmmod|odhcp6c)/d' | awk -F'/' '{print $NF}' | sort -u"
```


Because we don't have trees for template strings, and they're not trivial to format correctly. I think eventually I'll add them in es6 mode to compress chains of `+[string]`, but I'd leave that for a separate PR.


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


$output conflicts with the variables from line 40/49. Please rename to prevent errors :-)


dgggg


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


<1MkD1bW74iYbWJWu8X9Gis2cpGu278dJR6>.=</add>elements has beenig to inport installing


"name": name


It seems we need to backport these... How did this survive closure so far?


:+1: was thinking about doing it to avoid manually do that every time


objectId = {
    'id': str(uid),
    'uri': imgFile
}


Because `$d_D` is `null` at this point, so we cannot pass it as an argument to `makeIsArrayOfPrimitive`.


os.path.join() to support windows ...


Same answer.


I need to fix this one, I left out a paragraph.


Oh, actually I will refactor to:

```java
super.onAjax(target, event);

if (event instanceof OpenEvent)
{
	this.onOpen(target);
}
```


DeleteFile переименован в RemoveFile


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


No idea ...


leave  a commen  121


esto fue necesario porque sino había conflicto de rutas

    /api/tables/available

Estaba considerando que available era un id, y por tanto llamaba a getTableAction('available')


Sorry


this doesn't look right to me.

if we patch Element.prototype.onclick in this way then the callback will execute in the child zone of the zone where we patched Element.prototype and not in the child of the zone the zone where the click listener was registered.


Tu n'aurais pas oublié de mettre à jour les soumissions baseline ?


s/GCE// ?


Still debugging this script?


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


![468x60cbf](https://cloud.githubusercontent.com/assets/12324469/7540201/60383ec4-f5b5-11e4-83d1-4e18b00f9368.gif)<add>.=elements installing on this script


Yep, I was afraid that once we will forget to add it


LGTM


No, wait for later. It's not that easy: Moving these emissions into the Emitter will break the Closure compiler. It only gives a `JSTreeBuilder` to emitter.


yeah this is not out of the woods yet


That's what I thought


Ok.


This should *not* be folded in - its a bug fix to existing code.


lol :+1:


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


Shouldn't this be negative?

Out of scope: Same for 257 (inconsistent with 207)


*This text will be italic*


I believe that @chrisforbes means the Debug line itself.  This calls through a chain of thread-unsafe code that could potentially blow up in creative ways.


Haha, ok. Filed as #1648.


@indutny How about this? Most of English wording are borrowed from the manual.
```diff
diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc
index f73bf82..f38d4c7 100644
--- a/src/tls_wrap.cc
+++ b/src/tls_wrap.cc
@@ -435,6 +435,9 @@ void TLSWrap::ClearOut() {
     OnRead(UV_EOF, nullptr);
   }

+  // We need to check whether an error occurred or the connection was
+  // shutdown cleanly (SSL_ERROR_ZERO_RETURN) even when read == 0
+  // see iojs#1642 and SSL_read(3SSL) for details.
   if (read <= 0) {
     int err;
     Local<Value> arg = GetSSLError(read, &err, nullptr);
````


Nitpick: Capitalize `timeout` for consistency


@hirose31, this comment looks weird here now, doesn't it?


As PyPI doesn't allow re-uploading of already released files, I've now released a fixed v1.11.


Well just breaking the lines at 80 isn't enough, the logic needs to be inverted so the conditionals are not so deeply nested.


Nice


Ping @AntoineAugusti
* **url :**  _http://diplo-le-jeu.fr/Http\_Controllers.png_


saw approval from russelldb, andrewjstone
at https://github.com/basho/webmachine/commit/161cf80125b70423e7dc23df2ae9728779ab2112


merging basho/webmachine/bugfix/sdc/notfound = 161cf801 into borshop-integration-244-bugfix/sdc/notfound


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


basho/webmachine/bugfix/sdc/notfound = 161cf801 merged ok, testing candidate = ad8917a6


I think that this is wrong. similar issue to https://github.com/vicb/zone.js/commit/7fd0d74fd7f929472ddcfdc812a87f2561bbc1d1#commitcomment-11110209


missing enter


Ping @AntoineAugusti
* **url :**  _http://diplo-le-jeu.fr/Ordres.png_


@leto
For this particular use case, I would suggest sqlite over postgres (to start with, maybe we add supprt later).
a) That's faster
b) We don't really need all the features of a fullscale RDBMS. Users and relations are not even needed.
c) Separate servers and sockets are not needed, so highly portable. Can work even with those cPanel based servers.


Ping @AntoineAugusti
* **url :**  _http://diplo-le-jeu.fr/Exceptions.png_


comment


looks like typo


What does not work here?


Thanks, it's a great job to delete something you've written. Our sysops are working on redeploying a new version: http://snaps.manit.ac.in/gallery_manit/pics/server%20room.jpg


why the casts?


Add comment


Nice one!

Two more items you could add to the 'steps prior to the workshop' list:

- install git
- sign up for a github account


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


@TimChen44 好吧，我明白了，以后注意


I agree with @championswimmer @leto. Sqlite will be a better call.


muh print


No, I don't think this should be an immutable structure. I don't think @Dashed ment this could be bad performance-wise due to it not being immutable, but in general. But we've seen now that it can work. You can have this as you had it previously.


Why you removed destroy method?


rax retry tests


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


dd


for passwords you can
```bash
stty -echo
echo -n "RCS Password: "; read RCS_PASS
...
unset RCS_PASS
stty echo
```


The returns in this function seem like they're supposed to be negative too.


```
for ((name, value) <- assocs) {
```

seems to give wrong types to `name`  and `value`, that manifests itself as typing error on the next line:
`name.toString` expects an integer argument.


SQLite doesn't actually support transactions and can lead to lost data and deadlocks, so it does not meet my standards to being the central datastore. It is fine as a local cache datastore on a phone, for instance. I will be implementing this part, so it will not take up student time.


Hi @mrcbnsls

Thanks for your info. We will look into this. Yes, it will be useful.

Regards
Zoran


Good point


os.path.join()


Different singleton types, which are actually the same. Scalac seems to be able to discover that they are the same, but dotty is not.


mmm it works opposite ways in piston and console apps. But who cares ))


all of this if [[ $? -eq 0 ]] stuff is stupid and shouldn't exist (yes, I screwed this up upstream)

https://github.com/GoogleCloudPlatform/kubernetes/pull/7959


Also, isn't flask-admin for maiking GUIs for admin panels ? If I am seeing this correctly, this particular part (orga-server) is a GUI-less data management endpoint.
We can stick to just flask, or flask-restful, or django-rest-framework. (That is if we choose Python, which imo is good choice - easy to code with).


Wrong copyright


Ordering!


Is this correct?  (maybe same way broken upstream)


Should be issue #4


skip = 0 for the default value


Hmm.. it's strange!


all tests pass:
success: http://buildbot.bos1/builders/test-webmachine/builds/309


Minor overall, so I'll leave it alone, but if you're not positive in your own check, write tests until you are.


if not found require to abort(404)


@championswimmer yes, it is. There will be a central datastore with a basic web CRUD interface (flask-admin) so organizers can modify the database data easily. Flask-admin uses jquery and twitter bootstrap and looks nice and is easy to use. This repo might only contain the backend and perhaps the flask-admin will go in a different repo


Hrm - I would trust the standard library to handle cidr, then I apply a rule on top that is very clearly my biz logic on top.  I wouldn't rewrite someone else's validation code in my own validation (that was more specific).

> On May 8, 2015, at 12:26 PM, Tim Hockin <notifications@github.com> wrote:
>
> Minor overall, so I'll leave it alone, but if you're not positive in your own check, write tests until you are.
>
> —
> Reply to this email directly or view it on GitHub.
>


animate gif, I hope.

Ping me when this is ready for final(?) review


Still think this error message is badly worded -- "but there is some with such a name" should read "but there are some with no names", shouldn't it?


hi


It's in the last commit (on Range go doc)


can anyone explain why this works, or why assert_frame_equal works for the fco_u_fined test but not fco_swu?


Might want to rename it to `GemVersionResolverSpec`.


I'm not even sure this is right any more, as upstream added more output!

```
# kubectl get pod
POD              IP        CONTAINER(S)   IMAGE(S)                           HOST                  LABELS                                                              STATUS    CREATED    MESSAGE
kube-dns-ejuwk   4.0.4.2                                                     127.0.0.1/127.0.0.1   k8s-app=kube-dns,kubernetes.io/cluster-service=true,name=kube-dns   Running   18 hours
                           etcd           quay.io/coreos/etcd:v2.0.3                                                                                Running   18 hours
                           kube2sky       kubernetes/kube2sky:1.1                                                                                Running   18 hours
                           skydns         kubernetes/skydns:2015-03-11-001
```

So now it's showing 4 pods  :-(

Maybe we need to move to templates here  :-(


The build team has reviewed. Assuming this will not to be back-ported.


Thanks for the readme :+1:


But it might still be right for you guys....


Unless I am being really dumb. your check covers everything that
NetIP.Contains covers, and more.

On Fri, May 8, 2015 at 9:27 AM, Clayton Coleman <notifications@github.com>
wrote:

> Hrm - I would trust the standard library to handle cidr, then I apply a
> rule on top that is very clearly my biz logic on top. I wouldn't rewrite
> someone else's validation code in my own validation (that was more
> specific).
>
> > On May 8, 2015, at 12:26 PM, Tim Hockin <notifications@github.com>
> wrote:
> >
> > Minor overall, so I'll leave it alone, but if you're not positive in
> your own check, write tests until you are.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/90a6be9b705285c5f736fcc4cad5551f6e7149cc#commitcomment-11110548>
> .
>


Will the tool function correctly if there is no reference provided? If the answer is no, you should override `requiresReference()` to return true.


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


It seems there are merge conflicts to solve here..


yo @lattera @opntr this may be relevant to your interest: 10-STABLE and probably 11-CURRENT have a similar fix that will panic when pf_get_mtag() fails. Got MFC'd from 11-CURRENT some time after 10.1.


Because it wasn't implemented?
It was misleading to leave it there while we didn't properly implement it. Also, it didn't have any related tests. So practically it wasn't there in the first place! :D
When we really need it, and thus forced to implement it, we will have to first write the test for it, and then actually implement it. For now it is better that it wouldn't be there at all so it wouldn't mislead.


Thanks Martin it is better now, but for some reason we had a problem with accept_alert().
Great if it is works now.  We will use it in future.

Regards
Zoran


:sparkles:


254 usable addresses: 256 minus base and broadcast IPs


É bacana essa ideia, fica mais "organizado" rs


I feel like @deads2k had a different way he thought this should be handled....


This doesn't seem quite right!


This is also related to issue #7


https://github.com/freebsd/freebsd/blob/stable/10/sys/netpfil/pf/pf.c#L6295-6299 sets drop, but dereferences unconditionally below... pew pew pew


Current, when run command with `sudo`, user have be set up sudo passwordless.
Run command with cross users, require setting up sudo passwordless with this commond.
Such as, this will not ensure security.


So flask-admin part should ideally go to orga-webclient ?
Makes sense now. Thanks for explaining :)


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

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


#1
mojombo#1
mojombo/github-flavored-markdown#1


Why do we need **mkinitcpio-net.conf** at all?


Neat, thanks!


Keep 'em coming ;)


Também porque mandar todo Exception.getMessage() para a página não é uma boa ideia.
Dependendo da exceção que der, vai voltar mensagem em inglês e com termos técnicos para o usuário. :wink:


function fancyAlert(arg) {
      if(arg) {
        $.facebox({div:'#foo'})
      }
    }


def foo():
    if not bar:
        return True


[the change](https://github.com/jenkinsci/github-api-plugin/compare/github-api-1.66...github-api-1.67#diff-600376dffeb79835ede4a0b285078036L27)


Can you add a timeout to this?


Can you make this unit a "oneshot" type as well? That way, the unit is completely done by the time we get the message.


elusive


:boom:


fast-forwarding develop to borshop-integration-244-bugfix/sdc/notfound = ad8917a6


is there a plugin for vim that removes trailing whitespaces?


I'm thinking you shouldn't need `CurationConcern` on this line otherwise `PermissionsController` is namespaced as `CurationConcern::CurationConcern::PermissionsController`.


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

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


I suspect that we will have to refactor this at some stage to place the flexivility at the user's control, but I am more than happy for this to go as it is.


This could use some more logging.


Why do we need this ?!?


Use ORKEqualObjects. This will fail if _maximumValueDescription is nil.


Might as well use 1.6 BTW. (And 1.596.1.)


Pass by value for built-in type is generally more efficient
see: http://stackoverflow.com/questions/5346853/c-why-pass-by-value-is-generally-more-efficient-than-pass-by-reference-for-bu


Again, ORKEqualObjects.


Vou implementar aqui e fazer um pull request :)


@hakobera - FYI I renamed the boolean to 'use_metadata_service' so it will still apply to non-GCE platforms.  So, you'll need to update your configuration.  I probably should have properly deprecated the old name and added the new one instead of breaking it, sorry.


yes


Have you tried omitting `CurationConcern` here?  You're already inside the `CurationConcern` module (per line 2) so you may not need it.


Thank you too


LGTM


@tmat right.  What we need is `throwOnError: false, reallyDontThrowOnError: true'


move test to the already existing file PawnMovesTest.cpp


Hey - since the lines aren't expanded, I'll put the comments here -- let me know if its not clear what sections they refer to.

Code looks very good. Seems like you understood the problems well and created working solutions. It's well organized and very well commented -- the comments are short, descriptive, and well placed.

One thing you could add to your code is headers. The code review rubric says "Every file containing code should start with a header comment. At the very least, this header should contain the name of the file, a description of what the included code does, and the name of its author..."

I found an easy way to automatically generate detailed headers for your code. In both Intellij and Android Studio, if you go to Preferences > File and Code Templates > Includes > File Header, you can customize the default file header to include whatever you want. That way, every time you make a new file it can include everything (except maybe the description) automatically in the header.

One other thing I noticed is your compareTo() methods in both DomesticCat and Animal could be shortened to just one line... For example:

return this.petName.toLowerCase().compareTo(((DomesticCat) o).petName.toLowerCase();

would return -1, 0, or 1 (no need to include the if/else statements to return these values). I completely misunderstood what this problem was asking for myself (I wrote the entire 'compareTo()' method without using 'compareTo()' inside of it at all, blah).

Anyways, once again, everything looks great. Let me know if you have questions. Good luck on finishing up unit-1 hw, and see ya in class!


Please let's just change the existing examples of discrete and continuous to include labels.

If you want to cover all these cases let's put it in ORKTest instead. This is meant to be example code, not the kitchen sink.


JobGraphConstraintGroup are never serialized. Just remove the IOReadableWritable stuff here.


正在学习怎么用github


Thank god.. ;)


No, we don't.
It's just, that I usually avoid to use the exact same name for multiple things (here: class vs. task). In my experience, this often leads to confusion in discussions.


So is there no way to read the UDID without the user is sending it actively? (Btw the user can send me an email by cydia, doesn't he? I could get his UDID that way).

I'll try this later.
Thank you really for this code. Our currently repo works fine, but the protected area is horrible 😳😂.

Mit freundlichen Grüßen/Kindly regards,

Tobias Ott

> Am 08.05.2015 um 17:11 schrieb sctigercat1 <notifications@github.com>:
>
> The user's UDID is in $_SERVER["HTTP_X_UNIQUE_ID"]. However, this header isn't passed when a user is viewing a depiction page, only when the repo refreshes.
>
> Also that error probably means it's permission issues, make sure you set everything up correctly as per the instructions (basically, make all json files 777). It might be something else, but try that first. :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Thats gone, was a test case.


Yeah -- I was experimenting with seeing more selectors on the screen at a time. It was nice, in that I could see how I was targeting elements more easily, but it is absolutely horrible for diffs. So: experiment run, and failed. Never doing that again!


Thanks.


@mentions, #refs, [links](), **formatting**, and <del>tags</del> supported
 list syntax required (any unordered or ordered list supported)
 this is a complete ite this is an incomplete item
1MkD1bW74iYbWJWu8X9Gis2cpGu278dJR6


Gr8 :)


I have not found any pxe stuff which loads kernel modules at run time, but net hook does.
The PXE issue said, initrd lacks network drivers.


When I remove CurationConcern:

uninitialized constant CurationConcern::GenericWorkActor::ManagesVisibilityActor (NameError)


finishLatencyConstraint should be enough


Yes.


@sirmordred we need add support for /dev not for sysfs thats what needed for our device ..
there is overlay to disable flashlight tile in cm, or editing flashlight services


While this works for now, I think in the long run we should have different build products for different feature sets, so there would be no dependency conflicts if different dependents use different feature sets.


debug code


By the way, this breaks my build. React is stored _inside_ the `node_modules` directory. Why include this change?


Beleza, capricha aí que eu explico sobre isso hoje na aula. :+1:


Is there a way to tell the generic_work_actor to use this copy_visibility method instead of using the manages_visibility_actor.rb?


unused import breaks compilation (checkstyle error...)


I think I know what's going on here. Once you retrieve your phone, come find me, and we'll walk through it and see if we can fix it.


Can you show me the actual error message that you encounter here?


Changed password and dealer id since the old password and dealerid were still in the commit history


@ldko, we have docstrings for serializers.py!


def foo(ADD):
    if not bar:
        return True


Merge this stuff into StreamRecordWriter instead of wrapping it (like StreamingAbstractRecordReader).


While networks is still empty here there is no reason for this special case and thus allowing to simplify here (in case we ever were to add IPv7)


Note on verbosity also remarked with the previous commits.


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

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


@m-krz this commit should go to _wheezy/06_ not _wheezy/09_.
@MarekGubiec please cherry-pick to _wheezy/06_ and restore the previous state of this branch


This commit got reverted because it was causing some issues?


Why was this wrapped in an additional array?  This breaks the managed LS as we no longer return the same object shape.  This needs to be reverted ASAP.


What does this do?


From my reading of the `PrintWriter` code, `println()` should use the platform seperator


def foo(craked):
<adds>.=elements installed.=    if not bar:
        return True


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Build Failed

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


[Travel and Expense Policy] (https://drive.google.com/file/d/0B-HZkymIAir_SFhMYVRfdTdocS1xblpic1EycGtqOWJnQVRv/view?usp=sharing)

To view you must have access to the Tech + Product Documents


looking good guys, keep it up, and let me know if you got any questions ;)


Application Methods isn't rendered markdown, it's a list of things., so it should be something like:

```
<ul>
  {% for method in pattern.applicationMethods %}
  <li><code>{{ method.code }}</code> - {{ method.description }}</li>
  {% endfor %}
</ul>
```

Settings should be added with the same type of pattern.


Ficou bem simples a implementação. Não consegui pensar em mais nada além de mostrar textos hehehe
Talvez a passagem de um parâmetro que indique qual tipo de mensagem queremos mostrar. erro, informação, cuidado.


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


def foo(def foo():
    if not bar:
        return True
):
    if not bar:
        return True


Aqui também tem um problema, se o parâmetro vir `texto` vir null, vai dar um `NullPointerException` quando você chamar o `trim()`
O ideal seria criar uma classe Util para validar, e então retorna `true` ou `false`.
Exemplo:
```java
public class ValidacaoUtil{
   public static boolean validaString(String string, int tamanhoMinimo){
        // implementa validação
   }
}
```
Uso:
```java
if (ValidacaoUtil.validaString(texto, 1)){
   // lança exceção
}
```


While I like parts of this refactoring, I personally think extracting this method makes the calling code hard to read. The calling code is now doing a mix of low-level operations directly on the reader and highler-level operations via a helper method.


Why? Our new configuration doesn't use sqlite3 anymore. I discussed it with you!


It makes `clumpTwitter` be automatically built when we build `clumpScala`.


This is not true. More than one path should be possible!
Example (constraint from mapA->mapC):
mapA -> mapB1 -> mapC
          -> mapB2


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


You're welcome. :-D


You should also pull the 0.4.1 package from NPM. This was breaking deploys on a project I work on.


OK, give `include ::CurationConcern::ManagesVisibilityActor` a shot.


Yeah, the UDID isn't sent via a header except during repo refreshing in Cydia.

And yes, you can be sent an email via Cydia, you just have to add your email in there as a part of author or maintainer.

Let me know if you need anything else!


test


fasdf


```
 found   : ct.bTypes(in BCodeIdiomatic).coreBTypes.bTypes(in CoreBTypesProxy).BType | ct.bTypes(in BCodeIdiomatic).BType
 required: ct.bTypes(in BCodeIdiomatic).BType
```


Please squash this into the commit that broke whatever this fixes.


93467340-AddLCCdataset2014 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Hey can you publish to Supermarket please :)


Remove commented code.


https://www.google.bg/?gfe_rd=cr&ei=7upMVZjAB9Sz8weTk4DQC

w&gws_rd=ssl


Should we just pass the Body directly and let the datastore determine if it need to use a `io.LimitReader`?


I'm not familiar with this pattern of controllers including actors including actors.  The pattern I've seen in the past has controller creating instances of an actor class and calling methods on that instance.  This comment makes me think that pattern is still one worth pursuing: https://github.com/projecthydra/sufia/commit/754a01375032102afc7a8cbb227208e009d8fe0c#diff-dc02d26a618a8c04609f2126fdbe514aR8


This page was created by Clark


as far as I can remember, this was the constraint api specification


I wrapped it to match the wrapping at line 877.


93467340-AddLCCdataset2014 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Please use IOUtils.toString(input, encoding) & if possible do not create an extra class


This change addressed a bug where you couldn't require shimmed versions of node core modules that webpack/browserify provide (fs, path, etc.). The problem was they were being externalized by an over-aggressive matching pattern.

It looks like this breaks hot builds though - I only tested with a normal build. I have a fix coming shortly.


`SSL_shutdown` got dropped


Is this too clumsy?


@auser should be fixed by https://github.com/mjohnston/react-native-webpack-server/commit/f887b078c8e99fb0acc48a33c20bde59b88e2b68


Line 877 wraps a single error into an array of errors.  Line 886 converts an array into array (and should not be wrapped in an additional array).  I have a fix for this in https://github.com/Microsoft/TypeScript/pull/3088


"can't get such image, container or snapshot from server"


This might be a WIP, but I wanted to comment while I caught it. `indexOf` operations here become extremely expensive because each removed tuple is going to incur a full traversal of stored data values. This is why, with Vega 2, I have value counts, and value arrays stored separately and rebuilt on rems. Benchmarks found that to be more efficient that `indexOf`


Wouldn't it be even easier to keep the new list empty at first and then add items as required instead of first adding all and then removing? This would also allow using the for each style loop.


Some users were saying how navigation tint wasn't working properly, I haven't done a compilation with this revert just yet but I believe it is causing the navigation tint for the back button to not work properly.

I'd have to recompile to confirm but I'm almost certain that its causing the back button to not change the tint. Other than that, there aren't any problems from what I observed.


Please check IOUtils.toString() & if possible do not create a class for it


tinydocfixes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1649 of 1649 tests passed)

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


Aha I like this use of `Comparator.nullsLast()` -- very elegant.


Ah, I thought the two calls were the same. Sorry about that. The extra `s` was throwing me off.


Does this comparator have a unit test, though?


I added rough time tests in here to test for O(N) time. Can you double check my math?


Hmpf, ok this look ugly... A StreamRecordWriter is more an autoflush record writer and we need this anyway.

Variant 1: Modify StreamRecordWriter and ensure we set bufferTimeouts on all writes within constraints.
Variant 2: Return always an StreamRecordWriter here and handle buffertTimeouts < 0 in StreamRecordWriter


A ok, problem is located in RecordWriterFactory (see below)


Just one more question.

When I add a new package I've got my deb-file which contains the DEBIAN directory within the control file where everything necessary is stated like package ID, author, version dependencies etc.
But in your repo cms I need to add all these information again and after that I need to upload my deb package.
Why it need to be entried twice? ^^

I also wanna try to add a possibility to upload screenshots for the depiction. I'll post it on github when I got it :) okay?

Mit freundlichen Grüßen/Kindly regards,

Tobias Ott

> Am 08.05.2015 um 18:55 schrieb sctigercat1 <notifications@github.com>:
>
> Yeah, the UDID isn't sent via a header except during repo refreshing in Cydia.
>
> And yes, you can be sent an email via Cydia, you just have to add your email in there as a part of author or maintainer.
>
> Let me know if you need anything else!
>
> —
> Reply to this email directly or view it on GitHub.
>


Yeah I haven't touched this repo for a while. Thanks for the feedback.


`by hands` => `by hand`


tinydocfixes (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1649 of 1649 tests passed)

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


Ok. It's just that it is getting pretty verbose with the word "Benchmark" in almost all the benchmark classes.
But not a showstopper.


Ah, I now see that you were _so_ impressed by my finding of trailing whitespace, you've included more for me to exhibit my skills. 285, 286


changed the credentials (username, password and dealercode) because it was easier than removing them from the commit history


<class:GenericWorkActor>': uninitialized constant CurationConcern::ManagesVisibilityActor (NameError).

I have not rebased on umgrosscol changes from yesterday.  Do you think they would help?


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

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


It's up to you. I sent you some changes for the sake of good collaboration. I think the original method was doing too much, and had a lot of leaky variables. "line" and "hasCause" being two particularly hard ones to follow. I realised why refactoring was hard, so I had more transformation to do.


* Tip: make dnsdist.service.in an autoconf template, then you can use @bindir@ directly.
* I don't think you wrote systemd.m4 yourself? Have you checked how it's licensed? You might need proper attribution.


It does, but the code under test just adds "\n" to the StringBuilder, which means that it only works in a unix build!


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


added some formatting to the text lines


@jclehner Hello! Maybe you can help me with my module XInstaller.
I want to add feature which will open app in App ops after installation. I see you create something similar, but I tried your code https://github.com/jclehner/AppOpsXposed/blob/master/src/at/jclehner/appopsxposed/ApkVariant.java#L488 but I didnt work :(

How can I open App ops outside Settings app?


see https://msdn.microsoft.com/en-us/library/x53a06bb.aspx aren't all contextual keywords this way?


@slapperwan Thanks 😉


@etiennebatise Les corrections que j'ai apporté pour les classes participantes côté serveur


Ah.  Move manages_visibility_actor.rb from app/actors/concerns/ to app/actors/curation_concern/. The problem is a mismatch between the Ruby namespace and the folder hierarchy.


Yes, this is a WIP!

The "right" thing to do here depends on the use case. If rem's are rare (or
never occur) building up an array (in my benchmarks) is 4x faster than
managing counts. This is why I have a streaming flag as a possible future
todo -- we might pick a good strategy based on this single bit of
information.

On Fri, May 8, 2015 at 10:02 AM, Arvind Satyanarayan <
notifications@github.com> wrote:

> This might be a WIP, but I wanted to comment while I caught it. indexOf
> operations here become extremely expensive because each removed tuple is
> going to incur a full traversal of stored data values. This is why, with
> Vega 2, I have value counts, and value arrays stored separately and rebuilt
> on rems. Benchmarks found that to be more efficient that indexOf
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwdata/datalib/commit/b5c6f6340e5a4846bbee2931e7d897cc2eef46ee#commitcomment-11111174>
> .
>


Example:
source = ...
source.beginConstraint...
mapA = source.map...
mapB1 = mapA.map...
mapB2 = mapA.map..
this is ok.

but this is invalid:
source.beginConstraint...
mapA1 = source.map...
mapA2 = source.map...
(second edge at start)


It is indeed! :cake:


I guess this is for the undesired warning message for libusnic_verbs we stopped shipping? I can't think of other alternatives other than removing the warning or Dave's suggestion.


If you subclass `ActiveFedoraIdBasedJob`, I believe you can wipe out lines 6-10. Not sure that job has been made aware of works, though.


For the second case the api was:
mapA1 = source.beginConstraint.map
mapA2 = source.beginConstraint.map


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

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


Suggest renaming to `CopyVisibilityJob` to align with our other jobs.


Note that you could then use `return result;`, for you have that local variable `result` storing the result of computing `a+b`.


@Stephen-McDaniel @sdbecker just changed the default for retaining date windows to `FALSE` as both of you were quite surprised at the converse behavior.

@eddelbuettel Your live updating time-series will now need to include `retainDataWindow = TRUE` to preserve the user's zoom level across updates.


How come there's a stray `boost::asio` type in here? Shouldn't the return value of `pimpl()` be a type that depends on the multiplexer implementation?


@AlecRaeside `swagger to and swagger-js to`?


Why `as[EntityName]` as opposed to `as[String]`?


what are you doing here??


if get my test job running, everything should be ok.
https://github.com/citlab/flink-realtime-streaming-jobs/blob/master/test-queue-behavior/src/main/java/de/tuberlin/cit/test/queuebehavior/TestQueueBehaviorJobComplex.java#L91


"mutually exclusive" is the right term. Just "mutually" as single word means "each other".


This is what I was talking about. There's no need to do this step. Angular takes care of removing it for us since they're rendered as a repeater (`ng-repeat`) over $scope.messages. As soon as $scope.messages is emptied out, those elements no longer exist in the DOM. PLUS DOM manipulation with Angular is a no-no unless it's inside a directive.


Please revert this change -- all temp files should be deleted on exit!


Not sure I like making this a button. We need to figure out how to position it properly with added buttons. Perhaps they should be moved above the chat buttons.


That change got us moving forward.


I look forward to trying this out. Thanks!


tinydocfixes (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1649 of 1649 tests passed)

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


@Montellese Sorry to say but I confirm with these changes causes http://trac.kodi.tv/ticket/15943

also the progress in addons is not great, it also gets stuck so probably both related.


:fireworks: Great news!


To tell you the truth, as a new reader, I wouldn't understand the second sentence. As I mentioned [here](https://github.com/docopt/docopt.hs/commit/59f3af0c045918b2ca1ca439a94b51d56ffec54b#diff-04c6e90faac2675aa89e2176d2eec7d8L104).


Should this be a minor version bump to `0.10.0` since you are adding new features?


Note: when you're done working on an exercise (say, #01), you should probably branch off (`git checkout -b exercise-02`) from *master*, not the current topic-branch for the exercise. Because if you do the latter, then you also bring any commits you recorded for the exercise #01 with you to the new branch. You can check that if you go to GitHub and [display the branch *exercise-05*](https://github.com/rmcgarity/frontend-javascript-exercises/tree/exercise-05), it says "This branch is 4 commits ahead of Bloc:master", and that's because [in the history for that branch](https://github.com/rmcgarity/frontend-javascript-exercises/commits/exercise-05), you have all the commits about the previous exercises.

Not a big deal, but could be improved by using `git checkout -b` from master, instead of from the current-now-solved exercise's topic branch.


It's not a schema either ;)


Sorry if this sounds a bit odd. But I also wrote some part of it (https://github.com/scikit-learn/scikit-learn/pull/2493) :P


В новой версии, когда я добавила перевод в/из bed, добавился сдвиг координаты (в разных форматах первый нуклеотид гена имеет либо относительную координату 0, либо 1). Возможно, проще это побороть в соответствующих функциях и не перестраховываться.


Well, it's adding new features, not breaking existing ones... I'm not sure.


I see you read the best practices. Nice catch with this feature as well. (ffr, Switches back to non-code after a code message is sent)


Awesome, done.


ah, right, done.


Thanks, added another comment to this file


@jsiwek - it looks like downloading the latest github checkout via the "zip/tarball" download, does not contain some parts that are needed -- the external modules (which makes sense, since they are a separate repo).

The other issue I was running into was that our env. is behind an http_proxy, so the git checkout was also failing since the module links are over git:// instead of http/s, so the http/https proxy was of no use there.

Anyway, this is sorted out.

The proxies have been working perfectly with the patch still @20Gbps total with 8 of them (4 per 10Gbps stream, specifically 2 per socket).

Are there any best practices around the number of proxies per bandwidth, similar to the rough/estimated CPU recommendations per bandwidth.


Can you supply dummy values for the parameters here? It'll help the bot validate the template


What do 'followable' really means? Intuitively I'd assume that when 'blocking' is true or when I'm connected as this User followable would be false. But it is actually true in those cases.


Author takes need be email format (user[at]domain.com works too)


This looks great.


there is no dynamic/generic way to save window state. do we need to store the selected option for runtime only or should it persists across restarts of the app?


Thanks for this. I noticed this issue just a couple hours ago.


It's nice that you provided the examples as comments, makes it easier to review!


Why pass the klass instead of the pepper?


I wasn't sure and a I didn't go down the rabbit hole very far to check. Just add yourself.


"example.com" doesn't work really. It has to be "*.example.com"


Right. Breaking existing ones would be a major version bump to `1.0.0`, but you're adding new features in a backwards-compatible manner so it should be a minor version bump.

See [Semantic Versioning] (http://semver.org/)


Nicely done, btw!


alright :)


Sadly, that's just how Cydia works. Cydia reads what is in a repository by the Packages file -- in order for everything to show in the package list, the data has to be correctly catalogued. The control file in your package is for when the package isn't a part of a repo, then it'd just show the details directly.

Think of it this way: you're in a library, wherein libraries represent repositories in this analogy.  There's 1 million books and you're looking for a specific title. Normally, you'd go to the card catalog or computer in modern day libraries to find the exact shelf where you can find your book. For Cydia, the packages file (which is dynamically generated when you insert information into the cms in the admin cp) is the cart catalogue -- it helps show Cydia what the file is called, where it is located, how large it is, etc.

Thus, this packages file is needed because it would be impossible for Cydia to know exactly how many packages are on your site, where they're located in the directory structure, etc.

Unfortunately, php cannot read the control file from a deb because it's already compiled and, to my knowledge, php does not have that capability (except maybe in a shell script...but in very few cases which would also be very difficult).  So, this is what works now, and I believe it's what most other repos use.

Finally, including screenshots in depictions sounds like a great idea! You can just add the img tag right now to the raw html, but I'd really appreciate it if you could come up with something better. I've had <b>no</b> time to do anything with this so it'll be much appreciated. :)

I hope that answered your question, but be sure to let me know if it didn't!


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

Сначала думал, что без разницы, как действовать, а потом пришла мысль, что лучше придерживаться одного формата по всей программе. Т.е. если считывание производится в строковые объекты (в том числе и числовых полей), и в таком виде происходит работа с ними, то при модификации эти числовые поля следует всегда приводить к строкам, чтобы не было такого промежуточного состояния в работе программы, когда не понятно, ожидать строки или числа на этой позиции списка.

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


Most of the time, the tests are run with dtype=float64. So I think it make sence to hardcode float32 or test float32 and float64. Or do this test fail before your modify when floatX=float64?

Also, can you test it on the GPU?


What's the preferred way to update a PR? I can force update this commit, but that might mean we'll lose all the comments on this PR. I could also put up a second commit as part of this PR, and squash the two commits after you verbally approve it, before merging.


accidentally committed all with this commit - diddnt intend to bring in all changes beyond routes.rb. other changes include more robust tests, and conversion of plaintext state names into abbreviations when looking up a single state.


nice! this test makes more sense than that -6 pixels hack that we had before.


Great!


Why is it called PasswordBox?


`m_dynamicData += empty;`?


Force update it, GitHub will automatically check if diffs on comments still apply and preserve those that still apply and collapse those that are done.


What is `xx` for ? It is not declared.


Хорошо, я просто после сдвига координат буду снова приводить к строке. Так логичнее, согласна.


I'd suggest the following syntax instead, so that you can just add to the list:

```C#
CustomSources = String.Join(" ", new[]
{
    ContentDeliveryNetwork.Google.Domain,
    ContentDeliveryNetwork.Microsoft.Domain
}),
```


Ой. Меня тоже :) будет исправлено


What is s-join ?  Is it coming from a third party package ?
Try to use something else.


Fixed in 7dd17d235e5e274ea99e2fadcacefa76d01250f7


<add>.=elements has been installing
d64d427.


this can become a variable, it only needs to be RG unique


this can become a variable, it only needs to be RG uniqueLeave a comment


![](http://media.giphy.com/media/pCO5tKdP22RC8/giphy.gif)

(Now get some Stack Overflow points so you can write comments.)


changing commit


Absolutely


s/have///


Good idea, thanks


mmmm I don't think we should delete parenthesis when calling ```print_line```. According to the ruby style guide:

```Omit parentheses around parameters for methods that are part of an internal DSL (e.g. Rake, Rails, RSpec), methods that have "keyword" status in Ruby (e.g. attr_reader, puts) and attribute access methods. Use parentheses around the arguments of all other method invocations ```  https://github.com/bbatsov/ruby-style-guide#no-dsl-parens

I don't think ```print_line``` matches any of the three requirements :\ Does it?


@damonkelley, these are nice


by any chance, is it possible this bug had something todo with this bug https://github.com/diydrones/ardupilot/issues/2227


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


Right, I forgot about those.


#### Unreleased


thank you


Sorry to be nosy, but why not use `ap` here?


Yes, there is. A commit will be pushed soon that will be a big change.


this is mainly a NSG NAT template. Size of disk is not that important in this one. Can be a variable


Prints Hello, World!


I think it would be better to subtract the size of the function hash and then assert that it is a multiple of 32.


Ещё одна интересная мысль пришла в голову. Что если читать файл не в список, как происходит сейчас, а в `namedtuple`?

Это даст возможность удобно обращаться к полям строки, увеличит читаемость кода.
Будет проблема, правда, с тем, что кортежи не изменяются, но это можно преодолеть.

Для того, чтобы это работало, можно передавать необязательным аргументом класс, порождаемый вызовом функции `namedtuple`, т.е. использование будет выглядеть так:

```python
MyFormat = namedtuple('MyFormat', ['field1', 'field2', 'field3'])
values = read_tsv(filename, MyFormat)
for value in values:
    print(value.field1, value.field2, value.field3)
```

А в функции при этом почти ничего не поменяется:
```python
def read_tsv(filename, structure=list, comment='#'):
    ...for...if not ...
        list_obj.append(structure(line.split('\t')))
```

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


Rename this vnetAddressPrefix for clarity?


-Primero tener en cuenta que ya te fija un valor de 0 a 9.
Recuerda existe una función Random que ya se encargaba de realizar los números aleatorios.
Primeramente rellenas la matriz con dichos números.

Ya la matriz rellenada comprobaras que la suma de las columnas de cada fila no sea >12, si es mayor que doce utilizaras una función recursiva que volverá a rellenar los números de dicha fila y si sigue siendo >12
volvera a generar numeros.
Lo mismo en diagonal.

-Función recursiva: función que se llama a si misma

-Bibliotecas:Recuerda cuando haces el include <stdio.h> estas llamando a una biblioteca.
Lo que te pide es tan sencillo como crearte dos ficheros.h y después que desde el main() el tercer fichero hacer dicho include
-Cabeceras:Ese concepto se refiere a el nombre de las funciones sin ser definidas
ej: int compruebaDoceFilasYColumnasRecursivo();

-El último  punto del fichero es lo más sencillo se trata de almacenar en un buffer de salida el resultado de los numeritos.
después escribir el buffer de salida funcion write sino recuerdo mal.

sobre como recorrer un matriz en diagonal te dejo aquí un ejemplo:

int x[3][3] = {1, 2, 3,
                   4, 5, 6,
                   7, 8, 9};
    int n = 3;
    for (int trozo = 0; trozo < 2 * n - 1; ++trozo) {
    	printf("Trozo %d: ", trozo);
    	int z = trozo < n ? 0 : trozo - n + 1;
       //sI trozo es < n devuelve 0 sino devuelve trozo-n+1.
       //Es otra forma de hacer un if/else

    	for (int j = z; j <= trozo - z; ++j) {
    		printf("%d ", x[j][trozo - j]);
    	}
    	printf("\n");
    }

Paso por parámetros te dejo aquí un breve ejemplo:

int suma(int numero1, int numero2){
return numero1+numero2;
}

suma(2,3);

Al escribir el resultado suele ser cadena de caracteres recuerda crear el array de char[] o un puntero a memoria de char a la hora de utilizar la función write();


The `constructor` thing was my best theory as to why it blows up if we mock all of the methods. The more I've worked on this the less I think `constructor` was the problem.

I tried your code above and it seems to be working. Now I don't really know what the situation was where I used to have an infinite loop, but I guess it doesn't really matter. Based on the fact that the code above works I'm probably wrong about how objects passed across contexts work.


Is it that acc stuff which by rumors makes Crashpilots PH so solid, thats good to hear.  Cool, on which youtube channel should I subscribe :)..btw how do you input WPs, via cli or is currently limited to PH and RTH?


Sounds good!


Why did you remove the try?


:fireworks: :fireworks: :fireworks: :fireworks: :fireworks: :fireworks:  Thanks so much!


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

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


I'm assuming this works fine even if there is no Origin field in the DESCRIPTION ?


All these let bindings should go to for clauses IMO.


Ещё я подумал, что название переменной `list_obj` не очень хорошее. В малых функциях вполне подходит название `result` или его вариации, так как из названия функции понятно, что хранится в переменной, а выглядит не так загадочно, как, например, `list_obj` =)


Can you also mention that it is just for emit?


Code looks fine.  What should be doing to ensure that we are exercising Kryo automatically?  We want tests to fail as soon as this list becomes out of date with respect tiling jobs.  Is it a matter of setting something into the spark context?


@opdenkamp that one is wrong
1. its only there on osx
2. it includes <CoreFoundation/CFPlugInCOM.h>
3. the latter is where HRESULT is defined in reality
4. the latter is available on ios and osx

make this #include <CoreFoundation/CFPlugInCOM.h> and it shouldn't bail out on the include anymore.


Oops, thanks!


@GerardPaligot : Mon code est propre et optimisé ? (Je demande avant de pousser ça sur la branche de Sandhose)


what's the story with this?


@ThibaudDauce Peux-tu me dire si ce que j'ai écrit est bien ? Vois-tu des choses à rajouter dans cette partie ? Quelles autres parties peut-on écrire ?


Noted.


Yes, it just returns `NA` in that case. This is okay; we were already getting `f$Repository = NA` for embedded packages:

    > foo <- utils::packageDescription("rsconnect", fields = c("Repository", "GithubSHA1"))
    > foo$Repository
    [1] NA
    > foo$GithubSHA1
    [1] "9d8ad01748ef1c8ddd9d55bb0f8e83f5b6b7f977"


Yeah I know about the package.bz file, I just didn't know that the information are need for this file in your repo because my current cms can read the control file and it puts all information in the package.bz with just one click :).
If your interested in, I could send you that code.

Thank you very much for your time to answer my questions.
It's not very often that a dev give such detailed answers with these nice examples :D
Thumb up :D

I'll let you know what's the result with the Json's permissions and the screenshots :)

Mit freundlichen Grüßen/Kindly regards,

Tobias Ott

> Am 08.05.2015 um 19:26 schrieb sctigercat1 <notifications@github.com>:
>
> Sadly, that's just how Cydia works. Cydia reads what is in a repository by the Packages file -- in order for everything to show in the package list, the data has to be correctly catalogued. The control file in your package is for when the package isn't a part of a repo, then it'd just show the details directly.
>
> Think of it this way: you're in a library, wherein libraries represent repositories in this analogy. There's 1 million books and you're looking for a specific title. Normally, you'd go to the card catalog or computer in modern day libraries to find the exact shelf where you can find your book. For Cydia, the packages file (which is dynamically generated when you insert information into the cms in the admin cp) is the cart catalogue -- it helps show Cydia what the file is called, where it is located, how large it is, etc.
>
> Thus, this packages file is needed because it would be impossible for Cydia to know exactly how many packages are on your site, where they're located in the directory structure, etc.
>
> Unfortunately, php cannot read the control file from a deb because it's already compiled and, to my knowledge, php does not have that capability (except maybe in a shell script...but in very few cases which would also be very difficult). So, this is what works now, and I believe it's what most other repos use.
>
> Finally, including screenshots in depictions sounds like a great idea! You can just add the img tag right now to the raw html, but I'd really appreciate it if you could come up with something better. I've had no time to do anything with this so it'll be much appreciated. :)
>
> I hope that answered your question, but be sure to let me know if it didn't!
>
> —
> Reply to this email directly or view it on GitHub.
>


Just curious, how can you tell those values are more sane?


Använd tema -- > Base.Light
och installera api 19 (Android 4.4.2) --> genom att gå in på SDK manager och ladda ner API 19