[R/block.R:2:32:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/block.R#L45) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/block.R:49:58:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/block.R#L52) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/boot_jaatha.R:24:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/boot_jaatha.R#L28) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~~~
```
[R/boot_jaatha.R:41:76:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/boot_jaatha.R#L43) *style:* **Trailing whitespace is superfluous.**
```r

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

^~
```
[R/boot_jaatha.R:49:5:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/boot_jaatha.R#L51) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/boot_jaatha.R:52:39:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:45:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L48) *style:* **Trailing whitespace is superfluous.**
```r

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

^~
```
[R/coala_interface.R:53:38:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L60) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:61:64:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L69) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:70:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

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

^~~~
```
[R/coala_interface.R:80:11:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L90) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:93:14:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L95) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:99:20:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L102) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:106:49:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L109) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/coala_interface.R:119:4:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L128) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/coala_interface.R:177:26:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L183) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~
```
[R/coala_interface.R:199:20:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/coala_interface.R#L201) *style:* **Trailing whitespace is superfluous.**
```r

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

^~
```
[R/fit_glm.R:8:52:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/fit_glm.R#L21) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/fit_glm.R:24:21:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/fit_glm.R#L28) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/fit_glm.R:42:5:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/fit_glm.R#L59) *style:* **Commented code should be removed.**
```r
#   list(smooth=smooth_glm)
    ^~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:1:70:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L1) *style:* **Trailing whitespace is superfluous.**
```r
get_start_pos <- function(model, data, reps, sim, init_method, cores,
                                                                     ^
```
[R/initialization.R:3:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L3) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:6:18:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L6) *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)
                 ^~~~~~~~~~~~~~~~~
```
[R/initialization.R:8:18:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L8) *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:12:62:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
    start_pos <- matrix(runif(reps * model$get_par_number()),
                                                             ^
```
[R/initialization.R:17:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L17) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:18:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L18) *warning:* **no visible global function definition for ‘assert_that’**
```r
  assert_that(is.matrix(start_pos))
  ^~~~~~~~~~~
```
[R/initialization.R:19:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L19) *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:19:19:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L19) *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:27:21:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L27) *warning:* **no visible global function definition for ‘determine_bpp’**
```r
  blocks_per_par <- determine_bpp(par_number, reps)
                    ^~~~~~~~~~~~~
```
[R/initialization.R:28:13:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L28) *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:29:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L29) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:31:62:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
  estimates <- lapply(blocks, estimate_local_ml, model, data,
                                                             ^
```
[R/initialization.R:37:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L37) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/initialization.R:39:76:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L39) *style:* **Trailing whitespace is superfluous.**
```r
  best_indexes <- order(vapply(estimates, function(x) x$value, numeric(1)),
                                                                           ^
```
[R/initialization.R:41:55:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L41) *style:* **Trailing whitespace is superfluous.**
```r
  t(vapply(estimates[best_indexes], function(x) x$par,
                                                      ^
```
[R/initialization.R:62:17:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L62) *warning:* **no visible global function definition for ‘create_block’**
```r
    return(list(create_block(basic_block)))
                ^~~~~~~~~~~~
```
[R/initialization.R:64:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L64) *style:* **Trailing whitespace is superfluous.**
```r

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

^~
```
[R/initialization.R:86:17:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L86) *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:90:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/initialization.R#L90) *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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha_log.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_log.R:26:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha_log.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha_log.R:47:76:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha_model.R#L43) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~~~~~~~
```
[R/jaatha_model.R:55:9:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha_model.R#L62) *style:* **Trailing whitespace is superfluous.**
```r

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

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

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

^~~~~~~~
```
[R/jaatha_model.R:74:7:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha_model.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

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

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

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

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

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

^~~~~~
```
[R/jaatha_model.R:134:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L65) *style:* **Trailing whitespace is superfluous.**
```r
                   repetitions = 3,
                                   ^
```
[R/jaatha.R:66:54:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L67) *style:* **Trailing whitespace is superfluous.**
```r
                   max_steps = 100,
                                   ^
```
[R/jaatha.R:68:64:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L74) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:76:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L83) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/jaatha.R:85:10:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L89) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~~~
```
[R/jaatha.R:103:64:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L105) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/jaatha.R:106:19:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L113) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~
```
[R/jaatha.R:131:65:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/jaatha.R#L135) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:1:52:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L3) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~
```
[R/likelihood.R:11:49:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:14:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

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

^~
```
[R/likelihood.R:50:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L65) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:66:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:79:5:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L80) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/likelihood.R:82:22:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L83) *style:* **Trailing whitespace is superfluous.**
```r

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

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

^~
```
[R/likelihood.R:92:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L117) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:118:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L124) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:135:5:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/likelihood.R#L137) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/likelihood.R:138:3:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/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/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/par_ranges.R#L29) *style:* **Commas should never have a space before.**
```r
      exp(value * private$log_range_width + private$log_range[ , 1]) +
                                                              ~^
```
[R/par_ranges.R:29:71:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/par_ranges.R#L29) *style:* **Trailing whitespace is superfluous.**
```r
      exp(value * private$log_range_width + private$log_range[ , 1]) +
                                                                      ^
```
[R/par_ranges.R:36:23:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/par_ranges.R#L36) *style:* **Commas should never have a space before.**
```r
      (private$range[ , 2] - private$range[ , 1]) / 2 + private$range[ , 1]
                     ~^
```
[R/par_ranges.R:36:45:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/par_ranges.R#L36) *style:* **Commas should never have a space before.**
```r
      (private$range[ , 2] - private$range[ , 1]) / 2 + private$range[ , 1]
                                           ~^
```
[R/par_ranges.R:36:72:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/par_ranges.R#L36) *style:* **Commas should never have a space before.**
```r
      (private$range[ , 2] - private$range[ , 1]) / 2 + private$range[ , 1]
                                                                      ~^
```
[R/simulation_cache.R:22:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/simulation_cache.R#L22) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/simulation_cache.R:27:7:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/simulation_cache.R#L27) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
      }, logical(1))
      ^
```
[R/simulation_cache.R:29:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/simulation_cache.R#L29) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/simulation_cache.R:37:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/simulation_cache.R#L37) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/simulation_cache.R:40:7:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/simulation_cache.R#L40) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
      }, logical(1))
      ^
```
[R/stat_basic.R:8:1:](https://github.com/statgenlmu/jaatha/blob/4fa1dcbea76f8b3a6890cc003cc7971407700b77/R/stat_basic.R#L8) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```


This indentation `void octaveChanged();` does not match the other lines in this file, i.e. `void updatePositionAccompany(const MidiTime & t );`


Minor: use `struct` instead of `class` here?


Same as above, this indentation does not match the other lines in this file.


Is there an issue if the HttpResponseMessage isn't being disposed?


Couldn't have done it better myself. Like #19066, will conflict with #19133.


i think it's a motion sensor
https://github.com/MSM8226-Samsung/android_kernel_samsung_ms013g/blob/cm-13.0/arch/arm/configs/cyanogenmod_ms013g_defconfig#L3647

??


Hm, TIL.

of / if is still misused, won't worry about another PR, someone will do a larger edit i'm sure :tongue:


Was this fixed on 8.0?


I really want to prefix (and would have a liked a more clear startswith() function). This is similar code to https://github.com/diydrones/ardupilot/blob/master/libraries/AP_HAL_Linux/I2CDriver.cpp#L72-L75

And the string I'm searching for comes from this table:
https://github.com/diydrones/ardupilot/blob/master/libraries/AP_HAL_Linux/HAL_Linux_Class.cpp#L57-L63


You're right, @robdodson. I swapped out the code sample with one from the PSK.


GroundDomain-Output-Variable-Name-Changes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Btw. consider the following:

T1:
lock (m1);
cond.wait();

T2:
lock (m1);
do something;
unlock(m1);

T3:
lock (m1);
do something else T1 is wayiting for
cond.signal();
unlock (m1);

So even T1 is waiting for T3 it would always (!) be woken up whenever T2 is run before and unlocking the mutex - that's not what we want, right?


This should use misc.get_scratch_devices, which maybe should be enhanced with some of this code


Correct. It would probably be wise to add the check for 'sc' below and remove the null pointer check.


c9 commands
```
cd exp2014
DEBUG=exp2014:* npm start
&& ^C
```


I think you mean "will conflict with #19068" here. :)


uploaded whole =D


Could you add variables independent on theme mode?

https://github.com/StartPolymer/polymer-starter-kit-plus/blob/master/app/themes/default-theme/variables.css#L38-L43


This doesn't seem to go with the 'use _get_dev_list'; what's the motivation for this?


nice solution :+1:


seems consistent with #19054 &  #18779. Good to get it out of api/types, I think.


There's a new PR https://github.com/PolymerElements/paper-styles/pull/89/files, that cleans this up a bit more


I'm not sure how comfortable to be with this.


I think this must not be necessary?  IIRC run() takes a string or an array (conveniently)


It seems I miscounted - 9 times, not 15 :smile:


same as before


etc


Ver comentarios en #17


this is not an optimization. variable declaration should always be at the top of the function to avoid hoisting confusion. the JS compiler will basically split the code up to what I have originally written anyways, so it's better to be clear about it code.


we use condition variables with predicates. you can signal as often as you like. that only triggers a check of the predicate


With 0 commits :|


Remove the file instead if `newBlobSums` is empty?


understood, thanks.


Sure. I will then turn `run` into `string -> string array option -> process`, since it is more easy to factorize.


Concerning your notes:

- FirebrandFireball and SwordPuff lacks of Obituary
- AstroDrone, AstroCyborg1-2 & AstroRobot have Nazi obituaries
- G43Tracer has the same Obituary of Kar98kTracer
---- Can you fix these?
- We still need Cartridge53 sprites
---- I don't understand that
- Miniship enemy lacks of effects and debris while exploding
---- You could take care of that as well, considering that you are into the whole debri thing :)
- Geist, Hitler Ghost & Undead Monk needs to be improved
---- What do you have in mind?
- MadCook and MadCook2 shares same sprites
---- That's not ultra dramatic I think


I don't see the purpose of this code movement by itself.


Not really.

Fine:
```
while (something != true)
cond.wait();
```

Not fine:
```
cond.wait();
```

So with that unlock change you would get spurious wakeups by design :-)


Remember your own suggestion of splitting SPIDeviceProperties our of SPIDeviceDriver. This way we avoid a conceptual cyclic dependency between SPIDevice and SPIDeviceDriver (later on).


The original autotest test passed `poo` to aio-stress. Maybe a small note explaining the options you used?


(Well, not super important though)


Can you go ahead with this change and add a test? The test would be important here but I have no idea on how to get started with it.


Wew full of kang :|


@justinsb this seems to compile.  what do we do next?


Nice use of custom Handlebars helpers!


Cool! Thanks! :tada:


hello, this is my first contribution, I add the property "isdispose" in class "Song". to look like than "xna"


Any reason why to use `/usr/bin/env python` directly, instead of simply python? By the way, isn't compilebench directly executable?


It's just a preparation for 3adaf3b (Merge router/local package into router package). I separated the change into two commits to allow for easier review.


Ack. btw, it's just the filename created to make the tests. I don't have a better reason to call it 'foo' than one could have to call it 'poo'. I will make it more clear anyway :)


add-default-build-type (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (927 of 927 tests passed)

![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-26.13%25-green.svg)


So after talking with a few other devs here, we've decided that NPM is sufficient. I have heard of plans for ADV to host a bower repo, but for ozone purposes (my purposes) it's now irrelevant.


*fix


That's actually not a bad idea. +1


Thanks! I forgot how well array to string worked on the sql side, I will switch over our description formulas to that and then I can remove the extra processing step in Python.

working on this now


will remove, thanks.


Minor: if you want to embrace OwnPtr, you could use it here, drop the deletes and when pushing back, leak the value out of it.


take a closer look at CSharedSection and you'll see. in particular:

```
XbmcThreads::TightConditionVariable<XbmcThreads::InversePredicate<unsigned int&> > cond;
```

read the comment attached to definition of TightConditionVariable


Belay that. I think we can do this without plumbing `flags` through at all.


not in love with the name 'sticky' how bout 'inheritNotes' or something that describes this better


Are you encountering a place where the user is getting more than one _user doc?


Skimmed this only. The general comments I made in the `Linux::I2CDevice` should be considered here too.


@volaya @medspx does the unit test need to be updated to reflect this change or has it introduced a regression? The processing tests are failing since this commit.


added to pow


This change in variable order is not consistent with the order that is in toolkit.version file. I will change/correct the other usages of the method to provide the correct variables in next revision commit.


The problem is that the content of the returned value is dependent on the platform.
This is the same reason why `daemon.ContainerInspect()` also returns `interface{}`.

I think that it's okay here because the `interface{}` is only ever returned to the client after serializing it via JSON, and never used directly by the daemon itself, so type safety is not an issue here.
If the value would be used by the daemon code itself, I'd fully agree that it should be a type safe value.


Advanced search page still has some problems @omermtemel


It's weird, but that seems to be caused by me piping it into `head`. I don't normally get that from pipelines on Darwin, but it might be some weird interaction with `xargs`. Doesn't seem to adversely affect stdout though.


Disabled it


could this be multiple handlers?


GroundDomain-Output-Variable-Name-Changes (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1990 of 1995 tests passed)

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


I know it just crashed xD


This overwrites `$this->email` set in the constructor, could have led to horrible behaviours.


Hmm, servers is a map[string]struct{} is copy the right word to use in this case?


lets make sure we have at least one thing you can actually change in there.
i though:
- switch for enabling and disabling tracking
- switches for different map overlays (kreisfahrt and sternfahrt should be enough for now)
- maybe in the future some kind of field that lets you input a string you want to be identified by so people can track you on the map

more suggestions?


compilebench has this in its head:
~~~
$ head -n1 compilebench-0.6/compilebench
#!/usr/bin/env python2.4
~~~
Original autotest test solves it by using `patch -p1 < compilebench.path`, with a one-line patch changing the head to `/usr/bin/env python`. I simplified it with the command line, but yes, we can use simply `python`.


Again, this is fixed.


How would you feel about reverting all this plumbing through of `flags` and manually maintaining a stack of state in `collectMatchingElements` instead? Basically trading off adding complexity here for adding complexity there, but I think it'd make things a little simpler overall because it doesn't add confusing extra arguments here.

That said, I'd understand if you're pretty over this by now and don't want to undertake such a complex rewiring at this point; I could do it in a follow-up instead.


Good idea. WIll work on that.


It's weird that we have to handle things in this way -- Anyone knows when a 404 will happen for `ct=10`?


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


`one and only one argument` should be changed to `two and only two arguments`


DeploymentController::abort is missing a docblock.


just noticing this now ... but I would like if these fields were a bit more descritpive as well.

beginTimeMillis
beginTimeMicros


Where is this read?


Hooray!


Good catch, my bad, didn't notice it.


Nice test! It may seem like overkill to test for each individual attribute, and in some cases it may be, but by testing each separately, you ensure that not only will the directive renders as expected, but it also forces you to carefully construct the HTML and not include unnecessary attributes/classes.


Breaks all test PRs :)


These too...

endTimeMillis
endTimeMicros


should we offer durationMillis and durationMicros?


Looks good to me.


Also, I just noticed `level` - doesn't seem to be used in this change but I'm assuming it gets used later on. That would make my idea impractical...


.. me and matcracker tried. Only wire works? And no inputs. actually, i'd love to work on the input rewrite.
And btw, crash :stuck_out_tongue_winking_eye:
```
[19:01:58] [CRITICAL]: ClassNotFoundException: "Class pocketmine\block\Level not found" (EXCEPTION) in "/src/spl/BaseClassLoader" at line 144
[19:01:58] [DEBUG]: #0 /src/pocketmine/block/Tripwire(92): spl_autoload_call(string pocketmine\block\Level)
[19:01:58] [DEBUG]: #1 /src/pocketmine/level/Level(1625): pocketmine\block\Tripwire->onUpdate(integer 1)
[19:01:58] [DEBUG]: #2 /src/pocketmine/block/Block(758): pocketmine\level\Level->setBlock(pocketmine\block\Tripwire 0, pocketmine\block\Tripwire 0, boolean 1, boolean 1         )
[19:01:58] [DEBUG]: #3 /src/pocketmine/level/Level(1979): pocketmine\block\Block->place(pocketmine\item\StringItem Item String (287:0)x1, pocketmine\block\Air Block[Air         ] (0:0), pocketmine\block\Grass Block[Grass] (2:0), integer 1, double 0.10161590576172, double 1, double 0.91950988769531, pocketmine\Player Player(221))
[19:01:58] [DEBUG]: #4 /src/pocketmine/Player(2076): pocketmine\level\Level->useItemOn(pocketmine\math\Vector3 Vector3(x=49,y=3,z=220), pocketmine\item\StringItem Item          String (287:0)x1, integer 1, double 0.10161590576172, double 1, double 0.91950988769531, pocketmine\Player Player(221))
[19:01:58] [DEBUG]: #5 /src/pocketmine/network/RakLibInterface(145): pocketmine\Player->handleDataPacket(pocketmine\network\protocol\UseItemPacket object)
[19:01:58] [DEBUG]: #6 /src/raklib/server/ServerHandler(90): pocketmine\network\RakLibInterface->handleEncapsulated(string 151.49.36.31:19132, raklib\protocol\Encapsula         tedPacket `.hv..l.......1.........=...?...?ke.BE.7@...CZ........., integer 0)
[19:01:58] [DEBUG]: #7 /src/pocketmine/network/RakLibInterface(86): raklib\server\ServerHandler->handlePacket(boolean)
[19:01:58] [DEBUG]: #8 /src/pocketmine/network/Network(165): pocketmine\network\RakLibInterface->process(boolean)
[19:01:58] [DEBUG]: #9 /src/pocketmine/Server(2542): pocketmine\network\Network->processInterfaces(boolean)
[19:01:58] [DEBUG]: #10 /src/pocketmine/Server(2276): pocketmine\Server->tick(boolean)
[19:01:58] [DEBUG]: #11 /src/pocketmine/Server(2154): pocketmine\Server->tickProcessor(boolean)
[19:01:58] [DEBUG]: #12 /src/pocketmine/Server(1755): pocketmine\Server->start(boolean)
[19:01:58] [DEBUG]: #13 /src/pocketmine/PocketMine(469): pocketmine\Server->__construct(pocketmine\CompatibleClassLoader object, pocketmine\utils\MainLogger object, str         ing phar://C:\Users\Administrator\Desktop\Server\MinecraftPE\ImagicalMineTestServer\ImagicalMine-IC_1.0dev#348.phar/, string C:\Users\Administrator\Desktop\Server\Minec         raftPE\ImagicalMineTestServer\, string C:\Users\Administrator\Desktop\Server\MinecraftPE\ImagicalMineTestServer\plugins\)
[19:01:58] [DEBUG]: #14 (1): require_once(string phar://C:/Users/Administrator/Desktop/Server/MinecraftPE/ImagicalMineTestServer/ImagicalMine-IC_1.0dev#348.phar/src/poc         ketmine/PocketMine.php)
[19:01:58] [DEBUG]: [RakLib Thread #4332] An E_WARNING error happened: "date(): It is not safe to rely on the system's timezone settings. You are *required* to use the          date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspel         led the timezone identifier. We selected the timezone 'UTC' for now, but please set date.timezone to select your timezone." in "/src/pocketmine/utils/MainLogger" at lin         e 197
[19:01:58] [DEBUG]: #0 /src/raklib/server/SessionManager(88): pocketmine\utils\MainLogger->send()
[19:01:58] [DEBUG]: #1 /src/raklib/server/SessionManager(311): pocketmine\utils\MainLogger->notice()
[19:01:58] [DEBUG]: #2 /src/raklib/server/SessionManager(285): raklib\server\SessionManager->blockAddress()
[19:01:58] [DEBUG]: #3 /src/raklib/server/SessionManager(106): raklib\server\SessionManager->receiveStream()
[19:01:58] [DEBUG]: #4 /src/raklib/server/SessionManager(96): raklib\server\SessionManager->tickProcessor()
[19:01:58] [DEBUG]: #5 /src/raklib/server/SessionManager(84): raklib\server\SessionManager->run()
[19:01:58] [DEBUG]: #6 /src/raklib/server/RakLibServer(235): raklib\server\SessionManager->__construct()
[19:01:58] [DEBUG]: #7 /src/raklib/server/RakLibServer(0): {main}()
[19:01:58] [NOTICE]: [RakLib Thread #4332] Blocked 151.49.36.31 for 5 seconds
[19:01:58] [DEBUG]: [RakLib Thread #4332] An E_WARNING error happened: "date(): It is not safe to rely on the system's timezone settings. You are *required* to use the          date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspel         led the timezone identifier. We selected the timezone 'UTC' for now, but please set date.timezone to select your timezone." in "/src/pocketmine/utils/MainLogger" at lin         e 210
[19:01:58] [DEBUG]: #0 /src/raklib/server/SessionManager(88): pocketmine\utils\MainLogger->send()
[19:01:58] [DEBUG]: #1 /src/raklib/server/SessionManager(311): pocketmine\utils\MainLogger->notice()
[19:01:58] [DEBUG]: #2 /src/raklib/server/SessionManager(285): raklib\server\SessionManager->blockAddress()
[19:01:58] [DEBUG]: #3 /src/raklib/server/SessionManager(106): raklib\server\SessionManager->receiveStream()
[19:01:58] [DEBUG]: #4 /src/raklib/server/SessionManager(96): raklib\server\SessionManager->tickProcessor()
[19:01:58] [DEBUG]: #5 /src/raklib/server/SessionManager(84): raklib\server\SessionManager->run()
[19:01:58] [DEBUG]: #6 /src/raklib/server/RakLibServer(235): raklib\server\SessionManager->__construct()
[19:01:58] [DEBUG]: #7 /src/raklib/server/RakLibServer(0): {main}()
[19:02:05] [DEBUG]: Packet pocketmine\network\protocol\UseItemPacket 0xaa0000003100000004000000dd043ec000003f0e98b83f773d00423f431440b3d70a435bf2cd011f0100000000
[19:02:05] [CRITICAL]: ClassNotFoundException: "Class pocketmine\block\Level not found" (EXCEPTION) in "/src/spl/BaseClassLoader" at line 144
[19:02:05] [EMERGENCY]: An unrecoverable error has occurred and the server has crashed. Creating a crash dump
[19:02:05] [EMERGENCY]: Please upload the "C:\Users\Administrator\Desktop\Server\MinecraftPE\ImagicalMineTestServer\CrashDump_Wed_Jan_6-19.02.05-GMT_2016.log" file to t         he Crash Archive and submit the link to the Bug Reporting page. Give as much info as you can.
```


But before the crash happened, it just blocked me. I'll check tripwire


looked like not more complex


Oh, here's that extra whitespace!


Trivial: We should separate this line from the next one, as the comment only applies to this line.


I think what happened is I got tripped up by the dsl-like nature of rake.

In ruby you can't do this:

```
foo = 1

def bar()
  puts(foo)
end

bar() # says it can't find foo
```

So for example in lib/helpers.rb I load the yaml file in every method where I need it.

You could get around this with a global variable but I like to avoid those.

Anyway, in rake the task() method is superficially the same, so it could be that that tripped me up.
But I would still like confirmation that the rake targets work after this commit.


Seems this crash is not this commit  fault ..


:+1: But what about togglePowered? Doesn't toggle at all.


parameterized and parametrized are both correct


least and minimum are superfluous


make this final


final


`grep ! <pattern> <file>` does not work. You should `! grep <pattern> <file>` instead


The use of `first` here protects against more than one value (or completion not) coming from a resolver which can result in hanging in `zip`. There are specific tests for that.


:grinning:


Shouldn't this be Mono rather than Flux? We are expecting a single argument value as the Javadoc states and we are applying `defaultIfEmpty` + `first` after that in `InvocableHandlerMethod`.


@thebigsmileXD
What do you mean ?
It seems works well


Using a silent grep (`grep -q`) would be better here


tbh I think it's kind of weird to take a StringIO because in most cases you're going to have the features as a string if you have it in memory, not StringIO so that could just be an implementation detail of feaLib, which would remove the need for adding an attribute to it in the first place, i.e. we could have two constructors of the builder object, one that takes a file path (or FileIO instead) and one that takes a string and a working directory (so, this would use an alt. constructor w a classmethod that calls \__new__). This would be like freetype where you have distinct FT_New_Face or FT_New_Memory_Face (e.g. I've made an alt. constructor for it for the py bindings in rougier/freetype-py#33).


I'm not doing a release for this.


Is this really a necessary condition? Could there be situations (on a private registry) where slashes are used for something other than delimiting accounts from repository names, or where a particular account has access to multiple base paths?

But it probably makes sense to at least prioritize candidates that meet this constraint...


This pattern is not correct. The second [] should be ().
The architecture is only valid on IE 10 under Windows 8 or later.

This needs to be studied a bit further but I don't think this change is correct.


ack ... (IntelliJ dictionary didn't think so)


Okay, i see. The redstone wire was powered next to a tripwire. Fixed.
And.. Pressure plates do not work, redstone torches only invert if on the side of a block, everything only toggles with redstone wire (like inverting torches), redstone wire doesn't reset if attached to redstone lamp, daylight detector still gives 0 power (i might fix that and add the real calculation).. And could you make it so noteblocks get powered?


done


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10833](https://teamcity.mesosphere.io/viewLog.html?buildId=100915&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:12


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


should all tests be marked final then?


"it would be converted to a matrix"
you mean "vector" probably....yes quite annoying.


Thanks for advice buddy, but you have to strictly respect author/proprietary of any single commit. Take a look at what I'll push. Thanks again.


It ends up in a file called `local.go` one level up. In the end it seems to split the functions from the interfaces. I dunno. Stylistic differences.


YES


Well, haven't checked after this commit, but before this, the power didn't toggle but the state of the button (pressed|non pressed) does


:hushed: I can't believe this. Thanks a lot


I don't want to brute force through all possible sources, and there's really no other useful heuristic I'm aware of for assuming access similarity. I'm sure this could be improved in the future...


look at the diff ;)


Mmmh, mmmh - not fully convinced. You could just return NotifyAll for Notify() in general if that behaviour would be intended ... I need to think a bit more about it.


i can buy that :)


work_progress is actually a CtaWorkProgress object, so it counts per block work, and should get Reset once per block. The reason I added blockIdx.x == 0 for partitioned strategy is that I do not use work_progress, so I just keep the interface but want to save the efforts. Anyway, shouldn't be the problem. I'm going to debug this a bit today.


>How does that sound?

Sounds good to me! :+1:


Can you add a 1-line comment to explain we are verifying that parent account exists if `parentAccountId` is not null?


Its in pocketmine.yml
PLEASE REVERT THIS
And kicking the players is not necessary its alreasy done by shutdown!


Makes sense.


Good catch


it is a call to the std function with std_tag nobody is compelled to use it


Make sure to document the DDL changes in the migration page: https://github.com/killbill/killbill/wiki/Kill-Bill-Migration-0.16.x-to-0.18.x.

See https://github.com/killbill/killbill/wiki/Kill-Bill-Migration-0.14.x-to-0.16.x for an example.


@thebigsmileXD
Try the latest code , and conpare with local mode
If you find any bug plz give a screenshot .... I cant understand clearly , sorry for my poor english


Alright, sweet. That does keep things a bit cleaner. To me it feels like the trend in JavaScript land nowadays is to just use NPM for everything anyway.


Nice work here. FYI: It is considered better security practice to merely state something like "Incorrect user or password" for either username or password mismatch so as to give it less info.


I think is worthwhile clarifying in the commit message that, if we ever want a full-duplex capability back, it's just a matter of adding such function to SPIDevice interface.


I finally thougth they where worth nothing


ok


By way of comparison, https://github.com/cgwalters/rpm-ostree/commit/4a9f20da3b37abaabcf7dac73fb3f8d7220a41b1#diff-9c9868f10e45d9c09feaeef0c8a162c6R451 simply doesn't write a ref, which is presently possible with the C API...I could try to convert the introspection client to it, but eh, the integrated version should be the future, this is just a stopgap.


youre right


Good work! :+1:


This changes the behavior of random_layout. It no longer places nodes randomly in [0,1). In fact, the randomness appears in a complex way because the values provided are scaled (after they are chosen) to lie in [-1.0, 1.0].  So if you by chance get a max value of 0.5 in the random numbers, your positions get scaled so the max is 1.0. Using this scaling addition you no longer have uniform random position of nodes.

I am sure we don't want to use the ```_rescale_layout``` function here.
We probably also don't want the scale argument for random_layout.
And even the center argument (which is a recent addition) is suspect because the doc_string indicates it is the center of the points while it is actually the lower left possible value of the points.

I guess we could use ```pos = (np.random.random(shape)-0.5) * 2 * scale + center``` so that it lies randomly in ```[center-scale, center+scale]```. But that's not how random numbers are usually specified and it feels like we are pushing a round hole onto this square peg.  I'll make this case more in #1749. But the ```_rescale_layout``` should definitely be removed here.


:rocket:


More whitespace weirdness :frowning:


Esquece, esta treta não ia funcionar ... Ou se calhar ia, mas não consegui. Ainda fiz uns testes com uma struct, que acabei por publicar no meu pastebin ( podes ver aqui pastebin(dot)com/YG0rYJJf ) mas aqui não consegui


Thanks!


Thanks for opening this PR. It is a good start for the iOS implementation.

Bringing the iOS implantation on apr with the Android one needs some planning. As already seen with the `watch` method there are differences in the properties of the returned object. We might need to consider a redesign of the API to accommodate both platforms.

Another idea is to split the plugin in two parts one for publishing and one for discovering services. This approach could make easier to have parity and we decouple the two parts as not all apps ends both.


I've had just "alpine" as base with consecutive `apk upgrade --update` to ensure latest version and security updates would get installed. I'm not seeing you doing `apk upgrade`, which would be good.. unless I've missed it.


What's with the race db in the script folder?


ich verstehe nicht ganz, was du machen möchtest. aktuell schiebst du die schwarze Figur auf Position null - das wird zu einer Null Pointer Exception führen. Möchtest du die schwarze Figur vom Spielbrett nehmen? Dies geschieht automatisch, dadurch dass die weiße Figur an die Stelle gesetzt wird durch aBoard.setFigureAt(aBoard.getFigureAt(aSource), aTarget);


Só é suposto copiar as unidades de uma sala para a outra. Por isso mesmo é que se criar uma sala nova.
Falta adicionar as restantes unidades dentro do ciclo


suppessed


The above discussion is a really really good argument for why we will need good comments in our complex code so that anyone following you guys can make sense of design decisions/code that you wrote. :)


Com ID é melhor. ID só á um por unidade, mas podem haver duas unidades com o mesmo nome na mesma sala.


Instead of deleting the line and having to generate an empty controller file, perhaps we could replace it with a self documenting line, i.e: ```var args = $.args;```


Couldn't we chain all the way? In other words:
````java
return Flux.fromIterable()
        .concatMap()
        .next()
        .then()
        .then()
        .otherwise();
````


Could you explain what makes this better?
Do we need glibc-bin?


Q: are you changing the default in this commit or is this just a fix?


not used elsewhere


`blackout()` sounds good to me. I'll make ours use a fixed-size buffer so other things can remain constant.


I would call it “the menu attached to `attributionButton`”, because developers viewing the documentation in Chrome _still_ won’t see the emoji…


Foi necessário o construtor por cópia para as unidades, por este motivo. Caso contrario, ao fazer delete da sala, removia também as unidades. Assim são criadas outras unidades iguais e adicionadas á nova sala, e estas serão eliminadas sem causar problemas


Mi ero lasciato fuorviare dalla stesura in italiano. Correggerò nel
prossimo commit.
Il 06/gen/2016 19:43, "Davide Polonio" <notifications@github.com> ha
scritto:

> I nomi dei file non erano in inglese?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BugBusterSWE/documentation/commit/48a50ec8fbdf257c20a190c0a944d68ec8568b23#commitcomment-15297520>
> .
>


note that this are not posix condvars, those are defined in our code. as mentioned, you can signal as often as you want or even do a busy loop. important is the value of the predicate and this is protected by the mutex.


Isto aqui por baixo é importante


they where missing, but atan2d  was not in the CMakelist


GroundDomain-Output-Variable-Name-Changes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (927 of 927 tests passed)

![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-26.13%25-green.svg)


Agreed -- will make that change


**Code Quality - Critical**

You don't need this line to retrieve views again.

Line 16 has the number of views already, so I expect to just see

``php
return response()->json($project->views);
```


LGTM.


Nice idea! To add extensibility to this and aid readability, it might be worthwhile to name the value something other than a true/false boolean.


Wow so much kang, 0 skill

Learn to give credits mate.


These now look like:
(SemiboldItalic does not have a long s)

![x](https://cloud.githubusercontent.com/assets/1371085/12152312/cd88b566-b4ab-11e5-9dd6-f3378776dc0e.png)


TeamCity DNN JavaScript Libraries :: CI [Build 129](http://teamcity.engagesoftware.com/viewLog.html?buildId=407&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 129](http://teamcity.engagesoftware.com/viewLog.html?buildId=407&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:9


Would more naturally be the other way around, or both extending a common `abstract` class, but close enough.


That conforms to my understanding. Okay :+1:


4.3? You're not going to back-port a big feature like this, are you?

In any case, I would just leave the version number out of the comment, it doesn't really add any value. Just explain what the code does as it stands.


I know is legacy code. But why 10?


Sure, I will.


this goroutine is not stopped, but I guess it's not that bad because it'll be stopped when chihaya exits?


Ah, now I get why the earlier change here happened.


Decidi alterar estas funções, porque quando criei fiz de forma a retornar apenas um valor. Mas se a prof nos pedisse para retornar 3 valores estávamos feitos ... Então assim é fácil.
E agora que me lembro fiz mal o delete disto, mas depois eu corrijo.


The remaining content of `router/local/local.go` finally ends up in `router/image/image.go` (bacdfe6).

One reason why the commits are split this way is that I didn't want to mix the move of `local.router` to `image.imageRouter` with the move of `local.NewGetRoute` to `router.NewGetRoute` (etc.), especially to make reviewing the quite large diff of the latter change easier.


Já está tudo explicado!


Should we just not remove these lines altogether then


`this.target` should do


Can you explain the `legend` logic here?


Probably OK, but…why?


Much better. Thank you. :)


Does not need the validation at this point (would return an empty list).


Stray from another change? It's also not reflected in the JSDoc.


Ok $row ['link'];
114 if (OPF_SHORT_URL) {$view_url = WB_URL. $ row ['link']; }


I was running into this issue as well. Out of curiosity, why was the `assert_receive` line not receiving `{:exit, "shopping", ^bucket}`? I was always getting a `process mailbox is empty` error whenever I ran this test.


`window.innerHeight` alone is fine, we're not supporting IE8 and nor are Microsoft =]


`mkdir -p` succeeds silently if the directory already exists. I think you could lose the `if`.


Добавлено отображение страницы с информацией о товаре
URL вида: /products/product/[_id_товара_]

1. Добавлен метод product контроллера Products.php
2. Добавлен хелпер getHtmlForProduct с «рыбой» под отображение инфы о товаре
3. Добавлен метод getProductById в модель Products_Model.php для получения информации о товаре из СУБД по его ID
4. Добавлен шаблон product.twig для метода product контроллера Products.php


@jancborchardt - let me hear your opinion about this google keep-like card-layout instead of the current list-layout?


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


Using an alert here is probably not the most elegant way to do this. Can you think of another approach? Perhaps a way to redirect to a login view?


:scream:


Maybe you could also mention:

1. Whether to install `gulp` globally.
2. That `npm test` can be used (for all the tests).
3. If you are amenable to it, I think it is less intimidating to say a PR should *ideally* be squashed, as I personally don't work very linearly, and it can be a deterrent (or at least an obstacle) for me to contribute to projects which insist strictly on this.


Add an ordering: `<defaultOrderBy()>`.

See https://github.com/killbill/killbill/blob/ce36048b30a7b89d8a369fadf7d685a7444d4724/payment/src/main/resources/org/killbill/billing/payment/dao/TransactionSqlDao.sql.stg#L55 for an example.


You are right, it should be "sz_arg", not "n_in". And below, "n_out" changes to "sz_res" in the declaration of "res". The the for loops remains unchanged though.


Swap the `equals` to avoid NPE.


Now the buttons power, but don't unpower


yes, except for the ones intended for extension, e.g. EvictorTestBase