1: D100 Missing docstring in public module
24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: I100 Imports statements are in the wrong order. import os should be before import time
26:1: E402 module level import not at top of file
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. import ConfigParser should be before import sys
28:1: E402 module level import not at top of file
29:1: E402 module level import not at top of file
30:1: E402 module level import not at top of file
31:1: I201 Missing newline before sections or imports.
31:1: E402 module level import not at top of file
32:1: I201 Missing newline before sections or imports.
32:1: E402 module level import not at top of file
34:1: I100 Imports statements are in the wrong order. from intbitset should be before from six
34:1: F999 redefinition of unused 'intbitset' from line 31
34:1: E402 module level import not at top of file
35:1: I201 Missing newline before sections or imports.
35:1: E402 module level import not at top of file
36:1: E402 module level import not at top of file
39:1: I100 Imports statements are in the wrong order. from invenio.ext.cache should be before from invenio.modules.indexer.tokenizers.BibIndexJournalTokenizer
39:1: E402 module level import not at top of file
40:1: I101 Imported names are in the wrong order. Should be get_collection_reclist, search_pattern, search_unit
40:1: E402 module level import not at top of file
41:35: E126 continuation line over-indented for hanging indent
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
45:1: I101 Imported names are in the wrong order. Should be task_get_option, task_get_task_param, task_sleep_now_if_required, task_update_progress, write_message
45:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from invenio.modules.knowledge.api
45:1: E402 module level import not at top of file
46:22: E126 continuation line over-indented for hanging indent
48:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_utils should be before from invenio.legacy.bibsched.bibtask
48:1: E402 module level import not at top of file
49:1: E402 module level import not at top of file
50:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.legacy.docextract.record
50:1: E402 module level import not at top of file
53:20: E126 continuation line over-indented for hanging indent
56: D103 Missing docstring in public function
64: D103 Missing docstring in public function
72: D103 Missing docstring in public function
102: D400 First line should end with a period (not 'g')
102: D205 1 blank line required between summary line and description (found 0)
149:55: E127 continuation line over-indented for visual indent
159: D103 Missing docstring in public function
204: D400 First line should end with a period (not 'e')
209: D103 Missing docstring in public function
226: D400 First line should end with a period (not 'd')
226: D200 One-line docstring should fit on one line with quotes (found 2)
240: D400 First line should end with a period (not 's')
240: D200 One-line docstring should fit on one line with quotes (found 2)
257: D400 First line should end with a period (not 'g')
272: D202 No blank lines allowed after function docstring (found 1)
272: D400 First line should end with a period (not 't')
281: D400 First line should end with a period (not 'e')
386: D400 First line should end with a period (not 'd')
454: D400 First line should end with a period (not 'm')
463:46: E227 missing whitespace around bitwise or shift operator
465:48: E227 missing whitespace around bitwise or shift operator
470: D401 First line should be in imperative mood ('Scan', not 'Scans')
470: D400 First line should end with a period (not 'd')
470: D205 1 blank line required between summary line and description (found 0)
470: D208 Docstring is over-indented
536:30: E128 continuation line under-indented for visual indent
538:25: E127 continuation line over-indented for visual indent
555:31: E127 continuation line over-indented for visual indent
559:40: E127 continuation line over-indented for visual indent
578:9: E303 too many blank lines (2)
580:34: E128 continuation line under-indented for visual indent
582:36: E127 continuation line over-indented for visual indent
586:39: E128 continuation line under-indented for visual indent
588:36: E127 continuation line over-indented for visual indent
600:29: E128 continuation line under-indented for visual indent
604:29: E128 continuation line under-indented for visual indent
607:25: E127 continuation line over-indented for visual indent
623:39: E127 continuation line over-indented for visual indent
626:39: E127 continuation line over-indented for visual indent
636:39: E127 continuation line over-indented for visual indent
644:34: E127 continuation line over-indented for visual indent
665:19: E128 continuation line under-indented for visual indent
666:19: E128 continuation line under-indented for visual indent
667:19: E128 continuation line under-indented for visual indent
671: D400 First line should end with a period (not 't')
671: D205 1 blank line required between summary line and description (found 0)
671: D208 Docstring is over-indented
725:41: E127 continuation line over-indented for visual indent
736:36: E127 continuation line over-indented for visual indent
765:47: E127 continuation line over-indented for visual indent
784:34: E127 continuation line over-indented for visual indent
823:34: E127 continuation line over-indented for visual indent
862:34: E127 continuation line over-indented for visual indent
896:34: E127 continuation line over-indented for visual indent
922:34: E127 continuation line over-indented for visual indent
950:44: E127 continuation line over-indented for visual indent
957:49: E127 continuation line over-indented for visual indent
979:50: E127 continuation line over-indented for visual indent
990:21: E128 continuation line under-indented for visual indent
1011:29: E127 continuation line over-indented for visual indent
1032:29: E127 continuation line over-indented for visual indent
1053:29: E127 continuation line over-indented for visual indent
1070:29: E127 continuation line over-indented for visual indent
1109: D400 First line should end with a period (not 'e')
1109: D205 1 blank line required between summary line and description (found 0)
1123: D401 First line should be in imperative mood ('Thi', not 'This')
1123: D200 One-line docstring should fit on one line with quotes (found 3)
1135: D400 First line should end with a period (not 'w')
1135: D205 1 blank line required between summary line and description (found 0)
1148: D400 First line should end with a period (not 'e')
1155: D400 First line should end with a period (not 'd')
1155: D205 1 blank line required between summary line and description (found 0)
1185: D400 First line should end with a period (not 'd')
1185: D205 1 blank line required between summary line and description (found 0)
1217: D208 Docstring is over-indented
1234:27: E127 continuation line over-indented for visual indent
1240: D209 Multi-line docstring closing quotes should be on a separate line
1240: D400 First line should end with a period (not 'e')
1240: D208 Docstring is over-indented
1248: D400 First line should end with a period (not 'l')
1248: D205 1 blank line required between summary line and description (found 0)
1271: D400 First line should end with a period (not ']')
1281: D400 First line should end with a period (not 'e')


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


1: D100 Missing docstring in public module
22:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
25:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.dbquery
25:1: I201 Missing newline before sections or imports.
25:1: E402 module level import not at top of file
26:1: I201 Missing newline before sections or imports.
26:1: E402 module level import not at top of file
27:1: I100 Imports statements are in the wrong order. from invenio.ext.cache should be before from invenio.legacy.miscutil.data_cacher
27:1: E402 module level import not at top of file
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from operator should be before from invenio.legacy.dbquery
29:1: I201 Missing newline before sections or imports.
29:1: E402 module level import not at top of file
30:1: I201 Missing newline before sections or imports.
30:1: E402 module level import not at top of file
33: D203 1 blank line required before class docstring (found 0)
33: D204 1 blank line required after class docstring (found 0)
33: D400 First line should end with a period (not ',')
33: D205 1 blank line required between summary line and description (found 0)
38: D102 Missing docstring in public method
75:32: E114 indentation is not a multiple of four (comment)
75:32: E116 unexpected indentation (comment)
92: D401 First line should be in imperative mood ('Return', not 'Returns')
92: D400 First line should end with a period (not 'y')
92: D205 1 blank line required between summary line and description (found 0)
113: D400 First line should end with a period (not 'd')
120: D400 First line should end with a period (not 'd')
134: D400 First line should end with a period (not ',')
134: D205 1 blank line required between summary line and description (found 0)
134: D208 Docstring is over-indented
150:21: E721 do not compare types, use 'isinstance()'
165:25: E128 continuation line under-indented for visual indent
166:25: E128 continuation line under-indented for visual indent
175:5: W191 indentation contains tabs
175:5: E101 indentation contains mixed spaces and tabs
178:30: E128 continuation line under-indented for visual indent
179:30: E128 continuation line under-indented for visual indent
187:26: E128 continuation line under-indented for visual indent
187:62: E502 the backslash is redundant between brackets
188:26: E128 continuation line under-indented for visual indent
193: D400 First line should end with a period (not 's')
193: D205 1 blank line required between summary line and description (found 0)
221: D400 First line should end with a period (not 's')
221: D205 1 blank line required between summary line and description (found 0)
249: D400 First line should end with a period (not 'm')
249: D205 1 blank line required between summary line and description (found 0)
280: D401 First line should be in imperative mood ('Return', not 'Returns')
280: D400 First line should end with a period (not 'd')
280: D200 One-line docstring should fit on one line with quotes (found 2)
288: D400 First line should end with a period (not 'e')
288: D205 1 blank line required between summary line and description (found 0)
288: D208 Docstring is over-indented
301: D400 First line should end with a period (not 'n')
301: D205 1 blank line required between summary line and description (found 0)
331: D400 First line should end with a period (not 'e')
331: D205 1 blank line required between summary line and description (found 0)
331: D208 Docstring is over-indented
351: D400 First line should end with a period (not 's')
351: D205 1 blank line required between summary line and description (found 0)
351: D208 Docstring is over-indented
373: D103 Missing docstring in public function


My concern is that it might not end up being general (catching all corner cases) that the `ndarray` approach attempts to. The other advantage to using `np.array(pos, np.float32)` is that you can leave it in that format, as it is what eventually gets sent to the GPU (avoids another conversion). In other words, if it's not already an `ndarray` at this point, it will be later, so this approach shouldn't actually be faster.


25:1: I100 Imports statements are in the wrong order. from cryptography.hazmat.backends should be before from cryptography.hazmat.primitives.ciphers


1: D100 Missing docstring in public module
41:1: I100 Imports statements are in the wrong order. import ConfigParser should be before import sys
47:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.dbquery
47:1: I201 Missing newline before sections or imports.
48:1: I201 Missing newline before sections or imports.
48:1: F401 'CFG_ETCDIR' imported but unused
49:1: I101 Imported names are in the wrong order. Should be task_get_option, task_sleep_now_if_required, task_update_progress, write_message
54:1: I101 Imported names are in the wrong order. Should be compute_friends_self_citations, compute_simple_self_citations, get_authors_tags, update_self_cites_tables
54:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.selfcites_indexer should be before from invenio.legacy.dbquery
55:47: E126 continuation line over-indented for hanging indent
58:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_searcher should be before from invenio.legacy.bibrank.selfcites_indexer
59:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_indexer should be before from invenio.legacy.bibrank.citation_searcher
60:1: I101 Imported names are in the wrong order. Should be fromDB, intoDB
64: D400 First line should end with a period (not 'e')
65:65: E127 continuation line over-indented for visual indent
90:16: E128 continuation line under-indented for visual indent
91:16: E128 continuation line under-indented for visual indent
107: D400 First line should end with a period (not 't')
133: D400 First line should end with a period (not 'h')
144:1: E302 expected 2 blank lines, found 1
144: D400 First line should end with a period (not 'k')
151: D400 First line should end with a period (not 's')
151: D200 One-line docstring should fit on one line with quotes (found 2)
161: D400 First line should end with a period (not 'n')
175: D401 First line should be in imperative mood ('Update', not 'Updates')
175: D400 First line should end with a period (not 'e')
192: D401 First line should be in imperative mood ('Thi', not 'This')
192: D205 1 blank line required between summary line and description (found 0)
233: D401 First line should be in imperative mood ('Return', not 'Returns')
233: D400 First line should end with a period (not 'e')
242: D401 First line should be in imperative mood ('Self-cite', not 'Self-cites')
242: D400 First line should end with a period (not 'd')
255: D401 First line should be in imperative mood ('Thi', not 'This')
255: D400 First line should end with a period (not 's')
267: D401 First line should be in imperative mood ('Thi', not 'This')
267: D400 First line should end with a period (not 'a')
309: D400 First line should end with a period (not 'e')


28:1: I100 Imports statements are in the wrong order. from sqlalchemy should be before import sqlalchemy.dialects.postgresql
28:1: F401 'types' imported but unused
29:1: F401 'compiles' imported but unused
35:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy.types should be before from sqlalchemy_utils
63:5: B901 blind except: statement
68:5: B901 blind except: statement
120:5: B901 blind except: statement


27:1: I101 Imported names are in the wrong order. Should be Cookie, add_cookies, get_cookie
29:1: I100 Imports statements are in the wrong order. import random should be before from invenio.legacy.wsgi.utils
30:1: F401 'zlib' imported but unused
31:1: I201 Missing newline before sections or imports.
32:1: I100 Imports statements are in the wrong order. import re should be before from six.moves
32:1: I201 Missing newline before sections or imports.
33:1: F401 'sys' imported but unused
34:1: I100 Imports statements are in the wrong order. import os should be before import sys
36:1: F401 'datetime' imported but unused
36:1: F401 'timedelta' imported but unused
40:1: I101 Imported names are in the wrong order. Should be blob_to_string, run_sql
40:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.utils.date
41:1: I101 Imported names are in the wrong order. Should be CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSEARCH_PREV_NEXT_HIT_FOR_GUESTS, CFG_WEBSESSION_EXPIRY_LIMIT_DEFAULT, CFG_WEBSESSION_EXPIRY_LIMIT_REMEMBER, CFG_WEBSESSION_IPADDR_CHECK_SKIP_BITS, CFG_WEBSESSION_STORAGE
41:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.dbquery
41:1: F401 'CFG_WEBSESSION_EXPIRY_LIMIT_DEFAULT' imported but unused
48:1: I101 Imported names are in the wrong order. Should be CFG_WEBSESSION_CLEANUP_CHANCE, CFG_WEBSESSION_COOKIE_NAME, CFG_WEBSESSION_ONE_DAY
49:39: E128 continuation line under-indented for visual indent
50:39: E128 continuation line under-indented for visual indent
51:1: I100 Imports statements are in the wrong order. from invenio.ext.cache should be before from invenio.legacy.websession.websession_config
62:1: E302 expected 2 blank lines, found 1
87: D203 1 blank line required before class docstring (found 0)
107: D102 Missing docstring in public method
110:22: F821 undefined name 'CFG_WEBSESSION_ENABLE_LOCKING'
153:30: E261 at least two spaces before inline comment
166: D401 First line should be in imperative mood ('I', not 'Is')
166: D400 First line should end with a period (not '?')
166: D200 One-line docstring should fit on one line with quotes (found 3)
172: D400 First line should end with a period (not 'r')
172: D205 1 blank line required between summary line and description (found 0)
181: D102 Missing docstring in public method
186: D102 Missing docstring in public method
203: D205 1 blank line required between summary line and description (found 0)
218:25: E266 too many leading '#' for block comment
224:29: E131 continuation line unaligned for hanging indent
232:25: E266 too many leading '#' for block comment
238:29: E131 continuation line unaligned for hanging indent
254: D400 First line should end with a period (not 'd')
254: D205 1 blank line required between summary line and description (found 0)
264: D200 One-line docstring should fit on one line with quotes (found 3)
270:13: E266 too many leading '#' for block comment
271:36: E203 whitespace before ':'
272:21: E128 continuation line under-indented for visual indent
272:31: E203 whitespace before ':'
273:21: E128 continuation line under-indented for visual indent
274:21: E128 continuation line under-indented for visual indent
274:31: E203 whitespace before ':'
275:21: E128 continuation line under-indented for visual indent
275:31: E203 whitespace before ':'
276:21: E128 continuation line under-indented for visual indent
276:32: E203 whitespace before ':'
277:21: E128 continuation line under-indented for visual indent
277:35: E203 whitespace before ':'
278:13: E124 closing bracket does not match visual indentation
288:9: E266 too many leading '#' for block comment
291: D200 One-line docstring should fit on one line with quotes (found 3)
298: D200 One-line docstring should fit on one line with quotes (found 3)
311: D400 First line should end with a period (not 'g')
311: D205 1 blank line required between summary line and description (found 0)
339:5: E303 too many blank lines (2)
339: D400 First line should end with a period (not 's')
339: D205 1 blank line required between summary line and description (found 0)
349: D400 First line should end with a period (not 's')
349: D205 1 blank line required between summary line and description (found 0)
359: D200 One-line docstring should fit on one line with quotes (found 3)
366: D200 One-line docstring should fit on one line with quotes (found 3)
373: D205 1 blank line required between summary line and description (found 0)
380: D205 1 blank line required between summary line and description (found 0)
387: D200 One-line docstring should fit on one line with quotes (found 3)
395: D102 Missing docstring in public method
399: D102 Missing docstring in public method
402:5: E266 too many leading '#' for block comment
403:5: E266 too many leading '#' for block comment
404:5: E266 too many leading '#' for block comment
405:5: E266 too many leading '#' for block comment
406:5: E266 too many leading '#' for block comment
407:5: E266 too many leading '#' for block comment
408:5: E266 too many leading '#' for block comment
409:5: E266 too many leading '#' for block comment
410:5: E266 too many leading '#' for block comment
413:1: E302 expected 2 blank lines, found 1
413: D200 One-line docstring should fit on one line with quotes (found 3)
421:1: E302 expected 2 blank lines, found 0
421: D205 1 blank line required between summary line and description (found 0)
440:1: E302 expected 2 blank lines, found 1
440: D400 First line should end with a period (not 'p')
440: D205 1 blank line required between summary line and description (found 0)
466:24: F821 undefined name '_get_generator'
479:1: E302 expected 2 blank lines, found 1
479: D400 First line should end with a period (not 'P')
479: D200 One-line docstring should fit on one line with quotes (found 3)
490:1: E303 too many blank lines (3)
490: D101 Missing docstring in public class
491: D102 Missing docstring in public method
493: D400 First line should end with a period (not 'd')
493: D205 1 blank line required between summary line and description (found 0)
504: D102 Missing docstring in public method
510: D102 Missing docstring in public method
514: D102 Missing docstring in public method
530:13: E128 continuation line under-indented for visual indent
533: D101 Missing docstring in public class
535: D102 Missing docstring in public method
538: D102 Missing docstring in public method
541: D102 Missing docstring in public method
544: D102 Missing docstring in public method


no problem, I recommend also placing the .setFastestInterval on this request as well so it can receive from other requests otherwise the priority is set to the defaults


Since `store_input_hook` and `store_output_hook` are globally set configs, I think it would still be nice to be able to get that out of an individual use of `run-hook` regardless of these config settings; maybe it should be an option that you can pass to the command to turn the somewhat noisy inclusion of input/output in `run-hook`'s response on or off.


The problem is that I'm using the 'eventorganiser_save_event' hook to do
other stuff - specifically, to save group associations. So if I want to
use 'save_post' for creating activity, it'll be too soon to catch
connected groups.

I can just abandon the idea of using 'eventorganiser_save_event'. I'll
try doing a priority dance to see if I can get things to fire correctly.

On 04/22/15 14:06, r-a-y wrote:
> Hmm, I would rather not introduce a special after save hook when the
> |'save_post'| hook should do the job in normal circumstances.
>
> What if I brought back the autosave feature? And when someone clicked on
> the "New Event" button, this would pull up the last event draft for the
> current user if one exists and displays a message like "It looks like
> you have an existing draft. Please continue editing or discard this
> draft and create a new event" above the editor? If an event draft does
> not exist, it would create a new auto-draft post and autosave would kick
> in as normal.
>
> Would this fix the majority of these problems and not require you to
> jump through hoops to determine if an event is new or not?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuny-academic-commons/bp-event-organiser/commit/2e920d71ea807022d31eceeac81f3afe41d3b0f9#commitcomment-10852865>.
>


OK, then you need to double check it. I loaded a chord example in the editor, and when drawing a note to a unison it seems to crash on this assert. Let me know if you cannot reproduce it and I'll make an example file.


That makes sense, I would add that in your PR back to the repo


Shouldn't this be '[optionsSchema]'?


@maryam-osama fine :)


All the lonely people....


what did you change in projectDetails.html? Can you write it in plain text?


That solves the bug of not sending messages. Thanks!


eg. see here. We need to know if the encoding here was correct or not. So in harness you need to decode this string and verify that it matches the one that's in sourceMapData list that has non encoded version of this.


Do send me the example.


Yay! I won't touch these styles in my patch then :)


trivial: missing backtick after `@EnableWs


@e-kolpakov Not quite :-p

```python
>>> "".strip() == ""
True
>>> "".isspace()
False
```


Added this. It exposed a bug in strict checking. That was fixed on master when I fixed the todo.

I rebased and added more tests.


Medio raro lo de la tasa de importación... está bueno que sacaron la clase porque (como van a ver la clase que viene) esa clase realmente no aportaba nada y no está bueno; pero en realidad dice que la tasa es siempre 30 y no variable por cada clase que definimos, por lo tanto no sé cuanto sentido tiene que venga como parámetro.
Yo la dejaría fija para la clase y manejaría algún valor para determinar si es o no nacional.


These changes shouldn't be here, for the reason that the README.md defines what is going on here overall. It isn't for train of thought. Also the images won't show up, if you want you can create an `imgs` folder put them there and link to them like this: `![design 1](/img/design1.img)`


This line looks weird (mixed CDATA + child elements?).


@e-kolpakov Oh yeah. It was going to be a parametrized test, but in the end I realized it was a more comprehensive test if done this way, where blocks are tested in pairs. So I don't need the `ddt.ddt` anymore.


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


Es una pavada, pero en realidad la idea de la marca es que le hacia un recargo al precio final, o sea a "(valorFijo + this.precioBase()) * tasaImportacion" y no solo al precio base.
Presten atención al enunciado para el TP :)


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Do we need this Suppression on public api?


Me gusta. Una interfaz para no perder la posibilidad de herencia.
Y he aquí un strategy.


see above


Ok. Will revert my commit. Funny that not a single test fails due to that change. That tells you a lot about the code quality in ATContentTypes. ;)


Not sure if this can really happen - could we somehow avoid throwing exceptions? - I'm just curious


New error message looks like the following:

Deploy `/Users/smith/myapp-1.0.15.tgz` to `http://localhost:60077` failed.
An error occurred while processing the application tarball: Invalid tgz file
HTTP Code: 400


see above, unused suppression on public api


Hmm I got little confused with inline sources and inline sourcemap.. I meant that you need to verify that this text here (inline sourcemap) is same as the decoded one..


sacrificial *


Note: This commit belongs to #266 not #216.


But nor working :(


Makes sense that this is pointing to a release tag, but maybe we should have some instructions in here for how to update as well.


We should not use "REGS".  Maybe use one of our variables from the config?


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1647 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)


En otro lado usan un static, ahora acá usan un valor hardcodeado para el precio. Medio raro, no? Analizando las cualidad de diseño, les parece consistente?
Aparte... todos odiamos el hardcode :P


See this red icon - I think Git prefers a newline at the end of each file?


data-slider=... ? Do we need a value here? No we don't


Do you mean instructions for migrating from the Framework integration?


Macs are picky that way! @timothykang


See comment above - use org-name or something.


`isVisible` may have issue if your question list gets long and flows off-screen!  you may consider `moveToObject` or similair.  http://webdriver.io/api/action/moveToObject.html


moved


Worked!!! thank you sir


Would be great if we could choose a free port dynamically (like in other ITs where we start a local jetty or something)


vehicle.sqf is still mentioned here: https://github.com/A3Wasteland/ArmA3_Wasteland.Altis/blob/Development_main/server/functions/serverCompile.sqf#L75


This commit broke the build. Please remove ASAP!


Yes, I agree with both of these points. I will put together a definition
of the wreath product as well as an example and some motivation. I
certainly needed that for myself.

On 4/21/15 3:15 PM, Erick Matsen wrote:
>
> This is a great start, but given that the audience of the paper will
> be computational biologists it would be good to give a bit more
> definition.
>
> Also, I don't think that it's the d-fold wreath product, but rather
> the wreath product of the automorphism group with the symmetric group
> on d letters. Take a look at
> https://en.wikipedia.org/wiki/Wreath_product, especially
>
> image
> <https://cloud.githubusercontent.com/assets/112708/7263728/14e25dda-e839-11e4-97ea-1f7c639462dd.png>
>
> image
> <https://cloud.githubusercontent.com/assets/112708/7263721/102f5d56-e839-11e4-890e-5de780c4f8b1.png>
>
> We wouldn't have to introduce the wreath product in general, but (I
> think) just the wreath product with a symmetric group.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/matsengrp/tangletex/commit/ed94316aea9c7cb6139bbb5f01f91be50bebef9f#commitcomment-10835596>.
>


What does this mean for apps? Do you plan to update rest.ip?


Wonder if we should display the help icon in front of the text? Adding `<span class="cf-icon cf-icon-help-round"></span>` might give the button/link a little more weight visually.


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (341 of 341 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)


More like "You can look here (https://github.com/URXtech/urx-sdk-ios/releases) for new releases, and just increment the version number to the latest version and rerun pod install" (I'm assuming that cocoapods will just pull down the package if you point it to another commit/tag, this might be totally technically inaccurate).


This looks good. I think there's a few other directive we could go back and give this treatment too. Not all of them should be restricted to Element directives.


Nice one. Just some trivials and nice to have.


So clean :sunglasses:


I didn't spend a lot of time on the blog stuff - does our list page show the complete entry of each post?  If not, we'll need the ability to click through to a full post page.  Unless maybe I'm reading this wrong?


Fixed in v13.


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


Ok, this makes sense.


```lua
image.display({image.lena():transpose(2,3),image.hflip(image.lena():transpose(2,3))})
```
![screenshot from 2015-04-22 14 33 17](https://cloud.githubusercontent.com/assets/2119355/7282220/80ae9c04-e8fc-11e4-955c-bb02aefb27a8.png)


Makes sense so far


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


After looking at the `'eventorganiser_save_event'` and `'save_post'` hooks some more, it looks like we need to hook into 'save_post_event' to do the initial save and check the third parameter to see if $update is false.

Something like:
```
add_action( 'save_post_event', function( $post_id, $post, $update ) {
	// bail if update
	if ( true === $update ) {
		return;
	}

	// create initial activity item here
}, 10, 3 );
```


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (340 of 341 tests passed)

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


It's actually only in the editor. Maybe because the order of the notes is
not updated when redrawing.

On Wed, Apr 22, 2015 at 8:18 PM, Andrew Horwitz <notifications@github.com>
wrote:

> Do send me the example.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DDMAL/verovio/commit/7ec015c29fc7c800152c50755c30ce2be32d7aaf#commitcomment-10853138>
> .
>


Wow it's true that we create a ton of clients. I had never paid attention to that...

We can easily refactor this by creating one client per destination cluster during initialization, and then re-using those cached clients whenever needed.

We should perhaps make this change as a separate commit however, and fix it everywhere in one shot...


# WRONG!


Just realized: we should start the timer in `viewDidAppear` and stop it in `viewWillDisappear`. You might need to track the timer ID or token.


Looks really good. Tested 3 different TV shows. Before that, only the first episode was working. Thanks man.


Was there a reason for removing zsh support?


I have no problems with this being removed if it's not easily fixable.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/a542ec01f695ab89118da537ca7d4917340966e0/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/a542ec01f695ab89118da537ca7d4917340966e0/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/a542ec01f695ab89118da537ca7d4917340966e0/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/a542ec01f695ab89118da537ca7d4917340966e0/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/a542ec01f695ab89118da537ca7d4917340966e0/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Are we checking the *contiguity* of `src` or `dst`?


U serious? No mysqli please! Use database.php! It already does everything for you, with no magic strings or anything! (By magic strings I mean putting the database details directly in as strings. We obviously will need to change things later so it's better to keep them all variables so we only have to change one thing)


rax retry tests


Yeah, that's what I've been trying to do for several hours, but there
are some irregularities in the way that EO works that make it impossible
to trust `$update`. I'll continue to hack at it.

On 04/22/15 14:33, r-a-y wrote:
> After looking at the |'eventorganiser_save_event'| and |'save_post'|
> hooks some more, it looks like we need to hook into 'save_post_event' to
> do the initial save and check the third parameter to see if $update is
> false.
>
> Something like:
>
> |add_action( 'save_post_event', function( $post_id, $post, $update ) {
>      // bail if update
>      if ( true === $update ) {
>          return;
>      }
>
>      // create initial activity item here
> }, 10, 3 );
> |
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuny-academic-commons/bp-event-organiser/commit/2e920d71ea807022d31eceeac81f3afe41d3b0f9#commitcomment-10853460>.
>


Also, what name should we add to the credits?

On Wed, Apr 22, 2015 at 1:14 PM, Ferran Pujol Camins <
notifications@github.com> wrote:

> Thank you for the fixes @rryan <https://github.com/rryan>! Shall I sign
> the contributor agreement?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mixxxdj/mixxx/commit/3ddc39393cf0d4e055ab06daece80021d705c512#commitcomment-10851827>
> .
>


pourquoi avoir supprimer tous les espaces ?


Cool. Thing is, I can't log in locally. I'm going to add a `&& $_SERVER['HTTP_HOST'] != "localhost"`


ton commit ne va pas du tout. Je ne sais pas pourquoi tu touches à l'indentation des fichiers comme ça. Et pourquoi tu supprimes les retour à la ligne.

A refaire.


Nice :+1:


This line introduces an annoying dependency that prevents using ThermoDatabase without without creating an RMG object.

It's not just an unwanted dependency from a module that is higher in hierarchy, but it also forces the existence of the RMG object at runtime.

I believe we could tackle this, by improving the design of handling thermochemistry in the liquid phase.


Ok, I agree with @benfrancis. I need to go back and clarify some of these and we might need to add new metadata names to use with meta elements.


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


Would we ever want the auto controller to increase the altitude target while we are landed and the motors are off?  The only thing I could think of, would be if it needed to do this while disarmed or something?  But I'm not sure why we'd ever do that.


yes, just need them to be in the same order every time


'OGC' should not be hardcoded. We should be able to read all config files, then sort them by organization and then  alphabetically.


@rmjarvis or @rmandelb , could one of you take a look at `SBInterpolatedKImage::SBInterpolatedIKImageImpl::getRealKImage()  (and getImageKImage())`?  I'm suddenly getting a segmentation fault in my unit tests (as of 299dd4ad) which used to pass.  My guess is I'm doing something memory unsafe here, which happened to work before but isn't guaranteed to work in general (possibly using `ImageAlloc` incorrectly?).  If this isn't it, then I'll keep looking, but I want to sanity check this first.


Felix, I agree we should fix the existing issues in one single commit. But My point was let us not introduce the new ones in the mean time.


You have lots of options.  I think we'll want to change the integration to:
```
pod 'URXSearch',  '~> 0.3.1'
```

Or, the user can just use:
```
pod 'URXSearch'
```

Which should pull the latest version.  I can make these changes.  I did this before I set up the private CocoaPods repository so I could only test via Github.  Since we plan to push this to the public repo, we should avoid `:git` and `:tag` as they are handled through the repo and Podspec.


review is done


Have the parameter in the Database object (which would exist at runtime) instead?
And have the reading of an input file etc. set this in the Database object?


Is this at risk of blowing the 5 second timeout?


What about checking if the `post_date` and `post_modified` date are the
same?  Yeah, I know that is even hackier.


Hurrah for automatic initializers!


Ну кароч, в результате-то все тосты выкинуть надо из moveSense. Нужно, чтоб они срабатывали из main.


I think the `to` at the end of this line is a typo and could be removed


That's what I originally did in 81d3ac8923b6796, but I switched away because it was a crummy hack :) But it's looking like there's no way to make this a non-crummy hack, so maybe I should just switch back.


que tal `map_and_find` ? pq ele primeiro rola o map pra fazer o find em cima do map.


pretty sure it’s 2015 now


rax retry tests


Не получилось сделать вывод таблицы рекордов из файла.
Не пойму как структуру из файла записать в массив.
Сюда пока не выкладывал, еще буду думать как сделать.


Added the Gmail Stats example.


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


This Commit causes errors when using POS in conjuction with the restaurant addon.
addons/point_of_sale/static/src/img/scroll-down.png and "addons/point_of_sale/static/src/img/scroll-up.png" are missed then.


TROUSERS


We don't have much in the way of written docs yet for SCI2, but we can talk about how to coordinate things. Do you use IRC? Most of us talk in #scummvm on freenode.


Hello,

Can you also add Contact items?
That would be awsome


I think I may have commented on the wrong post, but I am trying to figure out how to get Java3D working with 3D viewer in ImageJ. I have installed the Java3D plugin, but when I launch 3D Viewer in ImageJ, it gives me an error that "Java 3D version 1.3 detected, but version 1.5 is required." I launched the demo jar file to test if Java 3D is installed and it works fine. If you can point me in the right direction on how I can resolve this issue, I would greatly appreciate it. Thanks


fixed, thx


All `encode` instances are the identity.  This surprises me.  Given that `T` extends JSON, what are you anticipating here?


время исходя из ловкости вообще четко считаешь. Лойс тебе. Другой вопрос мы на вход moveSense даем (контекст, силу, код действия, ловкость) Так вот Сила кароч нужна чтоб при тряске определить сколько раз нужно тряхнуть. Так что высчитай: сила может быть от 1 до 100 и чем ее больше тем меньше нужно трясти раз, при условии, что разов может быть от 2 до 8 где-то так.


This is so cool! I can't wait to see it live! :heart:


это получается таймер каждую секунду отрабатывает. Я думаю, нужно почаще, поставь (time, 500) или еще чаще


U dont need to think about recursive inclusion, as long as u have ifnded, define ... at the beginning of header files.
I agree with u probably we dont need to add get


Seems like this was renamed from `invalidOwner` to `invalidEnclClass`


Sure, but that'll be in a different pull request.

I'd have to get some of those sensors first. What kind are you using? Door/window sensors?


I thought you had nice response logging?  We should use it more generally.  Are we worried about the log leaking on iOS?


Homeworks


тута у нас количество встрясок. цифра 18,, это вроде 3-4 встряски. Нужно чтоб число тут было завязано на силу персонажа так, как я писал выше


Java-Course


I understand this desired, but it doesn't require `shared_ptr` in the interface.  With a small addition (additional inheritance for Data/Interest abstraction), it is possible to obtain shared_ptr from the object instance.

    class Name : public std::enable_shared_from_this {
    ...
    };

Specifically for Name abstraction, I would suggest that we don't worry about copying it.  In theory, it should be made cheap.  Getting `shared_ptr` of the name in the interest filter is a little bit weird for me...


@rodrigopavezi is there an extra bracket `(` here? Thanks.


Looking at this a little bit more, I'm concerned about the effect on Heli.  I think it may be non-critical, but the logic gets a bit messed up.

Ok, I'm going to straighten out the logic a bit just by renaming output_min() in Arducopter into enable_motor_output(), to avoid confusing this function with output_min in AP_Motors.  That's part of the confusion about what and when "output_min" does.


i just bought them didn`t know i couldn`t use them yet,
I have the klikaanklikuit/coco amst-606 and the awst-6000 motion sensor


**Resumen del PR #156**

:rocket: Es una primera versión sencilla (fiel a los reports `html`), pero e intentado diseñarlo de manera que sea fácil modificar: añadir más columns, otros modelos...

Resumiendo (solo por si alguien le interesa):

* Los reports son services (app/services/report/) `Report::CSV` y `Report::PDF` donde cada uno define su metodo `run`.

```
Report::CSV.run(headers, rows) # se encargar de generar el csv o pdf
```

* Cada modelo que tenga "report" se define en una clase propia `Report::CSV::Member`, ...

```
report = Report::CSV::Member.new(org, collection)
report.run
report.name
report.mime_type
```

* Cada colección necesita ser "decorada" (app/decorators/) para generar el contenido (`report.run`): los headers y los rows.

```
decorator = MemberReportDecorator.new(org, collection)
decorator.rows
decorator.headers
```

* Las mismas acciones de controller que generan el report `html` se encargan de generar los reports, respondiendo a su correspondiente mime_type:

```
respond_to do |format|
  format.html

  format.csv do
    report = Report::CSV::Member.new(current_organization, @members)
    send_data report.run, filename: report.name, type: report.mime_type
  end

  format.pdf do
    report = Report::PDF::Member.new(current_organization, @members)
    send_data report.run, filename: report.name, type: report.mime_type
  end
end
```


Ooops! I've intentionally re-enabled the dark theme for GTK3, but forgot to submit a simplified css theme file... I'll fix that, thanks for pointing it out!

Anyway, the GTK3 theme is still quite buggy and not looking as good as the GTK2 one (which was actually derived from RawTherapee). I would definitely need help from someone with a good knowledge of GTK3 theming.


also at this stage we don't need utils.cpp, will see if we need it later.


Can we give `f` a better name?


Suggest factoring these two lines into an `_itemActivate` method


Consider moving all public api stuff above other methods but just below the specific lifecycle callbacks.


Zsh support was never up to par. It seemed better to reduce complexity than to offer poor support.


Is it possible to allow a selector like we have in more modern elements?


This looks weird.  If there wasn't  `<>`, you'd write it
```
factor(f: JSON -> T) -> (String -> T?) to emphasize the HOF, right?


Just FYI I think this reintroduces issues with instances that don't have GLIBC_2.14, due to changes in 3.0.0 of pngquant-bin (at least that's my guess on what happened).  We had a build that worked last night but not this morning and it came down to finding these commits here.  Updating our dependency from "^4.0.0" to "~4.0.0" worked for us for now but I thought I'd just let you know about this in case others come up with the same issue.


Кароч тут где-то пиздани статический метод публичный. В который на вход принимаются данные из moveSense, к примеру в виде массива. И этот метод вызывай из moveSense когда его отключаешь, т.е. в isUnregistred. Вот у меня в магии есть такая хуйня
![2015-04-22 21-55-47](https://cloud.githubusercontent.com/assets/11233404/7282706/6cdb9748-e93a-11e4-8b17-67d09db492ee.png)
Её я вызываю из класса где магичат, и на вход отдаю, какие клетки были зажаты. Как-то так.


We have `String.utf8data` IIRC.


Looks like we now need to filter `hscolour` options a-la `filterConfigureFlags` to support setup scripts built with pre-1.23 Cabal.


@alex-deaconu good catch. I will remove it.


It's not crummy if it doesn't require adding a bunch of new code (especially `debug_backtrace()`) to try and address this ;)

The chances of the post_date and post_modified date being the same after the initial post creation should be quite rare.  I'd be okay with switching back if it's okay with you.


Tag interface (no methods) ? hmmm....


We don't have a release of this yet do we?


nit: dedent `])...`


@jonathanong Are you having any specific trouble? I haven't come across any problems covering generators. The reason for using Babel is to cover features that istanbul itself does not understand, so if you blacklist regenerator, I think istanbul will barf until it is up to date. Also, what would including the runtime accomplish? Thanks.


@h-kataria Is this a bug?  Shouldn't this be using descendents rather than subclasses?


ExtractCoremesh --> ExtractCoreMesh?


Worth a comment on the IVs here.  Does `iv: nil` generate a fresh IV, or is it like all zeroes or something?  Is there a Sync limitation on the IVs?


Yeah, you're right, but we don't have separate file for ```Composer``` view yet, I can move it to ```views/thread/thread.css```, does that sound good for you?


`cabal-install` has `setenv` in `Distribution.Client.Compat.Environment`.


Yeah. Okay. Blech. I'll do it.

On 04/22/15 14:58, r-a-y wrote:
> It's not crummy if it doesn't require adding a bunch of new code
> (especially |debug_backtrace()|) to try and address this ;)
>
> The chances of the post_date and post_modified date being the same after
> the initial post creation should be quite rare. I'd be okay with
> switching back if it's okay with you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuny-academic-commons/bp-event-organiser/commit/2e920d71ea807022d31eceeac81f3afe41d3b0f9#commitcomment-10853897>.
>


Possible alternative verbiage:

       * The event that fires from items when they are selected. Selectable
       * will listen for this event from items and update the selection state.
       * Set this to empty string if you don't want to listen to any events.


This is a non-functional chance, just semantics for clarity.


Though not `unsetEnv`, but that could be added. No big deal if it's only test suite.


I think the solution will be cleaner if we fix it in one shot, rather than with a one-off. I don't mind whether it's fixed everywhere as part of this change or not, but if we are planning to re-visit this soon anyway, it's probably not worth doing a one-off...


I have prepared the following commit for KotlinJavascriptSerializationUtils, which should work
for both old and new names of metadata files for JS (up to M12, when FallbackPath will disappear)
https://github.com/JetBrains/kotlin/commit/1a420f9ffd9b5901282b2e597e8cce74820a93ca


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1647 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)


let's use https://developers.google.com/closure/compiler/docs/js-for-compiler#tag-enum ?


Hi @mattab This error is always irritating, **"Your Piwik codebase is running the old version %1$s and we have detected that your Piwik Database has already been upgraded to the newer version  Maybe your Piwik administrators are currently finishing the upgrade process. Please try again in a few minutes."** because even after one hour, this error does not go away and it seems that the developer is struck with the index page and could not do anything.  It would be great if the given codebase version and database version always stay in sync because in that case developer is free to access local piwik site until new (codebase + database) version update is available. I have noticed this error several times. Maybe, we need more sensible and accurate update reporting.


Did you mean to *add* a new curry?


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


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (341 of 341 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)


Dangerous.  Log here, for sure!


As this is, it won't create a folder, it tries to execute that file. Is that what you intended?


Sorry @bgruening, just noticed this now that I'm converting things to v2.


Verify that the intention is for the clients/tabs synchronizer to manage this timestamp?


Hi Lalit,

The reason for this note is to introduce myself to you, and to make you aware of a senior role we have to fill. Our client is a technology company that successfully applies disciplined, process-driven investment strategies based on statistical models developed using rigorous mathematical analysis. They are recruiting for a senior level software engineer to build proprietary algorithmic research systems capable of executing extremely high levels of computation and will detect anomalies in massive datasets i.e. Big Data. Candidates will likely hold a degree in Computer Science or Electrical Engineering, however all backgrounds will be considered, and experience building highly available systems in an industrial setting.

Would you like to learn more about this opportunity? Kindly let me know your thoughts.

Brian McNeill
Alpha Advisors
(914) 584-9471
bmcneill@alphaadvisorsllc.com
http://www.alphaadvisorsllc.com/
http://www.linkedin.com/pub/brian-mcneill/43/a4b/343/en


@d1manson, sorry, that's not going to happen :-(

These are our reasons:

1. We also removed support for html support in our qtconsoles since Spyder 2.3.3 to follow Pandas, which had also removed support for it a long time ago.
2. I also did several other changes to only support IPython 3 (like removing conditional imports). The thing is supporting several IPython versions is not easy, and we prefer to maintain compatibility with the latest one only :-)


I don't think, that changing now to spaces indents is the best idea. I don't like tab indents too, but reformatting entire project seems to me pointless.


wow, that was really 2013!

thanks for this use case. now i understand.
hm.... i'll look into it.


I want to have all the XYZTempResult classes implement this.


@candelom @Kirill23 One of the recent commits to search_api_solr broke facets. The make dropdown doesn't work anymore with the new branches. However, if I revert back to the older version with just geolocation change, then it returns. So you two need to look into this and see where it broke.


now having a bug where a download appears as fully downloaded immediatly... this never ends.


This feels pretty weird.  Can we get @st3fan to comment on whether this is a reasonable approach?


I think that having in the input file the libraries separated i.e. thermoLibraries = [] and liquidThermoLibraries = [] is a good idea.
But I (personally style wise) don't like that there is solvation stuff in thermo.py, anyway. I feel like it should be all in solvation.py. I can see why not, though: solvation.py all deals with solvation "corrections" and this is about thermo libraries that have already been "corrected" (gas+solvation thermo).


`eventorganiser_details_save()` is intended only for saving events created using the normal WordPress' "post" UI - so it only ever *updates* events because by the time the process reaches it, the event is already created by WordPress.

If you need to check when the event is first created, couldn't you use a metadata flag?


:+1: for getting this upstream - it seems like jenkins core is pushing hard to deprecate the Build Flow Plugin in favor of the workflow-plugin, and with the latest LTS 1.596.2 they've broken things so that flow jobs no longer automatically trigger instantiation of EC2 on-demand slaves.


@dwaynebailey You are right. I just went crazy. Fixing it.


Could we use Notification Center to serve as a decoupled message bus here?  So that when Sync sees a tab, it just broadcasts, and the ambient `UIApplication` -- if there is one -- reacts as appropriate?


(By 'saving' I mean saving the meta data / dates etc)


Début de modification de abonné


This requires an `app` and otherwise fails silently, which sounds terrible.


i'm just getting errors where it says `runtimeRegenerator is not defined`. i'm guessing it's because you don't include the runtime, but since i'm using iojs, we should just blacklist generators.


@brandondunne the code above is using SUBCLASSES constant that's defined in each of those models, if replacing those calls with descendants fixes the issue, then i good with that. There might be more places in code where we use SUBCLASSES in think in treebuilders while building VM explorer trees, look in presenters directory.


Your Playlist class already is an ArrayList<Track>, so it shouldn't contain one. :) You can use 'this' instead of pl.


The fact that this is necessary strongly suggests our abstraction is wrong here.  We should think carefully about this.


> eventorganiser_details_save() is intended only for saving events created using the normal WordPress' "post" UI - so it only ever updates events because by the time the process reaches it, the event is already created by WordPress.

Right. @r-a-y's got a fancy system that apes the Dashboard interface while on the front end. So we effectively are using the "post" UI. The "event is already created" bit is just what the problem is in this case.

> If you need to check when the event is first created, couldn't you use a metadata flag?

Blech :) There is no good solution here. At least the post_date/post_modified comparison won't leave cruft in the database.


Could you confirm I signed the right thing? (A google docs form)
You should add "Ferran Pujol Camins".

Thank you all for the support through my first contribution :)

2015-04-22 20:39 GMT+02:00 RJ Ryan <notifications@github.com>:

> Also, what name should we add to the credits?
>
> On Wed, Apr 22, 2015 at 1:14 PM, Ferran Pujol Camins <
> notifications@github.com> wrote:
>
> > Thank you for the fixes @rryan <https://github.com/rryan>! Shall I sign
> > the contributor agreement?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mixxxdj/mixxx/commit/3ddc39393cf0d4e055ab06daece80021d705c512#commitcomment-10851827
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mixxxdj/mixxx/commit/3ddc39393cf0d4e055ab06daece80021d705c512#commitcomment-10853565>
> .
>


Granskad!
![fg2wcv7](https://cloud.githubusercontent.com/assets/442870/7282989/be72cc36-e933-11e4-82cc-9bc0d70a0064.gif)


Apparently this got better in 1.2?


Sure sir


Granskad!
![ubgsuoj - imgur](https://cloud.githubusercontent.com/assets/442870/7283008/d42ed8da-e933-11e4-8979-2be749d078dd.gif)
Teamwork!


I understood this as saying that we will use tabs, which have a size equal to 4 spaces. This seems reasonably compatible to me since we currently use tabs. The number of spaces shouldn't be terribly important, as long as all the editors are configured to leave a tab character rather than a space character.


I think we ended up on `org.mozilla.ios.FennecAurora`, so this should follow suit.


these symbols need to be prefixed with `mono_`


`ctx` is destroyed by the `deflate_rsrc_dtor()` destructor which is called whenever the associated resource is garbage collected. It doesn't need to be freed if an error occurs inside `deflate_add()`.


probando


This is funky -- we should reflect something better here.  iPad vs. iPhone at least; and hopefully size and/or model.


In KoreanTokenJava class, the result of offset() method seems weird.
Input:
"한국어를 처리하는 예시입니닼ㅋㅋㅋㅋㅋ한국어를 처리하는 예시입니닼ㅋㅋㅋㅋㅋ"

Output:
[한국어(Noun: 0, 3), 를(Josa: 3, 1), 처리(Noun: 5, 2), 하다(Verb: 7, 2), 예시(Noun: 10, 2), 이다(Adjective: 12, 3), ㅋㅋ(KoreanParticle: 15, 2), 한국어(Noun: 17, 3), 를(Josa: 20, 1), 처리(Noun: 22, 2), 하다(Verb: 24, 2), 예시(Noun: 27, 2), 이다(Adjective: 29, 3), ㅋㅋ(KoreanParticle: 32, 2)]


Sorry, this text was not clear enough.
I would tell that a tab has a size of 4
And Tabs should be used for indention.


couldn't we just have C functions with inline asm?


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (341 of 341 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)


I know, but (for me) "tag interfaces" is not a good signal of a sound design.

This bug? are you able to run FrostWire? How?


@dvacherot great file!

Another reason to be wary of fixed bid contracts is that good, experienced developers tend not to accept this form of payment.  In other words, if you do land such a contract, you've likely hired a bad or inexperienced team!

Additionally, one small typo to fix before submission:
- Line 17 should read, "incentive" rather than "inventive"


I've removed the 5 second wait and now I'm making it non-blocking on the UI thread


Nevermind, I can't reproduce the issue. I may have used an old build earlier. Sorry about that.


Granskad!
![hkmuq - imgur](https://cloud.githubusercontent.com/assets/442870/7283083/3a8b6378-e934-11e4-9368-51b7ef5fde07.gif)


This has been a bugbear for ages.  Let's reduce this, a lot, and upload our client record more frequently too.


I ran out of steam part-way through this commit. Eating lunch instead.


Sorry to nit, but closing tag as well.


@timothykang Your changes to the management agent make the webserver die on our boxes.  I worked my way up through changes and everything works up till this one.  Perhaps you changed the config file as well which is why it wouldn't be in the management demo code?  Though I am not 100% sure.  I just know the head of master doesn't work and the last commit 8641fb3 does work.


this looks unrelated


Plugins have been renamed and moved to npm. Please read http://cordova.apache.org/announcements/2015/04/21/plugins-release-and-move-to-npm.html


ditto


Ok, so here's an issue.  I had planned to link-up the motor interlock Aux Sw function with the tradheli motor control switch (currently hard-coded to Ch8).  Motor Interlock would replace much of the logic for switching the motor on/off in tradheli.  However, if I change the interlock status to output_min instead of output_armed_zero_throttle, then tradheli will fire this function ever time the motor interlock is disabled:

https://github.com/diydrones/ardupilot/blob/master/libraries/AP_Motors/AP_MotorsHeli.cpp#L264

That would mean that not only the motors will stop, but also the servos will stop.  So obviously that won't work at all.

So if I do what you request here, I'll need some complicated logic in Tradheli to straighten this out.

Alternatively, could keep output_armed_zero_throttle and just add functions to Tradheli, tricopter, etc.

Another alternative, would be some kind of virtual function, where all the multirotors would basically just run output_min, while tradeheli would do something else.  I think this is the best choice actually.

Make a virtual function for output_armed_zero_throttle where the basic definition is simply to do output_min.  Then Tradheli would overload it with something where the servos move but the motors stop.

Does that make sense?  I'll hold off on this until I hear back.


Heh, looks like I added this confusing rule a while ago :) Yeah, will leave it here, we'll refactor our styles file by file later on anyway


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (341 of 341 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)


I don't think you need to quote nil. Nil is nil...


Thanks @mwringe


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/1a396be69daaffdf4a178a194d7925b1245b45db/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/1a396be69daaffdf4a178a194d7925b1245b45db/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/1a396be69daaffdf4a178a194d7925b1245b45db/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/1a396be69daaffdf4a178a194d7925b1245b45db/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/1a396be69daaffdf4a178a194d7925b1245b45db/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (339 of 340 tests passed)

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


yes, [here](https://github.com/iojs/nan/blob/207a62d6449129d1b8779109fb7c09d97b8e10bd/nan.h#L821) it is


Gah. Stupid closing tags. Who needs them? They've been holding me back all day!

(Thanks)


and the old Size is deprecated:
```
    NAN_DEPRECATED NAN_INLINE int Size() const {
      return size;
    }

    NAN_INLINE int length() const {
      return size;
    }
```


Not bad. But why?


That should be fine.


method_missing (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (341 of 341 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)


Fixed in v17.


It's also the curse of documentation, which tends to get out of synch -- especially since it's not checked by validator, etc.


method_missing (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-warnings-yellow.svg)


Will there be a release with this in it anytime soon? It would be nice to be able to build the latest release with up-to-date versions of Xcode again. (Homebrew/homebrew#38494)


Are you enabling the config option?

On Wed, Apr 22, 2015 at 4:52 AM, girish-raghavendran <
notifications@github.com> wrote:

> After doing these changes, still I am not able to get the swsusp command
> listed in help. Code compiled without any error. Is there anything need to
> be done apart from the things mentioned here
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/russdill/u-boot/commit/92915e99bacab949e0904ca8a6f3da0319d6c0f0#commitcomment-10845223>
> .
>


Yeah. It'll be gone in two days anyway.


Ah, that does make sense. I'm pretty sure blacklisting won't work in your case at all. If regenerator is blacklisted, istanbul will not be able to actually process the original code. I'm not 100% certain of that though.

You should edit the installed node_module to check this out. Line 403 of `lib/instrumenter.js` is where the compiler gets its options, so add in regenerator as a blacklist there. If you run your tests with that done and it all works, I'll add it in here.

Keep in mind this is pretty much a stop-gap until istanbul catches up, so anything we do here is pretty much a hack. Also, if you want to fix your problems right now, you can require the `babel-runtime` once in your own tests.


Nice!


Hi Brian,
This opportunity sounds interesting to me. Do give me a call: 919-601-3402,
anytime after 1pm tomorrow to discuss.

Thanks,
Lalit

On Wed, Apr 22, 2015 at 3:03 PM, Brian McNeill <notifications@github.com>
wrote:

> Hi Lalit,
>
> The reason for this note is to introduce myself to you, and to make you
> aware of a senior role we have to fill. Our client is a technology company
> that successfully applies disciplined, process-driven investment strategies
> based on statistical models developed using rigorous mathematical analysis.
> They are recruiting for a senior level software engineer to build
> proprietary algorithmic research systems capable of executing extremely
> high levels of computation and will detect anomalies in massive datasets
> i.e. Big Data. Candidates will likely hold a degree in Computer Science or
> Electrical Engineering, however all backgrounds will be considered, and
> experience building highly available systems in an industrial setting.
>
> Would you like to learn more about this opportunity? Kindly let me know
> your thoughts.
>
> Brian McNeill
> Alpha Advisors
> (914) 584-9471
> bmcneill@alphaadvisorsllc.com
> http://www.alphaadvisorsllc.com/
> http://www.linkedin.com/pub/brian-mcneill/43/a4b/343/en
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lponnala/lponnala.github.io/commit/7bd9f05cb50b69e75d2fef1ca79e432512091098#commitcomment-10854001>
> .
>


extra line that shouldn't be in here


You're not applying a patch, you need to merge his commit


ça a l'air mieux


LGTM


Yeah, I probably compiled this using a newer version. It's really hard to keep backwards compatibility with CentOS and all other distros. I'll see if I can fix another binary.


> Keep in mind this is pretty much a stop-gap until istanbul catches up, so anything we do here is pretty much a hack.

i don't think istanbul will ever catch up =/ babel moves too fast lol


It's complicated.  The string types are different between release/debug builds (GUI/console), and then there's ANSI/Unicode.

I've simplified this a bit in v17, and I still believe querying via GetCommandLine() is fine.


Yes, I was thinking to make it applicable to more use cases we'd add a
method limit() to Elements.

On Wed, Apr 22, 2015 at 12:00 AM, Sravan Ravi <notifications@github.com>
wrote:

> Hello,
>
> Good point. I added this as a convenience method only for this particular
> one because this is the one I use most.
>
> Are you suggesting that a chain-able trim(int limit) method be added?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sravan953/jsoup/commit/d4f190c436271530753e30e2e5c36dcb5cff068b#commitcomment-10840586>
> .
>


Yep, that's my excuse.  I think it is fine.


Never refer to the data layer from within the UI layer,  always use the business objects instead.


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


I've pushed similar fixes.

The release version 1.4.0, and isn't "release_1_4_0".  Other than that, most of the changes look good.


True story!


Почему это $mas[$i] > 0 ? А LIMIT 0,5  как же будет работать?
Вообще тут возможно 2 варианта - либо один аргумент, либо два. Можно просто второй необязательным сделать, зачем так все сложно делать?)


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


Неплохо было бы функцию упростить и сделать более читаемой.


@CaptainElliott Well I'm too young to be sued - however, this repository can be DMCA'd.

And wouldn't I have to pull it on another branch then merge master and the second branch then push it?


That looks useful but what bothers me is that we are now defining the list of packages twice. I think it would be nice to define it in this package-pinned-packages variable and use just that variable to load the required packages. Something like:

```lisp
(let ((package-pinned-packages (append (...) exordium-extra-pinned)))
  (dolist (pkg package-pinned-packages)
    (let ((p (car pkg)))
      ;; existing code should work
```

What do you think?


А почему бы просто не сделать так?
```
type(x) is int and 1 <= x <= 9
```


JSP scriptlets don't have to be complete java blocks,  they can contain as much or as little java as is necessary.  In this case we can break out the "if" block into two separate scriptlets which means we are able to embed the HTML directly in the page without having to escape the Strings or mess with any of that stuff.


hahaha, vilken rolig gif


зачем этот метод если есть execute? а вот fetchAll я бы вынес.


no json


oh yeah...
2015年4月22日 13:25于 "Mohammad Etemaddar" <notifications@github.com>写道：

> Congratulations.
> We can say Happy new year :)
>
> Mohammad Etemaddar
> metemaddar.ir
>
> * in <http://www.linkedin.com/pub/mohammad-etemaddar/9/832/462>*
> <http://goog_1283906581>
> Programming services, C++, Python, Linux
>
> On Tue, Apr 21, 2015 at 11:25 AM, knight2995 <notifications@github.com>
> wrote:
>
> > +_+ Good...
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/torvalds/linux/commit/39a8804455fb23f09157341d3ba7db6d7ae6ee76#commitcomment-10818018
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/torvalds/linux/commit/39a8804455fb23f09157341d3ba7db6d7ae6ee76#commitcomment-10839714>
> .
>


erm, I think you would need to add cardinal as a new remote and grab and merge the commit, try a bit of googling


Wrong name?


Когда много сравнений, можно сделать как-то так
```
valid_options = ('photo', 'video', ...)
return x in valid_options
```


@noobtrader15 Which version of which operating system? Which version of Java?


not congruent with external package


Use `npm` scripts please to define the task commands


wrong scheme


не понял что тут происходит. куда ты пишешь результат.


looks good.


попробуй отлавливать исключения кусками поменьше. а то весь код обернул)


needs to be validated before commit.  Think about what names are allowed.  no '/' or  '%' for sure (break REST API)


не очень понятно что тут происходит. по идее метод должен что-то возвращать.


validate ObjectMeta


use 	etcdgeneric "github.com/GoogleCloudPlatform/kubernetes/pkg/registry/generic/etcd"


The comments below actually refers to this line of code. My bad.


Hey @helfrij, if you are wanting to re-implement the setControl method, you need to be careful.  If you want to reimplement the method you should make it virtual rather than doing a "silent overload".  The other issue you may run into is the fact that you should never use virtual functions inside of constructors.  Because the class will not have been fully implemented yet you will have no guarantee that the proper version of the class will be called.  I know that there is no call to setControl() inside this commit, but I do recall setControl being used in the pull request I merged in earlier.


:facepalm:


follow the example here: https://github.com/openshift/origin/blob/master/pkg/oauth/registry/oauthclient/registry.

It seems like a lot of work, but it is more inspectable than the old mechanism and you end up with only validation tests.


Mac OSX 10.9 Java 8. Do I need to be running an older version of Java?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (340 of 340 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)


I'm also waiting for it! :) Killer feature!
Docs on mesosphere already mention this functionality so I upgraded mesos but I was amazed that marathon 0.8.2 wasn't released yet
https://mesosphere.github.io/marathon/docs/native-docker.html


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


This took considerable reverse engineering, but the reason we issue this series of events is that certain websites have a very complex series of actions that are triggered by the various keyboard events. I need to reacquaint myself with this script before I can comment more than that.


I don't understand how you can say that the code is sufficiently general for use in LegacyCompatibleGitAPIImpl when the setupSubmoduleUrls() method and the fixSubmoduleUrls() method are untouched by automated tests.

It seems that adding them to the LegacyCompatibleGitAPIImpl without automated tests is making a huge assumption that they will work correctly with the JGit implementation, or that JGit users don't actually need those two methods.


fixed y2 step pin


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


Any chance there's some info in your console on how/why it dies? I checked both management-service-demo/manager-config and PlatformManagerAgent/config and they both have the updated settings:
```
    ...
    "server" : {
        "host": "127.0.0.1",
        "port": 8080
    },
    ...
```


What the hell is this anyway?


Was uppercasing everything else in that line intentional?


@CaptainElliott will do! I had to go.


Nevermind: https://github.com/fruux/sabre-katana/commit/faae0e0 :)


The original intention was to replace the command line in the Android version.


I believe some of the forms on facebook were particularly challenging, but I don't remember which one it was -- and it has likely since changed.


I'm wondering if you would want to strip the whitespace from the string and output before the assertion so you don't have to worry about the formatting of the compiled code.


Your invoice.tpl does not proper work (Just footer and header show up correctly).
1. Blunder: you cannot use hyphen in file names and underscores instead of the hyphens in invoice.tpl.
2. Nevertheless it doesn't even work after corrected these mistakes, because the variables show zero value in my test shop.
3. It seems that the styles you sourced to invoice.style-tab.tpl instead of a css file don't work.


Что будет, если я подам в это функцию число? А список? А словарь? :-)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (339 of 340 tests passed)

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


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (340 of 340 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)


Colour also should be color, since FillSprite also uses color.


@macpie drop the --insecure


That one slipped through. Current devel version is 2.0, right?


ZOMG


This can be removed if you use WIN32 in the add_executable


What are you referring to when you say "injected environment"? Are you talking about a content script?


hhhh


URLs in edit comments do not get linked automatically.
You might use `[[Wikidata:Primary sources tool]]` instead.


:)
![whm75vh](https://cloud.githubusercontent.com/assets/442870/7283690/95cf619a-e938-11e4-9787-69200025c288.gif)


Can you please add a metadata description of the flag, stating that it's related to EC support in openssl?


@dannypernik You've disregarded my comment on the special meaning of the percent (%) sign. Here it is again:
https://github.com/permalife/zaplings-full/pull/124/files#diff-d75b7b863f912cf9e6eb960ca2ab4ad3L781


we probably need to be more comprehensive here:

1. names vs. ids?
2. if names supported, fail on unsuccessful lookup?
3. fail on invalid input

If this stuff is all stop-gap to be replaced by systemd functionality we probably shouldn't invest much effort here however.


yeah i think so I have confused my pull requests


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


the second file is mistaken and should not have changed (appactivate(bool, long)


Why this big jump?


I was just checking to see if there is a way to customize what the pebble says.  So this was just a test to see how to do it.


Is this condition ever true:  BestMoveChanges == 0 ?
Comparing double with integer


I don't think we need it its a list, and technically it supports them both so if anything it could be an and or type thing.


Besides, I think, that such informations should be included in wiki, with reference to it in README.md


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


Right. You just need to be careful about where this is placed. It could lead to invalid code, unfortunately. Thinking this over, it might not be a bad idea to look at implementing it in `split_penalty.py`. Test both options and see which one you think works best (i.e., is minimally intrusive). :-)


Lol for enforcing C# naming recommendations.


In the log file we are getting 2015-04-22 12:41:10,744 (platformmanageragent-0.1 17874) <stdout> INFO: ('Registering (vip_address, vip_identity)\n\t', 'ipc:///tmp/v1home/run/vip.socket', 'platform_manager')  after your changes

2015-04-22 12:47:45,133 (platformmanageragent-0.1 18755) <stdout> INFO: ('Registered: ', {'identity_params': {'name': 'platform', 'uuid': 'third_platform'}, 'peer_address': 'ipc://@/tmp/v3home/run/vip.socket', 'ctl': None, 'external': True})

So based upon that it's the @ not being inserted/removed properly.


looks good  I am still not !00% happy with the support forum section.   I may reword it again later if i come up with something better.


@jlturner I updated the README.md.


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


Isn't locale always undefined here, I don't see where you retrieve it in ping (as opposed to fetch)


On 22.04.15 22:40, Justin wrote:
>
> This can be removed if you use WIN32 in the add_executable
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brezerk/Cataclysm-DDA/commit/068393d5eefa5f8d9b5c76022e188568494bcb15#commitcomment-10854699>.
>
Updated. Thanks.

--
Regards,
John Brezerk


Great! Thanks!


rax retry tests


> alph

Close enoug.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (340 of 340 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)


Tests are fixed. Service tenant checking was moved from upload_sec_group_rules to get_sec_gr_and_rules following same logic as in keystone resource.


Hahahaha fuck it.


I had to read this a few times...the code was confusing before this even =)  It seems OK to me.  Would be happier with a test case, but not a blocker.


style nit, but this is unnecessarily verbose IMO:

```
pexit_if(*root && chroot(root) == -1,
         "Chroot \"%s\" failed", root);

pexit_if(*cwd && chdir(cwd) == -1,
         "Chdir \"%s\" failed", cwd);

pexit_if(gid > 0 && setresgid(gid, gid, gid) == -1,
         "Setresgid \"%s\" failed", gid_str);

pexit_if(uid > 0 && setresuid(uid, uid, uid) == -1,
         "Setresuid \"%s\" failed", uid_str);
```

But do we want these calls to be optional anyways?  I'm not sure it makes sense to continue executing if we don't perform any these steps.  Almost like we should change it to:

```
pexit_if(!*root || chroot(root) == -1,
         "Chroot \"%s\" failed", root);

pexit_if(!*cwd || chdir(cwd) == -1,
         "Chdir \"%s\" failed", cwd);

pexit_if(gid <= 0 || setresgid(gid, gid, gid) == -1,
         "Setresgid \"%s\" failed", gid_str);

pexit_if(uid <= 0 || setresuid(uid, uid, uid) == -1,
         "Setresuid \"%s\" failed", uid_str);
```


no worries, did that and a couple of other minor doc fixes directly


:+1: from me


##Milestone 1 Feedback
Hi Linda, I think your outlined purpose, audience, and goals are excellent. This well-defined and narrow scope is exactly what we were looking for. This is a great place to start planning your project. Unfortunately, I must sign off on this Milestone at Tier I only because it was not completed on time this morning. Otherwise this is a job well done.

###Suggestions
Your HTML looks good - great job! Add lang="en" to either the <html> or <head> tag, and the <br/> tags are unnecessary. Going forward, we will be using CSS to alter the look and feel of the elements on the page. Remember to push your Git commits early and often, and integrate meaningful Git commit messages with each change. I'm looking forward to seeing your project take shape!


Why is it required to have both (single) `Port` and (plural) `Ports` at the same time? I'd have assumed that the latter would replace the former with the support for multiple service ports introduced along this PR.


@snopoke fyi, it looks like this breaks. even if you don't use the return value you still need to have the result to do the fancy call here: https://github.com/dimagi/commcare-hq/blob/598e00828821b7123cb10ab39cf24f93d7d3edd4/custom/ilsgateway/tanzania/warehouse/updater.py#L245 otherwise it just hangs.


You're right. I didn't recognize, there is a wiki on github. But not today


:ship: @dgearhart new owner of trippathbuilder


Can you index roleAssignments by roleAssignment.recipient -- i.e. convert it into a dict-of-lists, where the dict key is roleAssignment.recipient? That should reduce the big-O time complexity of this loop, so seems worthwhile.


So pluginId is unique across all usePlugin calls?
So say I have DBSource -> FilterTransform -> FilterTransform -> TableSink. Now configureAdapter should assign ids, 0, 1, 2, 3 for all these? And we allow each individual plugin to load its own plugin (only one level deep) through say PipelineConfigurer it will prefix the id so that subsequent calls. For example, the DBSource can use PipelineConfigurer to "usePlugin" and that will get an id of 0.1? The ids assigned at the first level, 0, 1, 2, 3 can be sent to MR Driver. How will the individual plugins, say DBSource, load the JDBC Driver class using this framework?


ok，那就用“配色方案”了


プルリクほしいです！ƪ(Ơ̴̴̴̴̴̴͡.̮Ơ̴̴͡)ʃ


好吧，“制图艺术博大浩瀚”这个我觉得可以。


Hey Zach,

We were comparing our output to yours and noticed for the
```
*** Life<ConwayCell> 109x69 ***
```
test, you print out the 10th test even though you are not supposed to:
```
...
Print the first 10 grids (i.e. 0, 1, 2, ... 9).
...
```
Other than that though your Conway tests match ours =)


何も確認せずにコメントして申し訳ないのですが、vim script だと`3/2`は1になるのですが、この辺考慮出来ていると思って良いでしょうか。


ditto


既存を作った者としてツライ感ありますが、matchstr の方が良い？


not in


Hey, I think you guys got the wrong email address.
Best
Rohit
On Apr 21, 2015 9:02 PM, "Chris Sheahan" <notifications@github.com> wrote:

> Hey Zach,
>
> We were comparing our output to yours and noticed for the
>
> *** Life<ConwayCell> 109x69 ***
>
> test, you print out the 10th test even though you are not supposed to:
>
> ...
> Print the first 10 grids (i.e. 0, 1, 2, ... 9).
> ...
>
> Other than that though your Conway tests match ours =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cs371p-spring-2015/life-tests/commit/9120bd414ac0bc7ad395e03d9086a9fb302cd6f4#commitcomment-10838267>
> .
>


はい、できています。


meh...I have reworded the paragraph for clarity.


Whoops accidentally squashed two commits while rebasing. This commit includes fixing a test that was  being ignored, some pep8, and removing unused names.


did you ever know that you're my hero... you're everything I wish I could be... If I can fly higher than an eagle.. for you are the wind beneath my wings..


確かにsubstituteの部分はmatchstrで代替可能ですね。(`v:version < 703`のケースですので、そんなにシビアにならなくてもいい気はしています…)


50 != 30 ;)


好，就这么着。那这一节的内容你梳理下然后就提交吧，要不要试试pull request？ @yarray


I guess the code is wrong if the `dir` kwarg is not supposed to do any normalization? Otherwise the test should probably use `Base.samefile` ?


That's why it says "Interim commit" and is also not in any Travis build ;-). That test obviously doesn't make any sense yet.

I just wanted to stash that very work-in-progress code so that I could switch my local setup to a different branch and didn't feel like "git stash".


バージョンというよりも、substitute はマッチしなかった場合に全体が返るという仕様と、置き換え動作が発生するので若干遅いというのが気になってます。


Probably the test is wrong; I'm not sure how to correct the test, though.   We could use `dir=\`, but that would fail on Windows.


sorry just interested ;) Agree with you re stash.


I hate that you can't edit the name/description of a commit...


Done


can he make a custom attribute? Instead of ```id```, say ```nodeid```.


done


Beginner Game demonstrating the basic understanding of building a platforming game within Unity 3D.


Ohh :P
Will Fix It


Consider adding support Unicode Escape sequences found in MS Word HTML docs.


Last time I tried (about a year ago), I still had to hack convert-cache.py depending on if its run on OSX or unix because of sed incompatibilities...


I know that pyne doesn't use the install phase of cmake, so I cant copy the delta (or alphad) executable's as I usually would, so instead I am trying to copy over the delta executable to the binary install directory using a custom_command.  Is there a better or more proper way I should be doing this?  If not, do you know why the above line does not copy the delta executable to the binary install directory? @scopatz


Well that is because whoever wrote convert cache didn't do a good job :)


From a fully cleaned directory tree (`git clean -d -x -f`), these `<Target>`s prevent Visual Studio from launching the Windows 8 application:

> Error : DEP1700 : The recipe file "C:\dev\OSS\rabbitmq-dotnet-client\projects\examples\WinRT\Subscriber\build\bin\Debug\RabbitMQ.Client.Examples.WinRT.Subscriber.build.appxrecipe" does not exist. You may need to build your project.

Removing lines 165 through 173 and re-loading the project will enable the Windows 8 application to be deployed and launch.


Typically I would not commit these auto-generated files (similar for `users.coffee`, `images.scss`, etc). Better to be purposeful and explicit and only commit the files you create / care about


This is fine, but you can use the more direct `render :new` here (and in general).


This method should be `gallery_params`, not `user_params`. Guessing this was a carry over from when you implemented strong params as part of user stuff, so no worries. The way to think of it is you want to create a method that slices out the safe params for a given controller. `user_params` for a `Users` controller, `gallery_params` for a `Galleries` controller, etc.


Per the comment below, the params building method should be named for this controller, specifically `gallery_params`. Obviously this works, but is confusing when read.


If you update the route to be nested (see comment in routes file), then this can build from the `params[:gallery_id]`. Otherwise this is great


Ух какой файл!!!


More auto-generated files I would avoid (although obviously not a real problem)


firstName


birthDay


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


верно, супер


Since `link_to` is a ruby method, there is no need to interpolate the `name` into a string. You can just use it directly as:

``` ruby
<%= link_to gallery.name, gallery_path(gallery.id) %> -
```


The first bug that nothinger fixed, you are so low.


Yes, I think setting that to true should be ok.  The only thing I could imagine is that setting it to true would stop the auto controller from increasing the altitude target while landed.  I've just tested in SITL though and it seems fine.


папки test-output, bin, .settings в репозиторий не заливаем
Заливаем только "полезный" код, папку src


Typically you want to indent within a block, so lines 5-12 here would be indented to clarify that they are part of the form building block.


Sure, let's say `data-nodeid`


ok great.  so the only remaining critical issue is using the output_min vs output_throttle_zero so the build goes ok for the coaxcopter, tri, etc


This is impressive! You can get by with a simpler technique using the routing, but this sort of thing is often needed to pass additional data, so awesome that you are comfortable with it


do we need these toggled variables?


Hi. Thanks for your work, but you missed many strings in PackageManagerService.java.
I used it from CM.


If you nest images within galleries then you get the associated in `Images#create` for free. Also, I would be explicit about the actions that are live (this simple `resources` call lights up all 7 routes). This would look like:

``` ruby
resources :galleries, only: [:index, :show, :new, :create] do
  resources :images, only: [:create]
end
```


Overall this is great. This is a lot of solid rails functionality. I would typically break this down into multiple commits, possibly one per bullet point on my list. If you work in a branch then you can even combine commits later, but in general it is better to have more, smaller check points along the way. Beyond that, I added some comments inline, but overall this looks really solid. Nice job!


I don't think so- if they're causing issues they can go as far as I know
________________________________
From: Derek Hua <notifications@github.com>
Sent: Tuesday, April 21, 2015 10:33:39 PM
To: dhua20/UFBands
Cc: Meredith,Benjamin B
Subject: Re: [UFBands] toggle testing (b803534)

do we need these toggled variables?

-
Reply to this email directly or view it on GitHub<https://github.com/dhua20/UFBands/commit/b8035344a22e17ab2fe95d3b9c681818d05500b1#commitcomment-10838493>.


I've never used Clojure before, so I'm a little lost. I installed Lein and lein-exec, but I can't figure out how you are executing the program from the CLI.


delete CNAME


@YusufSoomro you spelled my name wrong! fix it!


Please examine latest version of seeds.rb on GitHub, branch: seed-data-assignment.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Monday, April 20, 2015 5:25 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Complete Seed Data assignment (46cc040)

Good idea with the Comment.count - comment_count_pre and using find_or_create_by.

You removed the old code, but could you now bring it back so that your seed both creates 50 posts and 100 comments as well as your one predefined post and comment (if they didn't exist before)?

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/46cc040d67822e708ca2387bf732c7f703aac42e#commitcomment-10799093> .  <https://github.com/notifications/beacon/AKgq0H4is9VqF7ky-mlSLkMyHfCWQZZfks5oBL1VgaJpZM4EEIoO.gif>


We need to discuss this.  I think we may be able to use a single thread mapper for all thread types and in that case we wouldn't need a forum thread mapper class.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (339 of 340 tests passed)

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


Did we want the delete review button to be visible?


I'm still confused how this happened... https://github.com/JuliaLang/Compat.jl/issues/70


Can't do this - it duplicates something I already did...:smile:


This actually confused me when I saw this project, making me search through the source code to see if the plugin supported reading credentials externally instead of embedding them.  I recommend reverting the commit and and adding a comment pointing to the AWS docs. I'll make a pull request.


done


According to [this meteor tutorial](http://manual.meteor.com/#deps-accessingcurrentcomputation) we should use computation object passed as argument to autorun function.


We probably won't need this.  It'll be the rare case when we ask for a list of post revisions (view post history with versions).  I think fetching that list can be a distinct call.  My guess is that ```getPost()``` will be just fine fetching post data plus latest revision data and then we'll have a separate call for ```getPostRevisions()``` because we'd rarely call it.  We'll need to add raw content and rendered content fields into the ```Post``` domain object and populate those via a join to the latest revision.


why?


The null change is actually to fix the IntelliJ warning.


seems this function is not exposed in session_functions?  is that  by intentional?


average of females?


Simple test with three questions (first real program)


We'll talk more about the thread mappers and post mapper as soon as we have a chance, but this is a good start.


libcxx_ubuntu_14_04 (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (336 of 337 tests passed)

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


自动布局约束


I like that you end each push with tried not to break anything :)


Não colocar as classes de teste no arquivo de Steps


お、とすると推測するに`Throws`は関数スタイルでも使えるのかな...!


http://www.sitepoint.com/rails-security-pitfalls/ This article also suggests whitelisting the keys *before* casting to a symbol (see the "Denial of Service" section)... not sure how we could do this here though without it becoming very verbose.


All changes you suggested have been executed.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Monday, April 20, 2015 5:16 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Complete crud-assignment (bdc6e47)

The rest of the code is good. The index view using media classes is nice.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/bdc6e47197f3b97aa9534f9ce46f897c114d65fa#commitcomment-10798972> .  <https://github.com/notifications/beacon/AKgq0Gq59iYdzRAeWE-5cBVWqQh4EUz_ks5oBLtVgaJpZM4EEIEf.gif>


帰宅後修正しますー


could use Mutex::Locker for this.


could use "can_inc" for the variable name .


No, you're not using `unsigned int`. Integer literals are **_signed_**. Which means any literal above 32767 will be a signed long.


I changed it to this
   <input type=entry name="roomnum" size=1 value="<?php echo $obj{"roomnum"};?>" >


This is weird, I don't have this behavior. For me pressing ESC does not
close the company popup.

I tried with stock Spacemacs and auto-completion layer and indeed it closes
the popup buffer.

There is still the issue of company used with RET at end of line. I need at
least `fd` for this case.

I will revert ESC soon but I want to know why some of us get a different
behavior.

Le mardi 21 avril 2015, Tristan Hume <notifications@github.com> a écrit :

> OH GOD THE PAIN! This is a terrible change.
>
> The problem with this is that company pops up frequently and most of the
> time you can just ignore it, but now it completely breaks muscle memory for
> all of vim. In insert mode and want to go down 2 lines? Well if company
> happened to pop up you now need to press ESC ESC instead of ESC. At first
> I didn't even notice it was company mode I just thought escape was broken
> and finicky and it annoyed the shit out of me. The presence or absence of a
> company popup should never change how you use vim editing commands.
>
> Before the behaviour was that escaping to normal state wouldn't exit the
> company popup but it would close whenever any other command was executed.
> This could be fixed by just exiting company *and* going to normal state
> on pressing ESC.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/syl20bnr/spacemacs/commit/cdd1eddd3d9b88ed4c4be59b54e2906b723f8b97#commitcomment-10838111>
> .
>

--
-syl20bnr-


Maybe...


Can you look at the comment above this one , By Tony , and tell me how to translate the item inside the function?


大家准备好写一些代码了吗?


Any tips on how to deploy this to heroku?


lgtm


This should be replaced by the Forge method to check if a block is supposed to be an air block, can't remember the exact name. Otherwise things like Railcraft tracking blocks will break.


done


Ditto.


这个.....用这句总感觉不对劲又不知道怎么翻译