Cool!

That's good news.
Thank you both!

However there is a notice on SO: https://stackoverflow.com/questions/13018625/how-to-exclude-all-artifacts-from-a-group-in-maven:

> Works for me on maven version 3.3.1, triggers warning on maven version 3.0.4

Not sure how should we still be compatible with that old Maven version...


mini project


u hard coded this key-value pair, u should use the input parameters and would be perfect


same as above


You don't need this extra `input` variable, use the `email` parameter instead.


Yes this would also solve that issue as well, I currently added these for now becuase i'm not sure if ignoring updates is a good idea or not.


I write about this variant 5 post up.
But I will not implement this until I see the screenshots of how it all looks.


# PBL 과제_01

* 마프 02분반 6조
* 2016136053 백소정


This is a working solution, however it could be more simple.
For example:
```
string[] emailParts = email.Split('.', '@');
string firstname = emailParts[0];
string lastname = emailParts[1];
```


BTW, i've tested all these formats with FFmpeg without any issue. I added them because of graphist users working with Adobe or macOS, like our forum moderator, that ask why it doesn't support all the formats is it is called "Universal" ?
As i did, so i did added them on the way.


can you remove the debug so we can merge?


is this really necessary? I just updated an installation to the latest version, the customer is now annoyed about this newly and unexpected behaviour. Wouldn't it be a better idea to make this kinda optional?


my new post


You also missed to get the last name, combine the first name and the last name, and make the first characters uppercase.


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152520 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152520") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152527 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152527") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170925-152532 "deployed-2-env-Prod-20170925-152532") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152541 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170925-152541") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


pushed a couple of minutes ago :)


TeamCity SayMore :: SayMore pull requests [Build *.*.53.9ebe5db8751ab80f26727a3476c1b3b1cd33b864](http://build.palaso.org/viewLog.html?buildId=98106&buildTypeId=SayMore_SayMorePullRequests) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 655, ignored: 4 Build time: 00:01:27
### Failed tests
```
SayMoreTests.dll: SayMoreTests.Model.SessionWorkflowInformantTests.GetCategorizedSessionsByField_PassGenre_ReturnsTwoLists: <no details avaliable>

SayMoreTests.dll: SayMoreTests.Model.SessionWorkflowInformantTests.GetCategorizedSessionsFromListByField_FromSpecifiedList_ReturnsTwoLists: <no details avaliable>

SayMoreTests.dll: SayMoreTests.Model.SessionWorkflowInformantTests.GetCategorizedSessionsFromDoubleKey_PassGenreAndStatus_Return: <no details avaliable>

```


Not quite yet...


The return type should be `List<string>` here.


Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.


GTM


@iwalkalone69 See if this works better for you. It's the same API other than the adding of tabs is now `kiwi.addTab(name, component, [props]);`.

The main difference is that I've now removed all the event triggering/binding so that it's easier to follow and work with. It now also no longer needs to mess around with mounted() functions or anything.

~~~javascript
kiwi.addTab('info', {
    data: {},
    template: '

hi
<button @click="clicked">click me</button>
',
    methods: {
        clicked: function(e){ alert("clicked"); }
    }
});

var el = document.createElement('a');
el.innerText = 'hi';
el.addEventListener('click', function(){ kiwi.showTab('info'); });

kiwi.addUi('input', el);
~~~


Your list in which you'd like to go through with your `foreach` loops, is `url`.
Instead of looping through the `url` list, you started to create recursion.
`foreach (var urlsFromHandle in url)`


I posted on the Google Group thread about robot training. I'm not sure about how Trade and Account are involved in robot training except I thought those were separate processes.


not a good comment!


this should be enough to make us aware of this situation


Instead of returning, you should add these urls to a new list first, and then return with this new list.


the function will be created every time when you create a Rectangle


Hola @JuliColombo,

probe la rama `home_categories` y funcionó. Por eso hice el merge a `master`. Sin embargo, ahora probando este cambio en la `master` al intentar instalar con `install/install.sh` devuelve este error:

```bash
django.db.utils.IntegrityError: Problem installing fixture '/home/geonode/geoandino/geoandino/fixtures/initial_data_geoandino.json': Could not load geoandino.SiteConfiguration(pk=1): null value in column "publisher_id" violates not-null constraint
DETAIL:  Failing row contains (1, 2017-08-15 15:09:51.99-05, 2017-08-15 15:09:51.99-05, Portal de Juli, , t, f, , , , http://www.facebook.com/portal, http://www.twitter.com/portal, http://www.github.com/portal, contact@portal.com, , , , , f, t, , , , , , , , , , , null).
```

saludos!


How about "scala.reflect-based macros" vs. "Scalamacros"?

With scalameta/paradise, gestalt and scalamacros, "new macros" are kind of confusing when explaining the background to people who want to get involved.


Why isn't it a One2many?


Change this please


In tradition with nsc, maybe we should go with `nsm`: new scala macros 😄


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/44632/testReport/) for this commit**


ja ja, but u didn't use the input parameter again. u already got the hunagrian word, shouldn't ask user again for that


Resolves #44


Hi i have error in this code
1) The directory /Users/macbook/Desktop/XOutOf10-master is registered as a Git root, but no Git repositories were found there. Configure
2) Error:This Gradle plugin requires Studio 3.0 minimum


Because it does not matter


Yes i still need to add the strings, but this takes care of the hidden messagesOn Sep 25, 2017 17:27, Laurent Pugin <notifications@github.com> wrote:Not quite yet...

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


same as above, and u should search in the values and give back the key


title:baofeidyz
google_analytics:baofeidyz


no comment


Change of color


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-14: OK (2171 of 2171 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)


It matters because you have a intermediate table in Many2many that you don't have in One2many and don't have to migrate. :)


Maybe you mean 3.5.2 and 3.5.1?   Worth a link to the issue?


You don't need this extra empty line here.


qweqweqwe


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/44634/testReport/) for this commit**


I've told you about code style! What is this??


I was only commenting about the attempt to correct this English ;)


"is has" typo


Otherwise, it's a perfect solution, grats :)


Для справки. У вас неправильно был собран jar'ник. =)
Скорее всего, когда вы откроете его как архив, там будут папки META-INF и target (или не таргет, в зависимости от того в какой папку вы компилили исходники). Класс Printer лежит в пакете com.blablabla.
Соответственно в джарнике у вас лежит невалидный класс com.blablabla.Printer в пакете target.com.blablabla.

Самый простой способ корректно собрать джарник - перед сборкой перейти в папку target и собрать его от туда, с последующим переносом на папку выше.


I meant to have 2.87 as the title for this commit, but i followed from 2.85 to 2.86 and forgot that 2.86 was the question about Book objects being immutable.


unnecessary empty line


First


no need for internal class StringBuilder


it should eat a list of string, not a single one :/


One solution would be to fix the currently unused screenScaleFactor and use that instead of Screen.devicePixelRatio. See https://github.com/Ultimaker/Uranium/commit/7ae37a8b57ad4f98829185a632b249fdb1c54a76 and discussion about why the calculation of screenScaleFactor was wrong here: https://github.com/Ultimaker/Uranium/pull/199

The calculation should be something like this:
```
    def _screenScaleFactor(self):
        # OSX handles sizes of dialogs behind our backs, but other platforms need
        # to know about the device pixel ratio
        return 1.0 if Platform.isOSX() else self.devicePixelRatio()
```
I can make a pull-request for that, if you want.


Looks good 👍 👀


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/44635/testReport/) for this commit**


You should put your exception package in the layer using this exception.
So, for the CacheException - `by.bsuir.library.cache.exception.CacheException` and so on.


this breaks the intended functionality. The additional parameters (BUILD_ARGS) should only be added if the MSVC_64_TOOLSET is specified.


no, it is non-stored

On Mon, Sep 25, 2017 at 5:36 PM, pimodoo <notifications@github.com> wrote:

> It matters because you have a intermediate table in Many2many that you
> don't have in One2many and don't have to migrate. :)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/odoo-dev/odoo/commit/aeff8997e978e95973ced5bd28f440ea3591e65e#commitcomment-24547146>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ADp7e6cZGAraWRVUUNqzc1MrFwb3hNIlks5sl8iPgaJpZM4Pi7Ik>
> .
>


this should be part of the OddFilter function


@Atinux Sure :) The generate/ssr optimizations is going to be directly into render so i think is safe :)


Because it not worked. This function called from `test_pipeline`, but not `pytest` directly


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/44636/testReport/) for this commit**


Maybe, but it is at least 10.000 times more beautiful.


Thanks. :)


u summed all odd numbers until the given parameter, u should add the odd numbers of the original list to a new list


No, that's an unrelated but similar change.


nice job, but folder structure could be better :)


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/44637/testReport/) for this commit**


recomendo tirar o item tibiacoin para evitar colapso financeiro do servidor caso haja bug de duplicar item.


deprecated
use `__construct` instead of `foo`


Indeed, this function isn't a valid test. Do we need parametrise test_pipeline()? To make multiple granular tests insert one method which checks everything?


this is hilarious !!


Note 1.25 (not 1.00) is the default.


why change the boiler plate? We already gave that list for u


Ça fonctionne probablement de cette manière mais vous n'avez normalement pas besoin de copier les css de `agency` et `sb-admin` dans ce dossier. Elles ne doivent pas sortir de votre dossier `node-modules`. Normalement tout se fait dans le fichier `app.scss` ([petite piste](http://sass-lang.com/documentation/file.SASS_CHANGELOG.html#CSS__import_Directives)).
Il en va de même pour les fichiers `js`. Le principe est quasiment le même. Tout se fait dans `app.js`. Pas besoin de copier le `js` des dépendances dans ce dossier.


Before:

After:


May be better to do this in temp dir, not in project source dir?


You're welcome!


are you planning to build a zombie mode over these configurations?


thanks @pb82 & @grdryn


// lets start parsing


However I like that you are thinking in separating codes to different functions, it's quite unnecessary here.
Your solution is quite difficult, this could be solved in a much simple way:
```
string[] emailParts = email.Split('.', '@');
string firstName = emailParts[0][0] + emailParts[0].Substring(1);
string lastName = emailParts[1][0] + emailParts[1].Substring(1);
return string.Format("{0} {1}", lastName, firstName);
```


Thanks!


A less verbose and less fragile way to accomplish this would be to downcase the email address before looking it up in the map. Then it would also handle cases like STUDENTS@, sTUdents@, etc.


wtf, why did u convert int to string and back to int again? the ? operator is not usefull anytime...like here, a single if would be easier.
why void? it shoud return the addElements list


Hey @goussaka,

1. Seems like the git registered for the project looks for Windows directory ("C:/Users/<Username>/...") while you have a Macbook. Try to reconfigure the git of your project directory.
2. As the error says, I use Android Studio 3.0 for developing this app, and the Gradle plugin used is supported only on Android Studio 3.0.

Enjoy,
Idoideas.


Confused


'or False' seems useless


overall nice job, but folder structure could be better


Many2many offer flexibility and we want to be flexible

On Mon, Sep 25, 2017 at 5:42 PM, pimodoo <notifications@github.com> wrote:

> Maybe, but it is at least 10.000 times more beautiful.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/odoo-dev/odoo/commit/aeff8997e978e95973ced5bd28f440ea3591e65e#commitcomment-24547289>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ADp7ewFJpZBZTADF1XqNOLvj5Cyo5T3lks5sl8nNgaJpZM4Pi7Ik>
> .
>


How do get other users off


can't we do it in one line?


I'd strongly advise either bumping this (and CMake) back down or planning to release 1.5.1 next as anyone installing this isn't going to be able to distinguish it from the proper future 1.5.0 release. What's the v1.5 time frame, btw?


ok, better, what if
if(list[i] % 2 != 0)
{
    oddElements.Add(list[i]);
}
isn't it more simple?


Did you forget to rename the module? Or you intentionally left as `git-repositories`?


I'd where


Hallo

die Version 0.9.32 lässt sich nicht mehr unter Debian "Buster" kompilieren. Die Version 0.9.31-intermediate hingegen schon.

Hier die Fehlermeldung zur weiteren Analyse:
>   CC [M]  /opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.o
/opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.c: In function ‘dma_alloc’:
/opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.c:258:38: error: ‘__GFP_RETRY_MAYFAIL’ undeclared (first use in this function)
    dma->vbuf[i] = kmalloc(dma->size, __GFP_RETRY_MAYFAIL);
                                      ^~~~~~~~~~~~~~~~~~~
/opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.c:258:38: note: each undeclared identifier is reported only once for each function it appears in
/usr/src/linux-headers-4.12.0-2-common/scripts/Makefile.build:307: die Regel für Ziel „/opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.o“ scheiterte
make[5]: *** [/opt/dvb/dddvb-0.9.32/ddbridge/ddbridge-core.o] Fehler 1
/usr/src/linux-headers-4.12.0-2-common/scripts/Makefile.build:566: die Regel für Ziel „/opt/dvb/dddvb-0.9.32/ddbridge“ scheiterte
make[4]: *** [/opt/dvb/dddvb-0.9.32/ddbridge] Fehler 2
/usr/src/linux-headers-4.12.0-2-common/Makefile:1532: die Regel für Ziel „_module_/opt/dvb/dddvb-0.9.32“ scheiterte
make[3]: *** [_module_/opt/dvb/dddvb-0.9.32] Fehler 2
Makefile:152: die Regel für Ziel „sub-make“ scheiterte
make[2]: *** [sub-make] Fehler 2
Makefile:8: die Regel für Ziel „all“ scheiterte
make[1]: *** [all] Fehler 2
make[1]: Verzeichnis „/usr/src/linux-headers-4.12.0-2-amd64“ wird verlassen
Makefile:8: die Regel für Ziel „all“ scheiterte
make: *** [all] Fehler 2
>

Mangels Issue Seite poste ich das hier. Ich Danke vorab schon mal für die Hilfe ;-)


Try to use solutions that you understand, not just a working one from Google.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.408.54f94f49f8bd250712dedd1f5619e5cd3ec99dc8](http://build.palaso.org/viewLog.html?buildId=98110&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:07


No, main function in this test is `run_pipeline`, it's not normal unit test, and it can't be granular.


This function should **return** the result, not just printing it.


TeamCity libpalaso :: palaso-win-master Continuous [Build 4.0.101.54f94f49f8bd250712dedd1f5619e5cd3ec99dc8](http://build.palaso.org/viewLog.html?buildId=98113&buildTypeId=Libpalaso_PalasoWinmasterContinuous) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:32


I thought it to be more beautiful in 2


Where are you trying to run this test?


Oups ;)On Sep 25, 2017 17:37, Laurent Pugin <notifications@github.com> wrote:I was only commenting about the attempt to correct this English ;)

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


Do you know what caused all those white space changes? If possible, that could be reverted and squashed.


I haven't tried yet, I assume it could be launched in TeamCity in our docker img?


Created multidev environment [natalie-sandbox#pr-narrativ](https://dashboard.pantheon.io/sites/ee1a9f88-1cc8-4eca-a8a6-7dc1433bb37e#pr-narrativ).

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


at least it is before when the state timeout is started


[R/helper_funs.R:63:21:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/helper_funs.R#L63) *style:* **`%>%` should always have a space before it and a new line after it, unless the full pipeline fits on one line.**
```r
    factor_levels %>% str_split(":", simplify = TRUE) %>%
                  ~~^
```
[R/processing.R:55:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L55) *style:* **Lines should not be more than 80 characters.**
```r
    message(sprintf(paste0("\n'sperrorest()': Non-convergence during model fit.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/processing.R:122:16:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L122) *warning:* **no visible global function definition for ‘remove_missing_levels’**
```r
    nd_test <- remove_missing_levels(fit, nd_test)
               ^~~~~~~~~~~~~~~~~~~~~
```
[R/processing.R:181:18:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L181) *warning:* **no visible global function definition for ‘remove_missing_levels’**
```r
      nd_test <- remove_missing_levels(fit, nd_test)
                 ^~~~~~~~~~~~~~~~~~~~~
```
[R/processing.R:189:41:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L189) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (is.null(current_res[[j]]$test)) { # nocov start
                                        ^
```
[R/processing.R:320:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L320) *style:* **Lines should not be more than 80 characters.**
```r
  # then is_factor_prediction is not set within runfolds and we need to set it here
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/processing.R:379:7:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/processing.R#L379) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if ((do_gc >= 1) & (do_gc < 2)) {
      ^
```
[R/resample.R:60:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/resample.R#L60) *style:* **Lines should not be more than 80 characters.**
```r
    stop(paste0("'strat' must either be a vector of factor type, or the name", # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/resample.R:154:27:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/resample.R#L154) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (is.null(param$fac)) { # nocov start
                          ^
```
[R/sperrorest_error.R:3:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' @description Calculate a variety of accuracy measures from observations and predictions
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_error.R:70:25:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L70) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (any(is.na(pred))) { # nocov start
                        ^
```
[R/sperrorest_error.R:91:30:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L91) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (nlevels(obs) == 2) { # nocov start
                             ^
```
[R/sperrorest_error.R:122:18:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L122) *warning:* **no visible global function definition for ‘prediction’**
```r
      predobj <- prediction(pred, obs)
                 ^~~~~~~~~~
```
[R/sperrorest_error.R:123:16:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L123) *warning:* **no visible global function definition for ‘performance’**
```r
      auroc <- performance(predobj, measure = "auc")@y.values[[1]]
               ^~~~~~~~~~~
```
[R/sperrorest_error.R:174:34:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L174) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                rmse = sqrt(mean((obs - pred)^2)), mad = mad(obs - pred),
                                 ^
```
[R/sperrorest_error.R:174:46:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_error.R#L174) *style:* **Put spaces around all infix operators.**
```r
                rmse = sqrt(mean((obs - pred)^2)), mad = mad(obs - pred),
                                            ~^~
```
[R/sperrorest_misc.R:44:32:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L44) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (method == "euclidian") { # nocov start
                               ^
```
[R/sperrorest_misc.R:301:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L301) *style:* **Lines should not be more than 80 characters.**
```r
tile_neighbors <- function(nm, tileset, iterate = 0, diagonal = FALSE) { # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_misc.R:301:72:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L301) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
tile_neighbors <- function(nm, tileset, iterate = 0, diagonal = FALSE) { # nocov start
                                                                       ^
```
[R/sperrorest_misc.R:470:13:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L470) *warning:* **no visible global function definition for ‘validate.resampling’**
```r
  stopifnot(validate.resampling(object))
            ^~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_misc.R:508:38:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L508) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
print.resampling <- function(x, ...) { # nocov start
                                     ^
```
[R/sperrorest_misc.R:575:27:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L575) *warning:* **no visible binding for global variable ‘validate.resampling’**
```r
  valid <- sapply(object, validate.resampling)
                          ^~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_misc.R:576:20:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L576) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (any(!valid)) { # nocov start
                   ^
```
[R/sperrorest_misc.R:591:41:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L591) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
print.represampling <- function(x, ...) { # nocov start
                                        ^
```
[R/sperrorest_misc.R:632:45:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_misc.R#L632) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
summary.resampling <- function(object, ...) { # nocov start
                                            ^
```
[R/sperrorest_resampling.R:72:20:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L72) *warning:* **no visible global function definition for ‘as.resampling’**
```r
      resampler <- as.resampling(resampler)
                   ^~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:77:19:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L77) *warning:* **no visible global function definition for ‘as.represampling’**
```r
    resampling <- as.represampling(resampling)
                  ^~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:158:15:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L158) *warning:* **no visible global function definition for ‘as.represampling’**
```r
    repres <- as.represampling(repres)
              ^~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:211:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L211) *warning:* **no visible global function definition for ‘as.resampling’**
```r
    fac <- as.resampling(fac)
           ^~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:218:17:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L218) *warning:* **no visible global function definition for ‘as.represampling’**
```r
    represmp <- as.represampling(represmp)
                ^~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:266:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L266) *style:* **Lines should not be more than 80 characters.**
```r
    warning("'nfold' should be <= nlevels(fac); using nfold=nlevels(fac)\n") # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:414:36:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L414) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  } else if (rotation == "random") { # nocov start
                                   ^
```
[R/sperrorest_resampling.R:467:30:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L467) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (length(dsplit) == 1) { # nocov start
                             ^
```
[R/sperrorest_resampling.R:482:40:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L482) *style:* **Put spaces around all infix operators.**
```r
      r <- phi[as.character(cnt)] * 180/pi # nocov start
                                      ~^~
```
[R/sperrorest_resampling.R:495:31:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L495) *style:* **Put spaces around all infix operators.**
```r
      x_delta <- diff(x_range)/nsplit[1]
                             ~^~
```
[R/sperrorest_resampling.R:496:31:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L496) *style:* **Put spaces around all infix operators.**
```r
      y_delta <- diff(y_range)/nsplit[2]
                             ~^~
```
[R/sperrorest_resampling.R:520:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L520) *style:* **Lines should not be more than 80 characters.**
```r
      x_split <- seq(x_range[1], x_range[2] + x_delta, by = x_delta) # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:543:47:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L543) *warning:* **no visible global function definition for ‘as.tilename’**
```r
          tile[sel_x & sel_y] <- as.character(as.tilename(c(ix, iy)))
                                              ^~~~~~~~~~~
```
[R/sperrorest_resampling.R:550:16:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L550) *warning:* **no visible global function definition for ‘get_small_tiles’**
```r
    s_tiles <- get_small_tiles(tile, min_n = min_n, min_frac = min_frac)
               ^~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:553:21:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L553) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (reassign) { # nocov start
                    ^
```
[R/sperrorest_resampling.R:582:15:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L582) *warning:* **no visible global function definition for ‘as.resampling’**
```r
      tile <- as.resampling(tile)
              ^~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:691:15:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L691) *warning:* **no visible global function definition for ‘as.resampling’**
```r
      tile <- as.resampling(tile)
              ^~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:696:19:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L696) *warning:* **no visible global function definition for ‘as.represampling’**
```r
    resampling <- as.represampling(resampling)
                  ^~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:833:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L833) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste0("empty training set in 'partition_disc': 'buffer'", # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest_resampling.R:1167:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L1167) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (oob) { # nocov start
           ^
```
[R/sperrorest_resampling.R:1256:22:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest_resampling.R#L1256) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if (missing(data)) { # nocov start
                     ^
```
[R/sperrorest.R:326:48:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L326) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (class(par_args$par_mode) == "numeric") { # nocov start
                                               ^
```
[R/sperrorest.R:327:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L327) *style:* **Lines should not be more than 80 characters.**
```r
      stop("'par_mode' has to be specified using an explicit parallel mode name.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:407:27:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L407) *warning:* **no visible global function definition for ‘availableCores’**
```r
    par_args$par_units <- availableCores()
                          ^~~~~~~~~~~~~~
```
[R/sperrorest.R:434:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L434) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    } else { # nocov end
           ^
```
[R/sperrorest.R:461:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L461) *style:* **Lines should not be more than 80 characters.**
```r
        message(sprintf("Using 'parApply' parallel mode with %s cores.", # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:526:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L526) *style:* **Lines should not be more than 80 characters.**
```r
            stop(paste0("No output was received from sperrorest.\n", # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:538:9:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L538) *warning:* **no visible global function definition for ‘plan’, Did you mean 'lag'?**
```r
        plan(par_args$par_option, workers = par_args$par_units) # nocov
        ^~~~
```
[R/sperrorest.R:541:9:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L541) *warning:* **no visible global function definition for ‘plan’, Did you mean 'lag'?**
```r
        plan(par_args$par_option, workers = par_args$par_units)
        ^~~~
```
[R/sperrorest.R:585:5:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L585) *warning:* **no visible global function definition for ‘registerDoFuture’**
```r
    registerDoFuture()
    ^~~~~~~~~~~~~~~~
```
[R/sperrorest.R:594:13:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L594) *warning:* **no visible global function definition for ‘makeCluster’**
```r
      cl <- makeCluster(par_args$par_units, outfile = out_progress, ...)
            ^~~~~~~~~~~
```
[R/sperrorest.R:595:7:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L595) *warning:* **no visible global function definition for ‘plan’, Did you mean 'lag'?**
```r
      plan(cluster, workers = cl)
      ^~~~
```
[R/sperrorest.R:595:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L595) *warning:* **no visible binding for global variable ‘cluster’**
```r
      plan(cluster, workers = cl)
           ^~~~~~~
```
[R/sperrorest.R:601:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L601) *style:* **Lines should not be more than 80 characters.**
```r
        message(paste0("Please see 'sperrorest_progress' in your current", # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:602:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L602) *style:* **Lines should not be more than 80 characters.**
```r
                       " working directory for progress output on Windows.")) # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:606:7:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L606) *warning:* **no visible global function definition for ‘plan’, Did you mean 'lag'?**
```r
      plan(sequential)
      ^~~~
```
[R/sperrorest.R:606:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L606) *warning:* **no visible binding for global variable ‘sequential’**
```r
      plan(sequential)
           ^~~~~~~~~~
```
[R/sperrorest.R:616:45:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L616) *warning:* **local variable ‘pooled_pred_train’ assigned but may not be used**
```r
                        pooled_obs_train <- pooled_pred_train <- c()
                                            ^~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:617:44:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L617) *warning:* **local variable ‘pooled_pred_test’ assigned but may not be used**
```r
                        pooled_obs_test <- pooled_pred_test <- c()
                                           ^~~~~~~~~~~~~~~~
```
[R/sperrorest.R:620:25:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L620) *warning:* **local variable ‘current_impo’ assigned but may not be used**
```r
                        current_impo <- resamp[[i]]
                        ^~~~~~~~~~~~
```
[R/sperrorest.R:623:40:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L623) *warning:* **no visible global function definition for ‘map’, Did you mean 'mad'?**
```r
                        current_res <- map(resamp[[i]], unclass)
                                       ^~~
```
[R/sperrorest.R:725:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L725) *style:* **Lines should not be more than 80 characters.**
```r
                                       not_converged_folds = runfolds_merged$not_converged_folds)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:729:15:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L729) *warning:* **no visible global function definition for ‘stopCluster’**
```r
      on.exit(stopCluster(cl))
              ^~~~~~~~~~~
```
[R/sperrorest.R:744:15:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L744) *warning:* **no visible global function definition for ‘map’, Did you mean 'mad'?**
```r
  check_na <- map(my_res, function(x) all(is.na(x)))
              ^~~
```
[R/sperrorest.R:766:17:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L766) *warning:* **no visible global function definition for ‘transfer_parallel_output’**
```r
  my_res_mod <- transfer_parallel_output(my_res, res, impo, pooled_error)
                ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sperrorest.R:802:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/sperrorest.R#L802) *style:* **Lines should not be more than 80 characters.**
```r
    cat(sprintf("%s folds of %s total folds (%s rep * %s folds) did not converge.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/summary_functions.R:52:37:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L52) *style:* **Variable or function name should be snake_case.**
```r
                                    na.rm = TRUE, ...) {
                                    ^~~~~
```
[R/summary_functions.R:112:59:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L112) *style:* **Variable or function name should be snake_case.**
```r
summary.sperrorestreperror <- function(object, level = 0, na.rm = TRUE, ...) {
                                                          ^~~~~
```
[R/summary_functions.R:140:61:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L140) *style:* **Variable or function name should be snake_case.**
```r
summary.sperrorestimportance <- function(object, level = 0, na.rm = TRUE,
                                                            ^~~~~
```
[R/summary_functions.R:161:12:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L161) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    } else { # when does this happen?
           ^
```
[R/summary_functions.R:249:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L249) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/summary_functions.R:250:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L250) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/summary_functions.R:251:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/R/summary_functions.R#L251) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-processing.R:81:11:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L81) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
          ^
```
[tests/testthat/test-processing.R:81:22:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L81) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                     ^
```
[tests/testthat/test-processing.R:81:39:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L81) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                      ^
```
[tests/testthat/test-processing.R:81:50:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L81) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                                 ^
```
[tests/testthat/test-processing.R:86:63:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L86) *style:* **Commas should always have a space after.**
```r
    if (!is.null(fac)) pred <- majority_filter(pred, newdata[,fac])
                                                              ^
```
[tests/testthat/test-processing.R:121:71:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L121) *style:* **Commas should always have a space after.**
```r
  mypred_rpart <- function(object, newdata) predict(object, newdata)[,2]
                                                                      ^
```
[tests/testthat/test-processing.R:169:63:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L169) *style:* **Commas should always have a space after.**
```r
    if (!is.null(fac)) pred <- majority_filter(pred, newdata[,fac])
                                                              ^
```
[tests/testthat/test-processing.R:194:3:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L194) *style:* **Variable or function name should be snake_case.**
```r
  imp.one.rep <- readRDS("inst/test-objects/imp.one.rep.rda")
  ^~~~~~~~~~~
```
[tests/testthat/test-processing.R:219:3:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L219) *style:* **Variable or function name should be snake_case.**
```r
  imp.one.rep <- readRDS("inst/test-objects/imp.one.rep.rda")
  ^~~~~~~~~~~
```
[tests/testthat/test-processing.R:270:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-processing.R#L270) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-resample.R:13:52:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-resample.R#L13) *style:* **Only use double-quotes.**
```r
                              param = list(strat = 'slides', nstrat = 100))
                                                   ^~~~~~~~
```
[tests/testthat/test-resample.R:15:32:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-resample.R#L15) *style:* **Only use double-quotes.**
```r
  expect_equal(sum(d$slides == 'TRUE'), 100) # == 100
                               ^~~~~~
```
[tests/testthat/test-resample.R:25:55:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-resample.R#L25) *style:* **Only use double-quotes.**
```r
  d <- resample_uniform(ecuador, param = list(strat = 'slides', n = 400))
                                                      ^~~~~~~~
```
[tests/testthat/test-resample.R:28:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-resample.R#L28) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-sperrorest-resampling.R:23:48:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L23) *style:* **Only use double-quotes.**
```r
  parti <- partition_cv_strat(ecuador, strat = 'slides', nfold = 5,
                                               ^~~~~~~~
```
[tests/testthat/test-sperrorest-resampling.R:37:44:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L37) *style:* **Only use double-quotes.**
```r
  parti <- partition_factor(ecuador, fac = 'zclass')
                                           ^~~~~~~~
```
[tests/testthat/test-sperrorest-resampling.R:49:47:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L49) *style:* **Only use double-quotes.**
```r
  parti <- partition_factor_cv(ecuador, fac = 'zclass', nfold = 5)
                                              ^~~~~~~~
```
[tests/testthat/test-sperrorest-resampling.R:131:63:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L131) *style:* **Commas should always have a space after.**
```r
  parti <- represampling_tile_bootstrap(ecuador, nsplit = c(4,2),
                                                              ^
```
[tests/testthat/test-sperrorest-resampling.R:143:65:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L143) *style:* **Commas should always have a space after.**
```r
  parti <- represampling_kmeans_bootstrap(ecuador, nsplit = c(4,2),
                                                                ^
```
[tests/testthat/test-sperrorest-resampling.R:172:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L172) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-sperrorest-resampling.R:173:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-resampling.R#L173) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-sperrorest-summary.R:241:11:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-summary.R#L241) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
          ^
```
[tests/testthat/test-sperrorest-summary.R:241:22:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-summary.R#L241) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                     ^
```
[tests/testthat/test-sperrorest-summary.R:241:39:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-summary.R#L241) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                      ^
```
[tests/testthat/test-sperrorest-summary.R:241:50:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-summary.R#L241) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                                 ^
```
[tests/testthat/test-sperrorest-summary.R:263:57:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest-summary.R#L263) *style:* **Commas should always have a space after.**
```r
  out <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                        ^
```
[tests/testthat/test-sperrorest.R:46:67:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L46) *style:* **Commas should always have a space after.**
```r
            out <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                                  ^
```
[tests/testthat/test-sperrorest.R:74:73:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L74) *style:* **Commas should always have a space after.**
```r
                                                              newdata)[,2]
                                                                        ^
```
[tests/testthat/test-sperrorest.R:101:13:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L101) *style:* **Variable or function name should be snake_case.**
```r
            par.nsp.res <- sperrorest(data = ecuador, formula = fo,
            ^~~~~~~~~~~
```
[tests/testthat/test-sperrorest.R:103:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L103) *style:* **Lines should not be more than 80 characters.**
```r
                                      model_args = list(cost = 10000, gamma = 0.0001,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sperrorest.R:208:21:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L208) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
                  x[ fac == lev ] <- majority(x[ fac == lev ])
                    ^
```
[tests/testthat/test-sperrorest.R:208:32:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L208) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
                  x[ fac == lev ] <- majority(x[ fac == lev ])
                               ^
```
[tests/testthat/test-sperrorest.R:208:49:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L208) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
                  x[ fac == lev ] <- majority(x[ fac == lev ])
                                                ^
```
[tests/testthat/test-sperrorest.R:208:60:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L208) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
                  x[ fac == lev ] <- majority(x[ fac == lev ])
                                                           ^
```
[tests/testthat/test-sperrorest.R:214:73:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L214) *style:* **Commas should always have a space after.**
```r
              if (!is.null(fac)) pred <- majority_filter(pred, newdata[,fac])
                                                                        ^
```
[tests/testthat/test-sperrorest.R:228:67:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L228) *style:* **Commas should always have a space after.**
```r
            out <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                                  ^
```
[tests/testthat/test-sperrorest.R:259:11:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L259) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
          ^
```
[tests/testthat/test-sperrorest.R:259:22:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L259) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                     ^
```
[tests/testthat/test-sperrorest.R:259:39:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L259) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                      ^
```
[tests/testthat/test-sperrorest.R:259:50:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L259) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                                 ^
```
[tests/testthat/test-sperrorest.R:265:63:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L265) *style:* **Commas should always have a space after.**
```r
    if (!is.null(fac)) pred <- majority_filter(pred, newdata[,fac])
                                                              ^
```
[tests/testthat/test-sperrorest.R:279:57:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L279) *style:* **Commas should always have a space after.**
```r
  out <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                        ^
```
[tests/testthat/test-sperrorest.R:402:11:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L402) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
          ^
```
[tests/testthat/test-sperrorest.R:402:22:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L402) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                     ^
```
[tests/testthat/test-sperrorest.R:402:39:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L402) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                      ^
```
[tests/testthat/test-sperrorest.R:402:50:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L402) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        x[ fac == lev ] <- majority(x[ fac == lev ])
                                                 ^
```
[tests/testthat/test-sperrorest.R:408:63:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L408) *style:* **Commas should always have a space after.**
```r
    if (!is.null(fac)) pred <- majority_filter(pred, newdata[,fac])
                                                              ^
```
[tests/testthat/test-sperrorest.R:418:68:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L418) *style:* **Commas should always have a space after.**
```r
  res_lda_sp_par <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                                   ^
```
[tests/testthat/test-sperrorest.R:469:67:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L469) *style:* **Commas should always have a space after.**
```r
            out <- sperrorest(fo, data = maipo, coords = c("utmx","utmy"),
                                                                  ^
```
[tests/testthat/test-sperrorest.R:499:73:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L499) *style:* **Commas should always have a space after.**
```r
                                                              newdata)[,2]
                                                                        ^
```
[tests/testthat/test-sperrorest.R:639:1:](https://github.com/pat-s/sperrorest/blob/b8664495c85e5c2b0e0027200495cec885ef68e1/tests/testthat/test-sperrorest.R#L639) *style:* **Lines should not be more than 80 characters.**
```r
  readRDS("/Users/pjs/Servers/GIServer/home/shares/data/LIFE/mod/survey_data/data-clean.rda") %>%
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


It's fails with 'Segmentation fault (core dumped)'


There's an extra comma at the end of your list initialization.


unnecessary empty line


The lock_id is available just from logging into your account via a browser.

It can be seen in the URL, and also in the settings page.


same as above


nice 👍


ok


shorter like this any((m.quantity_done > 0 and m.state not in ['done', 'cancel'])for x in order.move_finished_ids | order.move_raw_ids)


po


原因：缺少`withRouter`包裹，导致NavLink无法获取router相关属性


here we go :)


It works, but it's a quite complicated solution.
A more simply one:
```
for (int i = 0; i < list.Count; i++)
{
    list[i] = "https://github.com/greenfox-academy/" + list[i];
}
return list;
```


Good idea. Could you implement this at some point?

You could also make it even more general with a regex match to deal with the lists@ thing. Something like `(?i)students@(.*)team254.com`


@torbenott
I'm only halfway through this job, but maybe I could get some input here.
I changed this line cause Spikegadget's exportmda function names files differently than "tetrodeN.mda".
It is unclear to me at this point how is datasets.txt used, but I suppose it should reflect the actual names of the mda files - is that correct?


Why do you need it? Why not use 'has_tracking'


sfsdffghfgh


Da moglo je i ovako da se odradi. Mali komentar(mozda je bolja opcija da se loklano obrise fajl jer tako moze da se eliminise opcija da se pogresno obrise neki falj).


Line 10


Yes please! I wanted to do it myself, but now I'm not certain what I will break...


Спасибо. Обязательно изучу материал.

2017-09-25 16:26 GMT+03:00 Igor Sukach <notifications@github.com>:

> Вот тут почитай про форматирование: https://docs.google.com/
> document/d/1llehTrlRkh7-r2F7gU7gRT7QjUG9PogVMIZHEqCAjxk/edit
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ShtanovaVika/itAcademy/commit/7dcc2bbf3f38d3a8ea906341811ce9c9f5108716#commitcomment-24544038>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AeUxfkfjqUMVLRo8MwxTAHIgdvzMfupmks5sl6oCgaJpZM4PiuDB>
> .
>


Not working interference