@fpgamcu , I don't understand very well this commit. It seems that everything is done outside of the Engine and normally, the messages available in the parametrizacion.cpp should be processed by the Engine. So these ifs in order to detect if the command is 1; or 3; should not be here. Are you performing all the simulation here?


Las pruebas son de división y de estadísticas, no de resta


There were the following issues with your Pull Request

  * Commits must be in the following format: **%{type}(%{scope}): %{description}**

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


please disregard about Channels


Maybe re-write this in Go? ;)


Added badge.


Pipeline script


Thanks for noticing; yes that is indeed correct. I added it by accident to the first fixup and did not bother to create another one.


Hi @danibudi! Jessica from SurveyMonkey here.

I'm curious, why hard pin the requests version? Is there something specific in the latest version that you need to have here?

Asking because applications integrating with kount may already have a specific requests version set. This will cause a dependency conflict. Maybe open up the version range `requests>=2.11.1` or something?


中英文切換的按鍵在appbar那邊，一開始預設是中文，按一下會變成英文並顯示出toast告知使用者

![21103746_1583770548340531_1302036181_o](https://user-images.githubusercontent.com/19555724/29673682-8bc513a2-8922-11e7-9d19-66a378e0810e.jpg)


That is correct. So when you pass an ES6 class definition (or function definition for that matter) to an Angular component it creates a new instance of it. That is typical class syntax for a method. At run time you access them with the "this" keyword. In AngularJS components you access the instance of the controller using the "$ctrl"


Yeah if the gawk build system doesn't detect libsigsegv it falls back to just calling `signal(SIGSEGV, ...)`. And all it does in the handler (independent of whether libsigsegv is available or not) is to print "fatal error: internal error: stack overflow" and call `abort()`. Definitely no reason to have `libsigsegv` slowing down stdenv builds...


Is this release available for download anywhere? I can't find it on pypi nor in the release section here on github.


An attempt to add the url for the badge to the readme fild https://travis-ci.org/RobertHagan/DaysSinceWhen.svg?branch=master


Fix for issue #237.

The title of this commit should have been "Write correct output to log and stderr for fastq_chars"  (stderr, not stdout).


Elsewhere we use dirs/dirs.go for this sort of thing.


Looks ok to me


Does this replace the reinhard tonemapping, or is it additional? I would rename my checkbox... or replacing it by a combobox.


HotFix-AirterminalSizingDivByZero (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2784 of 2784 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)


That won't work if .Junk already exists. For cleanup, I think we want a script that goes through and renames or merges all .Spam into .Junk. I don't know that you  can do that later with a mailfilter .


Так это уже будут другие темы для стихов?


Yikes, thanks – fixed both issues in Transifex.


@daniellandau what was the purpose of this function? I'm trying to fix a bug and understand exactly what you were trying to do! :)


Near 24. If you want, you can check the duration limit isn't `0` and if it fails the test, the contract won't deploy.


While not currently used in our policy, we should maybe list rlimit here for completeness. rlimit is an old feature that was added in apparmor 2.3 (the same time the /sys/kernel/security/apparmor/features was added). I confirmed that Ubuntu 12.04 3.2 kernels have it and unpatched 4.9 Debian has it.


HotFix-AirterminalSizingDivByZero (mjwitte) - 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)


you might want to write:
```
// let parsedTicker = this.parseTicker (ticker); // this line is excessive :)
let symbol = this.markets_by_id[marketId]['symbol;
result[symbol] = this.parseTicker (ticker);
```


Is zero matches expected?


@Pro
Imho this is not the place to fix the error.
The variant contains a scalar. I don't see why the next else/if should not cover nodeclasses.


But don't worry, I'll help clean it up, if you submit)


There is a big loophole to close. This is non-obvious but important to understand.

The "secrets" are revealed to the whole world when the transaction is transmitted via gossip and before it's mined. Miners have the option to include or not include the transaction. That means they or someone else can race to issue another transaction containing the secret and try to steal the funds. Do you see how that window of opportunity is a problem?

You can fix that by assuming (forcing) the contract to know "Carol's" address and only let her be the one to withdraw funds.


I don't like this much. Many compilers can detect wrapped headers and avoid re-processing them when they are included multiple times, but this change will prevent that, requiring the entire 3000+ lines to be re-scanned every time. And having stuff outside the wrapper is just kinda gross. Is this problem that important? And if so, is there no other way to deal with it?


I suggest to remove the `&& defined $group_id` part of the if statement, because it's always defined now.


I'll add that! Thank you for noticing.


Create it


Make it a habit to add `public` to make your intention clear and return something.


1) Can confirm. Seems like a webkit issue to me (so nothing I can do about it)
Can you try disabling this line https://github.com/llelectronics/webcat/blob/master/qml/pages/FirstPage.qml#L361 on your installation and try again (I don't have a facebook account to test this).

2) Works fine for me with the videos. Maybe that IP is blocked. Try maybe another exit.
Note also that clicking on the play icon on the page itself does not work anymore. It opens an empty page for whatever reasons.
Clicking on the play button works however.
I cannot integrate youtube-dl as that would make webcat not going into harbour. I suggest using LLs VideoPlayer instead if you like to use youtube-dl. You can even set it as external player in the settings for webcat.

Would be nice if you could create an issue for the facebook thingy instead of writing it here in the comments.
Thanks :)


Hey Xavier, thanks for taking a look! I've adjusted the code based on the feedback for the other comments.

Here, when isActive = false, all users can still withdraw their funds, they just can't split anymore. I could add an "unkill" function to enable splitting again though


You can put it in its own file and `import`.


That's what `.transfer` would do.


You should update the state __before__ you talk to another untrusted contract.


You could get carried away and let this co-purchase contract interrogate the shopfront to find out what the current extended price is for a sku and qty instead of having this function ask for it as input.


a tua alteração causa um erro relativo a abertura de certos ficheiros causando array out of index. Por exemplo o trabalho final de asc.
java.lang.ArrayIndexOutOfBoundsException: -1
	at java.util.ArrayList.elementData(ArrayList.java:418)
	at java.util.ArrayList.get(ArrayList.java:431)
	at jasmin.core.Parser.parse(Parser.java:165)
	at jasmin.gui.SyntaxHighlighter.parseLine(SyntaxHighlighter.java:205)
	at jasmin.gui.SyntaxHighlighter.parseAllOnToDoList(SyntaxHighlighter.java:380)
	at jasmin.gui.SyntaxHighlighter.insertString(SyntaxHighlighter.java:124)
	at javax.swing.text.DefaultEditorKit.read(DefaultEditorKit.java:273)
	at javax.swing.JEditorPane.setText(JEditorPane.java:1416)
	at jasmin.gui.JasDocument.loadFile(JasDocument.java:349)
	at jasmin.gui.MainFrame.<init>(MainFrame.java:84)
	at jasmin.Main.main(Main.java:27)


The problem is pretty bad, because even if you know what you're running into, it's not always simple to find the arrangement of headers to prevent it from happening (especially because nobody includes `windows.h` directly intentionally, it usually comes from somewhere else indirectly). And, worse, many people have no idea about what is going on and are completely stumped by the mysterious linking errors they get (typically something like `GetClassInfoW() not found`).

As for another way to deal with it -- I don't see any. We could advise people to include all wx headers at the end but this is going to be confusing because we used to advise exactly the converse (for the same reason, too). And it's far from certain that anybody is going to follow this advice anyhow...

I admit that I hadn't thought about the implications of this on build time. I could measure it, but I honestly can't believe it's going to be noticeable. Do you have any reasons to believe otherwise?


![](https://github.trello.services/images/mini-trello-icon.png) [(3) As a PO, I want to have an opportunity to make improvements to Terminal Quest [13]](https://trello.com/c/UZ7nzhPl/1239-3-as-a-po-i-want-to-have-an-opportunity-to-make-improvements-to-terminal-quest-13)


It was possible to not remove USE and BARTER, like with them there are no problems.


`LogRemittanceCreated` to avoid confusion.


Currently replaces it (can be swapped in code swapped by changing TONEMAP(X) macro to be defined as ReinhardTonemap(X))

I've renamed my checkbox to 'Apply Tonemapping' for now, reinhard will be back as an option when I add selectable output profiles (options something like: LOG-A, LOG-B, LOG-C, Reinhard tonemapped, Uncharted2 tonemapped)


It is probably sufficient to say if the features directory isn't present, then to return None, but aa_is_enabled (http://bazaar.launchpad.net/~apparmor-dev/apparmor/master/view/head:/libraries/libapparmor/src/kernel.c#L106) is a little more robust and might be useful for logging why we are returning None. Note that aa-enabled is a small C program wrapper around aa_is_enabled, so calling it directly from the core snap might be best.


For Windows Server users the "-- output /dev/strerr" is a problem, as /dev does not exist. Maybe drop the "--output" option all together?


`length = remittances.push`.


commit has been made.


Created multidev environment [jaypeak#ci-148](https://dashboard.pantheon.io/sites/63ccddbd-731f-4f84-badc-3edee2e28541#ci-148).

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


To avoid wrong values:
`Remittance({ funder: msg.sender, collector: collector... })`


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


ahm, sorry, this had to be reply on our previous conversation...


How do you ensure that this `combinedPassword` was not disclosed earlier.


You should emit an event here too.


It might be nice to return the 'why' here for future logging. Eg:
```
    return None, fmt.Errof("...")
    return Partial, fmt.Errof("missing x, y, z")
    return Full, nil
```


![](https://github.trello.services/images/mini-trello-icon.png) [Sperren von tiapz.com](https://trello.com/c/ACgkGmsD/283-sperren-von-tiapzcom)


+event


I think using dirs/dirs.go will make the mocking a little easier.


No deadline check?


The direction of this commit is fine. Couple of small comments.


HotFix-AirterminalSizingDivByZero (mjwitte) - Win64-Windows-7-VisualStudio-14: OK (2749 of 2750 tests passed, 0 test warnings)

Messages:

 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 608
 * Failed: 1

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


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


That looks difficult. Could you have a way for a single source of truth for both Solidity and Js?


Better deploy a new one every time instead of reusing the same tired instance.


Ah cool. So a checkbox "Profile" would fit better for the future.


The main code for the assessment of viability of biological species.


hi moiz . There is an extra bracket here please remove it please.


It's really difficult to find this bug. But l think it may be better to record some bytes in entry tail(no more than 2 bytes) and give it back after memmove. It will save 4 bytes :)


Вообще да, языковая модель меняется, только надо не так топорно делать.


One issue with the extra bracket fix it and commit the remaining code.


Is this intended to have two consecutive "!"

Would this compile to not-not and this be simply

`if (job->isActive)`

Is that the intend?


Two security holes.

1. You should take steps to ensure the same passwords aren't reused because they are passed in the clear.
2. There is a window of opportunity during which the funds can stolen. This isn't obvious. Consider how the proposed transaction is broadcast via gossip before it's included in a block. It might even fail. Even if it has enough gas, etc., a miner might decide to ignore it or even substitute a transaction of his own (now that he knows the passwords). Or, consider a race condition where lots of people release proposed transactions with the overheard passwords.

Add a check to make sure only the intended account can withdraw the funds, and this should plug the hole.


Можно ещё по WordNet'у считать расстояния от понятий, и на основании этих расстояний перевзвешивать языковую модель.


Ironically it's also a chore because of cyclic imports. I'm untangling that and I may end up being able to actually import dirs here as well. I'll try.


hey, add a comment over the method.


That's a good idea, I'll make that so


This first step (not finished), (plugin doesn't compile with this state of code), consists in :
1) putting the code of Categories/CategoryTypes and Tree from demandcenter plugin to plugin-ticketing.
2) remove all the classes of ticketing about categoy/type/precision, business, services and web objects.
3) Change all the calls of formers methods to get the former categories the new methods of TicketCategoryService

This first step won't allow to switch levels of the tree; but it will facilitate this future work.


this line should be deleted


Fix: #12


oh.. that's a typo. Drat. Thanks for spotting, will fix it!


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2140 of 2141 tests passed, 0 test warnings)

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


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


Changing these to have a longer common prefix like "PullRequestActionAssigned" would make IDE completion a little more reliable.


Welp that's sufficient reason to patch.


bam! fixed thx


Remove commented out line. ;-)


getField method expects P[3] component to be time.

In the code below getField method is called with p[4] array (declared at the line 127).
If we want to update the cached field value at the beginning of the step we may initialize p[4] before this call.


HotFix-AirterminalSizingDivByZero (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1530 of 1530 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-39.28%25-green.png)


Good point!


Sample Serenity project using JBehave and WebDriver


@guibou : merci de la contribution au projet ! Je n'ai pas encore compilé le code Haskell car je suis en train de refaire une installation propre du compilateur et autres dépendances (platform). Mais je l'ai déjà intégré dans "core/src/language". On verra plus tard pour la mise au point.

Je ne publie pas de code en ce moment car je suis en pleine réflexion sur l'orientation à donner à l'utilitaire. Du genre interface à la ligne de commande ou en mode interactif à la console. il y a aussi la nécessité d'inclure certaines fonctions de la libc dans le code Fortran (exercice nouveau pour moi !).

Mais je vais communiquer plus sur l'avancement de GPL_Torture via des pages Wiki. Encore merci !


It is really diffcult to find this bug. But l think it may be better to record some bytes in entry tail(no more than 2 bytes) and give it back after memmove. It will save 4 bytes in memory. :)


Does not `packedHexHash` take 2 arguments, as in line 53?


xd


Check the expected event was sent?


@MarkRedeman, eslint op travis is strenger dan eslint op de tv-unit. Is het voor jou eenvoudig om deze commit even te fixen zodat eslint happy is?


Change this sync call into async.


может, rubber? =)


И ещё раз: через запятую.


[inst/examples/example_distanceFromEachPoint.R:37:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/inst/examples/example_distanceFromEachPoint.R#L37) *style:* **Commented code should be removed.**
```r
#beginCluster(3) # can do parallel
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_distanceFromEachPoint.R:40:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/inst/examples/example_distanceFromEachPoint.R#L40) *style:* **Commented code should be removed.**
```r
#endCluster() # if beginCluster was run
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_spread.R:54:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/inst/examples/example_spread.R#L54) *style:* **Commented code should be removed.**
```r
## Interrupt a spread event using iterations - need `returnIndices = TRUE` to
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_spread.R:247:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/inst/examples/example_spread.R#L247) *style:* **Commented code should be removed.**
```r
## Note: `neighProbs = c(0,1)` forces each active pixel to move to 2 new pixels
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_spread.R:248:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/inst/examples/example_spread.R#L248) *style:* **Commented code should be removed.**
```r
## (`prob = 0` for 1 neighbour, `prob = 1` for 2 neighbours)
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:243:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L243) *style:* **Commented code should be removed.**
```r
# ### INCORPORATE RELEVANT PARTS OF THIS OLD INIT FUNCTION INTO INITCOORDS()
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:248:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L248) *style:* **Commented code should be removed.**
```r
# #'                      (currently only \code{RasterLayer} or
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:249:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L249) *style:* **Commented code should be removed.**
```r
# #'                      \code{SpatialPolygonsDataFrame}) accepted.
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:259:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L259) *style:* **Commented code should be removed.**
```r
#     ext <- extent(agentlocation)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:261:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L261) *style:* **Commented code should be removed.**
```r
#       #            nonNAs <- !is.na(getvalue(probinit))
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:262:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L262) *style:* **Commented code should be removed.**
```r
#       nonNAs <- !is.na(getValues(probinit))
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:263:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L263) *style:* **Commented code should be removed.**
```r
#       wh.nonNAs <- which(nonNAs)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:264:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L264) *style:* **Commented code should be removed.**
```r
#       #            ProbInit.v <- cumsum(getvalue(probinit)[nonNAs])
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:265:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L265) *style:* **Commented code should be removed.**
```r
#       ProbInit.v <- cumsum(getValues(probinit)[nonNAs])
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:267:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L267) *style:* **Commented code should be removed.**
```r
#         ran <- runif(numagents,0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:268:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L268) *style:* **Commented code should be removed.**
```r
#         fI <- findInterval(ran, ProbInit.v)+1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:269:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L269) *style:* **Commented code should be removed.**
```r
#         fI2 <- wh.nonNAs[fI]
          ^~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:270:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L270) *style:* **Commented code should be removed.**
```r
#         last.ran <- runif(numagents,0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:271:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L271) *style:* **Commented code should be removed.**
```r
#         last.fI <- findInterval(last.ran, ProbInit.v)+1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:272:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L272) *style:* **Commented code should be removed.**
```r
#         last.fI2 <- wh.nonNAs[last.fI]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:274:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L274) *style:* **Commented code should be removed.**
```r
#         #                va <- getvalue(probinit)[nonNAs]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:275:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L275) *style:* **Commented code should be removed.**
```r
#         va <- getValues(probinit)[nonNAs]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:276:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L276) *style:* **Commented code should be removed.**
```r
#         ran <- runif(length(va), 0, 1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:277:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L277) *style:* **Commented code should be removed.**
```r
#         fI2 <- wh.nonNAs[ran<va]
          ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:279:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L279) *style:* **Commented code should be removed.**
```r
#         last.ran <- runif(length(fI2), 0, 1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:280:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L280) *style:* **Commented code should be removed.**
```r
#         last.fI <- findInterval(last.ran, ProbInit.v) + 1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:281:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L281) *style:* **Commented code should be removed.**
```r
#         last.fI2 <- wh.nonNAs[last.fI]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:283:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L283) *style:* **Commented code should be removed.**
```r
#         #                last.ran <- runif(length(fI2),0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:284:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L284) *style:* **Commented code should be removed.**
```r
#         #                last.fI2 <- wh.nonNAs[last.ran<va]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:287:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L287) *style:* **Commented code should be removed.**
```r
#         position <- xyFromCell(agentlocation,fI2,spatial = TRUE)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:289:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L289) *style:* **Commented code should be removed.**
```r
#         position <- coordinates(agentlocation)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:291:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L291) *style:* **Commented code should be removed.**
```r
#         stop("need raster layer or Spatial Points object")
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:293:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L293) *style:* **Commented code should be removed.**
```r
#       numagents <- length(position)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:301:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L301) *style:* **Commented code should be removed.**
```r
#           colnames(xy) = c("x", "y")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:302:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L302) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(xy)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:303:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L303) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(sampleRandom(agentlocation, numagents, xy = TRUE, sp = TRUE))
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:305:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L305) *style:* **Commented code should be removed.**
```r
#           sam <- sample(1:length(agentlocation),numagents)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:306:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L306) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(agentlocation[sam,])
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:308:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L308) *style:* **Commented code should be removed.**
```r
#           stop("need raster layer or Spatial Points object")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:315:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L315) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(agentlocation)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:317:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L317) *style:* **Commented code should be removed.**
```r
#           stop("need raster layer or Spatial Points object")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:319:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L319) *style:* **Commented code should be removed.**
```r
#         numagents <- length(position)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:327:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L327) *style:* **Commented code should be removed.**
```r
#         numagents <- length(position)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:331:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L331) *style:* **Commented code should be removed.**
```r
#     stop("Need to provide agentlocation, which can be a map layer")
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:333:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L333) *style:* **Commented code should be removed.**
```r
#   heading1 <- runif(numagents, 0, 360)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:334:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L334) *style:* **Commented code should be removed.**
```r
#   distance <- runif(numagents, 0.1, 10)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:336:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L336) *style:* **Commented code should be removed.**
```r
#   .Object@ID <- as.character(1:numagents)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:337:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L337) *style:* **Commented code should be removed.**
```r
#   .Object@spatial <- position
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:338:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L338) *style:* **Commented code should be removed.**
```r
#   .Object@heading <- heading1
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:339:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L339) *style:* **Commented code should be removed.**
```r
#   .Object@distance <- distance
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:341:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/initialize.R#L341) *style:* **Commented code should be removed.**
```r
#   return(.Object)
    ^~~~~~~~~~~~~~~
```
[R/mapReduce.R:50:38:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/mapReduce.R#L50) *style:* **Variable or function name should be lowerCamelCase.**
```r
  fullRasterVals <- fullRasterVals[, row_number := 1L:.N]
                                     ^~~~~~~~~~
```
[R/mapReduce.R:53:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/mapReduce.R#L53) *style:* **Variable or function name should be lowerCamelCase.**
```r
  BsumVec <- reduced[fullRasterVals] # join
  ^~~~~~~
```
[R/neighbourhood.R:112:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L112) *style:* **Variable or function name should be lowerCamelCase.**
```r
adj.raw <- function(x = NULL, cells, directions = 8, sort = FALSE, pairs = TRUE,
^~~~~~~
```
[R/neighbourhood.R:114:21:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
                    match.adjacent = FALSE, cutoff.for.data.table = 2e3,
                    ^~~~~~~~~~~~~~
```
[R/neighbourhood.R:114:45:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
                    match.adjacent = FALSE, cutoff.for.data.table = 2e3,
                                            ^~~~~~~~~~~~~~~~~~~~~
```
[R/neighbourhood.R:117:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L117) *style:* **Variable or function name should be lowerCamelCase.**
```r
  J <- NULL
  ^
```
[R/neighbourhood.R:232:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L232) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
            ^
```
[R/neighbourhood.R:232:14:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L232) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
             ^
```
[R/neighbourhood.R:232:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L232) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
              ^
```
[R/neighbourhood.R:233:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L233) *style:* **Lines should not be more than 100 characters.**
```r
              ((adj[, "from"] %% numCol + adj[, "to"] %% numCol) == 1)) # | #right & left edge cells, with neighbours wrapped
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/neighbourhood.R:233:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L233) *style:* **Place a space before left parenthesis, except in a function call.**
```r
              ((adj[, "from"] %% numCol + adj[, "to"] %% numCol) == 1)) # | #right & left edge cells, with neighbours wrapped
               ^
```
[R/neighbourhood.R:237:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L237) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
            ^
```
[R/neighbourhood.R:237:14:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L237) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
             ^
```
[R/neighbourhood.R:237:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L237) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]) | # top or bottom of raster
              ^
```
[R/neighbourhood.R:238:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L238) *style:* **Lines should not be more than 100 characters.**
```r
              ((adj[, "from"] %% numCol + adj[, "to"] %% numCol) == 1)) # | #right & left edge cells, with neighbours wrapped
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/neighbourhood.R:238:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L238) *style:* **Place a space before left parenthesis, except in a function call.**
```r
              ((adj[, "from"] %% numCol + adj[, "to"] %% numCol) == 1)) # | #right & left edge cells, with neighbours wrapped
               ^
```
[R/neighbourhood.R:248:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L248) *style:* **Put spaces around all infix operators.**
```r
        numCol*(adj[whLefRig, "from"] - adj[whLefRig, "to"])
             ~^~
```
[R/neighbourhood.R:248:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L248) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        numCol*(adj[whLefRig, "from"] - adj[whLefRig, "to"])
               ^
```
[R/neighbourhood.R:249:20:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L249) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      whBotTop <- ((adj[, "to"] - 1) %% numCell + 1) != adj[, "to"]
                   ^
```
[R/neighbourhood.R:296:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L296) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((to - 1) %% numCell + 1) != to) |  #top or bottom of raster
            ^
```
[R/neighbourhood.R:296:14:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L296) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((to - 1) %% numCell + 1) != to) |  #top or bottom of raster
             ^
```
[R/neighbourhood.R:296:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L296) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          !((((to - 1) %% numCell + 1) != to) |  #top or bottom of raster
              ^
```
[R/neighbourhood.R:297:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L297) *style:* **Lines should not be more than 100 characters.**
```r
              ((from %% numCol + to %% numCol) == 1))# | #right & left edge cells, with neighbours wrapped
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/neighbourhood.R:297:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L297) *style:* **Place a space before left parenthesis, except in a function call.**
```r
              ((from %% numCol + to %% numCol) == 1))# | #right & left edge cells, with neighbours wrapped
               ^
```
[R/neighbourhood.R:313:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L313) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
              ^
```
[R/neighbourhood.R:313:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L313) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
               ^
```
[R/neighbourhood.R:313:17:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L313) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
                ^
```
[R/neighbourhood.R:313:52:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L313) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
                                                   ^
```
[R/neighbourhood.R:318:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L318) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
              ^
```
[R/neighbourhood.R:318:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L318) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
               ^
```
[R/neighbourhood.R:318:17:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L318) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
                ^
```
[R/neighbourhood.R:318:52:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L318) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            !((((to - 1) %% numCell + 1) != to) | ((from %% numCol + to %% numCol) == 1))
                                                   ^
```
[R/neighbourhood.R:327:22:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L327) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        whBotTop <- ((adj$to - 1) %% numCell + 1) != adj$to
                     ^
```
[R/neighbourhood.R:342:22:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L342) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        whBotTop <- ((adj$to - 1) %% numCell + 1) != adj$to
                     ^
```
[R/neighbourhood.R:854:29:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L854) *style:* **Variable or function name should be lowerCamelCase.**
```r
setGeneric("wrap", function(X, bounds, withHeading) {
                            ^
```
[R/neighbourhood.R:863:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L863) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:880:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L880) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:890:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L890) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:891:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L891) *style:* **Variable or function name should be lowerCamelCase.**
```r
    X <- wrap(X, bounds = extent(bounds))
    ^
```
[R/neighbourhood.R:900:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L900) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:901:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L901) *style:* **Variable or function name should be lowerCamelCase.**
```r
    X <- wrap(X, bounds = extent(bounds))
    ^
```
[R/neighbourhood.R:910:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L910) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:913:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L913) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds))
      ^
```
[R/neighbourhood.R:925:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L925) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:950:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L950) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:951:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L951) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds), withHeading = withHeading)
      ^
```
[R/neighbourhood.R:960:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L960) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:963:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/neighbourhood.R#L963) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds), withHeading = withHeading)
      ^
```
[R/RcppExports.R:67:20:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/RcppExports.R#L67) *style:* **Variable or function name should be lowerCamelCase.**
```r
runifC <- function(N) {
                   ^
```
[R/RcppExports.R:70:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/RcppExports.R#L70) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/SELES.R:53:23:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/SELES.R#L53) *style:* **Variable or function name should be lowerCamelCase.**
```r
numAgents <- function(N, probInit) {
                      ^
```
[R/spread.R:429:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L429) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if ((minValue(spreadProb) > 1L) || (maxValue(spreadProb) < 0L) ||
            ^
```
[R/spread.R:434:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L434) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if (((minValue(spreadProbLater) > 1L) || (maxValue(spreadProbLater) < 0L) ||
              ^
```
[R/spread.R:434:16:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L434) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if (((minValue(spreadProbLater) > 1L) || (maxValue(spreadProbLater) < 0L) ||
               ^
```
[R/spread.R:538:8:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L538) *style:* **Commented code should be removed.**
```r
      #isNASpreadProb <- is.na(spreadProb[])
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread.R:540:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L540) *style:* **Commented code should be removed.**
```r
      #   isNASpreadProb <- is.na(spreadProb[])
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread.R:541:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L541) *style:* **Commented code should be removed.**
```r
      #   spreadProb[isNASpreadProb] <- 0L
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread.R:783:43:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L783) *style:* **Place a space before left parenthesis, except in a function call.**
```r
              if (any(circleMaxRadius <= ((n - 1) * res(landscape)[1]))) {
                                          ^
```
[R/spread.R:807:19:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L807) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if (any((size + len) > maxSize & size <= maxSize)) {
                  ^
```
[R/spread.R:927:75:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L927) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            toKeepSR <- !(eventCells[, "id"] %fin% as.numeric(names(which((shouldStop)))))
                                                                          ^
```
[R/spread.R:937:17:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L937) *style:* **Place a space before left parenthesis, except in a function call.**
```r
            if ((returnDistances | spreadStateExists) & !allowOverlap) {
                ^
```
[R/spread.R:942:41:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread.R#L942) *style:* **Place a space before left parenthesis, except in a function call.**
```r
              notDupsEvents <- notDups[-(1:(nrSpreads - nrPotentials))]
                                        ^
```
[R/spread2.R:532:29:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L532) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        whNeedJump <- which(((clusterDT$numRetries + 1) %% 10) == 0)
                            ^
```
[R/spread2.R:532:30:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L532) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        whNeedJump <- which(((clusterDT$numRetries + 1) %% 10) == 0)
                             ^
```
[R/spread2.R:621:32:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L621) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          effDists <- dists * ((2 - angleQualities[, "angleQuality"]) / 2 *
                               ^
```
[R/spread2.R:699:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L699) *style:* **Commented code should be removed.**
```r
            # colnamesDtPot <- colnames(dtPotential)
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:700:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L700) *style:* **Commented code should be removed.**
```r
            # ll <-  lapply(colnamesDtPot, function(x) dtPotential[[x]][!spreadProbNA])
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:701:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L701) *style:* **Commented code should be removed.**
```r
            # names(ll) <- colnamesDtPot
              ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:702:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/R/spread2.R#L702) *style:* **Commented code should be removed.**
```r
            # dtPotential <- as.data.table(ll)
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-adj.R:53:38:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-adj.R#L53) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                             if (sum((adjDT - adj2[, colOrd]) ^ 2) == 0) {
                                     ^
```
[tests/testthat/test-adj.R:57:46:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-adj.R#L57) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                               expect_gt(sum((adjDT - adj2[, colOrd]) ^ 2), 0)
                                             ^
```
[tests/testthat/test-examples.R:2:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-examples.R#L2) *style:* **Commented code should be removed.**
```r
#   fns <- ls("package:SpaDES.tools")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:23:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L23) *style:* **Commented code should be removed.**
```r
  #expect_equal(sort(unique(getValues(biomass))), sort(reducedDT$biomass))
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:27:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L27) *style:* **Commented code should be removed.**
```r
#   library(data.table)
    ^~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:28:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L28) *style:* **Commented code should be removed.**
```r
#   library(raster)
    ^~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:31:6:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L31) *style:* **Commented code should be removed.**
```r
#    detach("package:data.table")
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:35:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L35) *style:* **Commented code should be removed.**
```r
#   ras <- raster(extent(0,15,0,15), res=1)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:36:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L36) *style:* **Commented code should be removed.**
```r
#   fullRas <- randomPolygons(ras, numTypes=5, speedup=1, p=0.3)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:37:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L37) *style:* **Commented code should be removed.**
```r
#   names(fullRas) <- "mapcodeAll"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:38:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L38) *style:* **Commented code should be removed.**
```r
#   uniqueComms <- raster::unique(fullRas)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:42:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L42) *style:* **Commented code should be removed.**
```r
#     biomass=rnbinom(length(uniqueComms), mu=4000, 0.4)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:44:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L44) *style:* **Commented code should be removed.**
```r
#   biomass <- rasterizeReduced(reducedDT, fullRas, "biomass")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:46:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L46) *style:* **Commented code should be removed.**
```r
#   expect_equal(sort(unique(getValues(biomass))), sort(reducedDT$biomass))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:47:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L47) *style:* **Commented code should be removed.**
```r
#   expect_equal(length(unique(getValues(biomass))), length(unique(getValues(fullRas))))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:49:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L49) *style:* **Commented code should be removed.**
```r
#   setkey(reducedDT, biomass)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:50:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L50) *style:* **Commented code should be removed.**
```r
#   communities <- rasterizeReduced(reducedDT, fullRas, "communities")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:51:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L51) *style:* **Commented code should be removed.**
```r
#   expect_equal(sort(unique(getValues(communities))), sort(reducedDT$communities))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:52:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L52) *style:* **Commented code should be removed.**
```r
#   expect_equal(length(unique(getValues(communities))), length(unique(getValues(fullRas))))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:56:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L56) *style:* **Commented code should be removed.**
```r
#   library(data.table)
    ^~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:57:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L57) *style:* **Commented code should be removed.**
```r
#   library(raster)
    ^~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:60:6:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L60) *style:* **Commented code should be removed.**
```r
#    detach("package:data.table")
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:64:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L64) *style:* **Commented code should be removed.**
```r
#   ras <- raster(extent(0, 15, 0, 15), res = 1)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:65:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L65) *style:* **Commented code should be removed.**
```r
#   fullRas <- randomPolygons(ras, numTypes = 5, speedup = 1, p = 0.3)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:67:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L67) *style:* **Commented code should be removed.**
```r
#   uniqueComms <- raster::unique(fullRas)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:71:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L71) *style:* **Commented code should be removed.**
```r
#     biomass=rnbinom(length(uniqueComms), mu = 4000, 0.4)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:73:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L73) *style:* **Commented code should be removed.**
```r
#   biomass <- rasterizeReduced(reducedDT, fullRas, "biomass")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:75:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L75) *style:* **Commented code should be removed.**
```r
#   setkey(reducedDT, biomass)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:76:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L76) *style:* **Commented code should be removed.**
```r
#   communities <- rasterizeReduced(reducedDT, fullRas, "communities")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:77:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L77) *style:* **Commented code should be removed.**
```r
#   expect_equal(sort(unique(getValues(communities))), sort(reducedDT$communities))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:81:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L81) *style:* **Commented code should be removed.**
```r
#   library(data.table)
    ^~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:82:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L82) *style:* **Commented code should be removed.**
```r
#   library(raster)
    ^~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:85:6:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L85) *style:* **Commented code should be removed.**
```r
#    detach("package:data.table")
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:89:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L89) *style:* **Commented code should be removed.**
```r
#   ras <- raster(extent(0,15,0,15), res=1)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:90:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L90) *style:* **Commented code should be removed.**
```r
#   fullRas <- randomPolygons(ras, numTypes=5, speedup=1, p=0.3)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:91:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L91) *style:* **Commented code should be removed.**
```r
#   names(fullRas) <- "mapcodeAll"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:92:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L92) *style:* **Commented code should be removed.**
```r
#   uniqueComms <- raster::unique(fullRas)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:96:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L96) *style:* **Commented code should be removed.**
```r
#     biomass=rnbinom(length(uniqueComms), mu=4000, 0.4)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:98:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L98) *style:* **Commented code should be removed.**
```r
#   biomass <- rasterizeReduced(reducedDT, fullRas, "biomass")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:100:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L100) *style:* **Commented code should be removed.**
```r
#   setkey(reducedDT, biomass)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:101:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L101) *style:* **Commented code should be removed.**
```r
#   communities <- rasterizeReduced(reducedDT, fullRas, "communities")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mapReduce.R:102:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-mapReduce.R#L102) *style:* **Commented code should be removed.**
```r
#   expect_equal(length(unique(getValues(communities))), length(unique(getValues(fullRas))))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:151:8:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-splitRaster.R#L151) *style:* **Commented code should be removed.**
```r
      #detach("package:raster")
       ^~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:247:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-splitRaster.R#L247) *style:* **Commented code should be removed.**
```r
  #expect_equal(min(values(m)), min(values(r))) ## Error: cannot allocate vector of size 4.8 Gb
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:266:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-splitRaster.R#L266) *style:* **Commented code should be removed.**
```r
  #expect_equal(max(values(m1)), max(values(r)))
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:267:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-splitRaster.R#L267) *style:* **Commented code should be removed.**
```r
  #expect_equal(min(values(m1)), min(values(r)))
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread.R:209:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread.R#L209) *style:* **Lines should not be more than 100 characters.**
```r
    if (interactive()) Plot(circEdge, addTo = "circs", cols = c("transparent", rainbow(numCircs)[whCirc]))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread.R:601:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread.R#L601) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  expect_true((radius * 1.5) %>=% max(dists[, "dists"]))
              ^
```
[tests/testthat/test-spread2.R:474:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L474) *style:* **Lines should not be more than 100 characters.**
```r
  ##                                    expr       min        lq      mean   median        uq       max neval
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:475:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L475) *style:* **Lines should not be more than 100 characters.**
```r
  ##         iterativeFun(ras, FALSE, n, sp)  3.096979 12.642477  73.02248 35.17520  91.10528  764.4073   300
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:475:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L475) *style:* **Commented code should be removed.**
```r
  ##         iterativeFun(ras, FALSE, n, sp)  3.096979 12.642477  73.02248 35.17520  91.10528  764.4073   300
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:476:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L476) *style:* **Lines should not be more than 100 characters.**
```r
  ##      nonIterativeFun(ras, FALSE, n, sp)  1.509484  6.555565  31.18444 14.91066  42.78317  158.5237   300
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:476:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L476) *style:* **Commented code should be removed.**
```r
  ##      nonIterativeFun(ras, FALSE, n, sp)  1.509484  6.555565  31.18444 14.91066  42.78317  158.5237   300
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:477:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L477) *style:* **Lines should not be more than 100 characters.**
```r
  ##           origSpread(ras, FALSE, n, sp)  5.154006  7.555631  14.87158 11.49005  17.50599  231.5487   300
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:477:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L477) *style:* **Commented code should be removed.**
```r
  ##           origSpread(ras, FALSE, n, sp)  5.154006  7.555631  14.87158 11.49005  17.50599  231.5487   300
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:478:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L478) *style:* **Lines should not be more than 100 characters.**
```r
  ## origSpreadIterations(ras, FALSE, n, sp) 10.754669 51.524368 141.48620 93.61996 169.10808 2110.2683   300
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:478:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L478) *style:* **Commented code should be removed.**
```r
  ## origSpreadIterations(ras, FALSE, n, sp) 10.754669 51.524368 141.48620 93.61996 169.10808 2110.2683   300
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:702:38:](https://github.com/PredictiveEcology/SpaDES.tools/blob/7daf331d2f9cdb98f48b6837f3a76be78a753f91/tests/testthat/test-spread2.R#L702) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  ciCentre[seq_len(ncell(ciCentre))[-(ncell(ciCentre) / 2 - ncol(ciCentre) / 2)]] <- NA_integer_
                                     ^
```


You cannot add gas to wei. You can add gas * gasPrice to wei.


Done


CITL was fine with uploading to Youtube?


not need this folder


@jimchu10: Fixed by #1174. To use it on your branch you should cherry-pick all three commits 1ae3a89b24122f7cdedf0bbfd6dd568e1da5ea9e, 8cae0ad20247acc56ace88b68e8f1b0c777006fe, 95251f5a808fb9b055703d3200efc52286ba9d8c to avoid a merge conflict.


I've added logging (well, errors that can be logged). Looking at the referenced apparmor library code I can easily check for `/sys/module/apparmor/parameters/enabled`. I want to balance correctness and simplicity here and I'd rather not call into a C program to get the answer.


This is not working in eclipse pydev


Didn't ask. That's why they're unlisted for now. Wanted for the the beginning of the semester though. Will probably remove/move later if needed.

(In theory, I can absolutely upload the "unedited" versions, since those are 100% mine. This was just quicker.)


Posthook still to do


Good work ! add relevant comments where possible and complete the code.


Time for 1.31?


Try your hand at `require()`.


Hi
I got a soft keyboard app so how can I import this as module and what I have to change
thanks


What's the status of this branch? My research group has wanted to add inlet profiles to our own fork of the code, and we don't want to duplicate effort.

How buggy are these two new options (`INPUT_FILE` and `VEL_PROFILE`)? How general purpose are they? Are they in a useable state? Or should we roll our own implementation?


Menos mal que te diste cuenta de esa linea.


Maybe you could try to stop the test, and click just after that on "modify" to make it re-enter the loop.

If I remember correctly, the stopping will shake it a bit (purge it) and most of the tasks with high residuals will disappear from the results.


Probably need to include `re.IGNORECASE` here, too.


You cannot add wei to gas. Different units. wei + gasUsed * gasPrice, yes.


RTD is busted, but that's not the permanent URL for the new gecko docs, and will likely go dead soon too.

https://bugzilla.mozilla.org/show_bug.cgi?id=1392765


There is no `kCFNumberUInt8Type` (which is odd, because `NSNumber` has an initializer from an `unsigned char`), so the way I chose to fix this in my copy of the code is to change the type of `value255` to `SInt16`, and use `kCFNumberSInt16Type`.


第一课时


[Hello World.txt](https://github.com/krcurry/datasciencecoursera/files/1249558/Hello.World.txt)


Previously, the for loop would try and search through the previously stored teams to check if the desired team already exists. On a fresh install, when no teams exist, the loop does not execute and everything breaks because 'notexists' is not set to true or broken out.


develop (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)


Passwords are already disclosed when lock in? Anyone can use them then. No secret, no challenge.


Yay!


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)


Build has started :hourglass_flowing_sand:


Build Passed :white_check_mark:


Tom, please review.  Thanks.


fix #27


Please change to `AlphaSOC NFR/`


Awesome! No need to make editing affect everything -- just wanted a convenient way to add to multiple sections at once. :-)

> On Aug 24, 2017, at 10:23 AM, Caleb Sander <notifications@github.com> wrote:
>
> The database currently doesn't track which assignments were added in a group (although I guess you could guess at it by looking for assignments with the same parameters assigned to sections of the same course), so editing (which should be possible soon) and deleting don't act on all of them. Not sure whether this should be changed. We could probably add section checkboxes to the edit popup.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
> .
>
>
> .


Should this be NFR? Not sure where the text is used..


Good!


How about calling the logout at kick()/close() too?


spaces


@chapmanb When I run the current master branch (post- #3351) the length descriptions look correct -- MPOS, MMQ, and MBQ all have `NUMBER=A` in the vcf header, and all have values for just the alt alleles in the GT field.  Are you seeing something else / am I misunderstanding the issue?  I wanted to omit the ref allele from these annotations to save visual space in the vcf, but I'm not opposed to including it in the output if users want it.


Again, NFR maybe?


Why do we need safekey here? Can they ahab code figure it out using the public key? I think we've talked about this.


NFR?


I thought it was `read` and not `start` ?


Okie dokie @mattkrick I think this may be a degree cleaner…

…we still have to make consideration for the contents of the `Panel` children. In the case that it only `Row`s and there is no panel header (with the label) we can put the burden of handling the first border issue on the `Panel` context. This removes the burden from the provider rows and the `Row` element.

Also, I don’t always want to swallow the first pixel of the children. That might cause a visual polish bug in the opposite since “hey what’s happening here? I gotta add a pixel of padding”.

Still one thing to remember but at least there is less prop nesting hell.

I made the inner border color more consistent to reflect original design intent and to tighten up the existing instances of panels within the app.


According to http://docs.platformio.org/en/latest/librarymanager/config.html#license

a license key could be define in `library.json`

I think despite it's an optional field, it's an important information for developper.

According to https://github.com/PaulStoffregen/ht1632c/blob/master/COPYING.txt it should be

    "license": "GPL-3.0",

License should clearly appears on:
http://platformio.org/lib/show/1799/ht1632c/


Is this meant to mutate the previous descriptor?


Created multidev environment [build-tools-634#ci-1](https://dashboard.pantheon.io/sites/ad877c9d-3aa8-4a55-82ba-bab904217626#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-634.pantheonsite.io/)


That's `.transfer`.


😊 I should've noticed it's static-only.  I got lucky this time.

If it's always static-only, we don't need it in `allowedRequisites` at all, but it's unlikely to harm as it is.


Created multidev environment [demobuild#ci-1](https://dashboard.pantheon.io/sites/3adea5f3-4d90-4309-94c4-cbcedb00fe4d#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-demobuild.pantheonsite.io/)


Update the state before interacting with an untrusted address, avoid doing TheDAO ;)


Created multidev environment [build-tools-631#ci-1](https://dashboard.pantheon.io/sites/2f2e4282-3432-4791-9fcc-d521c2090674#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-631.pantheonsite.io/)


you need a feature or bugfix that didn't exist before 0.3.1?


Nope, this is config path for nfr (binary one).


Same as above


No deadline check?


OK these are fine then


@lightsighter Seems reasonable as a first pass. Would like to do performance tests on this to see how it compares to the old branch.


Thanks for giving feedback. Please check new commits on my branch. Thank you.


Created multidev environment [demobuild#pr-didthing](https://dashboard.pantheon.io/sites/3adea5f3-4d90-4309-94c4-cbcedb00fe4d#pr-didthing).

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


This is useful!


Stephane rides again! Good luck!


Whitespace here.


Consider logging an event.


ok


> 100 milli-satoshis

Typo for 1000.


Created multidev environment [dgreat#pr-cog](https://dashboard.pantheon.io/sites/41cad7fe-472d-43a9-a470-d391b9622376#pr-cog).

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


Configuration Servert Prodution


@ivancich The subops are being removed (they were only for jewel compatibility).  I have a feeling this code really supposed to be matching MOSDRepOp[Reply?] instead?


`LogSplit`?


Oh yes very true


This project was meant to illustrate the use of dictionaries and how to modify them once created


What is this test? If the sender has 10 and he sends 6, now he has 4, so this is not ok?


Wow, this is one nasty hack :)

I think the right one is just

   return (ptr - (addr_t)&env->hvf_emul->regs[0]) < sizeof(env->hvf_emul->regs);

If it works, please squash this small change into this commit.


Still not handling odd numbers.


LGTM Dag, good work on the multiple hosts support, there's just one 'cosmetic' thing: when you deploy a multi hosts env(containing kvm, xen and vmware hosts) looks like trillian is unable to determine correctly that all the default templates are downloaded. We get the usual failure:
```
FAILED - RETRYING: TASK: Wait for default template to be ready before returning (4 retries left).
FAILED - RETRYING: TASK: Wait for default template to be ready before returning (3 retries left).
FAILED - RETRYING: TASK: Wait for default template to be ready before returning (2 retries left).
FAILED - RETRYING: TASK: Wait for default template to be ready before returning (1 retries left).
fatal: [trl-875-k-cs411-bstoyanov-mgmt1]: FAILED! => {"attempts": 300, "changed": true, "cmd": "cloudmonkey list templates listall=true templatefilter=all | jq '.template[]| select(.templatetype==\"BUILTIN\") | .isready'", "delta": "0:00:00.308744", "end": "2017-08-24 14:23:00.202610", "failed": true, "rc": 0, "start": "2017-08-24 14:22:59.893866", "stderr": "", "stdout": "true\ntrue\ntrue", "stdout_lines": ["true", "true", "true"], "warnings": []}

cmd: cloudmonkey list templates listall=true templatefilter=all | jq '.template[]| select(.templatetype=="BUILTIN") | .isready'
```
But all 3 default templates are there ready..


This commit was collected as part of "project-1".
 If you think this was a mistake or you want to submit this assignment late, please fill out the late form.


a tiny nitpick about the naming of this variable -- `Math` is a built-in JS object, and it might be confusing to have something else named `math`


Created multidev environment [demobuild#ci-3](https://dashboard.pantheon.io/sites/3adea5f3-4d90-4309-94c4-cbcedb00fe4d#ci-3).

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


That's not going to work, please use proper parameters. I just wrote it like that in the comment to suggest what the parameter values should be, sorry for the confusion.


I want to upload Chinese


So why isn't it called `harvestTimeInSeconds`, like the build time? 😇


Consider sticking to camelCase.


Please add return ticket: at the end of this line. Thank you.


Your code need some review. Thanks


Again `msg.sender.balance >= msg.value`?


Feels like polymorph should be handled by aurascript.... overall the system is way too heavy for what it still does. Seems like a small thing to rewrite.


sugestão que me lembrei é fazer: size = arguments.get(Math.max(arguments.size()-1, 0)).address.size;


methods should be camelCase :c


Please input current temperature in Farenheit.
90
That's really hot!
The tempature is 32.22222222222222 degrees Celsius. Have a nice day!


I think you only have false/false, true/true and true/false. You can consider replacing these 2 `bool` with a single `enum`.


Revert this commit and apply https://github.com/Exynos7580/android_device_samsung_exynos7580-common/commit/2f44734e5f74695b74f3e68dbd7150c789829170
This fix problem


What do you mean @EmmanuelVella? In this case it should be translating `bar` right?

@asenor


How about:

```solidity
struct RemittanceVenueStruct {
      bool is_value;
      bool is_active;
      uint fee;
      mapping (bytes32 => RemittanceRequestStruct) remittanceRequests;
    }
```


The first entry is wrong: ";" missing...


I have a couple of comments for this one. First of all, we should use pem filenames to specify the server *principal* on SAFE. As we discussed, a pem filename is much easier for anyone who will run this code. Second, we've also discussed to post the id sets for the principals when we launch the instances for them, instead of using a separate script to get this done at a  later time. We need only post identity set of a principal once. Why not do it at the launch time by the controller? An additional benefit is that the controller can get the ids from the post responses  and keep them for later use.  A controller is designed to the manage a slice and will inevitably  touch them.


No particular reason, just demonstrating how to access the indexes. As you say, it has no effect since they are both 0


i'll try later, i'm finishing build


I've bundled all of that into a Docker image :wink: https://github.com/DeadlySystem/ninspect


Have you checked whether keeping the reference to `remittanceRequests[_exchange_venue][_public_key]` will save gas?


This commit seems to be causing an error "export 'default' (imported as 'VueAdsense') was not found in 'vue-adsense'


Removing `|\\\\` would mean that no backslashes are allowed unless they're part of an escaped char.
There's nothing in the spec that would support this.  You could have, for example `a\b` (`b` is not escapable).


You probably don't need to repeat that here.


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - x86_64-MacOS-10.9-clang: OK (2743 of 2744 tests passed, 0 test warnings)

Messages:

 * 1 test had: AUD diffs.
 * 1 test had: BND diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 605
 * Failed: 1

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


No need to pass `_public_key`, just calculate it.


this is in the developer branch which not affect to the main branch


__apiary.manifest contains reference to non-existent file in repository.__ (Autogenerated message from apiary.io)
+ apiary.apib


Having an amount should be sufficient a test.


We're still in the deprecation phase for `x?` being a ternary (deprecated in favor of `x ?`) but we should get a PR ready to go, to be merged closer to 1.0. Right now so much is broken in packages on 0.7 that no one seems to be bothering to fix deprecations, so implementing it right now would be pretty jarring.


How do you make sure that a key whose private parts have been disclosed is not reused?


just single commit


No, because in the code just bellow, it translates `$fieldDescription->getName()`, which is `foo`


So I can just find an expired remittance, and claim it for myself? Big bug.


We need at least three SAFE server addresses: SDX, alice, and bob.


Thanks for pointing out additional cases that we missed.  Would you be interested in opening an pull request against https://github.com/openzfs/openzfs/pulls, and we'll get these changes integrated there and to illumos?


147146473-Issue6173-VAV-airflow-problems-with-other-zone-HVAC-equipment (lgu1234) - i386-Windows-7-VisualStudio-14: OK (2749 of 2750 tests passed, 1 test warnings)

Messages:

 * 2 tests had: EIO diffs.
 * 1 test had: Table small diffs.
 * 1 test had: AUD diffs.
 * 1 test had: BND diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

Failures:

regression Test Summary
 * Passed: 608
 * Failed: 1

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


add Readme


But that code wasn't changed on the PR


HotFix-AirterminalSizingDivByZero (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2141 of 2141 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-71.88%25-green.png)


I prefer having single 'var', Whats others preferences?


HotFix-AirterminalSizingDivByZero (mjwitte) - 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)


Pourquoi ne pas faire une fonction `get_gouv_api_address()` retournant des données différentes à chaque appel ?


Reason behind removing font-awesome icons and using gif directly?


This was a Project for The Tech Academy to illustrate HTML, CSS, and JavaScript.


Created multidev environment [example-wordpress-composer#dev](https://dashboard.pantheon.io/sites/46afe9ed-ebf0-44fe-acac-229694bfd8b3#dev).

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


channel destination needs to be determined if all song wanted to do cmd_np on play


Si j’ai bien compris, cette fonction retourne une liste d’addresses. Ne devrait-elle pas être nommée `gouv_api_addresses` ?


Reviewed and approved.


I bet they all pass ;)


Pourqoui ne pas remplacer ça par un `str.join(',')` ?


The following doc(s) have been deployed to the [composer](https://composer-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/composer-json](https://composer-static-docs.pantheonsite.io/docs/composer-json)
- [/docs/guides/build-tools](https://composer-static-docs.pantheonsite.io/docs/guides/build-tools)
- [source/docs/assets/images/pr-workflow/github-pr-diagram.png](https://github.com/pantheon-systems/documentation/commit/6f3148691a50e5b434463e38af9a3402b34ea153/source/docs/assets/images/pr-workflow/github-pr-diagram.png)


I'd be clear about "Java frontend".

Nit: it's ➡ its, twice.

Same comment as elsewhere: we still use `signons.sqlite` to drive the UI and content — not `logins.json`, and not `browser.db`, because that migration is not complete.


I liked it more previously, when it avoided the `for` loop :( Is there anything wrong with doing it like this?

    lambda_dBr = np.ones(V.shape) * np.inf * units.m
    indices = V.value == 0
    lambda_dBr[indices] = h / (m*V[indices]*Lorentz_factor(V[indices]))


Yes, but the check on line 120 changes the behavior :

Before the change, if :
- label option is not set : label = translated field name (foo)
- label option is set (! condition) : label= label option

After the change, if :
- label option is not set, label = translated field name (foo)
- label option is set (!false condition), label = translated field name (foo)


Note: The 4.x branch does not implement `withProtocolVersion()`
There is an interface on the Response object in [App::respond()](https://github.com/slimphp/Slim/blob/4.x/Slim/App.php#L670)


s/ although due/. Due

Mismatch between "and" and "&".

iOS is now in even better shape than Android: database schemas are defined in precisely two files, one per database (`BrowserSchema.swift`, `LoginsSchema.swift`).

It might be worth mentioning here that iOS has very strong encapsulation of storage: front-end code uses specific, narrow, safe APIs to read and write data, and storage manages sync metadata.


En fait, elle retourne une liste contenant uniquement `GOUV_API_ADDRESS` pour imiter le retour de l'API en question (exemple : https://api-adresse.data.gouv.fr/reverse/?lat=47.1913&lon=-1.5610 à partir de `features`). Quand l'adresse demandée n'est pas trouvée, l'API renvoie une liste `features` vide (d'où le []).


The `config` directory isn't added to package `files` (and we don't want it to be). All new installs will throw an error because this file is missing.


Be consistent: JSON, SQLite.

dependency on, not dependency to.

s/gecko side/Gecko-side/


Build failed with output:
```Running in Debug mode (configured by setting <Production> in /Config/Server.xml

       )
```


C’est assez long tout ça. Cette fonction `get_all_addresses()` fait visiblement à la fois des requêtes vers l’API et du parsing des résultats. Je pense que ça serait mieux de le découper un peu :upside_down_face:.


s/not written/often not written/

"are lost, resulting in potential inconsistency between prefs and state elsewhere in the system".


In the Apache commons lang I can do `"${uixName}"`, correct? Or should I just stick with `"{}", uixName`?


Even better: harvestTimeInSecondsRelativeToHarvestCapacity ? ;-)

> Op 24 aug. 2017 om 13:57 heeft Arjen van der Ende <notifications@github.com> het volgende geschreven:
>
> Shouldn't this be *2 like all other speeds? If not, why?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Why did *you* make this comment?


`passwordHash` was sent in `init`? It is no longer secret, it is no longer a challenge. Carol can checkout at any time without Bob.


Testing this out again. this is excellent


👍  Nice logic to get the only header properties


replace DateTime with exitDateTime.


How about only updating a balance field and let the owner withdraw all at a later date?


add relevant comment where possible


It is lacking some events.


We should avoid rebuilding sources. In preferred way is to make modifications to the configuration files. But that doesn't rebuild. If rebuild is necessary here for now, why is that?


@abhishekjain88 It would be great to add comments here regarding: Authorization header to take effect user-name needs to be empty or removed...


No balance update of any kind? Feels like high trapeze without a safety net.


Can I call this function again and overwrite the previous values?
What happens to the previous ether in that case?


Oops. ;-)


Color for found shares not work. Sorry for my bad english


Alright, thanks for the review. I will upload the edited file with all due editions.


this commit looks like:
![gif](http://g.recordit.co/NTQquszSH6.gif)


are you on linux or windows? (windows does not work)


At the smallest viewport width there still is much room for anything other than microscopic text.

Also, I generally like the placeholder text to represent what the text will look like (aka same size).

The only catch here is that the card immediately drops in height when you go from the 2-line placeholder to the first bit of typed text.


@jordanh


What is the example? Can we write a couple of paragraphs here describing the example scenario implemented by this class? Also, the name of this class must be specific to the example. *Example* is just too general.


As `new String[0]` is merely used to transport the result type the version with `.toArray()` is more clear. e. g. by  just adding the cast `(String[]) list.toArray()`, but with less type safety. Unfortunately the Java Collection API for List is a bit old fashioned here.