[R/feature_ignore_singletons.R:39:15:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_ignore_singletons.R#L39) *style:* **Commas should never have a space before.**
```r
    segsites[ , !is_singleton]
             ~^
```
[R/feature_ignore_singletons.R:50:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_ignore_singletons.R#L50) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:54:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_ignore_singletons.R#L54) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:58:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_ignore_singletons.R#L58) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_mutation.R:140:9:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_mutation.R#L140) *style:* **Commented code should be removed.**
```r
#       model <- addFeature(model, "gamma.categories", parameter=categories.number)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_mutation.R:143:7:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_mutation.R#L143) *style:* **Commented code should be removed.**
```r
#     return(model)
      ^~~~~~~~~~~~~
```
[R/feature_mutation.R:152:5:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_mutation.R#L152) *style:* **Commented code should be removed.**
```r
#   model <- addFeature(model, 'mutation', parameter = middle_rate, group = group)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_recombination.R:47:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_recombination.R#L47) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.recombination <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_selection.R:178:36:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_selection.R#L178) *style:* **Commas should never have a space before.**
```r
                        n_pop, " " , paste(start_freq, collapse = " "), " ",
                                  ~^
```
[R/feature_selection.R:191:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_selection.R#L191) *style:* **lines should not be more than 80 characters.**
```r
                         ifelse(feature$get_start(), 0, feature$get_time()), ", ",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_selection.R:202:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_selection.R#L202) *style:* **lines should not be more than 80 characters.**
```r
                         ifelse(feature$get_start(), 0, feature$get_time()), ", ",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:18:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_sumstats.R#L18) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:22:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_sumstats.R#L22) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:26:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/feature_sumstats.R#L26) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.seg_sites_feat <- conv_to_ms_arg.seg_sites_feat
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/import_popgenome.R:17:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/import_popgenome.R#L17) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.GENOME <- function(data, only_synonymous = FALSE, ...) {
^~~~~~~~~~~~~~~~~~
```
[R/import_popgenome.R:41:43:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/import_popgenome.R#L41) *style:* **Commas should never have a space before.**
```r
      seg_sites <- bam[c(pops, outgroup), , drop = FALSE]
                                         ~^
```
[R/model_getters.R:74:21:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model_getters.R#L74) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                   ~^
```
[R/model_getters.R:74:60:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model_getters.R#L74) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                                                          ~^
```
[R/model_getters.R:81:25:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model_getters.R#L81) *style:* **Commas should never have a space before.**
```r
  min(which(cumsum(llm[ , "number"]) >= locus))
                       ~^
```
[R/model_getters.R:124:46:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model_getters.R#L124) *style:* **Commas should never have a space before.**
```r
  numbers <- get_locus_length_matrix(model)[ , "number"]
                                            ~^
```
[R/model.R:93:39:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model.R#L93) *style:* **Commas should never have a space before.**
```r
  sum(get_locus_length_matrix(model)[ , c(1:2, 4:5)]) > 0
                                     ~^
```
[R/model_simulate.R:36:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/model_simulate.R#L36) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  }, mc.set.seed = TRUE, mc.cores = cores)
  ^
```
[R/parameter.R:132:11:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/parameter.R#L132) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if ((!is.numeric(value)) || length(value) != 1) {
          ^
```
[R/segsites.R:18:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/segsites.R#L18) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.matrix.segsites <- function(x, ...) get_snps(x)
^~~~~~~~~~~~~~~~~~
```
[R/segsites.R:31:19:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/segsites.R#L31) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  create_segsites((matrix(c(1, 1, 0, 1, 1,
                  ^
```
[R/segsites.R:96:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/segsites.R#L96) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.default <- function(data, ...) {
^~~~~~~~~~~~~~~~~~~
```
[R/segsites.R:101:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/segsites.R#L101) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.list <- function(data, ...) {
^~~~~~~~~~~~~~~~
```
[R/simulator_class.R:55:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_class.R#L55) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  }, character(1L))
  ^
```
[R/simulator_class.R:67:44:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_class.R#L67) *style:* **Commas should never have a space before.**
```r
  grouped_commands <- unique(sim_commands[ , 2])
                                          ~^
```
[R/simulator_class.R:70:36:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_class.R#L70) *style:* **Commas should never have a space before.**
```r
    sum(sim_commands[sim_commands[ , 2] == cmd, 1])
                                  ~^
```
[R/simulator_msms.R:101:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_msms.R#L101) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_msms.R:101:52:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_msms.R#L101) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/simulator_ms.R:71:9:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_ms.R#L71) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
        }, character(1))
        ^
```
[R/simulator_ms.R:86:39:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_ms.R#L86) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                     ~^
```
[R/simulator_ms.R:86:50:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_ms.R#L86) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                                ~^
```
[R/simulator_scrm.R:84:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_scrm.R#L84) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_scrm.R:84:52:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_scrm.R#L84) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/simulator_seqgen.R:54:4:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_seqgen.R#L54) *style:* **Commented code should be removed.**
```r
  #print(cmd)
   ^~~~~~~~~~
```
[R/simulator_seqgen.R:129:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_seqgen.R#L129) *style:* **lines should not be more than 80 characters.**
```r
                                        sample_seed(length(trees[[locus_group]])))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/simulator_seqgen.R:154:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/simulator_seqgen.R#L154) *style:* **lines should not be more than 80 characters.**
```r
                                                              ignore_variation = TRUE),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_ihh.R:19:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L19) *style:* **lines should not be more than 80 characters.**
```r
    initialize = function(name, population, max_snps, calc_ihs, transformation) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_ihh.R:52:45:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L52) *style:* **Commas should never have a space before.**
```r
                      iHS = data.frame(ihh[ , 1:2], iHS = rep(NA, n_snps))))
                                           ~^
```
[R/sumstat_ihh.R:54:15:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L54) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if ((n_snps < 50)) freqbin <- 0.90
              ^
```
[R/sumstat_ihh.R:55:20:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L55) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          else if ((n_snps < 100)) freqbin <- 0.45
                   ^
```
[R/sumstat_ihh.R:56:20:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L56) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          else if ((n_snps < 200)) freqbin <- 0.225
                   ^
```
[R/sumstat_ihh.R:57:20:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L57) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          else if ((n_snps < 400)) freqbin <- 0.1
                   ^
```
[R/sumstat_ihh.R:60:55:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_ihh.R#L60) *style:* **Commas should never have a space before.**
```r
            data.frame(ihh2ihs(ihh, freqbin)$res.ihs[ , -4])
                                                     ~^
```
[R/sumstat_omega.R:14:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_omega.R#L14) *style:* **lines should not be more than 80 characters.**
```r
    initialize = function(name, min_win, max_win, grid, binary, transformation) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat.R:192:6:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat.R#L192) *style:* **Commented code should be removed.**
```r
    #assert_that(is.list(tree_list))
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat.R:193:6:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat.R#L193) *style:* **Commented code should be removed.**
```r
    #assert_that(length(tree_list) == get_locus_number(model))
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/sumstat_seg_sites.R:41:38:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/R/sumstat_seg_sites.R#L41) *style:* **Commas should never have a space before.**
```r
    trio_segsites <- seg_sites[[i]][ , left | middle | right]
                                    ~^
```
[tests/testthat/test-feature-ign-singletons.R:13:45:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-feature-ign-singletons.R#L13) *style:* **Commas should never have a space before.**
```r
               list(create_test_segsites()[ , -c(1, 3)],
                                           ~^
```
[tests/testthat/test-feature-ign-singletons.R:14:45:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-feature-ign-singletons.R#L14) *style:* **Commas should never have a space before.**
```r
                    create_test_segsites()[ , -c(1, 3)]))
                                           ~^
```
[tests/testthat/test-model-class.R:241:44:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-model-class.R#L241) *style:* **Commas should always have a space after.**
```r
               data.frame(name = c("theta","tau"),
                                           ^
```
[tests/testthat/test-search-executables.R:30:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-search-executables.R#L30) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-segsites-class.R:15:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L15) *style:* **Put spaces around all infix operators.**
```r
  expect_error(create_segsites(snps, 1:4/10))
                                       ~^~
```
[tests/testthat/test-segsites-class.R:16:46:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L16) *style:* **Put spaces around all infix operators.**
```r
  expect_error(create_segsites(snps, pos, 1:4/10))
                                            ~^~
```
[tests/testthat/test-segsites-class.R:22:40:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L22) *style:* **Put spaces around all infix operators.**
```r
  segsites <- create_segsites(snps, 1:3/10)
                                      ~^~
```
[tests/testthat/test-segsites-class.R:30:31:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L30) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  segsites <- create_segsites((matrix(c(1, 1, 0, 1, 1,
                              ^
```
[tests/testthat/test-segsites-class.R:36:26:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L36) *style:* **Commas should never have a space before.**
```r
  ss_subset <- segsites[ , 1:2]
                        ~^
```
[tests/testthat/test-segsites-class.R:38:64:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L38) *style:* **Commas should never have a space before.**
```r
  expect_equivalent(as.matrix(ss_subset), as.matrix(segsites)[ , 1:2])
                                                              ~^
```
[tests/testthat/test-segsites-class.R:41:30:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L41) *style:* **Commas should never have a space before.**
```r
  expect_equal(dim(segsites[ , 1]), c(3, 1))
                            ~^
```
[tests/testthat/test-segsites-class.R:42:33:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L42) *style:* **Commas should never have a space before.**
```r
  expect_equal(dim(segsites[1:2 , ]), c(2, 3))
                               ~^
```
[tests/testthat/test-segsites-class.R:43:32:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-segsites-class.R#L43) *style:* **Commas should never have a space before.**
```r
  expect_equal(dim(segsites[-2 , 1:3]), c(2, 2))
                              ~^
```
[tests/testthat/test-simulator-ms.R:7:73:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-simulator-ms.R#L7) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(parse_ms_positions("positions: 0.1 0.2 0.3 0.4 0.5"), 1:5/10)
                                                                       ~^~
```
[tests/testthat/test-simulator-seqgen.R:213:6:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-simulator-seqgen.R#L213) *style:* **Commented code should be removed.**
```r
  #  model.addMutationRateHeterogenity(model.hky, 0.1, 5, categories.number = 5)
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-simulator-seqgen.R:236:6:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-simulator-seqgen.R#L236) *style:* **Commented code should be removed.**
```r
    #feat_migration(par_zero_inflation(1, .5), symmetric = TRUE)
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-simulator-seqgen.R:274:27:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-simulator-seqgen.R#L274) *style:* **Commas should always have a space after.**
```r
  model <- coal_model(c(5,5,2), 1, 100) +
                          ^
```
[tests/testthat/test-simulator-seqgen.R:274:29:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-simulator-seqgen.R#L274) *style:* **Commas should always have a space after.**
```r
  model <- coal_model(c(5,5,2), 1, 100) +
                            ^
```
[tests/testthat/test-sumstat-jsfs.R:4:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L4) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  seg.sites.3.pop <- list(create_segsites(matrix(c(1, 0, 1, 0,
  ^~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:10:60:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L10) *style:* **Commas should never have a space before.**
```r
                                                   1, 0, 1 ,0 ),
                                                          ~^
```
[tests/testthat/test-sumstat-jsfs.R:10:61:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L10) *style:* **Commas should always have a space after.**
```r
                                                   1, 0, 1 ,0 ),
                                                            ^
```
[tests/testthat/test-sumstat-jsfs.R:11:54:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L11) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                    ~^~
```
[tests/testthat/test-sumstat-jsfs.R:11:62:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L11) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                            ~^~
```
[tests/testthat/test-sumstat-jsfs.R:13:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L13) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  jsfs.3.pop <- calc_jsfs(seg.sites.3.pop, list(1:2, 3:4, 5:7))
  ^~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:13:27:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L13) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  jsfs.3.pop <- calc_jsfs(seg.sites.3.pop, list(1:2, 3:4, 5:7))
                          ^~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:14:13:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L14) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  expect_is(jsfs.3.pop, "array")
            ^~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:15:21:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L15) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  expect_equivalent(jsfs.3.pop, array(c(0, 0, 0, 0, 0, 0, 0, 0, 0,
                    ^~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:18:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L18) *style:* **lines should not be more than 80 characters.**
```r
                                        0, 1, 0, 0, 0, 1, 0, 0, 0 ), dim=c(2,2,3)+1))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:18:73:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L18) *style:* **Put spaces around all infix operators.**
```r
                                        0, 1, 0, 0, 0, 1, 0, 0, 0 ), dim=c(2,2,3)+1))
                                                                       ~^~
```
[tests/testthat/test-sumstat-jsfs.R:18:78:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L18) *style:* **Commas should always have a space after.**
```r
                                        0, 1, 0, 0, 0, 1, 0, 0, 0 ), dim=c(2,2,3)+1))
                                                                             ^
```
[tests/testthat/test-sumstat-jsfs.R:18:80:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L18) *style:* **Commas should always have a space after.**
```r
                                        0, 1, 0, 0, 0, 1, 0, 0, 0 ), dim=c(2,2,3)+1))
                                                                               ^
```
[tests/testthat/test-sumstat-jsfs.R:18:82:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L18) *style:* **Put spaces around all infix operators.**
```r
                                        0, 1, 0, 0, 0, 1, 0, 0, 0 ), dim=c(2,2,3)+1))
                                                                                ~^~
```
[tests/testthat/test-sumstat-jsfs.R:21:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L21) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  seg.sites.3.pop <- list(create_segsites(matrix(c(1, 0, 1, 0,
  ^~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:27:60:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L27) *style:* **Commas should never have a space before.**
```r
                                                   1, 0, 1 ,0 ),
                                                          ~^
```
[tests/testthat/test-sumstat-jsfs.R:27:61:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L27) *style:* **Commas should always have a space after.**
```r
                                                   1, 0, 1 ,0 ),
                                                            ^
```
[tests/testthat/test-sumstat-jsfs.R:28:54:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L28) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                    ~^~
```
[tests/testthat/test-sumstat-jsfs.R:28:62:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L28) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                            ~^~
```
[tests/testthat/test-sumstat-jsfs.R:30:3:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L30) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  jsfs.3.pop <- calc_jsfs(seg.sites.3.pop, list(1, 2:6, 7))
  ^~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:30:27:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L30) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  jsfs.3.pop <- calc_jsfs(seg.sites.3.pop, list(1, 2:6, 7))
                          ^~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:31:21:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L31) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
  expect_equivalent(jsfs.3.pop, array(c(0, 0, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0,
                    ^~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:32:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L32) *style:* **lines should not be more than 80 characters.**
```r
                                        0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0 ), dim=c(1,5,1)+1))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-jsfs.R:32:82:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L32) *style:* **Put spaces around all infix operators.**
```r
                                        0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0 ), dim=c(1,5,1)+1))
                                                                                ~^~
```
[tests/testthat/test-sumstat-jsfs.R:32:87:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L32) *style:* **Commas should always have a space after.**
```r
                                        0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0 ), dim=c(1,5,1)+1))
                                                                                ^
```
[tests/testthat/test-sumstat-jsfs.R:32:89:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L32) *style:* **Commas should always have a space after.**
```r
                                        0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0 ), dim=c(1,5,1)+1))
                                                                                ^
```
[tests/testthat/test-sumstat-jsfs.R:32:91:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L32) *style:* **Put spaces around all infix operators.**
```r
                                        0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0 ), dim=c(1,5,1)+1))
                                                                                ~^~
```
[tests/testthat/test-sumstat-jsfs.R:33:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L33) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-sumstat-jsfs.R:53:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L53) *style:* **Commas should always have a space after.**
```r
  jsfs <- calc_jsfs(seg.sites, list(c(1,3), c(2,4)))
                                        ^
```
[tests/testthat/test-sumstat-jsfs.R:53:49:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L53) *style:* **Commas should always have a space after.**
```r
  jsfs <- calc_jsfs(seg.sites, list(c(1,3), c(2,4)))
                                                ^
```
[tests/testthat/test-sumstat-jsfs.R:58:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L58) *style:* **Commas should always have a space after.**
```r
  expect_error(calc_jsfs(seg.sites, c(1,3), c(2,5)))
                                        ^
```
[tests/testthat/test-sumstat-jsfs.R:58:49:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L58) *style:* **Commas should always have a space after.**
```r
  expect_error(calc_jsfs(seg.sites, c(1,3), c(2,5)))
                                                ^
```
[tests/testthat/test-sumstat-jsfs.R:59:41:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L59) *style:* **Commas should always have a space after.**
```r
  expect_error(calc_jsfs(seg.sites, c(1,7), c(2,3)))
                                        ^
```
[tests/testthat/test-sumstat-jsfs.R:59:49:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L59) *style:* **Commas should always have a space after.**
```r
  expect_error(calc_jsfs(seg.sites, c(1,7), c(2,3)))
                                                ^
```
[tests/testthat/test-sumstat-jsfs.R:141:40:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-jsfs.R#L141) *style:* **Put spaces around all infix operators.**
```r
                                    1:4/4))
                                      ~^~
```
[tests/testthat/test-sumstat-mcmf.R:13:51:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-mcmf.R#L13) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(calc_mcmf(seg_sites, 2:4, FALSE), 2/3)
                                                 ~^~
```
[tests/testthat/test-sumstat-mcmf.R:70:63:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-mcmf.R#L70) *style:* **Put spaces around all infix operators.**
```r
                              coal_model(4) + locus_trio()), 1/3)
                                                             ~^~
```
[tests/testthat/test-sumstat-omega.R:7:5:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-omega.R#L7) *style:* **Use two spaces to indent, never tabs.**
```r
1.11	1.000000
    ^
```
[tests/testthat/test-sumstat-omega.R:8:5:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-omega.R#L8) *style:* **Use two spaces to indent, never tabs.**
```r
2.22	0.200000
    ^
```
[tests/testthat/test-sumstat-omega.R:9:5:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-omega.R#L9) *style:* **Use two spaces to indent, never tabs.**
```r
3.33	0.030000
    ^
```
[tests/testthat/test-sumstat-pi.R:7:67:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-pi.R#L7) *style:* **Put spaces around all infix operators.**
```r
                                        0, 1, 1, 1, 1), 5, 3), 1:3/5)
                                                                 ~^~
```
[tests/testthat/test-sumstat-pi.R:26:67:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-pi.R#L26) *style:* **Put spaces around all infix operators.**
```r
                                        0, 1, 1, 1, 1), 5, 3), 1:3/5)
                                                                 ~^~
```
[tests/testthat/test-sumstat-segsites.R:28:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-segsites.R#L28) *style:* **lines should not be more than 80 characters.**
```r
  seg_sites_trio[[1]] <- create_segsites(get_snps(seg_sites_trio[[1]][ , c(1:2, 4:6, 8:9)]),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-segsites.R:28:72:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-segsites.R#L28) *style:* **Commas should never have a space before.**
```r
  seg_sites_trio[[1]] <- create_segsites(get_snps(seg_sites_trio[[1]][ , c(1:2, 4:6, 8:9)]),
                                                                      ~^
```
[tests/testthat/test-sumstat-sfs.R:62:40:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-sfs.R#L62) *style:* **Put spaces around all infix operators.**
```r
                                    1:4/4))
                                      ~^~
```
[tests/testthat/test-sumstat-tajimas-d.R:14:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-tajimas-d.R#L14) *style:* **lines should not be more than 80 characters.**
```r
                                 0, 0, 0, 0, 0, 0, 0, 1, 1, 0, 1, 0, 0, 0, 0, 0),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-tajimas-d.R:27:1:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-tajimas-d.R#L27) *style:* **lines should not be more than 80 characters.**
```r
  expect_equal(tajd$calculate(list(create_empty_segsites(10)), NULL, NULL, model), NaN)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-sumstat-trees.R:6:30:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L6) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  expect_true(requires_trees((coal_model(5) + sumstat_trees())))
                             ^
```
[tests/testthat/test-sumstat-trees.R:7:30:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L7) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  expect_true(requires_trees((coal_model(5) + sumstat_sg_trees())))
                             ^
```
[tests/testthat/test-sumstat-trees.R:8:34:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L8) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  expect_false(requires_segsites((coal_model(5) + sumstat_trees())))
                                 ^
```
[tests/testthat/test-sumstat-trees.R:9:34:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L9) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  expect_false(requires_segsites((coal_model(5) + sumstat_sg_trees())))
                                 ^
```
[tests/testthat/test-sumstat-trees.R:73:64:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L73) *style:* **Commas should always have a space after.**
```r
                                 locus_trio(locus_length = c(1,3,5),
                                                               ^
```
[tests/testthat/test-sumstat-trees.R:73:66:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L73) *style:* **Commas should always have a space after.**
```r
                                 locus_trio(locus_length = c(1,3,5),
                                                                 ^
```
[tests/testthat/test-sumstat-trees.R:74:60:](https://github.com/statgenlmu/coala/blob/f86dec59ca90948a048a85172ed7656a22d95e98/tests/testthat/test-sumstat-trees.R#L74) *style:* **Commas should always have a space after.**
```r
                                            distance = c(2,4)))
                                                           ^
```


I am fine with that. I am optimizing the code right, I realize that, if I change .value to a string, it will update ._num_value to the same string. If we say .num_value can only be a None or a Number, this will not match the idea. To make that happen, we still need to go through this long check for different type of value.

On the other hand, how do we make sure the user do not accidentally set .num_value to a number for string parameters, for example PSR, because it will update the .value. Maybe run the parameter value type again?
Or can we put a flag saying if the value is a number or related classes?

-Luo Jing


put it within module SiteSettings and some variable like `SiteSettings::CreatingYears`


looks good now!


добавить блок range


Ok, there are some new ones, smaller, without regulator and level shifter.
I'm using
* genuine si7021 works fine
* ebay Chinese [si7021/htd21D](http://www.ebay.com/itm/Si7021-Industrial-High-Precision-Humidity-Sensor-I2C-Interface-for-Arduino-/201371694878?hash=item2ee2b0371e:g:rroAAOSw2VJVfqFt) with regulator and level shifter works fine (which I suspect to be HTU21D)

Is it the 2nd you have ? with regulator and level shifter onboard ?

May you share the code you're using (not the library one) your setup and loop ?

Anyway, thanks for the head up I will look into this.


typo?


naruto mais ça a besoin d'être peaufiné


So by default (e.g. when reading a file) the orientation will be wrong for geometries?


it is not necessary comment it. You have already commented this links


Nice additions!


                            от
                            <input class="min-price" type="text" value="0"> до
                            <input class="max-price" type="text" value="15 000">


@dpgeorge: This is final change (+ a pending change to micropython-lib) to make unix port "ipv6 clean". So, in 1.6 changelog, feel free to include IPv6 support for unix port.


looks good


I'm against this.
Previous commit fixes warning in generated code, not in actual confetti source.
So there is no reason to remove -Werror from source code's Makefile at the moment.


What if we changed this to the following?

```ruby
command = @options.fetch(:command, 'bin/delayed_job')
```


au ok :)


That do you mean wrong?
It is now differ from that was before, but corresponds to that I see in ROOT.
And we could always agree on some other default settings


Sure I will update the PR with tests..


Complete the first section as well (announcements, rules), add a link to thread list in general discussion and make the panel 1 single block.


Good idea, but instead of changing default configuration, I prefer to let user do the change and put this to true by default.


prevent error in puppet parses for: error in value: Error 400 on SERVER: Evaluation Error: Error while evaluating a Function Call, undefined method `length' for 300:Fixnum


This is a comment about your code. Please fix it before I pull.


один блок form


Wrote a little comment about that FOR loop


Ok, I already did fix the typo.


I think Y should be up by default (maybe)


jan das hier ist weg z.B. :P


br не нужны здесь


Removing this line looks like a mistake :neutral_face:


checked добавить


But just start tutorials/geom/geodemo.C.
All examples there show Z in up direction


кнопка в форме и не ссылка
input type="submit" value="Показать"


Wow,Thanks very much,I really cant suffer Baidu any longer~


Please can you rebase and squash so that these commented-out lines don't make it into the history unnecessarily? Thanks.


You don't want to add the next character to `n` since it's being reassigned at the beginning of every loop of the `each` bock, you'll want another variable defined outside of the `each` and add `n` to that .


what a lovely commit you have done


ah et je sais pas si tu a vu ma video sur la forum, mais outrun a encore quelque probleme, deja euu la couleur de la voiture sur le menu, c'est tout dorée je sais pas pourquoi, mais ingame pas ce probleme.
Ensuite une fois ingame il y a un probleme, ya un effet violet sur le devant du véhicule au bout d'un moment, et ensuite la route disparait d'un coup et réapparait comme ça lol, et je peut pas redémarrer une cours sinon le jeux freeze et m'affiche des erreur d'instruction rsx, le jeux a encore besoin detre travailler pour etre vraiment en jouable ^^


**protractor myfile.js --multiCapabilities.0.browserName firefox --multiCapabilities.0.count 3**
doesn't work on protractor 3.0.0.
Any solution?


in the canvas, yes, but not in OpenGL...


One doubt here, if the simple test fork bomb bug happens, will avocado handle things correctly just by passing `--job-timeout` to it? Isn't it better to add a timeout to the `process.run` call?


Would be the user-defined setting `siteBaseUrl` a better fit? It was added in v2.2.0, see these [notes](https://www.accesstomemory.org/en/docs/2.2/admin-manual/installation/upgrading/#set-site-base-url).
e.g. `$repositoryIdentifier = QubitSetting::getByName('siteBaseUrl')->getValue(array('cultureFallback' => true)`
Off the top of my head I think the benefit would be to allow the administrator to manually define the URL when the application is running behind some kind of proxy. You could use `parse_url` to extract the host component of it.


Unfortunately, confetti does not have a separate build configuration for development (aka debug build). -Werror in release builds in a really bad practice and causes lots of fallouts after each compiler update. See e.g. https://gcc.gnu.org/ml/gcc/2016-01/msg00101.html


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 133](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=725&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


The documentation on fluentd.org for this change is really lacking.


Despite it was documented compatibility with 1.9, found one broken import in https://github.com/django-oscar/django-oscar-api/blob/master/oscarapi/utils.py#L5 so far, since now this module being imported from python directly.


@gabrielamatias should be 'fast-paced' with a hyphen


> Sadly it's not the cause of that mac-enum-serialization-crash.

Bummer, because that sure seemed like the behavior (at least on Windows) -- it didn't necessarily crash, but it hangs for a long time deep in serialization logic. e.g.:

![image](https://cloud.githubusercontent.com/assets/2192439/12352916/43455358-bb47-11e5-9c51-053658caf353.png)

I'll try it on Windows later today.


div.hover>a+p+a.btn


ok


Thanx. Works great once you know where to put it :)


список


Thus au test


Please can this be rebased/squashed away? Thanks!


One which rocks server does this version `20120820-1` of `lbase64` exist?

Can't see it on `moonscript.org`, for instance:

```
$ luarocks --server=http://rocks.moonscript.org search lbase64

Search results:
===============

Rockspecs and source rocks:
---------------------------

lbase64
   20120807-3 (rockspec) - http://rocks.moonscript.org
   20120807-3 (rockspec) - https://luarocks.org
   20120807-1 (rockspec) - http://rocks.moonscript.org
   20120807-1 (src) - http://rocks.moonscript.org
   20120807-1 (rockspec) - https://luarocks.org
   20120807-1 (src) - https://luarocks.org
   20100323-1 (rockspec) - http://rocks.moonscript.org
   20100323-1 (src) - http://rocks.moonscript.org
   20100323-1 (rockspec) - https://luarocks.org
   20100323-1 (src) - https://luarocks.org
   20070628-1 (rockspec) - http://rocks.moonscript.org
   20070628-1 (src) - http://rocks.moonscript.org
   20070628-1 (rockspec) - https://luarocks.org
   20070628-1 (src) - https://luarocks.org
```


Seems to be, you are right.
Than I will do Y default option and Z only when drawing TGeo inside TCanvas.


@gabrielamatias should be customers instead of customer


Thanks for all :+1:


ul
li.active
li a
li a
li a


Что за хуйню я делаю?


Do we still need instructions on how to create the offline bundle? For example is `react-native bundle --platform ios --dev false --entry-file index.ios.js --bundle-output iOS/main.jsbundle` no longer needed? Thanks.


ЙОбанаврот как это работает?


I know is not modified in this PR, but the comment above this line is wrong.


Sounds reasonable :+1:


Cool


La meilleure amélioration du site depuis longtemps, permet d'obtenir une grande fluidité dans la validation des compétences en utilisant un écran 120 Hz.


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


cimer


On 01/27/2016 11:19 AM, Jing Luo wrote:
> I am fine with that. I am optimizing the code right, I realize that, if
> I change .value to a string, it will update ._num_value to the same
> string. If we say .num_value can only be a None or a Number, this will
> not match the idea.

But that is ok if num_value is known to be only numeric values or numbers.

> To make that happen, we still need to go through
> this long check for different type of value.

yes, we will definitely need to check types.  I don't see any way around
that -- that's because paramters have several different types!

> On the other hand, how do we make sure the user do not accidentally set
> .num_value to a number for string parameters, for example PSR, because

You don't allow someone to set num_value to anything other than a number
or a None.  That is something that the setter can check.

> it will update the .value. Maybe run the parameter value type again?
> Or can we put a flag saying if the value is a number or related classes?

We could use a flag, or we could possibly figure it out from some of the
other initialization arguments for Parameter() (like the parse_value field?)

Scott

>
> -Luo Jing
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/luojing1211/PINT/commit/5ca25e27a72cc1dbcb472c5b4ba13f0849f30ec1#commitcomment-15710059>.Web
> Bug from
> https://github.com/notifications/beacon/AAIHIqqSVmY1m3KCYyevGelZ6_J3Qle4ks5peOWFgaJpZM4HNCac.gif
>

--
Scott M. Ransom            Address:  NRAO
Phone:  (434) 296-0320               520 Edgemont Rd.
email:  sransom@nrao.edu             Charlottesville, VA 22903 USA
GPG Fingerprint: 06A9 9553 78BE 16DB 407B  FFCA 9BFA B6FF FFD3 2989


I would advise against saving generated files in version control. Instead you could use the `master` branch as usual and only push the `dist` directory to an orphan `gh-pages` branch.

Here’s a simple example of how that could work: https://github.com/transparantnederland/browser/commit/a166348e105cfa730a8dd465b217fd3ba4d7f896


сортировать - в отдельный  li


home

store

cosmatics

food

burger

pepsi

choclate

dairy products

products


whoops, misspelled the title


@gabrielamatias can we change this to "if you're unsure, please apply - we'd love to hear from you" ? Since they'll be asked about CV and cover letter on the application page anyway. We can ask them the fintech question then too.


закрыть div


Nitpick-Change it to client_original_seq_tbl_.  Add a comment "// <client_id, original_seq>".


@gabrielamatias three changes and then i'm done, promise!! thank you!


:joy:


Snap is kind of useful


hspec is kind of useful


Aeson need ceral


This module for configuring and locking the IMRs to enforce a particular policy is logically distinct from the basic driver for accessing the IMR registers.


Nitpick - move it above o_seq. Pair the accessor and mutator for each variable.


Ah I see. It is escaped, while it shouldn't be. This was already present before this commit, I escaped everything correctly. Fixed: https://github.com/Dennis123789/folio/commit/632efdb013e6400e2f4bf2391490d42cf6d21fb9


Yes these are different cases...


Avocado should behave correctly and not "fork bomb" no matter if `--job-timeout` is used or not.

The point of using `--job-timeout` is that the functional test finishes properly, and it also tests how that part of avocado behaves when things go awry (like in a fork bomb). We could indeed swap the timeout from the avocado job to `process.run` but I fail to see the point. Those functional tests can be run without the fix and they report the failure. Again, the "better behavior" of `main()` (it's hard to call it the "right behavior) does not depend on the timeouts.


Eventually I will make everything support multiline labels :-)


Looks good!


I did a summary about the type of  parameter.value we can get.
Numberable
1. Number(float,numpy.longdouble)
2. Astropy quantity/Astropy Angle (They can treated as the same, coding wise)
3. Astropy time object
Non-Numberable
1. str(PSR/PSRJ,  Binary)
2. bool(Planet-Shapiro delay )
Other type
1. Command parameters (JUMP/START...)


I think I'm at the 'track down slightly annoying stuff phase'. Thanks for the catch! I clearly didn't know how to deal with that one. We'll see if we can get a clean build today.


Hello, can you please suggest how to override these attributes?
I have a main recipe, in which i am adding `node.default['apache_spark']['standalone']['master_host'] = 'foo.com'`

And executing it using chef solo like:
```ruby
run_list(
  'recipe[main]',
  'recipe[apache_spark::spark-standalone-worker]'
)
```

But that does not seem to work.


maybe instead of selecting one over the other, throw an exception if both are there? Since that would probably indicate a bug?


что это за блок? в макете не вижу


file is still missing


Yes that looks very related indeed.


In the upcoming revisions of my protection domain patches I reference the DMA section addresses for a purpose other than configuring IMRs.  Thus, I think the concept of a DMA-accessible memory region should be kept distinct from IMRs.


Ah, I honestly wasn't thinking about simplifying the method - was just avoiding the join duplication. I'll give it a shot, thanks


Setting the PenStyle doesn't build in qt5
should be p.setPen(Qt::SolidLine); and not p.setPen(Qt::PenStyle::SolidLine);


Yeah...I wonder if better method names are necessary? #top_left_x, #top_left_y?


Graph identifies if it is running inside a box in admin/workflows or not (inside a popup window in issues).


Yes, I'll add something similar to the description in the commit message.


You need to add the archive for this version too. Moving the `.txt` file doesn't work.


Don't call atoi multiple times. int client_id = atoi(addr.c_str()); Then use client_id in if and later assignment.


Это все же слайдер и желательно его обернуть в отдельных блок.
Далее слайдер состоит из
![2016-01-27 23-41-40 nerds-index-1000 jpg 1000x1880](https://cloud.githubusercontent.com/assets/8250032/12620647/49dc284a-c547-11e5-85c7-65168e993537.png)


The real-time evolution fails even on the CPU:

    kerneltest.cpp:137:Assertion
    Test name: my_test<CpuKernelTest>::rotating_frame_of_reference_test
    assertion failed
    - Expression: std::abs(ini_tot_energy - tot_energy) < TOLERANCE*10.


Man, who chose _that_ colour?


IMO this approach is no good for long term cross platform goals. It is known that JIT won't be enabled until it is implemented. However, Interpreter mode shouldn't be far away from a usable performance.

See `O3 optimized` LLVM IR below for `inline assembly` vs fallback suggested by this PR.

**Fallback;**
```
define i32 @mul(i64 %left, i64 %right, i64* %pResult) #0 {
  %1 = alloca i64, align 8
  %2 = alloca i64, align 8
  %3 = alloca i64*, align 8
  store i64 %left, i64* %1, align 8
  store i64 %right, i64* %2, align 8
  store i64* %pResult, i64** %3, align 8
  %4 = load i64* %1, align 8
  %5 = load i64* %2, align 8
  %6 = mul nsw i64 %4, %5
  %7 = load i64** %3, align 8
  store i64 %6, i64* %7, align 8
  %8 = load i64* %1, align 8
  %9 = icmp ne i64 %8, 0
  br i1 %9, label %10, label %20

; <label>:10                                      ; preds = %0
  %11 = load i64* %2, align 8
  %12 = icmp ne i64 %11, 0
  br i1 %12, label %13, label %20

; <label>:13                                      ; preds = %10
  %14 = load i64** %3, align 8
  %15 = load i64* %14, align 8
  %16 = load i64* %1, align 8
  %17 = sdiv i64 %15, %16
  %18 = load i64* %2, align 8
  %19 = icmp ne i64 %17, %18
  br label %20

; <label>:20                                      ; preds = %13, %10, %0
  %21 = phi i1 [ false, %10 ], [ false, %0 ], [ %19, %13 ]
  %22 = zext i1 %21 to i32
  ret i32 %22
}
```

**Inline assembly**
```
define i32 @mul(i64 %left, i64 %right, i64* %pResult) #0 {
  %1 = alloca i64, align 8
  %2 = alloca i64, align 8
  %3 = alloca i64*, align 8
  %_ = alloca i64, align 8
  store i64 %left, i64* %1, align 8
  store i64 %right, i64* %2, align 8
  store i64* %pResult, i64** %3, align 8
  %4 = load i64** %3, align 8
  %5 = load i64* %1, align 8
  %6 = load i64* %2, align 8
  %7 = call { i64, i64 } asm "mulq ${2:q}", "={ax},={dx},%0,%1,~{dirflag},~{fpsr},~{flags}"(i64 %5, i64 %6) #2, !srcloc !2
  %8 = extractvalue { i64, i64 } %7, 0
  %9 = extractvalue { i64, i64 } %7, 1
  store i64 %8, i64* %4, align 8
  store i64 %9, i64* %_, align 8
  %10 = load i64* %_, align 8
  %11 = icmp ne i64 %10, 0
  %12 = zext i1 %11 to i32
  ret i32 %12
}
```


thanks!


Are we making the assumption that everyone editing the source code is using vim?


Let me know if you need me to look at anything/test it in my docker container (with address sanitizer :) )


It might be best if you add the latest version (2.3.8) here as well. This lets users install the latest version as a specific version so that their docset doesn't get updated in the future.


@MrBaNkS is this line related to  removing PowerHal ???


Remove comment when done.


I could not find any example where protected is used or documentation that explains, how to migrate this. Could you add a hint for it in the docs?


Remove this when done debugging.


Расскажи, почему ты дал им разные классы?


First Description ng problem, "the client does not know whether if he should take~~" hindi ba parang wrong grammar siya?


Done.
http://jsroot.gsi.de/dev/?nobrowser&json=../files/simple_alice.json.gz
But still it is different from previous version:
http://jsroot.gsi.de/4.1/?nobrowser&json=../files/simple_alice.json.gz
With old version one had seen negative side of Z axis.
Was it intended? I can set other view point for camera.


For information, we reproduce with a colleague the mentioned notices
Imo, these lines should be removed :
https://github.com/glpi-project/glpi/blob/master/install/update_0901_091.php#L173-L199

If an admin has changed his super-admin profiles, with theses, his modification are erased.
If the purpose was to add unlock right to super-admin, we need a more simpler way to do that


is meant to be in this way @giogonzo ?


Rather than passing the context, what about throwing an exception, then logging it elsewhere? This way, `parseTime` will be callable even from context that doesn't implement `trigger`.


yup, looks much better


I was just identifying the commit to show that it is a copy/paste error.  The error you corrected does exist in the master branch.


I actually wrote this many months ago; I just broke it out from my protection domains PR to form part of this PR.  However, I will need to update this once I update the comment as you suggested below, so thanks for the reminder. :)


Yes, of course that's a good idea. I'll do that and move it to KeyValueStore.


no clue, don't know what js engine or browser it is talking about.


лучше задать здесь класс (нужно стилизовать будет) и выбрать другой тег. Т.к. это все же не параграф.


I think I did it right... the previous view is the correct one


@gordonwoodhull will do. I further indented some of the selectors to show their relative position in the hierarchy. I won't do that either.


depending on users--encryption link is quite dangerous practice, as when someone adds new link and forgot this case statement if behave strangely. Better specify third link and raise exception in else with unknown value.


Is using CM's setupwizard working fine?
for use it doesn't start restoring apps process


I handled without differently, this code was a workaround because keys array wasn't uniqued during merge. without is actually a bad deopt at the time it happens, it is used only in NativeArray but happens very early on.


Oh yes, I forgot it did. Carry on.


Merge sha c84549db is stale.


merging rabbitmq/lager/rotate-command = c278368a into borshop-integration-311-rotate-command


rabbitmq/lager/rotate-command = c278368a merged ok, testing candidate = c973e94b


как думаешь, может лучше разделить на две колонки?
![2016-01-27 23-47-54 nerds-index-1000 jpg 1000x1880](https://cloud.githubusercontent.com/assets/8250032/12620851/259f9f24-c548-11e5-9369-6b8cf5f21b92.png)


Аналогично предыдущему. Это не параграф


NP: you can use `users.first` instead `users[0]`


Here's the URL for a GM post on the information I provided.

https://elanthipedia.play.net/mediawiki/index.php/Post:Random_Rare_Rates_-_07/13/2011_-_10:55

And the quote:

Re: Random Rare Rates · on 07/13/2011 10:55 AM CDT	1633
>>You're more likely to get a random rare/very rare drop in a room with common metals than a room with uncommon/rare metals
No, not correct.
Rare metals only have a chance to drop when the % mined is below half. This % increases the closer a vein is to being fully depleted. Also, rare and very rare metal can spawn naturally as entire veins from time to time. Just need to keep an eye out for it!
This message was originally posted in Lore \ General Discussions - Lore, by DR-KODIUS on the play.net forums.


I'm going through the Symfony 3.0 book and at the step when I had to install `symfony demo` I encountered a problem. There is no `bin/console` file. It is still in the `app` directory.


All smbx65-38A are adding into lvlx too. The thing is: because LVLX supports markets, it can have unlimited extra parameters set and it will be understandable by older versions of parsers


`var` один раз.


why stream, not optional?


Good point, thanks, updated.


Fixed


<img width="1094" alt="2016-01-28 12 49 12" src="https://cloud.githubusercontent.com/assets/1846356/12620899/1521367e-c559-11e5-9471-fcc2008587cc.png">
番组列表好像在 Safari 上有奇怪的 bug


I agree with you to remove the lines.

-------- Original message --------
From: Alexandre Delaunay
Date:27/01/2016 17:45 (GMT+01:00)
To: glpi-project/glpi
Cc: "Moron, Olivier"
Subject: Re: [glpi] Lock Mechanism Implementation (ff247d9)

For information, we reproduce with a colleague the mentioned notices
Imo, these lines should be removed :
https://github.com/glpi-project/glpi/blob/master/install/update_0901_091.php#L173-L199

If an admin has changed his super-admin profiles, with theses, his modification are erased.
If the purpose was to add unlock right to super-admin, we need a more simpler way to do that

—
Reply to this email directly or view it on GitHub<https://github.com/glpi-project/glpi/commit/ff247d90000a275f3114b54b4db32640f67b9655#commitcomment-15710815>.


"two additional requirements"


По-моему этот метод нигде не вызывается.


hi


need help on line 23. I thought that reassigning the value of n to the value of i+1 would let me start checking for the second word, but it's not.


@Cis112233
Wow, this dead, decaying body is really skinny! I wish I could be that skinny!


Note to self: I don't quite understand this part, make sure we're testing this.


1st problem, symptoms,  "the client finds it hard to monitor (who/what?) properly" =))


Also done.
http://jsroot.gsi.de/dev/?nobrowser&json=../files/simple_alice.json.gz


file is still missing for me


Тоже артефакт, метод описан ниже.


Thank you for adding comments! Two notes:

1) Java comments for a method or class should be /** */ style comments
2) Methods in java should start with a lower case character

Can you address these changes?


`#top_left.x` sounds great


My main recipe: https://github.com/Vibhuti/chef-main


@jdamerow  Deleted


Thanks! :-)


Better in jmap.json?


:8ball: :mahjong:


I haven't had a chance to try this yet, but  the solution looks reasonable and makes sense. If you want to create a pull request I'll merge it in. Thanks!


Yeah, it wasn't on my radar to simplify it until I was trying to read 200+ characters to figure out what the change was.


Yeah, that was not intended. I fixed that on my next commit.


Wait, is `match` faster or `walk`?


Thanks for the source!


Sure, I'll remove this unused feed.


@nhervieu Check out the `np.where()` function.
```
index = np.where(rgal.value < 1200)
```
then later you can run
```
plt.loglog(mytable['MASS_EXTRAP'][index],mytable['VIRMASS_EXTRAP_DECONV'][index],marker='.',c='b')
```
without a `for` loop.


Здесь и дальше должны быть приватные методы, начинаются с `_`.


Зачем два одинаковых метода? Этот и следующий


Besides proclaimed speed, are there any other advantages over using the one from node?


I just saw this (auto-generated HTML documentation) code base. Do you need that page created on Wordpress to port in or does it generates itself?


@sasha0 I see now that this is not the only issue with Django 1.9. I'll created an issue for this #57, so we need to fix this.


to use flatmap


of course


"... in build time, *but* we feel that the ..."


That may be possible I am not sure


need to discuss line 97, because i cant understand why this line works.


I can connect using your mechanism (and resource 'bot'), but then the server sends something that seems to trigger the client to say its a legacy XMPP 0.9 connection and then i get a disconnect.

```
DEBUG    Event triggered: auth_success
DEBUG    SEND (IMMED): <stream:stream to='chat.hipchat.com' xmlns:stream='http://etherx.jabber.org/streams' xmlns='jabber:client' xml:lang='en' version='1.0'>
DEBUG    RECV: <presence to="***********@chat.hipchat.com/bot||proxy|pubproxy-c200.hipchat.com|5272" from="**************@chat.hipchat.com/qt_windows||proxy|proxy-c203.hipchat.com|5232">
WARNING  Legacy XMPP 0.9 protocol detected.
DEBUG    Event triggered: legacy_protocol
DEBUG    RECV: <show>away</show>
DEBUG    RECV: <query xmlns="jabber:iq:last" seconds="900" />
DEBUG    RECV: <delay xmlns="urn:xmpp:delay" stamp="2016-01-27T16:55:27Z" />
DEBUG    RECV: <x xmlns="http://hipchat.com"><client_type>http://hipchat.com/client/qt/windows</client_type></x>
DEBUG    End of stream recieved
DEBUG    reconnecting...
DEBUG    Event triggered: session_end
DEBUG    SEND (IMMED): </stream:stream>
INFO     Waiting for </stream:stream> from server
DEBUG    Event triggered: disconnected
DEBUG     ==== TRANSITION connected -> disconnected
```

Did you encounter this or work around it? When I connect with plain auth, I don't get the legacy warning or disconnect.


I think it's `D`, for gh handle is case-sensitive.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


I thought about that quickly, and concluded that at the end we want to catch any IntegrityError at this stage (given we then display the error message to the user). What do you think?


We want the behavior to log the warning but keep going so throwing an error wouldn't work.

However I see your point about the contract -- I considered making this a simpler onError callback and leaving it up to the adapter to emit the event. That's probably cleaner and would enable better unit testing of parseTime.


I went back and forth on that, but I see your point. I'll work on it.


I'm not really a fan of writing settings/methods that are an all in one get and set. I think get_output_state and set_output_state is more appropriate. Then the setting has one and only one job. Any setting that sets something on an instrument should then query the instrument to ensure that this actually went through. This may sound stupid but instruments are strange and sometimes they won't you to set things based on the instruments current configuration. You can try to put as many of these checks in software as possible but at the end of the day, querying it is best.


fast-forwarding master to borshop-integration-311-rotate-command = c973e94b


It's the base for what will be the documentation of Natron. Basically it auto generates pages for each Node based on their description from the OpenFX specification (parameter list, their tooltip etc...) Then whenever you will navigate the documentation, Natron will automatically generate pages for you.

Note that we will also include in that the user guide. We will soon start writing the table of content for the user guide and will need the help of the community to actually fill-in the content for each section of the user guide.
It will be uploaded on a separate github


This is just a start, we will generate HTML's for local and online use.


@nhervieu -- Here, let's use `mytable['MASS_EXTRAP']` for the cloud rather than the virial mass since the virial mass is defined as being proportional to `sigma**2 * R` where `sigma` is the velocity dispersion.


header & footer


So I'll keep this helper, using the revised functions? I'd like to go over this with you, because this is a mess and I'm sure you can tell me how to do it more elegantly.


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


Previous approach didn't allow for a value of "0"


It works in the browser natively without needing the browserify shim.

-m

> On Jan 27, 2016, at 8:56 AM, Balázs Kutil <notifications@github.com> wrote:
>
> Besides proclaimed speed, are there any other advantages over using the one from node?
>
> —
> Reply to this email directly or view it on GitHub.
>


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


Here, we need to discuss whether lcg-lr return logicalFile or vector<SE>. If we want lcg-lr to return vector<SE>, then we need to add new method in LFCMessage.


lol


It looks like $quoted_v would not be present if $v == git


![screenshot 2016-01-27 at 08 59 04](https://cloud.githubusercontent.com/assets/54884/12621245/8ad5641a-c4d4-11e5-874a-273e93e5f079.png)

is how https://travis-ci.org/git/git/branches is shown to me. I do not quite get why the tip of 'master'
shows that commit. There is some confusion between a proposed change and the history of the
project proper?


Ha, thanks, yes! This is a good idea!

@por And is there a way to tell Webpack to leave CSS URLs (images, fonts) alone?


`--package_path` should not be used in common. It will break `version` and `help` and some other subcommands.


Currently, tests keep re-using `opts` and we previously assigned `wrap_attributes_indent_size = 8`


Good catch, thanks.


I got into issues here. The output I receive to `$ which rhino` is `/usr/bin/rhino`. Actually I can run it from command line. but Emacs cannot find the engine and opening a `js-sub-process`. So `Mx- run-js` it doesn't.
Any help, please??


Uh, maybe we'd want to use a transparent png?


remove if not needed


@xyyx thanks! pushed it to gerrit already


The actual merging issue is caused by pervious "revert" commits in master branch: I wrongly committed changes to master, @yzhwang helped move them into dev and revert them in master; when merging @wetliu 's branch (which was based on master) into dev, the reverts took effect in dev too (because reverts are more recent than the original commits), resulting in lost of update in original commits.
The step took to resolve this was reverting the revert commits, which essentially enabled changes from the original commits.
Lesson? Start from dev branch, instead of master, when starting new development and not absolutely sure that should directly apply to master. Merge back / pull into dev also.


As a general rule we try to avoid forward declarations when possible.  If you reorder `zvol_create_minors_cb()` so it is defined before `zvol_create_minors()` you can remove both of these.


I didn't run into this; can you figure out what is triggering the `legacy_protocol` event? If the stream doesn't contain a `version` attribute the spec says you should assume 0.9, but it looks like there's one there. Is there something else being sent that's not being logged?


No clue about how closure (or groovy) works - maybe it is better to get Jono or Jorge to take a look at this one?


@xyyx didn't you miss it? http://gerrit2.xenonhd.com:8081/#/c/1599/


it looks good to me, but I have never looked at the groovy jobs. Maybe you want someone else to take a quick look for peace of mind?


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 133](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=725&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:37:02


Oh yes, this is much, much better. Thank you!


Are you using sleekxmpp 1.3.1 or one of the branches?


Might as well just try and see if it works @xiam -- shouldn't break anything newer at least!


I would suggest using an even more permissive license for this if you want high adoption.


Not sure if there’s a default setting. You could probably also just use an ‘empty’ Webpack loader for those kind of files.


are u keeping those comments for u to remember. rails projects / developers don't likey comments. :)


I'd suggesting bring this all in a level by changing it to..

```
if (zvol_inhibit_dev)
        return (0);

parent = ...
```


perfecto!


don't need the string interpolation here


> Are you using sleekxmpp 1.3.1 or one of the branches?

This branch, so whatever it's forked from (master at the time I made the PR, presumably).


same here with the comments.


With regard to talking about other distance sampling in R packages early on, I don't think this is a good idea. This is a paper about our software -- we should do due dilligence to talk about other people's packages, but to talk about them early on distracts from our stuff: the point of the paper.

If the point transect stuff is confusing then we should improve it. I think it's essential to keep points in there and give them equal exposure to line transects. In a lot of DS papers line transects are king and points get second class treatment, I think we need to avoid that here and show that `Distance` treats them as equals.

I also think that the basic intro to distance sampling stuff needs to stay if we intend to submit to JSS, as no-one there will know anything about distance sampling (see e.g. the [`unmarked` paper](www.jstatsoft.org/article/view/v043i10/v43i10.pdf)).


After I wrote I went back and looked and noticed you're off the top of the current develop branch. I'll try that and see if that fixes my problem. Sorry!


It would be better to use Kb instead of K, same for Mb, Gb, ...


This is the only thing that's actually changed in this file!

The rest is the autoformatter removing / adding spaces :)


But what if @options[:root]?


:+1: nice


Easy peasy - if this comes out as True (i.e., it is not False) then fail the test straight away.


Most of this commit is cruft you can ignore - just config files and whatnot.


Make these all abstract. Note that clients should return null if there are no known algorithms (i.e., never been connected).

Also rename this to "getKnownKeyAlgorithmsForServer"


![uuuuuuuu
 20160116_224354](https://cloud.githubusercontent.com/assets/16417822/12621567/2654b982-c547-11e5-9aa8-22eacfe1f809.jpg)


It's actually a much better idea to make each test as small and specific as possible - then if any fail, we immediately know exactly what's going wrong!

Note how I've named them in the format "given x, and we do y, we should expect z"


I'm ok with this, but this was not the original idea of Antoine's PR.
Did you see with Inbox team that ThreadId and mailboxIds should not be always present?


LINQ is awesome!

And since we've already written four tests showing that the left is correct, and the right still passes them, we know all is well :)


reverting the revert, sounds like a good name for a band, or album.


If you want to, you can actually reduce that code even further, but this may take some thinking! Have a look at the `maybe` function.

    let deposits = maybe [] accountTransaction $ Map.lookup accountToLook accts

maybe has this type:

    maybe :: b -> (a -> b) -> Maybe a -> b

the return type is noted as `b` here... so for us that would be [Transaction]... and it takes a function from (a -> b), and for us that is (Account -> [Transaction]), and then a `Maybe a`, which is for us what Map.lookup is returning... (Maybe Account).

What it will do is the same as that case statement. It will evaluate the Maybe a, and if it's nothing, it will return the first argument. If it's Just something, it will apply the second argument (the function) to it... which is exactly what we wanted to do..

That's if you *want* to use this. It's a very cool function. Takes some getting used to and thinking about though.


Yup.....it is there.....i succeed


@pfalcon ok, great!  I'll write a note to put it in main comment of v1.6 log.


other array tests are in `standardClasses/`


@andrewgarner I know we'll have to clean up the `wait_for_ajax` stuff, but that dataLayer stuff is pretty crazy!


Thanks for the interaction here by the way.. Jumping to develop doesn't seem to help me. I must be doing something wrong..

I pip uninstalled sleekxmpp twice, changed my requirements.txt line to
-e git://github.com/fritzy/SleekXMPP.git@develop#egg=SleekXMPP

I confirmed that's the develop branch in my virtualenvs

Yet I still get the same disconnect. I'll keep poking at it..


@jmchilton seems to work with:
docker run --rm -p 8080:80 -i -t --privileged=true quay.io/bgruening/galaxy-openms

and this is the Dockerfile: https://github.com/bgruening/docker-recipes/blob/master/galaxy-openms/Dockerfile


la route qui disparait c'est probablement une histoire de spu ou de LLE mal émulé.


I think just using the fallback is definitely not the way to go long term yeah. Whether we want to just inline assembly or add some existing mul128 implementation with fine tuned assembly for different platforms, not sure. I thought the focus was first to get things compiling and then improve from there, but we can definitely do everything in one go here for sure.

If you ask me I'd search for a solid mul128 implementation and add it as a new file in here to be used by Linux/Unix ports instead of inlining assembly.


This would be a fraction clearer:

    let maybeAccount = Map.lookup accountToLook accts
        deposits = maybe [] accountTransaction maybeAccount

If you think this is harder to understand, then don't worry.


No. The preview needs to have no HTML extensions to work. Do not fool with it.
I was going to do color scheme. I have changes in my working branch.
I scrolled down. It still needed work.
Never attempt to use that color scheme again.


Let me try this with rubymine.


I haven't checked the api. Do you have and example of how `artist`/`artists` are used? (just wondering if it makes sense to combine them)


Don't even.


You didn't use this rule in the CSS or JS. Why even name the class?


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


@batmat I seem to have lost my push rights to this repo now:

```
ERROR: Permission to jenkinsci/phabricator-plugin.git denied to sectioneight.
fatal: Could not read from remote repository.
```


We probably want to document the `bbox` configuration option on deployment/installation in the `docs/` folder.

/cc @reve0716


hmm j'y est pensé mais je pensait plus a un probleme graphique, mais d'un coup elle réapparait lol donc je sais pas, je vois pas comment debuger ca, si toi tu sais, la route disparait toujours au meme endroit donc plutot simple a trouvé, c'est pas un truc aleatoire ^^


This morning there was a planned outage for the Total Connect Comfort web server. However, after the server came back online again I don't see my Evohome updated using the evohome-client2 script . It looks like the web server can not be reached anymore. Has Honeywell switched to another server?
The iOS app is updating itself with the latest info via the Toal Connect Comfort server.


If necessary, I can make a patch.
But I think the information about the module in the second infobare enough.
skins Magic(SD), PLI-HD and PLI-FULL HD


The fixups for this are a bit weird as there are subsequent commits which change the spec file.

The first (main) round of fixups to address the problems flagged here is [a5f05b8](https://github.com/mysociety/alaveteli/compare/6a3c8b2...a5f05b8#diff-8)

Then there are other fixups to keep things up to date:
* [9d05dc0 ](https://github.com/mysociety/alaveteli/compare/7e0477b...9d05dc0#diff-5) fixup! Split ResponseController#show_response into individual actions
* [6f454e1](https://github.com/mysociety/alaveteli/compare/af98b71...6f454e1#diff-5) fixup! Prevent success message from being shown for duplicate messages
* [aaca02a](https://github.com/mysociety/alaveteli/compare/d894a4f...aaca02a#diff-5) fixup! Fix handling of followups to closed requests
* [e6ec79f](https://github.com/mysociety/alaveteli/compare/8f24155...e6ec79f#diff-1) fixup! Add ResponseController#preview


I've removed the B to use it on states too. States are number of states instead of bytes.


oke, so if I understand it right it will give a account or a empty list ?


with my implementation ( https://github.com/SickRage/SickRage/pull/772) there is no problem because i'm using terms of API.
Best regards


The compiler can't find the elc.h header

"elena-lang\elenasrc2\elc\vs\elc11.vcxproj" (default target) (1) ->
(ResourceCompile target) -> resource.h(1): fatal error RC1015: cannot open include file 'elc.h'. [\elena-lang\elenasrc2\elc\vs\elc11.vcxproj]"


Why the "WTF"?

And why did you want to decompile mcpe


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1253](http://teamcity.monogame.net/viewLog.html?buildId=14077&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


i think this needs to be "1.3.0" i get "TypeError: Invalid Version: 1.3" when i run "bower update angular-aside-menu"


Gibt es ein Issue dem die Gründe für diese Änderung zu entnehmen sind?


Oh I missed the further indenting. That's not a bad idea. (I guess you have seen how disorganized our main css is, the incomprehensibly named bootstrap-rcloud-tweaks.css.) I just auto-formatted the whole thing to get rid of the tabs...


When I type in qt:expand I see the Alfred prompt, but when I complete qt:expand it doesn't actually do anything and the option disappears to expand a template. Here is the debug information:

    [ERROR: alfred.workflow.input.scriptfilter] Code 1: Templater: : getDirectoryContents: does not exist (No such file or directory)
    [INFO: alfred.workflow.input.scriptfilter] <?xml version="1.0"?><items>
    [ERROR: alfred.workflow.input.scriptfilter] XML Parse Error 'The operation couldn’t be completed. (NSXMLParserErrorDomain error 5.)'. Row (null), Col (null): 'Extra content at the end of the document' in XML:
    <?xml version="1.0"?><items>


Dobytku!!


Looks like the version in npm wasn't built correctly. The lib directory is missing.


par contre, pour les voiture doré sur le menu tu a une idée ? la sur ta branch d3d12 c'est tout cassé donc peut plus tester la ^^


On topic of not made pull requests: would you mind commenting on http://forum.drdteam.org/viewtopic.php?f=24&t=6584 - at least telling why noone else seems to be seeing the problem.

That hack is kind of silly, as it enforces my bias towards 4:3 display of the 320x200 content, yet that aside, yet it's at least a proof of concept (and you might find my stab at moving savpic offscreen in the latest version useful).


@kbenne your cmake-fu is stronger than mine


Closes #1


Yeah it was just there for the color scheme and yeah it still needs a bit of work for the header but that is mostly making it more slick and such


ok, thanks


Oh, cool :)


`add(pattern(ARRAYS, "<init>(Int,Function1)"), arrayFromFun);` doesn't work?


Let me know if you find anything; I don't have time to test this right now, but I didn't notice any problems before. It could very likely be something I broke though.


Cpom, this broke it. Shit


No, that's impossible - they're different types. A function can't return different types.

What it actually does is applies accountTransaction to the value after unwrapping the Just... if it's a Nothing, it returns [].

It might be easier to see with a simpler example... these are equivalent expressions:

    case (Just 8) of
      Just x  -> (+2) x
      Nothing -> 1

    maybe 1 (+2) (Just 8)


could use a different name and an explanation. I have no idea what this means and what it would contain:\


yeah, I got those the wrong way round :disappointed:

is fixed... somewhere, most clearly by a5f05b8


I put in some more debugging and it looks like something is getting into the start stream handler that shouldn't.

```
DEBUG    RECV: <stream:stream version="1.0" from="chat.hipchat.com" id="6969495c879b00cd">
INFO     6969495c879b00cd
INFO     1.0
INFO     False
....
DEBUG    SEND (IMMED): <stream:stream to='chat.hipchat.com' xmlns:stream='http://etherx.jabber.org/streams' xmlns='jabber:client' xml:lang='en' version='1.0'>
DEBUG    RECV: <presence to="*****@chat.hipchat.com/Ym90||proxy|pubproxy-d200.hipchat.com|5252" from="*****@chat.hipchat.com/qt_windows||proxy|proxy-c215.hipchat.com|5282">
INFO
INFO
INFO     False
```

first info is stream id, 2nd is version, 3rd is is_component.. In the previous stream statement, i gets version 1.0 and a correct id.
so it looks like because hipchat sent that stream to with no id, it made things angry and didn't process the version?


Thanks for doing this @kmetsalu - silly oversight on my part!


It looks like `@init` is looking for a variable called `Escher.__inits__` while it should be looking for `Requires.__inits__`...

This problem does not occur on my computer (Julia 0.4.2)

@one-more-minute, what's the correct way to use `@init` ?

I had to resort to:
https://github.com/shashi/ComposeDiff.jl/blob/master/src/ComposeDiff.jl#L547-L549

in ComposeDiff package for which I *was* getting this error on my computer.


:exclamation: Назначение backround'а ВСЕМ, без разбора, контейнерам.


This commit looks like a great improvement. Does any genius can introduce how will the ACTION_T_DYNAMIC_MOVEMENT strengthen the creature AI? Thanks so much.


Don't forget to change the date to 2016


bb


:exclamation: Изменение поведения класса со строгой семантикой. Как-то не клеится.


Ok, I forget than the "vs_release" is just for release mode! sorry!


hi


Ich glaube nicht, dass im zweiten Hauptsatz das Wort "Wärmepumpe" benutzt wird


Makefile already updated to include the library,
on Linux systems `cc` is `gcc` anyways.

Rejecting


+1 - this should be in!


Da ist ein Komma zu viel


Excuse me, what did you fork this for?


*Verhältnis


I think the `extern` keyword suggestion in https://github.com/JuliaLang/julia/pull/6884 could help deal with this.

Not sure this one method would merit an IJuliaBase (or some such) package, there's not much other stuff that can be abstracted out.


No problem, I've used SASS quite a bit recently which can help with unruly CSS.


Good catch! But those `!=` comparisons should be `==`. I'll update the article.


OK. I didn't realize that localField access is so much faster than
this.member, but OK. I learned something new today!

On Wed, Jan 27, 2016 at 8:02 AM, Tim Fennell <notifications@github.com>
wrote:

> I'm not sure I understand your comment. The point of doing this is to turn
> it into a local reference so that there's no indirection since it is
> accessed many times in the loop below, that's all.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/broadinstitute/picard/commit/187dcc5a956fc0fd8ddbc020cc7acd40984d7987#commitcomment-15704844>
> .
>


Understood. I'll be more careful about commit messages and code commenting from now on.


:exclamation: Изменение класса Twitter Bootstrap'а, кроме того назначение свойства изменяющего внешний вид абсолютно всех элементов которым назначен данный класс.


I suspect I'm in over my head. I confirmed that the presence line is getting into this section

```
1525     def __read_xml(self):
1526         """Parse the incoming XML stream
1527
1528         Stream events are raised for each received stanza.
1529         """
1530         depth = 0
1531         root = None
1532         for event, xml in ET.iterparse(self.filesocket, (b'end', b'start')):
1533             if event == b'start':
1534                 if depth == 0:
1535                     # We have received the start of the root element.
1536                     root = xml
1537                     log.debug('RECV: %s', tostring(root, xmlns=self.default_ns,
1538                                                          stream=self,
1539                                                          top_level=True,
1540                                                          open_only=True))
1541                     # Perform any stream initialization actions, such
1542                     # as handshakes.
1543                     self.stream_end_event.clear()
1544                     self.start_stream_handler(root)
```

and then into
```
   202     def start_stream_handler(self, xml):
   203         """Save the stream ID once the streams have been established.
   204
   205         :param xml: The incoming stream's root element.
   206         """
   207         self.stream_id = xml.get('id', '')
   208         self.stream_version = xml.get('version', '')
   209         self.peer_default_lang = xml.get('{%s}lang' % XML_NS, None)
   210
   211         log.info(self.stream_id)
   212         log.info(self.stream_version)
+  213         log.info(self.is_component)
+  214
   215         if not self.is_component and not self.stream_version:
   216             log.warning('Legacy XMPP 0.9 protocol detected.')
   217             self.event('legacy_protocol')
```


Should LongType, etc be supported ?

I can send a PR if that is the case.


capitalization sensitivity is the devil
i tell you


Note: (since can't put TODOs in JSON): This was to fix the Visual Studio compiler (still using DNX) even though dotnet compile works just fine. We can removed this TFM once VS tooling updates.


:exclamation: Вложенные контейнеры.


:exclamation: Назначение класса контейнера всем без разбору элементам.


OK cool will work on that then :-) Good point on points!


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


PS that link to unmarked paper did not work for me


@saghul you're missing the support for PyPy 4.0.1 **portable version**, which is supposed to be installed on platforms which are not Ubuntu, to quote the PyPy download page:
>stating it again: the Linux binaries are provided for the distributions listed here. If your distribution is not exactly this one, it won't work


:exclamation: Если что-то откуда-то копируем, то хотя бы убираем то, что не относится к новому макету или секции.


Updated the link, should work now :)


No. Double define.


Thanks for your code!! But It still doesn't work so I tracked it what's the problem was.. local player = go:GetLootRecipient() doesn't return any value I guess.. If I remove the next condition (if player then) and I got nil value on the player from return value of go:GetLootRecipient()..


:exclamation: После расширения файла, никаких _/750х450_ не может быть априори (_только если файл не является скриптом_).


Pre-built DLL only works with 1.34 of Steamworks and last build of SteamworksPy.  Please compile from scratch until this file is updated.


Runtime: O(n)


@Makman2 interesting detail:

If you do `tr = TextRange.expand(self, file_contents)` in a **SourceRange** and you pass **self**...

... you will end up with `TextRange.expand()` calling `return self.__class__.from_values(...)`.

notice the self? yup, that is `SourceRange`'s self... and `.from_values` is called with the wrong parameters...

m(


// FIXME: This is horrible, find a better way to do this


\REBELinBLUE\Deployer\Console\Commands\ResetApp::class, // FIXME: Figure out if we can overwrite the constructor and only add this in dev mode


// TODO: See if we can get these from StoreUserRequest somehow?


// FIXME: This is horrible!


The problem here is that I use the value to set the SDK as well, and I don't have that 10.7 SDK anymore. The older one I could find was 10.9. Does this change work in your machine?


It's probably easier to let you paste the compile errors than to get 32-bit godoc running ;)

> On Jan 27, 2016, at 7:15 AM, petevine <notifications@github.com> wrote:
>
> 32-bit discrimination! ;)
>
> # github.com/lunixbochs/usercorn/go/native
> go/native/statfs_linux.go:9: cannot use s.Type (type int32) as type int64 in field value
> go/native/statfs_linux.go:10: cannot use s.Bsize (type int32) as type int64 in field value
> go/native/statfs_linux.go:17: cannot use s.Namelen (type int32) as type int64 in field value
> go/native/statfs_linux.go:18: cannot use s.Frsize (type int32) as type int64 in field value
> go/native/statfs_linux.go:19: cannot use s.Flags (type int32) as type int64 in field value
> make: *** [usercorn] Error 2
> Have you seen this funny bug? in usercorn?
>
> —
> Reply to this email directly or view it on GitHub.
>


great explanation - it's very clear, without feeling over-simplified or dumbed down.


Does the test fail if you leave it out?


oke, so it gives [] or a all Transaction in a list.


Yes, I think that could help. I haven't used it myself but I have a PR on one of my other projects where it cleans things up a lot.

I'll open an issue on modernizing our build system.


please re-check since i run ```npm run start``` doesn't show any thing.
@kamilkisiela http://localhost:8080/webpack-dev-server/ show some file? is that ok? or it should show page like http://angular-formly.com/ ?
Sorry i just new, i wont break any thing intensionally :worried:


@Makman2 which is not an Issue.... because you don't really need `self.__class__` or `type(self)` anymore anyways. Since this function needs to only return TextRanges now, a static `TextRange.from_values(...)` works like a charm :)


yeah, this was a hasty refactor when I added enable/disable here (initially was `return {DOM: toggle.DOM}`). interestingly, it appears to work as written, for some value of "work" including "updates the full DOM when toggle.DOM changes".


lowercase `d` is fine, I can't figure out how to change it on my github account:(


I'm not sure this is the right taxonomy.


Actually, replace `@` with `'`.


nvm it finally worked, I'm now @dignifiedquire :)


Hey, enjoy it while it lasts! :)


What is making it more slick? I seriously couldn't understand this message.


lol nice...


What about?
```
Model.where(active: true).fetch_multi(object_ids)


let me try phantom quickly, I suspect this was for a browser we don't support


5 stars :)


phantom 1.x and 2.x seem fine, i suspect we are good to kill this


This change has an out-of-band task for adding the variables `SUPPORT_EMAIL` to the build-environments stored within secrets.


Ahhh, this is what I was assuming :)


Haha, sorry about the line endings. :-1:


I think John was looking at obfuscated JS programs, like: http://patriciopalladino.com/blog/2012/08/09/non-alphanumeric-javascript.html

Just for fun, I'd guess.


I mean $12, not "2"


I think we should skip using that specific SDK if we can get away with it -- I'll test that directly with Derek.


This should be `dsname` not `name`.  This was causing the failure in zconfig.sh because __zvol_create_minor() was failing correctly with EEXIST.  After making this change zconfig.sh works properly for me.


@macdaddyaz  Why thank you!  The file looks nice too!


my gut feeling is that this expression is likely costly, should we attempt to special case the NativeArray case ? We can follow up with a later PR, im just using this as a good opportunity to discuss


I agree, this should be removed. I removed it earlier today as a test but
it doesn't fix the bottom fiducials. I will be working on that next.

On Wed, Jan 27, 2016 at 1:33 AM cri-s <notifications@github.com> wrote:

> sorry, the diff displays wrong, inside fiducialLocator there is this old
> code that should simply be removed.
> the check for getSide() and inversion of coordinates.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openpnp/openpnp/commit/52c8351e7721df0c0cd6b555530ae767e00259dd#commitcomment-15700334>
> .
>


Dur, didn't realize I was reading compiled code until later down :/