[inst/examples/example_distanceFromEachPoint.R:37:2:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/inst/examples/example_distanceFromEachPoint.R#L40) *style:* **Commented code should be removed.**
```r
#endCluster() # if beginCluster was run
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:82:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L82) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
    if ( method != "RMexp") {
        ^
```
[R/initialize.R:237:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L237) *style:* **Variable or function name should be lowerCamelCase.**
```r
  X <- c(jitter(sort(rbeta(nPoints, betaPar, betaPar) * (maxX - minX) + minX)),
  ^
```
[R/initialize.R:239:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L239) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Y <- c(jitter(sort(rbeta(nPoints / 2, betaPar, betaPar) * (maxY - meanY) + meanY)),
  ^
```
[R/initialize.R:243:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L243) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Sr1 <- Polygon(cbind(X + xAdd, Y + yAdd))
  ^~~
```
[R/initialize.R:244:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L244) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Srs1 <- Polygons(list(Sr1), "s1")
  ^~~~
```
[R/initialize.R:341:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L341) *style:* **Commented code should be removed.**
```r
# ### INCORPORATE RELEVANT PARTS OF THIS OLD INIT FUNCTION INTO INITCOORDS()
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:346:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L346) *style:* **Commented code should be removed.**
```r
# #'                      (currently only \code{RasterLayer} or
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:347:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L347) *style:* **Commented code should be removed.**
```r
# #'                      \code{SpatialPolygonsDataFrame}) accepted.
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:357:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L357) *style:* **Commented code should be removed.**
```r
#     ext <- extent(agentlocation)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:359:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L359) *style:* **Commented code should be removed.**
```r
#       #            nonNAs <- !is.na(getvalue(probinit))
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:360:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L360) *style:* **Commented code should be removed.**
```r
#       nonNAs <- !is.na(getValues(probinit))
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:361:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L361) *style:* **Commented code should be removed.**
```r
#       wh.nonNAs <- which(nonNAs)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:362:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L362) *style:* **Commented code should be removed.**
```r
#       #            ProbInit.v <- cumsum(getvalue(probinit)[nonNAs])
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:363:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L363) *style:* **Commented code should be removed.**
```r
#       ProbInit.v <- cumsum(getValues(probinit)[nonNAs])
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:365:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L365) *style:* **Commented code should be removed.**
```r
#         ran <- runif(numagents,0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:366:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L366) *style:* **Commented code should be removed.**
```r
#         fI <- findInterval(ran, ProbInit.v)+1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:367:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L367) *style:* **Commented code should be removed.**
```r
#         fI2 <- wh.nonNAs[fI]
          ^~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:368:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L368) *style:* **Commented code should be removed.**
```r
#         last.ran <- runif(numagents,0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:369:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L369) *style:* **Commented code should be removed.**
```r
#         last.fI <- findInterval(last.ran, ProbInit.v)+1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:370:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L370) *style:* **Commented code should be removed.**
```r
#         last.fI2 <- wh.nonNAs[last.fI]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:372:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L372) *style:* **Commented code should be removed.**
```r
#         #                va <- getvalue(probinit)[nonNAs]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:373:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L373) *style:* **Commented code should be removed.**
```r
#         va <- getValues(probinit)[nonNAs]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:374:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L374) *style:* **Commented code should be removed.**
```r
#         ran <- runif(length(va), 0, 1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:375:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L375) *style:* **Commented code should be removed.**
```r
#         fI2 <- wh.nonNAs[ran<va]
          ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:377:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L377) *style:* **Commented code should be removed.**
```r
#         last.ran <- runif(length(fI2), 0, 1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:378:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L378) *style:* **Commented code should be removed.**
```r
#         last.fI <- findInterval(last.ran, ProbInit.v) + 1
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:379:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L379) *style:* **Commented code should be removed.**
```r
#         last.fI2 <- wh.nonNAs[last.fI]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:381:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L381) *style:* **Commented code should be removed.**
```r
#         #                last.ran <- runif(length(fI2),0,1)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:382:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L382) *style:* **Commented code should be removed.**
```r
#         #                last.fI2 <- wh.nonNAs[last.ran<va]
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:385:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L385) *style:* **Commented code should be removed.**
```r
#         position <- xyFromCell(agentlocation,fI2,spatial = TRUE)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:387:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L387) *style:* **Commented code should be removed.**
```r
#         position <- coordinates(agentlocation)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:389:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L389) *style:* **Commented code should be removed.**
```r
#         stop("need raster layer or Spatial Points object")
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:391:9:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L391) *style:* **Commented code should be removed.**
```r
#       numagents <- length(position)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:399:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L399) *style:* **Commented code should be removed.**
```r
#           colnames(xy) = c("x", "y")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:400:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L400) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(xy)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:401:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L401) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(sampleRandom(agentlocation, numagents, xy = TRUE, sp = TRUE))
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:403:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L403) *style:* **Commented code should be removed.**
```r
#           sam <- sample(1:length(agentlocation),numagents)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:404:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L404) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(agentlocation[sam,])
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:406:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L406) *style:* **Commented code should be removed.**
```r
#           stop("need raster layer or Spatial Points object")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:413:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L413) *style:* **Commented code should be removed.**
```r
#           position <- SpatialPoints(agentlocation)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:415:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L415) *style:* **Commented code should be removed.**
```r
#           stop("need raster layer or Spatial Points object")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:417:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L417) *style:* **Commented code should be removed.**
```r
#         numagents <- length(position)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:425:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L425) *style:* **Commented code should be removed.**
```r
#         numagents <- length(position)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:429:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L429) *style:* **Commented code should be removed.**
```r
#     stop("Need to provide agentlocation, which can be a map layer")
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:431:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L431) *style:* **Commented code should be removed.**
```r
#   heading1 <- runif(numagents, 0, 360)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:432:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L432) *style:* **Commented code should be removed.**
```r
#   distance <- runif(numagents, 0.1, 10)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:434:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L434) *style:* **Commented code should be removed.**
```r
#   .Object@ID <- as.character(1:numagents)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:435:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L435) *style:* **Commented code should be removed.**
```r
#   .Object@spatial <- position
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:436:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L436) *style:* **Commented code should be removed.**
```r
#   .Object@heading <- heading1
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:437:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L437) *style:* **Commented code should be removed.**
```r
#   .Object@distance <- distance
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialize.R:439:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/initialize.R#L439) *style:* **Commented code should be removed.**
```r
#   return(.Object)
    ^~~~~~~~~~~~~~~
```
[R/mapReduce.R:53:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
                    match.adjacent = FALSE, cutoff.for.data.table = 2e3,
                                            ^~~~~~~~~~~~~~~~~~~~~
```
[R/neighbourhood.R:233:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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:238:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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:297:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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:813:29:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L813) *style:* **Variable or function name should be lowerCamelCase.**
```r
setGeneric("wrap", function(X, bounds, withHeading) {
                            ^
```
[R/neighbourhood.R:822:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L822) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:839:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L839) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:849:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L849) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:850:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L850) *style:* **Variable or function name should be lowerCamelCase.**
```r
    X <- wrap(X, bounds = extent(bounds))
    ^
```
[R/neighbourhood.R:859:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L859) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:860:5:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L860) *style:* **Variable or function name should be lowerCamelCase.**
```r
    X <- wrap(X, bounds = extent(bounds))
    ^
```
[R/neighbourhood.R:869:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L869) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds) {
                        ^
```
[R/neighbourhood.R:872:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L872) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds))
      ^
```
[R/neighbourhood.R:884:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L884) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:909:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L909) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:910:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L910) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds), withHeading = withHeading)
      ^
```
[R/neighbourhood.R:919:25:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L919) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(X, bounds, withHeading) {
                        ^
```
[R/neighbourhood.R:922:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L922) *style:* **Variable or function name should be lowerCamelCase.**
```r
      X <- wrap(X, bounds = extent(bounds), withHeading = withHeading)
      ^
```
[R/neighbourhood.R:1059:40:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/neighbourhood.R#L1059) *style:* **Commas should always have a space after.**
```r
  cc <- cbind(pureCircle2[, "x"] - xy[,"x"], pureCircle2[, "y"] - xy[, "y"])
                                       ^
```
[R/probability.R:41:35:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/probability.R#L41) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    1 / sqrt(var * 2 * pi) * exp(-((theta - mu + 2 * pi * k) ^ 2) / (2 * var))
                                  ^
```
[R/probability.R:44:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/probability.R#L44) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Next <- term(theta, mu, var, k)
  ^~~~
```
[R/probability.R:45:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/probability.R#L45) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Last <- Next
  ^~~~
```
[R/RcppExports.R:68:20:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/RcppExports.R#L68) *style:* **Variable or function name should be lowerCamelCase.**
```r
runifC <- function(N) {
                   ^
```
[R/RcppExports.R:71:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/RcppExports.R#L71) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/SELES.R:53:23:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/SELES.R#L53) *style:* **Variable or function name should be lowerCamelCase.**
```r
numAgents <- function(N, probInit) {
                      ^
```
[R/spades-tools-deprecated.R:53:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spades-tools-deprecated.R#L53) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/splitRaster.R:51:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/splitRaster.R#L51) *style:* **Lines should not be more than 100 characters.**
```r
  function(r, nx = 1, ny = 1, buffer = c(0, 0), path = file.path(getwd(), names(r)), cl, rType = "FLT4S") {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/splitRaster.R:111:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/splitRaster.R#L111) *style:* **Lines should not be more than 100 characters.**
```r
      clusterApplyLB(cl = cl, x = seq_along(extents), fun = croppy, e = extents, r = r, path = path, rType = rType)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread.R:538:8:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/R/spread.R#L541) *style:* **Commented code should be removed.**
```r
      #   spreadProb[isNASpreadProb] <- 0L
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread.R:942:41:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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:5:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L5) *style:* **Lines should not be more than 100 characters.**
```r
                           "quantityAdj", "quantityAdj2", "state", "size", "tooBigByNCells", "V1", "proportion"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:674:12:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L674) *style:* **Commented code should be removed.**
```r
          #pureCircle <- unique(pureCircle)
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:759:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L759) *style:* **Commented code should be removed.**
```r
          # colnamesDtPot <- colnames(dtPotential)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:760:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L760) *style:* **Commented code should be removed.**
```r
          # ll <-  lapply(colnamesDtPot, function(x) dtPotential[[x]][!spreadProbNA])
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:761:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L761) *style:* **Commented code should be removed.**
```r
          # names(ll) <- colnamesDtPot
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:762:13:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L762) *style:* **Commented code should be removed.**
```r
          # dtPotential <- as.data.table(ll)
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:772:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L772) *style:* **Lines should not be more than 100 characters.**
```r
                                                                   .I[sample.int(length(numNeighs), size = numNeighs, prob = spreadProbRel)],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:787:7:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L787) *style:* **Variable or function name should be lowerCamelCase.**
```r
      NAaSP <- !is.na(actualSpreadProb)
      ^~~~~
```
[R/spread2.R:789:42:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L789) *style:* **Commas should always have a space after.**
```r
        dtPotential <- dtPotential[NAaSP,]
                                         ^
```
[R/spread2.R:830:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L830) *style:* **Lines should not be more than 100 characters.**
```r
        # this is to prevent overlap within an event... in some cases, overlap within event is desired, so skip this block
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:848:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L848) *style:* **Lines should not be more than 100 characters.**
```r
        #successCells <- na.omit(successCells[!dupsWithinDtPotential]) # remove the dupsWithinDtPotential
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/spread2.R:848:10:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/R/spread2.R#L848) *style:* **Commented code should be removed.**
```r
        #successCells <- na.omit(successCells[!dupsWithinDtPotential]) # remove the dupsWithinDtPotential
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-examples.R:3:47:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L3) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (grepl("VIC-", Sys.info()["nodename"]))  { # for debugging only
                                              ^
```
[tests/testthat/test-examples.R:4:11:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L4) *style:* **Commented code should be removed.**
```r
    cat("#START##############\n", file = tmpExFile, append = FALSE)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-examples.R:14:15:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L14) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
    setwd(owd)}
              ^
```
[tests/testthat/test-examples.R:16:46:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L16) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (grepl("VIC-", Sys.info()["nodename"])) { # for debugging only
                                             ^
```
[tests/testthat/test-examples.R:17:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L17) *style:* **Lines should not be more than 100 characters.**
```r
    cat(paste("All files exist: ", isTRUE(all(file.exists(exFiles))), "\n"), file = tmpExFile, append = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-examples.R:22:48:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-examples.R#L22) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (grepl("VIC-", Sys.info()["nodename"])) { # for debugging only
                                               ^
```
[tests/testthat/test-mapReduce.R:23:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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/42db054a05e893b07334a28fceddcc826dfe276b/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-randomPolygon.R:25:60:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-randomPolygon.R#L25) *style:* **Put spaces around all infix operators.**
```r
  expect_true(base::abs(base::abs(polyArea - area)) <  area/4) ## TODO: why is this area/4?
                                                          ~^~
```
[tests/testthat/test-splitRaster.R:140:38:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L140) *style:* **Commas should always have a space after.**
```r
  expect_identical(dataType(y4[[1]]),"INT1U")
                                     ^
```
[tests/testthat/test-splitRaster.R:155:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L155) *style:* **Lines should not be more than 100 characters.**
```r
    tmpdir <- file.path(tempdir(), "splitRaster-test-parallel") %>% reproducible::checkPath(create = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:158:8:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L158) *style:* **Commented code should be removed.**
```r
      #detach("package:raster")
       ^~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:216:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L216) *style:* **Lines should not be more than 100 characters.**
```r
  tmpdir <- file.path(tempdir(), "splitRaster-test-large") %>% reproducible::checkPath(create = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:254:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L254) *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:273:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L273) *style:* **Commented code should be removed.**
```r
  #expect_equal(max(values(m1)), max(values(r)))
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-splitRaster.R:274:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-splitRaster.R#L274) *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/42db054a05e893b07334a28fceddcc826dfe276b/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-spread2.R:148:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L148) *style:* **Commented code should be removed.**
```r
  #quickDT <- data.table(id = seq_along(sams), initialPixels = sams, key = "id")
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:150:4:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L150) *style:* **Commented code should be removed.**
```r
  #cirOut <- quickDT[cirOut, on = ]
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:473:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L473) *style:* **Lines should not be more than 100 characters.**
```r
  ##                                    expr       min        lq      mean   median        uq       max neval
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:474:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L474) *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:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L475) *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:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L476) *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:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L477) *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:688:38:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L688) *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_
                                     ^
```
[tests/testthat/test-spread2.R:884:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L884) *style:* **Lines should not be more than 100 characters.**
```r
                       spreadProb = 0.23, persistProb = 0, iterations = 10, directions = 8L, plot.it = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:886:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L886) *style:* **Lines should not be more than 100 characters.**
```r
                      spreadProb = 0.23, persistProb = 0.8, iterations = 10, directions = 8L, plot.it = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:891:3:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L891) *style:* **Variable or function name should be lowerCamelCase.**
```r
  M <- matrix(0.8, nrow = 50, ncol = 50)
  ^
```
[tests/testthat/test-spread2.R:898:35:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L898) *style:* **Put spaces around all infix operators.**
```r
  start <- c(50, length(landscape)-49)
                                 ~^~
```
[tests/testthat/test-spread2.R:902:1:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L902) *style:* **Lines should not be more than 100 characters.**
```r
                        spreadProb = 0.23, persistProb = persistRas, iterations = 10, directions = 8L,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-spread2.R:916:14:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L916) *style:* **Use <-, not =, for assignment.**
```r
  spreadProb = landscape
             ^
```
[tests/testthat/test-spread2.R:918:72:](https://github.com/PredictiveEcology/SpaDES.tools/blob/42db054a05e893b07334a28fceddcc826dfe276b/tests/testthat/test-spread2.R#L918) *style:* **Commas should always have a space after.**
```r
  expect_silent(spread2(landscape = landscape, spreadProb = spreadProb,start = start,
                                                                       ^
```


does this mean I need to create my concrete Cart at project initialisation?


Also, created kickOut


Created multidev environment [sfgov#ci-376](https://dashboard.pantheon.io/sites/91d50373-c4cf-40e4-a646-cb73e16a140c#ci-376).

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


Good job breaking this up into components and nice, small commits!


I think this import is necessary to read/generate the .K files.


At rest, the accelerometer still measures gravitational acceleration (~9.8 m/s^s). I thought this was counter intuitive at first, but I read up on it and it actually makes some sense ([https://www.reddit.com/r/AskElectronics/comments/2ufunr/why_is_my_atrest_accelerometer_showing_1g_with/]).

So this condition might output true even at rest assuming that it is reading acceleration magnitude. @Gazzar95


should this timeout be in sync with the 20 seconds value below for status timer?


@colinhowells my gift to you: https://github.com/Automattic/simplenote-electron/releases/download/v1.1.5/Simplenote-macOS-1.1.5.zip


If m_num_bit is 64, m_subfield_max won't make sense and we should be sure not to use it.


Enforces users to sign `feeRecipient` as well, which:
1. Removes the possibility to steal the fee after the transaction is published to the network. For example, when DreamTeam publishes transaction with the signature to the network, the signature becomes public. Attacker can write a simple sniffer program which will publish the same transaction with higher gas price: this way, if the transaction will be mined before DreamTeam's transaction, attacker can steal the fee while DreamTeam will get `revert` exception. This issue becomes critical only when fee is set to big enough number comparing to gas used (which may happen).
2. Allows DreamTeam to receive fees to a particular wallet instead of fees going to the wallet that executed the transaction.


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


the timestamp is just informational, not functionat. The font engines don't care about that. The only purpose is for you the developer to know when that binary font was compiled, that's all. If the only change is in the timestamp, and that makes the checksum change, the font is still the same by all means. I wouldn't worry too much about it.
If these "mod tools" as you call them are run automatically by the CI build script at every commit, then I guess it's ok that you use the same SOURCE_DATE_EPOCH timestamp derived from the git commit, and prevent it from being updated to whatever is the current time.


Nice job using CSS in JS.


<details>

<summary>Submitting the task to Taskcluster failed. Details</summary>

You do not have sufficient scopes. You are missing the following scopes:

{
  "AnyOf": \[
    {
      "AnyOf": \[
        "queue:create\-task:highest:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:very\-high:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:high:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:medium:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:low:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:very\-low:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:lowest:aws\-provisioner\-v1/servo\-docker\-worker"
      \]
    },
    {
      "AnyOf": \[
        "queue:create\-task:aws\-provisioner\-v1/servo\-docker\-worker",
        {
          "AllOf": \[
            "queue:define\-task:aws\-provisioner\-v1/servo\-docker\-worker",
            "queue:task\-group\-id:taskcluster\-github/X1bfQY0AQjOc7ROIwel5MA",
            "queue:schedule\-task:taskcluster\-github/X1bfQY0AQjOc7ROIwel5MA/X1bfQY0AQjOc7ROIwel5MA"
          \]
        }
      \]
    }
  \]
}

You have the scopes:

```
\[
  "assume:project:taskcluster:mozilla\-github\-repository",
  "assume:repo:github.com/servo/servo:pull\-request",
  "notify:email:\*",
  "notify:irc\-channel:\*",
  "notify:irc\-user:\*",
  "queue:create\-task:aws\-provisioner\-v1/github\-worker",
  "queue:create\-task:aws\-provisioner\-v1/github\-worker\-windows",
  "queue:create\-task:aws\-provisioner\-v1/servo",
  "queue:create\-task:aws\-provisioner\-v1/win2012r2",
  "queue:route:garbage.\*",
  "queue:route:index.garbage.\*",
  "queue:route:notify.email.\*",
  "queue:route:notify.irc\-channel.\*",
  "queue:route:notify.irc\-user.\*",
  "queue:scheduler\-id:taskcluster\-github",
  "secrets:get:garbage/\*",
  "secrets:set:garbage/\*"
\]
```

This request requires you to satisfy this scope expression:

```
{
  "AllOf": \[
    {
      "AnyOf": \[
        {
          "AllOf": \[
            "queue:scheduler\-id:taskcluster\-github",
            {
              "AnyOf": \[
                "queue:create\-task:highest:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:very\-high:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:high:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:medium:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:low:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:very\-low:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:lowest:aws\-provisioner\-v1/servo\-docker\-worker"
              \]
            }
          \]
        },
        {
          "AnyOf": \[
            "queue:create\-task:aws\-provisioner\-v1/servo\-docker\-worker",
            {
              "AllOf": \[
                "queue:define\-task:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:task\-group\-id:taskcluster\-github/X1bfQY0AQjOc7ROIwel5MA",
                "queue:schedule\-task:taskcluster\-github/X1bfQY0AQjOc7ROIwel5MA/X1bfQY0AQjOc7ROIwel5MA"
              \]
            }
          \]
        }
      \]
    }
  \]
}
```
----
method:     createTask
errorCode:  InsufficientScopes
statusCode: 403
time:       2018-06-15T16:02:35.635Z

</details>


I don't honestly think anyone will be using anything but the Show instance.
Remember these are only in exceptions! I do think a deepseq dependency is
fine, since basically everything depends on it and it's a GHC boot package.
If we don't already depend transitively on hashable, I don't really see the
point.

On Fri, Jun 15, 2018, 11:57 AM chessai <notifications@github.com> wrote:

> do you think Hashable and NFData are fine? hashable and deepseq are
> pretty light
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/nikita-volkov/refined/commit/34728034e7e29dd0d7cc028c7d4302890ce2ee2c#commitcomment-29381977>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABzi_V--wIkHvnso_qM76D4BjRAbRHTjks5t89lrgaJpZM4UpCJo>
> .
>


Did this ought to be updated for v1.2?


Also, consider adding `indexed` to the fields that will likely be filtered from the client-side.


Good attempt!


In line with the above comment, we may be looking at an acceleration of close to 9.8 at this point as well. Although I'm not certain.. we should test it. @Gazzar95


These 2 should be unnecessary as all values are initialised as `0`.


Sorry I think that you misunderstood (or maybe I misunderstood you).  Not for our build workflow/approach.  I meant how do we address this for other typeface projects that might want to use the font-v tool for versioning.  I am trying to make it generally useful outside of the scope of our own build workflows.


This is CVE-2018-12458


a typo?


See above comment regarding acceleration. We likely will not see a zero output at rest.


This is CVE-2018-12459


Fine ! :+1:
I'll try to move higher the occurrence thing on my pile of things to work on !


accepted


This is CVE-2018-12460


Since anyone other than those who have anything to withdraw will see a `recipientBalance` of `0`, some of these address checks are unnecessary.
On the other hand, a `require(recipientBalance > 0)` could notify callers of "nothing is going to happen here".


It is fine to subtract fee from the allowed amount:
1. In case of user wants to withdraw tokens to their account, we're fine.
2. In case of user wants to transfer tokens somewhere else, it makes sense to get fee from the allowed balance, as user may not have tokens on their balance.


you simply suggest them to `export SOURCE_DATE_EPOCH="$(date +%s)"`


Presumably this check is superfluous.


fg


![](https://github.trello.services/images/mini-trello-icon.png) [31 - Tela "Sobre"](https://trello.com/c/oGbzwi6r/145-31-tela-sobre)


Make sure your components are capitalized like "class Footer extends Component" to keep with convention.


Good work!


Overflow should add m_subfield_max + 1; i.e. 1<<m_num_bit.


Instead of letting the transaction go through if there is nothing to do, prefer `require(remainders != 0)`.


You could also just prepend r to the string and not require the additional slash, which would make it follow consistent escape rules as the line above it.


There's no need in this condition, it is always `true`.


Transfer tokens directly instead of approving.


But it's still here


- Colorblind mode removed, again.
- New analytic events: `verify_account_app_opened`, `change_log_cta_clicked`

## Strings
#### Added
- `VERIFICATION_VERIFIED: "Email Verified!"`
- `VERIFICATION_OPEN_DISCORD: "Continue to Discord"`
#### Removed
- `GAME_FEED_READ_ACTIVITY_ACTION: "Read"`
- Also the colorblind mode strings.


I'm not seeing the homework with this commit. Make sure you're pushing the right code!


`date +%s` is the seconds since the Unix epoch up to now, I guess you actually want `git log -1 --format=%ct"` for the unix timestamp of the latest commit, HEAD


or whatever timestamp you need


But that assumes that they have used the fontmake compiler to compile the binary that is being modified, no? Are other compilers using the same environment variable approach?


Good work. Try to make smaller commits that break up different work so you can refer back to them later and so we can see what each commit actually does. Otherwise, nice job!


Yes. The only difference is that the smart contract is not dealing in conversions. The smart contract takes and stores. Only.


Aight let's see if it works


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.9.7681f73fb3f9122b58b789b451e21e9d975a529b](http://build.palaso.org/viewLog.html?buildId=122600&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:26


TeamCity Glyssen :: Glyssen pull requests [Build *.*.702.7681f73fb3f9122b58b789b451e21e9d975a529b](http://build.palaso.org/viewLog.html?buildId=122601&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **FAILURE**
Summary: Compilation error: GlyssenShared\GlyssenShared.csproj Build time: 00:00:40


Good work!


and still maintains a time dependency if the post compile mods occur outside of the subshell where the original compile process occurred.  As I say, still trying to think it through :) Don't even fully understand the problem yet.


Good attempt!


You may want to add cleanup:

```
rm -f v${PROTOC_VER}.tar.gz
rm -rf protobuf-${PROTOC_VER}/
```


윈도우용


ok, I see your point. You can add a command line option to your script to `--no-recalc-timestamp`, and load TTFont with recalcTimestamp=False. Maybe it can be a opt-in, disabled by default. That is, by default you load TTFont with recalcTimestamp=True. Which will set it to the current time, or to the SOURCE_DATE_EPOCH if that is set. If one doesn't want to be bothered with timestamp changes, the use --no-recalc-timestamp


Checkmarx scan results 15 Jun 2018 19:13 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a23cad998261c81bf63aa21ac681a2e8b5303e7c	@web-flow


리눅스 용


이 문법도 안맞고 쓸모 없는 놈들


`remData[hashPass].deadline` is another `SLOAD`. Can you find a way to avoid reading this piece of information from storage?
Also, you should be consistent in your spacing: 1 space `, hashPass`, or no space `,remData[hashPass].deadline`.


@mattkrick this doesn’t seem to work…I think it’s because `BaseButton` is a component with state? Is that right, or is there something else at play?


스레드는 퍼오기만 하고 언제 이해하니


It's good that you tried! You can commit some of those attempts with a "WIP (work in progress)" commit so we can see where you're going wrong. Also, don't be afraid to ask me or Davey questions in class or on Slack!!


At first I made weight and gender as _int_ but then I realized that by doing it I cannot check if they are null.
Good work!!


Misschien zit de bug hier ergens?


Hi @vertexvaar as I said in the related issue, I proposed a more general fix, so that you only have to deal with a single timeout.

Btw, thanks for your contribution, that's appreciated.


So the password check is done against `msg.sender` but the `.transfer` is done on `tempData.recipient`?
Can you change it so that it appears consistent?


Good work breaking these up into components.


Perhaps you should have saved `hashHelper(firstPassw, msg.sender)` in a variable instead of calculating it here a second time.


> Don't even fully understand the problem yet.

the problem is that there is no problem. Your font-v tool _did_ change the binary font file, eg. by inserting some strings in the name table. So having the head.modified be updated by default is actually a sensible default behavior. If one is concerned with builds being reproducible, then one can either let fonttools read SOURCE_DATE_EPOCH if that is already set (like with Hack fonts), or one can decide to not update the timestamp with some command-line option.


Zmienne powinny być prywatne


You are reading `tempData.balance` from storage 3 times. Can you find a way to read it only once?


What happens if the recipient calls this function twice, with the same parameters?


i have made changes that were listed how do i find out if my application has been successful thanks


Luca vs SenseTime
1:0


hello world


That seems like the best approach.  I agree with that.  Thanks for your input here.  Will open a new issue for the addition of a new option.  This project includes a publicly accessible library so will include optional approach to these writes there as well.


Hi,

Is there will be Yii 2.1 or only 3.0 ?


Is this function reserved for the owner or can any call it?


Good work. Try to make smaller, more granular commits.


Solution works but it is a brute force method that extends the time limit.


Perhaps here too you can avoid calculating the hash twice.


Is it necessary that the password be sent here? Why?


Take a look at the documentation for .map() and make sure you're understanding how it works. Please don't hesitate to reach out to me or Davey on Slack!


Good job using CSS in JS.


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


Sounds good - thank for the update.


Good attempt!


hello,I am a student!


Try to be a little more descriptive in your commit messages. Also, you can use "WIP (work in progress + whatever it is you're attempting to do" as a message so you can refer back and understand what you were attempting in that commit!


Created multidev environment [sfgov#ci-377](https://dashboard.pantheon.io/sites/91d50373-c4cf-40e4-a646-cb73e16a140c#ci-377).

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


@WesIngwersen  Hi wes I just updated this file with two corrections.
One is I re-changed it back to the old style to read the new eGRID_req_fields.txt file with column type.

Also added steam output


Good point. Could you raise an issue? That's an easy one to miss given the version is otherwise templated in the RAML. I'll make sure we address that in the next patch release which I expect we'll be aiming to make a release for shortly.


I would rather logic for both OSs:

```
export BUILD_TAG=${BUILD_TAG}_${TRAVIS_OS_NAME%%*-}
```


@WesIngwersen  Wes, i added the steam output to the required fields.


I am on apache2.4 and i see the commited code but still i am getting this message "[auth_gssapi:error] [pid 121622] [client 10.60.160.134:8591] NO AUTH DATA Client did not send any authentication headers," as an error which should come as INFO.

Any idea why?


@RowenStrafe I just uploaded the 1.10.2 alpha build [here](https://github.com/Tommsy64/SmartMoving/releases/tag/1.10.2-16.4-RC3).


Nice job breaking it up!


Needs testing before merge to `18.04`


Here value of `length args` is bound to the `otherwise` symbol which propably isn't wanted. In `case` expressions wildcard: `_` is used for undefined values


👍 Good job using CSS in JS.


Good work and way to think through a solution.


@jperelli, this is not the only fix. More fixes are coming. ;) I plan to make a pull request, when all of them are ready. You can import this change before, if you want, though.


Nice work!


Great use of map!


Way to go! 👍


Good job using CSS in JS.


These links should be slotted, so that we can easily recompose the Navbar for things like subnavs etc.


`a >>= b . c` form of expressions has order of operations a c b. It could be rewritten as `c . b =<< a` but it may be my preference to have operations in order of execution


Không phải tạo `DataFakerController` ntn.
Ý anh là dùng Faker để tạo seed data ấy. :(


Good work breaking these out into components.


Good attempt at this.


safd


@ThuyNT13 keep in mind that there's a time complexity of O(n) to this array.includes function. This could potentially lead to a worst case complexity of O(n^2) for your algorithm, since the includes is nested in a for loop. Consider using a different data structure that's more efficient for membership testing, ie: dictionary or set, which have a search time of O(1).

Here's an example alternative implementation:

```javascript
const containsDuplicate = function (nums) {
    const numSet = new Set(nums);
    return numSet.size != nums.length;
};
```


Yes, you are right.


Why would someone use DNS over HTTPS instead of DNSSEC?


Fixes #1


Fixes #2


Yeah, probably need to specialise the whole class for Ind2D/Ind3D, as presumably none of the z offsetting should do anything with an Ind2D


@fabiosantoscode `-d MOZ_SourceMap=require('source-map')` will not work in a browser. `require()` should not appear in the bundle.

What about just adding `MOZ_SourceMap` in as a second parameter to the IIFE and let the user pass the `source-map` module reference in only if needed?


 'commented' => 'hat deinen Post kommentiert.'
Suggest a shorter version

 'commented' => 'kommentierte deinen Post.',


@xavierlepretre Anyone can call the function, since any address can be passed to the password generator. Consequently, the user does not have to be the owner, but the money must be sent to the owner.

To understand what struct we are calling. We can pass the hash. But I do not see a particular difference in this situation


Nice job!


Using Data.Maybe from standard library it could be written as:

    eval env (List ((Atom "if"):pred:conseq:t)) =
      do result <- eval env pred
        case result of
          Bool False -> eval env . fromMaybe result . listToMaybe $ t
          _                -> eval env conseq

And with LambdaCase language extension you could write it without `do` as:

    eval env pred >>= \case result of
      Bool False ...


Fixes #3
Fixes #4


Good job adding CSS in JS to these components.


The other problem is the worker, as you see we have a hardcoded version in there. Need to do something fancy to get that problem resolved as well.


I have removed all 64 bit field definitions from the MSRs, so this should not be a problem.  Perhaps a throw would be appropriate if line 96 is hit.


optimized the binary search a bit.


Fixes #5


Fixed, and pushed up to this branch.  Thanks!


Jane, cohort death is handled separately by a different set of functions.  Ultimately it ends up at the CohortDied function in succession.  (In NECN we renamed it CohortTotalMortality for clarity.)  In Biomass Succession, this could be improved similar to what we did in NECN, removing that confusing table.


If you don't, then it's an element of `Line B` rather than a path type, and we're constructing an element of the Fiber type, which involves paths, not lines.


Updates #6


Upload file


Fixes #6


Good attempt. Don't hesitate to reach out on Slack if you're running into an error!


Try to make smaller commits with more descriptive messages.


Try to authenticate with GitHub and see if you can make that work! Good try!


Thank you sir! Compiling apps myself is tricky, at least for me, so I appreciate it


@xavierlepretre I'm not sure. We will create a new transaction, but since we are replacing the value     `remData[tempHash].balance = 0;`
it will have zero balance and it should not go further.


Fixes #17


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.10.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122602&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:27


This is awesome @Dax74, a doubt about this settings...

If I add a new line with:
`es: Insertar tu enlace social`

That line applies into Spanish translation? 🤔


TeamCity Glyssen :: Glyssen pull requests [Build *.*.703.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122603&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:36


Fixes #17 not #16


Before we make this a public API, this would be an internal class in `/Common/` on the CoreFX side.

Eventually, we could make this part of an out-of-band assembly.


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.11.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122604&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:34


Good work. Try to make smaller commits!


Good job using CSS in JS here.


Thanks for pointing this out, we fixed docs in this matter: https://docs.coinapi.io/#list-all-assets


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.12.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122605&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:31


Please open a github issue (against the main project) and include what version of  mod_auth_gssapi you're running.  Thanks!


@remicollet Is dropping bundled libzip scheduled for PHP 7.3? If so, I suggest to remove it right away. Otherwise this note should be removed.


Updates #16


I had this done locally, but forgot to commit it 🤦♂


<details>

<summary>Submitting the task to Taskcluster failed. Details</summary>

You do not have sufficient scopes. You are missing the following scopes:

{
  "AnyOf": \[
    {
      "AnyOf": \[
        "queue:create\-task:highest:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:very\-high:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:high:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:medium:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:low:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:very\-low:aws\-provisioner\-v1/servo\-docker\-worker",
        "queue:create\-task:lowest:aws\-provisioner\-v1/servo\-docker\-worker"
      \]
    },
    {
      "AnyOf": \[
        "queue:create\-task:aws\-provisioner\-v1/servo\-docker\-worker",
        {
          "AllOf": \[
            "queue:define\-task:aws\-provisioner\-v1/servo\-docker\-worker",
            "queue:task\-group\-id:taskcluster\-github/C9PJl6QvQkaoKskStvaMnA",
            "queue:schedule\-task:taskcluster\-github/C9PJl6QvQkaoKskStvaMnA/C9PJl6QvQkaoKskStvaMnA"
          \]
        }
      \]
    }
  \]
}

You have the scopes:

```
\[
  "assume:project:taskcluster:mozilla\-github\-repository",
  "assume:repo:github.com/servo/servo:pull\-request",
  "notify:email:\*",
  "notify:irc\-channel:\*",
  "notify:irc\-user:\*",
  "queue:create\-task:aws\-provisioner\-v1/github\-worker",
  "queue:create\-task:aws\-provisioner\-v1/github\-worker\-windows",
  "queue:create\-task:aws\-provisioner\-v1/servo",
  "queue:create\-task:aws\-provisioner\-v1/win2012r2",
  "queue:route:garbage.\*",
  "queue:route:index.garbage.\*",
  "queue:route:notify.email.\*",
  "queue:route:notify.irc\-channel.\*",
  "queue:route:notify.irc\-user.\*",
  "queue:scheduler\-id:taskcluster\-github",
  "secrets:get:garbage/\*",
  "secrets:set:garbage/\*"
\]
```

This request requires you to satisfy this scope expression:

```
{
  "AllOf": \[
    {
      "AnyOf": \[
        {
          "AllOf": \[
            "queue:scheduler\-id:taskcluster\-github",
            {
              "AnyOf": \[
                "queue:create\-task:highest:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:very\-high:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:high:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:medium:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:low:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:very\-low:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:create\-task:lowest:aws\-provisioner\-v1/servo\-docker\-worker"
              \]
            }
          \]
        },
        {
          "AnyOf": \[
            "queue:create\-task:aws\-provisioner\-v1/servo\-docker\-worker",
            {
              "AllOf": \[
                "queue:define\-task:aws\-provisioner\-v1/servo\-docker\-worker",
                "queue:task\-group\-id:taskcluster\-github/C9PJl6QvQkaoKskStvaMnA",
                "queue:schedule\-task:taskcluster\-github/C9PJl6QvQkaoKskStvaMnA/C9PJl6QvQkaoKskStvaMnA"
              \]
            }
          \]
        }
      \]
    }
  \]
}
```
----
method:     createTask
errorCode:  InsufficientScopes
statusCode: 403
time:       2018-06-15T16:58:12.572Z

</details>


Updates #17


TeamCity Architecture Team :: Java Platform :: otj-kafka :: BuildTest [Build 177](https://teamcity.otenv.com/viewLog.html?buildId=8748008&buildTypeId=ArchitectureTeam_JavaPlatform_OtjKafka_BuildTest) is now running


Well, this TF process (internally called already) must be on some GPU (in hvd, it happens to be based on the local rank anyway). But, if a user decides to use a different GPU at the TF-script level, it will conflict with that GPU assignment. Try without cudaSetDevice, and it should work without that. I don't think my research version has it either.


Created multidev environment [sfgov#pr-71](https://dashboard.pantheon.io/sites/91d50373-c4cf-40e4-a646-cb73e16a140c#pr-71).

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


TeamCity Architecture Team :: Java Platform :: otj-kafka :: BuildTest [Build 177](https://teamcity.otenv.com/viewLog.html?buildId=8748008&buildTypeId=ArchitectureTeam_JavaPlatform_OtjKafka_BuildTest) outcome was **SUCCESS**
Summary: Tests passed: 9 Build time: 00:01:40


If an error happened, the periodic task gets an exclamation icon in the tasks list:

The error is also shown in the edit form:


red file


> It depends on where the current Koha-community master is.

Does that impacts whether this commit should also be included in bug 12586?

> -replicate the bug I found (maybe it no longer exists),

So I maybe should:
- try bug's 12586 test plan on master. To confirm that the bug exists
- if that doesn't work, try on a version of Koha from the time of the patch was done. To be sure that I correctly ran the test plan
- if that works (initial test plan fails but works with bug's 12586 patches) then I should try with this commit on master. To check whether it's a dependency of bug 12586 now or if the patches are just broken due to the changes on master.

Does that makes sense or is there a better way?

By the way, which test plan? Is [this test plan](https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12586#c5) the right one? Because it wasn't included in the signed off commit.

>  -use my solution as a basis to make an even better one to fix the possible issue upstream.

bug's 12586 patch or this commit?

After a deeper reading this commit seems to be a feature addition that should not impact the rest of C4::Matcher right?


You're quite welcome! I'd be happy to take a look again when you want some more eyes.


@xavierlepretre We raised this issue and how much I understood it - you need to specify the recipient in the constructor.


Thank you for your great work!

As far as I see you have declared Laravel 5.4 compatibility. Laravel 5.4 itself is PHP 5.6.4+ compatible. But your component has higher requirements. This frustrates those developers who are still forced to stay on the fifth version.

I suggest to drop this unnecessary rule as far as composer.lock. Please refer the composer docs:
https://getcomposer.org/doc/02-libraries.md#lock-file

> For your library you may commit the composer.lock file if you want to. This can help your team to always test against the same dependency versions. However, this lock file will not have any effect on other projects that depend on it. It only has an effect on the main project.


The issue where async prefetching is not working only happens in Sonic Colors.


@viniciusaac there is no `filter` parameter in this REST API, please check the REST API documentation: https://woocommerce.github.io/woocommerce-rest-api-docs/


Good job using CSS in JS.


Good job!


If node_ is nullptr, is it okay to dereference a nullptr here?


:p


-Os = Optimize for size


Good work breaking these up.


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.13.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122608&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:34


Make sure your commit messages are descriptive. Maybe commit more often and break up the code so you can refer to specific commits. Otherwise, good work!


@fabiosantoscode Node 10 seems to be having [trouble](https://travis-ci.org/fabiosantoscode/terser/jobs/392782541#L3865-L3886) with the JetStream benchmark:

curl -s https://browserbench.org/JetStream/SimplePayload.js | bin/uglifyjs -mc


TeamCity Architecture Team :: Java Platform :: otj-metrics :: BuildTest [Build 215](https://teamcity.otenv.com/viewLog.html?buildId=8748055&buildTypeId=ArchitectureTeam_JavaPlatform_OtjMetrics_BuildTest) is now running


He did not work without it, i think it gets call later in the code.
I could move the ddl_init to later in the code.


TeamCity Architecture Team :: Java Platform :: otj-kafka :: BuildTest [Build 178](https://teamcity.otenv.com/viewLog.html?buildId=8748056&buildTypeId=ArchitectureTeam_JavaPlatform_OtjKafka_BuildTest) is now running


TeamCity Architecture Team :: Java Platform :: otj-filterorder :: BuildTest [Build 34](https://teamcity.otenv.com/viewLog.html?buildId=8748057&buildTypeId=ArchitectureTeam_JavaPlatform_OtjFilterorder_BuildTest) is now running


Obrigado Professor!


3.0 only.


I was hoping moving to just copy_dirs would help, but it causes other issues; https://pastebin.ubuntu.com/p/XdRGDNBrQq/

It shouldn't go to dirs that are symlinks by default.


So as the commit says, it should fix invalid gralloc usages right?
Because HIDL gralloc works with 64bit types only as I see in hardware_interfaces


Note: there is a `suppress_warning do ... end` helper for this in `mspec`.
This is fine too, but the block would have the advantage to only ignore warnings in the specific examples, not all examples.


TeamCity Architecture Team :: Java Platform :: otj-filterorder :: BuildTest [Build 34](https://teamcity.otenv.com/viewLog.html?buildId=8748057&buildTypeId=ArchitectureTeam_JavaPlatform_OtjFilterorder_BuildTest) outcome was **SUCCESS**
Summary: Tests passed: 7 Build time: 00:01:17


TeamCity Architecture Team :: Java Platform :: otj-kafka :: BuildTest [Build 178](https://teamcity.otenv.com/viewLog.html?buildId=8748056&buildTypeId=ArchitectureTeam_JavaPlatform_OtjKafka_BuildTest) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 8 Build time: 00:01:35
### Failed tests
```
com.opentable.kafka.util.OffsetMetricsTest.testMissingTopic: <no details avaliable>

```


TeamCity Architecture Team :: Java Platform :: otj-metrics :: BuildTest [Build 215](https://teamcity.otenv.com/viewLog.html?buildId=8748055&buildTypeId=ArchitectureTeam_JavaPlatform_OtjMetrics_BuildTest) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 00:01:59


Rob, I get that. The cohort is dying correctly. But because of the way this if-else statement is set up, the dead wood does not get transferred to the woody pool if 100% is killed by a partial mortality event. You have to also call ReduceCohort for that transfer to happen. Can you see that in the logic? One way to demonstrate this would be to turn off the ReduceCohort call in the if statement, add only have a ReduceCohort statement in the else part. Then we can run biomass insects and see that there is in fact dead wood transferring to the woody pool. I accidentally did this earlier this week.


Nice!


Consider `isStopped` for clarity.


@joshuous are there supposed to be any changes to wcd9335.h?


Updates #19


(61.257633, 73.378265


Good attempt.


run image_sample.py to generate more sets of 36 random images


Consider `required(isStopped)`


![](https://github.trello.services/images/mini-trello-icon.png) [T127: Tasche mit falschen items zu kombinieren scheint mehr taschen zu produzieren || Item history code scheint nicht korrekt zu sein](https://trello.com/c/3XARhcN9/146-t127-tasche-mit-falschen-items-zu-kombinieren-scheint-mehr-taschen-zu-produzieren-item-history-code-scheint-nicht-korrekt-zu-sei)


Consider `onyIfRunning`


Maybe `<=` to respect "Maximum". That's how I would interpret it.


Kaito, would you kindly reconsider removing the HeavenMS label on ```var servers = ["HeavenMS", "MapleSolaxia", "MoopleDEV", "MetroMS", "BubblesDEV", "ThePackII", "OdinMS", "Contributors"];``` ?

New servers are supposed to hold the names of previous server names, I can't see the point on removing HeavenMS of the name stack at all. If you please, add your server name before HeavenMS, and add a new function with your name on top of it.


15.06.2018 - Primeiro Commit.


^ up one line. Not quite. `hashForPassword(_recipient, _password)`. That makes it so the withdrawing party has to be the remittance agent.


No. Here, the msg.sender can withdraw with knowledge of just the hash and the fact that is has expired.


There are still some bugs with text select right now. Need to continue to refine this behavior.


does nccl work? nccl needs GPU assignment in advance.


What happened to :db/id?


I think your use of "receiver" and "remitter" is confusing. Remember the intended beneficiary is not a party to the onchain transactions. They just walk to the remittance agent "Carol's money shack" and bring their password. Then Carol uses the password given and because she "is" the message sender, she is able to claim ETH. This is presumably witnessed by the beneficiary who can then witness a fair conversion to local currency and leave with cash.


This is totally out of place here. The goal of hc.b.context is for functions that manipulate the context.


NCCL works but it is not initialized until later. So I  will try same thing for DDL.


The circular dep is this reference from routing -> context (for the legacy function that isnt even used, this dep is actually dead on HEAD), context -> routing is legal.


LT & NS


lzap, I had to intent the :isolinux and :files to get the error to go away:

Example:
  test "full host image generation generates via PXELinux type" do
    @host.expects(:generate_pxe_template).with(:PXELinux).returns("Template")
    @host.operatingsystem.expects(:pxe_files).returns([])
    ForemanBootdisk::ISOGenerator.expects(:generate).with({
      :isolinux => "Template",
      :files => []}, anything)
    ForemanBootdisk::ISOGenerator.generate_full_host(@host)


Could this functionality be used to create both on- and off-cpu flame graphs ( see http://www.brendangregg.com/blog/2015-02-26/linux-perf-off-cpu-flame-graph.html ) ?

We are debuggin MySQL CPU scalability problems on OSv (both upstream and our research fork), and we want to find out where MySQL spends its time... cc @nyh


IBM Cloud [toolchain](https://console.ng.bluemix.net/devops/toolchains/c77a559b-d72a-4a4d-a905-14211fb2c5c1?env_id=ibm%3Ayp%3Aus-south "cloud-native-toolchain-tutorial-20180614215947753"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/68997e04-9227-4fd8-aa17-f1a006326c88?env_id=ibm%3Ayp%3Aus-south "sample-cloud-native-toolchain-tutorial-20180614215947753") deployed [sample-cloud-native-toolchain-tutorial-20180614215947753](https://github.com/ganaom/sample-cloud-native-toolchain-tutorial-20180614215947753/releases/tag/deploy-ibm-yp-us-south-ganesh.akarapu-aa.com-prod-20180615-172730 "deploy-ibm-yp-us-south-ganesh.akarapu-aa.com-prod-20180615-172730") to [prod](https://console.bluemix.net/apps/4b1b5bcf-8717-4eed-bbfd-b651b8d2ebf3?env_id=ibm:yp:us-south "ibm:yp:us-south:ganesh.akarapu@aa.com:prod"), including this commit


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.14.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122611&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Compilation error: GlyssenShared\GlyssenShared.csproj (new) Build time: 00:00:37


I don't see you checking that there isn't already a remittance in the `hashPass` slot. If there is, then it would be overwritten by the new one (destroying the first one). Another consideration is you really don't want the same passwords used again because then Carol would be able to take the money before someone arrives with the password (she would already know the password).

A solution to that problem is make sure you don't overwrite and then even after everything is settled, keep them around and prevent overwrites - thus forcing the selection of a new and unique password.


Even though I deleted the “readme-edits” branch per the tutorial instructions, can I still somehow look at it later?


Ok, I've changed those 2 functions at my last commit. Thank you!


> Tim kommentiere deinen Post.

sounds like the Pixelfed translators are incapable of speaking german.


change to std::lower_bound as bsearch can give you any random identical vertex, and you really want a unique redirect to a unique vertex


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.15.0c75be0e7dd8dd07ea097ccccbe208a6190e5b71](http://build.palaso.org/viewLog.html?buildId=122612&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Compilation error: GlyssenShared\GlyssenShared.csproj Build time: 00:00:39


First Commit


i am sorry, i am new on this api, i read all documentation and believed i could do that.
I used this part of documentation above as example and tryed to return all on sale products. So I tryed this two options: products?filter[on_sale]=true and products?on_sale=true  and on all of this calls i get all products as return. So i believe i understand everything wrong.
Can you help me? Is it possible to list all on sale products? Can you show me an example of how can i do that on node.js?

Thank you so much. And i am sorry again, but i am newbie and  trying to learn.

>
> Parameters
> All endpoints accept optional parameters which can be passed as an HTTP query string parameter, e.g. GET /orders?status=completed. There are common parameters and endpoint-specific parameters which are documented along with that endpoint.
>
> Filter Parameter
> All endpoints accept a filter parameter that scopes individual filters using brackets, like date filtering:
>
> GET /orders?filter[created_at_min]=2013-11-01
>
> Multiple filter parameters can be included and intermixed with other parameters:
>
> GET /orders?status=completed&filter[created_at_min]=2013-11-01&filter[created_at_max]=2013-11-30
>
> Note that the following filters are supported for all endpoints except the reports endpoint, which has it’s own set of filters that are documented along with that endpoint.

>


Ah, I understand the problem, but don't quite understand the fix yet.  If reduction = biomass, then it is removed, correct?  The RemoveCohort function should eventually lead to CohortDied in succession with wood transfer.  Could you give a line number?  We're probably talking past each other.


IBM Cloud [toolchain](https://console.ng.bluemix.net/devops/toolchains/c77a559b-d72a-4a4d-a905-14211fb2c5c1?env_id=ibm%3Ayp%3Aus-south "cloud-native-toolchain-tutorial-20180614215947753"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/68997e04-9227-4fd8-aa17-f1a006326c88?env_id=ibm%3Ayp%3Aus-south "sample-cloud-native-toolchain-tutorial-20180614215947753") deployed [sample-cloud-native-toolchain-tutorial-20180614215947753](https://github.com/ganaom/sample-cloud-native-toolchain-tutorial-20180614215947753/releases/tag/deploy-ibm-yp-us-south-ganesh.akarapu-aa.com-test-20180615-173856 "deploy-ibm-yp-us-south-ganesh.akarapu-aa.com-test-20180615-173856") to [test](https://console.bluemix.net/apps/f483d4e9-3ad6-4e15-8413-a1bc74fc36dc?env_id=ibm:yp:us-south "ibm:yp:us-south:ganesh.akarapu@aa.com:test"), including this commit


Checkmarx scan results 15 Jun 2018 20:40 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a23cad998261c81bf63aa21ac681a2e8b5303e7c	@web-flow