Please, don't waste any more time on this. The PR was intended to be 'cosmetic'. It adds no real value to the code base. Kemal is probably the coolest thing since Sinatra. Please, focus on making it even cooler.

Thank you all.


Just in case someone else finds this again: https://github.com/meteor/tutorials/issues/175
The correct way of fixing this would be
```bash
meteor remove blaze-html-templates
meteor add static-html
```
This guide explains it, but it's missing in the tutorial https://guide.meteor.com/react.html#using-with-meteor


If no history file exists, we should do a verbose log statement.


+1

Sent from MailDroid

-----Original Message-----
From: Dimitry Sibiryakov <notifications@github.com>
To: FirebirdSQL/firebird <firebird@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Sent: вт, 17 июл 2018 14:05
Subject: Re: [FirebirdSQL/firebird] Fix CORE-5874 - Provide name of read-only column incorrectly referenced in UPDATE ... SET xxx. (32ca23b)

Without proper versioning of msg file it is a really bad idea to change existing messages. Either add a new message or add a secondary code.

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/FirebirdSQL/firebird/commit/32ca23b4fc17d01c09e6400a41e313163ee4559a#commitcomment-29733068


Ok, sure


Ass


Well done ;)


![](https://github.trello.services/images/mini-trello-icon.png) [Wire up your static React App to our Notes API.](https://trello.com/c/8ZXeYcd6/9-wire-up-your-static-react-app-to-our-notes-api)


What's up with the function name? Should it not be OnLoadCustomDatabaseTable()?


Fixed


#6517-MissingEqNumRefsInEngrgRef (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2940 of 2982 tests passed, 11 test warnings)

Messages:

 * 11 tests had: AUD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 42 tests had: Table big diffs.
 * 1 test had: ERR diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.

Failures:

regression Test Summary
 * Passed: 601
 * Failed: 42

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


The following doc(s) have been deployed to the [rsyncupdat](https://rsyncupdat-static-docs.pantheonsite.io/docs) Multidev environment:
- [scripts/deploy-live.sh](https://github.com/pantheon-systems/documentation/commit/b0aec4a3b165fc9008e146250de802494a8ac9ec/scripts/deploy-live.sh)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/b0aec4a3b165fc9008e146250de802494a8ac9ec/scripts/deploy-multidev.sh)
- [/docs/cache-control](https://rsyncupdat-static-docs.pantheonsite.io/docs/cache-control)
- [/docs/caching-advanced-topics](https://rsyncupdat-static-docs.pantheonsite.io/docs/caching-advanced-topics)
- [/docs/code](https://rsyncupdat-static-docs.pantheonsite.io/docs/code)


Couldn't you calculate all of the rammed pawns in one operation?

score -= Rammed * popcount(pos.pieces(Us, PAWN) & shift<Down>(pos.pieces(Them, PAWN)));

or similar?


You did it so thank your @norgalades


How are you compiling this so that it emits a warning? What warning do you get?

None of the following commands give me any warnings pertaining to `find-tag-marker-ring`:
1. `make emacs=emacs-27.0.50 compile`
2. `make emacs=emacs26 compile`
3. `make emacs=emacs25 compile`
4. `make emacs=emacs24 compile`

My corresponding Emacs versions are the following:

1.
       In GNU Emacs 27.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
        of 2018-07-14 built on thunk
       Repository revision: 155d7303808345dd73427302d9a352ec5461c11a
       Windowing system distributor 'The X.Org Foundation', version 11.0.11906000
       System Description: Debian GNU/Linux buster/sid

2.
       In GNU Emacs 26.1.50 (build 1, x86_64-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
        of 2018-07-07 built on thunk
       Repository revision: 40e1db8ccd1239fc7da5ccd3f5f79017b2b44afc
       Windowing system distributor 'The X.Org Foundation', version 11.0.11906000
       System Description:     Debian GNU/Linux testing (buster)

3.
       In GNU Emacs 25.2.1 (x86_64-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
        of 2018-05-07, modified by Debian built on binet
       Windowing system distributor 'The X.Org Foundation', version 11.0.11906000
       System Description:     Debian GNU/Linux testing (buster)

4.
       In GNU Emacs 24.5.1 (x86_64-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
        of 2017-09-12 on hullmann, modified by Debian
       Windowing system distributor `The X.Org Foundation', version 11.0.11906000
       System Description:      Debian GNU/Linux testing (buster)

This is with a checkout of Swiperc commit d8d3cbd32bb132b4819340d2fffed4fb52007962.


"to fix bug with Yomichan"


Should be good now. Please confirm.


I didn't see a specific commit for comments so i'll just toss a message here.  What you have looks good but more would be better! By that I mean descriptions at the tops of methods describing what is going on inside that method.  It shouldn't be longer than 10 words for the most part.  Otherwise the code looks good! The styled components are pretty unique.


#6517-MissingEqNumRefsInEngrgRef (RKStrand) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@elevowofficial i can finish code, please message me at Dansteveade1


Do we need to check for isNativeSession?


Comments look great!  Not overly verbose but informative.  Alec strikes again! :+1:


i can continue the project just send me a message at "Dansteveade1" on fiverr
@elevowofficial , @dansteve


這個修正到 zhs2zht 了， zht fixed in https://github.com/bowans/slay-the-spire-zht-localization/commit/4bf64e4dfc16750339b5bc40afce674a331ee0ba.


Adding to run get pull


I Solved my issues https://github.com/luisreiskeys/react-native-background-timer


Thanks for getting back to me. The versions of phpunit and phpunit-mock-objects did not change before and after this change. I'm using an exact version constraint on the codeception package to reproduce the error. I get the error when I call `Stub::make()`.

`codeception/codeception              2.4.1 => 2.4.2`
`codeception/stub                            1.0.4 => 2.0.1`
`phpunit/phpunit                               7.1.5`
`phpunit/phpunit-mock-objects      6.1.2`


FYI. MPL is already listed in our scotzilla sheet. If we add this package, please update our tracking of licenses.


提交


Resolução do exercício.


**Fix** [`View [auth.passwords.reset] not found.`](https://github.com/mtvbrianking/multi-auth/issues/4)


is this really austin or a ghost. blink once for yes and twice for no.


Does this test the case where the  there is an existing list of timestamps {3,1} and we add the timestamp 2, and assert the final list is {3,2,1}?


В данной сортировке наличие дефолтного компаратора не имеет смысла, задумайтесь, что будет от такой сортировки ожидать код, который не знает деталей реализации. Дефолтное сравнение имеет место быть только в случае, когда для типов выполняется отношение порядка (IComparable), что не логично для двух элементов типа int[].


>[...] The PR was intended to be 'cosmetic'. It adds no real value to the code base. [...]

@lebogan Well, it adds value in a sense of a 🕦💣 (ticking bomb) waiting to explode on CI at very next release. I know what I'd do in your position: I'd send a followup PR fixing this.


Die englischen Texte enthalten zwar auch oft "he" anstatt [PAWN_pronoun], aber die Änderungen sollten innerhalb des Spiels zu keinen Problemen führen, da das Geschlecht der Person das Pronomen damit automatisch anpasst.
Auf diese Weise kommen weniger Fehler zustande, falls es sich mal doch um eine weibliche Person handelt.


Thanks for the feedback! I fixed that in 4cfb164
`os.getenv` actually returns `None` if the environment variable isn't set, so simply setting the list to `[]` fixes the issues.


In this case we don't actually know what the size of sensor_segmented_planes (vector of vector of planes) is going to be. We don't know how many unique observations are going to be there for the given number of sets before hand..


We might want to log a verbose message as we try to unlock the file so we can tell if the backup is hanging trying to acquire the lock.


Tested your version, works fine.

But I fixed it this way on my side:
https://github.com/fragcolor-xyz/nimpy/commit/dac219c8d6fae6a796ef8d930c4383a0ef30d4d0

not sure what's the right way but on python manual says it should inject current path if you pass 0
https://docs.python.org/2/c-api/init.html#c.PySys_SetArgvEx

I'll pull your version later anyway :+1:


If you open an issue yourself you use checkboxes instead of bullet points and click each point as it gets done ;)


Merged devlatest to master


Looks really good!


Oh that's even cleaner. I'll do it your way.


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

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


I guess you actually want `rtrim` instead


These should be in "LinuxScripts/InstallInfluxRepo.sh" not here. Please move them there.


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


Should `ivy-alist-setting` be fixed accordingly, or should it be avoided here? I vote the former.


Test `comment`


Will the collection of defining attributes always have the same sequence? I don't know how the API orders them. It might be worth it to try saving this collection as a list in a separate variable before the loop, and then ordering the a schema attributes by their name or db id. If the API somehow returns them in a different order now and then, I think it might affect how the key gets built.


I wasn't spending anytime at all personally,  and solely watch this repo because i have a few internal projects using Crystal & Kemal, and just wanted to point out that there already exists a utility for verifying the validity a version in core, and simply thought it might save all you excellent contributors some headache.


> What do you mean by 'again'. The segs array should be filled only once by the parent tag?

Bad wording. `bounce_bus_dmamap_load_buffer()` calls `_bus_dmamap_reserve_pages()` ([busdma_bounce.c:712](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L712)). If there is a shortage, `_bus_dmamap_reserve_pages()` returns `EINPROGRESS` and the map is put on a waiting list.

When the shortage is over, the entire process is done again. `busdma_swi()` calls `bus_dmamap_load_mem()` ([busdma_bounce.c:1292](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L1292)), which in turn starts the load process all over again ([subr_bus_dma.c:501](https://github.com/freebsd/freebsd/blob/master/sys/kern/subr_bus_dma.c#L501)).

This is what I mean by again.

> Hm, no, I don't think so. Another load will allocate a new segs array, and the previous one will be left in temp_segs.

A load does *not* allocate a new segs array. It uses the segs array tag->segments, unless the client specifies a segs array, in which case that array is filled (subr_bus_dma does not provide a segs array. It simply passes NULL to indicate that the tag's array should be used). A new segs array for each map is not allocated in any case. The segs array belongs to the tag, not the map. Check [busdma_bounce.c:76](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L76), [busdma_bounce.c:707](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L707), [busdma_bounce:296](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L296), [busdma_bounce.c:914](https://github.com/freebsd/freebsd/blob/master/sys/x86/x86/busdma_bounce.c#L914).

> Also, if the client is not supposed to try to issue concurrent loads using the same tag, why are we doing all this? Isn't it enough to just store whatever segs array the parent tag returns and keep that until the load is finished?
>
> The client issuing another load while this is still being processed is not supported as you say above.

The client is not supposed to issue a load while another load is being "processed". What that means is, until `bus_dmamap_load()` does not return, another load should not be issued. But, if a load returns `EINPROGRESS`, another load can be called while the previous load is waiting for resources. It is possible for the latter load to succeed if it needs less resources. Also note that `bus_dmamap_load()` never sleeps, it either succeeds, returns an error, or returns `EINPROGRESS` which indicates the load will be completed from a deferred context.

The bottom line is, the segs array *can* be over-written.


feel free to submit a PR


패킷을 관찰하던 중 IP Header가 존재하지 않으면 TCP Header도 존재하지 않는다고 판단하여 IP Header의 Checksum으로 Control Flow를 구성하였습니다.


This collection should also be sorted somehow, to ensure that the keys are generated the same way each and every time.


Sounds good. Then our look-ups will still work as required.


It


#6842-WrongTempUsedInCalcRadSysHXEffectTerm (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2942 of 2942 tests passed, 8 test warnings)

Messages:

 * 8 tests had: AUD 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)


all done


Why is whitespace stripped out here? Is it possible that two objects might be almost identical, except for whitespace in one attribute?


PYNQ image 2.1


YES


Oh god..I'm on a business trip and really shocked to see this much bikeshedding going on for this...if we don't like it or it breaks..we can just revert it


![](https://github.trello.services/images/mini-trello-icon.png) [Allow editing of an existing note](https://trello.com/c/fnOUfg0O/8-allow-editing-of-an-existing-note)


git commit -m "Refactor használhatósági tesztek.
https://github.com/Tomi1986/Heinekem/commits?author=Tomi198
https://github.com/Tomi1986/Heinekem/commit/5162ee3a0124498a71fedbb705c8bab9ea887457#diff-04c6e90faac2675aa89e2176d2eec7d86

git commit -m "Refactor használhatósági tesztek.

Elnézést kérek a cselekedetemért..Nem megy nekem sokat kell még tanulni..Bocsánat


It seemed that each time the defining attributes were pulled out in the same order but your right, it would make sense to order them alphabetically so that if that ever stopped we were still getting the same keys


ok 😄 Great to hear you like the new version btw 🎉


Thanks for this branch! Even though it's not pulled into the main repo yet, I'm able to make use of it :) Lists are very important!


Ah, no, I was just thinking a Rust enum. I can't decide if it would be annoying or comforting to have an enum of all the jobs you can enqueue or not...


Guys may I jump in? I think all maintainers try to do their best and spend
their precious time here. Lets bring up the positive in each other this
always works best long term for all parties.

This does not mean a good discussion is possible ;)

On Wed, Jul 18, 2018 at 3:16 PM Erik Dannenberg <notifications@github.com>
wrote:

> So basically you don't care that pretty much all other maintainers made an
> effort to improve git discipline over the course of this year, because you
> prefer easy over proper?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/OpenMage/magento-lts/commit/afc58e89714d5ec3c7a24b4af571bc3d4d66f0e3#commitcomment-29749515>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAn0a2TyvDnCcOEY-a6so7Sln1xGD5vbks5uHzVBgaJpZM4VS3n4>
> .
>


Trying to re-add


Ok, that makes more sense. Thanks!


Why were these markers removed?


Why was this marker removed?


The one that was coming up at this location was the name attribute, and it was my personal preference that the keys be a continuous string of characters


THE
A


This function needs prefixing with the component name to ensure it doesn't conflict with others in the global scope (e.g. `mod_book_check_progress`


As above, prefix this function name with the component name.


Es-tu sûr que tu t'en souviens?


Maybe also point people to the https://github.com/carver/ethereum-python-project-template/blob/master/.project-template/fill_template_vars.sh script to fill the module name (and other variables) everywhere.


Since this function generates markup, is there a reason it's not in `mod_book_renderer`? As it stands it's not possible for themes to override the markup generated here.


You *could* also dive into the API code to see how it assembles these lists and collections (I haven't done that recently, which is why I can't remember if the order is defined or not)... But I think generally it's a good idea to make sure you know that the order will always be the same. Another question: If the order changed, how would you know? Would your code alert you to the attributes being in a different order than last time?
You could also write a unit test that calls the getCacheKey on the same thing many many times (hundreds? thousands?) and fails if the returned key ever differs from the rest.
*Another* option is to make the key a *map* of values, rather than a string of concatenated values. This is probably not necessary in this case, but if the key were a map, then the sorting the inputs for making the key wouldn't matter so much.


The result is better than nothing... but full CPU on all cores for minutes... hahahaha.


![](https://github.trello.services/images/mini-trello-icon.png) [Realizar Backup programado](https://trello.com/c/FNJkHQzD/215-realizar-backup-programado)


wtf dude !?!

On 18 July 2018 at 13:39, Bruno Chapdelaine <notifications@github.com>
wrote:

> Es-tu sûr que tu t'en souviens?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/EtienneTetreault/hello-world/commit/c64385c48375f41bf70abc455225bb81d05a2a11#commitcomment-29753521>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AeBejN1CLYQgaGjY7DCXFizCHIouedXeks5uH3LmgaJpZM4VVBM3>
> .
>


in she goes


When using `moodle_url` for URLs with GET parameters, use the `$params` argument for them, e.g.

```php
$chapterurl = new moodle_url('/mod/book/view.php', [
    'id' => $cmid,
    'chapterid' => $currentchapter->chapterid,
]);
```


@acidsploit @zanza321
I've done these changes. You can also upload more to the folder if you don't like these.
It's currently not merged but you can find it in this branch, please use only for testing until it is merged to master, unless you test it yourself before.
https://github.com/sanchaz/bitcoin-cash-payments-for-woocommerce/tree/selectable_checkout_icon


[inst/examples/example_experiment.R:91:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L250) *style:* **Commented code should be removed.**
```r
      #urls <- moduleMetadata(module, path)$inputObjects$sourceURL
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:262:27:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L262) *style:* **Commas should always have a space after.**
```r
    chksums2 <- chksums[0,]
                          ^
```
[R/downloadData.R:263:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L263) *style:* **Commented code should be removed.**
```r
    #children <- moduleMetadata(module, path)$childModules
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:266:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L266) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
            ^
```
[R/downloadData.R:266:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L266) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                 ^
```
[R/downloadData.R:266:54:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L266) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                                                     ^
```
[R/downloadData.R:266:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/downloadData.R#L266) *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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/R/module-repository.R#L402) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/moduleMetadata.R:72:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/moduleMetadata.R#L72) *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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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/4334935323c75ef567a084d0abe54a7187d36bd7/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:128:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L128) *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:186:12:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L186) *style:* **Variable or function name should be lowerCamelCase.**
```r
           NaNRetries = NA, logObjFnVals = FALSE, weights, useLog = FALSE) {
           ^~~~~~~~~~
```
[R/POM.R:196:25:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L196) *style:* **Variable or function name should be lowerCamelCase.**
```r
                        NaNRetries, logObjFnVals, weights, useLog) {
                        ^~~~~~~~~~
```
[R/POM.R:213:28:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L213) *style:* **Variable or function name should be lowerCamelCase.**
```r
    if (is.na(NaNRetries)) NaNRetries <- 1
                           ^~~~~~~~~~
```
[R/POM.R:237:11:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L237) *style:* **Variable or function name should be lowerCamelCase.**
```r
          sim_ <- Copy(sim)
          ^~~~
```
[R/POM.R:255:66:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L255) *style:* **Put spaces around all infix operators.**
```r
          envPOMCalled <- sys.frame(min(grep("POM", sys.calls()))-1)
                                                                ~^~
```
[R/POM.R:322:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L322) *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:360:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/4334935323c75ef567a084d0abe54a7187d36bd7/R/POM.R#L360) *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]])])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


If whitespace is a valid character(s) for an attribute in the database, it still can contribute to the key. Stripping it out *probably* won't cause any problems, but can that be guaranteed?


TeamCity Plugins / CSV [Build 85](http://ci.n5o.green/viewLog.html?buildId=1610&buildTypeId=Plugins_Csv) is now running


TeamCity Plugins / CSV [Build 85](http://ci.n5o.green/viewLog.html?buildId=1610&buildTypeId=Plugins_Csv) outcome was **SUCCESS**
Summary: Running Build time: 00:00:28


#6517-MissingEqNumRefsInEngrgRef (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1694 of 1694 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.03%25-green.png)


TeamCity Plugins / CSV [Build 86](http://ci.n5o.green/viewLog.html?buildId=1611&buildTypeId=Plugins_Csv) is now running


This applies to all of the caches in this commit: Does it ever happen that you need to replace something in a cache with an updated version of the same instance?


TeamCity Plugins / CSV [Build 86](http://ci.n5o.green/viewLog.html?buildId=1611&buildTypeId=Plugins_Csv) outcome was **SUCCESS**
Summary: Running Build time: 00:00:50


Should we just say ''%s' is not a valid integer value to iterate on"?


TeamCity Plugins / CSV [Build 87](http://ci.n5o.green/viewLog.html?buildId=1612&buildTypeId=Plugins_Csv) is now running


TeamCity Plugins / CSV [Build 87](http://ci.n5o.green/viewLog.html?buildId=1612&buildTypeId=Plugins_Csv) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


You might want to amend the commit title


Try with last commit.

> Also the option seems to be missing in the full build - choice number 4 in the ini-file.

I don't want to add an untested lib to built-in options. That's the point.


Collision 14.30.1~15


There's no license on this repo yet, defaulting it to proprietary, so I'm not sure I can assume that.


oops. sorry about that. pushed the fix...


Regarding the second question (though this applies to the overall philosophy behind this):
The cache is only used during runtime for this species, theres (currently) no persistence outside of that. What this new code is supposed to do is reduce the **fetchIdenticalInstances** calls. Caching had been happening for ewas', complexes, entitysets, etc but hadn't been implemented for the identical instance check. This was costing us time since it's an expensive action (particularly so with this API) that happens many times for each instance being inferred.

Since we only care about making sure that identical instance calls happen only once, I think we need only care about a unit test that ensures that calling getCacheKey _n_ times results in the same order being returned _n_ times.


You know how many sensors are involved in your calibration, and how many observations of each one you have. Thats enough to set the size of vv_clouds, and the 2 upper levels of vvv_planes. Then you can avoid doing:
vv_clouds[i].push_back(cloud);


psexec in windowsservercore gives me the error

The handle is invalid.
Error establishing communication with PsExec service on F1C39EDC02B1:
The command 'cmd /S /C c:\windows\temp\psexec.exe -accepteula -s cmd /c whoami' returned a non-zero code: 6

Does psexec work on a windowsservercore container at all? I struggle to find good examples.


This in Bags too but I can’t get the cool down to show :(


Created multidev environment [mailinblue#ci-3](https://dashboard.pantheon.io/sites/7c88439c-0f59-4679-be2f-1f02e1043d37#ci-3).

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


The final option you suggested might be useful as well depending on if keys are similar, such as in a theoretical case where all defining attributes return null. The map could be based on defining attribute names and the value


But since there is more information than just the plane cloud (like the normals), you can find a better name, maybe: publishPlane3D


OPT: Could be inlined with the first chown but anyway this also works.


I suppose that is a good point -- I think I irrationally distrust whitespaces for stuff like this


nice catch


Yea this is something you've brought up from the beginning. I don't _think_ so, but I don't know 100%. Thats on my list of things to check


Oh god yes @kzemek - this is the commit of the century - very, very nice.


Why you remove this?


Then you could try to find more descriptive names:
"start calibration" would start the planes segmentation --> Extract planes
"proceed calibration" would trigger the matching, and so on --> Calibrate


Is it necessary?


#6517-MissingEqNumRefsInEngrgRef (RKStrand) - Win64-Windows-7-VisualStudio-14: OK (2900 of 2942 tests passed, 11 test warnings)

Messages:

 * 11 tests had: AUD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 42 tests had: Table big diffs.
 * 1 test had: ERR diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.

Failures:

regression Test Summary
 * Passed: 581
 * Failed: 42

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


jjlkj


If you would like to make a PR against the TAMUctf one we will most likely accept it.


typo


I only commented out code in HeatMapTask.java, as my benchmark for whether or not the implementation was successful was whether or not the heat map displayed. Easily reversible.


add first message


Don't forget, you'll need a kernel with dm-verity disabled for this to work!


Desenvolvi a solução no **console**.

carro.adicionarPessoas = pessoas => {
    if (carro.quantidadePessoas === carro.assentos) {
        return "O carro já está lotado!";
    } else if ((carro.quantidadePessoas + pessoas) > (carro.assentos)) {
        return `Só cabem mais ${(carro.assentos - carro.quantidadePessoas)} ${((carro.assentos - carro.quantidadePessoas) === 1)} : 'pessoa' ? 'pessoas'`;
    } else if ((carro.quantidadePessoas + pessoas) <= carro.assentos ) {
        carro.quantidadePessoas = carro.quantidadePessoas + pessoas;
		return `Já temos ${carro.quantidadePessoas} pessoas no carro!`;
    } else {
        return 'Deu ruim!';
    }
}

_A única coisa que deve ser reparada é o if ternário para saber se deve ser exibido pessoas ou pessoa._


If you find situations where many/all attributes are null, you can still use the string-key, just instead of only concatenating attribute values, prefix each value with its' attribute name.


Hopefully this fix will help the import problem in #1565


The following doc(s) have been deployed to the [rsyncupdat](https://rsyncupdat-static-docs.pantheonsite.io/docs) Multidev environment:
- [scripts/deploy-live.sh](https://github.com/pantheon-systems/documentation/commit/b91d70b16a430b7707ec442f7102021160e19afb/scripts/deploy-live.sh)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/191541496290d077b324491594c4abd32c819195/scripts/deploy-multidev.sh)


This is actually 1.03.7...  I'm just stupid.


FanEnergyIndex (XuanLuoLBL) - x86_64-MacOS-10.9-clang: Tests Failed (1386 of 2945 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 1
 * Failed: 625

regression Test Summary
 * Passed: 622
 * Failed: 2

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


i hope these would be fixed in a commit later.


![](https://github.trello.services/images/mini-trello-icon.png) [Allow editing of an existing note](https://trello.com/c/fnOUfg0O/8-allow-editing-of-an-existing-note)


we avoid using ! conditions as long as its not necessary.

and we have two options why go with ! first. ?


remove these test files from now on.


From now on remove these test related files as we don't need them.


#6842-WrongTempUsedInCalcRadSysHXEffectTerm (Myoldmopar) - Win64-Windows-7-VisualStudio-14: OK (2942 of 2942 tests passed, 11 test warnings)

Messages:

 * 11 tests had: AUD 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)


indentation....ignoring the other places this happened in this commit.
but fix it please.


Created multidev environment [mailinblue#pr-1](https://dashboard.pantheon.io/sites/7c88439c-0f59-4679-be2f-1f02e1043d37#pr-1).

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


There's quite a lot of duplication here between all of the properties of the record and the `$chapterid` conditon. We're also not returning the modified record. Can we pull the record object out of the conditional and just add properties to it? e.g.:

```php
if (!$chapterid) {
    return;
}

$progressrecord = mod_book_current_progress($userid, $bookid);
if ($progressrecord) {
    $progressrecord->chapterid = $chapterid;
    $DB->update_record('book_progress', $progressrecord);
} else {
    $progressrecord = (object) [
        'userid' => $userid,
        'bookid' => $bookid,
        'chapterid' => $chapterid,
    ];
    $progressrecord->id = $DB->insert_record('book_progress', $progressrecord);
}

return $progressrecord;
```


indentation is not right, we are going to follow a 2 space nested spacing model.


indentation issue.


@mcwumbly
https://github.com/pivotal-cf/om/commit/1d1fa9354d4cf41de1ebca22c172b7c4404e8911


My mistake in the commit message here, it is `[desktop]`


Interesting question :).

My first idea was: also a shallow copy.

But on second thought I think a deep copy can make sense as well. If you have a piece of code that wants to modify the copy you can use any PointList, including a shallow copy for this method. Performance wise there shouldn't be downside eithers?


indentation...


Very nice!

On Tue, Jul 17, 2018 at 6:41 PM, Ming Zhang <notifications@github.com>
wrote:

> also there's a typo in the commit message
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/michaelasper/Holograph/commit/414f295d01c7e39c8613f3ccddc6ed182d9352c8#commitcomment-29742970>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACV0aFKJOhAtwDDtznpAT2n0adDF3RaWks5uHpI3gaJpZM4P_HpB>
> .
>


We could. Actually, that's growing on me as we could also use that datakind as an index into the state type (I always like it when inapplicable fields just aren't there).

- Luke

> On Jul 18, 2018, at 9:20 AM, Tristan Ravitch <notifications@github.com> wrote:
>
> Could we avoid some bad states here if we used a phantom type? For example, we could change the type of the second callback to TermStmt s ret -> Annotator Term ext s ret () (the first callback would become Stmt ext s -> Annotator NonTerm ext s ret (). Then we could make it so that addStmtAfter is only callable in the NonTerm context (addStmtBefore could be totally unconstrained).
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.


Is there a reason we can't do this before sending the header with `redirect()`?


1) Contract: Splitter
base.js:266
       Testing the constructor behaves correctly under correct and incorrect usages.
         Constructor succeeds, state correctly populated given different addresses.:
     TypeError: promise is not a function
      at chain.reduced.chain.then (test/sequentialPromise.js:13:33)
      at tryCatcher (node_modules/bluebird/js/release/util.js:16:23)
      at Promise._settlePromiseFromHandler (node_modules/bluebird/js/release/promise.js:512:31)
      at Promise._settlePromise (node_modules/bluebird/js/release/promise.js:569:18)

Somewhere around here. Can't see what's wrong with what I've done.


no i was surposed to do that will push fix for typo soon


spacing issue in between ||


i guess both validations for username can be added in one line.


… in commit a1cec5a, I see. Thank you!


use link_to instead of these <a....> tags


caret --> recipes --> ddalpha --> sfsmisc


move this to coffee files.


if this was to happen in one commit then we could have worked with 1 migration here, instead of two.


![](https://github.trello.services/images/mini-trello-icon.png) [Gerar estatísticas na forma de relatório para o ADM](https://trello.com/c/BgeJC9Kj/218-gerar-estat%C3%ADsticas-na-forma-de-relat%C3%B3rio-para-o-adm)


any reason why we are so choosy about the version here?


indentation is not right here.
Please keep in mind we need to follow a two space nested spacing model.


Realized that I haven't been using ModelUtils in a static context, will fix this in next commit.


spacing issue.


#6517-MissingEqNumRefsInEngrgRef (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2322 of 2322 tests passed, 0 test warnings)

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


External library.


Git GUI lets you easily amend the last commit.
Now that you fucked up two though, you'll need to rebase/squash :p

Last but not least, don't foget to force push


What surprises me are the high number of crashes...


Oh, duh, thank you! I've re-spun the commit.


gcc/cmake? Uh?


change index name


@pashamesh solved, thank you.


Actually I wouldn't revert it, because it doesn't have any side effects and it may be safer for future modifications. The ```m_controller->action(PA_STEER_LEFT, 0);``` just sets ```m_steer_val = m_steer_val_r``` and it makes sure that ```m_steer_val_l = 0```. Or am i missing something?

Ok I will host a soccer server too.

Btw. testing if a server is still alive could be useful. So that once per 10 minutes/hour/whatever I can "ping" it in the script and automatically restart if needed. It happened for me only once so far (the server was available for LAN connection, but it wasn't available for WAN anymore). But it may happen when internet connection is lost etc.


@berkaygure awesome, pull request is already waiting for merge.


do you have an example for DeviceEventEmitter.addListener on secure profile please? I would like to be able to read Beacon Pros and also get the battery level of eddystone or Ibeacon.
Thanks!


This should be VMT undetected now right?


![](https://github.trello.services/images/mini-trello-icon.png) [Edit an existing note](https://trello.com/c/1A0kqY33/4-edit-an-existing-note)


Created multidev environment [mailinblue#ci-9](https://dashboard.pantheon.io/sites/7c88439c-0f59-4679-be2f-1f02e1043d37#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-mailinblue.pantheonsite.io/)