94: D205 1 blank line required between summary line and description (found 0)
117: D209 Multi-line docstring closing quotes should be on a separate line
117: D202 No blank lines allowed after function docstring (found 1)
117: D400 First line should end with a period (not 's')
117: D205 1 blank line required between summary line and description (found 0)
139: D209 Multi-line docstring closing quotes should be on a separate line
139: D202 No blank lines allowed after function docstring (found 1)
139: D205 1 blank line required between summary line and description (found 0)
139: D208 Docstring is over-indented
164:17: E128 continuation line under-indented for visual indent
173:21: E128 continuation line under-indented for visual indent
185:21: E128 continuation line under-indented for visual indent
194:21: E128 continuation line under-indented for visual indent
199:21: E128 continuation line under-indented for visual indent
218: D202 No blank lines allowed after function docstring (found 1)
219:9: E128 continuation line under-indented for visual indent
219:25: E251 unexpected spaces around keyword / parameter equals
219:27: E251 unexpected spaces around keyword / parameter equals
220:9: E128 continuation line under-indented for visual indent
220:25: E251 unexpected spaces around keyword / parameter equals
220:27: E251 unexpected spaces around keyword / parameter equals
236:13: E128 continuation line under-indented for visual indent
240:1: E302 expected 2 blank lines, found 1
240: D210 No whitespaces allowed surrounding docstring text
240: None: None
240: D209 Multi-line docstring closing quotes should be on a separate line
240: D202 No blank lines allowed after function docstring (found 1)
240: D400 First line should end with a period (not 's')
240: D205 1 blank line required between summary line and description (found 0)
259:79: E203 whitespace before ':'
268:1: E302 expected 2 blank lines, found 1
268: D210 No whitespaces allowed surrounding docstring text
268: None: None
268: D202 No blank lines allowed after function docstring (found 1)
268: D400 First line should end with a period (not 'm')
268: D205 1 blank line required between summary line and description (found 0)
291:13: E128 continuation line under-indented for visual indent
294:13: E128 continuation line under-indented for visual indent
314:17: E128 continuation line under-indented for visual indent
320:71: E502 the backslash is redundant between brackets
320: D202 No blank lines allowed after function docstring (found 1)
321:5: E128 continuation line under-indented for visual indent
321:57: E502 the backslash is redundant between brackets
322:5: E128 continuation line under-indented for visual indent
322:5: E125 continuation line with same indent as next logical line
351: D210 No whitespaces allowed surrounding docstring text
351: D401 First line should be in imperative mood ('thi', not 'this')
351: None: None
351: D209 Multi-line docstring closing quotes should be on a separate line
351: D202 No blank lines allowed after function docstring (found 1)
351: D400 First line should end with a period (not 't')
351: D208 Docstring is over-indented
352:9: E128 continuation line under-indented for visual indent
374:9: E125 continuation line with same indent as next logical line
377:13: E128 continuation line under-indented for visual indent
377:55: E202 whitespace before ')'
385: D210 No whitespaces allowed surrounding docstring text
385: None: None
385: D209 Multi-line docstring closing quotes should be on a separate line
385: D202 No blank lines allowed after function docstring (found 1)
385: D208 Docstring is over-indented
404: D210 No whitespaces allowed surrounding docstring text
404: None: None
404: D209 Multi-line docstring closing quotes should be on a separate line
404: D202 No blank lines allowed after function docstring (found 1)
404: D205 1 blank line required between summary line and description (found 0)
404: D208 Docstring is over-indented
427: D210 No whitespaces allowed surrounding docstring text
427: D401 First line should be in imperative mood ('function', not 'functions')
427: None: None
427: D209 Multi-line docstring closing quotes should be on a separate line
427: D202 No blank lines allowed after function docstring (found 1)
427: D205 1 blank line required between summary line and description (found 0)
435:9: E128 continuation line under-indented for visual indent
438: D209 Multi-line docstring closing quotes should be on a separate line
438: D202 No blank lines allowed after function docstring (found 1)
438: D400 First line should end with a period (not ',')
438: D205 1 blank line required between summary line and description (found 0)
458: D210 No whitespaces allowed surrounding docstring text
458: None: None
458: D209 Multi-line docstring closing quotes should be on a separate line
458: D202 No blank lines allowed after function docstring (found 1)
458: D400 First line should end with a period (not 'l')
458: D205 1 blank line required between summary line and description (found 0)
477:9: E128 continuation line under-indented for visual indent
479:9: E128 continuation line under-indented for visual indent
509:5: E303 too many blank lines (2)
528:5: E303 too many blank lines (2)
538:17: W503 line break before binary operator
538:17: E128 continuation line under-indented for visual indent
539:17: E128 continuation line under-indented for visual indent
552:21: E128 continuation line under-indented for visual indent
557:17: E128 continuation line under-indented for visual indent
577:17: E128 continuation line under-indented for visual indent
581:5: E303 too many blank lines (2)
583:1: E302 expected 2 blank lines, found 1
583: D210 No whitespaces allowed surrounding docstring text
583: None: None
583: D209 Multi-line docstring closing quotes should be on a separate line
583: D202 No blank lines allowed after function docstring (found 1)
583: D400 First line should end with a period (not 'l')
583: D205 1 blank line required between summary line and description (found 0)
583: D208 Docstring is over-indented
584:9: E128 continuation line under-indented for visual indent
658:17: E128 continuation line under-indented for visual indent
678:17: E128 continuation line under-indented for visual indent
698:21: E128 continuation line under-indented for visual indent
703:21: E128 continuation line under-indented for visual indent
727:21: E128 continuation line under-indented for visual indent
731:21: E128 continuation line under-indented for visual indent
738:21: E128 continuation line under-indented for visual indent
744: D210 No whitespaces allowed surrounding docstring text
744: D401 First line should be in imperative mood ('thi', not 'this')
744: None: None
744: D209 Multi-line docstring closing quotes should be on a separate line
744: D202 No blank lines allowed after function docstring (found 1)
744: D400 First line should end with a period (not 's')
744: D205 1 blank line required between summary line and description (found 0)
744: D208 Docstring is over-indented
745:9: E128 continuation line under-indented for visual indent
771:9: E128 continuation line under-indented for visual indent
779:9: E128 continuation line under-indented for visual indent
794:13: E116 unexpected indentation (comment)
804:27: E127 continuation line over-indented for visual indent
807:45: E261 at least two spaces before inline comment
814:39: E128 continuation line under-indented for visual indent
815:39: E128 continuation line under-indented for visual indent
816:39: E128 continuation line under-indented for visual indent
817:39: E128 continuation line under-indented for visual indent
818:39: E128 continuation line under-indented for visual indent
823: D209 Multi-line docstring closing quotes should be on a separate line
823: D202 No blank lines allowed after function docstring (found 1)
823: D400 First line should end with a period (not 'e')
823: D205 1 blank line required between summary line and description (found 0)
824:9: E128 continuation line under-indented for visual indent
839:19: E261 at least two spaces before inline comment
842:5: F841 local variable 'dummy' is assigned to but never used
858:36: E128 continuation line under-indented for visual indent
859:36: E128 continuation line under-indented for visual indent
860:36: E128 continuation line under-indented for visual indent
865: D209 Multi-line docstring closing quotes should be on a separate line
865: D202 No blank lines allowed after function docstring (found 1)
865: D400 First line should end with a period (not 'e')
865: D205 1 blank line required between summary line and description (found 0)
866:9: E128 continuation line under-indented for visual indent
904:9: E128 continuation line under-indented for visual indent
913:9: E128 continuation line under-indented for visual indent
916: D209 Multi-line docstring closing quotes should be on a separate line
916: D202 No blank lines allowed after function docstring (found 1)
916: D400 First line should end with a period (not 'n')
916: D205 1 blank line required between summary line and description (found 0)
929: D209 Multi-line docstring closing quotes should be on a separate line
929: D202 No blank lines allowed after function docstring (found 1)
929: D400 First line should end with a period (not 'e')
929: D205 1 blank line required between summary line and description (found 0)
969:54: E128 continuation line under-indented for visual indent
970:54: E128 continuation line under-indented for visual indent
971:54: E128 continuation line under-indented for visual indent
975: D210 No whitespaces allowed surrounding docstring text
975: None: None
975: D202 No blank lines allowed after function docstring (found 1)
987:1: E302 expected 2 blank lines, found 1
987: D209 Multi-line docstring closing quotes should be on a separate line
987: D202 No blank lines allowed after function docstring (found 1)
987: D400 First line should end with a period (not 'n')
994:9: E128 continuation line under-indented for visual indent
999: D210 No whitespaces allowed surrounding docstring text
999: None: None
999: D202 No blank lines allowed after function docstring (found 1)
1004:13: E128 continuation line under-indented for visual indent
1009: D210 No whitespaces allowed surrounding docstring text
1009: None: None
1009: D202 No blank lines allowed after function docstring (found 1)
1014:13: E128 continuation line under-indented for visual indent
1019: D209 Multi-line docstring closing quotes should be on a separate line
1019: D202 No blank lines allowed after function docstring (found 1)
1019: D205 1 blank line required between summary line and description (found 0)
1024:9: E128 continuation line under-indented for visual indent
1032: D210 No whitespaces allowed surrounding docstring text
1032: None: None
1032: D202 No blank lines allowed after function docstring (found 1)
1045: D209 Multi-line docstring closing quotes should be on a separate line
1045: D202 No blank lines allowed after function docstring (found 1)
1045: D205 1 blank line required between summary line and description (found 0)
1053: D209 Multi-line docstring closing quotes should be on a separate line
1053: D202 No blank lines allowed after function docstring (found 1)
1053: D205 1 blank line required between summary line and description (found 0)
1059:9: E128 continuation line under-indented for visual indent
1066: D202 No blank lines allowed after function docstring (found 1)
1071:9: E128 continuation line under-indented for visual indent
1081: D401 First line should be in imperative mood ('return', not 'returns')
1086:1: E302 expected 2 blank lines, found 1
1086: D401 First line should be in imperative mood ('Return', not 'Returns')
1096: D210 No whitespaces allowed surrounding docstring text
1096: None: None
1100:9: E128 continuation line under-indented for visual indent
1105: D210 No whitespaces allowed surrounding docstring text
1105: None: None
1105: D202 No blank lines allowed after function docstring (found 1)
1110:9: E128 continuation line under-indented for visual indent
1114:1: E302 expected 2 blank lines, found 1
1114: D202 No blank lines allowed after function docstring (found 1)
1123:1: E302 expected 2 blank lines, found 1
1123: D202 No blank lines allowed after function docstring (found 1)
1137:1: E302 expected 2 blank lines, found 1
1137: D202 No blank lines allowed after function docstring (found 1)
1145: D210 No whitespaces allowed surrounding docstring text
1145: None: None
1159: D202 No blank lines allowed after function docstring (found 1)
1159: D205 1 blank line required between summary line and description (found 0)
1173: D103 Missing docstring in public function
1176:9: E128 continuation line under-indented for visual indent
1182:1: E302 expected 2 blank lines, found 1
1182: D209 Multi-line docstring closing quotes should be on a separate line
1182: D202 No blank lines allowed after function docstring (found 1)
1182: D205 1 blank line required between summary line and description (found 0)
1197: D202 No blank lines allowed after function docstring (found 1)
1202:9: E128 continuation line under-indented for visual indent
1207: D202 No blank lines allowed after function docstring (found 1)
1212:9: E128 continuation line under-indented for visual indent
1216:1: E302 expected 2 blank lines, found 1
1216: D202 No blank lines allowed after function docstring (found 1)
1228: D103 Missing docstring in public function
1232:5: E125 continuation line with same indent as next logical line
1242: D202 No blank lines allowed after function docstring (found 1)
1264:1: E302 expected 2 blank lines, found 1
1264: D202 No blank lines allowed after function docstring (found 1)
1274:1: E302 expected 2 blank lines, found 1
1274: D400 First line should end with a period (not 'r')
1274: D205 1 blank line required between summary line and description (found 0)
1299:9: B901 blind except: statement
1320: D202 No blank lines allowed after function docstring (found 1)
1331:49: E502 the backslash is redundant between brackets
1332:41: E127 continuation line over-indented for visual indent
1349:24: E721 do not compare types, use 'isinstance()'
1359:17: E128 continuation line under-indented for visual indent
1359:5: E125 continuation line with same indent as next logical line
1371: D209 Multi-line docstring closing quotes should be on a separate line
1371: D202 No blank lines allowed after function docstring (found 1)
1371: D205 1 blank line required between summary line and description (found 0)
1396:1: E302 expected 2 blank lines, found 1
1396: D202 No blank lines allowed after function docstring (found 1)
1454:29: E126 continuation line over-indented for hanging indent
1461: D202 No blank lines allowed after function docstring (found 1)
1461: D400 First line should end with a period (not 'n')
1461: D205 1 blank line required between summary line and description (found 0)
1483:1: E302 expected 2 blank lines, found 1
1483: D202 No blank lines allowed after function docstring (found 1)
1483: D400 First line should end with a period (not 'n')
1483: D205 1 blank line required between summary line and description (found 0)
1509: D401 First line should be in imperative mood ('find', not 'finds')
1509: D209 Multi-line docstring closing quotes should be on a separate line
1509: D202 No blank lines allowed after function docstring (found 1)
1509: D205 1 blank line required between summary line and description (found 0)
1530: D202 No blank lines allowed after function docstring (found 1)
1530: D400 First line should end with a period (not 'a')
1530: D205 1 blank line required between summary line and description (found 0)
1560:5: E125 continuation line with same indent as next logical line
1565:5: E303 too many blank lines (2)
1575:27: E261 at least two spaces before inline comment
1583:14: E713 test for membership should be 'not in'
1589:32: E261 at least two spaces before inline comment
1594:45: E202 whitespace before ']'
1601:20: E221 multiple spaces before operator
1610:37: E201 whitespace after '('
1610:48: E202 whitespace before ')'
1634: D209 Multi-line docstring closing quotes should be on a separate line
1634: D202 No blank lines allowed after function docstring (found 1)
1634: D400 First line should end with a period (not 's')
1634: D205 1 blank line required between summary line and description (found 0)
1649:9: E128 continuation line under-indented for visual indent
1655:9: E128 continuation line under-indented for visual indent
1662:51: E128 continuation line under-indented for visual indent
1663:51: E128 continuation line under-indented for visual indent
1664:51: E128 continuation line under-indented for visual indent
1670: D209 Multi-line docstring closing quotes should be on a separate line
1670: D202 No blank lines allowed after function docstring (found 1)
1670: D400 First line should end with a period (not 's')
1670: D205 1 blank line required between summary line and description (found 0)
1719:39: E128 continuation line under-indented for visual indent
1720:39: E128 continuation line under-indented for visual indent
1721:39: E128 continuation line under-indented for visual indent
1724: D202 No blank lines allowed after function docstring (found 1)
1724: D207 Docstring is under-indented
1725:9: E128 continuation line under-indented for visual indent
1726:9: E128 continuation line under-indented for visual indent
1747:1: E302 expected 2 blank lines, found 1
1747: D209 Multi-line docstring closing quotes should be on a separate line
1747: D202 No blank lines allowed after function docstring (found 1)
1747: D400 First line should end with a period (not 'g')
1747: D205 1 blank line required between summary line and description (found 0)
1763: D202 No blank lines allowed after function docstring (found 1)
1764:9: E128 continuation line under-indented for visual indent
1765:9: E128 continuation line under-indented for visual indent
1766:9: E128 continuation line under-indented for visual indent
1799:26: E128 continuation line under-indented for visual indent
1800:29: E121 continuation line under-indented for hanging indent
1801:29: E126 continuation line over-indented for hanging indent
1805:17: E128 continuation line under-indented for visual indent
1806:17: E122 continuation line missing indentation or outdented
1813:45: E128 continuation line under-indented for visual indent
1834:17: E128 continuation line under-indented for visual indent
1845:42: E127 continuation line over-indented for visual indent
1846:42: E127 continuation line over-indented for visual indent
1847:42: E127 continuation line over-indented for visual indent
1852:5: E303 too many blank lines (2)
1855: D209 Multi-line docstring closing quotes should be on a separate line
1855: D202 No blank lines allowed after function docstring (found 1)
1855: D205 1 blank line required between summary line and description (found 0)
1873:13: E128 continuation line under-indented for visual indent
1880: D209 Multi-line docstring closing quotes should be on a separate line
1880: D202 No blank lines allowed after function docstring (found 1)
1880: D400 First line should end with a period (not 'y')
1880: D205 1 blank line required between summary line and description (found 0)


If pickle is human-readable you may keep it.

On second thought feel free to use one file (.gtest-parallel-runtimes) instead of a .gtest-parallel directory, and just save the runs best-effort. So ignore the % hash(binary_name) part, and save (binary, test) -> runtime_ms in the format. Remove the average of recent runtimes, just use the last one, it should be a good-enough heuristic.


要想历史图好看的话就 rebase。
其实 merge 也没什么，我用惯 hg 了，基本都是 merge。


Looks good to me.


The changes should have been made in the /src version of the file then build into the /dist I corrected this when I merged it into my fork dsmorse/gridster.js


We definitely should. Thanks for pointing out.


oups. mauvais rebasage


Cheers.
It seems I left old debug string on that line :)


Can you tell me why you did this change? I'm using a lot of internal requests where I expect Eloquent collections and upgrading to 0.8.3 breaks a lot of functionality.


`.gitignore` 还有注释啊，很多都是用 GUI 前端几个按钮加进去的。
我觉得讲究一点的排个序，不讲究的爱怎么样怎么样都行。我是不是太不讲究了？


我对 rebase 有些犯怵，害怕把主分支也搞乱了。

说起来，owners 是不需要 at 也能收到消息吗？


:tada:


I will release this ASAP. Thanks!


Fixed at https://github.com/MSOpenTech/dash.js/commit/9bbfcd4c3f16a261832ae4d78f08d751b70357e9


`(default: %d)` - perhaps you've meant `(default: {})`.


views.py is getting a little long. I'm going to start breaking more stuff out. I'm wondering if you should put the core export code in a new/different function too?


Isn't `urllib.parse` enough? At least it works for the test cases you added:
```python
>>> from urllib.parse import urljoin
>>> urljoin('http://foo.com/', '//bar.com/')
'http://bar.com/'
>>> urljoin('https://foo.com/', '//bar.com/')
'https://bar.com/'
>>> urljoin('https://foo.com/', 'http://bar.com/')
'http://bar.com/'
```


There are plans to remove this class out of github and to include it from https://github.com/Collivery/Collivery-Client so can you revert this change and just pass in the province?


fetch/rebase 是把自己本地的的 master 重定位到远程的 master 后面，没什么吧。

这个是我偶然看到的，还是需要 at。


我是直接在命令行下操作的，修改 `.gitignore` 也是用 mVim 直接修改。看着挺乱的，所以就加了几条注释。

倒也不是讲究不讲究，主要是强迫症 XD。这东西给机器看的，怎么样搞都无所谓。

说起来，其实 `.gitignore` 用 GitHub 生成的会比较好，特别全，特别整洁。新建一个 Repo，选择 `.gitignore` 的时候选择 TeX 项目就好了。
那个相当不错。


Curious, what was the view bug that you fixed? Is this it?


I think this change should be on BrailleTextBorderFactoryService


Когда поменяешь чтобы `moves_of_bishop` возвращала список, можно будет всю функцию записать так:
```python
moves = [moves_of_bishop(x, y, dx, dy, self.color, field) for dx in (-1, 1) for dy in (-1, 1)]
return reduce(lambda x, y: x + y, moves)  # flatten a list of moves
```


Oh, I don't think it is really worth it just because L4 warning, but if you done it already it's ok.


Can somebody view my profile,please


This case was tested by the code above in a better way. I forgot about the other case. The tests must pass with many Theano mode. The code above handle it for FAST_COMPILE and DEBUG_MODE.

Can you remove the case here with including('unsafe') as it is done above, use the above version for the GPUs tests.

To make sure the tests pass in all mode, call "theano/misc/do_nightly_build" instead of nosetests. It accept the same parameter as nosetests. The only difference is that it run tests in all modes we need.


O ką, jei žaidėjas suomis su suomiškais simboliais varde? :)


The problem is that this test is *not* testing `infinity`.  All the test is confirming is that the database is storing the MIN and MAX values.  It's no longer checking for the conversion to and from `infinity`.

There are two things that need to be tested: inserting values into the database, and selecting them out.  If all we do is insert MIN and MAX and then select them and compare, then we have no idea whether the conversion to and from Postgres `infinity` is happening, right?  That's why I was inserting the literal `infinity` values.

To see what I mean, revert to a revision before any `infinity` values were handled, and then run this test.  It should pass because there is no `infinity` as part of the test.


@funkybob I thought you might like to know about this related issue: https://code.djangoproject.com/ticket/24603.


Can someone see my profile,please


Was there a significant gain in removing this from the config file? My only concern (admittedly very nitpick-y) is that automated deployment of InfluxDB nodes will now require the users config management code to lay down a custom init script or upstart config as opposed to just laying down a custom Influx config file and expecting the stock packaged startup mechanisms to work. Not onerous, but out of the norm it seems.

Certainly not a huge problem, but seems like a papercut that could avoided - unless I'm missing some benefit to doing it this way. :-)


maybe "# latest commits"


This is an example comment on a patch.


@AhmarG After watching it run I noticed the lights didn't come on until 8:21 today. So we need to figure out how to force the timer to start at the top of every hour (1:00, 2:00, 3:00).


我是新建了一个名为 `ctex-dev` 的本地分支，我的修改都在它上面操作。这样的 WorkFlow 可以保证远程 Master 有新的提交的时候，我需要 Push 的时候不会冲突。操作流程大约是

* commit to ctex-dev
* checkout to master
* fetch origin/master
* merge fetch_HEAD
* merge ctex-dev
* push master to origin/master

大概 `merge ctex-dev` 这一步需要改成 `rebase ctex-dev`?

对 rebase 犯怵是之前用 git svn 的时候，手册特别强调了避免使用 rebase。之后其实就一直没有用 rebase 这个功能。

好的，那就 @leo-liu 。


宏开发的话，我们一会儿 `.def` 一会儿 `.cfg` 的，其实未必够。同样的文件后缀 `ctex.sty` 就要忽略，`CJKpunct.sty` 就不忽略，这些没准的。


refs #13


The .gitmodule file is missing.


好的吧，我明白了。


Vale... pero el paper es de 4 páginas @JJ, he resumido todo lo que he podido. Si queréis que extienda tanto, lo dejo para otro más largo...


ggghhhhhhhhjjh


refs #1


Done in 9a2386a looks alot better too


I would prefer to be consistent about it and don't use CAutoPtr also here.


Entering insane values OOM's on the E310. I approve of this change.


Can you post the full stack trace place?  I'm not seeing this, even when I remove SLF4J jars from my classpath and add Log4J jars.


Yes, str(e) is readable; the current output gives e.g.

  out/Debug/audio_decoder_unittestss: [Errno 2] No such file or directory

Also, I don't think specifying that we encountered the error while listing tests is particularly useful; the errors here are going to be things like permission denied, file not found, etc., none of which is specific to listing the tests.


You still have to revert, though


@KozhinM Should `StreamProcessor.initialize()` take `mediaInfo` parameter to avoid separate call to `StreamProcessor.setMediaInfo()`?

Looking into this made me wonder...  The only other place that `StreamProcessor.setMediaInfo()` is called is in `DashAdapter.updateData()`.  It seems that updateData would make more sense as a function in `StreamProcessor` since its job is to update data on the `StreamProcessor`.  This would allow us to remove `StreamProcessor.setMediaInfo()`.  Thoughts?  I seem to be in "removing-lines-of-code-mode" right now :smiley:


Qual a função do método  [field resignFirstResponder]  ?
Pq ele é usado nos métodos "next" ,"previous" e "mudar" ?


Because it now uses impersonation, there is no guarantee that you can reuse the previous one - i.e. credentials are needed from each inbound request.


I'd do the strtolower in findBestVersionForPackage or wherever it makes most sense to have a greater surface area of coverage for the fix. Because here you basically only fix this one usage of the function, others could still be affected.


A palavra editada não está sendo exibida na TableView do TabBar


In this case, I prefer to define  `timeout` as proper parameter of the test class. say `...base_logdir=None, tag=None, job=None, runner_queue=None, timeout=0)`.


IMHO is better to extend ToTestBase here, instead of Factory


Correção feita!!
Label: #Bronze5


No problem ... have you ever looked at Octopus Deploy, Microsoft Release Management or even Config Transforms and Publish Profiles? All these are designed for deployment scenarios and work by injecting changes into .config files ...


@cboggs we were doing it for cleanliness since it's something that is only paid attention to on the first startup. And it's easier in our code to know that you're configuring for the first time if that flag is passed in.

If it makes config management much easier to have it in the config file, perhaps we can have a section in there titled `run_once` or something. What you think @jwilder or @corylanou?


OK. I will try to add that one. :)


I actually had misread the problem description and made timestep a constant. I realized on re-reading the AMPL version that its a variable - with this change it works perfectly!


Removing as a submodule and adding as a folder on the main project.


hhhh


This comment is wrong, irrespective of our fix.


please use ``plone.api.portal.get_tool`` instead


Oops, I just didn't read down to this line of code.  So, my new suggestion is to just move `DashAdapter.updateData()` in some form into `StreamProcessor`, then call it after `initialize()` and remove `StreamProcessor.setMediaInfo()`.


Fixed in 47a9d8950086b4788847b67df39ceb14e2ac8749


Fine with me, will change it in a minute.


Meanbee Limited is the official company name.


真复杂。还是下载个 source tree 吧。
本地 master 和远程 master 就是两个不同的分支了啊，没必要多建立一个分支。不想有 merge 的话你只需要 commit, fetch, rebase, push 就行了。


OK. I enhanced tests, pls check again.


Fixup: 1bd67b2


Do you really want to increment this when `itObject == pPage->objects.cend()`?


Agreed. Build is broken.


O método deve dar um pop para retornar para a view anterior da apresentada, e não somente alterar os dados da view corrente. Deve-se usar o navigation para dar push e pop nas views.


O método deve dar um push para incluir uma view na frente da view apresentada, e não somente alterar os dados da view corrente. Deve-se usar o navigation para dar push e pop nas views.


0830 に変更してください。


Source Tree 貌似会乱码，我也不知道咋了。

行，我等下开个 Repo 试试看。

—
Sent from Mailbox

On Wed, Apr 8, 2015 at 9:36 PM, Liu Haiyang <notifications@github.com>
wrote:

> 真复杂。还是下载个 source tree 吧。
> 本地 master 和远程 master 就是两个不同的分支了啊，没必要多建立一个分支。不想有 merge 的话你只需要 commit, fetch, rebase, push 就行了。
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CTeX-org/ctex-kit/commit/e1104e1abb69ebea9a6453440b524cdc10e1273a#commitcomment-10623815


Nice! Thanks!


This doesn't fix issue #1000 and greater. should be rejected


"50 - ambit::current_indent() - strlen(buffer)" gives an error on our cluster.  It says that strlen is not declared in the scope.


那自然是设置 source tree 的编码啊
![1](https://cloud.githubusercontent.com/assets/570611/7046382/d4bf9994-de37-11e4-9948-96cdf74b5333.png)


you pushed a bunch of merge conflicts... please do not push to master in the future. I will merge your branch for you.


This is another sample comment


You mean canvas size? I think canvas size should be handled outside the effect (in the app code).


Sounds good!


please add such comments on code also, so we don't have to guess later why you're doing this


我蠢了……

—
Sent from Mailbox

On Wed, Apr 8, 2015 at 9:40 PM, Liu Haiyang <notifications@github.com>
wrote:

> 那自然是设置 source tree 的编码啊
> ![1](https://cloud.githubusercontent.com/assets/570611/7046382/d4bf9994-de37-11e4-9948-96cdf74b5333.png)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CTeX-org/ctex-kit/commit/e1104e1abb69ebea9a6453440b524cdc10e1273a#commitcomment-10623886


`std::move`


These CLI options should be documented in command-line-flags.md


Use std::move() here too.


WRS is a major feature of the firmware, there's no way I'm going to remove it.  And as I said WRS has no impact on Asus's loopback.  Both actually work fine for me.


Yes urljoin is enough...fixed and thanks.


0830 に変更してください。


What's going on here?  You renamed the localization.

Was it broken before?


@fjvallarino sorry, README is out of date regarding the monitoring-filter version. Its latests version is 2.0.1. I think changing the version should resolve the issue. cheers.


Hi @ssk2,

I have a dejavu. Didn't I comment on this already?

Anyhow, looks great, just need some documentation.


this is something we really wanted to do before but we need to be careful: what happens on an existing cover using the old Deco 16 grid system?


@marcotuts, as you suggested. I've tested it and this seems to be the size we wanted.


Do we want to really do this or just fix all together. Wei do you have time to do this or should we just add it as an issue and move on. I would like this done before I do then next merge with AI2 Master.


I didn't even know it was possible to make comments like that. Wow.


This repo offer php 5.3 compatibility so far, and `[ ]` syntax is >= 5.4 compatible


what is ``PTC_FIXTURE``?


nodejs_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2101 of 2196 tests passed)

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


@KozhinM Do we need a private and public `videoModel`?


Sorry about that Kevin. I'll get a patch in shortly.

Justin Turney, Ph.D.
Senior Research Scientist
CCQC/UGA

On Wed, Apr 8, 2015 at 9:39 AM, Kevin Hannon <notifications@github.com>
wrote:

> "50 - ambit::current_indent() - strlen(buffer)" gives an error on our
> cluster. It says that strlen is not declared in the scope.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jturney/ambit/commit/af3ad5de3552827585ef7422801a060c40f59016#commitcomment-10623870>
> .
>


To usuń i `CommandSender sender` zamień na `Player player` ;)


Hmm, why not a `tgt.id in inc_edges` test?


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


Seems I did something wrong on rebase...


There is some "NO_BREAK_SPACE" remaining in your test cases ;)


That's included from zlib-asm (https://github.com/ukyo/zlib-asm/blob/master/zlib.js) a dependence of jsziptools.


CHERI is MIPS n64


Not sure I understand.
In  RPM/SPECS/kaltura-media-server.spec you do not declare a dep on kaltura-oostinst, and yet in  RPM/scripts/postinst/kaltura-media-server-config.sh you are calling kaltura-base-config.sh. A script that is part of the kaltura-postinst.

How is that suppose to work?


Guten Nacht~ ^_^


instead of removing this code, just skip it for Plone 5:

```python
@unittest.skipIf(PLONE_VERSION >= '5.0', 'Resource Registries not available')
def test_jsregistry(self):
    resource_ids = self.portal.portal_javascripts.getResourceIds()
    for id in JS:
        self.assertIn(id, resource_ids, '{0} not installed'.format(id))
```

and so on...


Thầy ơi, em xem trên group, thông tin nhóm trưởng chưa update


Okay, Kevin, give it a go.

Justin Turney, Ph.D.
Senior Research Scientist
CCQC/UGA

On Wed, Apr 8, 2015 at 9:45 AM, Justin Turney <justin.turney@gmail.com>
wrote:

> Sorry about that Kevin. I'll get a patch in shortly.
>
> Justin Turney, Ph.D.
> Senior Research Scientist
> CCQC/UGA
>
> On Wed, Apr 8, 2015 at 9:39 AM, Kevin Hannon <notifications@github.com>
> wrote:
>
>> "50 - ambit::current_indent() - strlen(buffer)" gives an error on our
>> cluster. It says that strlen is not declared in the scope.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jturney/ambit/commit/af3ad5de3552827585ef7422801a060c40f59016#commitcomment-10623870>
>> .
>>
>
>


He intentado poner el link_product con recordsets, he tenido problemas, ya que este parámetro se carga desde una función de otro módulo que se llama de otras funciones, aparte, luego este carga las líneas de productos planificados de una OF, y si le paso el recordset, en vez del id del producto, cuando ejecuta el onchange del producto da error porque espera un id.
Puede ser que no te haya entendido bien, y no haya intentado de la manera correcta. No lo descarto, la verdad.


Fix is going to be merged ;)


That unfortunately won't let you to easily specify timeout params, as you'd have to call the class constructor with the special parameter, instead of just passing yet another param from the multiplexer system.


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


Yes, that looks like it will test `SELECT` correctly.  I don't know how to test plain-sql `INSERT` though.  We might have to mix a plain-sql insertion with a regular-Slick selection and use `isInfinite`.  That's why I didn't write it myself. :)


Fancy updating this to return teapot status code?

    resp = jsonify(data)
      resp.status_code = 418
      return resp

as 418 = I'm a teapot (http://en.wikipedia.org/wiki/Hyper_Text_Coffee_Pot_Control_Protocol)


Would it make sense to limit the thread pool size?


try not to reintroduce dependencies on ``getToolByName``; use something like this, instead:

```python
qi = self.portal['portal_quickinstaller']
```

also, instead of making things complex, it should be better to split this in 2 different tests with one of them running only if PFG is installed.


change message to reflect what it is exactly


`cend()`


了解いたしました。至急修正します。


TeamCity Ybackend :: biocad/ybackend [Build 183](http://192.168.30.225:8111/viewLog.html?buildId=1104&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


Thanks <3


Yes I'm aware of the warning, but the thing is that I can easily list all files with python's distutils (and record them in INSTALLED_FILES), but it's not so trivial to change permissions on them, so I accompanied rpm macros to do so. Do you know for some better way to do so?


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


Project in developed in Visual basic en Plataform .net


@cboggs makes an excellent point. Forcing our users to have a custom init script is a non-starter because every time InfluxDB is upgraded, the new package unconditionally overwrites any existing script in `/etc/opt/init.d`. This is important because sometimes bug fixes require changes to the init scripts. If user-specific changes are in the init script, they will be lost on upgrade.

We have two choices are far as I can see -- back to an "initialization" section in the config file, which we had before but previously removed (this may have been a mistake), or allow the `join-urls` to be specified in `/etc/influxdb/defaults`, which is sourced by the init.d script on startup. I prefer option 1.


Make sure that this change in dependency will not break the uber build.


Ich hätte zwei Commits gemacht:
1. Löschen von unbenutzten Files
2. Add listener...

Ausserdem ```mvn -e clean install``` erstellt die eine ausführbare "Fat jar".


Merged as part of 3784baaf5e130018a22598d4a4881d8177b669b5, in a slightly different way. Thanks for the pointer!


Thank you for the quick fix.


When I look at the definition of AmbPlugin, I see "class AmbPlugin : public AbstractSource". As my plugin is a sink and not a source, I can keep using AbstractSink instead of AmbPlugin, right ?


Looks good to me.


Can you explain this implementation to me please? If you differentiate `map`, the derivative should be implemented in terms of another `map` right?
`der(map(f,x)) == map(der(f),x)`
I can't tell if you're doing that or not


what is this line for?


There's some sort of merge issue in this commit.  The *.csproj files are back (when an earlier commit in this PR removed them correctly).  Also, the Azure deployment tasks aren't using the "connectedService:Azure" input type as is latest in master.  Instead, they're using the older "azureConnection".  Some investigation needs to be done so we're sure the proper changes are going to get into master.


Yes, the flag parsing library only parses os.Args. The plugin args need to be parsed so we add them into os.Args. I Have not been able to find a way to change which args the library parses.


Of course, if end-users launch `influxd` using their own tools which specify all options, then perhaps they shouldn't be using the init.d script i.e. `service influxdb start` from those tools. In that case our init.d script is only useful for single-node systems, which I don't think is desirable.


I add this feature to use the "content" field  instead to "title" or "name" if i dont have description and short_descrition in my class... and i add "~" to get only 200 caracter from "content".


I don't think, we need to connect to the frontend anymore here.
I'll fix that.


TeamCity ChameleonForms :: Continuous Integration [Build 234](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3554&buildTypeId=bt20) is now running


変更を確認いたしました。


It's missing because "1.0.4.3" makes npm unhappy. Future releases will be node-semver compliant.


Think subject is `Dashboard.new` by default


Also remove old and unused out-commented code.


apparently that's all that needed to be added to get Smacker videos playing again :P


I think the changes in this file are more suited to the first change. I will move them.


We should only add this as a requirement where it is needed (in Python 2.6).


This isn't necessary.


I think this also needs to call super().__init__ in case `Cached` isn't at the end of the MRO.


sammy davis junior


Good idea, I was also thinking of that and implemented that now:

  https://github.com/mika/jenkins-debian-glue/commits/mika/pr114_finalize

Can you please let me know if those 4 commits on this branch implement what you need?


asdasdasd


@Paxxi this fix the GetCharWidth issue


Undefined. ;-)
`file:xyz` is no valid pagename, `File:Xyz` is valid. At least on commons.
It is wiki-dependent, whether namespaces are case-sensitiv or not. So this will need some work.


as of now we do need a copy. I have tried getting rid of the copy but, args is being manipulated by lines 202 - 205. I did not write the flag parsing portion of the plugin, but I will try other methods.


OK, this looks nice.


Witch political story this code is writting ?


Under which circumstances can this happen? Maybe it would be better to show a warning or fail with an error, if the meta data can't be found. Silently ignoring it might hide problems and makes the code harder to debug.


~~[[CDUserFactory alloc] init];~~
[CDIMConfig config].userDelegate = [CDUserFactory shareInstance];


Ahh - yeah ``singlePaired.input_mate1`` is a collection in the case of ``singlePaired['sPaired'] == "paired_collection"`` - so it won't have an ``ext``.

This variant of the tool would sort of breakup the collection - you would pass in a list or pairs and it would produce two disconnected lists of fastq files (which to be honest may have been what we should decided would be the canonical representation of such a data structure given the existing tools) - or if you passed it a single paired collection it would produce two disconnected datasets.

One can build workflows that use the disconnected lists approach - but my preference for dealing with this kind of operation would be a second tool that always takes in a paired input and always produces a paired dataset.

Do you want me to hunt down what the syntax for this should be or scope out a second variant of this tool that consumes and produces paired collections explicitly.


Creating tests for this is a good learning opportunity. Issue itself isn't urgent so I'll just target it to alpha 2.


"iff" isn't a typo, it means "if and only if" (http://en.wikipedia.org/wiki/If_and_only_if) :)


Cambiar por "Anster to your comment in #{nombre_del_articulo}"


`using tRegionList = std::list<std::unique_ptr<DVB_REGION>>;`


The formatting seems a bit too spacious, but I'm trusting you've selected something reasonable.


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest118+011](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3554&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3005 Build time: 0:0:0


Bad idea. I removed it.


It would look for something like "all" and turn it into a list of strings.  The comma delimited stuff is already handled by resource.Builder, it's just that resource.Builder shouldn't know about "all", it should invoke an interface.

If we add that method, then ExpandShortcutRESTMappers would handle translating "all" into the right thing.


Can we have this return an `io.ReadCloser` for streaming? The caller would have to wrap it in a `tar.Reader`, but that seems like a reasonable alternative.


Should be able to replace these two lines with `msbuild Managed-OSVR.proj /t:ClientKit` which do the same thing (actually, they build the debug versions, too, but it's fast enough that it shouldn't matter) and might be more reliable or compatible?


@rashmimittal Why are we adding these empty event listeners?


Sure :smile:
I was  litle cryptic I guess.
Standard folder structure is bmz_cache/folder1. This adds another folder bmz_cache/folder1/folder2

For people with limited number of files allowed in a folder, this will help.Also spreading the files over more folders helps lowering the load time of a site.

And as an added bonus for me, I finally start to understand the Image Handler code :laughing:


TeamCity Ybackend :: biocad/ybackend [Build 183](http://192.168.30.225:8111/viewLog.html?buildId=1104&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 72 Build time: 00:11:37
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


Is it typical to bundle the `NuGet.exe` file rather than using one on the system? If so, I assume we'll need a license file for it, etc.


ACK. Looks good for me.


If you define a class method as private then for some reason it isn't actually private.

If you do define a method on the metaclass then it should work http://stackoverflow.com/questions/4952980/creating-private-class-method

But I think then it will not be able to be called by the subclass so it should be protected.


The Plone 5.0 PLONE_FIXTURE does not install any content types.  PTC_FIXTURE is a backwards compatible fixture that installs the ATCT content types in Plone 5.0 (it is also available on Plone 4.x and behaves identically to the normal fixture there).


Thanks, that seems to make sense - I just cherry picked this to https://github.com/mika/jenkins-debian-glue/commit/ef89da8bc498a350a47dfd313d035dce2c15adfe


Add those also to `stdafx.h`. (probably `<array>` too)


Ok, so that's bad style, because it will break the shim:

from http://requirejs.org/docs/api.html#config-shim
> //Remember: only use shim config for non-AMD scripts,
    //scripts that do not already call define(). The shim
    //config will not work correctly if used on AMD scripts,
    //in particular, the exports and init config will not
    //be triggered, and the deps config will be confusing
    //for those cases.

Also, I think a shim configuration doesn't work with nodeJS anyways. So, if you choose another library, you'll probably be better off. I think I already suggested using http://stuk.github.io/jszip/

You can install it with bower and it is an AMD module http://stuk.github.io/jszip/documentation/examples.html

> If you use an AMD loader (RequireJS for example) JSZip will register itself : you just have to put the js file at the right place, or configure the loader (see here for RequireJS).


Pareciera que está todo bien! :)


nit: I think a cast to uint32_t is clearer than using a magic mask. i.e.
(uint32_t)x2.

Similarly for lines below.


Number of entries needs also be adapted. I'll correct it.


I think issues should also be put on the tracker in addition to in here so conversation can occur around them until they are resolved. What do you think?


@btfrise the build is failing because there is a missing comma after ['user/player/chrome-app','Chrome App Player'].
You added a new element to an array but the elements has to be separated by commas.

['user/player/chrome-app','Chrome App Player'],
 ['user/player/native-player','Native Player']

Let me know if that does not resolve the issue.

Cheers


nodejs_fixes (lefticus) - i386-Windows-7-VisualStudio-12: OK (2179 of 2196 tests passed)

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


I guess I was minimizing the changes incurred by this commit, but yeah, I don't see why `to_s` wouldn't do it there. Actually, at this point, why even bother to parse it with URI?


FYI, this does not (currently) respect Ghost subURIs (i.e. http://my.site/suburi/post-slug is presented as http://my.site/post-slug for the Published href).


Can you please elaborate why the components workaround should be removed? cowdancer is available only in universe on Ubuntu:

````
% rmadison -u ubuntu cowdancer
 cowdancer | 0.61 | lucid/universe                    | source, amd64, armel, i386, ia64, powerpc, sparc
 cowdancer | 0.67 | precise/universe                  | source, amd64, armel, armhf, i386, powerpc
 cowdancer | 0.73 | trusty/universe                   | source, amd64, arm64, armhf, i386, powerpc, ppc64el
 cowdancer | 0.73 | utopic/universe                   | source, amd64, arm64, armhf, i386, powerpc, ppc64el
 cowdancer | 0.73 | vivid/universe                    | source, amd64, arm64, armhf, i386, powerpc, ppc64el
 cowdancer | 0.73 | ubuntu-rtm/14.09/universe         | source, amd64, armhf, i386
 cowdancer | 0.73 | ubuntu-rtm/14.09-factory/universe | source, amd64, armhf, i386
````

and if COMPONENTS is unset bootstrapping the cowbuilder environment on Ubuntu fails, so I think that should stay as it is. Or is causing any side-effects in your setup?

Regarding the sed -i...:  I'm not sure whether modifying the changes file on-the-fly is really a good idea, can you please elaborate what you're trying to solve here?


Is it worth it to support these complex string paths, when they can just specify them using an array?

```javascript
'a[-1.23]["[\\"b\\"]"].c[\'[\\\'d\\\']\'].e'
// Seems needlessly complex compared to
['a', '-1.23', '["b"]', 'c', "['d']", 'e']


Ok, I found a way to test `inserting`. pls check again.


Done.


Where is serializers? This cannot be the web2py one. We cannot make pydal dependent on web2py.


Merged in master!
http://www.cmake.org/Bug/view.php?id=15500


LOL


Should we bother outputting this at all if we're on `localhost`? It's always nicest to not load any external resources when developing locally. :thought_balloon:'s?


TeamCity ChameleonForms :: Continuous Integration [Build 235](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3555&buildTypeId=bt20) is now running


Done.  I separated out the test for the enabled tile count, the specific enabled tiles, and the pfg tile being enabled.


Nice work and :+1: @AndyAbplanalp


We should be returning a []string.  The point is not to force people to have to parse the string.  If you get multiple, joining them via strings.Join(..., ",") is not difficult.  I don't think we should compromise this API just to reduce work in a higher level component.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hahaha :heart:


sounds good!


I pretty thoroughly tested to make sure that exactly these changes wouldn't break anything and would also be completely invisible to users. People using he dev client wouldn't even be able to see a difference when selecting their server.


ReplicationControllers at a minimum.


@mdipierro you're looking at a single commit. Refers to the complete code change: https://github.com/web2py/pydal/pull/129/files


This might be a bit aggressive.  It will break things like:

```
CHANNEL=#develop
EMAIL_DOMAIN=${EMAIL#*@}
QUOTING_DOESNT_EVEN_DO_ANYTHING="I don't know why this # is here"
```

OTOH, I think this problem existed before.


Started at #29...


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


The reason I changed []string to string is the way I am using this method inside the builder (see the use of `replaceAliases`). Please give me some feedback about that.


Do we need jline?

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=8722
https://dev.eclipse.org/ipzilla/show_bug.cgi?id=4027


![oohu9](https://cloud.githubusercontent.com/assets/11759204/7047040/5fbe9fde-dde0-11e4-9f5d-44ba6d338378.jpg)
Olha, é mais legal botar imagens aqui. Peço perdão pelo vacilo, estava só testando.


The majority of this Dockerfile would be better in the https://github.com/chop-dbhi/harvest repo. This Dockerfile is could extend it and set the `ENV` variables.


CQ?


Thanks for the valuable contribution!  The only thing I am wondering is why you chose to make suite a *required* parameter for SeleniumTestCase.  While tests are most often run within suites, doesn't it seem useful to be able to run tests on their own, without being part of any particular suite?


There are outstanding IP issues/concerns.  We need to resolve them first.


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0+011](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3555&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3005 Build time: 0:0:0


What about Option(cmd) and Option(id)? Your if statements looks like you expect possible null values there.


TubeTK_SOURCE_DIR should always be false. You might want to just do the find package.


Thanks


Try adding on some additional debugging:

1. Open the register.php file
2. On line 71/72 add the following before the curly brace (})
echo '
Response

' . htmlspecialchars($client->response, ENT_QUOTES) . '
';
3. Try registering again with a new user

This will post the verbatim error returned by Articulate Online and should provide more detail about what is going wrong.

-Dave


I figured it wouldn't be necessary to limit the pool size. In production we often see only one or two threads that perform linking concurrently (builder + editor) and therefore a pool size around 4-7. The smoke tests use more threads so the problem was more severe, there. Anyway, we could limit the number of threads artificially but places like that are hard to find as soon as we have a solution which allows to access EMF models concurrently per resource rather than per resource set. That's why I made the deliberate decision to use an executor that is only limited by the physically available resources on the machine.

The main purpose of this commit was to reduce the cost of the annotation processing.


@martynjames I changed the commit ID in here so that the changes I made in the edx-search repository regarding this ticket gets included.


what do you mean ?


This logic should definitely be moved to another class. I know that there is already business logic in our resource classes but we should avoid that.


This seems to be duplicated here: https://github.com/dimagi/Vellum/commit/ccf46d4f5c32a3e18737c4036cd64ff44829a648#diff-b860087dfb0ae7f09a955a95f517a309R150

(I realize it's been that way for a while. no need to change)


style: smoothingScale = 0.0


You're welcome :)

Do you have any plugins in mind you need? I will eventually port most/all the plugins of compiz but it would be nice to have a list :)


Could you add in the comment a short explanation of the parameters ?


It breaks, of course ;)

Thing is, it will break anyway if we keep Deco 16 as default, because the grid css has changed and all you will get is a stacked layout of all the tiles. We could probably add some sort of  upgrade step, but as you say, not easy.


Thought scala was already approved... this is not the jline:jline, but the org.scala-lang:jline if it makes a difference...


So you're passing the smc_fid back as r0, some unknown value back as r1, and x1-x3 back as r2-r7. You could do with documenting this "protocol" somewhere, even if it's just a comment.

Also, cast x1 to (uint32_t) to avoid the high bits polluting the context. Casting (x1 >> 32) would be clearer too


:+1: #81


Will do.


@clintonb thanks for the update!

@stroilova @mulby @brianhw @jab5569 -- would one of you please review these documentation changes?


describe your commit messages properly. Describe what you are doing for each commit.


Move it to `OpenDirHelper.cpp` maybe?


Should probably be collapsed by default


Unecessary


Thanks, this fixes one deprecation.
See also one of my comments in https://github.com/ktoso/maven-git-commit-id-plugin/issues/153

```
[WARNING] pl.project13.maven.git.GitCommitIdMojo#session:
[WARNING]   The syntax
[WARNING]     @parameter expression="${property}"
[WARNING]   is deprecated, please use
[WARNING]     @parameter property="property"
[WARNING]   instead.
```


Sorry, code should be:

`echo '<h2>Response</h2><pre>' . htmlspecialchars($client->response, ENT_QUOTES) . '</pre>'`


### [Codacy](http://localhost:9000/app/nunomiguelt_6/JS-Tests/commit?cid=1) detected an issue:
#### Message: `Avoid unused parameters such as '$arg3'.`

#### Occurred on:
+ **Commit**:  57c31c39f12922328ba7446ba435a9ed3f332792
+ **File**:    [spa ces.php](https://github.com/nmatpt/JS-Tests/blob/57c31c39f12922328ba7446ba435a9ed3f332792/spa ces.php)
+ **LineNum**: [12](https://github.com/nmatpt/JS-Tests/blob/57c31c39f12922328ba7446ba435a9ed3f332792/spa ces.php#L12)
+ **Code**:    `    public function Gac($text, $arg3 = [])`

#### Currently on:
+ **Commit**:  f98474dcbe545b89c6accd72cd284f363f83d86f
+ **File**:    [spa ces.php](https://github.com/nmatpt/JS-Tests/blob/f98474dcbe545b89c6accd72cd284f363f83d86f/spa ces.php)
+ **LineNum**: [14](https://github.com/nmatpt/JS-Tests/blob/f98474dcbe545b89c6accd72cd284f363f83d86f/spa ces.php#L14)


attempt_more_exception_reduction (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (335 of 335 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 approve this message.


I removed this because I dont think this particular miniheader integrates nicely into the the system, it just doesnt look like its fits... We had discussed this. I think that if we need some kind of miniheader, we should adjust the rest of the headings and subheadings in the system to make it so we dont have to work so hard to make a miniheader stand out
.


Great.

Minor comments:
/expetected/expected/ (too many 'e's)
/foldername/folder name/
/can not/cannot/


Minor style: use directly this->SmoothingMethodComboBox


Ok should I change this or will you?


i know, however it can affect logic of applications if developers already rely on that POST param, anyway not against reverting it


add () after method names  to match style generated when using `:func:`


See above (do it in the ui file)


@i5o, the code is a little convoluted for my taste :)
"not is_unshareable" is a double negation, and in fact, you only need do the "else" part if is shareable.
(You don't need hide because is not visible)
About the comment from @SAMdroid-apps, I think we should play safe, and show the menu by default. Only hide it if the activity have max_participants <= 1 in the activity.info file.


Different strokes for different folks. The example I have in the unit test is a bit extreme because that's what it's intended to test.


I agree that it makes it look like a link. We should make it a link and make it do something because the older styling looked a bit awkward. I think we have a lot of options here as to what to use this link for


I think having the close icon for now should be line for deselect


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


Again, there is no comment explaining what x1-x3 mean and what is returned.


I am not sure a blank line after `engine = ..` in these tests is helpful (especially when you have brackets on their own line which adds some whitespace).


虽然拦截了电话，但是通话记录中依然有来电记录。下一版去掉通话记录中的记录。


(type) is a c style cast. Use a static_cast instead (c++ style cast).


could be 1 lined


Whoops, seems I forgot to save after fixing this line. It was late... :flushed:


chop blankline?


Minor style: use a more descriptive variable name


typo


Is there another variable that I can check to see if TubeTK has already been found ?


I'd be willing to integrate this upstream, if there is either an empty implementation throwing a LogicException ("no implemented") with a comment explaining why there is no correct abstract method or PHP5.3.0-PHP7 work correctly with the method being omitted completely.


cd crawlers/nba/


minor suggestion: else if ? or switch case ?


attempt_more_exception_reduction (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

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


I think this needs one more space to be aligned. I didn't test it though.


cd crawlers/nba/
sh nba_odds.sh
less nba-odds.csv


Agree, fixed at https://github.com/MSOpenTech/dash.js/commit/a13a09e69e37d15787286f3bc5e2e2c00f604375


That's doens't matter because it doesn't work, this should be replaced like it has been done for display ebay widget


This should be []string - you're invoking a generic API to ask if a resource value maps to it.  I would call the method AliasesForResource


I don’t understand this example. Why would you run `html.escape` when setting `.src` directly (as opposed to writing said HTML into the attribute value)? Is that ever necessary?


The fundamental operation is "one" to "potentially many".


It might be helpful to include some `RemovedInDjango21Warning` comments in the areas of backwards compatibility code that can be eventually removed. It seems like there is some code away from deprecation warnings that could be inadvertently left behind otherwise.


用拼音命名不是个好习惯


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

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


No.  Since we aren't specifying a timeout value it should be identical in behaviour to pthread_join


This is potentially unsafe if the high bits of x1 happen to contain non-zeros. Perhaps instead:
x1 = (uint32_t)x1 | (x2 << 32)


@moonsin 为毛我只看到了你的提交记录...余浪的呢


Instead of aliases..., you can just do `strings.Join(aliases, ",")`


The tempfile dependency goes away if we use just one file. I'll do that. Oooh, but no: with one file, I'll be creating it under a temp name and then moving it into place.

gzipping the file adds one line for the import, one line when writing, and one line when reading, and the gzip module is already in the python standard library. I can certainly remove it, but why? Even with the binary format that pickle uses it compresses the file by a factor of ~3.

As for pickle: I could of course use a human-readable format instead, but it'll be slower and take more lines of code. So again: why?


you are on fire!!!


This is a no go for me.
```
The page isn't redirecting properly

Firefox has detected that the server is redirecting the request for this address in a way that will never complete.
```


There is no new line at the end of file.


This should have an additional field which is the UUID of the collection. API clients use this uuid to select the collection


could this change has possibly a 3rd effect.

when calling RoutesProxy#send, some_route_url on a Engine Routes proxy, it no longer returns the path with the mounted prefix.


:-1:
This spoils memory saving by MRB_WORD_BOXING on 32-bit processors.


Let it be known I am personally mourning the loss of `CLLLD_*` but I am going to let it go.


Sorry was reading the wrong line of docs.  pthread_tryjoin_np will not wait for the other thread to finish.

So the question is... do we need to wait or not? I will read the code some more.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (335 of 335 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 should be indented by 2 spaces


Cadê o when? O cenário é um teste, certo? O given diz a pré-condição, o ponto de partida para executar uma ação principal que vai ser testada, descrita no when. Por fim, o then checa o resultado dessa ação. Esse cenário e os próximos 3 não possuem o when. Corrigir.


Too much fun.


No, we don't. I have not replaced videoModel with this.videoModel only because it is a bit shorter to use it w/o "this". If you think that it is confusing, I can do this replacement.


not at all. The point was to create at runtime the static example shown in the link with the img tag. Trying to reproduce that problem in IE8 is not possible unless you inject via `innerHTML` some weird HTML with the backtick issue. Hence the following part that warns about this and points at alternatives.

I really **don't care** about backtick here ... people are still parsing everything wrong and there are alternatives. I prefer consistency with other common HTML escape/unescape or entities handlers.

I don't want the server to send me an encoded backtick that would be decoded in JS land.

Does this make sense?


cd crawlers/nba/


I noticed the same thing.  You should be fine with AmbPlugin because
AbstractSource inherits from AbstractSink.  You just won't implement any of
the source methods.
On Apr 8, 2015 6:54 AM, "Olivier Delbeke" <notifications@github.com> wrote:

> When I look at the definition of AmbPlugin, I see "class AmbPlugin :
> public AbstractSource". As my plugin is a sink and not a source, I can keep
> using AbstractSink instead of AmbPlugin, right ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlivierDelbeke/automotive-message-broker/commit/b43dabbf2c36ba3ef6e08ab195243185d657d32c#commitcomment-10624276>
> .
>


The release importer will only be used for a single collection at a time, so we can set this in `__init__`. Pass it in as a uuid string and get the `data.Collection` object in the class. Set `release.collection = self.collection` in `_create_release_object`


It is better to move event handler code to another line to make this easier to read.


I don't understand why we are joining the thread here... I am clearly not the person to be commenting.


find_package is smart enough to know if TubeTK is already found and not re-do everything. You don't have to worry about it. (For background, a Find*.cmake it generally checks *_FOUND where * is the name of your project.)

The intent of the if (TubeTK_SOURCE_DIR) / endif() was that if you build the tortuosity module from inside TubeTK, you don't want to do the find_package (a module cannot find itself, it wasn't built yet !). When the tortuosity module was moved to Vesselview, the if/endif stayed even if it has no reason to be here.


That could be due to the fact that there is nothing outputted to the browser. I'm guessing after it loops two or tree times then Firefox will mark it as never ending - does adding sleep(1); before ob_end_clean(); help this?

IE showed no symptoms of this after processing 3,763 posts


Yes, they seems good for me and enough generalized that can be useful but not a regression for others, thanks!


Щось я не можу зрозуміти що буде, коли сегмент повністю лежить на прямій. Ця функція повертає точку або викидає Exception, що немає перетину.

Ще ця функція схожа на попередню. Чи можливо переписати її використовуючи результат перетин відповідних прямих, а  потім просто перевірити чи точка належить сегменту чи ні?
@AnatoliyS


nit: I preferred returning early if ns is true. Although there is currently no difference in behaviour, if some extra code is inserted above this in future, you could be open to attacks from the ns world.


I like this!


Para não comprometer a legibilidade, substituir "Given" por "And".


file -> template?


You will have to create the permission:
https://docs.djangoproject.com/en/1.8/topics/auth/customizing/#custom-permissions


Fixes #200


:thumbsup:


I think you will be well suited to look into `watchify` for "watch"-ing concerning `browserify`. Much faster and more recommended. Also, I would drop the usage of `.jsx` specifying which files have `jsx` in them.

Also, I know Babel has great support for both browserify and jsx. Probably a good choice to look into.

Just let browserify w/ transform handle that for you. Chances are most of your files are going to end up w/ `jsx` in them in a React project :smile:


This is `needs_compile(Source, Target)` in rebar. Why is not so here as well?


@mlafleur could you please update this commit to avoid all these unnecesary line breaks. This makes this commit hard to review and doesn't aligned with JS coding style.


List contents available on Inovaula => List available contents on Inovaula


dddddddd


I know Octopus Deploy or MS Release management. These Release Management tools manage config file because it is a widely spread practice not because it is a good one :)

I recommand to read the Twelve Factor App : http://12factor.net/. Especially the section about config.


You should add
´´´
include ActionView::Helpers::TagHelper
´´´
It crashes on content_tag in ApipieHelper https://github.com/Apipie/apipie-rails/issues/353


what is exactly place you are catching errors in?


Не совсем корректно.
Нам нужна связь многие к 1.
А значит в классе Age (см выше) ставим has_many: movies.


space after function


I just tested it in our stage environment running PHP5.3.3 and removing the function definition works fine.


cd crawlers/nba/


sh nba_odds.sh


One more note, I have not used JSHint much myself, but it seems the general consensus is that JSHint is out and ESLint is in. I have been very happy w/ my ESLint experiences so far. Same feeling I get about Grunt vs. Gulp. I would recommend possibly swapping in ESLint. Who knows where else it may come in handy :wink:


less nba-odds.csv


has_many: movies

Потому что возрастных ограничений у нас всего штук 5 от силы. Мы же не будем заставлять пользователя вводить вручную и сохранять потом это для каждого фильма?


@excelle08 这边我只看到你们两个人的提交记录,不知道其他人的进度如何呢


chop 'in'


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


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2094 of 2201 tests passed)

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


Опічатка, треба @throws
@AnatoliyS


blabla


Changed


Thanks!


I was unable to hit these routes when using the server via `gulp`. No errors/etc... Do these not translate directly into `http://localhost:3000/vehicles` for development?


But we do want to output to show its progress `Processing posts...`. Adding `sleep(1)` did not helped.


@marjev - With which story is this associated? If it's still in an open PR, I'd add it in there; otherwise mention the story in the title so that it is understood what changes are associated with this hash change.


Done


Would be useful to include a link to the GitHub source browser (I think we do this in other places).


Oops, yes I did it wrong. I had lost the commit and manually reapplied to change and took the wrong file.


I think we should be fixing the newest issues first.


Please set to the lastest `1.0.20`. 1.0 is too dynamic and cause clients to upgrade on minor version release. We don't want to happen.


See https://github.com/iojs/io.js/pull/1301/files#diff-3deb3f32958bb937ae05c6f3e4abbdf5R547 -- the other checks don't actually matter.

If this is at all controversial, I'll just drop this commit, it doesn't matter.


This actually increases the log level to warn, instead of decreasing it to debug.
Is that a mistake?
Or is this line warning about a situation that should normally not occur and that might signal problems?


or to rephrase my POV: the IE < 9 issue exists if you create dirty chunks of HTML allowing users to inject element attributes. I would never encourage this technique and if we do things properly this IE < 9 only problem is not a real-world concern at all. This, unless I realize how frequent and common it is to create problems. Those examples didn't convince me enough :P


Don't you want to use more default interpolation like `%{url}` or `#{url}` ?


Sure doesn't. Edited.


This should be shanged in studetnview.php and teacherview.php


Couldn't you just use $_SERVER['PHP_SELF'] ?


`|url|` is default for RST


lua 5.1

lua fun_check_count.lua result_fc.txt
files: 228, functions: 7264, passed: 6622, failed: 642, uncompilable: 20, pass rate: 91.16%

lua fun_check_count.lua result_fc_c.txt
files: 11, functions: 6740, passed: 6223, failed: 517, uncompilable: 13, pass rate: 92.33%


It's pending the new mappings and templates being added to the cluster.


More readable page ... only significant numbers are enhanced.

![image](https://cloud.githubusercontent.com/assets/2932687/7047629/0331f562-de0e-11e4-9984-9c1dc96299a1.png)


i learned this! looks more clean to finish with an extra empty line ;) but not mandatory...


nit: prefer brackets around (delta * ops->clkmult) so not relying on operator precendence.


Hmm... I'll just remove it. It's easy enough to download. I will also add an exclusion for NuGet.exe in .gitignore so people can just download it into their working directory instead of modifying their PATH.


This change forced me to require this dependency directly in my `composer.json` file instead of letting Composer figure out the best version.

Looks like Composer was resolving to Zend Framework 2.4 and therefore was choosing the previous version of this package (1.0.3), which is the one with a compatibility issue with ZF 2.4. It couldn't choose version `1.0.4` because its incompatible with ZF 2.4. So kind of a catch-22, unless I'm missing something?

Like I said, the only way I could fix this is by explicitly requiring this package version `1.0.4`, which made Composer uninstall ZF 2.4 and install version 2.3.7.


Since you're constructing new promise object with `new WinJS.Promise()`, you should not return another promise from it, like you do here. Instead just call `deviceInfo.findAllAsync` and fullfill your promise from fullfillment callback (`c` in this case).

Basically you already doing that in lines 47 and 51, so


why do you need to handle it manually here then?


Prefer using UINT_MAX instead of 0xffffffff.


Probably obvious but might be helpful to add "returns True when the operation is successful" or something to these.

And do people think that raising exceptions is better than returning False?


Travis complains about the relative import:
`module_prototyper/models/module_prototyper.py:30: [W0403(relative-import), ] Relative import 'default_description', should be 'server-tools.module_prototyper.models.default_description'`


thanks, looks good


Por que há cenários em inglês e outros em português? E nesse caso em particular, as palavras reservadas em inglês e o texto de título e steps em português? É recomendável seguir um padrão.
O Cucumber provê suporte à vários idiomas (me refiro à linguagem usada para escrever os cenários, Gherkin), trata-se de uma questão de configuração. Ajustar.


Great idea! :smiley: Thanks for your work @z3ro01!

Just some general thoughts and ideas:

atm we use integers starting from 0 as player id -> if we just change this to guids we could store and sync profiles easily the via the webservice (take your personal profile to friends home/parties)

What about doing some kind of hashing of the songfiles (only notes without header) to identify a matching version.
Distributing songfiles is a nice idea, but if I can't get a perfectly matching mp3 version that fits to the songfile, I can not play this song.

How about "recording" a game: just the points an the bars -> could to ghost-duels/challenges

Server question: should we use php or asp?
on the one hand: with asp we could reuse our c# classes and use .NET serialization. (How much "client" code do we need on server side)
On the other hand it may would be nicer if we use an open exchange format as JSON to allow others to interact with the server.


It might be better to reject promise here because if there no available cameras you won't te able to scan anything :)


data is no longer optional.


I think this "filtering" would be better suited to living outside of the `render` method. For both clarity and performance. Perhaps the `filterList` method could handle the bulk of this and `setState({renderVehicles: renderVehicles})` on `filterText` change.

no need to re-process the filter operation on every render cycle. Only when `filterText` changes, yes?

Then you could do something along the lines of

```jsx
<ul className="list-group">
  {this.state.renderVehicles.map((vehicle) =>
    <GarageList.Vehicle key={vehicle.id} vehicle={vehicle}/>)}
</ul>
```

and drop everything above the `return (` line :smile:


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


Yeah- that's why I said that it didn't show anything back in the ticket. Looks like header is not going to be any use though. It will probably have to be done through the client side or have no output.

But that said, I'm not sure what is - JavaScript redirects shouldn't be used, and meta redirects can be disabled through both the MMC (and browser I think) : https://www.vanstechelman.eu/windows-group-policy-encyclopedia/10f1aad09e2b0df3a73f51a5889a5196/allow-meta-refresh

> This policy setting allows you to manage whether a user's browser can be redirected to another Web page if the author of the Web page uses the Meta Refresh setting (tag) to redirect browsers to another Web page.If you enable this policy setting a user's browser that loads a page containing an active Meta Refresh setting can be redirected to another Web page.If you disable this policy setting a user's browser that loads a page containing an active Meta Refresh setting cannot be redirected to another Web page.If you do not configure this policy setting a user's browser that loads a page containing an active Meta Refresh setting can be redirected to another Web page.


please delete part about collecting docstrings -- no longer true


Is this going to change the default behavior of the adaptor? Does this change the way in which we know/enforce mandatory parameter Vs optional?


In Python it's considered bad to just use "except:" without specifying which exceptions you're expecting to catch. "except:" even catches the KeyboardInterrupt when you press Ctrl+C.


Esse step é dispensável. Pense que posteriormente cada step deverá ser implementado... como vc faria para implementar isso, o desejo do usuário? O desejo dele é traduzido nas ações que ele realiza no sistema, já descrito no step seguinte (when).


It's explained in commit comment — "Used sed command because RST doesn't support highlighting in parsed-literal, which is the only way to use a substitution in code block."


please add a docstring here to explain why the description has been extracted.


What was merged here? And haven't I done these path adaptations in an earlier commit already?


It looks correct that this one is `warn`. It's an exceptional condition (it found a method that was already patched)


I think the above hunks relating to onChange could be reduced to something along the lines of

```jsx
<input
  // ...
  onChange={this.props.onUserInput}
```

I would be hesitant to start interacting directly with the DOM and favor setting/getting props/state appropriately


`opts_changed` is not in my version. What is it for and why its implementation has to be changed here?


`... CSV file for which ones...`


recreated a resume format found online.  Resume not actually added to the file yet


Comparing null against an integer is a bit weird anyway -- it might be safter to do the null check or use an `||`


I'm sure your fancy `lodash` library you've included could make handy work of this type of task :wink:


Não existe a palavra "logado" em português, apesar de usarmos informalmente.


DEF -> DEFAULT


do you really want to leave this open for multiple instances? I think `document.getElementById('garage-list')` may be more appropriate.


Since these branches are equal, let's simplify the code by collapsing them:

`if ((node.getType() == AND) || (node.getType() == OR)) {`


Create user with temporary password


nit: UINT32_MAX


啦啦啦关帝


If you have this log message, the comment form above is a bit unnecessary, right? How about moving it inside here instead.


@JPrause no backport needed as far as I know. We will use the Tuskar only in 3.2


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


O then deve expressar o resultado da ação realizada no when. Nesse caso, o link que direciona aos conteúdos do Inovaula é alterado, ok? Agora ainda falta informar o novo valor, que deveria estar explícito como parâmetro no when.


Why not use:

  -p, --password PASSWORD       encrypted password of the new account

for useradd instead?


This is missing the `reason`


We should check whether this succeeded and fail the action if it did not.


Where are my tests? :-)


Add a break to both cases


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

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


hm, zero index for accessing it, and actual size when initialising...


Hi @clintonb , this work was merged to edx/edx-platform months ago.
https://github.com/edx/edx-platform/pull/5635
Why is this being resurrected for the edx-solutions/edx-platform repo now?
Others, please hold off on spending time on this until we understand what the need actually is.


nit: 2015


Other than comments above it's ok.


Because of circular dependences with assert_log / assert_break


The idea is to use the `foreign_ref` attribute on the `BruseFile` to keep track of what we call the file on the server. I suggest we use `SecureRandom.uuid` to rename the files when they get uploaded. That way we keep the files anonymous and it will be harder to recognize them if someone without permission happens to access the server.


Do you think there could become an issue with leaving this mocks available for the remainder of the session? It seems optimal if we can unmock them after the test, but if it's harmless we can forget about it.


What does this mean? Does this mean the permissions are already 0700?


lowercase `l` (trivial)


Missed a quotation mark `"` in the `<img src=>`


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


is there a reason for this epsilon increase?


Suggest making the default empty:

```c
template <typename T>
class TimePoint;
```

and adding an explicit implementation for gpr_timespec:

```c
template <>
class TimePoint<gpr_timespec> {
  // ...
};
```

This will give clearer (IMO) error messages when the mechanism is misused.


I'd like to keep the google group referenced here-- for some folks, it's the best method to asynchronously get help, and it's where we generally post major announcements.


Feels like this should be done in the caller instead of doing it here and then returning it.


remove


I have to clean up these logs.  I was using them for debugging and now they're irrelevant.  Before I was doing a chmod here, but it would error with errno 17. I then moved it to a separate function but forgot to remove the conditional


This logic is really hard to understand. IMO its better to store current camera index in `selectedCamera` variable  instead of the whole `videoDevice` object. Then you'll be able to significantly simplify this logic.


Forget about my comment. It was nonsense. Of course `key[6]`has six bytes only and the initialization is perfectly fine.


disable PosDebug output when level is not DEBUG.


There is already an API for this.


Can you open a PR (including the composer-dependency change to `php: ~5.3` and an updated composer.lock) and confirm that phpunit is "green" on PHP5.3.3 in the descrption? That would be awesome!


será => é


Slightly disappointed that ```json.Unmarshal``` doesn't take an ```io.Reader```. Oh well.


We put the Signature in the "soft key". That way, we will never find by the key the c code, but we will generate the c code and compare it, so we won't recompile.

But we will create way too much useless keys in the keys.pkl file. So we need to make a difference for the signature in the key and the signature for merging. (or skip constant folding for that op for now).


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

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


might want to explain this change a little bit in detail:

 we can run into the situation where the missing shards cannot be reconstructed from what we have, when and only when in the first round of read. because we cancel the recovery in this case.


Exibir uma janela de confirmação é o resultado da ação de selecionar remover, daí que escrever o step dessa forma confunde, pois passa a ideia de que isso deveria estar no then. Uma alternativa seria...
When seleciono a opção “Remover” no menu
And clico em confirmar na janela de confirmação de exclusão


Don't specify any property tracking id in your config/environment and it
won't be included.
On Apr 8, 2015 7:10 AM, "BigBlueHat" <notifications@github.com> wrote:

> Should we bother outputting this at all if we're on localhost? It's
> always nicest to not load any external resources when developing locally. [image:
> :thought_balloon:]'s?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hypothesis/h/commit/f18647cc1fa22715625ebe37187f748ff2a41362#commitcomment-10624656>
> .
>


@lamagnifica Not sure. I received a notification this morning. My guess is that solutions is merging edx/edx-platform into edx-solutions/edx-platform. @smagoun would you confirm, please?


completion block przekazany do metody dismiss nie jest wolany. powinien byc przekazany do dismissViewControllerAnimated


Example_7.3.c
	解释为什当待排序有序是，排序的栈最大深度2N，根据二叉树的性质可知，二叉树最小深度lgN（完全平衡二叉树），最大深度N-1（退化为只有一个叶节点），此题出现2N是应为，每次push，操作两次STACKpush（）。还有一个主意的问题就是，记住应该每次先压入较大文件，这样确保栈的最大深度为lgN。


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

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


not really - open() or chmod() might have failed as well, no?


Can we just get rid of this method? It's confusing when it's used because it's not obvious what a stray boolean in the parameter list is, I always need to double check. Explicit method names are clearer when the actual functionality is being altered.


+1 for the `bin` declaration. New path: `%appdata%\Composer\vendor\bin`
I wish it put the files in `\ProgramData\CompserSetup\bin` as that path is automatically added by Composer upon installation. Alternatively, Composer should add the app data path to the `path` variable. Anyways, it's still better as I need only add one path.


exercise 7.11 and 7.12 different
	就是堆栈的深度不同，这个问题在实际应用中应该注意，像7.12这种做法他保证不了，总是压入较大文件，所以7.11算法从空间复杂度考虑优于7.12,7.11节省内存，但是7.11每次压栈前要判断那个文件大，这样判断要花费时间，而7.12每次压栈前不必要判断文件大小，这样节省时间，所以算法的时间复杂度和空间复杂度综合考虑，那个算法更优我没有研究，所以我不能在这个地方给出答案。


FINDING. THAT'S SUPPOSED TO BE FINDING.


You also create an username - let's give more details of what we do and why here.


There probably was, but I can't remember why.


And I can't remember if they needed to stay changed or not :/


Tem que descrever a ação que é realizada diretamente, não uma intenção. O que significa tentar acessar a página? Significa clicar em uma opção X que dá acesso à página.... logo, é isso que deve ser dito aí.


remove


Sorry I left a few comments in the commit instead of the PR :/


the original test confirmed not just the number of transactions but also the timesteps of their occurrence. I would prefer for that level of detail to be maintained.


This makes scan operation impossible to cancel! When you'll hit cancel button `destroyPreview()` and successCallback won't be called.


same note here


The general policy is for drivers to not call into platform code. You need to pass the required args (base address, multiplier and divider into init function).


I think this should be optional - maybe I don't want to use keys.


exercise 7.14.c
	7.14 和 Example_7.3.c 比较最大区别就是，Example_7.3.c代码没有判断压入堆栈时，没有检查文件大小为1或是0，这样导致文件大小为1或0的文件也压入堆栈，这样导致浪费时间，7.14代码就做到了，压入对堆栈前检查，压入的文件大小为多少，只要问价大小为1或0，就不用压入堆栈，经过两次测试100个和相同元素，Example_7.3.c压入堆栈的次数为258次，而改进版本的7.14.c代码压入堆栈次数为182次，经过改进就大大减少了，压入堆栈的次数，但还是没有解决这个问题，到底哪个运行的时间少，一个压入前要判断，一个压入前不用判断。


Only one way to find out =)

On Wed, Apr 8, 2015 at 10:01 AM, Robert Carlsen <notifications@github.com>
wrote:

> And I can't remember if they needed to stay changed or not :/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rwcarlsen/cycamore/commit/6b0385d3218730a8372020f7debefc9f3bc84ff7#commitcomment-10625941>
> .
>

--
Matthew Gidden, Ph.D.
Postdoctoral Associate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph. 225.892.3192


será => é


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


因为CDUserFactory 没有什么状态，没什么东西，创建一个就行。


@xray7224, I will ramble back atcha!

Some events will themselves _be_ terminal events, and therefore will not have a terminal event to link to, as you mention, so not _all_ events technically have a terminal event.  I also worry about asking users to create the terminal event first because it might not be scheduled/determined when the first prep events happen.  I think they might have to go back later and figure out what terminal events the prep events were leading up to, though of course that isn't ideal.  I think because their process is itself sort of non-linear that there's only so much we can do to smooth it out.

Thinking about it more, I think some events might just be regular monthly meetings (of parent mentors, for example, or even just staff meetings) and those won't necessarily have a terminal event, nor will they themselves _be_ terminal events.

I like your idea about auto-creating a placeholder terminal event, but I think they'll likely be doing prep for multiple terminal events at the same time, so it might not work out cleanly.

Good thoughts!  I agree that we should minimize the number of fields, in general, and maybe we can remove that checkbox even though  we don't have a clean prep/terminal division (in that there are some events that won't fit into those categories).


This is not accurate - this is the parent folder for the user folder based on the implementation.


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


unless I'm mistaken, the changes to each of these `test_foo_xactions` functions are unnecessary. can you please revert them or tell me why they're needed. thanks!


sou direcionado => é exibida


百万计运维经验。。。

我觉得现在我们的服务器上再放一个 mysql 都没问题。。

2015-04-08 20:41 GMT+08:00 yanjixiong <notifications@github.com>:

> redis和mongodb在内存上会不会有冲突呢？看到一篇文章说不能放到一台实例上。
> 百万级运维经验一：Mongodb和Redis数据不能放在同一个服务器
> <http://itindex.net/detail/49709-%E7%99%BE%E4%B8%87-%E7%BB%8F%E9%AA%8C-mongodb>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cnodejs/nodeclub/commit/67c098202b0de23d790ecd4486307dbe62be916b#commitcomment-10622797>
> .
>


Can you add the second layer from the minecraft 1.8 skins?


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


有空提交PR，一起来完善这个库:smile:


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


Please drop the quote at end of this paragraph. Thanks!


It's a standard Sublime API event.  I could... but we don't do that for the other Sublime APIs we use.  I'll add a link next update to the docs where this is covered.


此题课本上写的有错误。


They are indeed a mess; subtrees, the alternative, are complicated.
On Tue, Apr 7, 2015 at 23:43 msdos <notifications@github.com> wrote:

> Synchronizing your submodules gave me this error:
>
> ````Synchronizing submodule url for 'modules/completion/external'
> Synchronizing submodule url for 'modules/history-substring-search/external'
> Synchronizing submodule url for 'modules/prompt/external/agnoster'
> Synchronizing submodule url for 'modules/prompt/external/powerline'
> Synchronizing submodule url for 'modules/prompt/external/pure'
> Synchronizing submodule url for 'modules/syntax-highlighting/external'
> fatal: reference is not a tree: d0381d859c0eb8d52a6211281e622e89153ef812
> Unable to checkout 'd0381d859c0eb8d52a6211281e622e89153ef812' in submodule
> path 'modules/history-substring-search/external'
>
>
> I needed to run this command to, manually, change the commit hash without the need to remove/ add again the submodule:
>
> ```# http://blog.evan.pro/update-a-git-submodule-to-a-new-commit-without-cloning-the-submodule
> git update-index --cacheinfo 160000 6d7bc718aee81b9afa9e146b5567be014b3ff8f0 modules/syntax-highlighting/external```
>
> Submodules are a mess. :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sorin-ionescu/prezto/commit/c98da90662a11e7ac1e046ccd6a99edd21e68312#commitcomment-10616057>
> .
>


@xray7224
1. That's a good point.  All US addresses have a type (excepting some rural addresses, but that won't really come into play here), but not all have a direction.  I think it would be very rare for them to enter addresses outside of the city, but we may want to offer a null option for direction.
2. Agreed, I will add that.
3. Yes, the list of cities would be prohibitively large.


Why not merge the two and add the to the node API:

node.create_user(user, group, home, ssh_keys)

Then we can reuse this in other actions.


We've got our first snapshot release out here: https://oss.sonatype.org/content/repositories/snapshots/edu/wisc/my/portal/angular/angularjs-portal-home/4.1.1.19-SNAPSHOT/

With this configuration, `mvn deploy` uploads to the snapshot repository without running the javadoc/sources/gpg plugin, but they will be run with the release plugin


Breaks on Windows though as it can't find `autoload.php`.

Changing [line 19](/marcus-herrmann/wireshell/blob/master/wireshell#L19) of `wireshell` to this works:
```
if (file_exists(__DIR__.'/../../autoload.php')) {
    require __DIR__.'/../../autoload.php';
} else {
    require __DIR__.'/vendor/autoload.php';
}
```


@hrvojem I don't think the link to Consulting was properly formatted. I see "link to" at http://www.percona.com/doc/percona-playback/intro.html


TODO: Please add tests for this, similar to the tests for the other actions.


Thx!


Try to avoid global variables in tests. Use let syntax.


Awesome work, Stephan! But this added a ton of new features? Should probably have been version `2.2.0`, no?


File.read(Rails.root.join("spec/fixtures/response.json")) looks better imo :)


Indeed. This didn't work with `eval echo "${@//\\/\\\\}"` inside backticks, btw, only with `$(...)`.


typo: teh

Thanks!


Fix indentation


kein printStackTraces ... bitte mit Logging lösen.


I ad the extra line to push it to the 5.1 branch, i didn't want to change important things


@digitalextremist - there should only be one env variable: `CELLULOID_CONFIG_FILE` or `CELLULOID_CONFIG` with JSON/YAML or whatever.

CLLLD_FOO and CLLLD_BAR is still repetition.

Instead you could have `CELLULOID_CONFIG="{task:Fiber,group:Pool,debug:true,log:stderr}"`

I once wrote a gem: https://github.com/e2/nenv for this kind of stuff (YAML in ENV, bool handling, etc.).


O texto da feature não segue padrão algum. Adotar um dos padrões a seguir:
1 - As a ... I want to... So that...
2 - In order to... As a... I want to...


Nicht null zurückliefern. Entweder ein Ergebnis liefern, oder einen Fehler werfen.


Ressourcen sicher schließen. Entweder ein try-with-resources oder die Ressourcen in einem finally-Block schließen.


:+1: I get same hash.


@dkhamsing :v: :sparkles:


FYI @smagoun after October 2014 we moved all documentation rst files out of edx/edx-platform into edx/edx-documentation.


Deveria ter o step when para expressar a ação principal a ser testada. O problema se repete em todos os demais cenários.


Oh yeah, actually a good point ... I forgot about this :(