You imported the google font but never declared it anywhere so it hasn't been applied i.e font-family: "Open Sans";


@jsmeix I believe that HTML files should not be part of commits as these can be build from the source asciidoc files - want to avoid that developer start editing the html files instead of the source files (adoc)


Use `var` for non-primitive types


Nice :D


Thank you.


Yeah 😢 . Writing it like typical bash script crashes on my amazing fish shell.


Thanks, I was encountering quite a few issues whilst working on my improvements to arena chunk management -- I'll pull the latest changes and submit a pull request.


Never ever leave commented code in your commit


This is violent, you override all crontabs. The uniq is useless here.


`var`


Looks good overall. While at it, I'd update the commit message. No need to reference the LVM driver.

I'd go with the folliwing:

```
RemoteFS: revert snapshot support

This change implements the 'revert_to_snapshot' method at the remotefs base driver level in a generic way. This should work right away with most (if not all) remotefs based drivers.

Note that we only consider the case in which the latest snapshot is being reverted (the Cinder API won't allow reverting volumes to other snapshots anyway, at least for the time being).

The RemoteFS based drivers (e.g. NFS driver, SMB driver) use COW images (e.g. qcow2, differencing vhdx) in order to provide volume snapshot functionality. Reverting a volume to its latest snapshot is a trivial operation in this case and can be achived by simply cleaning up (e.g. recreating) the latest image from chain.

Implements: blueprint remotefs-revert-snapshot
```


leave me alone


Ahh yes. Good call. Can use `>>` instead?


Does this fix issue[#2075](https://github.com/codenameone/CodenameOne/issues/2075) too?


Suddenly a wild `switch` appears :(


Oh, you mean specifically that case in permute.c? Well, I tried to be explicit in what case the fall through happens. It is a void statement though. The fall through only happens when the condition `ti->group < perm->start_group` is not satisfied. As @yanntm mentioned it looks suspicious indeed.


I think you should use the fab append function to a file. It is meant for this, to not include several time a line in a file. Not sure what is the file for root contrab though. Maybe create a new file in /etc/cron.weekly/


Why not just use the normal Link procedure and use a clause like for cards that can't be Special Summoned from a certain location? Just have it check whether the summon is a Link Summon and the phase is not the MP2 and it should be fine. Am I missing something?


There is no point to leave empty else statements in your code


You fixed the indentation in a code you are not using, and it's commented also? 🐰 🐰


Split long strings in new lines so it will be more readable


Hi Mike:

This code is duplicated, for more info you can see this review:
* https://review.lineageos.org/#/c/178917/

Thanks.


wat is this indentation?


I forget that we are still in a switch-case, but i have that magic number feeling here `8, 7, 3` :)


indentation 👻


e.g.
You save"abc" to DataManager under "my_login" alias
then you want to save something else "xyz" to DataManager under "my_login" alias
But it won`t happen because there is already "abc" under this alias so it won`t be replaced by "xyz"
You have to first delete "abc" form DataManager then You can save "xyz" under this alias.


Ah of course, now I see. Thought it was some special switch/case syntax, the 'else'.
It is correct. The transitions before start_group are stored for later emission. The ones after start group can be emitted immediately (as in PermNone).


Empty case/if statements


Хуйня какая-то


Yes this is visible on github


And this one also, poor Peti :(


test


In bash you can do
(crontab -l ; echo 'some new line') |uniq|crontab
I'll look up the equivalent fish syntax.


I'm being pedantic but benefits could have slightly more margin bottom to match the design.


This constructor uses a dynamic array (near 22) which is a variable length interface. This is ill-advised because it cannot deployed by another contract owning to the variable-length interface. Support for that sort of thing is new and partial, but improving all the time.

The `for` loop near 26 is unbounded (has no limit). This is an anti-pattern because it will cease to work at a limit that isn't clear.

A solution to these problems is to pass in a fixed number of addresses (2?) in every case and deal with the data accordingly. Iterative processes should be pushed out to clients (invert control) that invoke the contract functions to deal with one iteration at a time (fixed gas cost per transaction).


So now there is "Plugins" and an "Extensions" menu?
Also: string freeze?


And the binaries are on github also, TodoApp.exe, and the *.pdb file, if you add the gitignore later or the binaries are already pushed, you have to delete them localy and commit/push those changes. (The IDE will recompile them)


**@omkarnathsingh

At least you should get the next update 4.4.1 source so that it does not

The mood of the soul that eliminated informal telegrams should at least update the source quickly and quickly.

Please download version 4.3.1 as soon as possible**


Yo my boy!!
Now the build Wont Fail :)


Fixed #1


That sqlite storage file surprised me


Close #1


You're heading in the right direction here with the fixed length interface. The assignment as framed deals with exactly two recipients and we want to lay down some strong patterns for the handling of the funds. If you want to stretch it to "two or more" then you will have to find a better way to deal with `ifExists` because iteration (38) is a bad idea.

Some ideas are laid out here: https://ethereum.stackexchange.com/questions/13167/are-there-well-solved-and-simple-storage-patterns-for-solidity


Cool, removed commented code and switch-case also! 🕺


p.s. it's clearly what I _meant_ to do in generate_certificate; but I borked it. Would be worth correcting there.
Though I love fish and use it, I think that making such things shell-agnostic is needlessly hard. The other option is to capture the option of crontab -l; manipulate it in python, and feed the result to crontab.


`EFFECT_SPSUMMON_PROC` don't check `EFFECT_SPSUMMON_CONDITION`.
So _副話術士クララ＆ルーシカ_'s Link Summon use manually.


interesting, I had major problems simply trying to play video on both platforms, would be great to know this is fixed. We had to move off cn1 for that project.


Well spotted! Indeed this is an error in the code, but I do not think it affects major numbers in the paper. I need to check.


Talking to more than one untrusted contract at a time is an anti-pattern because it opens up a DoS vunerability. What if one of the friends is a contract and it decides to `throw`? NO one will be able to get paid. Yuck.

A solution to this problem is implement a "withdrawal pattern" in which the recipients request their funds (prefer pull over push). This will also solve the problem of iterating over the list.

Since the `msg.sender` counts as the first "untrusted contract", you cannot talk to anyone else. That means this function must receive the money, do some accounting, declare victory and stop. Introduce a new `withdrawFunds()` function to enable funds to leave the contract.


What about eventual cards that treat a Special Summon as a Link Summon? This effect isn't limited to a Link Summon by its own procedure, the current script doesn't account for that.


This is a jailable offense. Github most likely isn’t reading this Matt, however your  Bezel hacker is all over this and so are you. I thought better of you. Feel sorry for your children and Lindsay


@ChristianMoesl please experiment with `-include inttypes.h` rather than `-Duint64_t='unsigned long long'` as well. Which one is better?

Also, I am getting:

`Makefile:10: *** environment variable CC has to be set to gcc or clang.  Stop.`

on my machine...


Ebben a projekben azt probaltuk megvalositani,hogy egy adott szoba homersekletet szoba homerseklet kozelebe tudjuk hozni. Ezt egy arduinoval probaltuk megvalositani,ha a homerseklet meghaladja a 28 Celsius-fokot egy piros led felgyullad es a piezo egy 200 tonusu hangon szolal meg,ha 20 Celsius-fok alatt akkor egy kek led gyullad meg ez koveti a piezo sipolasa. Ha 20-28 Celsius fok kozott van a szoba homerseklete a zold led gyullad meg ez jelzi hogy a szoba idealis homersekleten van.
Kep: https://i.imgur.com/TwNNPml.jpg


Those would be xbaddr


There is also warnings here of course. Can you find a config that prevents all warnings during bootstrapping but still makes `cc` output warnings if people change selfie code?


❌ This shouldn't be a boolean. It should always use the port allocator. Otherwise there is nothing preventing another test from allocating that port later.


Since this is an Xbox symbol, our convention is to compare with NULL instead of nullptr.


❌ The port needs to be released when the server is shut down.


@ChristianMoesl it's *conversion*, not *convertion* :)


My issue is the lack of block after the else, so it only takes the first statement of the next case in the switch, instead of the whole thing, and it is a semantic change w.r.t. the previous code that had no else in this position.

So, contrary to the fall thru comments, that are non semantic and just please the compiler, this particular else l.212 is a semantic change.
So in the else case, we do execute line 214, otherwise we don't, we go to line 215 without doing 214.

Previously, if the condition was false, both 214 and 215 got executed.

Basically I'm looking for semantic changes in recent patches, since I have regression in my test suite.
This particular change looked weird, but I don't know if it's a bug or deliberate.

Jeroen's commit message does say "this actually found bug", so maybe it's a deliberate semantic change, but the control structure is really weird in any case (if/else overlapping with switch case).


This was the decision made, just here to implement :P I did remove the plugin browser extension menu items though to not have duplication. There is indeed a string freeze but Ruben was OK with it.


Same remark as below regarding nullptr.
Besides that, please consider moving this statement block into a function?


@ChristianMoesl Nice. Thanks!


в задании было. подправлю readme тогда пожалуй


Don't need contract's own address.


The _ony_ way how to do that, is to cahnge input names to original ones, not filled/remembered by any autofiller, including LastPass Password manager


Второй параметр лучше вынести в отдельную переменную


Once this code is merged with XREF address derive functionallity, this should use that


it would be more readable if you would store this in a 2D array (rows and columns), which contains only 0-s and 1-s, representing the walls and floor.


Pull request created.
https://github.com/fritzy/SleekXMPP/pull/470/files


Где?


Something to think about next time: Always look for what is generic across the sections. Could container be defined once and used throughout? Could you set margin and padding on the .quote or . benefits class so you can start to separate this from groups of css declarations concerning font values .


Тут на скраме поясни


You can use `var` here, and the `2` in the variable name is not necessary


`var`


тут тоже


ok, changed it


как я понимаю этого конвертера больше нет?


Unused import


Nice and simple - preventing rendering failure in the absence of a category/categories. I really should have thought of that. Thanks @pabloroman


тут какая-то херота с коде стайл


This one here is cool, but nthElement name could tell more


Looks like you didn't want to remove the line, did you?


That space is not necessary between the methodname and the parameters


Let's make this into a general-purpose utility that anyone can use to send a split to anyone else. This will simplify things because there will be no privileged user. Since anyone will be able to deposit and anyone with a balance will be able to withdraw, you'll want a structure you can use to keep track of everyone's balance succinctly. Consider:

`mapping(address => uint) balances;`


I like, that the wall-checker and monster-checker methods are separate. So that they are absolutely reusable and reading their name, I totally understand what they are responsible for.


I dont understand it


This was supposed to be changed at 6 places


You only need this is there is a function or other privilege reserved for the owner.


There are a few differences that make it better to compare with nullptr.

nullptr is defined as being a pointer type, where NULL is an integer of value 0:

This means that if you have two functions/macros/a template/whatever with the same name but different arguments. for example:
```
void f(char const *ptr);
void f(int v);
f(NULL);
```
The int variant will most likely be called, even if that wasn't the intent, because NULL isn't a pointer, where nullptr is.

Additionally, NULL is NOT a keyword, it requires support from included headers, where nullptr is a keyword and requires no such support

https://stackoverflow.com/a/14112019 provides a good explanation on why NULL shouldn't be used.


Should be better to explain to the user how to define his own json_encoder instead of override the default one. If swagger is inited after app.json_encoder = MyCustomEncoder, your encoder override the encoder of the project and developers could get mad ;)


Since i am working as a single person, i did not create an organization. Do i have to create an organization although i am working single?


You can refactor this method like this: `return  int.TryParse(input, out _);`, this will return boolean value of wheter you can parse it to int or not, the `_` at the end is reffered to an unused variable, but since you have to give and output the it, you can mark it like this


тут нужны пояснения


the magic number, 72 could be stored somewhere else, in a more descriptive variable. Otherwise cool fields.


Этого теперь нет?


You don't need `type`. Generally, avoid the use of `string` if at all possible. It is a complex, variable length and expensive type that is only partially supported, meaning unexpected issues and limitations. If possible, use `bytes32` and require clients do conversion to speak the contract's language, never the other way around.


[Jenkins vg tests](http://jenkins.cgcloud.info/job/vg/731/) complete for merge to master.  View the [full report here](http://cgl-pipeline-inputs.s3.amazonaws.com/vg_cgl/vg_ci/jenkins_reports/branch/master/index.html).

10 tests passed, 0 tests failed and 10 tests skipped in 9726 seconds

Tests produced 2171 warnings. 2157 were for lower-than-expected alignment scores


yes, correct. It's better to store them in a list. :)


я бы вынес в отдельную переменную


200 rows of switch-case with comments, still not using else ifs? :)


Cool, but still.. poor Peti :(


i have allready a running environment what can i do to change the expiration without have to reinstall or create again clients?


нужны пояснения


I'm confused...


коде стайл


Removed comment and `return;` cool! 😎


yep


Almost nice commit messages at all by the way :)


implement _import(path)_ and _importSync(path)_. The first method should be able to
return promise with imported data. The second method is synchronous and should be
able to return all imported data


This whole function is hard to understand. I guess there are hidden assumptions about the flow and who is talking and doing what. It looks like you're trying to determine who invoked the contract? I'm not sure, so will just comment on some steps for style/feedback. Overall, possibly the confusion arises from not needing this.


Hub for personal software projects.


Guards up front. Good. I don't think it does what the comment says it does.


For performance reasons it's better to use `current.original.overriddenDescriptors` instead


もっと詳しくしてください！


This series of `if{}` blocks is a signal that something is wrong. Generally, functions should have guards up front and then proceed with a little more certainty. Your "spidy-sense" should tell you the logic isn't as simple as possible.


Again, `it.original.overriddenDescriptors`


Codes and Sql


@ljelinkova , please review.


I wonder why it wasn't necessary in the previous commit?


Ofcourse!


indentation 👻 . Pull your title and link tags back in line with the meta tags


Perfectly o.k. for me to not have also HTML files in the sources.
I find it already confusing to have both doc/rear.8
and doc/rear.8.adoc - I always edited both of them.
Should I edit only doc/rear.8.adoc because doc/rear.8
is autogenerated from doc/rear.8.adoc?


indentation 👻


`java.util.stream.Collectors#joining(java.lang.CharSequence)` just as in your Java classes homework :)


You are fine to use text-align: center here.


## Global

| Changed            |   Status |
|:-------------------|---------:|
| `metadata_passing` |       +2 |
| `metadata_total`   |       +2 |
| `nodes_count`      |     +164 |
| `texts_passing`    |       +1 |
| `texts_total`      |       +1 |

## Words

| Changed   |   Status |
|:----------|---------:|
| `grc`     |   +24011 |

## Units

| Changed                                               | Status   |
|:------------------------------------------------------|:---------|
| `./data/tlg1272/__cts__.xml`                          | New      |
| `./data/tlg1272/tlg001/__cts__.xml`                   | New      |
| `./data/tlg1272/tlg001/tlg1272.tlg001.1st1K-grc1.xml` | New      |
| `./data/tlg1272/__cts__.xml`                          | Passing  |
| `./data/tlg1272/tlg001/__cts__.xml`                   | Passing  |
| `./data/tlg1272/tlg001/tlg1272.tlg001.1st1K-grc1.xml` | Passing  |

*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/3051)*


I meant that Core used some portal sorting algorithm which could do the same thing automatically.


Unlike other languages, we prefer to "Fail Hard" instead of `return false`. If this is successful, then `return true`, if it is not acceptable, then `revert()`. False is an option, so consider these 'bool' responses as having up to three possible answers.

true = acceptable, success, yes
throw = unacceptable, illogical, invalid
false = no

So, for handling of money and state changes, fail is almost always the correct response in place of `return false`.

Where would you use `return false?`

`function isThingKey(bytes32 key) public constant returns(bool isIndeed) {}`


Please, use `// TARGET_BACKEND: JVM` instead


We never need to do this because everyone can see `address.balance` for all users at all times.


ok


Not sure it matters much, but the [official documentation ](https://docs.docker.com/engine/reference/run/#runtime-constraints-on-resources) uses small letter for unit.

`2g`


Same here.


Also true


text-decoration: none removes the underline on the link. Could you have defined some css that both the sign up button and the get it now button share and then used the id's just to change the colour? The goal is to write the least code possible :0)


Anyway Bitcoin nodes will drop S2X nodes as soon as fork happens.

Isn't it better if nodes (on both sides) are connected to nodes who will not ban them?

Even more so for the minority network (with minority of reachable validating nodes), wouldn't this mainly hit S2X? Harms both networks. What this is supposed to accomplish?

Only benefit coming to mind for me, would be to pretend minority network is in a good shape (by seeming to have more connections), is there something good resulting from using this option?


Ths Last Of Us도 정말 정말 정말 추천드립니다...!

즐거운 추석되시길 바랍니다! ^^


width:30% , margin: 0 auto would make this quote look more like the design.


Very good, that you already can turn your hero. It could be even better, if you resource the image setting part in the hero class. So that the class and the class's methods could be responsible for overriding its own fields. In this case, you will be able to only call the object's methods to change stuff.


The following works, albeit it is a bit hacked. I will propose a PR in the future to include a cleaner version in the `FormProperty` Class.
```ts
if(this.formProp.root && this.formProp.parent.schema["required"]) {
    const splits = this.formProp.path.split("/");
    const id:string = splits[splits.length-1];
    this.required = (this.formProp.root.schema["required"] as Array<any>).includes(id);
}
```


are the newly added `BeanDefinitionPostProcessor`s not supposed to be sorted as well?

we are having an ordering problem where one `BeanDefinitionPostProcessor` adds multiple other `BeanDefinitionPostProcessor`s. those additional ones get applied in arbitrary order (i.e. not in the order that would correspond to their `Ordered` precedence).

thank you for any feedback/opinion in advance!


hj


The logic will collapse (good thing) if you make the `deposit()` function `payable`.


This benefits section could do with some more padding-left. See how on the design the text here lines up with the Relaxer logo at the very top


Consider

`function sendSplit(address receiver1, address receiver2) public payable returns(bool success) {}`


Я понял о чем вы говорите и представляю как это сделать, но правильно ли я понимаю, что контроллер, при получении нового сообщения, будет информировать каждый из буферов о текущем состоянии системы?


Very Nice


My Galaxy S8 crash with this


I guess this is checking that the sender is the privileged user. You would be able to that with `require(msg.sender == owner);` if it was needed, but you can go ahead and make it so anyone is free to use the contract to split funds.


age state in shekli bood kheili karetoon rahat mishod:
state = {
  products: [],
  formData: {
    ....baghie fields ha
  }
}


lvec[i] = (lvec[i] * L * 4.0f * 2.0f);

is not

lvec[i] = (lvec[i] * L * (1.0f / KS)) ? :)


Használjatok https-t a linkben, és akkor menni fog. De mégjobb megoldás, ha bower-t és grunt-ot használtok, és csináltok egy dependency JS-t, ami az ilyeneket tartalmazza


Voilà https://github.com/Homebrew/homebrew-core/pull/18968


Both work and some examples they give in the documentation are with upper case, but you're right. I'll change it. Good spot!


in va renderProducts ro mishe stateless component kard be tore mostaghel


When this is removed, the 'calculateSlides' functionality is restored.


Ok, thanks! This make sense.


The sender is an "untrusted contract" because it could be, literally, anyone or anything that calls the function and we don't control that. You need to limit your conversations to one untrusted contract at a time to avoid DoS vulnerabilities and other nasty situations. Therefore, you cannot push the funds forward - that would mean talking to another untrusted contract. You're limited to accepting the funds and doing some accounting.


felan ok e vali to zehntooon bashe ke ino tabdil konid be ye file joda va harja kari ba localstorage dashtid oon file ro import konid


Third contract. This a classic error. You'll have to think about `function withdrawFunds() public returns(bool success)`


CartPage mitone stateless fucntion beshe be rahati


This is really awesome and most challenging thing inside this project.


age cart toye localstorage vojood nadashte bashe chi ?
ye check bekonid hatman


Ha egy függvénynek egyetlen paramétere van, ami a this kontextus, akkor gyanús, hogy annak a függvénynek ebben az osztályban kellene lennie. De ez csak tipp.
Másrészt szerintem a key ne törölje önmagát, az nem az ő feladata, hanem a canvas-e, vagy azé, ami azt irányítja. Szerintem egyszerűen renderelődjön ki a képből, és a canvas vegye ezt észre, és render helyett törölje az obsolate objektumot.


mitone khodesh ye component beshe (Stateless) 🦄


Fixed by https://github.com/geoadmin/geocat/commit/5ced2197848633b6af16811622ae1641dc3fe67b


Since we don't use the Raspberry Pi Camera module, these values don't apply to our camera. The reason behind our choice is the difficulty of sealing the RPi camera cable.


@kcc would it be possible to replace `thread_local` by `llvm::sys::ThreadLocal< T >`, especially for systems that lack thread_local support (eg OS X 10.6) ?


Próbáljátok ki a CSS-es verzót, sokat spóroltok vele.
És ahogy már említettem, erre a charMap-ra ott van a String.codePointAt() függvény


Its compatible with all DLCs


The .DS_Store files are gone and ignored correctly, but I can still see many .project, .classpath and .settings files in the repository. These should be removed.

The fabric8-analytics-lsp-server-test-devstudio folder is deleted but I cannot see where it is ignored in .gitignore

I think the root .gitignore file might look like this
target/
.project/
.settings
.classpath
.DS_Store

And the entries specific to com.redhat.fabric8analytics.lsp.eclipse.ui plugin should go to com.redhat.fabric8analytics.lsp.eclipse.ui/.gitignore
/bin/
fabric8-analytics-lsp-server-test-devstudio
server


@kcc ignore that, I just found https://reviews.llvm.org/D32312


Gotcha


ima li pruc


Hi @zent-lim,
It might be better for you to **fork** Usbong Store, and then me send **pull requests**, so that I can merge pertinent files accordingly.

Thank you.

Peace,
Mike


test test 1


Használjatok web-safe fontokat, vagy telepítsétek a kívánt fontot CSS-ből. Ez így nem lesz ugyanolyan minden böngészőben


indentation! 👻


Quoting the `read_verilog` help message:

> Note that the Verilog frontend does a pretty good job of processing valid
verilog input, but has not very good error reporting. It generally is
recommended to use a simulator (for example Icarus Verilog) for checking
the syntax of the code, rather than to rely on read_verilog for that.

At the moment `-sv` is only required for things that would have a different valid interpretation in standard Verilog. For example `assert property (...);` could mean in Verilog we instantiate a module `assert` and the name of the instance is `property`. SV features that simply would be syntax errors in Verilog are not disabled when `-sv` isn't present.


No, it is certainly not intended as a semantic change. It seems indeed valid C to have `else case ...` as two consecutive statements, I didn't think this was possible. The next `case` statement should indeed belong to the switch, not the else.
Let me double check this.


## Global

| Changed            |   Status |
|:-------------------|---------:|
| `metadata_passing` |       +2 |
| `metadata_total`   |       +2 |
| `nodes_count`      |      +76 |
| `texts_passing`    |       +1 |
| `texts_total`      |       +1 |

## Words

| Changed   |   Status |
|:----------|---------:|
| `grc`     |   +22207 |

## Units

| Changed                                               | Status   |
|:------------------------------------------------------|:---------|
| `./data/tlg2806/__cts__.xml`                          | New      |
| `./data/tlg2806/tlg001/__cts__.xml`                   | New      |
| `./data/tlg2806/tlg001/tlg2806.tlg001.1st1K-grc1.xml` | New      |
| `./data/tlg2806/__cts__.xml`                          | Passing  |
| `./data/tlg2806/tlg001/__cts__.xml`                   | Passing  |
| `./data/tlg2806/tlg001/tlg2806.tlg001.1st1K-grc1.xml` | Passing  |

*[Hook UI build recap](http://ci.perseids.org/repo/OpenGreekAndLatin/First1KGreek/3053)*


Most interesting and important topic.


What you have done is an option but it's best to keep all your styling in the .css file. Instead include the link to the google font as a link `<link href="https://fonts.googleapis.com/css?family=Open+Sans" rel="stylesheet">` in the head


I think `EFFECT_SPSUMMON_COST` works.
See _Spirit Message_ cards.


indentation! 👻 pull your title and link tags to be inline with your meta tags


Hi Yegor.
Wrike recently published a library for working with sentry. Let's try to join forces to make one excellent library.
We are ready to consider any proposals for improvement and collaboration.

https://github.com/wrike/sentry_client
https://pub.dartlang.org/packages/sentry_client


omgggggg


Added missing events to containers.


Isn't that _throwing the baby out with the bathwater_? What if there is a DNS change while Bazel Server is running?
Why not returning the cached hostname and executing the Hostname lookup in a secondary background thread and then updating cache. This way at least the next call gets a current value.


This font size is a bit big. If you look at the hierarchy of the design, the logo at the top is the biggest font on the page. Also for now - stick to px when defining your font size. If you do it by % it will change depending on the size of the screen. Ask Matt in class about this when we get to responsive websites :)


F$&¥%ck yeah!!!
Thanks!


> This commit turns kdeLocale5 (and hence kde-locale-5.nix) into dead code. Is my assumption correct that users have to use something like follows to use a locale with kde5 when switching from 17.03 to 17.09?

As of KDE Applications 17.04, Qt 5 localizations are distributed with each application instead of separately. When you install a KDE/Qt 5 application, all the locales for that application are automatically installed.


@rajagp this should be changed back to port 8080 to match what is on the 5.0 branch of try-cb-python.


I have a  graph file in MTX or txt,and this program can convert it to a binary file.
step1 you need compile it,like this:
`g++ -o main main.cpp`
step2 you can use it
`./main test.mtx  outtest.bin`


Could you have declared default stying for your a tags in this instance and then used the #id's to change the colours. The goal is to abstract out bits where possible.


풀이는 issues #1 에 올림


Да. + его придется расширить (проще говоря дублировать методы из статического класса). Но это не противоречит принципам solid.


no fun allowed


![](https://github.trello.services/images/mini-trello-icon.png) [Build a pipeline](https://trello.com/c/9JLf392y/1-build-a-pipeline)


@varlesh раз такое дело, давай удалим `ubuntu-mono-dark,Mint-X,elementary` из Inherits. Насчет ubuntu-mono-dark можно еще посомневатся, но elementary точно там не нужна, а Mint-X мы покрываем. Эти темы как warnings в KDE всплываются при запуске GUI програм через терминал. Про breeze я не в курсе, но может тоже уже не нужен.


Because it contains only a single member, this can be shorthanded with [`Collections.singleton()`](https://docs.oracle.com/javase/7/docs/api/java/util/Collections.html#singleton(T)). It also will be immutable, so, probably, we won't need to use `Collections.unmodifiableSet()` then.


[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") %>%
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


A


this was me trying to solve the conflict in this file, please check @lsh-0


HTML attribútumok kisbetűsek (id)


Typo and improvement proposal in the last line: "(for all Bootstrapbase based themes)."


This also needs to together with https://github.com/wireapp/wire-ios-sync-engine otherwise we'll crash on switching accounts.


this is tragic. bitcoin is about consensus not covert war tactics. this is open source software and masking clients to trick other open source software is just not OK by any community standards.


The most important classes _for Boost based themes_ might be...


See above


@varlesh `gnome` кстати лучше вернуть в ePapirus, т.к. некоторые пакеты копируют иконки именно туда.


Állítsatok be auto formázást az IDE-ben, ne legyen szétcsúszva a kód


Hi Mike,

Copy.

I'll start new procedure on next iteration.

Good evening!

On Wed, Oct 4, 2017 at 7:12 PM, Masarap Mabuhay <notifications@github.com>
wrote:

> Hi @zent-lim <https://github.com/zent-lim>,
> It might be better for you to *fork* Usbong Store, and then me send *pull
> requests*, so that I can merge pertinent files accordingly.
>
> Thank you.
>
> Peace,
> Mike
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/usbong/UsbongStore/commit/4f3255023f7f52308dce144eb0e0c794863d6297#commitcomment-24751889>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHplTw3k_SH8roLdrbG6Pl4w4lhAXltyks5so2gIgaJpZM4Ptcsc>
> .
>


good idea, i'll do it! thank you, Bogi :)


Понял, пойду пилить. Лучше это делать в той же ветке или в новой?


thank you! :))


done


it`s not, I removed it.


Hi ! Forum posts cannot be edited. I mentioned "semi-open", but in the position, black has a semi open on the e file, and later on the a file.
So maybe
```
if (rookDiff && pe->open_files() ==0)
   score += rookDiff < 0 ? ClosedForRooks : -ClosedForRooks;
```


I believe the lines above were removed mistakenly. \cc @evilaliv3


Да можно в новой) Это же просто рекомендация)


Thanks, cherry-picked this one as well.


@abehaskins That's right. Firebase serve will not deploy the Firestore rules, they need to either be deployed (using `firebase deploy --only firestore`) or copied manually.

I'll take a stab at adding this to the Readme


Just spotted this whilst updating my project to use netns v0.2.1. Looks like a regex gone amiss.


series


you


cuidado com tabs


В любом случае, запилить логгер различными приемлимыми способами - это ценный опыт. Увидеть плюсы и минусы различных реализаций. Пусть даже это просто рекомендация)


`epsg-index`  should be more up to date.


Done!

https://github.com/singularities/circular-work/commit/867923b36b1f650f95711a6b11bbaab3855ee323

Thank you


dessa maneira o contadorTempo vai cada vez ficando maior.... nao da pra fazer algo com menos variaveis:
```
float timeToNextBoss = 60;

update {
  timeToNextBoss -= timeElapsed;
  if(timeToNextBoss <= 0) {
    instancia
    timeToNextBoss = 60;
  }
}
```


Nesse aqui nao daria pra fazer um Find("ControlaJogo").TextoChefeAparece()... nao tem um metodo global que procura a instancia registrada de um game object?


But why?


After further investigation I've discovered this has nothing to do with Celery.

As mentioned in [The Hitchhiker's Guide to Python](http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments):

> Python’s default arguments are evaluated once when the function is defined, not each time the function is called (like it is in say, Ruby). This means that if you use a mutable default argument and mutate it, you will and have mutated that object for all future calls to the function as well.

```
Python 2.7.14 (default, Sep 29 2017, 15:33:24)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.37)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Python shell history and tab completion are enabled.
>>> def foo(l=[]):
...     print l
...     l.append(1)
...
>>> foo()
[]
>>> foo()
[1]
```


Waar blijft de presentatie? ):


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/57dc6ba0-88ac-4227-b8f1-0dcb39afe43f "beebot"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/89e75b99-5155-4e63-aa5d-49b09a72e7f5 "beebot") deployed [beebot](https://github.com/jobgitmww/beebot/releases/tag/deploy-ibm-yp-us-south-Martin-Weidauer-BeeBot-20171004-113833 "deploy-ibm-yp-us-south-Martin-Weidauer-BeeBot-20171004-113833") to [BeeBot](https://console.bluemix.net/apps/e7795b51-a2c6-49c3-a252-3eb54c321935?env_id=ibm:yp:us-south "ibm:yp:us-south:Martin Weidauer:BeeBot"), including this commit


[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,
              ^~~
```


![](https://github.trello.services/images/mini-trello-icon.png) [Test 01](https://trello.com/c/UIqAzkPk/34-test-01)


Why did not you keep the authorship?


aqui ficou um if/else que lembra switch... a medida que tem mais usuario, teria mais if/else e fica dificil de manter... aqui seria o bom de uso de interface... uma interface InimigoQueApanha e os dois implementam ela... ai o codigo aqui seria somente objetoDeColisao.GetComponent<InimigoQueApanha>().LevaTiro();
e ai cada inimigo tem sua implementacao. rola refatorar numa ultima aula?
e o if fica um unico, verifica se tem o componente inimigoqueapanha


CanoDaArma... mas SomTiro... eu prefiro padrão sempre com "De" e "Da"... mas vi que as vezes tem e as vezes não, tenta manter um padrão só.


mesma pergunta que fiz de antes sobre singletons e se tem um finder


@basthomas what’s the issue here? Reading the bug, It looks like this only applies to optional right?

TIL on the bug tho, should def be a compiler warning!


Пипец даже демо не с первой попытка


It is not a fake app.
Corrected comment to provide better feedback to developer.


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


o problema dessa abordagem de singleton (e singleton em geral) é que se algum codigo de alguma prte da aplicacao chamar ControlaAudio.instancia antes do metodo start aqui ser chamado, vai dar um null e erro. e nao tem como saber e ficar mandando todo mundo prevenir... singleton com variavel global publica é do mal :(

http://blog.caelum.com.br/singletons-e-static-perigo-a-vista/


done
split to two methods


done


https://github.com/Fluorohydride/ygopro-pre-script/commit/2764b3708c21e2f8ae73fda01bcff088e336cf1c updated.


Cause the Github page doesn't really need it


@abehaskins FYI


extrai esse aqui num metodo para ficar um nivel de if em cada metodo?
tambem jogaria uma variavel local para ficar mais facil de entender o que quer dizer essa comparacao. repara que comparacoes "complexas" ficam mais simples com variaveis locais:
```
bool toPertinho = agente.remainingDistance <= agente.stoppingDistance;

if(toPertinho) {
}
```


There goes all the fun in SPFx ;)


pq se vc nao tem a variavel descrevendo, toda vez vc tem que interpretar o que quer dizer as duas coisas e a comparacao


Corrected description.


Add package name


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/f24266f8-cf20-4141-a768-483b0bac7b1c "simple-toolchain-20170927012305383"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/f6a0df0f-302a-4bf6-ac0a-45f9e4164c8d "simple-toolchain-20170927012305383") deployed [smartbotanica](https://github.com/MUHAMMADRAZIN/smartbotanica/releases/tag/deploy-ibm-yp-us-south-rndspark.smartbotani-smartbotanica-20171004-114443 "deploy-ibm-yp-us-south-rndspark.smartbotani-smartbotanica-20171004-114443") to [smartbotanica](https://console.bluemix.net/apps/9cdefcbf-96e8-4d4a-92fd-2fa155210406?env_id=ibm:yp:us-south "ibm:yp:us-south:rndspark.smartbotani:smartbotanica"), including this commit


Review this


Working on it :)


o que é isso? não conheco :)


Changed methods name, done


Great update! Maybe you should consider accepting some wonderfull pull requests with valuable changes?


static это не по оопшному


personagem = jogador ou boss? se estamos falando do boss, deixar aqui boss explicito (mesmo que so no nome da variavel membro) vai ajudar a deixar explicito do que o codigo esta falando... ate ajuda mais pq repara que agora o nome da variavel nao diz nada alem do nome do tipo. é como criar um int int. ou uma string string. quando voce cria uma AnimacaoPersonagem animacaoBoss fica claro que essa instancia de animacao é especifica do boss. assim como num int idade fica claro que esse int é uma idade


idem


done


Nevím, osobně si myslím, že by tam měla patřit. Před účastníky by to mělo být skryté bez ohledu na to, zda to přijde v odpovědi. Tam si toho snad případně nevšimnou, na rozdíl od toho když to na ně vyskočí.


deixaria pro final... somente se sobrar tempo de gravacao... toa achando que tem muita coisa então colocaria o "degrade" somente no fim do curso


Okay, changed.


Thank you! Peace.


há, acho que tem os finders então :)


现在支持在server 加上延迟，示例：
./relay-server -l 500 12345


Run this file first


Changed.


Okay, MAP's solution is nearly identical to that is suggested here https://stackoverflow.com/questions/878600/how-to-create-a-cron-job-using-bash-automatically-without-the-interactive-editor.

It would have been better to use the `/etc/cron.d/weekly`, but they only run as sudo (https://help.ubuntu.com/community/CronHowto), which is not desirable for `brew`. So, I'll use MAP's solution. I will adapt the same thing for `generate_certificate`.


encapsulamento e tell don't ask aqui simplifica a responsabilidade:

sliderVidaChefe.inicia(status.vidaInicial)


new comment


alguns metodos estao com nome de verbos, outros n˜ ao, padronizaria tudo pra verbo... aqui por exemplo: "CausaDanoNoJogador" ou "BateNoJogador"


pois se ficar como substantivo confunde com variaveis


Has this malicious code been included in any release ?


por mais q seja super simples, prefiro em duas linhas qdoestamos ensinando, fica mais claro pro aluno:

```
dano = Random.Range(40,60);
```


@Verevka I extracted the previous file from my MIUI ROM. Sorry, no sign.


если уж делаешь через Message, тогда нужно чтобы состояния и формирование менялись на уровне мессаджей, а контроллер дергал только абстракцию, зная ничего о конкретном мессадже


aqu ie nos outros metodos a questao do verbo


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/0b77a219-c1e9-464f-bc76-befc9f46b387 "simple-toolchain-Finsa"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/f6dac069-c253-402e-8bbb-923c4716b09d "simple-toolchain-Finsa") deployed [simple-toolchain-Finsa](https://github.com/Javitron/simple-toolchain-Finsa/releases/tag/deploy-ibm-yp-us-south-javier.lisbona-es.ibm.com-dev-20171004-115037 "deploy-ibm-yp-us-south-javier.lisbona-es.ibm.com-dev-20171004-115037") to [dev](https://console.bluemix.net/apps/0052035e-f3c5-4707-8e09-800e220caaac?env_id=ibm:yp:us-south "ibm:yp:us-south:javier.lisbona@es.ibm.com:dev"), including this commit


test comment2   #2
 @chandrasen


Wrong path here: Using `'./state'` instead of `'./routes/state'`.


repara que essa linha ja podia estar dentro do CairPeloChao.... é o problema da varivel enabled ter ficado publica (decisao de API do Unity, claro...). ai nos ja temos um comportamento que esta dizendo que acabou o ciclo de vida do movimentoPersonagem, mas mesmo assim como temos acesso por fora, acabamos acessando por fora o enabled... mas manda o enabled pra dentro do cairpelochao.
pode fazer essa refaotracao discutindo com o aluno pra ele ir pegando o jeito


aqui ta dentro do proprio modelo... :)


I replace `autocomplete="new-password"` with `readonly onfocus="this.readOnly = false" onblur="this.readOnly = true" ` and auto-complete stop fill inputs.


nedir?


se isso for padrão para todos os kits medicos nao eh mais facil deixar no start deles?


Regarding the one about advancing, I'll add it, as @Arcanister pointed out that `{ADVANCE_UNIT}` is sometimes used for downgrading.

Regarding units which are killed by attacking, that didn't happen to me. The halo disappeared like it should.


Зачем пустой else?

п.с. вообще в энтрейпрайзе по правилам хорошего кода принято стараться не писать через ifele если это возможно.
такой код
```
public vooid blablabla() {
    if (a) {
        // some code
        return;
    }
    // other code
}
```

гораздо более читаемый чем if(a) {...} else {..}


eu prefiro o codigo como voce fez. uma variacao é status.Vida = min(vidaInicial, status.vida + vida); eu acho horrivel essa opcao que eu dei. prefiro a tua  pq fica clara


Please fix the state router path on api.js and then we're good to go.


aqui e em outros metodos usar verbos


Nedir?


Why didn't you just use the filters defined above? 😕

https://github.com/Microsoft/vscode/commit/3002756d7d399edbf1390e17b6bda8d65d6d88d1


evitar abreviar variaveis, mesmo quantidade


ne


doideira huehue, nao manjo, me explica? :)


好!


very good point.


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


@yanntm, I can not find evidence of a semantic change, the code seems to do what is intended (execute the next case statement as a whole), this is the Control Flow Graph of `permute.c`: https://pastebin.com/D8cr67CJ.
I generated the CFG with `make CFLAGS="-fdump-tree-all-graph"` after touching `permute.c`.

In the CFG of function `permute_one()` there is only one node in the CFG representing the `Perm_None` case block. Both the `Perm_None` case and the `else` have a `goto` to said block.

Can you confirm this?