TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14812](https://teamcity.mesosphere.io/viewLog.html?buildId=128096&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:25


@Optimusprimeums Yes, it might very well be a user error, but it sounded a bit strange that it would ask for Java 6.


dann am bestem in gazen reverten und den richtigen nochmal hinterherschieben


remove duplicate imports


FYI similar tests which actually passed STC/LTC:
http://tests.stockfishchess.org/tests/view/564a62ec0ebc5902cdc08218
http://tests.stockfishchess.org/tests/view/56406efd0ebc59227df1717d

In above tests, maxply was also considered. Good luck with your tests!


i removed all the nicetohave issues from the 1.3 milestone. would say we are ready, yes!


@tomenden - this without any wix js.


Outsch


nice, didn't know this!


unused?


Does `$inject` have to be called, even with an empty array? Or is this for explicitness?


It should be targeted to 3.5 not 3.0
Beside the fact is doesn't work on empty database + on database with sample data (I've inserted Raven-Entity-Label to one of empoloyee):

1. We have to maintain those labels somehow in code.
2. We have to keep mapping in sync with Raven-Entity-Name.

Better solution for me is to main mapping between Raven-Entity-Name to any name user wants and store this as document in database or in local storage. It won't cause performance impact on server side.


This looks good, assuming isUserAdmin works as expected. I did not do any testing yet.


Just for the reference. The problem with invalid values was fixed here: https://github.com/pgjdbc/pgjdbc/commit/e6f1beb19c1581b003ac85ab29454ac58b157d96


Thanks Emma!


остальное норм


Fogot about those two mods... Will be added in according script files


No newline at the end of the file.


No it's just for content negotiation.


Based on brief googling every Windows version since Win98 has notepad.exe in under system32. Based on that an ACK from me.


Process managers can also generate domain events.


Ah so the underlying bug is that we try to transmit on SPI when SPI is not
enabled? That seems well worth fixing, I'll look into it.


cacheInverse<-function(x,...)
                     {
                        Inv <- x$getreverse()
                        if(!is.null(Inv))
                        {
                           message("got the reverse matrix")
                           return(Inv)
                        }
                        else
                        {
                           Inv <- solve(x$get())
                           x$setreverse(Inv)
                           return(Inv)
                        }
                     }


You are never using `speed_x`


Change code from line 146 to 170 is just for rectifying code (not for currency conversion)


I recall these `function` syntaxes are changed later, but I'll just make a note of it here in case they're not.


It' just example of code usage. I've just replaced the ForceSSL variable with example one. Maybe it was not the best choice...


You shouldn't end functions with `pass`, it is there just to fix empty functions/blocks :smile:


use a c++ 11 range-based for loop?


const


I think d12fk or ValdikSS should have a quick look at this. I can't see any obvious issues here, but I'm not a C coder.


There's no `$inject` statement here?


Here too ctrl.panel.legend.rightSide?


Sorry for that extra coma here :octocat:


Maybe you should check it in relation to the window?


Hello fire855,
Is there something wrong with this repo? Why the changes are so big?
I have done a little patch manually, which is able to merge with the newest source and I can compile it without errors. It might be helpful for you :-D.
My code is at https://github.com/zhangjingye03/android_frameworks_base


@tresf would this solution not create an infinite loop as long as the qz.websocket.connect block keeps getting rejected and the origin is the launchQZ method?
Maybe some sort of max attempt setting in the _qz.websocket.connectConfig would be a way to go?


”検索結果の表示内容を変更”
のタスクをするにあたって、
QIDB/app/views/definitions/select.html.erb
のコードを書き換えたのですが、
コミットしてプッシュしたところ、このような状態になりました。

僕の想像では、showwin/QIDB内のapp/.../select.html.erbが変更したこのコードに差しかわると思っていたのですが、そうなっていないため、この作業が正しいのかどうかがいまいちわかっていません。
もし間違っているようなら、作業のアドバイスや改善点を教えていただきたいです。


I really would like to know why this happens.
Obviously master builds for us and for travis-ci (see green badge).
Which is your java version? Do you build with Android Studio or command line?


Note: you should `export` such variables, instead of hardcoding them. Like:
````gdscript
export var speed_y = -500.0
````

(The `.0` is to force a floating-point number)


### It was a success


As discussed offline, the double-buffering is still needed for contents which take time to render (svg, pdf) otherwise the texture is black until the new image is returned


Note that currently the player would sink a bit in platforms. Maybe you should give them more mass?


I think I recall this disappearing too, but as before I'll leave a note just in case.


This hack suck (I did that, my fault), lets remove it. A pull request is coming to standardize this. It wont allow everything this thing made possible, but trying to force some of those features into `widget.common` was a terrible idea.


Stopped test because I discovered only now that with threads = 23 (23 were suggested by Peter) we have the last 2 helpers again work with skipsize 1 (good quality) but current patch implementation (prototype) does exclude them as best-thread choice candidate.

@zamar Peter suggested me to use SPRT and his argumentation coincide with my thoughts:
also on a SPRT it's easy to calculate ELO continuously based on number of wins, losses, draws.
Given the case that after 10000 games the ELO-performance is promising, we simply could let the SPRT test go on (otherwise stop it) and thus save the framework from wasting this first 10000 games (=the case when first making a fixed game test and only then a SPRT). Please explain me what's unsound with this approach, thanks.

Günther


@md5 Syntax changed from 1.9 to 1.10.

Fix here  https://github.com/baptistedonaux/docker-gen/commit/b3b09d7553473113c4d935aed720dfb42f560359


Well, as far as I understand, render() can never return an array of elements. It always only return one element. However map() returns an array: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map composed of the elements returned by the call back. So, when I remove the key in this particular case, I get the following warning in console from React: "Warning: Each child in an array or iterator should have a unique "key" prop. Check the render method of `HelloWorld`. See https://fb.me/react-warning-keys for more information."


Readme updated


This should fix the broken job. However, it may never compile on Travis because I don't know whether the examples are ever built with c++11 on.


[free Magento extensions](https://www.envisionecommerce.com/store/free-items/free-magentor-extensions.html)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14814](https://teamcity.mesosphere.io/viewLog.html?buildId=128101&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:29


Ah, like that. Yes, you're right, I agree with you on that as a general rule.


[tests/testthat/test-scrm.R:37:28:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-scrm.R#L37) *style:* **Put spaces around all infix operators.**
```r
  output <- scan(file, what="character", sep="\n", quiet = TRUE)
                          ~^~
```
[tests/testthat/test-scrm.R:37:45:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-scrm.R#L37) *style:* **Put spaces around all infix operators.**
```r
  output <- scan(file, what="character", sep="\n", quiet = TRUE)
                                           ~^~
```
[tests/testthat/test-scrm.R:49:53:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-scrm.R#L49) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-scrm.R:58:53:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-scrm.R#L58) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-scrm.R:64:53:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-scrm.R#L64) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-summary-statistics.R:27:53:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-summary-statistics.R#L27) *style:* **Commas should always have a space after.**
```r
  expect_true(all(sum_stats$seg_sites[[1]] %in% c(0,1)))
                                                    ^
```
[tests/testthat/test-summary-statistics.R:28:53:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-summary-statistics.R#L28) *style:* **Commas should always have a space after.**
```r
  expect_true(all(sum_stats$seg_sites[[2]] %in% c(0,1)))
                                                    ^
```
[tests/testthat/test-summary-statistics.R:88:38:](https://github.com/scrm/scrm-r/blob/2c7e091665e5f16a2aa12dab40e2e5374f6ba92a/tests/testthat/test-summary-statistics.R#L88) *style:* **Commas should always have a space after.**
```r
    expect_equal(sum(sum_stats$sfs[i,]), ncol(sum_stats$seg_sites[[i]]))
                                     ^
```


Hey guys, just jumping in as someone else who would really like to see OpenTK continue. I meant to contribute in the past, but did not get around to it - and I dont trust myself enough to commit to something like that right now either.

However, since some of you are talking about replacing the official opentk website/forum, that is something I could help with. I'm not a web developer, but I got tons of unused webspace and wouldn't mind paying for a url either. I could set up whatever forum software we think is best, and keep it free of spam and do whatever other moderation is needed.

Is this something that would be helpful at all? If you agree that the current site/forum is not going to work, switching to a new one (maybe temporarily?) will still require all currently interested parties to sign up and give it some life. It also means we have to get people who have questions to actually find the forum.

I'm very willing to set something up (and maintain it), but only if there is somewhat of a consensus that it is a good idea. ("Splitting the community" sounds like a terrible idea.)


Are you proposing to create two simulator subclasses in order to handle "special cases" or to use explicitly two different interfaces within the manager?


`pendingBloodTypingMatchTests` isn't used in the frontend anymore and I think that it's best if it's just removed because I'm also not sure if it's correct that when the BloodTypingMatchStatus is MATCH there would be pending blood typing tests.

What do you think of removing the boolean from the overview endpoint?


No problem, it's quite a common problem that maven caches get inconsistent.


No. I just want to remove unnecessary patch that different to upstream approach.


There's a new sort method? Was this necessary for the refactor?


This has confused me so much. I prefer consistency over grammar.


I personally like the `qemu` option.
It requires `COPY . /usr/bin`, `ENV QEMU_EXECVE=1`, `RUN ["cross-build-start"]`, `RUN ["cross-build-end"]` statements in the `Dockerfile` when compiling for ARM on amd64.

But in this case, using Drone could be the better option


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14813](https://teamcity.mesosphere.io/viewLog.html?buildId=128100&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 222 Build time: 00:08:19


Fixes #41


Why did you remove the "-p" ?


I'm not sure, I'm just doing it in this instance in order to keep everything uniform


This shouldn't be taking a `stock` parameter.


@volumio OMG: Completely wrong indentation!!!! I'll fix this


@timler Okay, I will remove it.


I think it is better to name it `returnsMessageOrMessageList`.


Note for later to check the `function` syntax.


Added


Hi

I rebased your 3 deprecated method fixes into 1 commit for testing here:
https://github.com/bradymiller/openemr/commit/80fb6eaa8b5b9190835b2a4ebe0ae73a73cd474d

See the commit for comments(search for bradymiller to see the comments; disregard the unset() issue which I am looking into).

-brady


Sorry, can be restored.


Please add a comment why this number is used.


There's a new event? Is this part of an ES6 refactor?


Nat,
we don't seem to have a CLA in place for you as per the contribution guidelines... :-(
Also should this not be theme-able as per the title and favicon ? so that others can change it externally if necessary ?  (IE needs to be handled in red/api/theme.js as well - rather than being hard coded in index.mst


Fixes #40


Compile with mpich by typing "make" in the directory. Requires cBLAS and Lapack libraries.


Ok


Actually, we know nothing beside the collection and yes, how can we bind ourselves to any specific names.
In my opinion we should create them all created (if non-existent). If you agree @yvesgoeleven @SeanFeldman  I'm leaving the code as it is now - I've already done it this way.


@timler Please see https://github.com/jembi/bsis/commit/5b6036819d31bccad034ffa1e7734dcf24c73e0d and https://github.com/jembi/bsis-frontend/commit/75e7303350738712d396845df96f178e0259ed09.


This actually assumes the Sage doctesting framework, it doesn't work with Python's `doctest` module.


raising an exception is a bit harsh, don't you think?
Printing a log message to stderr should be enough - this allows us to enable it by default as well with graceful degradation.


Please add a newline.


Hi,

Recommend going back now and trying to get the original gacl smarty to work. You can revert your gacl commit in your branch by doing the following:
git revert f294a6c50815383f76d42d2f3d1927c1de0bd332
(this basically creates a new commit that reverts the commit)

Note the install worked for me and i saw no errors when I did the following(I am not using PHP7 though):
https://github.com/bradymiller/openemr/commit/5196924a0955084da7b43df55130469e5d89d046

-brady


Great. Thanks!


Do we really use the `HTTP/` prefix? I think in netlib we only use `"1.1"` and `"2.0"`.


[tests/testthat/test-scrm.R:37:28:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-scrm.R#L37) *style:* **Put spaces around all infix operators.**
```r
  output <- scan(file, what="character", sep="\n", quiet = TRUE)
                          ~^~
```
[tests/testthat/test-scrm.R:37:45:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-scrm.R#L37) *style:* **Put spaces around all infix operators.**
```r
  output <- scan(file, what="character", sep="\n", quiet = TRUE)
                                           ~^~
```
[tests/testthat/test-scrm.R:49:53:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-scrm.R#L49) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-scrm.R:58:53:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-scrm.R#L58) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-scrm.R:64:53:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-scrm.R#L64) *style:* **Put spaces around all infix operators.**
```r
  expect_true(file.exists(paste(tempdir(), file, sep="/")))
                                                   ~^~
```
[tests/testthat/test-summary-statistics.R:27:53:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-summary-statistics.R#L27) *style:* **Commas should always have a space after.**
```r
  expect_true(all(sum_stats$seg_sites[[1]] %in% c(0,1)))
                                                    ^
```
[tests/testthat/test-summary-statistics.R:28:53:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-summary-statistics.R#L28) *style:* **Commas should always have a space after.**
```r
  expect_true(all(sum_stats$seg_sites[[2]] %in% c(0,1)))
                                                    ^
```
[tests/testthat/test-summary-statistics.R:88:38:](https://github.com/scrm/scrm-r/blob/722ac2aecfb2743567e6574a30a8faeddfcdd8f2/tests/testthat/test-summary-statistics.R#L88) *style:* **Commas should always have a space after.**
```r
    expect_equal(sum(sum_stats$sfs[i,]), ncol(sum_stats$seg_sites[[i]]))
                                     ^
```


[R/cache.R:16:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/cache.R#L16) *warning:* **no visible binding for global variable ‘cache_env’**
```r
  cache_env[[model$id]][[name]]
  ^~~~~~~~~
```
[R/feature_growth.R:44:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_growth.R#L44) *warning:* **no visible binding for global variable ‘growth_class’**
```r
  growth_class$new(rate, population, time)
  ^~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:28:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L28) *warning:* **no visible binding for global variable ‘ign_singletons_class’**
```r
  ign_singletons_class$new()
  ^~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:40:29:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L40) *warning:* **no visible global function definition for ‘get_snps’**
```r
    is_singleton <- colSums(get_snps(segsites)) == 1
                            ^~~~~~~~
```
[R/feature_ignore_singletons.R:41:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L41) *style:* **Commas should never have a space before.**
```r
    segsites[ , !is_singleton]
             ~^
```
[R/feature_ignore_singletons.R:52:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L52) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:56:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L56) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:60:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_ignore_singletons.R#L60) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_migration.R:72:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_migration.R#L72) *warning:* **no visible binding for global variable ‘migration_class’**
```r
    return(migration_class$new(rate, time = time, symmetric = TRUE))
           ^~~~~~~~~~~~~~~
```
[R/feature_migration.R:74:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_migration.R#L74) *warning:* **no visible binding for global variable ‘migration_class’**
```r
    return(migration_class$new(rate, pop_from, pop_to, time))
           ^~~~~~~~~~~~~~~
```
[R/feature_mutation.R:178:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_mutation.R#L178) *warning:* **no visible global function definition for ‘conv_to_ms_arg.mutation’**
```r
  conv_to_ms_arg.mutation(feature, model)
  ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_outgroup.R:31:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_outgroup.R#L31) *warning:* **no visible binding for global variable ‘outgroup_class’**
```r
  outgroup_class$new(population)
  ^~~~~~~~~~~~~~
```
[R/feature_outgroup.R:40:34:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_outgroup.R#L40) *warning:* **no visible binding for global variable ‘is_feat_outgroup’**
```r
  mask <- vapply(model$features, is_feat_outgroup, logical(1))
                                 ^~~~~~~~~~~~~~~~
```
[R/feature_outgroup.R:49:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_outgroup.R#L49) *warning:* **no visible global function definition for ‘get_outgroup’**
```r
  outgroup <- get_outgroup(model)
              ^~~~~~~~~~~~
```
[R/feature_outgroup.R:51:20:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_outgroup.R#L51) *warning:* **no visible global function definition for ‘get_sample_size’**
```r
  outgroup_size <- get_sample_size(model, for_sim)[outgroup]
                   ^~~~~~~~~~~~~~~
```
[R/feature_pop_merge.R:47:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_pop_merge.R#L47) *warning:* **no visible binding for global variable ‘pop_merge_class’**
```r
  pop_merge_class$new(time, pop_source, pop_target)
  ^~~~~~~~~~~~~~~
```
[R/feature_recombination.R:35:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_recombination.R#L35) *warning:* **no visible binding for global variable ‘recombination_class’**
```r
  recombination_class$new(rate)
  ^~~~~~~~~~~~~~~~~~~
```
[R/feature_recombination.R:54:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_recombination.R#L54) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.recombination <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sample.R:44:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L44) *warning:* **no visible binding for global variable ‘sample_class’**
```r
  sample_class$new(individuals, ploidy, time)
  ^~~~~~~~~~~~
```
[R/feature_sample.R:59:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L59) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.logical(for_sim))
  ^~~~~~~~~~~
```
[R/feature_sample.R:60:18:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L60) *warning:* **no visible global function definition for ‘read_cache’**
```r
  sample_size <- read_cache(model, paste0("sample_size_", for_sim))
                 ^~~~~~~~~~
```
[R/feature_sample.R:63:40:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L63) *warning:* **no visible binding for global variable ‘is_feat_sample’**
```r
    feat_mask <- sapply(model$feature, is_feat_sample)
                                       ^~~~~~~~~~~~~~
```
[R/feature_sample.R:68:36:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L68) *warning:* **no visible global function definition for ‘get_ploidy’**
```r
      sample_size <- sample_size * get_ploidy(model)
                                   ^~~~~~~~~~
```
[R/feature_sample.R:70:36:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L70) *warning:* **no visible global function definition for ‘get_samples_per_ind’**
```r
      sample_size <- sample_size * get_samples_per_ind(model)
                                   ^~~~~~~~~~~~~~~~~~~
```
[R/feature_sample.R:72:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L72) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, paste0("sample_size_", for_sim), sample_size)
    ^~~~~
```
[R/feature_sample.R:80:34:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L80) *warning:* **no visible binding for global variable ‘is_feat_sample’**
```r
  mask <- vapply(model$features, is_feat_sample, logical(1))
                                 ^~~~~~~~~~~~~~
```
[R/feature_sample.R:91:18:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sample.R#L91) *warning:* **no visible global function definition for ‘get_sample_size’**
```r
  sample_size <- get_sample_size(model, TRUE)
                 ^~~~~~~~~~~~~~~
```
[R/feature_selection.R:176:21:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_selection.R#L176) *warning:* **no visible global function definition for ‘get_populations’**
```r
    n_pop <- length(get_populations(model))
                    ^~~~~~~~~~~~~~~
```
[R/feature_selection.R:191:36:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_selection.R#L191) *style:* **Commas should never have a space before.**
```r
                        n_pop, " " , paste(start_freq, collapse = " "), " ",
                                  ~^
```
[R/feature_size_change.R:47:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_size_change.R#L47) *warning:* **no visible binding for global variable ‘size_change_class’**
```r
  size_change_class$new(new_size, population, time)
  ^~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:18:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sumstats.R#L18) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:22:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sumstats.R#L22) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:26:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_sumstats.R#L26) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_unphased.R:52:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_unphased.R#L52) *warning:* **no visible binding for global variable ‘unphased_class’**
```r
  unphased_class$new(samples_per_ind)
  ^~~~~~~~~~~~~~
```
[R/feature_unphased.R:60:34:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_unphased.R#L60) *warning:* **no visible binding for global variable ‘is_feat_unphased’**
```r
  mask <- vapply(model$features, is_feat_unphased, logical(1))
                                 ^~~~~~~~~~~~~~~~
```
[R/feature_unphased.R:68:11:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_unphased.R#L68) *warning:* **no visible global function definition for ‘get_feature_unphased’**
```r
  feat <- get_feature_unphased(model)
          ^~~~~~~~~~~~~~~~~~~~
```
[R/feature_unphased.R:69:29:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/feature_unphased.R#L69) *warning:* **no visible global function definition for ‘get_ploidy’**
```r
  if (is.null(feat)) return(get_ploidy(model))
                            ^~~~~~~~~~
```
[R/import_popgenome.R:19:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/import_popgenome.R#L19) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.GENOME <- function(data, only_synonymous = FALSE, ...) {
^~~~~~~~~~~~~~~~~~
```
[R/import_popgenome.R:20:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/import_popgenome.R#L20) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("PopGenome")
  ^~~~~~~~~~~~~~~
```
[R/import_popgenome.R:43:43:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/import_popgenome.R#L43) *style:* **Commas should never have a space before.**
```r
      seg_sites <- bam[c(pops, outgroup), , drop = FALSE]
                                         ~^
```
[R/import_popgenome.R:49:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/import_popgenome.R#L49) *warning:* **no visible global function definition for ‘create_segsites’**
```r
    create_segsites(seg_sites, pos)
    ^~~~~~~~~~~~~~~
```
[R/import_popgenome.R:56:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/import_popgenome.R#L56) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("PopGenome")
  ^~~~~~~~~~~~~~~
```
[R/interface_abc.R:80:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/interface_abc.R#L80) *warning:* **no visible global function definition for ‘get_summary_statistics’**
```r
  sumstats <- get_summary_statistics(model)
              ^~~~~~~~~~~~~~~~~~~~~~
```
[R/interface_abc.R:84:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/interface_abc.R#L84) *warning:* **no visible global function definition for ‘assert_that’**
```r
      assert_that(is.numeric(value))
      ^~~~~~~~~~~
```
[R/interface_abc.R:86:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/interface_abc.R#L86) *style:* **lines should not be more than 80 characters.**
```r
        warning("Value of the ", stat$get_name(), " statistic is not a vector. ",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/locus.R:84:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/locus.R#L84) *warning:* **no visible binding for global variable ‘locus_class’**
```r
  locus_class$new(length, 1)
  ^~~~~~~~~~~
```
[R/locus.R:92:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/locus.R#L92) *warning:* **no visible binding for global variable ‘locus_class’**
```r
  locus_class$new(round(length), number)
  ^~~~~~~~~~~
```
[R/locus.R:151:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/locus.R#L151) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  llm <- get_locus_length_matrix(model)
         ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/locus.R:152:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/locus.R#L152) *warning:* **no visible global function definition for ‘get_locus_group’**
```r
  group <- get_locus_group(model, locus)
           ^~~~~~~~~~~~~~~
```
[R/model_build.R:10:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L10) *warning:* **no visible global function definition for ‘is.model’**
```r
  if (is.model(e1)) return(add_to_model(e2, e1, e2name))
      ^~~~~~~~
```
[R/model_build.R:10:28:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L10) *warning:* **no visible global function definition for ‘add_to_model’**
```r
  if (is.model(e1)) return(add_to_model(e2, e1, e2name))
                           ^~~~~~~~~~~~
```
[R/model_build.R:48:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L48) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/model_build.R:54:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L54) *warning:* **no visible global function definition for ‘add_variation’**
```r
  model <- add_variation(model)
           ^~~~~~~~~~~~~
```
[R/model_build.R:56:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L56) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/model_build.R:79:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L79) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/model_build.R:86:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_build.R#L86) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/model_check.R:18:16:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_check.R#L18) *warning:* **no visible global function definition for ‘get_simulator’**
```r
    simprog <- get_simulator(simprog_name)
               ^~~~~~~~~~~~~
```
[R/model_getters.R:15:13:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L15) *warning:* **no visible global function definition for ‘is.model’**
```r
  stopifnot(is.model(model))
            ^~~~~~~~
```
[R/model_getters.R:17:16:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L17) *warning:* **no visible global function definition for ‘read_cache’**
```r
  par_table <- read_cache(model, "par_table")
               ^~~~~~~~~~
```
[R/model_getters.R:35:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L35) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "par_table", par_table)
    ^~~~~
```
[R/model_getters.R:45:13:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L45) *warning:* **no visible global function definition for ‘is.model’**
```r
  stopifnot(is.model(model))
            ^~~~~~~~
```
[R/model_getters.R:59:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L59) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  llm <- get_locus_length_matrix(model)
         ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/model_getters.R:71:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L71) *warning:* **no visible global function definition for ‘get_locus_group’**
```r
    group <- get_locus_group(model, locus)
             ^~~~~~~~~~~~~~~
```
[R/model_getters.R:75:21:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L75) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                   ~^
```
[R/model_getters.R:75:60:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L75) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                                                          ~^
```
[R/model_getters.R:81:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L81) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  llm <- get_locus_length_matrix(model)
         ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/model_getters.R:82:25:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L82) *style:* **Commas should never have a space before.**
```r
  min(which(cumsum(llm[ , "number"]) >= locus))
                       ~^
```
[R/model_getters.R:87:8:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L87) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  nrow(get_locus_length_matrix(model))
       ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/model_getters.R:94:49:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L94) *warning:* **no visible binding for global variable ‘is_feat_sample’**
```r
  feat <- model$features[vapply(model$features, is_feat_sample, logical(1))]
                                                ^~~~~~~~~~~~~~
```
[R/model_getters.R:96:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L96) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(pop_number > 0)
  ^~~~~~~~~~~
```
[R/model_getters.R:105:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L105) *warning:* **no visible global function definition for ‘read_cache’**
```r
  llm <- read_cache(model, "llm")
         ^~~~~~~~~~
```
[R/model_getters.R:107:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L107) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(model$loci) >= 0)
    ^~~~~~~~~~~
```
[R/model_getters.R:115:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L115) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "llm", llm)
    ^~~~~
```
[R/model_getters.R:125:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L125) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  numbers <- get_locus_length_matrix(model)[ , "number"]
             ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/model_getters.R:125:46:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L125) *style:* **Commas should never have a space before.**
```r
  numbers <- get_locus_length_matrix(model)[ , "number"]
                                            ~^
```
[R/model_getters.R:127:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L127) *warning:* **no visible global function definition for ‘has_variation’**
```r
  if (has_variation(model) && !ignore_variation) return(1)
      ^~~~~~~~~~~~~
```
[R/model_getters.R:144:18:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L144) *warning:* **no visible global function definition for ‘get_sample_size’**
```r
  sample_size <- get_sample_size(model)
                 ^~~~~~~~~~~~~~~
```
[R/model_getters.R:145:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L145) *warning:* **no visible global function definition for ‘get_outgroup’**
```r
  outgroup <- get_outgroup(model)
              ^~~~~~~~~~~~
```
[R/model_getters.R:152:47:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L152) *warning:* **no visible global function definition for ‘get_ploidy’**
```r
  if (!haploids) sample_size <- sample_size / get_ploidy(model)
                                              ^~~~~~~~~~
```
[R/model_getters.R:155:17:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L155) *warning:* **no visible global function definition for ‘get_populations’**
```r
  if (!pop %in% get_populations(model)) stop("Invalid population")
                ^~~~~~~~~~~~~~~
```
[R/model_getters.R:163:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L163) *warning:* **no visible global function definition for ‘get_parameter’**
```r
  param <- get_parameter(model)
           ^~~~~~~~~~~~~
```
[R/model_getters.R:166:35:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L166) *warning:* **no visible binding for global variable ‘is.prior_par’**
```r
    param <- param[!vapply(param, is.prior_par, numeric(1))]
                                  ^~~~~~~~~~~~
```
[R/model_getters.R:174:28:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_getters.R#L174) *warning:* **no visible global function definition for ‘select_simprog’**
```r
  suppressWarnings(prog <- select_simprog(model))
                           ^~~~~~~~~~~~~~
```
[R/model_print.R:17:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L17) *warning:* **no visible global function definition for ‘print_model_part’**
```r
  print_model_part(x, "features", "Features")
  ^~~~~~~~~~~~~~~~
```
[R/model_print.R:20:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L20) *warning:* **no visible global function definition for ‘print_model_part’**
```r
  print_model_part(x, "parameter", "Parameter")
  ^~~~~~~~~~~~~~~~
```
[R/model_print.R:23:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L23) *warning:* **no visible global function definition for ‘print_model_part’**
```r
  print_model_part(x, "loci", "Loci")
  ^~~~~~~~~~~~~~~~
```
[R/model_print.R:26:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L26) *warning:* **no visible global function definition for ‘print_model_part’**
```r
  print_model_part(x, "sum_stats", "Summary Statistics")
  ^~~~~~~~~~~~~~~~
```
[R/model_print.R:29:31:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L29) *warning:* **no visible global function definition for ‘select_simprog’**
```r
  suppressWarnings(simprog <- select_simprog(x))
                              ^~~~~~~~~~~~~~
```
[R/model_print.R:35:23:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_print.R#L35) *warning:* **no visible global function definition for ‘get_cmd’**
```r
    cmd <- try(cmd <- get_cmd(x), silent = TRUE)
                      ^~~~~~~
```
[R/model.R:72:20:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L72) *warning:* **no visible global function definition for ‘feat_sample’**
```r
  model <- model + feat_sample(sample_size, ploidy)
                   ^~~~~~~~~~~
```
[R/model.R:76:22:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L76) *warning:* **no visible global function definition for ‘locus_averaged’**
```r
    model <- model + locus_averaged(loci_number, loci_length)
                     ^~~~~~~~~~~~~~
```
[R/model.R:92:11:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L92) *warning:* **no visible global function definition for ‘read_cache’**
```r
  name <- read_cache(model, "simprog")
          ^~~~~~~~~~
```
[R/model.R:98:18:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L98) *warning:* **no visible global function definition for ‘get_simulator’**
```r
      simprog <- get_simulator(simprog_name)
                 ^~~~~~~~~~~~~
```
[R/model.R:109:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L109) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "simprog", name)
    ^~~~~
```
[R/model.R:126:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L126) *warning:* **no visible global function definition for ‘get_locus_length_matrix’**
```r
  sum(get_locus_length_matrix(model)[ , c(1:2, 4:5)]) > 0
      ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/model.R:126:39:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L126) *style:* **Commas should never have a space before.**
```r
  sum(get_locus_length_matrix(model)[ , c(1:2, 4:5)]) > 0
                                     ~^
```
[R/model.R:132:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L132) *warning:* **no visible global function definition for ‘get_positions’**
```r
    pos <- get_positions(seg_sites[[locus]])
           ^~~~~~~~~~~~~
```
[R/model.R:133:21:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L133) *warning:* **no visible global function definition for ‘get_locus_length’**
```r
    locus_length <- get_locus_length(model, locus, total = FALSE)
                    ^~~~~~~~~~~~~~~~
```
[R/model.R:142:19:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model.R#L142) *warning:* **no visible global function definition for ‘get_trio_locus’**
```r
    trio_locus <- get_trio_locus(seg_sites[[locus]])
                  ^~~~~~~~~~~~~~
```
[R/model_scale.R:23:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_scale.R#L23) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.model(model))
  ^~~~~~~~~~~
```
[R/model_scale.R:23:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_scale.R#L23) *warning:* **no visible global function definition for ‘is.model’**
```r
  assert_that(is.model(model))
              ^~~~~~~~
```
[R/model_scale.R:24:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_scale.R#L24) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(scaling_factor))
  ^~~~~~~~~~~
```
[R/model_scale.R:25:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_scale.R#L25) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(length(scaling_factor) == 1)
  ^~~~~~~~~~~
```
[R/model_scale.R:27:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_scale.R#L27) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/model_simulate.R:40:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_simulate.R#L40) *warning:* **no visible global function definition for ‘select_simprog’**
```r
  simprog <- select_simprog(object)
             ^~~~~~~~~~~~~~
```
[R/model_simulate.R:44:21:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/model_simulate.R#L44) *warning:* **no visible global function definition for ‘prepare_pars’**
```r
    current_pars <- prepare_pars(pars, object)
                    ^~~~~~~~~~~~
```
[R/onLoad.R:3:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/onLoad.R#L3) *warning:* **no visible global function definition for ‘activate_scrm’**
```r
  activate_scrm()
  ^~~~~~~~~~~~~
```
[R/parameter_prior.R:30:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter_prior.R#L30) *warning:* **no visible binding for global variable ‘prior_par_class’**
```r
  prior_par_class$new(name, as.expression(substitute(prior)))
  ^~~~~~~~~~~~~~~
```
[R/parameter_prior.R:38:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter_prior.R#L38) *warning:* **no visible global function definition for ‘get_parameter’**
```r
  par <- get_parameter(model)
         ^~~~~~~~~~~~~
```
[R/parameter_prior.R:39:32:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter_prior.R#L39) *warning:* **no visible binding for global variable ‘is.prior_par’**
```r
  prior_par <- par[vapply(par, is.prior_par, logical(1))]
                               ^~~~~~~~~~~~
```
[R/parameter.R:123:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L123) *warning:* **no visible binding for global variable ‘parameter_class’**
```r
  parameter_class$new(as.expression(substitute(expr)))
  ^~~~~~~~~~~~~~~
```
[R/parameter.R:135:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L135) *warning:* **no visible binding for global variable ‘parameter_class’**
```r
  parameter_class$new(as.expression(constant))
  ^~~~~~~~~~~~~~~
```
[R/parameter.R:144:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L144) *warning:* **no visible binding for global variable ‘named_par_class’**
```r
  named_par_class$new(name)
  ^~~~~~~~~~~~~~~
```
[R/parameter.R:166:11:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L166) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if ((!is.numeric(value)) || length(value) != 1) {
          ^
```
[R/parameter.R:192:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L192) *warning:* **no visible binding for global variable ‘range_par_class’**
```r
  range_par_class$new(lower, upper, name)
  ^~~~~~~~~~~~~~~
```
[R/parameter.R:203:25:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L203) *warning:* **no visible binding for global variable ‘par_eval_func’**
```r
    par_env[["par"]] <- par_eval_func
                        ^~~~~~~~~~~~~
```
[R/parameter.R:209:25:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L209) *warning:* **no visible binding for global variable ‘par_print_func’**
```r
    par_env[["par"]] <- par_print_func
                        ^~~~~~~~~~~~~~
```
[R/parameter.R:225:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L225) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(pars))
  ^~~~~~~~~~~
```
[R/parameter.R:229:18:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L229) *warning:* **no visible global function definition for ‘get_par_names’**
```r
    par_names <- get_par_names(model, without_priors = TRUE)
                 ^~~~~~~~~~~~~
```
[R/parameter.R:237:11:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter.R#L237) *warning:* **no visible global function definition for ‘sample_par_priors’**
```r
  c(pars, sample_par_priors(model))
          ^~~~~~~~~~~~~~~~~
```
[R/parameter_variation.R:59:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/parameter_variation.R#L59) *warning:* **no visible binding for global variable ‘variation_par_class’**
```r
  variation_par_class$new(par, variance)
  ^~~~~~~~~~~~~~~~~~~
```
[R/segsites.R:47:11:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L47) *warning:* **no visible global function definition for ‘get_snps’**
```r
  snps <- get_snps(x)
          ^~~~~~~~
```
[R/segsites.R:48:28:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L48) *warning:* **no visible global function definition for ‘get_positions’**
```r
  colnames(snps) <- format(get_positions(x), scientific = FALSE)
                           ^~~~~~~~~~~~~
```
[R/segsites.R:62:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L62) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.matrix.segsites <- function(x, ...) get_snps(x)
^~~~~~~~~~~~~~~~~~
```
[R/segsites.R:75:19:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L75) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  create_segsites((matrix(c(1, 1, 0, 1, 1,
                  ^
```
[R/segsites.R:83:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L83) *warning:* **no visible global function definition for ‘create_segsites’**
```r
  create_segsites(matrix(0, n_ind, 0), numeric(0))
  ^~~~~~~~~~~~~~~
```
[R/segsites.R:103:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L103) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.list(left) && is.list(middle) && is.list(right))
  ^~~~~~~~~~~
```
[R/segsites.R:104:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L104) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(length(left) == length(middle))
  ^~~~~~~~~~~
```
[R/segsites.R:105:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L105) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(length(left) == length(right))
  ^~~~~~~~~~~
```
[R/segsites.R:142:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L142) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.default <- function(data, ...) {
^~~~~~~~~~~~~~~~~~~
```
[R/segsites.R:147:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L147) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.list <- function(data, ...) {
^~~~~~~~~~~~~~~~
```
[R/segsites.R:148:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L148) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(all(vapply(data, is_segsites, logical(1))))
  ^~~~~~~~~~~
```
[R/segsites.R:148:32:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/segsites.R#L148) *warning:* **no visible binding for global variable ‘is_segsites’**
```r
  assert_that(all(vapply(data, is_segsites, logical(1))))
                               ^~~~~~~~~~~
```
[R/simulator_class.R:28:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L28) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_simulator(simulator))
  ^~~~~~~~~~~
```
[R/simulator_class.R:28:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L28) *warning:* **no visible global function definition for ‘is_simulator’**
```r
  assert_that(is_simulator(simulator))
              ^~~~~~~~~~~~
```
[R/simulator_class.R:33:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L33) *warning:* **no visible binding for global variable ‘simulators’**
```r
  simulators[[name]]
  ^~~~~~~~~~
```
[R/simulator_class.R:40:19:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L40) *warning:* **no visible global function definition for ‘get_locus_length’**
```r
  locus_length <- get_locus_length(model, group = locus_group)
                  ^~~~~~~~~~~~~~~~
```
[R/simulator_class.R:41:25:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L41) *warning:* **no visible global function definition for ‘get_locus_number’**
```r
  total_locus_number <- get_locus_number(model, locus_group, TRUE)
                        ^~~~~~~~~~~~~~~~
```
[R/simulator_class.R:64:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L64) *warning:* **no visible global function definition for ‘reduce_sim_commands’**
```r
  reduce_sim_commands(sim_cmds)
  ^~~~~~~~~~~~~~~~~~~
```
[R/simulator_class.R:70:44:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L70) *style:* **Commas should never have a space before.**
```r
  grouped_commands <- unique(sim_commands[ , 2])
                                          ~^
```
[R/simulator_class.R:73:36:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L73) *style:* **Commas should never have a space before.**
```r
    sum(sim_commands[sim_commands[ , 2] == cmd, 1])
                                  ~^
```
[R/simulator_class.R:90:13:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_class.R#L90) *warning:* **no visible global function definition for ‘get_simulator’**
```r
    info <- get_simulator(simulator)$get_info()
            ^~~~~~~~~~~~~
```
[R/simulator_msms.R:109:41:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_msms.R#L109) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_msms.R:109:52:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_msms.R#L109) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/simulator_msms.R:158:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_msms.R#L158) *warning:* **no visible global function definition for ‘reset_cache’**
```r
  reset_cache()
  ^~~~~~~~~~~
```
[R/simulator_ms.R:67:37:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L67) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                    ^
```
[R/simulator_ms.R:87:39:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L87) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                     ~^
```
[R/simulator_ms.R:87:50:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L87) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                                ~^
```
[R/simulator_ms.R:136:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L136) *warning:* **no visible global function definition for ‘register_simulator’**
```r
  register_simulator(ms_class$new(priority))
  ^~~~~~~~~~~~~~~~~~
```
[R/simulator_ms.R:136:22:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L136) *warning:* **no visible binding for global variable ‘ms_class’**
```r
  register_simulator(ms_class$new(priority))
                     ^~~~~~~~
```
[R/simulator_ms.R:137:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_ms.R#L137) *warning:* **no visible global function definition for ‘reset_cache’**
```r
  reset_cache()
  ^~~~~~~~~~~
```
[R/simulator_scrm.R:11:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L11) *warning:* **no visible global function definition for ‘read_cache’**
```r
  cmd <- read_cache(model, "scrm_cmd")
         ^~~~~~~~~~
```
[R/simulator_scrm.R:17:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L17) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "scrm_cmd", cmd)
    ^~~~~
```
[R/simulator_scrm.R:83:41:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L83) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_scrm.R:83:52:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L83) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/simulator_scrm.R:129:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L129) *warning:* **no visible global function definition for ‘register_simulator’**
```r
  register_simulator(scrm_class$new(priority))
  ^~~~~~~~~~~~~~~~~~
```
[R/simulator_scrm.R:129:22:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L129) *warning:* **no visible binding for global variable ‘scrm_class’**
```r
  register_simulator(scrm_class$new(priority))
                     ^~~~~~~~~~
```
[R/simulator_scrm.R:130:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_scrm.R#L130) *warning:* **no visible global function definition for ‘reset_cache’**
```r
  reset_cache()
  ^~~~~~~~~~~
```
[R/simulator_seqgen.R:5:17:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L5) *warning:* **no visible global function definition for ‘read_cache’**
```r
  tree_model <- read_cache(model, "tree_model")
                ^~~~~~~~~~
```
[R/simulator_seqgen.R:20:29:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L20) *warning:* **no visible global function definition for ‘create_sumstat_container’**
```r
    tree_model$sum_stats <- create_sumstat_container()
                            ^~~~~~~~~~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:21:32:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L21) *warning:* **no visible global function definition for ‘sumstat_sg_trees’**
```r
    tree_model <- tree_model + sumstat_sg_trees()
                               ^~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:23:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L23) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "tree_model", tree_model)
    ^~~~~
```
[R/simulator_seqgen.R:41:20:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L41) *warning:* **no visible global function definition for ‘get_locus_length’**
```r
  locus_lengths <- get_locus_length(model, group = locus, total = FALSE)
                   ^~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:47:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L47) *warning:* **no visible global function definition for ‘sg_generate_opt_cmd’**
```r
  cmd <- sg_generate_opt_cmd(model)
         ^~~~~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:61:10:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L61) *warning:* **no visible global function definition for ‘read_cache’**
```r
  cmd <- read_cache(model, "seqgen_cmd")
         ^~~~~~~~~~
```
[R/simulator_seqgen.R:74:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L74) *warning:* **no visible global function definition for ‘cache’, Did you mean 'cycle'?**
```r
    cache(model, "seqgen_cmd", cmd)
    ^~~~~
```
[R/simulator_seqgen.R:222:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L222) *warning:* **no visible global function definition for ‘register_simulator’**
```r
  register_simulator(seqgen_class$new(binary, priority))
  ^~~~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:222:22:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L222) *warning:* **no visible binding for global variable ‘seqgen_class’**
```r
  register_simulator(seqgen_class$new(binary, priority))
                     ^~~~~~~~~~~~
```
[R/simulator_seqgen.R:223:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/simulator_seqgen.R#L223) *warning:* **no visible global function definition for ‘reset_cache’**
```r
  reset_cache()
  ^~~~~~~~~~~
```
[R/sumstat_dna.R:40:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_dna.R#L40) *warning:* **no visible binding for global variable ‘stat_dna_class’**
```r
  stat_dna_class$new(name, transformation)
  ^~~~~~~~~~~~~~
```
[R/sumstat_file.R:46:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_file.R#L46) *warning:* **no visible binding for global variable ‘stat_file_class’**
```r
  stat_file_class$new(folder)
  ^~~~~~~~~~~~~~~
```
[R/sumstat_four_gamete.R:73:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_four_gamete.R#L73) *warning:* **no visible binding for global variable ‘stat_four_gamete_class’**
```r
  stat_four_gamete_class$new(name, population, transformation)
  ^~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_ihh.R:53:45:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_ihh.R#L53) *style:* **Commas should never have a space before.**
```r
                      iHS = data.frame(ihh[ , 1:2], iHS = rep(NA, n_snps))))
                                           ~^
```
[R/sumstat_ihh.R:61:55:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_ihh.R#L61) *style:* **Commas should never have a space before.**
```r
            data.frame(ihh2ihs(ihh, freqbin)$res.ihs[ , -4])
                                                     ~^
```
[R/sumstat_ihh.R:148:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_ihh.R#L148) *warning:* **no visible binding for global variable ‘stat_ihh_class’**
```r
  stat_ihh_class$new(name, population, max_snps, calc_ihs, transformation)
  ^~~~~~~~~~~~~~
```
[R/sumstat_jsfs.R:60:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_jsfs.R#L60) *warning:* **no visible binding for global variable ‘stat_jsfs_class’**
```r
  stat_jsfs_class$new(name, populations, per_locus, transformation)
  ^~~~~~~~~~~~~~~
```
[R/sumstat_mcmf.R:44:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_mcmf.R#L44) *warning:* **no visible binding for global variable ‘stat_mcmf_class’**
```r
  stat_mcmf_class$new(name, population, transformation)
  ^~~~~~~~~~~~~~~
```
[R/sumstat_nucleotide_div.R:46:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_nucleotide_div.R#L46) *warning:* **no visible binding for global variable ‘stat_pi_class’**
```r
  stat_pi_class$new(name, population, transformation)
  ^~~~~~~~~~~~~
```
[R/sumstat_omega.R:156:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_omega.R#L156) *warning:* **no visible binding for global variable ‘stat_omega_class’**
```r
  stat_omega_class$new(name, min_win, max_win, grid, binary, transformation)
  ^~~~~~~~~~~~~~~~
```
[R/sumstat_omega.R:160:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_omega.R#L160) *warning:* **no visible global function definition for ‘search_executable’**
```r
  !is.null(search_executable("OmegaPlus", envir_var = "OMEGAPLUS"))
           ^~~~~~~~~~~~~~~~~
```
[R/sumstat.R:71:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L71) *warning:* **no visible global function definition for ‘get_id’, Did you mean 'getwd'?**
```r
  model$id <- get_id()
              ^~~~~~
```
[R/sumstat.R:87:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L87) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.model(model))
  ^~~~~~~~~~~
```
[R/sumstat.R:87:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L87) *warning:* **no visible global function definition for ‘is.model’**
```r
  assert_that(is.model(model))
              ^~~~~~~~
```
[R/sumstat.R:89:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L89) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(!is.null(segsites_list))
    ^~~~~~~~~~~
```
[R/sumstat.R:90:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L90) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(all(vapply(segsites_list, is_segsites, logical(1))))
    ^~~~~~~~~~~
```
[R/sumstat.R:90:43:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L90) *warning:* **no visible binding for global variable ‘is_segsites’**
```r
    assert_that(all(vapply(segsites_list, is_segsites, logical(1))))
                                          ^~~~~~~~~~~
```
[R/sumstat.R:93:24:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L93) *warning:* **no visible global function definition for ‘remove_singletons’**
```r
      segsites_list <- remove_singletons(segsites_list)
                       ^~~~~~~~~~~~~~~~~
```
[R/sumstat.R:96:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L96) *warning:* **no visible global function definition for ‘requires_trees’**
```r
  if (requires_trees(model)) assert_that(!is.null(trees))
      ^~~~~~~~~~~~~~
```
[R/sumstat.R:96:30:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L96) *warning:* **no visible global function definition for ‘assert_that’**
```r
  if (requires_trees(model)) assert_that(!is.null(trees))
                             ^~~~~~~~~~~
```
[R/sumstat.R:97:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L97) *warning:* **no visible global function definition for ‘requires_files’**
```r
  if (requires_files(model)) assert_that(!is.null(files))
      ^~~~~~~~~~~~~~
```
[R/sumstat.R:97:30:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L97) *warning:* **no visible global function definition for ‘assert_that’**
```r
  if (requires_files(model)) assert_that(!is.null(files))
                             ^~~~~~~~~~~
```
[R/sumstat.R:111:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L111) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.model(model))
  ^~~~~~~~~~~
```
[R/sumstat.R:111:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L111) *warning:* **no visible global function definition for ‘is.model’**
```r
  assert_that(is.model(model))
              ^~~~~~~~
```
[R/sumstat.R:120:20:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L120) *warning:* **no visible global function definition for ‘conv_for_trios’**
```r
      seg_sites <- conv_for_trios(seg_sites, model)
                   ^~~~~~~~~~~~~~
```
[R/sumstat.R:176:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L176) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.model(model))
  ^~~~~~~~~~~
```
[R/sumstat.R:176:15:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L176) *warning:* **no visible global function definition for ‘is.model’**
```r
  assert_that(is.model(model))
              ^~~~~~~~
```
[R/sumstat.R:179:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L179) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.list(trios))
    ^~~~~~~~~~~
```
[R/sumstat.R:180:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L180) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(!is.null(segsites_list))
    ^~~~~~~~~~~
```
[R/sumstat.R:186:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L186) *warning:* **no visible global function definition for ‘assert_that’**
```r
      assert_that(is.numeric(trio))
      ^~~~~~~~~~~
```
[R/sumstat.R:188:7:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L188) *warning:* **no visible global function definition for ‘assert_that’**
```r
      assert_that(length(trio) == 3)
      ^~~~~~~~~~~
```
[R/sumstat.R:196:22:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L196) *warning:* **no visible global function definition for ‘as.segsites’**
```r
    segsites_list <- as.segsites(segsites_list, ...)
                     ^~~~~~~~~~~
```
[R/sumstat.R:200:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L200) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(segsites_list) == get_locus_number(model))
    ^~~~~~~~~~~
```
[R/sumstat.R:200:42:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L200) *warning:* **no visible global function definition for ‘get_locus_number’**
```r
    assert_that(length(segsites_list) == get_locus_number(model))
                                         ^~~~~~~~~~~~~~~~
```
[R/sumstat.R:209:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L209) *warning:* **no visible global function definition for ‘calc_sumstats’**
```r
  calc_sumstats(model, segsites_list, tree_list)
  ^~~~~~~~~~~~~
```
[R/sumstat.R:214:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L214) *warning:* **no visible global function definition for ‘get_summary_statistics’**
```r
  any(sapply(get_summary_statistics(model), function(x) x$requires_segsites()))
             ^~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat.R:219:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L219) *warning:* **no visible global function definition for ‘get_summary_statistics’**
```r
  any(sapply(get_summary_statistics(model), function(x) x$requires_trees()))
             ^~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat.R:224:14:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat.R#L224) *warning:* **no visible global function definition for ‘get_summary_statistics’**
```r
  any(sapply(get_summary_statistics(model), function(x) x$requires_files()))
             ^~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_seg_sites.R:30:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_seg_sites.R#L30) *warning:* **no visible binding for global variable ‘stat_segsites_class’**
```r
  stat_segsites_class$new(name, transformation)
  ^~~~~~~~~~~~~~~~~~~
```
[R/sumstat_seg_sites.R:36:21:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_seg_sites.R#L36) *warning:* **no visible global function definition for ‘get_locus_length’**
```r
    locus_length <- get_locus_length(model, i, total = FALSE)
                    ^~~~~~~~~~~~~~~~
```
[R/sumstat_seg_sites.R:42:12:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_seg_sites.R#L42) *warning:* **no visible global function definition for ‘get_positions’**
```r
    pos <- get_positions(seg_sites[[i]])
           ^~~~~~~~~~~~~
```
[R/sumstat_seg_sites.R:51:38:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_seg_sites.R#L51) *style:* **Commas should never have a space before.**
```r
    trio_segsites <- seg_sites[[i]][ , left | middle | right]
                                    ~^
```
[R/sumstat_seg_sites.R:60:5:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_seg_sites.R#L60) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(nrow(seg_sites[[i]]) > 0)
    ^~~~~~~~~~~
```
[R/sumstat_sfs.R:38:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_sfs.R#L38) *warning:* **no visible binding for global variable ‘stat_sfs_class’**
```r
  stat_sfs_class$new(name, population, transformation)
  ^~~~~~~~~~~~~~
```
[R/sumstat_tajimas_d.R:62:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_tajimas_d.R#L62) *warning:* **no visible binding for global variable ‘stat_tajimas_d_class’**
```r
  stat_tajimas_d_class$new(name, population, transformation)
  ^~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_trees.R:28:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_trees.R#L28) *warning:* **no visible binding for global variable ‘stat_trees_class’**
```r
  stat_trees_class$new(name, identity)
  ^~~~~~~~~~~~~~~~
```
[R/sumstat_trees.R:57:3:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/R/sumstat_trees.R#L57) *warning:* **no visible binding for global variable ‘stat_sg_trees_class’**
```r
  stat_sg_trees_class$new("trees", identity)
  ^~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-feature-ign-singletons.R:13:45:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-feature-ign-singletons.R#L13) *style:* **Commas should never have a space before.**
```r
               list(create_test_segsites()[ , -c(1, 3)],
                                           ~^
```
[tests/testthat/test-feature-ign-singletons.R:14:45:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-feature-ign-singletons.R#L14) *style:* **Commas should never have a space before.**
```r
                    create_test_segsites()[ , -c(1, 3)]))
                                           ~^
```
[tests/testthat/test-model-cache.R:47:1:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-model-cache.R#L47) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-model-class.R:241:44:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-model-class.R#L241) *style:* **Commas should always have a space after.**
```r
               data.frame(name = c("theta","tau"),
                                           ^
```
[tests/testthat/test-segsites-class.R:30:31:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-segsites-class.R#L30) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  segsites <- create_segsites((matrix(c(1, 1, 0, 1, 1,
                              ^
```
[tests/testthat/test-segsites-class.R:36:26:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-segsites-class.R#L36) *style:* **Commas should never have a space before.**
```r
  ss_subset <- segsites[ , 1:2]
                        ~^
```
[tests/testthat/test-segsites-class.R:38:64:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-segsites-class.R#L38) *style:* **Commas should never have a space before.**
```r
  expect_equivalent(as.matrix(ss_subset), as.matrix(segsites)[ , 1:2])
                                                              ~^
```
[tests/testthat/test-segsites-class.R:41:30:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-segsites-class.R#L41) *style:* **Commas should never have a space before.**
```r
  expect_equal(dim(segsites[ , 1]), c(3, 1))
                            ~^
```
[tests/testthat/test-segsites-class.R:42:33:](https://github.com/statgenlmu/coala/blob/271265e95c160823a187e6298edc72aa62795e66/tests/testthat/test-segsites-class.R#L42) *style:* **Commas should never have a space before.**
```r
  expect_equal(dim(segsites[1:2 , ]), c(2, 3))
                               ~^
```


А почему отступы не одинаковые?


"romanaIP" is the GW address? Should we maybe call it that?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14815](https://teamcity.mesosphere.io/viewLog.html?buildId=128105&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:05:03


Maybe putting the return inside the if and else would make a better code


Please pull that out. I spend half of my last PR to turn similar stuff into tableOpts = opts_print....


Hi Stefano,

you can see at
https://github.com/pb00068/Stockfish/tree/skipsize
how skipsize can be calculated.


even if i have installed ipython , it is not working


I know you love Javascript but I think you mean Jbuilder, right?


The `desktopService` seems to have appeared!


Minor nitpick, in the other functions (e.g. getScatterOptsColormap) you pass the full struct. For consistency lets do the same here.


instead na ung nilagay na date ang lumabas ang nangyayare ung current date ang lumalabas.


> Ca vient de là c'est sûr

Quel était le problème ?


Should be isset($C->default_timezone);


If `foreground` is absent from the JSON object, it'll run as if the deafult value for it was `false`, won't it? If so, it goes against the documented default value of `true`.


Also on the different occurrences, please pull that into a separate opts_print.


Please add a newline.


For better readability when on cli


Before this read `selectionService.selectedStock() === stock`, not `stock.code` ...


Nevermind, I forgot to see the whole commit before writing stuff.


Note that this is only for development mode


What I don't understand though is that you are not testing that `force` is not stored given you are passing in `force` both times no?  Surely you need to check that calling `authorise` with no params a second time with no `force` param would not trigger a `request_token`?


This change doesn't seem to be iso-functional


this _will_ break inference of array type in certain cases. You really should've used a new type for the fixed size array. Just like what should've been done with `CompositeLiteral`. It results in much cleaner handling of these cases.


@pb00068: Just don't do it. It's against the rules.

There are good statistical reasons for this. If you don't know enough statistics to understand why the rules are in place, then just follow the rules.


if it was entirely up to me, I would leave everything on the same line...
#80columnsisstupid


Perhaps a better description:

A component that transforms HTTP status codes to UP and DOWN availability constants. Input are messages comming via HawkularMetricData JMS topic and the output is sent to Metrics via REST.


Ok, the SVG is indeed smaller and if it displays better then I'll apply it.


Good but a few points before a PR:

- Please always use braces even for 1 line if else's.
- The MinecraftVersion object has a `usesCoreMods()` method which returns a boolean on if it uses core mods or not, that would be a simpler and less error prone approach. Thankfully it also ties into the same time texture packs were phased out so will be a better solution.


What is it?


This should be left out as this feature is not currently supported in gintonicweb/requirejs.


Does your device also get stuck at "preparing contacts storage" on clean flash with a big gapps package like open gapps mini?


C'est ce build là qui posait problème (hors test_plf) pour la fiche de paie du salarié IPP.
Vérifie que tes tests passent bien (si tu en as). Mais cela devrait aller !


@jpkrohling are you putting the dependencies in some well defined order? FWIW, tend to use and prefer alphabetical order by groupId and arifactId.


Probably should rename this as "model_factory" or something


Please favor splitting lines between tags rather than attributes when possible.


These constants are not used in the end, so I will be removing them before merging.


Uploaded!


Concerning the LOD word... meh, I was just excited I think lol, but the problem here would be to apply sprites even to some actors which doesn't have fitting sprites but just placeholders (for example, burnt ones).

Concerning the SwitchableDecoration thing, well imagine this situation:
a player likes to play BoA with the default value for the LOD distance, which is 8192mp;
a mapper build a large area filled with trees and in the center of it we'll see a building that we need to enter; it has a lift and we'll go below once there and once we get on the lift the door in front of us will close forever.
After that the downhill sequence is starting then we'll need to create a fall-through teleport in order to simulate the "descent" with such elevator, so teleporting BJ into another part of the map but IN THE SAME MAP.
So with the SwitchableDecoration thing you can add a triggering line once if passed through will deactivate trees, with the result that those actors will never be renderized as 3d models at all even if the user set any kind of distance through BoA Options.
Ikr there is the SetState for acs, but this way is more easy for mappers to get this trick though.

Sorry for my bad english here, I am in a hurry but I had to explain absolutely what's my aim here with switchable stuff.

Thankx, Ozy81.


I don't know how eventloop integration works in node, especially since it has no sensible concurrency. You may not be able to avoid working with the edge-triggered FD, which I promise will be a headache.

There is a proposal to add a `socket.LEVEL_FD` that behaves sensibly, but that's probably some way off.


Need to update the comment here, since this u32 rule for traffic between host and endpoint doesn't exist anymore.


Doing it this way pollutes the global namespace. We should just return the created array value before we create the backing store.


And add a note about the DHCP that's also allowed.


Podのupdateで結構な量になってる


A component


first() works fine for arrays. But fore AR::Relations object, first() doesn't guarantee the zeroth element. To ensure you get first element you need to have ORDER BY some_column clause in your SQL query to get exactly what you expect. In other words AR::Relations doesn't guarantee order unless you order things yourself.


@Commit message: Cairo does have (since 1.12) raster sources: http://cairographics.org/manual/cairo-Raster-Sources.html

This is a cairo "surface" which you cannot draw to, but only use as a source (it's a pattern actually). I don't know Qt's brushes, but I guess since cairo still requires you to know the size in advance, this doesn't actually help, right?


A little example in the comment here would help.


@damiencaselli needed it for updating pypi.


No more log message?


policy up until now is that we never create destination queues, so if we cannot differentiate between destination queues & error/audit we should create none of the destination queues


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14816](https://teamcity.mesosphere.io/viewLog.html?buildId=128110&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:10:38


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 92](https://teamcity.labkey.org/viewLog.html?buildId=301586&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:06:30
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 4 more failed tests, see build details
```


Prachtig


Someone reading my blog! Amazing.
BTW, fixed


I think we can shorten this down a bit:

```python
from tempfile import TemporaryDirectory
with (TemporaryDirectory() as tempdir,
      make_temp(suffix='.orig', dir=tempdir) as orig_file,
      make_temp(suffix='.coafile', prefix='', dir=tempdir) as coafile,
      make_temp(dir=tempdir) as unrelated_file):
    execute
    ...

Note:

1. You can chain those things :)
2. Why do you mix make_temp and NamedTemporaryFile? I see no reason.
3. Use speaking names. f_a, f_b isn't particularly helpful. I know we do that too sometimes, that's a bug ;)
```


Have a look at https://duckduckgo.com/?q=how+to+write+good+commit+messages :) It may not seem that important right now, but once you need to trawl through the history at some point to figure out why a given mod was added where (or perhaps just in what commit a given mod was added), having properly written commit messages will be invaluable.


Hm, this was just a cherry pick of 6f890340011a5d48f30318cfd27b39ade78e94a4.


Then your commit makes no sense. somearray[0] isn't ordering anything more than somearray.first is. It does nothing but make the code slightly less readable.


Why `panic` instead of return `errors.New`?


"... in combination ..."


Documentation would be very helpful here. What are restrictions on value of `dir`? What format is it?


```Cha-ching!```?


`user.Current()` is not friendly towards cross-compilation.

What do you think about having an `os.Getenv("HOME")` fallback in case `user.Current()` returns a non-`nil` error? As long as `os.Getenv("HOME")` isn't empty string, of course.


I don't understand what you mean, but if you're looking for an example that's compatible with both 1.9 and 1.10, in addition to the snippet that was posted in the PR conversation, here's the one-liner we use: https://github.com/yourlabs/django-autocomplete-light/blob/master/src/dal/widgets.py#L62


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14817](https://teamcity.mesosphere.io/viewLog.html?buildId=128111&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:08


schön! :+1:


I suggest that you test django.VERSION < (1, 10) instead of a try/except block.


I don't think you need to support the choices arg in < 1.10, it's not used by anything in django.


:+1: Ok with the changes.

Something to take into account for the next iteration: if we move stuff from pl/python to pure python domain we would save a lot of headaches. Something like keeping the same interface but adding "hooks" for metrics might not need changes in the extension but just python lib code making everything easier to manage.

Here's the example that everything but the extension's entry point can be moved to pure python:

https://github.com/CartoDB/trade-areas-poc/blob/master/lib/python/cdb/cdb/here/types.py

It can be unit-tested in python domain by mocking `plpy` and just checking the `execute` queries are the ones expected.


like this.
![2016-02-08 18 06 48](https://cloud.githubusercontent.com/assets/10813852/12881793/e313334a-ce8f-11e5-951b-343d3feec80d.png)


@bauffman I tried Emmet a while back but didn't like it much. It's nice for longer, more complex snippets because you can name your vars, but as you can see I only need a handful and don't really have need for an additional syntax layer inbetween. I use https://github.com/alvan/vim-closetag to avoid repeating closing tags in {HT,X}ML.


Not sure where 4.4.1 comes from historically, but given that we generally want to use the lib provided by the container, the version should be `4.5` as found in WF 10.0.0.Final. At the same time the prop should be commented with something like `Keep in sync with httpclient in modules/system/layers/base/org/apache/httpcomponents/main of WF/EAP`


Why is it version `0.3.0`? we could have restarted the counter on `0.0.1`, there's none released yet.


I guess you have in mind a path for migrating to cdb_dataservices_server`, can you elaborate or is it documented somewhere?


This didn't help with /vendor/ paths because `srcDir` only has an effect for resolving relative import paths that are specified globally, I think. But all /vendor/ import paths are absolute.


I can re-order them. Usually, I put "our" components first (as they will get updated often), then Wildfly-provided dependencies, then "misc" dependencies, then test dependencies. I don't usually have a specified order for dependencies within each of those groups, though.


Habs rückgängig gemacht, Danke @gharlan


Debug class is already [Conditional("DEBUG)].


wrongly spelled


Mind that this also requires changes in the rails code.


TeamCity Civic Assembly :: Build backend + frontend [Build 77](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=838&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


I think it should be "destiny"!


We need to wait for https://github.com/pypa/pip/pull/3450 to be merged into pip.


ah nice addition to this! Thanks a bunch @jfonte


@zamar OK, got it. Aborted SPRT-tests are somehow prohibitive for further investigations, as someone could repeat them until one passes by lucky accident.


Thanks Marc, I'm glad to hear you like it and that it has value to you as an author. 👍🏼


Related to https://github.com/ezsystems/ez-js-rest-client/pull/72#discussion_r52101935

you can write:
```js
    var standardMethods = {"OPTIONS": 1, "GET": 1, "HEAD": 1, "POST": 1, "PUT": 1, "DELETE": 1, "TRACE": 1};

    if ( standardMethods[method] ) {
        method = "POST";
    }
```


style nit: spaces before and after `=`


PR #1788 fixes this compile error.


hi


Is cassandra-driver really necessary here?


This commit has been reverted. It should have been pushed to a feature branch instead of master.


> ITU REC 709 is an encoding standard, not a viewing standard.
>
In Natron, we also use OCIO to linearize the transfer function when decoding images and videos, and most videos nowadays seem to use Rec.709, Rec.2020, or one of the constructor-specific so-called "log" curves.

There is probably the same issue with blender when using it for compositing (what transfer function do the ToS raw footage use?).

Or should the blender config *not* be considered as a config to be used for decoding and encoding videos ?

In this case, should Natron prefer nuke-default, or can you help make a config based on this new blender config that would add transforms for decoding and encoding common video formats?
> Would you be interested in a custom crafted Natron config set? I am leaning towards one using sRGB / 709 primaries as a reference space, and perhaps another wide gamut that would be an introductory version of ACEScg, with fewer options to confuse folks?
>


Hi,

This breaks test.test_YoutubeDL:TestFormatSelection.test_youtube_format_selection :

FAIL: test_youtube_format_selection (test.test_YoutubeDL.TestFormatSelection)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/youtube-dl/test/test_YoutubeDL.py", line 323, in test_youtube_format_selection
    self.assertEqual(downloaded['format_id'], f1['format_id'])
nose.proxy.AssertionError: '17' != '36'
- 17
+ 36

    """Fail immediately, with the given message."""
>>  raise self.failureException("'17' != '36'\n- 17\n+ 36\n")


Shouldn't this be kept for code clarity?


Not the trace listeners ...


s/the yeGet/yeGet/


echo "hello world"


Yes, I plan to add some write-up.


@agershun  Question:

What are the ideas behind having this test to avoid defining the `saveAs` var from `FileSave.js`?

I have a feeling it would be better to place tests like this where we would access `saveAs`. Any inputs ?


Plz. define RX. RX extensions of what?