[R/block.R:2:32:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
block_class <- R6Class("Block",
                               ^
```
[R/block.R:8:29:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L8) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      assert_that(all(border[ , 1] < border[ , 2]))
                            ^
```
[R/block.R:9:31:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L9) *style:* **Commas should never have a space before.**
```r
      assert_that(all(border[ , 1] < border[ , 2]))
                             ~^
```
[R/block.R:9:46:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L9) *style:* **Commas should never have a space before.**
```r
      assert_that(all(border[ , 1] < border[ , 2]))
                                            ~^
```
[R/block.R:20:42:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L20) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      upper <- denormalize(private$border[ , 2], jaatha)
                                         ^
```
[R/block.R:20:44:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L20) *style:* **Commas should never have a space before.**
```r
      lower <- denormalize(private$border[ , 1], jaatha)
                                          ~^
```
[R/block.R:21:44:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L21) *style:* **Commas should never have a space before.**
```r
      upper <- denormalize(private$border[ , 2], jaatha)
                                          ~^
```
[R/block.R:25:27:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L25) *style:* **Commas should never have a space before.**
```r
      all(private$border[ , 1] - 1e-15 <= point &
                         ~^
```
[R/block.R:25:50:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
      all(private$border[ , 1] - 1e-15 <= point &
                                                 ^
```
[R/block.R:26:38:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L26) *style:* **Commas should never have a space before.**
```r
            point <= private$border[ , 2] + 1e-15)
                                    ~^
```
[R/block.R:28:27:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L28) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      m <- (private$border[ , 2] - private$border[ , 1]) / 2 +
                          ^
```
[R/block.R:29:29:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L29) *style:* **Commas should never have a space before.**
```r
      m <- (private$border[ , 2] - private$border[ , 1]) / 2 +
                           ~^
```
[R/block.R:29:52:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L29) *style:* **Commas should never have a space before.**
```r
      m <- (private$border[ , 2] - private$border[ , 1]) / 2 +
                                                  ~^
```
[R/block.R:29:63:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L29) *style:* **Trailing whitespace is superfluous.**
```r
      m <- (private$border[ , 2] - private$border[ , 1]) / 2 +
                                                              ^
```
[R/block.R:30:25:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L30) *style:* **Commas should never have a space before.**
```r
        private$border[ , 1]
                       ~^
```
[R/block.R:36:26:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L36) *style:* **Commas should never have a space before.**
```r
        private$border[i , , drop = FALSE]
                        ~^
```
[R/block.R:36:28:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L36) *style:* **Commas should never have a space before.**
```r
        private$border[i , , drop = FALSE]
                          ~^
```
[R/block.R:37:7:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L37) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
      }), KEEP.OUT.ATTRS = FALSE)
      ^
```
[R/block.R:45:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L45) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/block.R:49:58:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L49) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
                                  max = self$get_border()[ , 2]),
                                                         ^
```
[R/block.R:49:60:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L49) *style:* **Commas should never have a space before.**
```r
                                  min = self$get_border()[ , 1],
                                                          ~^
```
[R/block.R:50:60:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L50) *style:* **Commas should never have a space before.**
```r
                                  max = self$get_border()[ , 2]),
                                                          ~^
```
[R/block.R:52:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/block.R#L52) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/boot_jaatha.R:24:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L24) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("boot")
  ^~~~~~~~~~~~~~~
```
[R/boot_jaatha.R:28:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L28) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/boot_jaatha.R:33:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L33) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/boot_jaatha.R:38:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L38) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/boot_jaatha.R:41:76:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L41) *style:* **Trailing whitespace is superfluous.**
```r
                   file = tempfile(paste0("boot_log_", Sys.getpid(), "_")),
                                                                           ^
```
[R/boot_jaatha.R:43:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L43) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/boot_jaatha.R:46:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L46) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/boot_jaatha.R:49:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L49) *warning:* **no visible global function definition for ‘create_jaatha_data.default’**
```r
    create_jaatha_data.default(sim_data, model)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/boot_jaatha.R:51:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L51) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/boot_jaatha.R:52:39:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L52) *style:* **Trailing whitespace is superfluous.**
```r
  boot::boot(data, jaatha_stat, R = R,
                                      ^
```
[R/boot_jaatha.R:53:33:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L53) *style:* **Trailing whitespace is superfluous.**
```r
             sim = "parametric",
                                ^
```
[R/boot_jaatha.R:54:38:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/boot_jaatha.R#L54) *style:* **Trailing whitespace is superfluous.**
```r
             ran.gen = simulate_data,
                                     ^
```
[R/coala_interface.R:33:45:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L33) *style:* **Trailing whitespace is superfluous.**
```r
create_jaatha_model.coalmodel <- function(x,
                                            ^
```
[R/coala_interface.R:44:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:45:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L45) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("coala")
  ^~~~~~~~~~~~~~~
```
[R/coala_interface.R:48:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L48) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:50:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L50) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:53:38:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L53) *style:* **Commas should always have a space after.**
```r
  par_ranges <- as.matrix(par_table[,-1])
                                     ^
```
[R/coala_interface.R:60:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L60) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:61:64:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L61) *style:* **Trailing whitespace is superfluous.**
```r
  create_jaatha_model.function(sim_func, par_ranges, sum_stats,
                                                               ^
```
[R/coala_interface.R:69:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L69) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:70:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L70) *warning:* **no visible global function definition for ‘require_package’**
```r
  require_package("coala")
  ^~~~~~~~~~~~~~~
```
[R/coala_interface.R:71:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L71) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.string(jsfs_summary))
  ^~~~~~~~~~~
```
[R/coala_interface.R:71:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L71) *warning:* **no visible global function definition for ‘is.string’**
```r
  assert_that(is.string(jsfs_summary))
              ^~~~~~~~~
```
[R/coala_interface.R:72:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:75:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:80:11:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L80) *warning:* **no visible global function definition for ‘coarsen_jsfs’**
```r
          coarsen_jsfs(x[[name]], jsfs_part, jsfs_part_hi)
          ^~~~~~~~~~~~
```
[R/coala_interface.R:90:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L90) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:93:14:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L93) *warning:* **no visible global function definition for ‘create_jaatha_stat’**
```r
      return(create_jaatha_stat(name, function(x, opts) x[[name]]))
             ^~~~~~~~~~~~~~~~~~
```
[R/coala_interface.R:95:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L95) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:99:20:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L99) *style:* **Commas should never have a space before.**
```r
        x[[name]][ , c(1, 2, 6), drop = FALSE]
                  ~^
```
[R/coala_interface.R:100:7:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L100) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
      }, poisson = FALSE, breaks = four_gamete_breaks))
      ^
```
[R/coala_interface.R:102:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L102) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:106:49:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L106) *style:* **Trailing whitespace is superfluous.**
```r
                                poisson = FALSE,
                                                ^
```
[R/coala_interface.R:109:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L109) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:119:4:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L119) *style:* **Commented code should be removed.**
```r
  ## A[p[1,1]:p[1:2],p[2,1]:p[2:2],...] consists of the same values as A[v],
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/coala_interface.R:122:12:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L122) *style:* **Commas should always have a space after.**
```r
  v <- p[N,1]:p[N,2]
           ^
```
[R/coala_interface.R:122:19:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L122) *style:* **Commas should always have a space after.**
```r
  v <- p[N,1]:p[N,2]
                  ^
```
[R/coala_interface.R:125:28:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L125) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      v <- as.vector(outer((v - 1)*d[n], p[n, 1]:p[n, 2], "+"))
                           ^
```
[R/coala_interface.R:125:35:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L125) *style:* **Put spaces around all infix operators.**
```r
      v <- as.vector(outer((v - 1)*d[n], p[n, 1]:p[n, 2], "+"))
                                 ~^~
```
[R/coala_interface.R:128:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L128) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:177:26:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L177) *style:* **Commas should always have a space after.**
```r
      if (tail(part[[i]],1) >= upper[1]) {
                         ^
```
[R/coala_interface.R:183:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L183) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:187:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L187) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:189:18:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L189) *style:* **Trailing whitespace is superfluous.**
```r
  combinations <-
                 ^
```
[R/coala_interface.R:191:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L191) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:199:20:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L199) *warning:* **no visible global function definition for ‘multi_index_range’**
```r
    z[i] <- sum(ja[multi_index_range(d, p)])
                   ^~~~~~~~~~~~~~~~~
```
[R/coala_interface.R:201:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L201) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:204:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/coala_interface.R#L204) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/fit_glm.R:8:52:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L8) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
fit_glm.jaatha_model <- function(x, sim_data, ...) {
                                                   ^
```
[R/fit_glm.R:8:53:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L8) *style:* **Trailing whitespace is superfluous.**
```r
fit_glm.jaatha_model <- function(x, sim_data, ...) {
                                                    ^
```
[R/fit_glm.R:10:29:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L10) *warning:* **no visible binding for global variable ‘fit_glm’**
```r
  lapply(x$get_sum_stats(), fit_glm, sim_data, ...)
                            ^~~~~~~
```
[R/fit_glm.R:19:16:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L19) *style:* **Trailing whitespace is superfluous.**
```r
  X <- cbind(1,
               ^
```
[R/fit_glm.R:21:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L21) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/fit_glm.R:24:21:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L24) *style:* **Commas should never have a space before.**
```r
      glm.fit(X, Y[ , i], family = poisson("log"),
                   ~^
```
[R/fit_glm.R:24:51:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L24) *style:* **Trailing whitespace is superfluous.**
```r
      glm.fit(X, Y[ , i], family = poisson("log"),
                                                  ^
```
[R/fit_glm.R:28:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L28) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/fit_glm.R:42:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L42) *style:* **Commented code should be removed.**
```r
#   par_names <- names(sim_data[[1]]$pars)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:50:7:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L50) *style:* **Commented code should be removed.**
```r
#     colnames(pars) <- names(sim_result$pars.normal)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:51:7:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L51) *style:* **Commented code should be removed.**
```r
#     data.frame(pars, sim_result[[sum_stat$get_name()]])
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:57:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L57) *style:* **Commented code should be removed.**
```r
#   if (!smooth_glm$converged) stop("GLM did not converge")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:59:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/fit_glm.R#L59) *style:* **Commented code should be removed.**
```r
#   list(smooth=smooth_glm)
    ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:16:70:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L16) *style:* **Trailing whitespace is superfluous.**
```r
get_start_pos <- function(model, data, reps, sim, init_method, cores,
                                                                     ^
```
[R/initialization.R:18:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:21:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L21) *style:* **lines should not be more than 80 characters.**
```r
    start_pos <- do_zoom_in_search(model, data, reps, sim, cores, sim_cache, block_width)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:21:18:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L21) *warning:* **no visible global function definition for ‘do_zoom_in_search’**
```r
    start_pos <- do_zoom_in_search(model, data, reps, sim, cores, sim_cache, block_width)
                 ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:23:18:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L23) *warning:* **no visible global function definition for ‘do_initial_search’**
```r
    start_pos <- do_initial_search(model, data, reps, sim, cores, sim_cache)
                 ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:27:62:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L27) *style:* **Trailing whitespace is superfluous.**
```r
    start_pos <- matrix(runif(reps * model$get_par_number()),
                                                             ^
```
[R/initialization.R:32:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L32) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:33:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L33) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.matrix(start_pos))
  ^~~~~~~~~~~
```
[R/initialization.R:34:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L34) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(all((dim(start_pos) == c(reps, model$get_par_number()))))
  ^~~~~~~~~~~
```
[R/initialization.R:34:19:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L34) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  assert_that(all((dim(start_pos) == c(reps, model$get_par_number()))))
                  ^
```
[R/initialization.R:40:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L40) *style:* **lines should not be more than 80 characters.**
```r
  "determines starting postions by cutting the parameters in equally sized blocks
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:44:21:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L44) *warning:* **no visible global function definition for ‘determine_bpp’**
```r
  blocks_per_par <- determine_bpp(par_number, reps)
                    ^~~~~~~~~~~~~
```
[R/initialization.R:45:13:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L45) *warning:* **no visible global function definition for ‘create_initial_blocks’**
```r
  blocks <- create_initial_blocks(model$get_par_ranges(), blocks_per_par)
            ^~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:46:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L46) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:48:62:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L48) *style:* **Trailing whitespace is superfluous.**
```r
  estimates <- lapply(blocks, estimate_local_ml, model, data,
                                                             ^
```
[R/initialization.R:54:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L54) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:56:76:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L56) *style:* **Trailing whitespace is superfluous.**
```r
  best_indexes <- order(vapply(estimates, function(x) x$value, numeric(1)),
                                                                           ^
```
[R/initialization.R:58:55:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L58) *style:* **Trailing whitespace is superfluous.**
```r
  t(vapply(estimates[best_indexes], function(x) x$par,
                                                      ^
```
[R/initialization.R:80:17:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L80) *warning:* **no visible global function definition for ‘create_block’**
```r
    return(list(create_block(basic_block)))
                ^~~~~~~~~~~~
```
[R/initialization.R:82:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L82) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:90:45:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L90) *warning:* **no visible global function definition for ‘create_block’**
```r
      blocks[[(j - 1) * par_number + i]] <- create_block(new_block)
                                            ^~~~~~~~~~~~
```
[R/initialization.R:93:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L93) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:98:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L98) *style:* **lines should not be more than 80 characters.**
```r
do_zoom_in_search <- function(model, data, reps, sim, cores, sim_cache, block_width,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:98:85:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L98) *style:* **Trailing whitespace is superfluous.**
```r
do_zoom_in_search <- function(model, data, reps, sim, cores, sim_cache, block_width,
                                                                                ^
```
[R/initialization.R:100:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L100) *style:* **lines should not be more than 80 characters.**
```r
  "Starts with estimating parameters in the complete parameter space, an then iteratively
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:100:90:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L100) *style:* **Trailing whitespace is superfluous.**
```r
  "Starts with estimating parameters in the complete parameter space, an then iteratively
                                                                                ^
```
[R/initialization.R:109:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L109) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/initialization.R:111:17:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L111) *warning:* **no visible global function definition for ‘estimate_local_ml’**
```r
      middle <- estimate_local_ml(block, model, data, sim, cores, sim_cache)$par
                ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:112:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L112) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/initialization.R:117:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/initialization.R#L117) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  }, numeric(model$get_par_number())))
  ^
```
[R/jaatha_data.R:2:44:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_data.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
jaatha_data_class <- R6Class("jaatha_data",
                                           ^
```
[R/jaatha_data.R:16:57:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_data.R#L16) *style:* **Trailing whitespace is superfluous.**
```r
          stop("Data has missing values for statistic ",
                                                        ^
```
[R/jaatha_data.R:26:9:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_data.R#L26) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
        }, numeric(1))
        ^
```
[R/jaatha_data.R:84:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_data.R#L84) *warning:* **no visible binding for global variable ‘jaatha_data_class’**
```r
  jaatha_data_class$new(data, model)
  ^~~~~~~~~~~~~~~~~
```
[R/jaatha_data.R:90:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_data.R#L90) *warning:* **no visible global function definition for ‘create_jaatha_data’**
```r
  create_jaatha_data(test_data, model)
  ^~~~~~~~~~~~~~~~~~
```
[R/jaatha_log.R:1:42:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L1) *style:* **Trailing whitespace is superfluous.**
```r
jaatha_log_class <- R6Class("jaatha_log",
                                         ^
```
[R/jaatha_log.R:18:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_log.R:26:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L26) *style:* **lines should not be more than 80 characters.**
```r
      private$final_estimates <- private$estimates[[1]][rep(1, 5 * par_number), ]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/jaatha_log.R:44:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_log.R:47:76:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L47) *style:* **Trailing whitespace is superfluous.**
```r
        message(" Step ", step, ": Loglikelihood ", format(estimate$value),
                                                                           ^
```
[R/jaatha_log.R:50:66:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L50) *style:* **Trailing whitespace is superfluous.**
```r
        message(" Parameter: ", private$format_par(estimate$par),
                                                                 ^
```
[R/jaatha_log.R:51:53:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L51) *style:* **Trailing whitespace is superfluous.**
```r
                ", Loglikelihood ", format(old_llh),
                                                    ^
```
[R/jaatha_log.R:57:36:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L57) *style:* **Trailing whitespace is superfluous.**
```r
        message("Repetition ", rep,
                                   ^
```
[R/jaatha_log.R:63:31:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L63) *style:* **Use <-, not =, for assignment.**
```r
      private$converged[step] = TRUE
                              ^
```
[R/jaatha_log.R:67:60:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L67) *style:* **Trailing whitespace is superfluous.**
```r
      message("Determining starting positions using the '",
                                                           ^
```
[R/jaatha_log.R:88:45:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_log.R#L88) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      param <- as.numeric(best_estimate[1, -(1:3)])
                                            ^
```
[R/jaatha_model.R:4:46:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L4) *style:* **Trailing whitespace is superfluous.**
```r
jaatha_model_class <- R6Class("jaatha_model",
                                             ^
```
[R/jaatha_model.R:13:71:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L13) *style:* **Trailing whitespace is superfluous.**
```r
        stop("There already is a summary statistic with name '", name,
                                                                      ^
```
[R/jaatha_model.R:21:59:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L21) *style:* **Trailing whitespace is superfluous.**
```r
    initialize = function(sim_func, par_ranges, sum_stats,
                                                          ^
```
[R/jaatha_model.R:43:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L43) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_model.R:46:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L46) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_model.R:51:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L51) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:55:9:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L55) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
        }, error = function(e) {
        ^
```
[R/jaatha_model.R:56:22:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L56) *style:* **Use <-, not =, for assignment.**
```r
          error_dump = tempfile("jaatha_frame_dump_", fileext = ".Rda")
                     ^
```
[R/jaatha_model.R:59:76:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L59) *style:* **Trailing whitespace is superfluous.**
```r
          stop(paste(e$message, "[Frame dump written to", error_dump, "]"),
                                                                           ^
```
[R/jaatha_model.R:62:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L62) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:63:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L63) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:68:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L68) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:72:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:74:7:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L74) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
      }, mc.preschedule = !private$slow_sim, mc.cores = cores)
      ^
```
[R/jaatha_model.R:75:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_model.R:83:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L83) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:93:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L93) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_model.R:97:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L97) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:100:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L100) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~~~
```
[R/jaatha_model.R:103:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L103) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_model.R:134:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L134) *style:* **lines should not be more than 80 characters.**
```r
create_jaatha_model.default <- function(x, ..., scaling_factor = 1, test = TRUE) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/jaatha_model.R:167:61:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L167) *style:* **Trailing whitespace is superfluous.**
```r
                                         scaling_factor = 1,
                                                            ^
```
[R/jaatha_model.R:169:51:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha_model.R#L169) *style:* **Trailing whitespace is superfluous.**
```r
  jaatha_model_class$new(x, par_ranges, sum_stats,
                                                  ^
```
[R/jaatha.R:64:32:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L64) *style:* **Trailing whitespace is superfluous.**
```r
jaatha <- function(model, data,
                               ^
```
[R/jaatha.R:65:36:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L65) *style:* **Trailing whitespace is superfluous.**
```r
                   repetitions = 3,
                                   ^
```
[R/jaatha.R:66:54:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L66) *style:* **Trailing whitespace is superfluous.**
```r
                   sim = model$get_par_number() * 25,
                                                     ^
```
[R/jaatha.R:67:36:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L67) *style:* **Trailing whitespace is superfluous.**
```r
                   max_steps = 100,
                                   ^
```
[R/jaatha.R:68:64:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L68) *style:* **Trailing whitespace is superfluous.**
```r
                   init_method = c("zoom-in", "initial-search",
                                                               ^
```
[R/jaatha.R:74:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L74) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:76:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L76) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_model(model))
  ^~~~~~~~~~~
```
[R/jaatha.R:76:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L76) *warning:* **no visible global function definition for ‘is_jaatha_model’**
```r
  assert_that(is_jaatha_model(model))
              ^~~~~~~~~~~~~~~
```
[R/jaatha.R:77:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L77) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/jaatha.R:77:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L77) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/jaatha.R:78:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L78) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(repetitions))
  ^~~~~~~~~~~
```
[R/jaatha.R:78:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L78) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(repetitions))
              ^~~~~~~~
```
[R/jaatha.R:79:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L79) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(sim))
  ^~~~~~~~~~~
```
[R/jaatha.R:79:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L79) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(sim))
              ^~~~~~~~
```
[R/jaatha.R:80:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L80) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(cores))
  ^~~~~~~~~~~
```
[R/jaatha.R:80:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L80) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(cores))
              ^~~~~~~~
```
[R/jaatha.R:81:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L81) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(block_width) && length(block_width) == 1)
  ^~~~~~~~~~~
```
[R/jaatha.R:82:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L82) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(block_width > 0 && block_width < 1)
  ^~~~~~~~~~~
```
[R/jaatha.R:83:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L83) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:85:10:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L85) *warning:* **no visible global function definition for ‘create_jaatha_log’**
```r
  log <- create_jaatha_log(model, data, repetitions, max_steps, verbose = TRUE)
         ^~~~~~~~~~~~~~~~~
```
[R/jaatha.R:87:16:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L87) *warning:* **no visible global function definition for ‘create_sim_cache’**
```r
  sim_cache <- create_sim_cache(sim_cache_limit)
               ^~~~~~~~~~~~~~~~
```
[R/jaatha.R:89:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L89) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:94:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L94) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:100:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L100) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/jaatha.R:103:64:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L103) *style:* **Trailing whitespace is superfluous.**
```r
                                  estimate + block_width * .5),
                                                               ^
```
[R/jaatha.R:105:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L105) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha.R:106:19:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L106) *warning:* **no visible global function definition for ‘estimate_local_ml’**
```r
      local_ml <- estimate_local_ml(block, model, data, sim, cores, sim_cache)
                  ^~~~~~~~~~~~~~~~~
```
[R/jaatha.R:113:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L113) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha.R:118:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L118) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha.R:125:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L125) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:131:65:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L131) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    llh <- estimate_llh(model, data, as.numeric(best_values[i, -(1:3)]),
                                                                ^
```
[R/jaatha.R:131:73:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L131) *style:* **Trailing whitespace is superfluous.**
```r
    llh <- estimate_llh(model, data, as.numeric(best_values[i, -(1:3)]),
                                                                        ^
```
[R/jaatha.R:135:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/jaatha.R#L135) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:1:52:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L1) *style:* **Trailing whitespace is superfluous.**
```r
calc_poisson_llh <- function(data, stat, loglambda,
                                                   ^
```
[R/likelihood.R:3:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L3) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:7:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L7) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:10:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L10) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:11:49:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
  llh <- sum(data$get_values(stat) * loglambda -
                                                ^
```
[R/likelihood.R:13:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:14:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L14) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.finite(llh))
  ^~~~~~~~~~~
```
[R/likelihood.R:25:75:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L25) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
approximate_llh.default <- function(x, data, param, glm_fitted, sim, ...) {
                                                                          ^
```
[R/likelihood.R:25:76:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
approximate_llh.default <- function(x, data, param, glm_fitted, sim, ...) {
                                                                           ^
```
[R/likelihood.R:32:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L32) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/likelihood.R:32:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L32) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/likelihood.R:33:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L33) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(param))
  ^~~~~~~~~~~
```
[R/likelihood.R:34:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L34) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.list(glm_fitted))
  ^~~~~~~~~~~
```
[R/likelihood.R:42:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L42) *style:* **Variable and function names should not be longer than 30 characters.**
```r
approximate_llh.jaatha_stat_basic  <- function(x, data, param, glm_fitted,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/likelihood.R:42:75:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L42) *style:* **Trailing whitespace is superfluous.**
```r
approximate_llh.jaatha_stat_basic  <- function(x, data, param, glm_fitted,
                                                                          ^
```
[R/likelihood.R:44:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:48:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L48) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:50:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L50) *warning:* **no visible global function definition for ‘calc_poisson_llh’**
```r
  calc_poisson_llh(data, x, loglambda, sim, scaling_factor)
  ^~~~~~~~~~~~~~~~
```
[R/likelihood.R:59:25:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L59) *warning:* **no visible global function definition for ‘approximate_llh’**
```r
                        approximate_llh(model, data, param, glms, sim)
                        ^~~~~~~~~~~~~~~
```
[R/likelihood.R:61:41:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L61) *style:* **Commas should never have a space before.**
```r
                      lower = boundary[ , 1, drop = FALSE],
                                       ~^
```
[R/likelihood.R:61:60:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L61) *style:* **Trailing whitespace is superfluous.**
```r
                      lower = boundary[ , 1, drop = FALSE],
                                                           ^
```
[R/likelihood.R:62:41:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L62) *style:* **Commas should never have a space before.**
```r
                      upper = boundary[ , 2, drop = FALSE],
                                       ~^
```
[R/likelihood.R:63:43:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L63) *style:* **Trailing whitespace is superfluous.**
```r
                      method = "L-BFGS-B",
                                          ^
```
[R/likelihood.R:65:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L65) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:66:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L66) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(block$includes(best_value$par))
  ^~~~~~~~~~~
```
[R/likelihood.R:73:68:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L73) *style:* **Trailing whitespace is superfluous.**
```r
    sim_data <- model$simulate(pars = block$sample_pars(sim, TRUE),
                                                                   ^
```
[R/likelihood.R:75:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:79:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L79) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(sim_data) >= sim)
    ^~~~~~~~~~~
```
[R/likelihood.R:80:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L80) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/likelihood.R:82:22:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L82) *warning:* **no visible global function definition for ‘fit_glm’**
```r
    glms <- tryCatch(fit_glm(model, sim_data), error = identity)
                     ^~~~~~~
```
[R/likelihood.R:83:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L83) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:87:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L87) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/likelihood.R:91:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L91) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:92:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L92) *warning:* **no visible global function definition for ‘optimize_llh’**
```r
  optimize_llh(block, model, data, glms, length(sim_data))
  ^~~~~~~~~~~~
```
[R/likelihood.R:115:60:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L115) *style:* **Trailing whitespace is superfluous.**
```r
estimate_llh <- function(model, data, parameter, sim = 100,
                                                           ^
```
[R/likelihood.R:117:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L117) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:118:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L118) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_model(model))
  ^~~~~~~~~~~
```
[R/likelihood.R:118:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L118) *warning:* **no visible global function definition for ‘is_jaatha_model’**
```r
  assert_that(is_jaatha_model(model))
              ^~~~~~~~~~~~~~~
```
[R/likelihood.R:119:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L119) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_jaatha_data(data))
  ^~~~~~~~~~~
```
[R/likelihood.R:119:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L119) *warning:* **no visible global function definition for ‘is_jaatha_data’**
```r
  assert_that(is_jaatha_data(data))
              ^~~~~~~~~~~~~~
```
[R/likelihood.R:120:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L120) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.numeric(parameter))
  ^~~~~~~~~~~
```
[R/likelihood.R:121:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L121) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(sim))
  ^~~~~~~~~~~
```
[R/likelihood.R:121:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L121) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(sim))
              ^~~~~~~~
```
[R/likelihood.R:122:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L122) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.count(cores))
  ^~~~~~~~~~~
```
[R/likelihood.R:122:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L122) *warning:* **no visible global function definition for ‘is.count’**
```r
  assert_that(is.count(cores))
              ^~~~~~~~
```
[R/likelihood.R:123:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L123) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is_single_logical(normalized))
  ^~~~~~~~~~~
```
[R/likelihood.R:123:15:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L123) *warning:* **no visible global function definition for ‘is_single_logical’**
```r
  assert_that(is_single_logical(normalized))
              ^~~~~~~~~~~~~~~~~
```
[R/likelihood.R:124:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L124) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:135:5:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L135) *warning:* **no visible global function definition for ‘calc_poisson_llh’**
```r
    calc_poisson_llh(data, stat, log_means, sim, model$get_scaling_factor())
    ^~~~~~~~~~~~~~~~
```
[R/likelihood.R:136:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L136) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  }, numeric(1)))
  ^
```
[R/likelihood.R:137:1:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L137) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:138:3:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/likelihood.R#L138) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.finite(llh))
  ^~~~~~~~~~~
```
[R/par_ranges.R:2:42:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
par_ranges_class <- R6Class("par_ranges",
                                         ^
```
[R/par_ranges.R:13:32:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L13) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      assert_that(all(par_range[ , 1] < par_range[ , 2]))
                               ^
```
[R/par_ranges.R:14:34:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L14) *style:* **Commas should never have a space before.**
```r
      assert_that(all(par_range[ , 1] < par_range[ , 2]))
                                ~^
```
[R/par_ranges.R:14:52:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L14) *style:* **Commas should never have a space before.**
```r
      assert_that(all(par_range[ , 1] < par_range[ , 2]))
                                                  ~^
```
[R/par_ranges.R:21:53:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L21) *style:* **Commas should never have a space before.**
```r
      private$log_range_width <- private$log_range[ , 2] -
                                                   ~^
```
[R/par_ranges.R:21:59:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L21) *style:* **Trailing whitespace is superfluous.**
```r
      private$log_range_width <- private$log_range[ , 2] -
                                                          ^
```
[R/par_ranges.R:22:55:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L22) *style:* **Commas should never have a space before.**
```r
                                   private$log_range[ , 1]
                                                     ~^
```
[R/par_ranges.R:26:39:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L26) *style:* **Commas should never have a space before.**
```r
      (log_value - private$log_range[ , 1]) / private$log_range_width
                                     ~^
```
[R/par_ranges.R:29:64:](https://github.com/statgenlmu/jaatha/blob/a18f63d112a3eda92e45a06df9fb2922d9bbcbd4/R/par_ranges.R#L29) *style:* **Commas should never have a space before.**
```r
      exp(value * private$log_range_width + private$log_range[ , 1]) +
                                                              ~^
```


TeamCity Civic Assembly :: Build all [Build 22](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=401&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1; exit code 123 Build time: 00:00:32


I changed it to LDT. Though i don't think it matters much in this special case, the expressiveness of java.time.LocalDateTime#now() convinced me :)


Damn, my browser cache tricked me, fixed. Thanks


Instead of doing additional copies here, can't we use the same trick we use for the doc comment freeing? I.e. check whether const->ce == ce of the class?


Great addition, thanks!


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


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


Example YAML:
```yaml
location: byon3
services:
- type: org.apache.brooklyn.cm.salt.SaltEntity
  name: myent
  runList:
    - apache
  formulas:
    apache: https://github.com/saltstack-formulas/apache-formula/archive/master.tar.gz
```


vissible => visible


ich würds mit _class machen


Sign-off, please?


fair enough, removed.


"but there is another way to empty an array that is more performant"


I know that.  You know that.  The original writer went for it anyways....
(removed)


> @den99f is it a bug?

Remove duplication dumping


"Let's say you define an array and want to empty its contents"


Okay, sorry for merging... Feel free to work on this branch/pr or create a new one. (I won't merge that one :) )


"Usually, you would do it like this"


Looks good. Made a few comments. Great to have some tests :)
Some notes:
- We should consider doing something about indentation (mixed tabs and spaces right now).
- We should have tests for changing viewModel and disposal of subscriptions. We can do that later.


Hi @herby! Good point on this.

My main purpose on system-notification is to create high-level API for sending and scheduling local notifications on Android (or on iOS also in the future). And by Android design, GCM and notifications are two separated things in the system - so that leads to this architecture here.

But you're right about the complexity to users caused by installing two modules, and seems that most users will need push-notifications by common. We will discuss the details on merging these two :smile:


nit addressed.


I really thought I'd got the config email set right this time.


just as heads up, this also needs to go into mon/MonCommands.h's entry for 'osd set'.


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


This is an interesting test code of your import style guide. `... as Binary` doesn't work since that name is already imported, but I'd probably have imported it `... as Bin` or similar so as to get shorter function calls below. But it is (of course) OK as it is.


which means that references to the contents of the previous array are still kept in memory, leading to memory leaks.


However, if you have a copy of the array (A and Copy-A), if you delete its contents using list.length = 0, the copy will also lose its contents.


and?


@benrudolph not totally sure I'm understanding how your suggestion works. should the `item' property in `TableConfiguration` be a be a list of `ExportItem`s?

if so then, fwiw, this looks a lot like what the original code does. is it possible that the root problem is just the way the current code defines schemas, but the actual saved configs have a reasonable structure? not sure if reusing/leveraging bits of that will make anything easier


Hi Benjamin,

On Wed, Jan 20, 2016 at 3:07 AM, Benjamin Vernoux <notifications@github.com>
wrote:

> There is an other problem with actual firmware (v1.0.0-rc6) kernel driver
> it does not manage new commands
> see
> https://github.com/airspy/firmware/blob/master/common/airspy_commands.h
> actual kernel driver manage only up to CMD_SET_PACKING = 0x13, which is
> wrong as in new firmware
> 0x13 corresponds to AIRSPY_MS_VENDOR_CMD
>
I think firmware version can be seen by CMD_VERSION_STRING_READ,
so then if it is an old version then we can use the existing commands and
if it is the new firmware then we can use the new commands.

regards
sudip


:+1: that seems clearer to me (also what UCR does)


потому что эта штука в page controller'e отображается. там у него свой контент еще снизу.


I guess you'd still probably want to set `copy selection` and `pane reset mode` as the default behaviors for `MOUSEUP1_PANE` somewhere?

But yeah this makes sense! It seems like what I was hoping for. I'll pull this down and test it out tomorrow!


Minor: space is missing


@andyjmaclean yep, sure. I'll update it. (after conversations about functionality this morning this pattern will need to change anyway but i'll leave it in PL for future use)


> This is an interesting test code of your import style guide. `... as Binary` doesn't work since that name is already imported, but I'd probably have imported it `... as Bin` or similar so as to get shorter function calls below. But it is (of course) OK as it is.

(-:

I guess from three references on it gets tedious, but IIRC I only use it twice, so I'd like to keep it as is.


yes. I committed.


Done.

(adds note for future changes/PR)


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


If in a serial run writing to a previously not-existing dir still works then you did not break anything, I think.


@danmar This change completely removes the check equivalent to clang `-Wchar-subscripts` warning. That warning says the code is unclear and unportable because just `char` can be signed or unsigned depending on how code is compiled. Was that the intent?


hello


Awesome! Could it be possible to loop over to the main menu for asking the player "to continue or to quit" once all the lives are over.

I mean, to display the same menu image, but looped when all the lives are lost. This would prevent from abruptly closing the game window.

What do you think @bardlean86


is this ok?
1. when event is {stop, closed}
    change state from ready to connecting and process other events when state is connecting
2. other errors
   stop current gen_fsm, send msgs to ekaf_server (for the reason that gen_fsm do not know which partition startegy to use, it's better to send msgs to ekaf_server). this situation should deserialize the msgs.


?dividerHorizontal


нет


@DarknessVoided  still waiting for ButterAIO doge


[HelpdeskTemplatewithWorkflow.pdf](https://github.com/srirams007/PnP-Partner-Pack/files/98963/HelpdeskTemplatewithWorkflow.pdf)

Added the statemachine workflow. The attached file contains the screens of complete helpdesk template functionality.


没听懂。。。


nice job


```cpp
case TurnDirection::NoTurn:
      imageName = isNextTurn ? nil : @"straight";
      break;
```


Oh yeah ._. So busy with school nowadays ...


Лучше не сортировать


HAPPY NOW RHYS?!?!?


TeamCity pull requests :: netty [Build 1078](https://garage.netty.io/teamcity/viewLog.html?buildId=4944&buildTypeId=pulls_netty&guest=1) is now running


Not sure that we need it for user builds.


you can omit the return in this function.


Oh, hope you're doing good fam, get them A's <3 have fun and educate :)


 .page-item {
      &:first-child {
        .page-link {
 -        @include border-left-radius($border-radius);
 +        @include border-radius($border-radius,left);
        }
      }
      &:last-child {
        .page-link {
 -        @include border-right-radius($border-radius);
 +        @include border-radius($border-radius,right);
        }
      }
    }

ddddd


Sure I will


@rauchg The date of the release (2015-11-28) is apparently wrong.


[cheatsheet](https://github.com/NchingTW/cheatsheets.git)
[Group](https://github.com/NchingTW/biom262-hw.git)


you can skip the return (as mentioned above). This is applicable to all methods similar to those.


The problem persist in the list ticket.
The number of my ticket is correct but the ticket viewed in my ticket isn's still correct


Makes sense:)


This is not enough. If config.allowFreeText and there is no text, then the emptyTemplate should be displayed.


Вот ваще зря отсортировано


Please do not use underscore notation for "private" fields.


нафиг сортировку


first commit


remove the extra space


Это согласовано с Игорем?


I see thanks. Didn't check the issues.
On Jan 21, 2016 09:52, "Chris Rebert" <notifications@github.com> wrote:

> @XhmikosR <https://github.com/XhmikosR> Breaks the test suite; see #18890
> (comment)
> <https://github.com/twbs/bootstrap/issues/18890#issuecomment-171878944>
> et seq.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twbs/bootstrap/commit/cbeeb1ce080b8cfb41ffb7214906a758017e7ff1#commitcomment-15579099>
> .
>


lelo


@absimaldata
>>Why this condition is there? score can be negative or maybe positive?

Can be positive or negative, it does not matter, at the rootnode we always have to maximize
as it lies in the principle of any Minimax or NegaMax algorythm


> Make that "it uses the Win32 API to create windows and draws
on them with GDI", and I'd be with you.  (Might as well be
specific about _which_ API it's using.)

Sounds perfect. =)


Oops, this needs to go to the top, so it covers the case of a discarded byte at L208.


OK.


Нет, парент должен быть MwmTheme.Night


Array is reset below, can probably remove these two :)


Why can't _users.subList(from, to)_ be used here?


This comment is useless. You don't need to document good practice. ;)


И здесь


I've never had to do anything besides [refresh the index](https://github.com/dimagi/commcare-hq/blob/master/corehq/ex-submodules/pillowtop/tests/test_elasticsearch.py#L99)


Would it perhaps be better to do

```python
lnk_dir_template = os.path.join(os.sep, 'home', '{user}', '.kdesktop', 'Powerup.lnk')
for user in linux_users:
    lnk_dir = lnk_dir_template.format(user=user)
```


I was redirected to here from this link, found it via google: https://travis-ci.org/firegento/firegento-magesetup/jobs/75237184#L134

I'm having the same issue when trying to update a module via composer. I've edited the composer.json and ran `composer update module/name --ignore-platform-reqs` and got the same error:

`PHP Parse error:  syntax error, unexpected '[' in vendor/magento-hackathon/magento-composer-installer/src/MagentoHackathon/Composer/Magento/Plugin.php on line 113`

Any hints on how to fix this or what's causing it?


TeamCity Civic Assembly :: Build backend + frontend [Build 16](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=402&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 69, passed: 1598, ignored: 7 Build time: 00:17:50
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

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


if (XXXX) { while (XXXX) {...} should be written as while(XXXX) {....}
not as if (XXXX) { do .... while (XXXX)


安卓版软件？


Please mark as `inline` or move body to `.cpp` file. Comment above misses a `.` at the end of the sentence.


:dart:


Hi @clintongormley,

I thought I should brief you on what happened here. @dakrone found that the bot was unhappy, presumably due to changes in the search wrapper API.

The whole [tale-o-woe is immortalised in #infra](https://elastic.slack.com/archives/infra/p1453413542004202).

This is my evil hack to get some search results back, but it's obviously flawed. (Hard-coded ES version for one thing).


Sorry for delay in looking at this, I had a rough deadline to meet. We treat Bool() as UInt(1), so the vectorized version is doing the right thing, and the unvectorized version isn't. The final Halide code includes the right casts, and so does the llvm bitcode (cast from float to i1, then zero extend to i8 to store it), but the assembly is wrong (just casts from float to i8 directly)... *shakes fist at llvm*.

Ah! Rereading the llvm reference, it turns out that even though the underlying asm ops have the semantics we want, casting from float to an unsigned int has undefined behavior on overflow, so llvm makes optimizations that assumes it can't overflow. In this case that means the int must be 0 or 1, so the zero extend is unnecessary.

We'll have to do something different when we cast from float to int. I'll think about it.


I'll upload the files to the mirror in a little bit, was sick yesterday so didn't get to.


FYI driverNames is a slice and uploadMachineSchema() accepts a slice. You don't need the `[:]`.
..I think I've seen you do this in a few places.


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


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


Totally stealing this.


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


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


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


Closes #93


efc2e74 (on top of a rebase, sorry about that)


LGTM hahah


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


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


`_ Sample = (*Rejectioner)(nil)`


This class is not necessary, you can delete it


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 99](https://teamcity.labkey.org/viewLog.html?buildId=298562&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (2 new), passed: 275, muted: 1 Build time: 01:53:41
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

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

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


On Jan 21, 2016, at 3:56 PM, Lydia Duncan <notifications@github.com<mailto:notifications@github.com>> wrote:

Ah, there we are. We can get a ModuleSymbol passed to us by BlockStmt::moduleUseAdd, which gets called in a number of places in the compiler. It seemed simpler at the time to just do the SymExpr wrapper around the module in the constructor rather than insert that wrapping at all the places where moduleUseAdd was called on a plain module symbol. I'm willing to bite the bullet and do it if you don't think that laziness is justified.

I don’t think it’s a question of laziness.  I think it’s a question of API design.  When one glances at the
constructor one observes that it appears to accept any BaseAST.  Some analysis shows that it actually
accepts a UnresolvedSymExpr, SymExpr, CallExpr, and (I think) a ModuleSymbol.  One could claim that
it’s unfortunate that one needs to work reasonably hard to figure that out.

Mike

—
Reply to this email directly or view it on GitHub<https://github.com/lydia-duncan/chapel/commit/7f9e6aea7ebd4e0b7e5cf643d09c5ccbaf32b9e0#commitcomment-15601201>.


Yep. It should.


This constructor should assign first parameter to nameAuthor and second to message


I agree with using a shorter name because of the small scope of the variable, but isn't the new name misleading? Perhaps just ```rooms``` would be best.


3266254 brings them out, for clarity's sake.


Instead of string and string2, you should use meaningful names, for example name and msg


You can delete this class, because it doesn't covert the exercise


You should use a newer version, for example 4.12 version


Can you just add a comment as to why we do this for future context


:+1: I see the scope of the variable is larger and you were matching the external name. It would have been clearer without the last commit being separate.


Vou fazer a modificação..


H5py will not configure for me either (Ubuntu 15.10)
I've installed python-h5py but it doesn't find it so builds its own copy,
but then that fails too with ImportError (when I add a line to print the
exception: "cannot import name _errors")

On 19 January 2016 at 15:04, RebeccaFarrington <notifications@github.com>
wrote:

> no worries :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/underworldcode/underworld2/commit/2aa51432318a306b05a8a7d9a409ef69ea49508e#commitcomment-15521265>
> .
>

--
*OWEN KALUZA*
Senior Visualisation Specialist

*Immersive Visualisation Platform (eResearch)*
Monash University
Room 337-25, 20 Research Way, Clayton Campus

T: +61 3 9902 9809
M: +61 410 545 824
E: owen.kaluza@monash.edu
monash.edu/mivp


Actually, it does not make too much sense to make these configurable - or at least not per programmer. I am not aware of any other programmers where changing these options is supported - and they are not needed for any flash chips flashrom supports yet.

Any specific reason you are implementing them?


It's not a big problem because, after a fetchnodes, node_updated is called with NULL meaning that all nodes have been just received, and just after that callback the `changed` struct is set to 0. The only adverse effect would be that apps could see flags set if they request them in `fetchnodes_result`, or `nodes_updated` (when the `nodes` parameter is NULL), but it doesn't make much sense because the app already knows in that moment that all nodes have been just received.

Anyway, I agree that it's better to not set the flag in that case, but I would manage it in a centralized way in `notifynode`, instead of individually manage it for each case. In this case `procph` is only called in `CommandFetchNodes::procresult` but we may use it to process action packets in the future.

Additionally, action packets are already being used in the context of a `fetchnodes` for huge accounts when the server caches the response, so setting or not the flag depending if we are processing an action packet or not, is not a solution for this issue.

A management inside `notifynode` depending on the value of the flag `MegaClient::fetchingnodes` seems more appropriate to me. But it's not a problem specific to this pull request so we should take care of it in a different one.


This should happen in the transaction instead (inside `dao.changeInvoiceStatus`).


Just in case: it is easy to merge RAW and AxCrypt (see 01e053b972a8ed73c141356489333eb3a83f9ff3).

- I see no drawback or benefit.


I must say I'm not much of a fan of abbreviated flag names. Why `SPF_RELPOS` instead of `SPF_RELATIVEPOSITION`? No other codepointer, to my knowledge, does this kind of abbreviation.


These were some issues I spot on map c1m5...
![c1m5 3](https://cloud.githubusercontent.com/assets/11136108/12498675/9386e342-c0a4-11e5-974c-a94d45774acf.jpg)
...where many of those toilets were wrongly positioned, and on map c1m0...
![c1m0 1](https://cloud.githubusercontent.com/assets/11136108/12498685/b24f0804-c0a4-11e5-8b74-4934d8727d51.jpg)
![c1m0 2](https://cloud.githubusercontent.com/assets/11136108/12498686/b250de36-c0a4-11e5-8272-97001feb5633.jpg)
![c1m0 3](https://cloud.githubusercontent.com/assets/11136108/12498687/b254d8ce-c0a4-11e5-9c34-ccad574482db.jpg)
![c1m0 4](https://cloud.githubusercontent.com/assets/11136108/12498688/b255c69e-c0a4-11e5-8baa-5a452bd37cb7.jpg)
![c1m0 5](https://cloud.githubusercontent.com/assets/11136108/12498689/b25d3ffa-c0a4-11e5-9977-f2230f14ffec.jpg)
...where I discovered that RTCWPhone and Plant1/2 actors had their radius too much large + trees surroundings that building where too much near it.

Map c1m6 have the same problem with trees at its very start (bridge).


Well, the navigator seems very detailed, but the problem is that the buttons are large and it takes more then one click to get to things like Target selector.
What about implanting the clickgui in the navigator? The clickgui could be the "basic" view, and you could easily switch back and forth between the "advanced" view (navigator main screen) and the "basic" one. This would make it easier for people that are used to a clickgui.


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


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


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


it doesn't do a dam thing if you ask me. I'm sure someone will prove me wrong though.


> LLVM?

Just pushed a Clang compatibility commit, it's basically just using the GCC options.


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


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


If I understand correctly, it should be Math.max instead of Math.min.


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


Will equities/forex data be included in these files?


It's the standard 2-clause BSD license. It's just badly formatted.


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (968 of 968 tests passed, 0 test warnings)

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


That answers my question.  I think it'd be nice to have that in a comment in the code here though.


These all are global configurations, and allowed to be changed by users. Most of the config can be overwritten in instances.


Actually, this test is not correct. It does not have a version field below the baseclass.


I'm not sure if this fix broke things or if it is a coincidence but I now get errors when the SMIL data is retrieved. I will try a previous version and see if that fixes it.


I think it's OK for this to give an undefined value, but it should be limited to the range of the target type at least, or bounds inference breaks and we might read out of bounds. This is only a problem for i1s because they're stored in i8s. So I'll do all float->i1 casts via i8. This generates the correct masking in the asm, because the i8->i1 conversion is well defined.


Cool ! Is it already uploaded to pypi ? It doesn't seem so. How can I know every time you update the pypi package ?

(ps: I'm the guy from reddit^^)


Lose one blank line here.


this line isn't needed, since `nvm install` runs `use` for you.


> command has some cost to pay

That makes sense. I'll fix it.


Combine this on one line.


Maybe, but why is this true for VersionedTest_Subclass? It differs only in the fact that the Versioned Extension is reapplied. From my understanding there should be no difference in behaviour between VersionedTest_Subclass and VersionedTest_AnotherSubclass, or am I wrong?


assert(n > 0);
assert(n < 64);


Addressed in 7e1059191dce18e87c21ce5bd36cddf546d6dbd0 :+1:


> The instruction set being 32 or 64 bits is not what's important, it's which vector extensions are available.

I do agree with you, but as already stated I just reused someone else's target architecture detection script. The target architecture is important though, because there is no SSE compiler support detection but user options. I use this only to prevent displaying and using the `BASE64_WITH_SSSE3` and `BASE64_WITH_AVX2` switches see `CMakeLists.txt:26`.


Since you already have a generator expression you might as well do the filtering in there: `set(p for p, _ in updates if has_roles(p, [BATCH_NORM_POPULATION_MEAN]))`.


Had to tweak it to support Lucee's lowercase session cookies
https://github.com/blueriver/MuraCMS/commit/21e91713cc5abadb73d15de4955b1e61b0fd5019


Do we need to use intrinsics like `_rotr64` or is GCC and clang already smart enough to do the right thing? Can you run the numbers on 32-bit and 64-bit GCC and clang builds? I will do the Windows builds.


Yes, we decided to leave it like this because of `CAST` which varies on the return type


ive actually got a pull request in already to fix this


This (sharing memory between goroutines) is an anti-pattern in golang. It falls under the idea of "Do not communicate by sharing memory; instead, share memory by communicating." You can read this if you're interested: https://blog.golang.org/share-memory-by-communicating

But, the most proper solution involves some somewhat complex channel logic. Soo, I think  a resonable compromise would be to protect the errs = append(errs, err) with a mutex


LGTM


so i can longer not have a title?  and if i want two links with the same rel with different titles, i'm out of luck there too?  this is really weird and forceful.


Does this compare probabilites?


@ibuildthecloud any input on this ^^^
Think just protecting with a mutex is sufficient?


I guess this means, I don't have to answer the following anymore:
> What does this mean for cross-compiling? I understand the use-case for detecting the architecture of the current machine, but a library like this should be easy to build inside a cross-compile environment. Would it be possible to override the detected architecture when using these buildscripts?


I was unclear but I did mean for the name of this function to start with a verb. So please prefix with `gather` or `fetch` or `get` depending on how many characters it's worth :-)


New-Fan-Object (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.

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


Want to lowercase this function so that it isnt exported? UpdateDrivers should be the only exported func, right?


:-1: Props to you for going to the effort to document your thinking here!


In THIS current version we aren't using the ini. If you decide to switch it so it uses the ini that your choice. ItsYuka had his version using the ini. It's basically personal preference. Having all your statements in one place is nice in my opinion. Makes it quick to add more and don't have to search through files to find them.


No as this has no need for backwards comparability as it is new. Also this is an outdated commit so please comment on or #1995


I'll inline the first one


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


`++`和`--`也是代码


I changed the inner interface to `ImplementationDependency`


> This is called a "computed include" and is implementation-defined behaviour...

If this is a problem for you, we'll find a different solution to the problem that with CMake you usually don't do in source builds and thus the config.h won't be in the same directory as `codec_choose.c`. This was just the solution that required the least amount of change in order to keep your Makefile and my CMake script compatible.


How about `reusePort` to (hopefully) allow multiple instances to use the same interface?

http://stackoverflow.com/questions/14388706/socket-options-so-reuseaddr-and-so-reuseport-how-do-they-differ-do-they-mean-t/14388707#14388707

> SO_REUSEPORT is what most people would expect SO_REUSEADDR to be. Basically, SO_REUSEPORT allows you to bind an arbitrary number of sockets to exactly the same source address and port as long as all prior bound sockets also had SO_REUSEPORT set before they were bound.

(I'd open an issue, but they don't seem allowed on this projet.)


文中所有的代码是否需要统一格式？


> This commit does not take into account the stuff in  /test , which is built with its own Makefile. That directory would also need a CMake makeover for optimal cross-platform compatibility.

Indeed. If  I get bored enough this weekend I'll probably add your tests to the build system.


第二个`//`格式问题


also this means if i reuse a rel in two ways (something generic like x:parent) i get the same title each time?


简化编程逻辑，或简化处理逻辑会更清晰一点吧


主要是文中代码是否要统一作为代码格式


This would never get executed because the query would never return the results. However, it was still stupidly lazy coding, and shouldn't have been there to begin with. This has been fixed.


Thank you.


hahaha,my name is fys!!!!!!!!!!


Класс!
А почему такой маленький? Есть фотографии распечатанного результата? Не верится, что там всё хорошо видно.


closed #4


The `interceptors` argument is before `state` argument, becouse is mandatory and it is better then add other `null` when calling this method.


ah I had left it to keep it to implement keeping last build but delete all before it.


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1519 of 1519 tests passed, 0 test warnings)

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


@MCDad Sorry this took so long. I have pushed a fix to the issue you mentioned.


Yes, they should always be the same, maybe we can remove QuoteCurrencySymbol.
Same in the Forex class, there are Cash QuoteCurrency, string BaseCurrencySymbol, string QuoteCurrencySymbol.


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


"https://yourdomain.com"
Needs to be changed on line 12 to
"https://popcorntimeappletv.herokuapp.com/"


I wouldn't do this. If the bins argument changes (or the calculated number of bins changes because the source data changed), the size of the histogram will change. Here, I would rather update the relevant axes on `out`.


created new global function to get app domain


I guess I wrecked my newlines in this commit without realizing it :frown:


`#next_level_threshold` because it's an instance method.


Maybe something like this would be more informative:

Rails now generates weak ETags to properly handle conditional GET requests when the content encoding is changed by an intermediate web server like NGINX.


not sure if this is necessary?