@fichtner I'm not sure I follow you here, I've tested this and it worked locally. the end result should be that squid starts a bit later then usually, due to the sleep in setup.sh. the normal bootup with ipfw will pass in the mean time.


# 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)|


类型判断 Error
```
template: Home/default/View/svg.html:9:29:
executing "Home/default/View/svg.html" at <gt $.AdminId 0>: error calling gt: invalid type for comparison
```


Oh, should I use settings.dev in dev? I missed it


currently this means setup.sh in first look and later still ipfw start -> squid start in second loop


With this change, `yarn test` will not be to able to run the test of a specified package, is it intentionally?


Sorry will do. [There was a few times I thought I saw something suspicious that looked like it wouldn't expand, but I didn't mean to introduce any of that myself.]


@cweithoeft My Python SDK for this is named "Python 3.6 conda-stix2" because I use conda and the virtual environment for this project is named stix2. If you use something other than conda I am happy to rename to something more abstract like "Python 3.6 stix2", just let me know.


ipfw is started using plugins, or am I mistaken here? https://github.com/opnsense/core/blob/c61babf7bb7db6fc6a1196899a637ec44baa8934/src/etc/inc/plugins.inc.d/ipfw.inc#L35-L46


s/DNS/DSN/ plz!


#6888-EngRefEquationFixPart2 (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1717 of 1717 tests passed, 0 test warnings)

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


It was my original idea for W'all.
Y'know, like a chair shot in pro wrestling.


Alas that method is protected.
```
[ 43%] Building CXX object Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/qMRMLPlotView.cxx.o
In file included from /home/pieper/slicer4/latest/Slicer/Libs/MRML/Widgets/qMRMLPlotView.cxx:53:0:
/home/pieper/slicer4/latest/Slicer-superbuild/VTKv9/Charts/Core/vtkChartXY.h: In member function \u2018void qMRMLPlotView::RemovePlotSelections()\u2019:
/home/pieper/slicer4/latest/Slicer-superbuild/VTKv9/Charts/Core/vtkChartXY.h:285:8: error: \u2018void vtkChartXY::ReleasePlotSelections()\u2019 is protected
   void ReleasePlotSelections();
        ^
/home/pieper/slicer4/latest/Slicer/Libs/MRML/Widgets/qMRMLPlotView.cxx:1084:40: error: within this context
   this->chart()->ReleasePlotSelections();
                                        ^
Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/build.make:1725: recipe for target 'Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/qMRMLPlotView.cxx.o' failed
make[2]: *** [Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/qMRMLPlotView.cxx.o] Error 1
CMakeFiles/Makefile2:2556: recipe for target 'Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/all' failed
make[1]: *** [Libs/MRML/Widgets/CMakeFiles/qMRMLWidgets.dir/all] Error 2
Makefile:162: recipe for target 'all' failed
make: *** [all] Error 2

```


py-gnureadline was a working workaround for buggy Apple's libedit behaviour (https://trac.macports.org/ticket/48807)


Move this var, and probably some others to defaults. We want to be able to use `eas.settings.dev` in the development server.


Unless there are other ideas probably best to just stub this out (with a warning message?).


feature are copied from 3.1.0, guess this was by mistake.


WHATS THE CODE?


Removing that line "unfixes" the problem that @naveen2507 had addressed by adding that line as part of #428 (09c9f272e9498d7463938973f9743883ded8133b).

So we are trading one problem for another one here. The must be a better fix.


Reset for Interrupt Trigger is missed


The little method that could


Copy of this build can be found at https://www.mewbuilds.com/build/a3cbb0909736b82464536347b413104913641b13/index.html


# 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)|


you might want to make it not go through the ground...


I also think is an interesting patch/idea...

Not to be a "negative Nellie" I don't think this patch will influence game play much...and the STC was a lucky run.

The reason being is that bestMoveIsDraw becomes true only 56 times during bench.

0 hits at depth 25 at the start position.


here finished MVP #1


Agry


oops :-)


Looking good. I always prefer keeping my braces nice and tight. I helps to fit more into a smaller vertical space.


Hey, I like the looks of this! A few thoughts:

- Exporting `MagentoRouter` is exactly the right thing to do. That should be a public API.
- Pushing the ugly Webpack hack away into that router is also very nice; thank you for doing that.
- I'm not sure if the CSS should be so opinionated; if we want this to end up a reusable construct, and we do, then it might be better to style it minimally and expose its classes via `classify`.
- A few other inline comments.

Looking forward to seeing this as a PR.


🤐 I cannot say, but feel free to [guess](https://2018-asc-secret.glitch.me/)!


By the way, @gorhill should we convert all existing `script:inject` filters to `+js` or do you want us to wait?


Why do you prefer to pass an instance here, instead of passing the Loader constructor itself? Are you trying to simplify the API by disallowing props in a custom loader? If so, I'm listening, but I think a custom loader will at least want some context about what it's waiting for.


Culprit for crashes found in #18


What was your reasoning for switching from JSX to direct invocation here? We want to use JSX whenever possible; if we call directly like this, we should add an explanatory comment.


Were you running into a bug with trailing slashes? If so, could we contain this to the router and not make it Peregrine's responsibility?


It looks like you maybe left some debug here.


I posted a mistaken comment before. Good on you for moving this into the router.


This is not a RootComponent! A RootComponent should handle a Magento content type. This belongs in `src/components`, not `src/RootComponents`.


Beautiful style and syntax, super easy to read! One small thing is, if you use semi-colons at all, make sure you are using them in every place expected - so the end of lines 4-6 and 8.


In my local fix I also included the networks: in influxdb and grafana.
And ...

```
version: "3"

networks:
   mainflux-base-net:
      driver: bridge

services:
...
```
...


Might be easier to grab these elements with the ID rather than having to dig into the input properties here - but both ways work!


Asking @jimbo to take a look here, to determine whether this classnames API can be made more opener.


Looks great @hmesander 🎉


y'all realise this commit is from like 7 months ago? lol


This seems unrelated to the HOC change. Did it turn out to be necessary for some weird reason?


Fix manual sync for SVG images.


Love the way you used the template literal here!


Since these won't be reassigned, I'd use a const here instead of let


@RoDuth , keep in mind my mathematical foundation, so don't please say "Always" when you mean "often enough" ;-)

About the csv export, I only consider it useful for backup.  Restoring an old backup will destroy all existing information in the tables you are restoring, and the links to the other tables will only make sense if the object id match.  This makes sense and sounds acceptable from a backup, not for importing contributed data.  Given this consideration, I definitely discourage its use as an export.  Please notice: not in absolute, only in particular if you are looking for ways to export stuff that you want to share with others.  If you quote me partially, or out of context, you will misunderstand me.  Exporting plants where the accession code is hidden behind a numeric id, and the plant genus behind three such jumps, I really have trouble calling it a viable export for sharing your data, I'm sure you see the point.

About the xslt reports plugin, I do not know how to use it, it lacks technical and user documentation.  From time to time users ask me about it, and I can't point them to documentation, nor am I able to assist them.  A garden you help with ghini would be able to provide at least some user documentation, but after years waiting and several times asking, they don't seem to be interested in contributing it, or are they?  IIRC, they aren't even willing to share their templates for me to review/adapt.  So, I guess the plugin is going to stay there until I make order in the GUI concerning reports, and I do not know when that will happen.  I always hope to hear good news from them, any time, also through you.  I don't like keeping undocumented untested code in my software, that I am not able to support.

anyhow, in version 3.1 the csv export is now called 'backup' and we have a new, dynamic csv export.

I agree that partial database restore is desirable, so I think it will stay as it is.

I do not think that there is any added safety behind requiring a user to click "yes, I know" more than once.  I think most users will feel abused after the third time we ask the same question.

in what you write you refer to `txt` not desirable as an extension, right?


[inst/examples/example_experiment.R:91:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_experiment.R#L91) *style:* **Commented code should be removed.**
```r
  # library(raster)
    ^~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:92:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_experiment.R#L92) *style:* **Commented code should be removed.**
```r
  # beginCluster(20) # if you have multiple clusters available, use them here to save time
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:96:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_experiment.R#L96) *style:* **Commented code should be removed.**
```r
  # endCluster() # end the clusters
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:152:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_experiment.R#L152) *style:* **Commented code should be removed.**
```r
  # fires <- lapply(sims, function(x) x$landscape$fires) %>% stack
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:49:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L49) *style:* **Commented code should be removed.**
```r
  # cl <- makeCluster(detectCores() - 1) # not implemented yet in DEoptim
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:84:4:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L84) *style:* **Commented code should be removed.**
```r
  #stopCluster(cl) # not yet implemented, waiting for DEoptim
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:107:34:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L107) *style:* **Variable or function name should be lowerCamelCase.**
```r
    params(sim1)$caribouMovement$N <- pars[2]
                                 ^
```
[inst/examples/example_POM.R:114:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
    nPattern_Out <- caribouFn(out$caribou)
    ^~~~~~~~~~~~
```
[inst/examples/example_POM.R:120:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L120) *style:* **Commented code should be removed.**
```r
    # cat(minimizeFn)
      ^~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:121:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L121) *style:* **Commented code should be removed.**
```r
    # cat(" ")
      ^~~~~~~~
```
[inst/examples/example_POM.R:122:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L122) *style:* **Commented code should be removed.**
```r
    # cat(pars)
      ^~~~~~~~~
```
[inst/examples/example_POM.R:123:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/examples/example_POM.R#L123) *style:* **Commented code should be removed.**
```r
    # cat("\n")
      ^~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:54:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L54) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.caribouMovement <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:86:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L86) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + P(sim)$.plotInterval, "caribouMovement", "plot", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:95:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L95) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + P(sim)$.plotInterval, "caribouMovement", "plot", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:102:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L102) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + P(sim)$.saveInterval, "caribouMovement", "save", .last() + 1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:114:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:121:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L121) *style:* **Variable or function name should be lowerCamelCase.**
```r
  N <- P(sim)$N
  ^
```
[inst/sampleModules/caribouMovement/caribouMovement.R:122:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L122) *style:* **Variable or function name should be lowerCamelCase.**
```r
  IDs <- as.character(1:N)
  ^~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:138:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/caribouMovement/caribouMovement.R#L138) *style:* **Variable or function name should be lowerCamelCase.**
```r
Move <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:20:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L20) *style:* **Lines should not be more than 100 characters.**
```r
    person(c("Alex", "M"), "Chubaty", email = "alexander.chubaty@canada.ca", role = c("aut", "cre")),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:21:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L21) *style:* **Lines should not be more than 100 characters.**
```r
    person(c("Eliot", "J", "B"), "McIntire", email = "eliot.mcintire@canada.ca", role = c("aut", "cre")),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:35:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L35) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("persistprob", "numeric", 0.00, 0, 1, "probability of fire persisting in a pixel"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:37:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L37) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("spreadprob", "numeric", 0.225, 0.05, 0.5, "probability of fire spreading into a pixel"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:38:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L38) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("startTime", "numeric", start(sim) + 1, 0, end(sim), "time of initial fire ignition"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:39:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L39) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", start(sim), start(sim), end(sim) + 1, "time to schedule first plot event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:41:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L41) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "time to schedule first save event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:42:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L42) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInterval", "numeric", NA_real_, NA, NA, "time interval between save events")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:59:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L59) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.fireSpread <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:87:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L87) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim), "fireSpread", "stats") # do stats immediately following burn
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:142:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L142) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:146:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L146) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- raster(extent(landscapes), ncol = ncol(landscapes),
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:150:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L150) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- setValues(Fires, 0)
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:153:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L153) *style:* **Variable or function name should be lowerCamelCase.**
```r
  landscapes$Fires <- Fires
             ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:159:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L159) *style:* **Variable or function name should be lowerCamelCase.**
```r
Burn <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:162:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L162) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- spread(landscapes[[1]],
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:174:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L174) *style:* **Variable or function name should be lowerCamelCase.**
```r
  landscapes$Fires <- Fires
             ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:181:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/fireSpread/fireSpread.R#L181) *style:* **Variable or function name should be lowerCamelCase.**
```r
Stats <- function(sim) {
^~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:33:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L33) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("inRAM", "logical", FALSE, TRUE, FALSE, "should the raster be stored in memory?"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:34:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L34) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("nx", "numeric", 100L, 10L, 500L, "size of map (number of pixels) in the x dimension"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:35:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L35) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("ny", "numeric", 100L, 10L, 500L, "size of map (number of pixels) in the y dimension"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:37:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L37) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", start(sim), start(sim), NA, "time to schedule first plot event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:38:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L38) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInterval", "numeric", NA_real_, NA, NA, "time interval between plot events"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:39:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L39) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "time to schedule first save event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:40:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L40) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInterval", "numeric", NA_real_, NA, NA, "time interval between save events"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:41:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L41) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".useCache", "logical", FALSE, c("init", "plot"), NA, "should the module result be cached for future use")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:55:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L55) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.randomLandscapes <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:75:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L75) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + P(sim)$.saveInterval, "randomLandscapes", "save", .last() + 1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:86:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L86) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:100:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/randomLandscapes/randomLandscapes.R#L100) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DEM <- gaussMap(template, scale = 300, var = 0.03, speedup = speedup, inMemory = inMemory)
    ^~~
```
[inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R:17:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R#L17) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", NA_real_, NA, NA, "This describes the simulation time at which the first plot event should occur"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R:18:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R#L18) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "This describes the simulation time at which the first save event should occur")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:231:50:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L231) *style:* **Commas should always have a space after.**
```r
          cat(crayon::blue("  Using ", fromWhere," copy of", cur$moduleName, "module\n"))
                                                 ^
```
[R/cache.R:242:50:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L242) *style:* **Commas should always have a space after.**
```r
          cat(crayon::blue("  Using ", fromWhere," copy of", cur$eventType, "event in",
                                                 ^
```
[R/cache.R:289:10:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L289) *style:* **Commented code should be removed.**
```r
        #checkPath(cacheRepo, create = TRUE) #SpaDES dependency
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:413:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L413) *style:* **Lines should not be more than 100 characters.**
```r
        #   makes soft copy of all objects, i.e., they have the identical objects, which are pointers only
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:441:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L441) *style:* **Lines should not be more than 100 characters.**
```r
        list2env(mget(lsObjectEnv[lsObjectEnv %in% createOutputs | lsObjectEnv %in% expectsInputs], envir = object@.envir), envir = object2@.envir)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:451:12:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L451) *style:* **Commented code should be removed.**
```r
          #object2@events <- unique(rbindlist(list(object@events, object2@events)))
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:455:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L455) *style:* **Lines should not be more than 100 characters.**
```r
        # This is for objects that are not in the return environment yet because they are unrelated to the
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:468:10:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L468) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      for(atts in attrsToGrab) {
         ^
```
[R/cache.R:472:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L472) *style:* **Commented code should be removed.**
```r
      # attr(object2, "tags") <- attr(object, "tags")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:473:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L473) *style:* **Commented code should be removed.**
```r
      # attr(object2, "call") <- attr(object, "call")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:474:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L474) *style:* **Commented code should be removed.**
```r
      # attr(object2, "function") <- attr(object, "function")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:507:66:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L507) *style:* **Variable or function name should be lowerCamelCase.**
```r
  setGeneric(".addTagsToOutput", function(object, outputObjects, FUN) {
                                                                 ^~~
```
[R/cache.R:530:48:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L530) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(object, outputObjects, FUN, preDigestByClass) {
                                               ^~~
```
[R/cache.R:622:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L622) *style:* **Variable or function name should be lowerCamelCase.**
```r
objSize.simList <- function(x, quick = getOption("reproducible.quick", FALSE)) {
^~~~~~~~~~~~~~~
```
[R/cache.R:649:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L649) *style:* **Variable or function name should be lowerCamelCase.**
```r
makeMemoiseable.simList <- function(x) {
^~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:657:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/cache.R#L657) *style:* **Variable or function name should be lowerCamelCase.**
```r
unmakeMemoiseable.simList_ <- function(x) {
^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkpoint.R:36:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/checkpoint.R#L36) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.checkpoint <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:86:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L86) *style:* **Lines should not be more than 100 characters.**
```r
                xAsCall <- .isLastLineSim(x = x, xAsString = bb[seq(funStarts[yy], funEnds[yy + 1])])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:90:16:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L90) *style:* **Commented code should be removed.**
```r
              #y = strsplit(bb[funStarts[yy]], split = "\\s+")[[1]][1]
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:91:53:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L91) *style:* **Commas should always have a space after.**
```r
              y <- paste0(cantCodeCheckMessage, "'",bb[funStarts[yy]], "'")
                                                    ^
```
[R/code-checking.R:144:39:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L144) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (identical(type, "returnSim")) { # This is intended for only the last line of a function
                                      ^
```
[R/code-checking.R:182:48:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L182) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      } else if (identical(x[[1]], quote(`<-`)) ) {
                                               ^
```
[R/code-checking.R:185:62:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L185) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
            if (any(grepl(x[[2]][[2]], pattern = ".envir"))) {# i.e., sim@.envir
                                                             ^
```
[R/code-checking.R:202:19:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L202) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          x <- x[-(1:2)]
                  ^
```
[R/code-checking.R:218:29:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L218) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          x[[2]] <- x[[2]][-(1:2)]
                            ^
```
[R/code-checking.R:286:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L286) *style:* **Lines should not be more than 100 characters.**
```r
  allChecks <- list(simAssigns = simAssigns, simGets = simGets, returnsSim = returnsSim, assignToSim = assignToSim, fg = fg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:319:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L319) *style:* **Lines should not be more than 100 characters.**
```r
    allChecks[anyCantCodeCheck] <- lapply(names(cantCodeCheck[anyCantCodeCheck]), function(objName) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:368:51:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L368) *style:* **Commas should always have a space after.**
```r
                                              " ",verb," declared in outputObjects, ",
                                                  ^
```
[R/code-checking.R:368:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L368) *style:* **Commas should always have a space after.**
```r
                                              " ",verb," declared in outputObjects, ",
                                                       ^
```
[R/code-checking.R:384:20:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L384) *style:* **Commas should always have a space after.**
```r
               " ",verb," declared in inputObjects, ",
                   ^
```
[R/code-checking.R:384:25:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L384) *style:* **Commas should always have a space after.**
```r
               " ",verb," declared in inputObjects, ",
                        ^
```
[R/code-checking.R:395:90:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L395) *style:* **Commas should always have a space after.**
```r
                                       paste0(paste(missingFrmMod, collapse = ", "), " ",verb,
                                                                                ^
```
[R/code-checking.R:440:75:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L440) *style:* **Commas should always have a space after.**
```r
             " but only for the 'get' functions, not the 'set' function ","
                                                                          ^
```
[R/code-checking.R:454:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L454) *style:* **Lines should not be more than 100 characters.**
```r
    missingInMetadata <- simAssignsNotInDotInputObjects[!(simAssignsNotInDotInputObjects %in% outputObjNames)]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:463:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L463) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                       ^
```
[R/code-checking.R:463:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L463) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                                   ^
```
[R/code-checking.R:464:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L464) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in outputObjects")
                                ^
```
[R/code-checking.R:464:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L464) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in outputObjects")
                                            ^
```
[R/code-checking.R:471:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L471) *style:* **Lines should not be more than 100 characters.**
```r
    missingInMetadata <- simAssignsInDotInputObjects[!(simAssignsInDotInputObjects %in% inputObjNames)]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:480:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L480) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                       ^
```
[R/code-checking.R:480:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L480) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                                   ^
```
[R/code-checking.R:481:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L481) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                ^
```
[R/code-checking.R:481:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L481) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                            ^
```
[R/code-checking.R:503:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L503) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," used from sim inside ",
                       ^
```
[R/code-checking.R:503:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L503) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," used from sim inside ",
                                   ^
```
[R/code-checking.R:504:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L504) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                ^
```
[R/code-checking.R:504:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L504) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                            ^
```
[R/code-checking.R:516:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L516) *style:* **Commas should always have a space after.**
```r
        " ",verb," used from sim inside ",
            ^
```
[R/code-checking.R:516:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L516) *style:* **Commas should always have a space after.**
```r
        " ",verb," used from sim inside ",
                 ^
```
[R/code-checking.R:518:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L518) *style:* **Commas should always have a space after.**
```r
        ", but ",verb," not declared in inputObjects"
                 ^
```
[R/code-checking.R:518:23:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L518) *style:* **Commas should always have a space after.**
```r
        ", but ",verb," not declared in inputObjects"
                      ^
```
[R/code-checking.R:661:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L661) *style:* **Commented code should be removed.**
```r
    #unique(pd[outerWh, "line1"])
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:665:34:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L665) *style:* **Commas should always have a space after.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                 ^
```
[R/code-checking.R:665:64:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L665) *style:* **Commas should always have a space after.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                                               ^
```
[R/code-checking.R:665:79:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/code-checking.R#L665) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                                                              ^
```
[R/copy.R:37:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/copy.R#L37) *style:* **Variable or function name should be lowerCamelCase.**
```r
            sim_ <- object
            ^~~~
```
[R/downloadData.R:54:96:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L54) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
    io <- .parseModulePartial(sim, modules = list(module), defineModuleElement = "inputObjects" )
                                                                                ^
```
[R/downloadData.R:242:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L242) *style:* **Commented code should be removed.**
```r
    # parsedModule <- parse(file = file.path(path, module, paste0(module, '.R')))
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:243:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L243) *style:* **Commented code should be removed.**
```r
    # urls <- .getSourceURL(pattern = fileToDownload, x = parsedModule)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:250:8:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L250) *style:* **Commented code should be removed.**
```r
      #urls <- moduleMetadata(module, path)$inputObjects$sourceURL
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:268:27:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L268) *style:* **Commas should always have a space after.**
```r
    chksums2 <- chksums[0,]
                          ^
```
[R/downloadData.R:269:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L269) *style:* **Commented code should be removed.**
```r
    #children <- moduleMetadata(module, path)$childModules
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:272:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L272) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
            ^
```
[R/downloadData.R:272:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L272) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                 ^
```
[R/downloadData.R:272:54:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L272) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                                                     ^
```
[R/downloadData.R:272:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/downloadData.R#L272) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                                                       ^
```
[R/experiment.R:200:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L200) *style:* **Commented code should be removed.**
```r
    #sim <- Copy(sim, objects = FALSE, queues = TRUE)
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:215:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L215) *style:* **Commented code should be removed.**
```r
    # cl <- tryCatch(getCluster(), error = function(x) NULL)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:216:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L216) *style:* **Commented code should be removed.**
```r
    # on.exit(if (!is.null(cl)) returnCluster(), add = TRUE)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:218:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L218) *style:* **Commented code should be removed.**
```r
    #if (length(modules) == 0) modules <- list(modules(sim)[-(1:4)])
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:222:11:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L222) *style:* **Commented code should be removed.**
```r
        # unlist(params[paramsTmp], recursive = FALSE)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:314:58:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/experiment.R#L314) *style:* **Commented code should be removed.**
```r
            params(sim_)[[mod[x]]][[param[[x]]]] <- val #factorialExp[ind,x]
                                                         ^~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:230:74:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/helpers.R#L230) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                             paste0("package:", .pkgEnv$corePackagesVec[-(1:2)]))
                                                                         ^
```
[R/helpers.R:260:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/helpers.R#L260) *style:* **Variable or function name should be lowerCamelCase.**
```r
all.equal.simList <- function(target, current, ...) {
^~~~~~~~~~~~~~~~~
```
[R/load.R:198:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/load.R#L198) *style:* **Lines should not be more than 100 characters.**
```r
                names(argument) <- c(nam, names(formals(getFromNamespace(loadFun[y], loadPackage[y])))[1])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/misc-methods.R:467:55:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/misc-methods.R#L467) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
          (inherits(get(w, envir = envir), "integer")) ) {
                                                      ^
```
[R/module-define.R:719:63:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-define.R#L719) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      exts <- na.omit(match(file_ext(fl), .fileExts[, "exts"]) )
                                                              ^
```
[R/module-dependencies-class.R:53:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-class.R#L53) *style:* **Variable or function name should be lowerCamelCase.**
```r
            out.df <- data.frame(
            ^~~~~~
```
[R/module-dependencies-class.R:172:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-class.R#L172) *style:* **Commented code should be removed.**
```r
    #   warning("input object data.frame should use colnames sourceURL.")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/module-dependencies-class.R:174:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-class.R#L174) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
    if ( !all(colnames(.outputObjects()) %in% colnames(object@outputObjects)) ) {
        ^
```
[R/module-dependencies-class.R:174:78:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-class.R#L174) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
    if ( !all(colnames(.outputObjects()) %in% colnames(object@outputObjects)) ) {
                                                                             ^
```
[R/module-dependencies-methods.R:43:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L43) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DT <- .depsEdgeListMem(deps, plot)
    ^~
```
[R/module-dependencies-methods.R:56:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L56) *style:* **Variable or function name should be lowerCamelCase.**
```r
  sim.in <- sim.out <- data.table(objectName = character(0),
  ^~~~~~
```
[R/module-dependencies-methods.R:56:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L56) *style:* **Variable or function name should be lowerCamelCase.**
```r
  sim.in <- sim.out <- data.table(objectName = character(0),
            ^~~~~~~
```
[R/module-dependencies-methods.R:61:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L61) *style:* **Variable or function name should be lowerCamelCase.**
```r
      z.in <- as.data.table(x@inputObjects)[, .(objectName, objectClass)]
      ^~~~
```
[R/module-dependencies-methods.R:62:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L62) *style:* **Variable or function name should be lowerCamelCase.**
```r
      z.out <- as.data.table(x@outputObjects)[, .(objectName, objectClass)]
      ^~~~~
```
[R/module-dependencies-methods.R:65:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L65) *style:* **Variable or function name should be lowerCamelCase.**
```r
        sim.in <<- rbindlist(list(sim.in, z.in), use.names = TRUE)
        ^~~~~~
```
[R/module-dependencies-methods.R:68:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L68) *style:* **Variable or function name should be lowerCamelCase.**
```r
        sim.out <<- rbindlist(list(sim.out, z.out), use.names = TRUE)
        ^~~~~~~
```
[R/module-dependencies-methods.R:80:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L80) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DT <- dx[, list(from = module, to = i.module,
    ^~
```
[R/module-dependencies-methods.R:83:15:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L83) *style:* **Variable or function name should be lowerCamelCase.**
```r
    if (plot) DT <- DT[!duplicated(DT[, 1:2, with = FALSE]), ]
              ^~
```
[R/module-dependencies-methods.R:85:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L85) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DT <- data.table(from = character(0), to = character(0),
    ^~
```
[R/module-dependencies-methods.R:164:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L164) *style:* **Variable or function name should be lowerCamelCase.**
```r
    M <- distances(simGraph, mode = "out")
    ^
```
[R/module-dependencies-methods.R:196:39:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L196) *style:* **`%>%` should always have a space before it and a new line after it, unless the full pipeline fits on one line.**
```r
      missingObjects <- simEdgeList %>% filter(from != to) %>%
                                    ~~^
```
[R/module-dependencies-methods.R:210:26:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L210) *style:* **`%>%` should always have a space before it and a new line after it, unless the full pipeline fits on one line.**
```r
    return(newEdgeList %>% data.table() %>% setorder("from", "to", "objName"))
                       ~~^
```
[R/module-dependencies-methods.R:246:79:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-dependencies-methods.R#L246) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
              loadOrder <- names(simGraph[[tsort, ]]) %>% .[!(. %in% "_INPUT_" )]
                                                                              ^
```
[R/module-repository.R:28:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L28) *style:* **Lines should not be more than 100 characters.**
```r
#'          (\url{http://spades-core.predictiveecology.org/articles/ii-modules.html#module-directory-structure-modulename}),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/module-repository.R:52:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L52) *style:* **Lines should not be more than 100 characters.**
```r
    zipFiles <- grep(file.path(name, "data"), zipFiles, invert = TRUE, value = TRUE) # remove any zip in data folder
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/module-repository.R:293:8:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L293) *style:* **Commented code should be removed.**
```r
      #versionWarning(name, version)
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/module-repository.R:305:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L305) *style:* **Lines should not be more than 100 characters.**
```r
        GET(zip, ua, config = list(config(token = pat)), write_disk(localzip, overwrite = overwrite))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/module-repository.R:401:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L401) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/module-repository.R:402:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/module-repository.R#L402) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/moduleMetadata.R:75:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/moduleMetadata.R#L75) *style:* **Commented code should be removed.**
```r
    #metadata <- eval(parse(text = x)) # can't be used because can't evaluate start(sim)
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-diagrams.R:78:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L78) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DT <- tail(completed(sim), n)
    ^~
```
[R/plotting-diagrams.R:173:28:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L173) *style:* **`%>%` should always have a space before it and a new line after it, unless the full pipeline fits on one line.**
```r
        sapply(ll, NROW) %>% sum() %>% `*`(., 26L)
                         ~~^
```
[R/plotting-diagrams.R:325:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L325) *style:* **Variable or function name should be lowerCamelCase.**
```r
      PlotRemovingDots <- function(modDia, plotFn, axes, ...,
      ^~~~~~~~~~~~~~~~
```
[R/plotting-diagrams.R:326:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L326) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.color,
                                   ^~~~~~~~~~~~
```
[R/plotting-diagrams.R:327:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L327) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.size,
                                   ^~~~~~~~~~~
```
[R/plotting-diagrams.R:328:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L328) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.size2,
                                   ^~~~~~~~~~~~
```
[R/plotting-diagrams.R:329:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L329) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.shape,
                                   ^~~~~~~~~~~~
```
[R/plotting-diagrams.R:330:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L330) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.label.cex,
                                   ^~~~~~~~~~~~~~~~
```
[R/plotting-diagrams.R:331:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L331) *style:* **Variable or function name should be lowerCamelCase.**
```r
                                   vertex.label.family,
                                   ^~~~~~~~~~~~~~~~~~~
```
[R/plotting-diagrams.R:345:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting-diagrams.R#L345) *style:* **Lines should not be more than 100 characters.**
```r
          c(nchar(namesModDia)^0.8 * 10) # use exponential to stretch out, and multiplication to make all bigger
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:6:31:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L6) *style:* **Variable or function name should be lowerCamelCase.**
```r
                              na.color, zero.color, length){
                              ^~~~~~~~
```
[R/plotting.R:6:41:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L6) *style:* **Variable or function name should be lowerCamelCase.**
```r
                              na.color, zero.color, length){
                                        ^~~~~~~~~~
```
[R/plotting.R:37:25:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L37) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        na.color, zero.color, length) {
                        ^~~~~~~~
```
[R/plotting.R:37:35:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L37) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        na.color, zero.color, length) {
                                  ^~~~~~~~~~
```
[R/plotting.R:86:22:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L86) *style:* **Put spaces around all infix operators.**
```r
      eval(parse(text=deparse(elems[[useElem]])), envir=tmp@.envir),
                    ~^~
```
[R/plotting.R:86:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L86) *style:* **Put spaces around all infix operators.**
```r
      eval(parse(text=deparse(elems[[useElem]])), envir=tmp@.envir),
                                                      ~^~
```
[R/plotting.R:87:12:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L87) *style:* **Put spaces around all infix operators.**
```r
      error=function(x) eval(parse(text=deparse(elems[[useElem]])), envir=envir))
          ~^~
```
[R/plotting.R:87:40:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L87) *style:* **Put spaces around all infix operators.**
```r
      error=function(x) eval(parse(text=deparse(elems[[useElem]])), envir=envir))
                                      ~^~
```
[R/plotting.R:87:74:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/plotting.R#L87) *style:* **Put spaces around all infix operators.**
```r
      error=function(x) eval(parse(text=deparse(elems[[useElem]])), envir=envir))
                                                                        ~^~
```
[R/POM.R:131:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L131) *style:* **Lines should not be more than 100 characters.**
```r
#' @param objFnCompare Character string. Either, "MAD" or "RMSE" indicating that inside the objective
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/POM.R:189:12:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L189) *style:* **Variable or function name should be lowerCamelCase.**
```r
           NaNRetries = NA, logObjFnVals = FALSE, weights, useLog = FALSE) {
           ^~~~~~~~~~
```
[R/POM.R:199:25:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L199) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        NaNRetries, logObjFnVals, weights, useLog) {
                        ^~~~~~~~~~
```
[R/POM.R:216:28:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L216) *style:* **Variable or function name should be lowerCamelCase.**
```r
    if (is.na(NaNRetries)) NaNRetries <- 1
                           ^~~~~~~~~~
```
[R/POM.R:240:11:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L240) *style:* **Variable or function name should be lowerCamelCase.**
```r
          sim_ <- Copy(sim)
          ^~~~
```
[R/POM.R:258:66:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L258) *style:* **Put spaces around all infix operators.**
```r
          envPOMCalled <- sys.frame(min(grep("POM", sys.calls()))-1)
                                                                ~^~
```
[R/POM.R:325:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L325) *style:* **Lines should not be more than 100 characters.**
```r
          cat(format(unlist(lapply(objectiveRes, function(x) x[["value"]])), digits = 4), dep = "\t")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/POM.R:363:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/478e68ede09cf03d81986515930b96cd72e2aba6/R/POM.R#L363) *style:* **Lines should not be more than 100 characters.**
```r
      par[whP] <- unlist(deps[[modName]]@parameters$default[deps[[modName]]@parameters$paramName == names(P(sim, modName)[whParamsByMod[whP]])])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Since your other functions are using ES5 syntax, maybe change this one to ES5 as well? I'd recommend just being consistent within each file.


Awesome job @hmesander - a couple notes for you, let me know if you would like to follow up.


I really appreciate this as an experiment, but I have to say I don't like the API. It seems to save some LOC at the expense of obviousness. The function-as-callback pattern of `Query` seems more readable to me; it's still fairly terse, and it allows you to implement custom state changing logic very naturally.

I've made a lot of things like this that didn't feel necessary after a few passes, so I'm right there with you. I think this one is not quite worth the refactor. Let me know if there's something I'm missing, of course!


This directive is for RootComponents only. You'll definitely want to remove it from non-RootComponents.


welp i guess that didn't work. oh well, not like "ducks" will be the password on prod anyways


If you create a global instance Random random = new Random(), the randomization will be the same "level" as random that you're now. I mean, I think I get what you mean but the pseudo-random generator is random enough for what you want to accomplish here. Anyway, it's just bad practice on how it is used in this commit but will not really affect performance that much. Just wanted to point that out before it becomes an habit!

But Good luck! You're doing great!


I like the way you abstracted this out!


Created multidev environment [yaw-d7-site#ci-9](https://dashboard.pantheon.io/sites/853a0c69-1521-4bb0-8af2-94ed5f6f208e#ci-9).

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


Did you consider using `FOLLY_ATTR_WEAK` from [here](https://github.com/facebook/folly/blob/master/folly/CPortability.h#L155)


I'd recommend grabbing these elements by ID here - readability and more commonly used😊


Typically you will see this on the first line of the function it lives in.


@jude-lawson Looks awesome - beautiful style and syntax, consistent spacing and use of semi-colons.


try this:
>>> pd.Series([1.,2,3] * ureg.meter,dtype=PintType()).values
PintArray([1.0 meter, 2.0 meter, 3.0 meter], dtype='pint')

That's really nice! It might be possible to make it something like this:
pd.Series([1.,2,3] ,dtype=ureg.meter)

By making the ureg.meter an extensiontype? Not sure that's worth the complexity for a small benefit, at least for a first version.


Thanks. Fixed: https://github.com/talwrii/i3-clever-layout/commit/af13e7723dea2024cc0688fa165bf0952f3915fc


This works and is great - another common pattern you will see is just appending all that into a container, using a template literal you can give those HTML tags ids/classes, etc. and interpolate variables.


@jude-lawson looks great💯


i wrapped the UpdateSymbolTable Method now.

   public class SymbolpatcherWrapper
    {

        private static SymbolTable initalCryptoKeys = null;

        /// <summary>
        /// use own symbolpatcher method.
        ///
        /// restore initial crypto keys after patching
        /// </summary>
        public static void UpdateSymbolTable()
        {

			SymbolPatcher64.UpdateSymbolTable(MemoryContext.Current);

			if(initalCryptoKeys == null) {
				//save inital crypto keys
				initalCryptoKeys = new SymbolTable();
				copyCryptoKeys(SymbolTable.Current, initalCryptoKeys);
			}
			else
			{
				//restore inital cryptokeys
				copyCryptoKeys(initalCryptoKeys, SymbolTable.Current);
			}
        }

        private static void copyCryptoKeys(SymbolTable source, SymbolTable dest)
        {
            dest.CryptoKeys.RActorACDID = source.CryptoKeys.RActorACDID;
            dest.CryptoKeys.PlayerDataActorID = source.CryptoKeys.PlayerDataActorID;
            dest.CryptoKeys.PlayerDataACDID = source.CryptoKeys.PlayerDataACDID;
            dest.CryptoKeys.ActorID = source.CryptoKeys.ActorID;
            dest.CryptoKeys.ACDActorSNO = source.CryptoKeys.ACDActorSNO;
            dest.CryptoKeys.ActorType = source.CryptoKeys.ActorType;
            dest.CryptoKeys.SSceneID = source.CryptoKeys.SSceneID;
            dest.CryptoKeys.SWorldID = source.CryptoKeys.SWorldID;
            dest.CryptoKeys.LevelAreaSNO = source.CryptoKeys.LevelAreaSNO;
            dest.CryptoKeys.TimedEventSNO = source.CryptoKeys.TimedEventSNO;
            dest.CryptoKeys.RActorActorSNO = source.CryptoKeys.RActorActorSNO;
        }
    }
```


That was a bug. I set DB_HOST and not DB_NAME


Hi folks, asserts in "production code" are not recommended in python, mainly because if the python code is "compiled" with some specific performance flags, then asserts are stripped from the code, which would impact on the project functionality.


add pug/jade  + boostrap in layout  in this version


Created multidev environment [yaw-d7-site#ci-12](https://dashboard.pantheon.io/sites/853a0c69-1521-4bb0-8af2-94ed5f6f208e#ci-12).

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


Fixed.

https://github.com/talwrii/i3-clever-layout/commit/cd472da9c867d8885ce0c81d1b3040dabf5299b2


This workaround allows the build to complete:

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=27349


no, rc.freebsd on bootup


Hi, do you are planing to create  a PR for this feature? Or is just prove of concept.

Regards


Thanks for quicknotes, didn't know for it! Will take a look!

Regarding SQL, well, I am without experience writing NC queries, so I found some example with ORM approach and used that. To be honest, it was pain in the ass to write it:) Yours seems more lightweight. I would prefer ORM-style (with query builders), mostly because:
* In your case, not sure how to inject parameters
* query builder seems easier when it comes to refactoring
* and it seems more abstract way to write SQL (relying on additional layers underneath to properly transform it to underlying vendor - SQLite, postgres, mysql...)

However, I am not ditching you off, if this continues to be pain in the ass - plain T-SQL, here I come!:)


return_codes_pull_request (michael-okeefe) - Win64-Windows-7-VisualStudio-14: OK (2972 of 2972 tests passed, 11 test warnings)

Messages:

 * 4 tests had: ERR diffs.
 * 6 tests had: AUD diffs.
 * 5 tests had: RDD diffs.
 * 1 test had: BND diffs.

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


What about including the case of best value over static eval?


dolboeb


ok, I'll have to check this out, could it be that we're running ipfw twice now? the fix did work on my end, which is odd if the code wouldn't reach it.


Ah! I was just about to submit a PR for this before realizing it was already fixed.

Can we get this code in the latest version? I just installed and am still seeing "Increment."

Thanks!


## Jenkins Build Summary

Built from [this commit](https://github.com/mellery451/rippled/commit/1e6e255caff78b39cb0061616d65a179b2b0d17c)

Built at __20180813 - 19:04:49__

### Test Results

Build Type | Log | Result | Status
---------- | --- | ------ | ------
rpm | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/rpm.txt) | 1035 cases, 1 failed, t: n/a | FAIL :red_circle:
msvc.Debug | [console](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//console) | no test results, t: n/a | FAIL :red_circle:
msvc.Debug,
NINJA_BUILD=true | [console](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//console) | no test results, t: n/a | FAIL :red_circle:
msvc.Debug
-Dunity=OFF | [console](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//console) | no test results, t: n/a | FAIL :red_circle:
msvc.Release | [console](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//console) | no test results, t: n/a | FAIL :red_circle:
gcc.Release
-Dassert=ON,
MANUAL_TESTS=true | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Release_Dassert_ON_MANUAL_TESTS_true.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug
-Dcoverage=ON | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug_Dcoverage_ON.txt) | no test results, t: 0s | FAIL :red_circle:
docs | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/docs.txt) | no test results, t: 0s | FAIL :red_circle:
clang.Debug | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/clang.Debug.txt) | no test results, t: 0s | FAIL :red_circle:
clang.Debug
-Dunity=OFF | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/clang.Debug_Dunity_OFF.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug
-Dunity=OFF | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug_Dunity_OFF.txt) | no test results, t: 0s | FAIL :red_circle:
clang.Release
-Dassert=ON | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/clang.Release_Dassert_ON.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Release
-Dassert=ON | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Release_Dassert_ON.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug
-Dstatic=OFF | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug_Dstatic_OFF.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug
-Dstatic=OFF -DBUILD_SHARED_LIBS=ON | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug_Dstatic_OFF_DBUILD_SHARED_LIBS_ON.txt) | no test results, t: 0s | FAIL :red_circle:
gcc.Debug,
NINJA_BUILD=true | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/gcc.Debug_NINJA_BUILD_true.txt) | no test results, t: 0s | FAIL :red_circle:
clang.Debug
-Dunity=OFF -Dsan=address,
PARALLEL_TESTS=false,
DEBUGGER=false | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/clang.Debug_Dunity_OFF_Dsan_address_PARALLEL_TESTS_false_DEBUGGER_false.txt) | no test results, t: 0s | FAIL :red_circle:
clang.Debug
-Dunity=OFF -Dsan=undefined,
PARALLEL_TESTS=false | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/debug-cleanup/1//artifact/clang.Debug_Dunity_OFF_Dsan_undefined_PARALLEL_TESTS_false.txt) | no test results, t: 0s | FAIL :red_circle:


TeamCity Bvcms / BuildTrigger [Build 69](https://builds.tpsdb.com/viewLog.html?buildId=170&buildTypeId=Bvcms_BuildTrigger) is now running


## Jenkins Build Summary

Built from [this commit](https://github.com/mellery451/rippled/commit/bec69fa21674beae475f923c311a42d0b39cfbef)

Built at __20180813 - 19:05:07__

### Test Results

Build Type | Log | Result | Status
---------- | --- | ------ | ------
rpm | [logfile](http://ci.ops.ripple.com/job/mellery-rippled/job/cmake-cleanup/1//artifact/rpm.txt) | 1035 cases, 1 failed, t: n/a | FAIL :red_circle:


oi


Absolutely Amazing. I will try it!


Yes, person table is more important, that's exactly why I didn't start with it:D

Last time I wrote PHP was in faculty, so...2003-2004:) I want to ramp up slowly and to leave "important" stuff for later, when I get used to NC and PHP. Plus, compared to regular work hours, I like that I can do stuff in my tempo, my priorities, my milestones, no customers, no backward compatibility... I think you can relate;)

Thanks for first commit, hopefully will keep them coming:D

I would **very** much like that you keep monitoring my commits and commenting on them. Look around to somehow "subscribe" to this branch, I don't know details.


You decide (I added this more five months ago: <https://github.com/gorhill/uBlock/commit/ec56165d0d36ab91fba7d5cf44a4552d19fefdd5#diff-1c951eedcd0be2e11c02da8fabcc46b5>).


Thanks for all the work you put in on this and other scripts, they were extremely useful. Sorry about the unappreciative whiners and best of luck with your work.


There's a typo here, it should be `archivematica-dashboard` instead of `archivematica-storage-service`


Тогда все ок. Меня смутило, что мы выбираем/создаем вагоны по кол-ву свободного места и я просмотрел, что там не только car_parking в методе написано. Кстати, этот метод я бы назвал #find_or_create_cargo_car


You are my personal hero.


I am also not genius for DNN:)

@matiasdelellis and me are working on [Face Recognition for Nextcloud](https://github.com/matiasdelellis/facerecognition). We choose your PHP binding - thanks for it! We now need face_descriptors (at least we think we need them🤪), and I saw that you already started working on it, so I thought "hey, let's not duplicate same code". I *cannot* work on your branch (this branch) as I don't know what are you doing! However, I can create PHP binding for face_descriptors on master branch, help you merge that to this branch, and you can use that to test PHP parts, and continue on top of that. Does that makes sense?


Ótimo Diagrama!!!!


А тут разве не должны быть () вместо [] ?


is this necessary?
it shouldn't be if it is


TeamCity Bvcms / BuildTrigger [Build 69](https://builds.tpsdb.com/viewLog.html?buildId=170&buildTypeId=Bvcms_BuildTrigger) outcome was **SUCCESS**
Summary: Running Build time: 00:07:59


TeamCity Bvcms / BuildTrigger [Build 70](https://builds.tpsdb.com/viewLog.html?buildId=171&buildTypeId=Bvcms_BuildTrigger) is now running


Ok, then I would say - just to be on the safe side - we start to use `+js` from now on and if nobody complains about this new syntax in the next few weeks, we convert all `script:inject` filters.


TeamCity Bvcms / BuildTrigger [Build 71](https://builds.tpsdb.com/viewLog.html?buildId=172&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 70](https://builds.tpsdb.com/viewLog.html?buildId=171&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:04


TeamCity Bvcms / BuildTrigger [Build 72](https://builds.tpsdb.com/viewLog.html?buildId=173&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 71](https://builds.tpsdb.com/viewLog.html?buildId=172&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:04


TeamCity Bvcms / BuildTrigger [Build 72](https://builds.tpsdb.com/viewLog.html?buildId=173&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


TeamCity Bvcms / BuildTrigger [Build 73](https://builds.tpsdb.com/viewLog.html?buildId=174&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 74](https://builds.tpsdb.com/viewLog.html?buildId=175&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 73](https://builds.tpsdb.com/viewLog.html?buildId=174&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


TeamCity Bvcms / BuildTrigger [Build 74](https://builds.tpsdb.com/viewLog.html?buildId=175&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:04


TeamCity Bvcms / BuildTrigger [Build 75](https://builds.tpsdb.com/viewLog.html?buildId=176&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 76](https://builds.tpsdb.com/viewLog.html?buildId=177&buildTypeId=Bvcms_BuildTrigger) is now running


TeamCity Bvcms / BuildTrigger [Build 75](https://builds.tpsdb.com/viewLog.html?buildId=176&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:05


TeamCity Bvcms / BuildTrigger [Build 76](https://builds.tpsdb.com/viewLog.html?buildId=177&buildTypeId=Bvcms_BuildTrigger) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:04


Why pass a `command` instead of passing the `artifacts` directly?
Also, what's the lifecycle of that `tempfile`, when will it be deleted?


remove?


Fix for issue #288


Could you filter out non-rust tasks / build targets? (that'd save a couple of expensive unnecessary `buck query` calls for non-buck targets?


И Макс, и я намекали, как могли про super. Сравнение можно унифицировать, потому-что логика одна и таже. А именно - занимается место, пока оно осталось. Только в случае с пассажирским вагоном будет всегда заниматься 1 место. То есть здесь можно вместо этого тела метода просто вызвать родителський метод с "захардкоденым" аргументом.

```
def take_place
  super(1)
end
```
 Таким образом в main этот метод будет вызываться без аргумента в соответствии с ТЗ


What if this fails? E.g. the size might be too big.


s/Taxe amont by group/Taxes amounts by group (same match later)


The current experience is non-optimal as users have to input & build buck targets from the toolbar & only then, chain extra `buck build` commands (without reporting progress), & finally, the users would get their language service working.


TeamCity LCM :: liblcm-win32-master Continuous [Build 256](http://build.palaso.org/viewLog.html?buildId=126729&buildTypeId=Liblcm_LiblcmWin32masterContinuous) outcome was **SUCCESS**
Summary: Tests passed: 2608, ignored: 44 Build time: 00:01:31


TeamCity LCM :: liblcm-win64-master Continuous [Build 213](http://build.palaso.org/viewLog.html?buildId=126731&buildTypeId=Liblcm_LiblcmWin64masterContinuous) outcome was **SUCCESS**
Summary: Tests passed: 2608, ignored: 44 Build time: 00:02:03


TeamCity LCM :: liblcm-linux64-master Continuous [Build 215](http://build.palaso.org/viewLog.html?buildId=126730&buildTypeId=Liblcm_LiblcmLinux64masterContinuous) outcome was **SUCCESS**
Summary: Tests passed: 2620, ignored: 32 Build time: 00:02:01


![](https://github.trello.services/images/mini-trello-icon.png) [Отчеты](https://trello.com/c/4fP15jov/31-%D0%BE%D1%82%D1%87%D0%B5%D1%82%D1%8B)


Oh, lol. Blame @DustVoice for the necro. I don't think I've been following the project for that long.


It is, unfortunately. Making the attack interruptible causes critical errors when interrupt() fires before init() does in recoverySlam. Besides, it's not a super-strong attack at the moment.


Automated Test Suite Message


Delete line 20 and you should be good to go.


it's pretty easy to put a space between the text and the */ end of comment marker :)