@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/35/violations
```
plone/recipe/codeanalysis/analyser.py:59:9: T003 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T003 print function found.
plone/recipe/codeanalysis/analyser.py:157:35: C812 missing trailing comma
plone/recipe/codeanalysis/analyser.py:190:77: C812 missing trailing comma
plone/recipe/codeanalysis/analyser.py:201:75: C812 missing trailing comma
plone/recipe/codeanalysis/analyser.py:206:60: C812 missing trailing comma
plone/recipe/codeanalysis/clean_lines.py:24:10: C812 missing trailing comma
plone/recipe/codeanalysis/clean_lines.py:55:63: C812 missing trailing comma
plone/recipe/codeanalysis/clean_lines.py:64:59: C812 missing trailing comma
plone/recipe/codeanalysis/check_manifest.py:21:17: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:40:20: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:87:74: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:96:70: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:97:14: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:155:32: C812 missing trailing comma
plone/recipe/codeanalysis/__init__.py:176:13: T003 print function found.
plone/recipe/codeanalysis/__init__.py:191:9: T003 print function found.
plone/recipe/codeanalysis/__init__.py:200:9: T003 print function found.
plone/recipe/codeanalysis/__init__.py:248:9: T003 print function found.
plone/recipe/codeanalysis/flake8.py:23:32: C812 missing trailing comma
plone/recipe/codeanalysis/flake8.py:35:36: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_jshint.py:113:29: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_jshint.py:148:37: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_check_manifest.py:24:42: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_check_manifest.py:29:37: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_check_manifest.py:35:75: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_check_manifest.py:40:67: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_check_manifest.py:52:67: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_jscs.py:82:36: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
plone/recipe/codeanalysis/tests/test_flake8.py:80:36: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_flake8.py:105:53: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_flake8.py:118:53: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_flake8.py:128:45: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_i18ndude.py:70:36: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_script.py:76:28: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_script.py:86:27: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_script.py:92:28: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:51:20: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:64:22: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:68:21: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:83:22: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:107:78: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:139:32: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:152:60: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_recipe.py:160:32: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_clean_lines.py:43:36: C812 missing trailing comma
plone/recipe/codeanalysis/tests/test_clean_lines.py:62:36: C812 missing trailing comma
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


You are perfectly right, thanks.


Deberíamos revisar todas las comprobaciones que se hacen de esta misma forma para no incluir return innecesarios


Have you find out any soluction for it

On 18-Aug-2017 4:57 PM, "Jasper Rietrae" <notifications@github.com> wrote:

> @rjeffm <https://github.com/rjeffm> thanks, I will look into the
> permissions. The device I'm testing this on is an OnePlus2.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/rjeffm/CallRecorder/commit/3a5462661ac93d938ce82cf4ede4f611558315b8#commitcomment-23719700>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ATOlYYoWW06ZnoR6JGIyYjc7Odkw1h4Tks5sZXUagaJpZM4O54Tx>
> .
>


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/36/violations
```
plone/recipe/codeanalysis/__init__.py:147:57: C815 missing trailing comma in Python 3.5+
plone/recipe/codeanalysis/__init__.py:182:61: C813 missing trailing comma in Python 3
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Uploaded the load tables code to github


input_processor_refactor (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (2063 of 2673 tests passed, 0 test warnings)

 * 544 tests had: AUD diffs.
 * 528 tests had: EIO diffs.
 * 228 tests had: RDD diffs.
 * 505 tests had: Table big diffs.
 * 157 tests had: ERR diffs.
 * 400 tests had: MTD diffs.
 * 289 tests had: DXF diffs.
 * 425 tests had: SHD diffs.
 * 434 tests had: BND diffs.
 * 319 tests had: MDD diffs.
 * 205 tests had: SSZ big diffs.
 * 14 tests had: ESO big diffs.
 * 9 tests had: MTR big diffs.
 * 13 tests had: ZSZ small diffs.
 * 28 tests had: ESO small diffs.
 * 22 tests had: MTR small diffs.
 * 3 tests had: delightin diffs.
 * 3 tests had: delightout diffs.

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


Hey,
danke für die Nachricht aber ich habe erst heuer nebenberuflich mit iOS
Entwicklung angefangen. Ich glaube für den job hab ich noch zu wenig
Erfahrung :)
lg Daniel

Ponsii <notifications@github.com> schrieb am Mo. 21. Aug. 2017 um 16:26:

> Hi Daniel,
> Ich würde gerne mal mit dir reden! Wir sind gerade auf der Suche nach
> einem Head of iOS für unser Team.
> Email: philipp@cashquizz.com
> Danke!
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/DanielR025/SkateApp/commit/b9ba374d8063048defa41cf2120e784b2cb38ad3#commitcomment-23770899>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AZjr8aWM2eouKo7bzEOeAmBZtFqz8MZEks5saZOVgaJpZM4O9aRD>
> .
>


accepted


input_processor_refactor (mbadams5) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/8689311c-5e17-4012-b396-6393f6148684 "simple-toolchain-888"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/354e3c79-9125-452a-9c47-9bee3a388d5d "simple-toolchain-888") deployed [simple-toolchain-888](https://github.com/phthom/simple-toolchain-888/releases/tag/deploy-ibm-yp-us-south-thomas1-fr.ibm.com-dev-20170822-095258 "deploy-ibm-yp-us-south-thomas1-fr.ibm.com-dev-20170822-095258") to [dev](https://console.bluemix.net/apps/3161936d-5559-4ffa-af1b-cebff1d13e24?env_id=ibm:yp:us-south "ibm:yp:us-south:thomas1@fr.ibm.com:dev"), including this commit


ERROR: Redundant 'public' modifier.


input_processor_refactor (mbadams5) - Win64-Windows-7-VisualStudio-14: Tests Failed (2062 of 2639 tests passed, 32 test warnings)

Messages:

 * 543 tests had: AUD diffs.
 * 527 tests had: EIO diffs.
 * 228 tests had: RDD diffs.
 * 504 tests had: Table big diffs.
 * 399 tests had: MTD diffs.
 * 149 tests had: ERR diffs.
 * 289 tests had: DXF diffs.
 * 425 tests had: SHD diffs.
 * 433 tests had: BND diffs.
 * 319 tests had: MDD diffs.
 * 205 tests had: SSZ big diffs.
 * 16 tests had: ESO big diffs.
 * 10 tests had: MTR big diffs.
 * 11 tests had: ZSZ small diffs.
 * 26 tests had: ESO small diffs.
 * 22 tests had: MTR small diffs.
 * 3 tests had: delightin diffs.
 * 3 tests had: delightout diffs.

Failures:

integration Test Summary
 * Passed: 610
 * Failed: 1

regression Test Summary
 * Passed: 33
 * Failed: 576

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


Yasuhiro Matsumoto wrote:

> Sorry, this break utf-8 string
>
> ```diff
> --- a/src/terminal.c
> +++ b/src/terminal.c
> @@ -2145,7 +2145,10 @@ term_update_window(win_T *wp)
>  		{
>  #if defined(FEAT_MBYTE)
>  		    if (enc_utf8)
> +		    {
>  			ScreenLinesUC[off] = NUL;
> +			ScreenLines[off] = NUL;
> +		    }
>  		    else if (!has_mbyte)
>  #endif
>  			ScreenLines[off] = NUL;
> ```
> thanks @k-takata

I was trying to avoid duplication.  Perhaps we should use:

#if defined(FEAT_MBYTE)
		    if (enc_utf8)
			ScreenLinesUC[off] = NUL;
		    if (enc_utf8 || !has_mbyte)
#endif
			ScreenLines[off] = NUL;

We are actually trying not to overwrite ScreenLines[] when it was set to
the second byte of a double-byte character.  It's easier to understand
if we set it here:

#if defined(FEAT_MBYTE)
		    if (enc_utf8)
			ScreenLinesUC[off] = NUL;
		    if (!enc_utf8 && has_mbyte)
			ScreenLines[off] = byte_two;
		    else
#endif
			ScreenLines[off] = NUL;

Hmm, I actually don't see how we can get here without FEAT_MBYTE, we
never have double-wide characters then, right?

--
It doesn't really matter what you are able to do if you don't do it.
				(Bram Moolenaar)

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


So, here's a question: what does the regen health system still do?

It was originally intended to cover resetting, but we do that in evade code now (and immediately set full health as outlined in 62e7d70 ) - so why do we still need this code?


test


Вопрос снимается. Увидел в предыдущих комитах она была добавлена по ошибке.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/37/violations
```
plone/recipe/codeanalysis/__init__.py:147:57: C815 missing trailing comma in Python 3.5+
plone/recipe/codeanalysis/__init__.py:182:61: C813 missing trailing comma in Python 3
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


maybe show how to get that distribution (ex with docker cp command)?


Added callback function for the minimization. Fixed the bug resulting in wrong standard errors.


@tkelman, this is an initial commit that I perhaps should have held back. Both IterableTable and FileIO would soon be bumped to later versions.
- IterableTables will be bumped to v0.5.0 [#10840](https://github.com/JuliaLang/METADATA.jl/pull/10840).
- FileIO will need to be bumped to the next version once bedgraph files are registered.
- DataFrames can be relaxed to v0.9.1 as v0.10.0 excludes features.


unnecessary let


The concatenation has been lost through `" .`


```
Warning: Maximum number of iterations has been exceeded.
         Current function value: 4.561534
         Iterations: 50
         Function evaluations: 1170
         Gradient evaluations: 78
Mean Estimates:
               Mean        SD      Income   Income^2       Age      Child
Constant  -2.010873  0.558518    2.293647   0.000000  1.284360   0.000000
Price    -62.791246  3.316215  589.490516 -30.252891  0.000000  11.050478
Sugar      0.116244 -0.005813   -0.385428   0.000000  0.052287   0.000000
Mushy      0.499868  0.093713    0.746653   0.000000 -1.353060   0.000000
Standard Errors:
               Mean        SD      Income   Income^2       Age     Child
Constant   0.327211  0.162715    1.210062   0.000000  0.630978  0.000000
Price     14.827514  1.342623  270.905894  14.125879  0.000000  4.122291
Sugar      0.016047  0.013513    0.121682   0.000000  0.026011  0.000000
Mushy      0.198686  0.185443    0.803185   0.000000  0.666919  0.000000
--- 602.1178584098816 seconds --
'''


Would be safer to do something like

try (LineIterator iterator = IOUtils.lineIterator()) {
  while...
}


Is there any guide how to use Three.js if this method was removed?


perfect!
Accepted


Nice!


@kauffj Fixed in https://github.com/lbryio/lbry-app/commit/48b02567a077ee9c4beee6aea319c89460a3a6a7

A few additional points.
- I had to call `this.forceUpdate()` in the settings page component, otherwise, the page retains the previous selected language.
- The **SETTINGS** and **HELP** tab labels do not update until you change the current tab. I think there is another component that is not being updated after the language has been changed.
- It looks like there is a bit of lag after selecting the language (~3s) before the app toolbar updates. Not sure if this is due to file I/O which seems unlikely because `forceUpdate` on the Settings component is almost instantaneous, or due to the app being run in a VM.


boa


Cut stings


sorry - what's the best way to figure out the version range this error affected?


Já removi, foi antes de implementar o teste eu tinha colocado pra ver se tava camelCase =B


It is


And it iz


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/3402a273-9de3-4d1a-89d7-287d6d66b81d "simple-toolchain-20170819193356682"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/dd298aad-dc3d-40de-8244-0ff58822b674 "simple-toolchain-20170819193356682") deployed [bluehack-drugs-poc](https://github.com/vverzellesi/bluehack-drugs-poc/releases/tag/deploy-ibm-yp-us-south-vbitelman-bluehack-20170823-001047 "deploy-ibm-yp-us-south-vbitelman-bluehack-20170823-001047") to [bluehack](https://console.bluemix.net/apps/63b403d0-c2f5-42b5-9cb3-dd682ec28b86?env_id=ibm:yp:us-south "ibm:yp:us-south:vbitelman:bluehack"), including this commit


Created multidev environment [wiegele#ci-58](https://dashboard.pantheon.io/sites/e5d4b65a-3be9-4852-bb2f-7b73d4f5eeaf#ci-58).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-58-wiegele.pantheonsite.io/)


[practica1 (fundamentos de programacion).docx](https://github.com/katsstella/practica1_pdf/files/1243770/practica1.fundamentos.de.programacion.docx)


@mdstaff Fixed in https://github.com/react-native-training/react-native-elements/commit/c173e96c2aeabcffa4a1e071f8a6ee98bd9210cf. You can use the next branch to use it right now. Will be released in 0.17


Is this in the main branch ? (is ble available in main micropython port for esp32) ?
Thanks


LocalNodeNetwork (XuanLuoLBL) - x86_64-MacOS-10.9-clang: OK (2743 of 2743 tests passed, 0 test warnings)

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


Ok


Will want to just have this in npm (`npm install --save axios`)


Closes #125


Indentation off? Or is that just GitHub?


This breaks building a multi-platform v6+v7 config.

```
arch/arm/mm/cache-v7.o: In function `rpi_dma_inv_range':
(.text+0x1fc): multiple definition of `rpi_dma_inv_range'
arch/arm/mm/cache-v6.o:(.text+0xb4): first defined here
arch/arm/mm/cache-v7.o: In function `rpi_dma_clean_range':
(.text+0x244): multiple definition of `rpi_dma_clean_range'
arch/arm/mm/cache-v6.o:(.text+0x100): first defined here
```


Not really, this just reflects that a mark4 header cannot even be initialized without knowing `ntrack` -- inferring it from the data would need to happen in `base.Mark4FileReader.find_frame`


It's nano technologies from monolith  %_%


I don't know if it's more efficient (in term of the time it takes to run, as the compiler makes many optimizations anyway), but it can be written shorter/cleaner. There are many ways to write things, but I would do something like..

Lol, I was about to start writing when I realized that this doesn't make any sense. You can't make something out of nothing, you try to get the renderer out of thin air. You can't do it like that. When you create new ```OutputParams```, you do that only using a ```PmsConfiguration``` instance. That doesn't have any information about the renderer in question (how could it?), which means that ```params``` doesn't know anything about the current renderer. That means that the renderer you get and use to read the parameters isn't the renderer you want, it's probably the default renderer or ```null```.

If the above code was valid, I'd write it like (only for education, as I said, the code won't work):
```java
RendererConfiguration renderer = new OutputParams(PMS.getConfiguration()).mediaRenderer;
int cbr_video_bitrate = renderer.getCBRVideoBitrate();
try {
    int max_video_bitrate = (int) Math.round(Double.parseDouble(renderer.getMaxVideoBitrate()));
} catch (NumberFormatException e) {
    ...handle...
}
```


fyi @ryanbr


Nope


@kevinzetterstrom
Found the issue with upload on develop. Prod is on 3.3.3 so it's safe.

https://vimean.atlassian.net/wiki/spaces/API/pages/68555956/Changelog
As a heads up, I see in the [PR](https://github.com/vimeo/vimeo-networking-java/pull/238) for this you link to the changelog, but you didn't update the breaking change in 3.3.4. `upload_link_secure` -> `upload_link`.

There would have been other issues (because of the upload code pointing to just `3.3`....), but we still have to be careful with updating the API versions. Anytime we update the API it should likely also come with upgrade path tests and a call out to QA to test as well.

When our models change (which ideally isn't very frequent), we have to consider if caches will break, including our persisted tasks for upload/download as well as request cache.

We have another required version bump for live (which you called out in your research doc) which means we'll have to be extra careful about all those individual changes. I believe this is all reflected in your research and subsequent tickets. But since we plan on cutting an RC next Friday, we'll have to pull in a ticket to patch this particular issue - separate from live.


Vai fazer o que mais tarde gatão?


I think it's not enough to simply strip `CASE` off the mods. There's other synthetic stuff that's added by the scala parser when it sees the `case` keyword. It needs to be removed as well.


Burro


注释分别是 容量  温度


AHHN OH AHHHHHH ÕÕÕH AHHHNNN OHHHH
VC CAIU NO GEMIDÃO DO ZAP


Created multidev environment [build-tools-606#ci-1](https://dashboard.pantheon.io/sites/bda76f56-309c-4250-886f-2f62ab390ac5#ci-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-1-build-tools-606.pantheonsite.io/)


bye-bye


Added space after StackReg. Without the space an error occurs: "Unrecognized command:"StackReg".


kiva-minor-mods (nealkruis) - Win64-Windows-7-VisualStudio-14: OK (2744 of 2748 tests passed, 605 test warnings)

Messages:

 * 609 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table small diffs.
 * 3 tests had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 605
 * Failed: 4

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


Created multidev environment [build-tools-605#ci-1](https://dashboard.pantheon.io/sites/f6c54874-1451-449c-9761-26f4ccb1a669#ci-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-1-build-tools-605.pantheonsite.io/)


I see that now. Cool.


Don't even need this apparently


é isso mesmo


test


> no wayyyyyy


```python
FROM:
    if len(reverse_path)>1 and path.count(os.sep):
TO:
    if not match and len(reverse_path)>1 and path.count(os.sep):
```
Was there a issue that it needed `not match` added as the first test?


Thank you for your hard work so far to continue this bot


this needs to be committed


Hi Esan,

Better if you delete the comment inside the constructor and getters, setters.
Added method comments for each method.
Author: Esan
Reviewer: Gulani
Mediator: Pradeep

Thanks.


RemoveSinCosSolarShading (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2138 of 2139 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 610
 * Failed: 1

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


quero dar a bunda


http://www.smogon.com/forums/threads/pokemon-sun-moon-battle-mechanics-research.3586701/page-70#post-7461464 As this post point out, Ate abilities should not affect Judgment and Technoblast even when there is no plate/drive present, i.e. they remain normal type with the absence of corresponding items, and this is the case not only in gen 7. Seems like the current code missed this, http://replay.pokemonshowdown.com/gen7balancedhackmons-621244144


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


This is not proper authorship


<!DOCTYPE html>

 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
 <title>Homework Week 2</title>

Sports I Do and Love

 Swimming

 I learned swimming at the age of 28.  First, I inhaled lots of water and got frustrated because I did not do things correctly, but I kept going! I overcame my fears! The joy and "I did it!" feeling that follows is so satisfactory and powerful that all my efforts were worth persisting!

 Running

 I started running two years ago with a local chapter of international Hash House Harriers. My goal was to become fit and broaden my circle of new friends. I am happy to say that I achieved both my goals - deciding to join Hash is one of the best decisions I have ever made.

 Hiking

 When I was a kid, I often visited my uncle's big family in a beautiful mountainous countryside. My crazy cousins often took me to the mountains to collect wild berries and firewood. I still love going up to the mountains whenever I have time and a good company of friends.


Without this the javascript will not load and the datepicker will not function unless you refresh the page.


This file isn't used at all, can be deleted


In no universe does deciding not to build a debugging file constitute the need for "proper authorship." Please @cannondalev2000 learn how the gpl works.


wow i guess i need to check my email more often ...... +1 something is being fixed :D


Does port mean copy and paste now?  Sorry for the trolling hehe


helle word


geometry_editor (macumber) - x86_64-MacOS-10.10-clang: Build Failed

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


Create resources directory


Hi Esan,

Add the header comment.
Add method header comment.
Author: Esan
Reviewer: Gulani
Mediator: Joel

Thanks.


Thanks for the answer, and download the file in Spanish, configure the locate in 'es' and continue the same in English.


Мб таймаут зафигачить в `TaskPool`?


TeamCity ChilliSource.Web :: ChilliSource.Web.Forms :: Pull-Request [Build 0.9.53](https://build.bluechilli.com/viewLog.html?buildId=71533&buildTypeId=ChilliSource_ChilliSourceWebForms_2_PullRequest) is now running


![](https://github.trello.services/images/mini-trello-icon.png) [Delete a Board](https://trello.com/c/nmnMFb6K/17-delete-a-board)


TeamCity ChilliSource.Web :: ChilliSource.Web.Forms :: Pull-Request [Build 0.9.53](https://build.bluechilli.com/viewLog.html?buildId=71533&buildTypeId=ChilliSource_ChilliSourceWebForms_2_PullRequest) outcome was **FAILURE**
Summary: Will fail Build time: 00:02:07


@jolting I wasn't able to write a valid enable_if_t expression here... any hint?


TeamCity Keyman :: Keymanweb :: Test: Pull Requests [Build 10.0.52.pull.219](http://build.palaso.org/viewLog.html?buildId=96464&buildTypeId=Keymanweb_TestPullRequests) outcome was **SUCCESS**
Summary: Running Build time: 00:02:16


Issue #52


TeamCity Keyman :: Keyman Android :: Test: Pull Requests [Build 10.0.33.pull.219](http://build.palaso.org/viewLog.html?buildId=96465&buildTypeId=KeymanAndroid_TestPullRequests) outcome was **SUCCESS**
Summary: Running Build time: 00:02:12


merging branches


Hi Esan,
Remove the white spaces.
Add the comments for methods and conditions.
Author: Esan
Reviewer: Gulani
Mediator: Joel
Thanks.


this line needs to be removed, showing as an error in master when pulled to local.


@dkorolev As discussed offline with @mzhurovich, I'm pushing this second breaking change to the same PR.

I have just realized I should have bumped the `c5t-current-schema-ts` module version in `package.json` because its dependencies have changed. Also, update the generated header comment to reference the right version of `io-ts` and `c5t-current-schema-ts`.


Break it for as much as you wish as long as by the time it's being merged it's doing the job and passing the tests! :)


Created multidev environment [stevector-d8-composer#ci-1](https://dashboard.pantheon.io/sites/ad741e40-0f96-47ac-b595-07ea93d47f6e#ci-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-1-stevector-d8-composer.pantheonsite.io/)


测试pullrequest


@Nadahar Thank you :+1:
Right, i did forget the format error handling...memory bird symptom.
The other part of your code look nicer, but i'm even able to make it compile, so i guess that's outside of my skill, but at least i get the confirmation that one more clear, short way was possible.

Yes, i was suspecting this, but when i tried `PMS.getConfiguration(renderer)` i got a compilation error and reverted back because i was only wishing be able to finally compile it, even if it was not working as with my much more simple precedent implementation.


LocalNodeNetwork (XuanLuoLBL) - i386-Windows-7-VisualStudio-14: OK (2749 of 2749 tests passed, 0 test warnings)

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


![](https://github.trello.services/images/mini-trello-icon.png) [Get a List with details](https://trello.com/c/gEU9u6z3/15-get-a-list-with-details)


Now, it keeps doing resize until there is an open index.


Good Idea,
I think it is working now:

C:\Users\Rick>C:\Python27\python.exe "C:\Program Files (x86)\OpenCPN 4.8.0\plugins\weatherfax_pi\data\foobar.py"
http://www.weathercharts.net/noaa_ukmo_analysis/previous/PPVA89_fullsize_00z_last.gif

C:\Users\Rick>C:\Python27\python.exe "C:\Program Files (x86)\OpenCPN 4.8.0\plugins\weatherfax_pi\data\foobar.py"
http://www.weathercharts.net/noaa_ukmo_analysis/previous/PPVA89_fullsize_00z_last.gif

So I need to use:

> C:\Python27\python.exe "C:\Program Files (x86)\OpenCPN 4.8.0\plugins\weatherfax_pi\data\foobar.py"

In that upper line?


## 1.1 git中的版本号
最新的版本会保存前一版的部分版本号，版本不断更新就会形成一串目录。


a


closes #2


kiva-minor-mods (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2779 of 2782 tests passed, 607 test warnings)

Messages:

 * 610 tests had: AUD diffs.
 * 3 tests had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 623
 * Failed: 3

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


Hey Kiran,
I don't think this code is right. Could you do it again? Please let me know if you have any difficulties.


I'd comment but I think you said it better

Wednesday, June 14, 2017 - 2:49 PM
RJ Trujillo (AOSPA): I don't have any respect for people that steal the work of others honestly. No matter how good it is, it is still stealing


@Sami32 The point is that you want the renderer, not the configuration, to get the bitrate values. To do that, you'll need to have the ```RendererConfiguration``` instance that applies to that request (All DIDL replies is generated from a request made by a renderer). The way to do this, is to add ```RendererConfiguration renderer``` as a method argument. But, it gets complicated quickly because that also means that you have to update all code that calles ```getRealVideoBitrate()``` to call ```getRealVideoBitrate(RendererConfiguration renderer)``` instead. But, since this method is ```public```, you can't do that without breaking the API (thus making plugins not work if they call ```getRealVideoBitrate()```). Finding all code that calls a method is easy using an IDE, but without that it's almost hopeless. I guess you could build it and fix one error at the time, but it would take "forever". In Eclipse, you simply click "Find References" and all code that calls the method is listed and you can click in the list to jump directly to that code.

The way this is usually solved is to make a new method ```getRealVideoBitrate(RendererConfiguration renderer)``` while keeping the old ```getRealVideoBitrate()```. You then make ```getRealVideoBitrate()``` deprecated and replaces its code with a simple call to ```getRealVideoBitrate(null)```. In this way the "old" method will call the new, but without the information about the renderer. The new method must then be able to handle that ```renderer``` might be ```null```, in which case you must skip doing your renderer specific tests.


@mzhurovich The `value_` should likely be reset to the analog of `nullptr` (default?) in the missing `else` here. Otherwise the optional seems to keep the old value implicitly.


Right, this does a forced recursive sub-directory of any folder is defined to be scanned. Wouldn't that include the root directory so all directories are being scanned at root call and then again at any path folder call w/sub-directories? and that can also triple or more if those sub-folders are also specified by Plex to be scanned? The question is to why this was done? Wouldn't this cause the scanner to run ALOT longer then wanted with many rescans of the same folders?

You can see this when you unique the log names.
![image](https://user-images.githubusercontent.com/16750699/29594876-966674e0-8779-11e7-8988-b7e478aee13e.png)

"Pan's_Labyrinth" & "`Movie" were scanned twice. Once from Root scan and then this piece of code of a force scan of sub-directories. Then again when directly called to be scanned by Plex.


+1, need an `else`.


kiva-minor-mods (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


In deregister I think you have to remove the not equals to sign.


Actually @CookieWookiee *


juikukiuuu


@dameng03 if the comment like this is no longer required, I'm not sure are they still required or I can remove it as I notice one. Can you please remove it from the code?
similar commented console.log codes are piling up in our codebase, unless it's a comment written to explain something. We have so far 100++ console.log in our code...


运行此文本


new


@Nadahar Hola, i stoppped when i read that you said "complicated", as i already understood that i should better keep with my hardcoded value ;-)

I will read it tomorrow, as for now my eyes close themself...


Homework 01 update


This is used to generate assignment names for both limits and warnings. While I agree this makes the warnings look less cluttered, I think it is useful to know due dates for limits that span multiple days.


@afanfa & @vehre: Do you know if this `#ifdef GCC_GE_7` wrapping the `caf_team_t` `type_def` is necessary? My tests with GCC/GFortran 6.4 (just using the test suite) don't see any failures by deleting this... but I know teams aren't implemented on the compiler side until GCC 8.x. Is there a danger to removing this so that 6.x compilation no longer fails?


The backtick (`) following **.typekit.net** is a typo?


Yup, sure @trtshen, I will remove it, those console logs needs to be removed, thx for reminding.


Nothing goes there. We have been duplicating effort in parallel. We need a higher-throughput communication channel already.


Foobar.py  top line
#! C:\Python27\python.exe

print "http://www.weathercharts.net/noaa_ukmo_analysis/previous/PPVA89_fullsize_00z_last.gif"

I changed the weatherfaxinternet.xml file line back to
"foobar.py" without the full path.

Darn, it is still a double path!!  What's happening here? Some Cache? I shut Opencpn down and reopened to run this one...second time.

![screenshot 124](https://user-images.githubusercontent.com/2363959/29595199-0542c8e6-8784-11e7-8fdd-0b7d310b0796.png)


@Slava

Awesome, I already know Im going to love it! It's already up in web store but it tells me that Store is empty even when `__RELAY_DEBUGGER__` contains correct store & all the data. I take it's not ready yet?


Hi! termpix (Rust) does the same thing with this code. Hope this can help you ;)

[find_colour_index()](https://github.com/hopey-dishwasher/termpix/blob/master/src/lib.rs#L51)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2136 of 2136 tests passed, 0 test warnings)

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


Lmao


Hmmmmmmmm I wonder


上楼我也是，报错


If this works fine, can you submit a pull request please?


写一个评论


I think there should only be a single exit point in this function; there's no need to introduce extra returns and my advice would be to revert the switch statement in its entirety to its original state.


#5768-Example-files-which-show-discontinutity-in-air-or-water-flow-rate-through-fans-and-pumps (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2138 of 2139 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 610
 * Failed: 1

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


E mai ok sa returnezi o eroare in cazul in care nu ai putut crea un FileWriter decat sa printezi un mesaj de eroare


[55 gaurav kumain.CSA.5th sem.docx](https://github.com/sgupta12392/Operating-System-Assignment-1-5th-sem/files/1245008/55.gaurav.kumain.CSA.5th.sem.docx)


Poti grupa apelurile si verificarea erorilor pentru a face codul mai compact


gregr


TODO: Nu am inteles de ce ai avut nevoie de o copie a variabilei `fileName`


Fix for issue #257


fgrgrg


Nu este nevoie sa initializezi si mutex-ul pentru ca este initializat deja cu valoarea default


I hope this commit very soon will be in release! Great job


La fel ca mai sus: returnam eroare in loc sa printam un mesaj de eroare


Try running npm install && bower install


I'm just going to take your points from the top:

- I cannot see any benefit in not letting it work with regression data. What are you afraid might happen? :-)
- I'm in the process of rewriting the general interface to avoid returning a function, so this is a temporary situation.
- no-one is forcing anything - the whole point of the `predict_model`/`model_type` interface is to make it easily extensible without having to overwrite the presupplied `predict` method. There's no way I'm going to implement converts for every model in existence which is why I'm targeting frameworks with consistent interface.

I think there's a very strong reason for a consistent and predictable API. Users will become accustomed to the format of the returned data.frame and can easily and quickly inspect the results without adjusting to the type of model or input data. I will flip it around and ask what benefit you see by introducing inconsistency?


@yomguy
Why mounting and copying "lib/plugins" ? What is/should be inside this directory ?
As the directory does not exist it causes an issue, see https://travis-ci.org/Parisson/Telemeta/builds/260887678?utm_source=github_status&utm_medium=notification

Can I remove it ?


#shortcut
`setRepliedMessagesReplyProperty([message], self.channelMessages)`
`  .then(deffered.resolve)`

پ.ن: ایندنت نشده


I would prefer to use `runOrdersOf` as it's accessing `RunOrder` instances from string.


Puzzle `6-8bd69a57` discovered in [`src/main/git-elegant`](https://github.com/extsoft/elegant-git/blob/master/src/main/git-elegant) and submitted as #30.


Puzzle `6-228df600` discovered in [`src/main/git-elegant`](https://github.com/extsoft/elegant-git/blob/master/src/main/git-elegant) and submitted as #31.


Am mutat functia asta direct pe obiectul `Request`.
Putem chiar sa facem validarea direct in functia `submit()` pentru ca este o singura linie de cod.


Please use singular, imperative style "Store", not "Stores"


Was this failing on some platform? This should be fixed in C++11. Can't imagine any compiler that doesn't support that these days.


fregfg


`isChannelIdActiveChannel()`


这里替换不是加号，而是空格，在URL的定义里是允许出现空格的，空格需要使用+号代替


Maybe check for `jQuery` too here? Also should this be `window.$`?


Yes why not.


Am initializat writerii in `main()` pentru a putea trata si erorile intoarse de noua functie `NewFileWriter`.

Numele writerilor sunt schimbate pentru a evita orice confuzie (initial, am crezut ca intr-un fisier sunt scrise textele venite cu prioritate, iar in alt fisier textele obisnuite).


When i mean `on heap`is allocating manually on heap, because internally std::vector allocate on heap an array.


Best practice: proprietatile sa fie adnotate cu numele atributele din obiectul JSON


Are translations going to be removed for some popular languages like vietnmaize and Korean? Because now I see English messages.


این چرا توی پابلیک ای‌پی‌آی هستش؟
استفاده‌ی بیرونی نداره
!کل این ریفکتور برای همین بود اصن


personal folders :gun:


Not my business, but usually any *NIX terminal when you open it by default is set as 80 characters, also it is more readable use in this way. IMHO we should follow 80 characters.


please let me know if this is too loud


Poti folosi `http.Error()` pentru a intoarce atat un cod HTTP, cat si un mesaj


`self.currentChannelMessages`


Statusul default returnat este `HTTP 200`


peut etre qu'un naming firstStepFormCreateFiche/secondStepFormCreateFiche serait plus adapté à ces fonctions :)


As discussed with @traversaro the orientation has been fixed in https://github.com/robotology-playground/icub-model-generator/commit/3bcd40872edf0812b0d1accf176e018904bd9493, while the new origin is not a problem because the shift is along the joint axis and the dependent `<origin>` are shifted accordingly.


Early return in loc de `switch{}`


Oluwatosin Akande (slack : tosinakande, github : tosinakande)


no, that breaks the interface implementation.


محتوای داخل این ایف رو بزار تو یه تابع جدا
مثلاً
`updateChannelMembersCount()`


More elegant, but it has the same impact. Only the first layer will be having associations. I want the full tree to have the associations.


@karpfen this should fix it. The columns can be labelled pretty much anything (reasonable) at all, and this should now all work


Hi,
I am already using php 7.1.8 and virtualbox 5.1.26.
What do you mean?

I finally got error messages about temporaryEjectDevice, nonRotationalDevice and setHotPluggableForDevice. I add some commits to fix that, based on [https://gist.github.com/derekschrock/f7c4ee9b1d3533088baa1e44ebc624c8](url). That works for me now.


After this commit all orbitals in FCIDUMP files generated by dmrgscf have irep 1.
Reverting the commit makes it fine again.


accepted


چرا تو این تابع بررسی نمی‌شه که آیا اصن پیام، ریپلای داره که بره کارای دیگه رو بکنه؟


[ASHIMA BALI CSA40.docx](https://github.com/sgupta12392/Operating-System-Assignment-1-5th-sem/files/1245041/ASHIMA.BALI.CSA40.docx)


Numele functiilor care intorc o valoare booleana sunt denumite de obicei in forma `isXXX()`, `areXXX()`, `shouldXXX()` etc., ca si cum ar raspunde la o intrebare.
`request.isValid()`: Este request-ul valid?


Weird. Why do you need to get rid of the dobblock then?


Because phpstan complains that this is a mock object instead of an Episode when we use it to build the Version object. Same is valid for the other typehints


Codechef Codes


Numele nou faciliteaza citirea for-ului: poti observa imediat ca for-ul merge cat timp workerii sunt activi.


Am tratat erorile aparute in urma scrierilor in fisiere


Fix for issue #258


Nu e nevoie de o functie separata deoarece e o singura linie de cod si nu face nimic special. E un simplu apel la `WriteLn()`.


dada


Este utila afisarea erorilor aparute in cazul in care server-ul nu poate porni (spre exemplu daca portul ales este deja ocupat)


I would have thought it would be clever enough to figure that out as you are passing in `Episode::class`. PHPStorm manages to work it out...
Does this mean EVERY mock needs a typehint?


Shame on you!


Prve dve naredbe ti vec radi remove() metod, iskoristi ga, gledaj da ne dupliras kod :)


This looks like cppcheck will perform UB in case we parse code like  ````x /= 0````. Can we handle this separately?


这两个设置server的貌似是互斥的，应该用if else吧

试了一下，就算指定了port，但是函数名不带ports的貌似会直接忽略掉这个port，返回值是success

而且set server也需要检测返回值，因为这个是外部输入的，很可能用了错误的格式，会返回ARES_EBADSTR


Descrizione del funzionamento del programma fwidbmgr.

In milanone è presente tutto il progetto C++ (?)


notice is missing.


`"notice"` is missing.


I think it is because I didn't add persistent notification.


为什么我用了没有效果呢


Theres also a crash on clang tests as seen on travis:

````
// ./test/SemaCXX/nullptr_in_arithmetic_ops.cpp
void foo() {
  int a;
  a = nullptr ^ nullptr; // expected-error{{invalid operands to binary expression}}
  a /= nullptr; // expected-error{{invalid operands to binary expression}}
}
````
````
Checking test/SemaCXX/nullptr_in_arithmetic_ops.cpp ...
lib/valueflow.cpp:1694:42: runtime error: division by zero
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior lib/valueflow.cpp:1694:42 in
ASAN:DEADLYSIGNAL
=================================================================
==13209==ERROR: AddressSanitizer: FPE on unknown address 0x000001296538 (pc 0x000001296538 bp 0x7fff512d2670 sp 0x7fff512ced60 T0)
    #0 0x1296537 in valueFlowForward(Token*, Token const*, Variable const*, unsigned int, std::__cxx11::list<ValueFlow::Value, std::allocator<ValueFlow::Value> >, bool, bool, TokenList*, ErrorLogger*, Settings const*) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/valueflow.cpp:1694:42
    #1 0x125c811 in valueFlowAfterAssign(TokenList*, SymbolDatabase*, ErrorLogger*, Settings const*) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/valueflow.cpp:2083:13
    #2 0x124dbe0 in ValueFlow::setValues(TokenList*, SymbolDatabase*, ErrorLogger*, Settings const*) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/valueflow.cpp:3075:5
    #3 0x110d25b in Tokenizer::simplifyTokens1(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/tokenize.cpp:1768:5
    #4 0xccd903 in CppCheck::processFile(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::istream&) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/cppcheck.cpp:356:37
    #5 0xcbd488 in CppCheck::check(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) /home/matthias/vcs/github/cppcheck_llvm_debug/lib/cppcheck.cpp:83:12
    #6 0x12eeafa in CppCheckExecutor::check_internal(CppCheck&, int, char const* const*) /home/matthias/vcs/github/cppcheck_llvm_debug/cli/cppcheckexecutor.cpp:863:41
    #7 0x12eadd5 in CppCheckExecutor::check(int, char const* const*) /home/matthias/vcs/github/cppcheck_llvm_debug/cli/cppcheckexecutor.cpp:196:16
    #8 0x130bb96 in main /home/matthias/vcs/github/cppcheck_llvm_debug/cli/main.cpp:136:21
    #9 0x7f0603bf2400 in __libc_start_main (/lib64/libc.so.6+0x20400)
    #10 0x7428a9 in _start (/home/matthias/vcs/github/cppcheck_llvm_debug/cppcheck+0x7428a9
````


Use priority que, which has always the command with the max_priority at the top. Complexity of STL implementation is at most `lg(n)` for exctraction and `lg(n)` for insertion whereas sorting takes `n*lg(n)` at best so if you repeat sorting during every extraction (your use of `startWork()` function where you use the sort only once is flawed, it mustn't be blocking - it blocks whole application until all commands are finishded).


Still fixing build errors


out_subformat should be optional, defaulting to None.


tea


accepted


Yes that's right I'll make that change this evening and merge that PR 👍


TODO: don't return and pass it to drive shaft instead?
NOTE: still could return for stats and such


Its need to be in release already. I hope this will happen very soon.


> usually any *NIX terminal when you open it by default is set as 80 characters

So you work in your default terminal and don't move it around? 😅

The situation where I only have 80 characters width available are rare. And in such situations, I can still device whether to wrap text or not.

> reviewers be the judge when to break that in favor of shorter lines.

Love it! We should make sure to always create PRs to change code on master. That increases the visibility and would outline too long lines.

> i strongly prefer to reduce indentation over reducing line-length.

👍 , and I would even go one step further and look at the method complexity as well.


`main.py` is better


This looks like that it will cause issues with config items that are not of type string. However, testing with startup doesn't show any errors.


There were the following issues with your Pull Request

  * A "License" and a "Signed-off-by" trailers are required. See [the guidelines about this](https://github.com/ipfs/community/blob/master/contribution-guidelines.md#a-license-and-a-signed-off-by-trailers-are-required)

We ask for a few features in the commit message for Open Source licensing hygiene and commit message clarity.
`git commit --amend` can often help you quickly improve the commit message.
[Guidelines](https://github.com/ipfs/community/blob/master/contribution-guidelines.md#commit-messages) and [a script](https://github.com/ipfs/community/blob/master/dev/hooks/setup_commit_msg_hook.sh) are available to help in the long run.
Your feedback on GitCop is welcome on [this issue](https://github.com/ipfs/infrastructure/issues/23).

---
This message was auto-generated by https://gitcop.com


self.stomach_filling = 0.0


`self.stomach_filling_upper_limit` is better


Напрягись и попробуй всё же ещё раз переосмыслить это условие


Consider creating class constant for 90 to avoid magic numbers: http://bit.ly/WAtpmT


Great! Thanks a lot

- sent from my mobile device

On 22 Aug 2017 12:36, "Jeremy Poulter" <notifications@github.com> wrote:

> Done now
>
> On Aug 20, 2017 12:46 AM, "Glyn Hudson" <notifications@github.com> wrote:
>
> > Did you make the change? Sorry I can't see the commit
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/OpenEVSE/ESP8266_WiFi_v2.x/commit/
> 96c633d0d88bf1106f256627d2c9ab45bb76e411#commitcomment-23747239>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-
> auth/AF1WgFNKhO9v0RUEKbEe-Q7-d61fhStAks5sZ3PngaJpZM4O5ySH>
> > .
> >
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/OpenEVSE/ESP8266_WiFi_v2.x/commit/96c633d0d88bf1106f256627d2c9ab45bb76e411#commitcomment-23792118>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAuUPKt5XXXmRV7Tdm6IM7hINS8D6B-1ks5sar1XgaJpZM4O5ySH>
> .
>


Great! Thanks a lot

- sent from my mobile device

On 22 Aug 2017 12:36, "Jeremy Poulter" <notifications@github.com> wrote:

Done now

On Aug 20, 2017 12:46 AM, "Glyn Hudson" <notifications@github.com> wrote:

> Did you make the change? Sorry I can't see the commit
>
> —
> You are receiving this because you commented.

> Reply to this email directly, view it on GitHub
> <https://github.com/OpenEVSE/ESP8266_WiFi_v2.x/commit/
96c633d0d88bf1106f256627d2c9ab45bb76e411#commitcomment-23747239>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-
auth/AF1WgFNKhO9v0RUEKbEe-Q7-d61fhStAks5sZ3PngaJpZM4O5ySH>

> .
>

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/OpenEVSE/ESP8266_WiFi_v2.x/commit/96c633d0d88bf1106f256627d2c9ab45bb76e411#commitcomment-23792118>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAuUPKt5XXXmRV7Tdm6IM7hINS8D6B-1ks5sar1XgaJpZM4O5ySH>
.


cip cip bales cip


This faulty query can be done by user ? If not : we must log it as error but not to user ? No ?


Am I wrong in thinking that we should click the button twice here? Seems like that's important.


Yes, if you get it from github, it will compile but it produces a lot of warnings.  I'm going through it and trying to resolve all of those and will try to submit a pull request within the next day or two.


Moreover, if you use Fibonacci heap as underlying data structure of `priority que` ADT, you get `Theta(1)` amortized for insertion.


Looks a bit redundant. This method already unnecessarily duplicates some code. I'd stick with the helper method builidng the qemu-img command.


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - i386-Windows-7-VisualStudio-14: OK (2599 of 2606 tests passed, 0 test warnings)

Failures:

EnergyPlusFixture Test Summary
 * Passed: 514
 * Failed: 4

integration Test Summary
 * Passed: 605
 * Failed: 3

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


Perhaps we should add a unit test to make sure things still work with jQuery. Or do the current tests handle this case?


Improvement suggestion: you can check stomach upper limit exceeding _before_ adding human weight to avoid Godzilla outweigh.


It's not =/ we can go two ways about this, typehint the objects or ignore the mock object error message. Either way if fine for me.


So we're changing the method signature. I know it makes more sense to have it like this rather than appending the new arg at the end of the args list, but in this case we have to make sure we're not breaking existing calls.


because phpstan was complaining that this typehint conflicts with the interface implementation.


accepted


Unnecessary extra line.


There were the following issues with your Pull Request

  * A "License" and a "Signed-off-by" trailers are required. See [the guidelines about this](https://github.com/ipfs/community/blob/master/contribution-guidelines.md#a-license-and-a-signed-off-by-trailers-are-required)

We ask for a few features in the commit message for Open Source licensing hygiene and commit message clarity.
`git commit --amend` can often help you quickly improve the commit message.
[Guidelines](https://github.com/ipfs/community/blob/master/contribution-guidelines.md#commit-messages) and [a script](https://github.com/ipfs/community/blob/master/dev/hooks/setup_commit_msg_hook.sh) are available to help in the long run.
Your feedback on GitCop is welcome on [this issue](https://github.com/ipfs/infrastructure/issues/23).

---
This message was auto-generated by https://gitcop.com


1-6, 8: accepted


Sounds good!


It happened for one of our LimeService customers, but I didn't track down the error completely. Maybe I should export his database and run it locally, to get more information. This was a quick fix to not show "Exception!" to the user.


We may want to check the '_get_vhd_type' call, validating its arguments.


My vote is show the error to the user, because then he/she can report it. Otherwise we'd never find out.


Identation


Fix for #250


Can you justify this change? Now `buildbench.sh` scripts builds everything. What's the purpose of this script then? We already have a script to build everything: `scripts/build/cardano.sh`. It has multiple features.
My understanding of `buildbench.sh` was that its goal is to build only what is necessary for benchmarks. That's what the name suggests. The intended usage is when you want to run benchmarks and don't want to waste time on building unnecessary stuff.
> CI builds these, so it's important to know that they pass before pushing
a patch.
If you need to know that they pass, why do you use `buildbench.sh` instead of `scripts/build/cardano.sh`?


Should do however I didn't test that when fixing this, the following lines after 129 account for this flow anyway:
```
     // Let's show the hidden tabs.
     for (let object of containerState.hiddenTabs) { // eslint-disable-line prefer-const
       browser.tabs.create(object.url || DEFAULT_TAB, {
         windowId: window.id,
         cookieStoreId: options.cookieStoreId
       });
}
```

Will check with the other work we are doing for this.


You can add a unit test for that sure but you already have the answer it still work with jQuery because in our units tests because we call `.alert` on jQuery objects


Released in [hail-0.2.0.0](https://hackage.haskell.org/package/hail-0.2.0.0)


Seeing this, would it be an idea (for future release, not now) to have system where at each "serious"(to be defined) error message, there is a button: report this and then a report with some diagnostics is sent to ollehar?


Why are we waiting for the response?


Done


This function is not needed as Bc::round always rounding up


this seems to be a typo


test comment 1503485327839


test comment 1503485331575


test comment 1503485332218


Also you're merging list[0] that is newTask  with itself


test comment 1503485356686


test comment 1503485360199


test comment 1503485360844


Do not use single startWork() function. Use asynchronous approach. Whenever command comes send to the serial if there is no command running or insert it in the command queue. Whenever a command is finished a command with maximal priority should be extracted from the queue. Do not forget to control if some command is not called more times (during the execution of some command, user can perform the same command or require to perform command which is already in the queue. If it is write command, its value should be replaced or merged with new command, if it is read command it should be discarded).


Hi, I couldn't find where container component `connect` to state...
Could you let me know how to connect them...?


Bunu degistirmek gerekiyor. Bu dosyalarda birden fazla degisken olabilmeli


Yes, that is a great idea!


Great, thank you


Bu da env dosyasindan gelmeli ve production da false olmali uzerindeki uyariya bak :)


dinamik olmali


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-MacOS-10.9-clang: OK (2594 of 2600 tests passed, 597 test warnings)

Messages:

 * 600 tests had: AUD diffs.
 * 600 tests had: EIO diffs.
 * 589 tests had: RDD diffs.
 * 3 tests had: Table big diffs.

Failures:

integration Test Summary
 * Passed: 602
 * Failed: 3

regression Test Summary
 * Passed: 600
 * Failed: 3

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


I can make a user interface for it, but you may have to code it yourself...


db.sqlite 3 dosyasi repo da olmamali


I'm not against adding a try/catch :) ( still dislike adding `$sql` like this, but it's another part ;).

But i think it must be admin optionnable to see to user or not :)


Done


Can this description be improved further anyhow?


PS : needed https://github.com/TonisOrmisson/yii1-querybuilder and remove the actual filter system :) . No ?


Merged, thank you for correction.


> So you work in your default terminal and don't move it around?

Yes I don't move my terminal around, I use i3wm and I don't need to resize my window because mostly code that I work respect the 80 characters, from FreeBSD ports to FreeBSD kernel, all respect 80 characters.

There is nothing more annoying than need to resize a terminal to be able to read a source code.


For the processing of relay responses, use asynchronous approach - connect `QSerialPort::readyRead()` signal to some slot in your class. This slot should decide according to `lastCommand` variable and the content of the message from the serial port how to treat the response (if it is a direct user interaction with the relay or if it is the response for the command which serves as a communication failure control).


@olleharstedt : you don't have a specific config system to log all error to a specific file in limeservice ? I think we have a feature request about logging error by default somewhere admin user can see it .


IMHO iocage is doing very good respecting the 80 characters, and brake it now, doesn't make any sense, at least for me.


raise(SIGKILL) is the only way to make an image fail on purpose. It is important for testing the failed images implementation. When you go without it, don't expect failed images to work as desired. When you move it into the ifdef block, you get different fail behaviour for programs with MPI-impls. that support failed images and ones w/o.


The order of `completedTaskControl()` and `qDebug()` should be reversed.


Will this text be comment or configuration?


These texts are comments.


What will this do?


These textboxes represent the commenter's text of the line directly above it.


Mozda je bolji naziv za ovaj metod printData() posto, sustinski samo vraca string reprezentaciju zivotinje (mada, najrelanije bi bilo da ovo override-uje samo toString() metod :) )


Prve dve naredbe si vec implementirao kroz remove() metod, iskoristi njegov poziv, izbegavaj duplikaciju koda :)


Probably a moot point based on the issue I have raised with this try catch wrapping. Is there a dot missing in Exception constructor call?


accepted, но есть 2 момента для улучшения:

1. `random_selection` можно упростить:
```
def random_selection(numb):
    return random.choice(code_table[numb])
```

2. цикл по обязательным символам можно записать проще:
```
for i in code_table:      # for obligatory symbols
    password.append(random_selection(i))
```
кроме того это будет правильней, т.к. в текущей версии `i` пробегает значения от 1 до 2, а не до 3.


Forgot the migration file, again


Hi NABEEL-AHMED-JAMIL.
I remove the hard code as you say. one think inform to you kindly first
complete the relation model then later added the login and other security.
because we have no idea about the front end view so that why first complete
the then we create the mock-up view for the front-end and the ionic-app

On Tue, Aug 22, 2017 at 5:50 PM, Nabeel Ahemd <notifications@github.com>
wrote:

> remove hard code
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/NABEEL-AHMED-JAMIL/AQA/commit/4f84b8a8b0340d33ddb349e4a867e3279710a116#commitcomment-23793476>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AdzD3jxisfo-GbIBD24ylp5APd2DqNQgks5sas59gaJpZM4O-kh8>
> .
>


автопрефиксер точно нужен?


delete the teat file


необязательно, у тебя же все подгружается в текущей папке через require_tree .


I try that then :)


лучше так: TestApp - <%= yield :title %>


Ok


same as dawei..


@elektroschule Bei 9-Bit, um einen Wert darzustellen, ergäben sich doch 2^9=512 mögliche Werte und nicht 257, oder? Zudem wird dieses zusätzliche Bit in der Methode `MCP4151.set_wiper()` nicht genutzt. Hier könnte ein Assertion helfen:

    assert value 0 <= value <= 255

Oder die Bits müssen etwas geschoben werden.


@davidsaOpenu I would like to follow-up on this thread:

compilation error comments: **https://github.com/davidsaOpenu/simulator/commit**/7d04be2bf4ba7c163c8af0ffd1b5a218d5cbe695

If I'm correct eVSSIM is forked from **https://github.com/nvmeqemu/nvmeqemu**
I tried to clone that too and ended up in same error (compilation error comments). This branch was created **6 years ago.**

Another fork is this:
**https://github.com/OpenChannelSSD/qemu-nvme**
This was maintained until **3 years ago**. I cloned this too and compiled and it worked well.

Is there any quick fix for this?


readme


Краще
Student current;
while (iterator.hasNext()) {
 -			current = iterator.next();
 -			if (current.getCourse().equals(course)) {
 -				System.out.println(current);
 -			}

щоб не плодити не потрібні обєкти студентів


I'll work on this tomorrow. Wanna change a couple of the GUI layouts etc

(Might make them like [TextDisplayer](https://github.com/boomboompower/TextDisplayer/))


No more updates for bleeding edge?


ми можемо використовувати перший сканер, нам не потрібно ще один


Another dereference without validation bug


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2634 of 2640 tests passed, 601 test warnings)

Messages:

 * 604 tests had: AUD diffs.
 * 604 tests had: EIO diffs.
 * 593 tests had: RDD diffs.
 * 3 tests had: Table big diffs.

Failures:

integration Test Summary
 * Passed: 605
 * Failed: 3

regression Test Summary
 * Passed: 620
 * Failed: 3

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


@m-stein: please add to your testing branch


BTW I find this a nice way to test precedence. I wonder if we have more precedence tests. If not, this can be reused for that.


Hi MEDICSi
I have complete the front-end app material. know will soon I will create the mock-up view and share with you


Привет!
Несколько советов:
1. Для решения первого вопроса стоит использовать команду [git log](https://git-scm.com/docs/git-log), которая позволяет посмотреть историю изменения не только всего нашего проекта, но и отдельного файла. [git blame](https://git-scm.com/book/ru/v1/%D0%98%D0%BD%D1%81%D1%82%D1%80%D1%83%D0%BC%D0%B5%D0%BD%D1%82%D1%8B-Git-%D0%9E%D1%82%D0%BB%D0%B0%D0%B4%D0%BA%D0%B0-%D1%81-%D0%BF%D0%BE%D0%BC%D0%BE%D1%89%D1%8C%D1%8E-Git) позволяет, посмотреть, кто в последний раз менял строки файла, поэтому это не подходит для решения первой задачи :)
2. Тоже самое: `git blame` решает другую задачу, тут нужно использовать `git log`
3. А вот тут как раз нужен git blame :)
4. все верно 👍


Done in 3ec9d1af3


That was fast :-)


a


Hello,

I got a question related to "knn" and would appreciate if someone could help or give an advice.
When I try to use knn-method, I get that 'train' and 'class' have different lengths.
From the very beginning, I thought that this is my mistake (kind of "typo"). Hence, I decided to copy the come from the book and from this source. Both are leading to the same mistake. I do not see what is wrong here or where the problem may be.

Thanks to everyone for the time and help.


@snicolet
Exactly the same was tested for two weeks.
http://tests.stockfishchess.org/tests/view/5988d2d10ebc5916ff64a462


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


cool


@ingbrzy just wondering shouldn't the file path be `extras/clock` instead of `extras/clocks` ?
Have I typoed it?


Mmhmhh


there are both actually.. we are using same translation for both..


Ah ok, so I leave it be like this. Thanks for the corrections btw.


See https://github.com/efluffy/gpdfand/pull/6 for this implemented with more fixes.


try removing the dot from here


`%OSTREE_REPO_LIST_REFS_EXT_EXCLUDE_REMOTES` — the `%` sign is gtk-doc syntax for ‘enum member’.


@ermau No it was already async. The change I made was going form `async void SetValue` to `async void SetValue<TValue>`


As @nop90 said, there appear to be stack issues in general use. It's not that compiling doesn't work.

Bear in mind that this repository is experimental and mainly here to illustrate compiling autotools based libraries. Several people seem to have forked and added libraries that probably aren't really suited for this environment in their current state. Many open source projects seem to have little concept of cross compiling.


This doesn’t need to be a `GPtrArray`, since its size is known beforehand.
```
const gchar *refs_dirs[] = { "refs/mirrors", "refs/remotes", NULL };
if (flags & OSTREE_REPO_LIST_REFS_EXT_EXCLUDE_REMOTES)
  refs_dirs[1] = NULL;
```


This doesn't print the pybind11 include path. It just prints the Python system include path.


グローバル変数が減って局所化が進められたのは大変めでたいんだけどその分煩雑にもなったし変数名の変更がまだ甘いから追加のリファクタリングは必要だなぁ。


有一个bug，搜索结果多选时，单击条目可正常选中，单击左侧的复选框无效果。


@gorhill While EasyList "tries" to block them it does not mean it is right. I do understand that the "_People You May Know_" feature appears frequently and it might be right to block it, but in an annoyance filter not the **main filter** as it is just that an annoyance.


Will this fix be available soon ?


add a dot here as well


What if the same value is used twice? Hmm I guess it doesn't matter if we confuse them :)


@fgsch
I would have preferred to return the hash as a BLOB and leave any encoding to https://code.uplex.de/uplex-varnish/libvmod-blobcode or other BLOB handling methods rather than serializing as HEX hard-coded.


Sorry for being pedantic, but can you please preserve the formatting of the changelog? Lines are wrapped at ~70, code references are single-quoted, R-tree, BigWig etc.


code duplication


Prve dve naredbe ti vec radi metod remove(), pozovi njega, nemoj da dupliras kod :)


U sustini, ne moras ni da prosledjujes animal - this.newAnimal ti je dostupno u ovom metodu, nad njim je i bind-ovano sve sa forme, samo njega ubacis u niz i resetujes na prazan objekat (sto vec i radis).


sshconfig


Three `==` in a single comparison! This is not very good for maintainability.


use operator_user and  operator_group


I think this can be a `const gchar *` and you can drop the `g_strdup()`s below.


Yes, will be deleted.


@igarashitm I think `org.apache.camel.component.jpa.JpaEndpoint#isSharedEntityManager` should be checked like in `org.apache.camel.component.jpa.JpaConsumer#doStart`:
```java
@Override
protected void doStart() throws Exception {
    // need to setup entity manager first
    if (getEndpoint().isSharedEntityManager()) {
        this.entityManager = SharedEntityManagerCreator.createSharedEntityManager(entityManagerFactory);
    } else {
        this.entityManager = entityManagerFactory.createEntityManager();
    }
    LOG.trace("Created EntityManager {} on {}", entityManager, this);
    super.doStart();
}

what do you think?
```


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - Win64-Windows-7-VisualStudio-14: OK (2599 of 2606 tests passed, 599 test warnings)

Messages:

 * 603 tests had: AUD diffs.
 * 603 tests had: EIO diffs.
 * 592 tests had: RDD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table small diffs.
 * 3 tests had: Table big diffs.

Failures:

integration Test Summary
 * Passed: 605
 * Failed: 3

regression Test Summary
 * Passed: 602
 * Failed: 4

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


Okay, was not completely sure about the values ;) thanks


- Fixed bug that caused weird resizing issues on mobile devices when changing viewing modes (portrait/landscape).


Fix for issue #240


Fix for issue #255


?
Strange system ;) readonly must be always 'read only' and readonly_when_active only whan active.
Some update somewhere at sometimes ?


Gcc didn't like it either, I added some more braces in a later commit. 4301e8c252


Ovde ti ng-container nije bitan - on samo sluzi da grupises nekoliko drugih elementata u celinu i nad njima zajedno (bez dodavanja novog elementa) primenis ngIf ili ngFor... Ovde mozes ngIf da stavis na <td> element.


It's a fix i found for the bug that is still open for jQuery UI.
https://bugs.jqueryui.com/ticket/9125
Our case uses IE11 but it's also in older versions.

Does this behaviour happen with all the datepicker types?
And in which browser version do you see this?

Kind regards, Marcus


my first php script


Prve dve naredbe si vec implementirao u remove() metodi, pozovi taj metod, izbegavaj dupliciranje koda.


Ne moras da prosledjujes newNanimal, on ti je vec dostupan kao this.newAnimal, posto se i to polje i taj metod nalaze u samoj klasi komponente :)


s/indexName/encodedIndexName


I managed to push with a typo .... shoudl be setURL


sahil


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1392 of 1392 tests passed, 0 test warnings)

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


fixed


Det var med vilje, men tilføjer det igen, bliver også logget på js siden.  Hvis det spammer loggen må vi slå det fra.


Who the hell gave you permission to scrape my website?


nice, useful for mini game servers to split the users on one server into several groups


Aren't you supposed to use this approach here?

` var self = this;
      context.res.on('finish', function () {
        self.exit(context);
      });`


good


@Hrxn If you look at the previous commit you will find that it was added due to @gorhill interpreting the joke as some sort of anti adblock mechanism.

I found it funny for two reasons, first being that anti adblock measures are being used so widely that even a little joke that impeded some sort of page functionality made the developer believe it was an anti adblock measure.

The second being the joke itself by **The Guardian**. The article title reads "**_How to tell if you damaged your eyes during the eclipse_**" and after a few seconds of page load the article page turns blurry (this was interpreted as anti adb) jokingly implying that yes you did damage your eyes.
And if you read the article (can do so by adding the removed filters above) you see that the article is a joke in itself. The introductory paragraphs quote an unnamed doctor saying about the damages of staring at the eclipse as "_**If you can’t read this piece, then ...**_" (referring to the fact that they haven't made it possible to do so) and the rest of the article contains the lyrics of the song "**Total Eclipse of the Heart**" by Bonnie Tyler.


Er ikke enig, man vinder ikke noget ved at erklære dem som final i toppen, medmindre de skal bruges på tværs af metoder. En streng literal er konstant, så den kan ikke muteres uanset hvad man gør.


Changed in next commit


its all awkwardly done because I was trying to get all the pages done before results day, and I was trying to get the slideshow on the right of the sports page but then appear on the left on the football page.


「～built-in type names, such as `int` or `string`, are not treated as type parameters.」とか。


Fix for #249


You're right -- I'll put up a new PR shortly to address this


This produces the PHP notice "Undefined index: locale in .../htdocs/lib/common.inc.php on line 589", if $_REQUEST['locale'] is not set.


LocalNodeNetwork (XuanLuoLBL) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (2173 of 2783 tests passed, 0 test warnings)

 * 610 tests had: AUD diffs.
 * 3 tests had: Table big diffs.

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


[R/onLoad.R:2:13:](https://github.com/steinbaugh/basejump/blob/6ca248e0ec350fc155e38bfab00c2e203bd7199e/R/onLoad.R#L2) *warning:* **Unneded concatenation of a constant. Remove the "c" call.**
```r
    pkgs <- c("annotables")
            ^
```


Jetzt sind es aber keine 15 min mehr?


ja, habe ich auch gerade gemerkt.. lade gerade die nächste version hoch...


The .X syntax is a new feature in Lark that allows giving priority to rules, as I understand it the rule names remain the same,


happy first day with github


jfmfh


@lsala the branch has been merged into develop! I will create yet another branch with my latest developments which should bring big performance improvements specially during assembly both integration and condensation


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/844f68ac-0e27-456c-b51d-3af8584b4a16 "simple-toolchain-20170823094933781"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/929ee60b-911b-4e58-81fe-a4812710ef40 "simple-toolchain-20170823094933781") deployed [simple-toolchain-20170823094933781](https://github.com/ranjitdn/simple-toolchain-20170823094933781/releases/tag/deploy-ibm-yp-us-south-ranjit.jha-in.ibm.com-cloudCertPrep-20170823-120024 "deploy-ibm-yp-us-south-ranjit.jha-in.ibm.com-cloudCertPrep-20170823-120024") to [cloudCertPrep](https://console.bluemix.net/apps/0ac4ed5a-4cc4-4f93-a1e8-c4a7f3eb4c4d?env_id=ibm:yp:us-south "ibm:yp:us-south:ranjit.jha@in.ibm.com:cloudCertPrep"), including this commit


is this up to date description?


Done.


این لاجیک نباید توی کنترلر باشه. باید توی
message.model
باشه.
مگه قرار نشد چیزهایی که مربوط به **یک** پیام هستن توی مدل باشن؟


looks like some ';' are missing here


Fix for issue #248


Yes, it failed on travis, because it uses an older compiler. I don't think we enable c++11 in all c++ sources, and I'm not sure the version in travis has (full) c++11. Might need to upgrade the toolchain.


Jessica's Assigments

Unit 1 Hello World


some wrong indentation?


یادت باشه هر موقع مرج کردی، این خط رو با تابع
`getMessageById`
که نوشتی، عوض کنی


@TheHollidayInn Did you see the code I put in https://github.com/HabitRPG/habitica/pull/8978/files ? Even though it's not working by itself, it contains some changes that are needed, for example about rewards and tasks versus task.


Do we really need a fallback path at all? If someone requested upscaling and it doesn't work, what's wrong with just dying? Especially given that since it needs a resolution set this isn't something we can ship a fix with enabled?

What sorts of things can go wrong - anything transient, or only user config errors?


@Fettabachi Hi Tim,

This isn't the right place for you to resolve this - I think you're going to have better luck with help if you use [Stack Overflow](https://stackoverflow.com) for this problem.


where this var is used?


again some indentation problems


مگه آی‌دی یه پیام قبل‌تر رو نگرفتی؟
چرا اسمش
`next`
داره؟


You can see in commit 223d42df34d5a6d9743c5cd1fddfa8b829547db0 how I preceived the usage of this feature by scripts. The advantage of my approach is that cut scripts can pull in multiple "library" scripts  easily.
But I guess that, in the short term at least, it wouldn't hurt to pull in the whole "rapido library" in every initrd anyway because it'd hardly be more than a few kB. That way we wouldn't need any code in the cut scripts, we'd just need to source the library files in the autorun script.
What do you think?


Although I only found combinations which have a printed part you're probably right. Deleting this part.


Tässä tuli nyt lisättyä #7 ongelmaan liittyen käännökset.


Ok


> There is nothing more annoying than need to resize a terminal to be able to read a source code.

@araujobsd you forgot the crucial qualifier "to me".

so, to me, nothing is more annoying than reformatting code: reason (an ocaml) and go and most lisps do the right thing by autoformatting it for you. recently someone wrote prettier for Javascript http://jlongster.com/A-Prettier-Formatter

unfortunately, as mentioned already, yapf does not work for python 3.6

worse only (to me) than reformatting code, it's reformatting it to fit an 1960 teletype and punch card. all this on a machine that's millions of times more powerful. I'm writing this on my PHONE! WHILE SITTING IN AN AIRPLANE!!


That should be happening. If it's not, can you please cite a URL?


it's better use more informative var names: `student`


it's better use more informative var names: `hw_result`

the same for below code.


- [ ] dfdf


Seems to fix the issue for responsive arrows for my situation


who are you


Looks good!


you can simplify this expression with ternary operator:
```
group.sort(key=lambda group: group.get(sort_by_key), reverse=(False if sort_by_key != "rank" else True))
```


"to me" and other thousand *NIX developers!
Standards are there to be followed.


Komentar


А игде сам шаблон? )


Looks good.


Another thing, we are talking about screen size and not how powerful is your CPU or your PHONE. ;)


This change cost me hours of debugging, In our team we have case just like maxlapko's case mentioned above.


- [ ] Naamgeving / conventies
- [ ] Commentaar
- [ ] Recursie
- [ ] Flar


what


but please remove `@chelmuth` before pushing ;-)


@locutus2 oops, thanks :-)


I would prefer to check `fil_crypt_threads_inited` instead.
Add a source code comment that explains when exactly the subsystem has not been started.
I would also prefer some debug assertion, such as `ut_ad(srv_is_being_started || !srv_read_only_mode)`.
It is possible that we can get here even if `!srv_is_being_started`, especially if purge threads can call this function.


Releases are cheap! :grinning:


Good improvement for my vnpc.inc


Ok. Would you mind adding these comments to the relevant issues? I'm afraid they're going to get lost on individual commits.


missing newline at end of file


uhhh nice stuff 👍


a test


you could pass the entire `user`: ctx.client.asUser(user) - IIRC we filter this / it's a convenience shortut


Are we really going to support russian ?


`@@retval` is not valid Doxygen notation.
`@retval` should occur after `@return`.

The name and the comment of the parameter `undo_tablespace` are not descriptive. It seems that the purpose of the parameter is to skip some steps.

I think it would be clearer to refactor those optional steps into a separate function, which would be called when needed.


@grgrzybek ah it looks like so. +1


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I don't think it's possible to have zero completetion buffer.


Дима, знаешь что я тут не пойму. У тебя есть описание параметра isHidden при этом у тебя в самой функции этого параметра нет.


Thank you, the logic looks OK now. The only thing that I would request is to split the function `fil_read_first_page()` into two parts. The first part should be invoked for all files except the undo tablespace, and the second part should be invoked for all tablespaces.


1. Да, так немного больше получается))
[temox@git zabbixapi]$ git log  | grep Date: | wc -l
417

2. В 2012ом?
[temox@git zabbixapi]$ git log -p Gemfile | grep Date:
Date:   Wed Apr 19 15:03:47 2017 -0500
Date:   Wed Jan 11 09:26:13 2017 -0600
Date:   Mon Oct 31 13:31:23 2016 -0500
Date:   Tue Dec 11 21:40:13 2012 +0100
Date:   Wed Nov 21 12:04:40 2012 +0400
Date:   Mon Nov 19 00:15:31 2012 +0400
Date:   Mon Nov 19 00:12:57 2012 +0400
Date:   Wed Oct 10 00:47:52 2012 +0400

3. Не совсем понимаю, как blame'ом искать изменения в функции?


this commit seems to be add regression when adding a new host : #1101


It's clearly described in update notes: https://github.com/yiisoft/yii/blob/master/UPGRADE#L47


Absolutely - thx for the clarification


fin


The decision is not mine, however as it is a python project, please follow at least PEP8 :), flake8, pylint and other static analysis tools are your friends too.


重构为maven项目


Yep, I noticed that only when I was told that the framework has been upgraded


Maybe the one in `README.md` should also be updated?


ac


Is the translation on ygorg wrong?
It is translated as a mandatory part of the effect to activate the field spell after returning the other to the hand.


what is this great thing?


Also tweaked polls and changed splash screen for iOS.


Why you comment "//"use-msi", 1,"?


See single version at [Dranggo's Code](http://paste.ubuntu.com/25376205/)

Problem Link:[POJ](http://poj.org/problem?id=2559) , [DBSDFZ](http://s.billchen.bid:8080/problem.php?cid=1400&pid=7)


![](https://github.trello.services/images/mini-trello-icon.png) [[1] create/edit event: allow dots/commas in stat weights](https://trello.com/c/pYwc6H1Y/304-1-create-edit-event-allow-dots-commas-in-stat-weights)


I looked into MAB 437. Most of it is additional material like "Anhang" so I'm going to remove it.


Should be self._output_device is None (none should be compared with is)


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 388](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1656&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


It's probably better to set this at priority 2, so it has the same priority as local outputs. All machine tied output devices are at priority 2, and they should get preference over this device.


Continuing to allow cut scripts to decide which library scripts they want to pull in makes sense.
E.g. cut_something.sh:
```
"$DRACUT"  --install "..." \
                     --include "${RAPIDO_DIR}/libs/<file1>" "/usr/lib/rapido/" \
                     --include "${RAPIDO_DIR}/libs/<file2>" "/usr/lib/rapido/" \
                     ...
```
or if the cut script wants to pull in everything:
```
"$DRACUT"  --install "..." \
                     --include "${RAPIDO_DIR}/libs/" "/usr/lib/rapido/" \
                     ...
```

The autorun usage would continue to match what you've done with https://github.com/mwilck/rapido/commit/223d42df34d5a6d9743c5cd1fddfa8b829547db0


Max Thread Exception

Here can make an exception with the thread count limitaiton with `threading` module.
And functional modifying with the list arranging method.


max thread exception


new list arranging method


Olá Mundo.!


Dict load : modified (with numeric count)


@rzats I've fixed all compile errors


Thanks Josh!


use "is None"


remote and local on same line


sfetsgrsgsgd


Oh damn nice catch!


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 388](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1656&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:01:57


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 80](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1657&buildTypeId=XbimEssentials_XbimEssentials) is now running


Hybrid_Unitary_Air_Conditioner_I6091 (spencerdutton) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1996 of 2000 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 604
 * Failed: 4

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.8%25-yellow.png) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.54%25-green.png)


use == instead of is. Note that you can also use "if job_name:" (empty string are also evaluated to false in python.


WHy do this? The multi-part message is already sent (as this happens when doing the post in line 150)


Sadly, if `overwrite` is on, this truncates the file before writing out each line.


No need to cast the exception to e. You can just use Logger.logException("e", "An exception occured during g-code upload"). This will print the exception and your message.


I see Product and Serializable are new parents. I tried to remove them but hit on other cryptic errors. I give up, this is beyond me.


Yep


yup, looks like there is


Why is there no Git tag on this @kohsuke? CC @olivergondza


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 80](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1657&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:03:04


Hello,

I recently installed Prestashop version 1.7, however I have a problem loading pages from my backoffice except the Module page.

Indeed, when I browse my back-office pages partially loads and the ajax load is always in infinity loading.

Can you help me solve my problem

I thank you in advance


@vakulich Hi, I was exploring this code and I found that this class is absolutely useless.
This class duplicates the `list` class functionality and you could use an inheritance from the `list` class to just use builtin `append` and `pop` methods. But simplifying this class become you understand that you could remove it at all , because instead of using `peek` method, you could just call `self.parent_types[-1]`, and instead of using `is_empty` method, you could just use `empty_flag = len(self.parent_types) == 0`.

I conclude that it is better remove this class at all.

FYI: @krasoffski


OK. I think I've come up with a saner introspection technique that should be more robust for ensuring `SIGKILL` is present, so let's just leave this as is. Sorry for the noise, and misguided proposed approach.


This code suggests that you expect `resources` to also include ignored files (and consequently even ignored files would be deleted by "Discard All Changes"). While I personally would consider that a bad idea, I also think that's not the case: `resources` seems to be populated by a call to `git status -z -u` from https://github.com/Microsoft/vscode/blob/87aa00a849ab1389a547728d103bcea049f3b41c/extensions/git/src/git.ts#L886 and `git status -z -u` doesn't include ignored files unless the option `--ignored` is also specified.


@lsala you probably need to merge develop into the hdg branch to be up to date with hdg work


Never mind, there is, GitHub just seems to be failing to calculate it properly.


I think you mixed up the tag names for the two calls to `localize`...


There's another one that is made for that. I'll look it up


> https://twitter.com/YuGiOh_OCG_INFO/status/900295299947651073
> http://yugioh-wiki.net/index.php?%A1%D4%A5%B4%A1%BC%A5%B9%A5%C8%A5%EA%A5%C3%A5%AF%A1%A6%A5%EA%A5%D5%A5%A9%A1%BC%A5%E0%A1%D5
> 通常罠
> このカード名の(2)の効果は１ターンに１度しか使用できない。
> (1)：自分フィールドの「ゴーストリック」フィールド魔法カード１枚を対象として発動できる。
> そのカードを持ち主の手札に戻す。
> その後、手札・デッキからフィールド魔法カード１枚を**発動できる**。
> (2)：墓地のこのカードを除外し、自分フィールドの
> 「ゴーストリック」Ｘモンスター１体を対象として発動できる。
> その自分のモンスターとカード名が異なる「ゴーストリック」Ｘモンスター１体を、
> 対象のモンスターの上に重ねてＸ召喚扱いとしてＥＸデッキから特殊召喚する。


Sorry for the delay, but finally my installation is working again properly...

I think I can't handle that exactly without doing something not that efficient as you could, I'm more the php guy...

But I implemented that it doesn't touch any completion providers when the relevant config didn't changed at all.


Note that this will only work if the task is visible.
This will break their position when resuming in IV and CP without having the task open.

Is this fix only for auto-align? Could we update`element.position` when auto-aligning?


可以考虑一个叫priority queue 的东西


Oh ty, i missed this: https://github.com/PureNexusProject-Mod/android_system_core/commit/3b6881b7545e4796b82e83a5e76e2093dbdcc164


Sass


Eee


Fixed in ba4b579


That makes sense, however why would we remove a test for it?


نايس


ماذا رأيكم


This looks good for taking snapshot, what about when deleting them?


Oh that's actually exactly what is happening. `DOM.select.elements()` emits twice and at each emission we click the button. But I admit it is not really easy to understand
Currently the first click works (we get the "message" event) but the second is never emitted (meaning that the second time the button didn't have the listeners bound)


There should be a space between the if and the opening parenthesis.


@bacek, thanks for pointing this out.  Let me get a fix.


waarom? zou je toch moeten compileren op je server ?


also i think self.resolve_backupable_id needs to be

`    def resolve_backupable_id(self, resource):
        return resource.get("DBClusterIdentifier", "DBInstanceIdentifier")`


I think the COP-cutoff is just relevant for HHP's. They switch to gas if the COP drops below the cutoff value. The other heat pumps don't have an other option.
notifying @ChaelKruip


Comments are generally discouraged by the coding style guidelines.


`pulsectl` controls PulseAudio volumes from Emacs, via `pactl`: [flexibeast/pulsectl on GitHub](https://github.com/flexibeast/pulsectl).

i am the maintainer.


 this patch is working for the latest LEDE checkout. Thank you!
0.20.8 has compile problems, at least in my environment.


Question (1) still remains open.


Oh sorry, that was just a test, you can remove it ;)


Hi Ilan,
We able to connect instance http://14.142.119.130:5000 .
Could you please suggest further steps to get service from API end point http://14.142.119.130:5000/redfish/v1/get_json_service

![screenshot from 2017-08-23](https://user-images.githubusercontent.com/9120829/29616892-8e0af886-8830-11e7-8b1f-a3a904826b76.png)

Thanks,
Sagar


omg such regex noob


Electron app link is broke


Should we change the message to "fastsum: Too many plans already allocated."?


servlet注册登录案例


@snicolet +1, let's trying to close the gap ;-)


Oooo what's up?


Its all good, this is pretty easy, dont worry ;)


还不怎么会用啊- -


@PositiveAlex It seems like you are right about removing this class for now.

Only one note not related to this class about inheritance from `list`.
The right approach is to inheritance from `UserList.UserList` for `Py2` or `collections.UserList` for `Py3` (the same for `UserDict` instead of `dict` and 'UserString` instead of `str`.

More info here: http://pypy.readthedocs.io/en/latest/cpython_differences.html#subclasses-of-built-in-types


package day01morning;
public class HelloWorld {
	public static void main(String[] abc){
		System.out.println("你好");
	}
}


hi哥们，这个代码几年前写的，用的utf8，一直内部用，前些天想弄成精简规范点放到外头，去掉了utf8的依赖，然后，是在公司里写的，写完想起公司开发机没法上外网，然后就直接在网页编辑了下，结果编辑错了好像。。。刚才提交了下，在家笔记本写代码也是蛋疼。。。你试下新提交的ok不，如果还有问题，我再改。后面闲了我再整理成精简版


package day01morning;
//单行注释，1对代码进行解释说明。2 对程序进行调试（把可能出错的代码注释掉）
public class HelloWorld {
	public static void main(String[] abc){
		System.out.println("你好");
	}
}


OK thanks. I will


tesekkurler hocam, size de iyi bayramlar @ozank


Inputs zijn ook te valideren via het `pattern` attribuut. Heb je daar misschien al naar gekeken of die oplossing ook voldoende was? In JS kan je dan alsnog checken of die input valideert met de Form Validation API


The upstream patch uses ```unsigned int``` but you are using ```int```. Any reason for using ```int``` instead? Does the value ever need to be negative?


somehow permission changed...download from repo, build, copy from usb?


@zbeekman maybe its feasible to look at the way autoconf is handling the lookup for raise or rather signal.h. At ftp://ftp.gnu.org/old-gnu/Manuals/autoconf-2.13/html_chapter/autoconf_4.html search for AC_DECL_SYS_SIGLIST. Maybe that's an inspiration.


`if(sha3(pwd1,pwd2) != withdrawCode) revert();` is sufficient and necessary for that.


De enige functie die binnen de event handler wordt aageroepen is de `init()`. Het handler argument van een event listener moet wel een functie zijn, maar omdat `init` ook een functie is, kan je dat herschrijven naar `document.addEventListener('DOMContentLoaded', init, false);`. Daarbij wordt `init` dus meteen aangroepen, ipv een anonymous function die weer een andere function aanroept. Doet precies hetzelfde, maar met minder code en minder functies :)


You can just put `n` after a numeral, rather than `:num`


Never use printStackTrace. You can inject a Logger and use this.logger.error() or warning() etc. depending on what you want to log exactly. See https://github.com/xwiki-contrib/mediawiki/blob/master/mediawiki-xml/src/main/java/org/xwiki/contrib/mediawiki/xml/internal/input/MediaWikiInputFilterStream.java#L601 for example.


timer added and done few correction


Hm, those `--include` lines are a bit verbose for my taste. Don't you think

```
_rt_require_rapido zram.sh nvme.sh
```
is a bit more friendly for the "user" (read: cut-script writer)?


Testkommentar


Je hebt hier te maken met een double negation. De naam van de variable is al een negation (invalid) en de waarde ook (false). Dit zou je kunnen herschrijven naar een `var valid = true` waarbij je `valid` alleen naar `false` zet wanneer de input niet matcht. Op die manier heb je imho een betere default value en naamgeving.


заимствовано


okay, got it :)


Yes you are right in this i'm going to revert this change and just change it in ```self.default = kwargs['default']```

The reasons in general behind this and other corrections included in this commit is that if of this kind that we want to enforce should be written eventually as an assertion so that they do not apply in production. implementing them as an if condition instead causes us to have to add a test case to cover such a possibility that would instead remain untested.

Anyhow for the cases evaluate in this commit in case of programming errors we will receive anyhow exceptions like the KeyError in this case so no need to add any check here.


+ dalsi drobne upravy


This code does not make much sense. You don't do anything with the input and you never close it.


ERROR: Line is longer than 100 characters (found 106).


FATAL: [MissingTranslation] "`organization_charts_feature_is_currently_only_available_on_the_web`" is not translated in "ja" (Japanese)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`you_are_unauthorized_to_access`" is not translated in "ja" (Japanese)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


오! immutability-helper를 쓰면 복사를 따로 안해도 되고 편하네요.


oh yeah, forgot to remove that. removing.


Shouldn't `static.surfconext.nl` be replaced with `static.{{ base_domain }}`??


Waarom gebruik je hier het click event van de submit button ipv het submit event formulier? Is dat met een bewuste reden gedaan?


Fixed by 4809d95


@tarsius done for https://github.com/aurelienbottazini/tronesque


А зачем, вроде как this на каждую функцию свой и они в такие же значения инитятся?


I don't know if 9999 is such a great idea since I think Dash Core is using this port for P2P communication.
Find a free/unregistered Port [Wikipedia](https://en.wikipedia.org/wiki/List_of_TCP_and_UDP_port_numbers#Registered_ports)


Since it seems like 'introtext_array' is supposed to be an array and not an object, I changed how introtext_array is initialized to match this assumption.


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/e28f9cf9-5c31-4058-9dc5-425f441d7298 "meghdoot-2-static"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/722e47cf-9199-4abd-875e-329889414dff "meghdoot-2-static") deployed [static-newsletter](https://github.com/shreyasmulay23/static-newsletter/releases/tag/deploy-ibm-yp-us-south-Meghdoot-dev-20170823-131915 "deploy-ibm-yp-us-south-Meghdoot-dev-20170823-131915") to [dev](https://console.bluemix.net/apps/39dc5c92-fcc7-46a7-9215-c741e2836c00?env_id=ibm:yp:us-south "ibm:yp:us-south:Meghdoot:dev"), including this commit


I know. This time I opted for `:num` because it is more descriptive for newcomers.


Dit zou misschien korter kunnen met een ternary statement:

`inputAll[i].style.borderColor = elInvalid ? 'red' : '#ccc';`


Of een class gebruiken ipv via JS het style attribuut aanpassen zou nog beter zijn ;) Op die manier hou je logica en styling gescheiden van elkaar.


汉诺塔


Side effect of the modification :

Before the modification the line 187 `rv = [cls(a*r, b, evaluate=False)], [], None` was equivalent to `rv = [b], [], None` if r==-1 and a == 1.


Hier zie je die double negation weer terug: "als het formulier niet invalid is" lees lastiger dan "als het formulier valide is".


@jkff , I noticed just now you had removed this ctor.
I was wondering if that was part of a clean up due to it being marked unused by the IDE within the Beam project scope?


Fixed in fcba62a


It might be helpful to add a note here about how to run the code, build mlpack and run bin/mlpack_test -t ... is just fine.


Move the filtering because we do not want to delete elements if  `evaluate=False`


Test comment


Updating test for `evaluate=False`


will cause an exception when extraHeaders is undefined.


Missing `extraHeaders` (object/shape?)


Actually we need both. See 91a5b6a4bc87ec0f183aa4a8ab99c3db2193af73


Why did the image file disappear? ...