Nope, not neccessary.

I'm using dot imports after following their examples at https://onsi.github.io/ginkgo/

While generally dot imports makes reading code confusing, in case of ginkgo and gomega, we have a lot of the same function calls. So just Describe instead of ginkgo.Describe will read better. So I think we can restrict dot imports to just ginkgo and gomega. What do you think?


Forks


@grabear Something's still broken. Check the [travis-ci log](https://travis-ci.org/datasnakes/Datasnakes-Scripts/jobs/286792064).

I'm 99% certain it's in the [tests](https://github.com/datasnakes/Datasnakes-Scripts/blob/master/Tests/test_manager.py).


> Now onto resolving the problem: What do you suggest as a solution?
> I guess that we could set it to output untranslated "Preacher" string set by user in settings, and to display translated Preacher string if it's not set in settings.
Right, this could be one solution.

The other solution which I would personally prefer and I have stated previously, would be to decouple the menu item, the preachers’ slug, and preacher titles. To make things more clear:
1. The menu item should always be a translation of “Preacher” or “Preachers” (in general, menu items should be in plurals preferably and they *usually* should not be user defined, unless they are page titles or file names etc). Alternatively, to make the Sermon Manager more neutral in this regard the term “Preacher” could be replaced by “Speaker” (see why in 3.).
2. The user defined slug for preachers (or *speakers*) should never be used in any message. It should be left alone to only serve its purpose as a URL slug, nothing more.
In fact, I think it would probably be best to get rid of it overall and only serve pages based on auto-generated or user defined slugs from user created taxonomies, including the existing predefined English slugs (like `sermons`, `books`, `series`,…) and their respective locale specific counterparts. Does this make sense? I mean, do you understand what I mean?
3. With regards to preachers (or *speakers*), as some may have titles in some churches, preacher (or *speaker*) titles (like “Father”, “Reverend”, “Bishop” etc) should be a property of a preacher (or *speaker*) taxonomy. Hence, the menu item “Preachers” (or “Speaker”, like in 2.) could have a sub-menu item “Titles” where title taxonomies could be created and managed. These “Titles” could then be selected by the user as a property when creating or editing a preacher (or *speaker*).
I do not know, but I suppose that this is why the preachers’ slug has been initially requested for or has been abused for.

This would decouple all three things, add a feature for some churches, and would also simplify messages, including any l10n.


Yes I agree that it makes the test readable.
May be not use dot import for non gingko packages (other packages in torpedo) cases like "torpedo/tests".
I will move the "Step" alias to a separate package so that it can still use the dot import.


This change is insiginificant.


approved A


WIN!!!


remove after testing


alpha and beta was forgotten to be used here


missing space before close of tag


@bcherny Was there any breaking change from 2.0.1 to 3.0.0?


I believe this is leading to some warnings under julia 0.6:

```
julia> using TensorOperations
WARNING: Method definition ntuple(F, Base.Val{N}) in module Compat at /home/garrison/.julia/v0.6/Compat/src/Compat.jl:525 overwritten in module TensorOperations at /home/garrison/.julia/v0.6/TensorOperations/src/TensorOperations.jl:13.
WARNING: Method definition reshape(AbstractArray{T, N} where N where T, Base.Val{N}) in module Compat at /home/garrison/.julia/v0.6/Compat/src/Compat.jl:523 overwritten in module TensorOperations at /home/garrison/.julia/v0.6/TensorOperations/src/TensorOperations.jl:11.
WARNING: Method definition (::Type{Base.Val{T} where T})(Any) in module Compat at /home/garrison/.julia/v0.6/Compat/src/Compat.jl:520 overwritten in module TensorOperations at /home/garrison/.julia/v0.6/TensorOperations/src/TensorOperations.jl:8.
```


Some side effects makes compiling with openmp break


Hmm…, yes, if we use ‘me’ topic as medium to self-talk feature. But when we use normal `p2p` topic then above condition won’t be happening isn’t it?

So the idea to access self-talk feature was by subscribing to user own id, for example if user have id `XXXXX`, then it would be subscribing to `usrXXXXX` rather than to `me`. By using this we would still use normal `p2p` topic, right?

Ah yes, the reason why I’m asking this question is just merely because I want to explore some aspect of Tinode which maybe I’m not aware of. So I think I will still follow your suggestion since it is simpler & the most important thing is it doesn’t break compatibility with original Tinode. But I’m just curious maybe there are some aspect which I’m still not ware of. Hehe...


One space after `if`.


One space after `else`.


This function could have another overload without needing to pass the `C: var tensor` just to be more ease of use.


redirect_to tweet_path(@comment.tweet)
みたいな書き方出来ないのかな…？


Uh, `pass`? What does this mean? I don't get it.


@jbkly We updated ngcomponent (which react2angular depends on) to a new major version. If you used ngcomponent as a standalone in your app, but didn't declare it as an explicit dependency in your package.json (and instead relied on it being brought in by react2angular), the change have the potential to be breaking for you.


Compiling in release warnings:

`p_operator_blas_l2l3.nim(150, 5) Hint: 'order_C' is declared but not used`


Good, but should store text in strings.xml


Good, but perhaps `"firstName"` should be a class constant.


Should be `MangledNameActivity`.


//Riders on the stone


Use a constant variable for the key `"firstName"`.


Need to check that `pass` is not null first.


Good.


Don't use leading spaces - your data is "dirty."


Need to check for null.


Not right. Use the `%` trick remember?


No space after `new Intent`. Should be `new Intent(...`


Can instead use `finishActivity()`.


Space between `)` and `{`.


I don't think this is correct. `SiteSetting.force_https` is meant to force HTTPS for requests going to the Discourse site and not external links. This is going to break any external links that do not support HTTPS.


Thanks!


The behavior isn't right here. Should just create a new complete name and update the EditText.


change merged to master: e334fd306e77111b5c53879ec6aad41d69b5b226


yes, it is a constant. `let` is for any variable that needs to be re-assigned at any point in the code, `const` is for everything else (variable only receives an initial assignment and that's it). `var` should not be used.


thank you


I mean 0, changed to zero. fml


@latobarita : retry


JW, does removing the trailing Z have any adverse side effects?

According to the [SVG spec](https://www.w3.org/TR/SVG/paths.html#PathDataMovetoCommands), there is a distinction between closing a path with `Z` and manually closing it yourself. Specifically:

> When a subpath ends in a "closepath," it differs in behavior from what happens when "manually" closing a subpath via a "lineto" command in how ‘stroke-linejoin’ and ‘stroke-linecap’ are implemented. With "closepath", the end of the final segment of the subpath is "joined" with the start of the initial segment of the subpath using the current value of ‘stroke-linejoin’. If you instead "manually" close the subpath via a "lineto" command, the start of the first segment and the end of the last segment are not joined but instead are each capped using the current value of ‘stroke-linecap’.


@skillinhow Não precisava ter tirado a chamada dessa função aqui , visto que na própria função, caso o usuário não tenha facebook setado, ele não vai ser exibido


If both path A and path B end with a Z, the Zs are added back at the end. It just simplifies the logic of the interpolation to remove them before doing the path expansion.


Everytime you pull the project, you should modify the password of database in the setting file.


SIGUSR1 зарезервирован для внутреннего использования сборщиком мусора. Тут точно всё ок?


@ssscassio Ele exibe o ícone, mas o link do ícone fica como undefined, retirei apenas pra sumir com o ícone, já que não aponta para link algum.


Of course, I'll do that soon. Thanks!


> This will emit an empty string and reset the v-model

Maybe after your changes, but right now on dev it doesn't.


Proper speeds from sniff added here:

https://github.com/cmangos/tbc-db/commit/7485c040f000567593c330d3dc838b5247127631

I don't see evidence of them running in sniff and it appears identical visually when compared to retail video.
Perhaps the problem is not so apparent without these changes.


@jasminejaksic avoid making direct commits to master unless there is an emergency and noone is around to approve a PR


Kinda useless...


> Hmm…, yes, if we use me topic as medium to self-talk feature. But when we use normal p2p topic then above condition won’t be happening isn’t it?

A normal p2p should be perfectly fine. Create a user Test or something like that and create P2P topics for everyone with that user. It will have an added value in that in the future you will be able to attach a bot to that Test user and have it reply to customers. It will not be possible (or much harder) to do with 'me'.

> So the idea to access self-talk feature was by subscribing to user own id, for example if user have id XXXXX, then it would be subscribing to usrXXXXX rather than to me. By using this we would still use normal p2p topic, right?

Not quite. 'me' may present itself to users the same way as a p2p topic, but it's not a P2P topic internally. 'me' was not intended to be used this way. It almost certainly will have unintended side effects.


更新了3.1.12版本后下拉刷新不会驻留，直接回去了，而且页面会上移10个像素


commit log says
`Add successfull -> successful`
but the actual commit is `droped->dropped`
why?


サイト名を考えよう。
僕のクジラポートフォリオは「LULLABY」、
「lambda(ラムダ)」という名前をつけた卒業生もいたよ。
参考までに。


Perhaps clang is the bad guy (although I still suspect a broken VM as well) - gcc did build for AVX even without this.


You'll want to `description` -> `render` this one, too.


Probably a bug in the script I use for commits.


Please don't push directly to master... We already have a bunch of things to fix because "we'll fix them later" and it's just piling up. I know we need new features ASAP but let's do it at least half properly - otherwise we won't fix it ever. Example being current Makefile, which is really hard to use and even harder to refactor into CMake, we don't want to repeat that mistake.

Let's think and discuss what we're going to do before actually doing it - will save us a lot of time in the long run.


张鑫更新异步通知


`<!-- /.menu_link -->`
このコメントは、不必要に思えます。
一行で完結しているため、書かなくてもわかります。
逆に、不必要なコメントが増えるとコードの可読性に悪影響を及ぼすので、そのコメントは必要かどうかを見極めましょう。


Isso é verdade para a parte de diretores, mas a de membros estava correta. O certo mesmo era adicionar a mesma chamada de metodo para o codigo do diretor


さっきと同じ、
いい名前を考えよう。


有道理！看来代码足够清晰是首要的！


Hi @Mogeng,

1) When you write in the code that covariates_transition = [], do you mean that for each time step, the transition matrix of hidden states has nothing to do with input? So with covariates_initial=[], covariates_transition=[], covariates_emissions=[[]], it just HMM?

2) I need P(X_t = j | X_t-1 = i, U_t ) rather than just P(X_t = j | X_t-1 = i). When I tried to give a reference to the transition matrix (covariates_initial=['a column in df'], covariates_transition=[''same column in df']), the program starts reporting some errors.
![image](https://user-images.githubusercontent.com/26510270/31474495-28cc176a-aec9-11e7-90b6-20e8b172b0d0.png)
I also tried covariates_initial=['Range', 'Range'], covariates_transition=['Range', 'Range'] (Range is a column in my df) since we have two state. However this time I get:
![image](https://user-images.githubusercontent.com/26510270/31474675-3707645a-aeca-11e7-9e29-ea93de339265.png)

3) Another thing is that, do I need to give the covariates_initial the same data as the covariate_transition? Because from the paper _A Generative Model of Urban Activities from Cellular Data_, it seems like the initial model is just the initial state distribution.


このあたりも、不必要そうなコメントが書かれています。
整理しましょう。


Necromancers everywhere.


Why would you want to refactor into cmake.  I don't want to do that.

But what jeff pushed to master was just a text file that isn't used.  I don't see a problem with that.


ここは、`<br />`で改行するのではなく、
リストにして整理しましょう。
コードの見やすさが格段にアップしますよ。


commited by Lena Arafa


あと`<br />`周りも整理しましょう。


Committed by Lena Arafa


ok


If it's not used then we don't want it in the repo.

Current Makefile is too hard to use, @navdeep-G @mmalohlava @RAMitchell and I spent a whole day now trying to add a new algorithm and still can't figure it out. If we have such problem with it we cannot expect from outside contributors to even try.

CMake is much cleaner and easier to use.


why are these values being grabbed if they aren't being used?


`<div id="profile">`
ここは、sectionタグを使ってもいいですね。
section、aside、article、div
これらの使い分けはちょっと難しいですが、余裕があれば調べてみましょう。


Thanks for these code cleanups!


Hmm but the `TEST="fresh test"` fails also (but is allowed to fail), and that's gcc. I wonder if it may happen that we build on one VM and execute tests on some other one? And they have a heterogenous cluster? Perhaps we need to add this to all configure lines.


This commit is the end of part 7 of this video series: https://www.youtube.com/watch?v=aZ16pkrMkZE&index=7&list=PLillGF-RfqbYRpji8t4SxUkMxfowG4Kqp


I've had major problems with cmake.  It hides too many things and if something doesn't work there is no way to make things work.  It took me 3 days of suffering with cmake to get nowhere for a related thing for xgboost.

I don't think the problem is cmake vs. make.

If you guys pointed me to the code, I could take a look.  Why would you spend all day without asking me?  All I heard from navdeep is the error he got.  You should share the PR and I can take a look.


`.create` isn't the method that explicitly renders the view. Like you answered in the second question the `.create` is just a compound method that first initializes a new model instance and then also saves it into the DB


ここは、navタグで囲ってもいいですね。
nav → ul → li
って構造だと、これはナビゲーションだと一目でわかります。


有木有测试账户，亲


```shell
if [[ $OSTYPE == darwin* ]]; then
```


Don't ever edit a migration after you've already run it!


great job overall! Take a look at my comments above


もし、このセクションの内容が「制作物」や「作品」というものであれば、
"content"よりも"works"の方がわかりやすいかと。


Entendi, irei retornar então as chamadas dos métodos dos membros, e adicionar as chamadas para os diretores, junto com as correções dos Nomes/Cargos dos membros


What jeff pushed was in preparation for adding cuda9 to jenkins.  He could have started a PR, but I don't see a major issue here.


great job!


If you only correct the typo, does that resolve the exception?


svg形式でサムネイルを用意したのは、Good!。


Já corrige, relaxe.
Foca em corrigir a parada da navbar agora


`Cat.save`


As for pushing to master.  I try to avoid it, except when it's jenkins related it's not possible.  That is, not everything in the pipeline is tested with a PR, like S3 push.  So there's no easy way to test that part of the pipline in a PR.  Maybe @mmalohlava has some ideas.


ここのリセットボタンには注意が必要。
ユーザーが間違ってクリック及びタップをしないようにする工夫が必要です。
配置の工夫をするかリセット確認をさせるかなど。


nice job!


Actually both `new` and `create` can make instances of a model with or without any attributes!


用来存放启动数据库的数据


you have new build mesen ,share to me?


Arena does something like this:
```
if line.startswith("*boxcap"):
   _, id=line.split()
   boxcaps[id]=currentTextureId
```
etc.


Overall, looks pretty good. But did you create any tests?


ここも、最後のコメントはなくても大丈夫です。


year, i find this problem and i will try your methods in night, think you~


I pulled this discussion into a separate issue #573


The issue is that the project should be clean and user-friendly. Let's have a proper commit history, no useless files, good structure etc. If it's just a preparation then we don't need it in master. Duplicating the same file with only 1 changed line usually becomes a problem in the long run.

As for CMake vs Make - I'm not an expert at either but from what I've seen CMake is much easier to use and at least @RAMitchell seemed to be pro it. In any way even if we don't migrate to CMake we have to clean up/upgrade our Makefile - currently we have to add way too much stuff in there to add new files/algorithms. From what I saw that's not the case in XGBoost.

If you want to test S3 push in a PR you can just, as a test, change the Jenkinsfile so it pushes when the branch name matches the PR branch name instead of `master`. Most things can be tested, if they are not testable now lets strive to make them testable.


What is the decision on this? Put back or leave out?


此处有错误，第17行：
else后面少了:
所以程序在运行时出现报错：

Exception Type: | SyntaxError
-- | --
invalid syntax (search.py, line 17)


Excellent! Thanks for writing tests.


declare it as static if it is used by SYS_INIT only.


test demo


Ok


应该是# -*- coding: utf-8 -*-
这里打错了


We should make it with a new version 0.4. I think. that way when customers pull the image, they will still use 0.3. except that they specify 0.4 in the settings.


No need support running this image locally.


put these in version 0.4


Yes, cmake or make, things need fixing.  One other example of an issue is dependencies need to be automatically generated.  This is easy, just takes someone to do it.  If someone wants to redo everything in cmake, that is fine, I just don't want to be the one.

But, why wasn't I let in on the all-day activity of adding something to the makefile?  I'm nearly certain I could figure it out in 10 minutes.

But by changing the jenkinsfile, it's not a full test of what happens when it goes to master.  It's a flawed process.

@RAMitchell was not always a pro at it, that took a long while and I was there as he overcame its difficulties.  If the concern is new users, we can't expect them to need such a learning curve.  I could represent the type of user we could have, and I really tried hard at some point to fix something in cmake, and it was impossible because things are only done at a high level.  That can be convenient if all you do are simple things, but it can also mean doing something simple (that's not already built-in) can be impossible.


log file name


My 2 cents on various things:
-cmake does have a learning curve but is significantly easier to maintain. It will also enable you to easily support windows. If you don't do this you are throwing away a large potential userbase.
-Agree with @mmalohlava about adding duplicate lines of code. Let's be honest no one ever goes back and fixes things. Either uphold standards for commits or live with this code forever.
- Its good to do PRs and let them sit for a day or so to allow comments. Nothing is ever so important it needs to get immediately pushed to master.
-In general this repo already has a large technical debt to the cuda code duplication and lack of standard structure across projects. It will take significant refactoring effort to reduce this debt.


I'm not sure if "onlyAdmin" modifier works if the function is constant read only, and usually there is no need for the onlyAdmin modifier if this is a read-only funciton.


else не принимает дополнительных условий для проверки, для этого есть elsif
И зачем проверять второе условие? в таком случае человек с ростом 110 останется без сообщения


All your if clauses basically do the same.
Does hoisting the duplicated while loops with strtol (not the operation itself though) really change some observable behaviour.
Just convert them all into an array of int beforehand, with error handling.

Separation of concerns, and avoidance of duplication of concerns


If UtilsCore.isAIR is doing the same, please remove commented out code instead leaving it here.


pickle should take filehandles in binary mode (cf
https://groups.google.com/forum/#!topic/nltk-users/ArDqxAyuBGA )


Is this no longer required?


Теперь у тебя определится либо прямоугольный, либо равнобедренный, либо равносторонний.
А бывает, что прямоугольный треугольник равнобедренный.
оператор if...elsif...end выполняет только одну ветку. ту, которая первой пройдёт условие.


.title


why ?


Addresses issue #17


Done. Thanks Philippe!


This commit fixes issue Issue #9235, which deals with the DC-2017-04-003 advisory released by DefenseCode. Magento calls it APPSEC-1626.

In Magento 1, similar vulnerabilities are fixed in version 1.9.3.3, or the security patch SUPEE-9767.

Hopefully this comment can help someone like me trying to Google the vulnerability and figure out whether or not it's still current.

Magento have not made it clear in the issue that they have fixed, or even looked at it, and are blocking the community from commenting on the issue, so nobody can help them shed light on it. According to DefenseCode, Magento had been aware of this remote code execution vulnerability for literally half a year when they released this update.


Google C++ Style Guide [says](https://google.github.io/styleguide/cppguide.html#Forward_Declarations):
> Avoid using forward declarations where possible. Just `#include` the headers you need.

But it seems like the #include not needed here at all. I'll remove it.


pas encore de fichier youtube, la fonction est pas faite.
Voir sur la gem carrierwave pour upload un fichier depuis la seed


@jaweii __
![qq 20171012180435](https://user-images.githubusercontent.com/32348909/31490908-228cf004-af78-11e7-9031-e7c49f9954dc.png)
路由可以正常转场，但是为什么报错了


textvariable = self.display not "self.display"


@tabrath I wanted to gauge your opinion on this change? I have been using alternate language versions of libsodium and noticed a difference in the box key generation code. The regular libsodium-net allows the creation of the key by providing a private key. However I found that providing a seed and it calling the underlying libsodium crypto_box_seed_keypair method to be useful as well. Obviously both take a byte[] so have elected to change the name of the method. Any thoughts?


It cause trouble on native hardware (Core2Duo with variant tsc) and trouble on Qemu on Linux with the timer* tests.


```
> make clean
> rm -f src/libsndfile*.def
> find src -name \*.def
src/test_endswap.def
> make
> find src -name \*.def
src/test_endswap.def
src/libsndfile-1.def
```
Is this a Cmake vs autotools problem?


Pour les tags, plutot faire Song.create(..., tag_list: "battle")


`path.isBlank()`


Als je op een kaart klikt veranderd deze nu naar zijn oorspronkelijke image. Deze wordt opgeslagen in een variabele. De tweede kaart die je klikt idem. Daarna wordt er gecheckt of ze gelijk zijn. Zijn ze gelijk dan blijven deze open staan.


Please reorder parameters so that the two flags are together in the parameter list


x = 3.14

pi = x

area = pi * radius ** 2
print(area)

need ad print on a end of code..then it works


LogicalPos is for an unfinished entity view mode, where it shows all entities in a flowchart/diagram view. It's a 2D vector for the object position, but you really don't need to do anything with it (other than perhaps maintain the exisiting value).


`newFiles.map`


It wont work with 3.0.0 it has to be the old version. Or did you get it to work?


`newFilePaths.sorted()`


I am not super fond of having runtime instead of linktime errores, but I guess it is hard to do better without using build system tricks. But it is fine for now on and we can wait for https://github.com/janestreet/jbuilder/issues/136 to land.


thanks :)


the runtime error should only happen with a program (not a library). Maybe change the error message to make that clearer?


Oops, I'm not into rust but there people around working on it.


![2017-10-12 6 16 50](https://user-images.githubusercontent.com/8784712/31491333-95ff02dc-af0c-11e7-9ec2-1a8e59f66268.png)

does this mean that it works?


I've updated the branch: https://github.com/Ladicek/arquillian-cube/commits/openshift-await-route

I still didn't even try it :-)


Try to open the report and see if the coverage and the code lines up. But it looks good that this part is there.


Named arguments here please?


从报错信息来看，是由于元素的classList属性是只读的，但是有对其进行写操作的行为，所以报错了

从图中看不出是哪儿造成的


почему первый раз по этому заданию не было замечаний?


Ram, have we tested this module?


I think this one should be merged, right? Because the dash is also used in the rwc2017 knowledge.


Ideally we should try to remove the generated cmo/cma and just install the cmis. Can you try with to tweak the pkg runes with something like:

```
Pkg.mllib ~api:["Digestif"] "common/digestif.mllib" ~cma:false ~cmxa:false ~cmxs:false ~dst_dir:""
```

or just:

```
Pkg.lib  "common/digestif.cmi"
```


![](https://github.trello.services/images/mini-trello-icon.png) [Content changes](https://trello.com/c/6eGBjsif/308-content-changes)


Does it make sense to support MPP here?


Static import please?


Dette burde vært en boolean, ellers kan man få NullPointerException hvis noen sender inn null en dag.


Мы должны хранить объекты поездов, а не их номера и типы.
Можно хранить их в хэше, по ключу - названию хранить объект поезд, но гораздо проще складывать в массив.


Using `project` as parent doesn't seem right. Use `executionEnviroment` instead?


Uncaught Error: Element ref was specified as a string (NAME) but no owner was set. You may have multiple copies of React loaded. (details: https://fb.me/react-refs-must-have-owner).


vhj


Is this final patch? Not sure why we set it to noop and then suddenly bfq.


x = 4

y = 5

a = 3*(x + y)

print(a)

#need to add "*" to multiple and need to put print to coorrrect the code


I think the issue has been fixed. You may see the OSK
![image](https://user-images.githubusercontent.com/28331388/31491767-a6333ab8-af72-11e7-95a7-b6c5f7de6497.png)

 and touch layout here.
![screenshot 2017-10-12 17 22 33](https://user-images.githubusercontent.com/28331388/31491805-c7edac1a-af72-11e7-9396-b817c0ba992e.png)

One the PR is merged, the update would take effect. Thank you!


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/discourse-build-is-broken/71925/1


Смотри, если ты в метод при вызове передашь объект класса Train, то не сработает удаление из хэша. @trains.delete(train) # здесь требуется название поезда, тк оно служит ключи для хэша
"#{train.train_name}" # здесь ты обращаешься к методу объекта поезда


So far the final version is running well too, but I want also ZaneZam to sanity check it first.
So it will be something for next weeks update.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2616](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45571&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


ah yeah it works, you can test it via `yarn example -- --coverage`


I guess you didn’t mean to leave this here


Awesome :)


It really bugged me to have to use such an old version.


@dmvolod Thanks for pointing this out. It has been addressed with the following commit already: https://github.com/apache/camel/commit/2439863b60dbf5ec182ab0f86266272f4220c6d5


orz你太强了


Dom元素（删除元素）.html

好


Removed.


Use `apply` and convert the methods called to extension functions?


In which case is this useful? I think that showing this as a completion variant will only confuse users into thinking that they have to use this syntax instead of passing the values directly.


Mir ist nicht klar was dies prüfen soll.

Ich hätte es so geprüft:
`if (message[i] == message[i+1])  `


I'm not very happy about this workaround, but I'm fine with it. However, shouldn't we wrap this piece in a try/finally clause in case an error occurs, so we don't leave the name modified?


@fabiangreffrath, this is probably another my suggestion.
Take a look: https://www.youtube.com/watch?v=gQSPoLEkvls


To be honest I haven't done a benchmark, as there are mor importent things to fix. This is the way bullhead does it. I'll probably drop this in future.


breaks IE 10


breaks IE10


import com.kymjs.rxvolley.toolbox.FileUtils;
找不到该类，编译未通过


Can't we write this like `this.props.blockRef` without arrow function?


Sure, I have no preference.


but this means that allready generated clients will expire?


You could set the default option to checkpoint=True if you like - it was only false while the checkpointing was broken.


Looking forward to Pre3 ;-)


by the way, when we merge https://github.com/typesafehub/akka-commercial-addons/pull/152 you have to comment out the below cassandra-journal-multi-dc-eu-west and cassandra-journal-multi-dc-eu-central until you test that thing, otherwise they will be picked up automatically


Thank you a lot. You make my day great.


Переменные нужно называть ближе к сути.
Здесь не понятно, что ты ожидаешь в new.
Ты добавляешь станцию? используй station


Holds PSVM(String [] args) and mainMenu()


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/discourse-build-is-broken/71925/1


тут снова проблема. По-хорошему, ты хранишь массив станций.
Удаление по названию из массива не пройдёт. Как и при создании, нужно передавать в метод станцию и её удалять


The check is not related to MPP: it's here just to disable this configuration for non-Kotlin/JS modules


Tests.py is stored in tests folder. It needs to know where db package located.


case-insensitive when comparing StoreLabel.key, the string store in StoreLabel is still case-sensitive


Yes, like I mentioned in the [issue](https://github.com/Source-Python-Dev-Team/Source.Python/issues/211), this is a lazy fix, but the only one that works with the current implementation of the logging API without having to re-design a good portion of it or duplicating the code with a new formatter. Error shouldn't occurs, but you are right; this is probably a good idea to add a try/finally.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2616](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45571&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 298, ignored: 3 Build time: 00:17:55


What's the impact of this on closure size? it seems undesirable to require a dependency on multiple versions of Qt...


туплю сама не знаю зачем проверять.
Не совсем поняла почему человек 110 останется без сообщения ему должно быть сооющение Your weight is perfect enough т.e 0 вес должен быть включен в сообщение Your weight is #{weight}? а все что меньше 0 уже падать во второе условие правильно поняла?


jzq太强了


# Zhiqian Jin太强了%%%


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2617](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45580&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


I haven't had a chance to properly clean up my code. But this file was being used to store application configs initially, I got issues and decided to bundle it all up in my app/views.py file. Part of my refactoring in the next few days will include moving configurations into a central file( in the init.py file)


Better one after all!


@Tcharl Nevermind, see https://github.com/jhipster/jhipster-dependencies/pull/8


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


I am not sure if this will work, `target->player->playerstate = PST_DEAD` is set 8 lines later. You don't need this check, as you are in `P_KillMobj()` anyway.


the scratch is not freed this way, won't this cause memory leak?


Тут нужно еще проверить, что вагонов больше 0, а то можно в отрицательные числа уйти


Is tests folder a package? When running python -m tests.test, we can expect
repo root will be in python path.

12 окт. 2017 г. 13:47 пользователь "Eugene Karimov" <
notifications@github.com> написал:

> Tests.py is stored in tests folder. It needs to know where db package
> located.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/mini-kep/db/commit/cc36ac7df92f61965b1d4aa2994f32f5c8e5ca96#commitcomment-24930116>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AI1grvkRRdRlK06qPF1mSFf5Zaorlc6Xks5sre5CgaJpZM4P20TS>
> .
>


почему массив? поезд может быть в один момент на одной станции


why it didn't work before? did they change something so it works now?


I like this idea, though I wouldn't add it to `P_PlayerThink()` which is called each tic. I would also add it to `P_KillMobj()`, just as the resetting of the `bonuscount` field.

BTW, what is `!INVERSECOLORMAP` supposed to mean? Not 32?

How about night visor effect, should this get preserved ot nor?


In the case of MPP, the module containing the sources will not be a Kotlin/JS module, but rather a common module with a JS implementation.


karthikeyan GIT project


![](https://github.trello.services/images/mini-trello-icon.png) [UC3: Der Anmeldebutton ist 10px hoch und 30px breit.](https://trello.com/c/GtTnxGS9/3-uc3-der-anmeldebutton-ist-10px-hoch-und-30px-breit)


Oh, I thought you meant as a method, but now I see what you mean. :)


![](https://github.trello.services/images/mini-trello-icon.png) [UC001: Als Student/User sollen jene Tastaturanschläge in einem schreibgeschützten Logfile aufgezeichnet werden, welche nicht im Prüfungsprogramm getätigt wurden bzw. im Internet erfolgt sind, somit kann der Lektor nachvollziehen was außerhalb der Prüfung eingegeben wurde.
Das Logfile ist gegliedert nach Tastaturanschläge, Internetseiten und Programmen, diese sind in separaten Abschnitten mit dem Datum und Benutzer dargestellt.](https://trello.com/c/2T90Zxoh/1-uc001-als-student-user-sollen-jene-tastaturanschl%C3%A4ge-in-einem-schreibgesch%C3%BCtzten-logfile-aufgezeichnet-werden-welche-nicht-im-pr)


Hmm, I was quite sure com.qualcomm.location caused the soft reboots. Took it out - no crashing, put it back in - crash.
Retrying with your newest vendor updates.


![](https://github.trello.services/images/mini-trello-icon.png) [UC002: Der Student kann lediglich das Programm aktivieren und nach der Prüfung deaktivieren, während der Prüfung hat der Student keinen Zugriff auf das Logg File.](https://trello.com/c/hDiNTJTS/3-uc002-der-student-kann-lediglich-das-programm-aktivieren-und-nach-der-pr%C3%BCfung-deaktivieren-w%C3%A4hrend-der-pr%C3%BCfung-hat-der-student-k)


Dang! Sure! >< I just need to place `target->player->bonuscount = 0;` right below `target->player->playerstate = PST_DEAD;`


No of triangles in pascal triangle


I don't like this part of code, `start_date` assigned in if statement, and referenced in another independent if statement. Of cource, this if statement is the same `if start_date_str: `, but it's really not a good practice


Changing the button to submit type implies that click handler should cancel the submit event. We may need to check if `js-save-address` event handler is still working..


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2617](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=45580&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 298, ignored: 3 Build time: 00:12:16


@diabl0 : You probably mean https://github.com/sensiolabs/SensioFrameworkExtraBundle/issues/519 ?


According to your notes, shouldn't this be
`single_dist_phase_op = intens_op * (1j * wavenum / 2 * plane_wave * prop_op * ray_trafo + plane_wave)`
?

If so, this can be simplified to
`single_dist_phase_op = intens_op * (1j * wavenum / 2 * prop_op * ray_trafo + prop_op.range.one())`
right?


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


you can gitignor this


Successfully tested by building for Arduino Zero


> Is tests folder a package?

No, it's not a package.
> When running python -m tests.test, we can expect
> repo root will be in python path.

Right, if you run it from root folder. I guess Travis CI runs command like `path_to_python_venv/bin/python3 path_to_project_folder/tests/test.py'


二分查找


Please make this configurable as different user has different TTL policy / standard.


I this is noting


What's this file?


要是menu的addEventListenr 和addEventListener('backPressed‘) 能获取 url就完美了。


По заданию нужно хранить не массив станций, а объект класса Route

> Может принимать маршрут следования (объект класса Route).


Раздели на три раздельных метода.
Методы текущей/предыдущей/следующей станции должны возвращать просто объект станции, без всяких puts. Что с ними делать потом - забота вызывающего кода.


Hello. Have you worked with 6688:6850 ID 203-RW device? Looks like it using different protocol and during the tag write the new tag id is encoded by the unknown algoritm during the sending from application to usb interface.

![software-screenshot](https://user-images.githubusercontent.com/416680/31492856-7d2f048e-af55-11e7-821a-ff082a9ecafb.png)

Software got from the same merchant:
[https://drive.google.com/file/d/0B-yCGjUOmtVsMHBfTnBORnhnVm8/view?usp=sharing](url)

Wireshark dump of the communication
[id-203-rw.pcapng.zip](https://github.com/merbanan/rfid_app/files/1379086/id-203-rw.pcapng.zip)


要是menu的addEventListenr 和addEventListener('backPressed‘) 能获取 e.title就完美了。就是获取网栏状态栏的标题


Hi @mrimal , @luizaandrade and @bbdaniels !

I changed the name of the github placeholder file command from iegitaddtxt to iegitaddmd as a markdown file is much more suitable for this than a txt file.

I made the content longer and more descriptive than in the txt version of this command. This is just a temporary file that I will remove after I have transferred this to the ado file.

Do anyone of you have any feedback? Add/Remove anything? Is something expressed in an unclear way? Can any language be improved?

Feel free to push any updates you might have!


you are right


Please replace this string value with parameter from inventory file that's available here https://github.com/shinesolutions/aem-aws-stack-builder/blob/master/ansible/inventory/group_vars/apps.yaml#L180


No I also used version 3 initially, but the setup was slightly different. I should have tried the later version again.


Помимо добавления индекса станции у поезда необходимо поддерживать актуальным список поездов на станции.
Для этого нужно вызывать соответствующий метод для текущей станции


Здесь тоже необходимо поддерживать актуальным список поездов на станции.
Схема такая: отправили с текущей станции, изменили индекс текущей станции у поезда, на новой текущей станции приняли.


:+1:  nicely done


замечание аналогично методу forward


[FAIR02.pdf](https://github.com/egonw/fopworkshop/files/1379092/FAIR02.pdf)
 aanvulling


Please an other format as *.PDF ...


@mark007 if you pulled in this revision of this library and used the the unbundled bootstrap file from the revision you are now that would be the only way at this stage, it may take some testing to ensure the versions work as desired though unfortunately.


The man page of GCC 6.3 `man -l /usr/local/genode-gcc/share/man/man1/genode-x86-gcc.1` states

>        -mcmodel=large
>           Generate code for the large code model.  This makes no assumptions about addresses and sizes of sections.
>           Pointers are 64 bits.  Programs can be statically linked only.

Do you have more information about the *static linking* restriction and if it affects our use case?


@prusnak I wonder if we can avoid using this if we move the firmware stack into CCMRAM? Right now, the firmware doesn't do anything with CCMRAM, except clear it. I don't know if you had any other plans for using it. The boardloader and the bootloader use CCMRAM extensively.


[R/helper_funs.R:63:21:](https://github.com/pat-s/sperrorest/blob/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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/dfaa3a9f0601a1185ebd7aab512823a178bfba35/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") %>%
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Thank you for your reply i plan to follow as many commits as you have for this kernel but on the s8 to get my alfa awus036h working, pcap capture apk works on it so power is no issue, should i do this from stock source or could i use a custom rom source as it already has been rooted

After i plan to tackle the HID


Добавляешь ты тоже не имя станции, а сам объект станции.
Понятно, что если передашь в метод объект, то всё отработает как надо, но аргумент стоит переименовать.


This is causing errors for me:

~~~
--- Lich: error: undefined method `each_key' for #<Array:0x3474f90>
	dependency:296:in `start_scripts'
	dependency:1230:in `_script'
--- Lich: dependency has exited.
~~~


> I guess Travis CI runs command like `path_to_python_venv/bin/python3 path_to_project_folder/tests/test.py'
We already have ```coverage tests/test.py``` that is picking path from root location, so ```python -m pytest tests.test```  or similar will do the same.

Generally, I see as path tweaking inside a module as last resort with other methods are not accessible, eg in Jupither Notebook in [here](https://drivendata.github.io/cookiecutter-data-science/#notebooks-are-for-exploration-and-communication), in other cases it does not seem very clean.


interesting, what does the filter line exactly do?

wouldn't the patchmanager folder also need to be included?


@Chey000 能够按照工作职责进行分类规划，这点不错，不足的地方除了泽宇 @ma283188200  提出的部分外，我补充下：规划主要要考虑逻辑、可行性。
- 逻辑性可以从以内容分类为顶层，在每个内容下根据时间节点进行细化任务；
- 可行性方面要注意两个关键要素：目标和时间，能够在规划时间内完成的目标，这样的规划才具备可行性。你的规划分类有了，也列出了部分内容，但是怎么去做？没有写明白。可以结合泽宇给的建议，根据内容分类-时间规划-目标拆解的递进方式重新梳理你的规划。


Why do you use flex and float at the same time ?


а поставить поезд на первую станцию?
было ведь правильно
`@index_station = 0`
а так методы передвижения поезда будут с ошибкой падать ри попытке изменения индекса


comparison starts here


[R/data.R:28:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/data.R#L28) *style:* **Lines should not be more than 80 characters.**
```r
#' @source Taken from \url{http://www.ats.ucla.edu/stat/r/dae/logit.htm}, direct download
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helper_funs.R:78:14:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/helper_funs.R#L78) *warning:* **no visible global function definition for ‘no_plot’, Did you mean 'boxplot'?**
```r
  plot_df <- no_plot(model)
             ^~~~~~~
```
[R/insert_or.R:90:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/insert_or.R#L90) *style:* **Lines should not be more than 80 characters.**
```r
insert_or <- function(plot_object = NULL, or_object = NULL, line_col = "red", # nocov start
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/insert_or.R:146:24:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/insert_or.R#L146) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      arrow_xloc_l <- -(max(plot_object$data$y) - min(plot_object$data$y)) *
                       ^
```
[R/or_gam.R:80:54:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L80) *style:* **Variable or function name should be snake_case.**
```r
                   percentage = NULL, slice = FALSE, CI = NULL) {
                                                     ^~
```
[R/or_gam.R:130:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L130) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam1_CI_low <- pred_gam1[1] - (2 * pred_gam1[2])
      ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:131:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L131) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam1_CI_high <- pred_gam1[1] + (2 * pred_gam1[2])
      ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:139:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L139) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam2_CI_low <- pred_gam2[1] - (2 * pred_gam2[2])
      ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:140:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L140) *style:* **Variable or function name should be snake_case.**
```r
      pred_gam2_CI_high <- pred_gam2[1] + (2 * pred_gam2[2])
      ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:142:24:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L142) *style:* **Use <-, not =, for assignment.**
```r
      result$predictor = pred
                       ^
```
[R/or_gam.R:151:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L151) *style:* **Put spaces around all infix operators.**
```r
      result$perc1[x] <- percentage*x - percentage
                                  ~^~
```
[R/or_gam.R:152:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L152) *style:* **Put spaces around all infix operators.**
```r
      result$perc2[x] <- percentage*x
                                  ~^~
```
[R/or_gam.R:178:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L178) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam1_CI_low <- pred_gam1[1] - (2 * pred_gam1[2])
  ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:179:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L179) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam1_CI_high <- pred_gam1[1] + (2 * pred_gam1[2])
  ^~~~~~~~~~~~~~~~~
```
[R/or_gam.R:186:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L186) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam2_CI_low <- pred_gam2[1] - (2 * pred_gam2[2])
  ^~~~~~~~~~~~~~~~
```
[R/or_gam.R:187:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_gam.R#L187) *style:* **Variable or function name should be snake_case.**
```r
  pred_gam2_CI_high <- pred_gam2[1] + (2 * pred_gam2[2])
  ^~~~~~~~~~~~~~~~~
```
[R/or_glm.R:62:39:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L62) *style:* **Variable or function name should be snake_case.**
```r
or_glm <- function(data, model, incr, CI = 0.95) {
                                      ^~
```
[R/or_glm.R:80:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L80) *style:* **Variable or function name should be snake_case.**
```r
  CI_low <- list()
  ^~~~~~
```
[R/or_glm.R:81:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L81) *style:* **Variable or function name should be snake_case.**
```r
  CI_high <- list()
  ^~~~~~~
```
[R/or_glm.R:86:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L86) *style:* **Variable or function name should be snake_case.**
```r
      CI_list <- as.data.frame(suppressMessages(confint(model,
      ^~~~~~~
```
[R/or_glm.R:92:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L92) *style:* **Lines should not be more than 80 characters.**
```r
      odds_ratios[[i]] <- round(exp(as.numeric(coef[[i]]) * as.numeric(incr[[i]])), 3)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:94:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L94) *style:* **Lines should not be more than 80 characters.**
```r
        CI_low[[i]] <- round(exp(CI_list[i, 1] * as.numeric(incr[[i]])), 3) # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:95:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L95) *style:* **Lines should not be more than 80 characters.**
```r
        CI_high[[i]] <- round(exp(CI_list[i, 2] * as.numeric(incr[[i]])), 3) # nocov
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/or_glm.R:98:7:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L98) *warning:* **local variable ‘or’ assigned but may not be used**
```r
      or <- odds_ratios[[i]]
      ^~
```
[R/or_glm.R:116:5:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L116) *style:* **Variable or function name should be snake_case.**
```r
    CI_low <- c(rep(NA, length(preds)))
    ^~~~~~
```
[R/or_glm.R:117:5:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/or_glm.R#L117) *style:* **Variable or function name should be snake_case.**
```r
    CI_high <- c(rep(NA, length(preds)))
    ^~~~~~~
```
[R/plot_gam.R:4:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L4) *style:* **Lines should not be more than 80 characters.**
```r
#' @description This function plots the smoothing function of selected GAM(M) models
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plot_gam.R:27:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L27) *style:* **Lines should not be more than 80 characters.**
```r
#' @param ci_alpha Numeric (range: 0-1). Opacity value of confidence interval shading.
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plot_gam.R:66:9:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L66) *warning:* **no visible global function definition for ‘gam_to_df’**
```r
  df <- gam_to_df(model, pred)
        ^~~~~~~~~
```
[R/plot_gam.R:103:28:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/R/plot_gam.R#L103) *warning:* **no visible global function definition for ‘ggtitle’, Did you mean 'title'?**
```r
    plot_gam <- plot_gam + ggtitle(title)
                           ^~~~~~~
```
[tests/testthat/test-insert-or.R:5:69:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L5) *style:* **Trailing whitespace is superfluous.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1 ^ 2)) + s(x2) + offset(x3) + x4,
                                                                    ^
```
[tests/testthat/test-insert-or.R:7:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L7) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:8:21:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L8) *style:* **Trailing whitespace is superfluous.**
```r
  library(oddsratio)
                    ^~~~~~~~~~~~
```
[tests/testthat/test-insert-or.R:10:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L10) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:14:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L14) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:16:8:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L16) *style:* **Use <-, not =, for assignment.**
```r
  ymin = ggplot_build(plot_object)$layout$panel_ranges[[1]]$y.range[1]
       ^
```
[tests/testthat/test-insert-or.R:17:8:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L17) *style:* **Use <-, not =, for assignment.**
```r
  ymax = ggplot_build(plot_object)$layout$panel_ranges[[1]]$y.range[2]
       ^
```
[tests/testthat/test-insert-or.R:18:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:23:1:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L23) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-insert-or.R:24:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-insert-or.R#L24) *style:* **Trailing whitespace is superfluous.**
```r
})
  ^
```
[tests/testthat/test-or-gam.R:9:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L9) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-gam.R:23:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L23) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-gam.R:43:36:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-gam.R#L43) *style:* **Put spaces around all infix operators.**
```r
  fit_gam <- gam(y ~ s(x0) + s(I(x1^2)) + s(x2) +
                                  ~^~
```
[tests/testthat/test-or-glm.R:18:3:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-glm.R#L18) *style:* **Variable or function name should be snake_case.**
```r
  fit_glmmPQL <-glmmPQL(y ~ trt + week, random = ~1 | ID,
  ^~~~~~~~~~~
```
[tests/testthat/test-or-glm.R:18:15:](https://github.com/pat-s/oddsratio/blob/631e92c75fd2fa9741b5e6d440e29a130fc51af1/tests/testthat/test-or-glm.R#L18) *style:* **Put spaces around all infix operators.**
```r
  fit_glmmPQL <-glmmPQL(y ~ trt + week, random = ~1 | ID,
              ^~~
```


If I understand correctly, the class `Loader` is only loaded when `getInstance()` is called (first time the VM need it, since it's static), and it's only at that time that `INSTANCE` is created.


Επικοινωνία Ανθρώπου-Υπολογιστή
Α.Μ 2016033 Μαρία Πασχάλη
 λινκ για super mario: https://github.com/courses-ionio/projects/blob/master/super-mario/index.md


@Chey000 所以说挑战才拉开序幕，内容营销部分涉及的知识相对庞杂，但是411法则的核心是不变的，你可以尝试多与泽宇沟通，相信你会进步飞速，加油！


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


Please replace this string value with parameter from inventory file. Introduce a new `dns_records.author_publish_dispatcher.record_set_name: author-publish-dispatcher` . Here's an example https://github.com/shinesolutions/aem-aws-stack-builder/blob/master/ansible/inventory/group_vars/apps.yaml#L181


Maybe, buffer overflow ? Anyway errno is set by _getcwd1().


@Yoeman321 在日志中可以尝试梳理下今天学到的技能、小技巧，当你回头看看这过往的日志，你会发现自己收获了不少，会很有满足感，加油！


Look like travis ci doesn't add root folder to sys.path


Addressed at fa7da78d803fdb1b6224be347e7ba8805e0a3d56


GUI界面中需要实现的功能如下：
1、 用户选择CanTool装置的COM口，并设置相应COM口的波特率、数据位数、停止位数；
2、 实现CanTool装置的CAN信息的设定（CAN速率、进入CAN工作状态、进入CAN初始化状态）；
3、 接受多个CAN信息，通过数据库内容进行解析，显示原始数据和CAN信号的物理值实时数据；
4、 用户选择多种形式查看CAN信号的物理值（仪表盘方式、实时曲线）；
5、 设置CAN信号的物理值，指定所发送的多个CAN信息和发送周期


Just put this
```
player->fixedcolormap = player->powers[pw_infrared] ? 1 : 0;
```
below `target->player->bonuscount = 0` in `P_KillMobj()`.


Thank you, fixed in 3aad9be85268e4a84b9bfcda1adc053b16a8fb0a


смотри, если бы код сработал по проверке weight > 0 потом weight < 0, то условие weight = 0 не попало бы ни туда, ни туда.
в задании

> Если идеальный вес получается отрицательным, то выводится строка "Ваш вес уже оптимальный"
0 - не отрицательное число
достаточно, например, проверить что weight >= 0 и выводим идеальный вес, а иначе текст, что уже всё ок.


self.class.post( base_uri + '/sessions')


Great, worked for me! Thanks!


hi i have a question. and i don't know where should i write it.
and it's how can i use this files 😸


You don't need to export the variables as they're not supposed to propagate to child processes


this is redundant


Thanks for the fix Dave

Tested-by: Bhupesh Sharma <bhsharma@redhat.com>


Ripping off my code, and violating the MIT license is shitty. Then having a fee go to you for an AMD miner which runs AMD code that you ripped off is even worse.

But... trying to re-license it? The code is mine, and I put it under MIT - for anyone to use under the restrictions of that license. Your codebase won't be infected with the GPL by using it, and quite honestly, this is the kind of thing that makes me stop releasing source code.


name added