[R/feature_ignore_singletons.R:41:15:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_ignore_singletons.R#L41) *style:* **Commas should never have a space before.**
```r
    segsites[ , !is_singleton]
             ~^
```
[R/feature_ignore_singletons.R:52:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_ignore_singletons.R#L52) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:56:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_ignore_singletons.R#L56) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:60:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_ignore_singletons.R#L60) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_recombination.R:54:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_recombination.R#L54) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.recombination <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_selection.R:191:36:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/feature_selection.R#L191) *style:* **Commas should never have a space before.**
```r
                        n_pop, " " , paste(start_freq, collapse = " "), " ",
                                  ~^
```
[R/feature_sumstats.R:18:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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:19:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/import_popgenome.R#L19) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.GENOME <- function(data, only_synonymous = FALSE, ...) {
^~~~~~~~~~~~~~~~~~
```
[R/import_popgenome.R:43:43:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/import_popgenome.R#L43) *style:* **Commas should never have a space before.**
```r
      seg_sites <- bam[c(pops, outgroup), , drop = FALSE]
                                         ~^
```
[R/interface_abc.R:86:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/interface_abc.R#L86) *style:* **lines should not be more than 80 characters.**
```r
        warning("Value of the ", stat$get_name(), " statistic is not a vector. ",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/model_getters.R:75:21:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/model_getters.R#L75) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                   ~^
```
[R/model_getters.R:75:60:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/model_getters.R#L75) *style:* **Commas should never have a space before.**
```r
  if (sum(llm[group , c(1:2, 4:5)]) == 0) return(llm[group , 3])
                                                          ~^
```
[R/model_getters.R:82:25:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/model_getters.R#L82) *style:* **Commas should never have a space before.**
```r
  min(which(cumsum(llm[ , "number"]) >= locus))
                       ~^
```
[R/model_getters.R:125:46:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/model_getters.R#L125) *style:* **Commas should never have a space before.**
```r
  numbers <- get_locus_length_matrix(model)[ , "number"]
                                            ~^
```
[R/model.R:126:39:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/model.R#L126) *style:* **Commas should never have a space before.**
```r
  sum(get_locus_length_matrix(model)[ , c(1:2, 4:5)]) > 0
                                     ~^
```
[R/parameter.R:166:11:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/parameter.R#L166) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if ((!is.numeric(value)) || length(value) != 1) {
          ^
```
[R/segsites.R:62:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/segsites.R#L62) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.matrix.segsites <- function(x, ...) get_snps(x)
^~~~~~~~~~~~~~~~~~
```
[R/segsites.R:75:19:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/segsites.R#L75) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  create_segsites((matrix(c(1, 1, 0, 1, 1,
                  ^
```
[R/segsites.R:142:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/segsites.R#L142) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.default <- function(data, ...) {
^~~~~~~~~~~~~~~~~~~
```
[R/segsites.R:147:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/segsites.R#L147) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.list <- function(data, ...) {
^~~~~~~~~~~~~~~~
```
[R/simulator_class.R:70:44:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_class.R#L70) *style:* **Commas should never have a space before.**
```r
  grouped_commands <- unique(sim_commands[ , 2])
                                          ~^
```
[R/simulator_class.R:73:36:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_class.R#L73) *style:* **Commas should never have a space before.**
```r
    sum(sim_commands[sim_commands[ , 2] == cmd, 1])
                                  ~^
```
[R/simulator_ms.R:67:37:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_ms.R#L67) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                    ^
```
[R/simulator_ms.R:87:39:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_ms.R#L87) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                     ~^
```
[R/simulator_ms.R:87:50:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_ms.R#L87) *style:* **Commas should never have a space before.**
```r
        paste("ms", sample_size, cmd[ , 1], cmd[ , 2])
                                                ~^
```
[R/simulator_msms.R:109:41:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_msms.R#L109) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_msms.R:109:52:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_msms.R#L109) *style:* **Commas should never have a space before.**
```r
        paste("msms", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/simulator_scrm.R:83:41:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_scrm.R#L83) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                       ~^
```
[R/simulator_scrm.R:83:52:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/simulator_scrm.R#L83) *style:* **Commas should never have a space before.**
```r
        paste("scrm", sample_size, cmd[ , 1], cmd[ , 2])
                                                  ~^
```
[R/sumstat_ihh.R:53:45:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/sumstat_ihh.R#L53) *style:* **Commas should never have a space before.**
```r
                      iHS = data.frame(ihh[ , 1:2], iHS = rep(NA, n_snps))))
                                           ~^
```
[R/sumstat_ihh.R:61:55:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/sumstat_ihh.R#L61) *style:* **Commas should never have a space before.**
```r
            data.frame(ihh2ihs(ihh, freqbin)$res.ihs[ , -4])
                                                     ~^
```
[R/sumstat_seg_sites.R:51:38:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/R/sumstat_seg_sites.R#L51) *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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-feature-ign-singletons.R#L14) *style:* **Commas should never have a space before.**
```r
                    create_test_segsites()[ , -c(1, 3)]))
                                           ~^
```
[tests/testthat/test-model-cache.R:47:1:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-model-cache.R#L47) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test-model-class.R:241:44:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-model-class.R#L241) *style:* **Commas should always have a space after.**
```r
               data.frame(name = c("theta","tau"),
                                           ^
```
[tests/testthat/test-segsites-class.R:30:31:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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-seqgen.R:274:27:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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:10:60:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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:19:42:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L19) *style:* **Put spaces around all infix operators.**
```r
                                      dim=c(2, 2, 3) + 1))
                                        ~^~
```
[tests/testthat/test-sumstat-jsfs.R:28:60:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L28) *style:* **Commas should never have a space before.**
```r
                                                   1, 0, 1 ,0 ),
                                                          ~^
```
[tests/testthat/test-sumstat-jsfs.R:28:61:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L28) *style:* **Commas should always have a space after.**
```r
                                                   1, 0, 1 ,0 ),
                                                            ^
```
[tests/testthat/test-sumstat-jsfs.R:29:54:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L29) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                    ~^~
```
[tests/testthat/test-sumstat-jsfs.R:29:62:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L29) *style:* **Put spaces around all infix operators.**
```r
                                                 nrow=7, ncol=4, byrow = TRUE),
                                                            ~^~
```
[tests/testthat/test-sumstat-jsfs.R:34:42:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L34) *style:* **Put spaces around all infix operators.**
```r
                                      dim=c(1, 5, 1) + 1))
                                        ~^~
```
[tests/testthat/test-sumstat-jsfs.R:55:41:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L55) *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:55:49:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L55) *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:60:41:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L60) *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:60:49:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L60) *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:61:41:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L61) *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:61:49:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-jsfs.R#L61) *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-trees.R:6:30:](https://github.com/statgenlmu/coala/blob/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/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/b3ab6cf5e991fb3c8c6e4fd4d3698660ba889c7e/tests/testthat/test-sumstat-trees.R#L74) *style:* **Commas should always have a space after.**
```r
                                            distance = c(2,4)))
                                                           ^
```


Yes, if you can simply add it to the constructor, the AppFramework is clever enough to be able to create an instance for you to use without having to create one first in `application.php`. That would be the best way to do things.


not so much! replaced both log and exception messages with something related to ACL update


= f.text_fieldじゃなくて= f.text_areaのが適切


Well done :)


ここも！


ハッシュロケット残っとるで！


it is actually package private. added @VisibleForTesting


You make a good point re failing fast: should we change this to more closely match `URI`?

```ruby
URI(' foo')
#=> URI::InvalidURIError: bad URI(is not URI?):  foo
```


ビューの適応のブランチやのにマイグレーションファイル作ったらあかーん


rollbackしよう！


-1 ... we should not be doing #ifdef with no content to have #elif following - you have a corrected patch file from me already that does the original change correctly ...


= if じゃなくて- if


=じゃなくて-


great, thank you! ;)


嗯，这里确实内存出了问题。
不设置bg_error，说明当前只有一台cs比较慢，sdk的写还可以继续下去，等文件关闭后，慢的这台会被gc掉。


perfect, thank you! :smile_cat:


ビューの適応のブランチやからdevise入れたらあかーん


А еще можно любую строчку откоментить )


Вот тут например начал филонить уже со стилями...


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2377 of 2397 tests passed, 0 test warnings)

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


Actionは処理の後にAction名のViewを見に行ってしまう。
対応方法1：処理後に他のActionに流す
対応方法2：処理後に他のviewをに流す
今回は方法2を採用


```
#!/usr/bin/env bash
```
... не под всеми системами работает. Точно не помню, где. Но способ не универсальный.


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2375 of 2397 tests passed, 0 test warnings)

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


It's not really that, since we supported #ARGB before. I would rather mention something about standard CSS format


Works like a charm for Tiled, good job! :)


Is it still a log level agnostic logger after this change? :)


@craig-mulligan removed unnecessary styles


removed


I think I'll want an Enum in addition to the other three fields.

I want those for a command help display system, and then we'll use the enumeration for testing what action was actually issued.  Does that seem reasonable?


Thanks for cleaning this up.  It is definitely worth doing it now rather than having difficult to maintain code later.

However, I think in this case the thing that is being parsed is a calibration file, not a config file, which was my point.

Anyway, it is probably clear enough at this point so maybe not worth going through this process again.  Only if you think it is worth it.


please include the (C) 2009-2016 OpenELEC line here too (above) and add your email adress from below after your name


What is this supposed to fix?


Obrigado @forsureitsme :)


I think that it would be better to be able to confirm deck.
Because opponent doesn't know that card can't be Special Summoned or added to hand.
http://yugioh-wiki.net/index.php?%B8%F8%B3%AB
デッキからサーチ･リクルートしたくてもカードがデッキに存在していなかった場合、本当に存在していない事を確認するために相手はデッキの公開を求める事ができる。
相手に《マジシャンズ・サークル》を発動された場合や、サーチ系を使った際チェーンでデッキから条件に合ったカードを失くされた場合などに起こりうる。
デッキやエクストラデッキに目標となるカードが存在しないことが「公開情報から」自明である場合には、確認させる必要はない。(09/08/24)


That is unless it's logger per google rules.  Other tools haven't caught up yet.  No harm in changing it to upper case though.

Sent from Outlook Mobile

On Wed, Mar 9, 2016 at 7:57 PM -0800, "dbwiddis" <notifications@github.com> wrote:

Constants should have names in all caps, e.g., LOGGER or LOG.

---
Reply to this email directly or view it on GitHub:
https://github.com/hazendaz/oshi/commit/0289fa8e11382ad5173e7a6fd77830ed56ef1b31#commitcomment-16607668


Doesn't this happen automatically because of `MAKEFLAGS` in `/etc/pkgmk.conf`?

if not, wouldn't it be better to use `-j$(echo $MAKEFLAGS | grep -o "\-j[0-9]*" | grep -o "[0-9]*")`


whoa, looks like you misspelled 'canola'


add a comment


> Shouldn't we still allow '-'?

Do you feel strongly about it?

On making the change, the only thing I *don't* like about not using - is
that if you just run qbatch without any arguments there is no help
printed. :-) I think that's a usability fail so I'm considering
reverting it... :-)


On Thu, Mar 10, 2016 at 9:42 AM, Steve Crook <notifications@github.com> wrote:
> I see your point now. I hadn't appreciated that "is True" was evaluated
> differently to just testing for truth. It's now fixed.

I agree, it doesn't look good but I guess it's the only way... When I
noticed that, I only found one other case (mentioned in #2) that does
the same thing, but it is checking a flag that is only handled by the
server and might not cause problems, but as you know the nymserv a lot
better than me, I think you should take a look into that and see if I
missed something.

Thanks!


Don't you mean 'adaption'?


Did you see this @Shaked ?


No


It gets more empty lines without it.  I was trying not to mess much with existing code.

Sent from Outlook Mobile

On Wed, Mar 9, 2016 at 7:48 PM -0800, "dbwiddis" <notifications@github.com> wrote:

We have complete control over what we add.  Do we ever add a `\n` by itself?  This seems a wasted check.   I'd suggest prepending every line of output with `\n` and removing the `%n` from the `String.format()`lines above.

---
Reply to this email directly or view it on GitHub:
https://github.com/hazendaz/oshi/commit/0289fa8e11382ad5173e7a6fd77830ed56ef1b31#commitcomment-16607585


Hi, I can rewrite it, its simple (and now I tested it - its working) change.

However I have another point: When we await exactly the same ammount of time we provide ti the docker stop command, there can be an unpredictable race condition and the await will actually finish BEFORE the docker stop is over - so we are setting and announcing task as KILLED while the task still "lives"


Hi Olivier,

Thanks for the tip. This is impressive... doesn't look like much was
needed to make it compilable.

Best,
Alex

On 2016-03-10 11:38, Olivier K. wrote:

> Hi Alex,
>
> you could use our kernel;
> https://github.com/lenovo-yt2-dev/android_kernel_lenovo_baytrail - it
> builds and boots on Stock and CyanogenMod 12.1
>> Hi Olivier,
>>
>> Unfortunately it doesn't. I am using it to track some changes in the
>> battery driver that Lenovo did from 4.4 to 5.0 and to 5.1. Specifically:
>>
>> http://forum.xda-developers.com/thinkpad-tablet/general/yoga-2-830-1050-wakelocks-cause-t3329821
>>
>> Cheers,
>> Alex
>>
>>
>> On Sun, Mar 06, 2016 at 06:29:11AM -0800, Olivier K. wrote:
>>> Just out of curiosity; does that kernel compile (and boot)?
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16523822
>>
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16585069
>
> --
> Reply to this email directly or view it on GitHub [1].

Links:
------
[1]
https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16613045


You should be checking that it does concern "bcc"
You should also check the matcher
You shold verify that this mailet is before LocalDelivery and RemoteDelivery.

If one of these three conditions is not met then it won't be called...


[R/get_nltt_values.R:50:48:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/R/get_nltt_values.R#L50) *style:* **Commas should always have a space after.**
```r
    m[row_from_index:row_to_index, 2] <- seq(0,1,dt)
                                               ^
```
[R/get_nltt_values.R:50:50:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/R/get_nltt_values.R#L50) *style:* **Commas should always have a space after.**
```r
    m[row_from_index:row_to_index, 2] <- seq(0,1,dt)
                                                 ^
```
[R/get_nltt_values.R:54:25:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                        ^
```
[R/get_nltt_values.R:54:29:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                            ^
```
[tests/testthat/test_get_nltt_values.R:170:17:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/tests/testthat/test_get_nltt_values.R#L170) *style:* **Commas should always have a space after.**
```r
      rep(seq(0,1,0.2),2),
                ^
```
[tests/testthat/test_get_nltt_values.R:170:19:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/tests/testthat/test_get_nltt_values.R#L170) *style:* **Commas should always have a space after.**
```r
      rep(seq(0,1,0.2),2),
                  ^
```
[tests/testthat/test_get_nltt_values.R:170:24:](https://github.com/richelbilderbeek/ribir/blob/979097f106cb645b989ac6d71d883bcfb139e3ff/tests/testthat/test_get_nltt_values.R#L170) *style:* **Commas should always have a space after.**
```r
      rep(seq(0,1,0.2),2),
                       ^
```


FirstUpload


Canola


Remove Unnecessary import


Just saw https://github.com/aduprat/james-project/commit/ae394588ffe410a9f7492f99bfcd313fa5648d56
But the remark for position in the processor and matcher still stands.


Done: #409


weird git spaces config issue. Will try to remove patch when rebasing PR.


Почему не используешь амперсанд как везде?


Already using it for some time and disabled the rule in Rubocop.

As a bonus, except the quite clear semantic implications, is that it's also quicker type: `%w[a b c]` is the quickest to type compared to `%w(a b c)` and `['a', 'b', 'c']`

some quick benches:

```ruby
require 'benchmark'

N = 1_000_000

Benchmark.bmbm do |x|
  x.report("[ ]") { N.times { %w[a b c d] } }
  x.report("( )") { N.times { %w(a b c d) } }
end
```

results

```
Rehearsal ---------------------------------------
[ ]   0.410000   0.000000   0.410000 (  0.414325)
( )   0.380000   0.000000   0.380000 (  0.386498)
------------------------------ total: 0.790000sec

          user     system      total        real
[ ]   0.380000   0.000000   0.380000 (  0.382488)
( )   0.390000   0.010000   0.400000 (  0.393868)
```


I'd like to see a more explicit error message, explaining it does involve field "bcc"


Remove commented code


I think this might just work (I'm checking with Steve about the oddities of X509V3_EXT_METHOD).

But make it a pointer to a lookup method?


Need to free this later.


Aren’t the news for 0.5.8 missing?


You're absolutely right, we should use what's in MAKEFLAGS. Waf doesn't look like it does it automatically from that variable.


ok


OK, I'll do it that way^ from now on. And you mean to enable colors in the progress bar itself? That sounds nice, yea.


Better. "Program" as a helper word is needed when you expect words replacing placeholder to have different genders and that is not the case here - isn't it?


[R/feature_ignore_singletons.R:52:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/feature_ignore_singletons.R#L52) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_msms_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:56:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/feature_ignore_singletons.R#L56) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_scrm_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_ignore_singletons.R:60:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/feature_ignore_singletons.R#L60) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.ign_singletons <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_recombination.R:54:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/feature_recombination.R#L54) *style:* **Variable and function names should not be longer than 30 characters.**
```r
conv_to_seqgen_arg.recombination <- ignore_par
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/feature_sumstats.R:18:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/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/922d28f90c42a148573adc39b7429c547faa9d9c/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/922d28f90c42a148573adc39b7429c547faa9d9c/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:19:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/import_popgenome.R#L19) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.GENOME <- function(data, only_synonymous = FALSE, ...) {
^~~~~~~~~~~~~~~~~~
```
[R/segsites.R:62:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/segsites.R#L62) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.matrix.segsites <- function(x, ...) get_snps(x)
^~~~~~~~~~~~~~~~~~
```
[R/segsites.R:142:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/segsites.R#L142) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.default <- function(data, ...) {
^~~~~~~~~~~~~~~~~~~
```
[R/segsites.R:147:1:](https://github.com/statgenlmu/coala/blob/922d28f90c42a148573adc39b7429c547faa9d9c/R/segsites.R#L147) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
as.segsites.list <- function(data, ...) {
^~~~~~~~~~~~~~~~
```


Change path to meaningful name


Btw, really like the work you did cleaning up all the code, in
particular in android_device_lenovo_YT2!

On 2016-03-10 11:38, Olivier K. wrote:

> Hi Alex,
>
> you could use our kernel;
> https://github.com/lenovo-yt2-dev/android_kernel_lenovo_baytrail - it
> builds and boots on Stock and CyanogenMod 12.1
>> Hi Olivier,
>>
>> Unfortunately it doesn't. I am using it to track some changes in the
>> battery driver that Lenovo did from 4.4 to 5.0 and to 5.1. Specifically:
>>
>> http://forum.xda-developers.com/thinkpad-tablet/general/yoga-2-830-1050-wakelocks-cause-t3329821
>>
>> Cheers,
>> Alex
>>
>>
>> On Sun, Mar 06, 2016 at 06:29:11AM -0800, Olivier K. wrote:
>>> Just out of curiosity; does that kernel compile (and boot)?
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16523822
>>
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16585069
>
> --
> Reply to this email directly or view it on GitHub [1].

Links:
------
[1]
https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16613045


class name 需要修改下


Test shouldn't be adding data, should be testing query


@duphungcong  bạn nên add cái thư mục `.idea` này vào `.gitignore`


Абсолютно независимые блоки предполагают отсутствие зависимости стилей. А тут одно правило для двух селекторов.


The test will fail.


bình thường nếu đưa vào `.gitignore` thì git không quản lý. nhưng nhỡ đã đưa lên github rồi thì có thể làm cho nó `quên` đi bằng cách
`git rm -r --cached .idea`


Uploaded!


The values of in are not changed, but the referenced object is. See https://github.com/imagej/imagej-ops/blob/sobel-hessian-filter/src/main/java/net/imagej/ops/filter/derivative/DirectionalDerivativeRAI.java#L86
This is necessary because the second iteration of the convolution has to be performed on the result of the first iteration.


was war bitte anders, als vorher? Doktor alfred?

2016-03-10 10:24 GMT+01:00 Eric Range <notifications@github.com>:

> alles idioten... außer ich :D :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericrange/uni-project/commit/19cea76962de5ec76d4fb4fbd8736d0cf0e50e32#commitcomment-16611508>
> .
>


Added these files for easy access. So, one can just clone the repo and run the notebooks without having to download the data files seperately.


Sure, there are even more typographical rules in `Misc`. Will do it.


update


Các phép về logic thì ở rails nên dùng `||`  hoặc là `&&` chứ không dùng `OR`, `AND`


I agree that would make things easier and more predictable. But it'd break backwards compatibility... we could release a `1.1` with this but if we are strict with [semver](http://semver.org/) that should be in `2.0`.
(Why didn't I think about that before?! :dizzy_face:)


Pre-existing but is this not normally expressed as a `while (true) { if (...) { break; } }`?


Added public selector highlightSectorAtIndex: to pieChart


The `user-fn` could be bound once outside of the function instead of every time you call it.


windoz (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2325 of 2395 tests passed, 0 test warnings)

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


Method trả về boolean cũng không nên dùng `is_something`. mà nên dùng `?`
`def admin?`


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


FYI: the numbers on the commit text are from the debug version.
Running it as release:
(13485/13485) (378 skipped, 111 failed)


added --quiet to both bower and npm, thanks @simison


Releasing a `2.0` would be fine.

It helps because I was trying to figure out "what does it *mean* to normalize a non-URN?" and it doesn't really make any sense if you can have `URN` instances which aren't actually URNs.


@sevon06 просто user и ничего более


@sevon06 убери лог


Gotcha


new plug


ở trong trường hợp này có thể viết lại
```ruby
unless current_user && current_user.admin?
  redirect_to....
end


What is the purpose of this boolean part?  I wouldn't think a sort-fn was valid if it didn't return a number.  It should hold up to the contract of a comparator right?  I also don't understand why you'd need to do a `boolean` coercion on these here either.  What else would they be?


windoz (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


could you remove the `!`, please? it's not necessary to put it anymore if you're the top level of `main.swift`.


and thanks a lot for this!!


#include<main.h>

int main()
{

}


also, we usually put the example before the `Installation` and call it `Usage`. I'll also create a guide for READMEs later.


dùng `.find` thì nó có raise `ActiveRecord::RecordNotFound` nếu không thấy (nên)
nhưng mà trong trường hợp này thì `current_user = User.find_by_id(session[:user_id])` là được. Nó = nil nếu không có


     service_name = (event =~ /\Abuild:/) ? :update_ddtf_build : :update_job
     Travis.run_service(service_name, event: event.to_s.split(':').last, data: payload)


Awesome! QOL upgrade


I agree. Although I wonder if it makes it easier for us but maybe it's more difficult for the user. See e.g.:

before:
```ruby
    def normalize_urns(list)
      list.map { |text| URN.new(text) }
        .select(&:valid?)
        .map(&:normalize)
    end
```
after:
```ruby
    def normalize_urns(list)
      list.map { |text|
        begin
          URN.new(text)
        rescue URN::InvalidURNError
          nil
        end
      }.compact
       .map(&:normalize)
    end
```


@StefanScherer Those are good arguments. So we leave it as it is.


Если помнишь, то у нас была проблема с checkbox, мы хотели выводить один checkbox а не два... а Live Composer выводил checkbox только два


Apparently not. I hadn't thought about this scenario so did a setup and test of a github-pr plugin, and things seem to work perfectly fine including reruns of such jobs which were originally trigged off with the older encrypted passwords. Looking at the Poller code for both [package-materials](https://github.com/gocd/gocd/blob/6556eae0b6501e605d3acb0832cf8b4cf0e321e7/server/src/com/thoughtworks/go/server/service/materials/PackageMaterialPoller.java#L59) and [pluggable-scm-materials](https://github.com/gocd/gocd/blob/6556eae0b6501e605d3acb0832cf8b4cf0e321e7/server/src/com/thoughtworks/go/server/service/materials/PluggableSCMMaterialPoller.java#L74), we seem to be getting all this information from config, not from db before invoking the plugin. Once the plugin responds with the changes, the respective updaters check if the db object needs to be updated or not ([here](https://github.com/gocd/gocd/blob/6556eae0b6501e605d3acb0832cf8b4cf0e321e7/server/src/com/thoughtworks/go/server/materials/PackageMaterialUpdater.java#L50) and [here](https://github.com/gocd/gocd/blob/6556eae0b6501e605d3acb0832cf8b4cf0e321e7/server/src/com/thoughtworks/go/server/materials/PluggableSCMMaterialUpdater.java#L50)).
To me it looks like we are covered unless I missed some scenario again :)


There is already a check performed during startup to see if config file differs from the last-known-good-version of the config (in config.git). The config will be fully validated if a difference was found. So, re-encryption is the first thing that happens to the config, followed by migration and then a validation if the config has really changed.
From the logs:

```
2016-03-10 16:07:58,661  INFO [main] ConfigCipherUpdater:71 - Found unsafe cipher on server, Go will make an attempt to rekey
2016-03-10 16:07:58,667  INFO [main] ConfigCipherUpdater:73 - Old cipher was successfully backed up to /Users/jsingh/projects/mygocd/target/go-server-16.3.0/config/cipher.original2016-03-10-16-07-58
2016-03-10 16:07:58,668  INFO [main] ConfigCipherUpdater:75 - Old config was successfully backed up to /Users/jsingh/projects/mygocd/target/go-server-16.3.0/config/cruise-config.xml.original2016-03-10-16-07-58
2016-03-10 16:07:58,670  INFO [main] CipherProvider:67 - Cipher not found. Creating a new cipher file
2016-03-10 16:07:58,751  INFO [main] ConfigCipherUpdater:110 - Successfully re-encrypted config
2016-03-10 16:07:59,041  INFO [main] MagicalGoConfigXmlWriter:91 - [Serializing Config] Generating config partial.
2016-03-10 16:07:59,043  INFO [main] GoFileConfigDataSource:124 - Config file changed at 1457606279000
2016-03-10 16:07:59,043  INFO [main] GoFileConfigDataSource:125 - Reloading config file: /Users/jsingh/projects/mygocd/target/go-server-16.3.0/config/cruise-config.xml
2016-03-10 16:07:59,137  INFO [main] MagicalGoConfigXmlWriter:91 - [Serializing Config] Generating config partial.
2016-03-10 16:07:59,237  INFO [main] CachedFileGoConfig:235 - About to notify config listeners
2016-03-10 16:07:59,237  INFO [main] CachedFileGoConfig:243 - About to notify config listeners
2016-03-10 16:07:59,237  INFO [main] CachedFileGoConfig:251 - Finished notifying all listeners
2016-03-10 16:07:59,238  INFO [main] CachedFileGoConfig:245 - Finished notifying all listeners
2016-03-10 16:07:59,251  INFO [main] BuildAssignmentService:170 - [Configuration Changed] Removing jobs for pipelines that no longer exist in configuration.
```


`unless current_user`


comment rồi


2076-liquid-flow-rate-ip-units (MatthewSteen) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


" , om tingen er købt..." kunne man ikke siget "om et item er købt" istedet? "Tingen" virker lidt tale agtigt


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


không cần `return` ở trong controller. Cái `redirect` hoặc `render` coi như là return rồi


Let's name event is `:erorred` not `:aborted`


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 39](https://teamcity.labkey.org/viewLog.html?buildId=309448&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12 (1 new), passed: 39 Build time: 01:15:21
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Hello,

Here I have posted both testcodes 1 & 2 with screenshots.


나도루비하다그랬걷느


So that the injector knows what services to inject and to avoid using annotation to prevent minification when using the other DI methods.


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


>デッキやエクストラデッキに目標となるカードが存在しないことが「公開情報から」自明である場合には、確認させる必要はない。

This indicates that comfirming deck is only a convention to avoid cheating and there always exists some cases that unnecessary confirming happens.


Typo in commit message (textredner).


:+1:  Test là tốt. Dev ruby muốn chuyên nghiệp phải viết tests đầy đủ, chuẩn.
Minitest này là đi kèm với Rails, đủ tốt nhưng mà mình không thạo lắm, vì các công ty thường dùng `RSpect` để test


Will need to ensure that only the recipient's email address is visible when they receive the notification. This will result in data on other participants.


You're missing some parentheses here: https://github.com/mapbox/tile-reduce/commit/1886b6840fe43c0c1fce316f175650081a1c6b06


2076-liquid-flow-rate-ip-units (MatthewSteen) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Does this mean Fingerprint can also implement LabelsProvider, DescriptionsProvider and AliasesProvider now?


sorry, one last minor nitpick. we've been prefixing with 'ostro.conf: ' changes to distro config.


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


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


А где дз?


Why do we have that? Should be `getQueryVar`.


Now you have the leaving option, you can get rid of this alert.


Wouldn't it be better to do here just a simple regex to see if the pasted text is URL, and do vimeo / youtube parsing only when `oembedProxy` is false? Because oEmbed supports many more URLs than just vimeo and youtube.


Looks like something that should not have been commited.


Der Button zum Bubble Buddy war durchaus sinnvoll für PC Nutzer. Wär praktisch wenn der wieder rein kommt.


Trong ruby không cần chữ `return`, nó trả về câu cuối cùng nên false là được.


Can you please explain a little bit more what you are doing here and why? Thanks! :)


Thanks @Jfach for handling this.


Just looking for some precedent in `URI` that might help us: how about exposing `URN.extract` (c.f. [`URI.extract`](http://ruby-doc.org/stdlib-2.0.0/libdoc/uri/rdoc/URI.html#method-c-extract))?

Then you might get something like:

```ruby
list.flat_map { |text| URN.extract(text) }.map(&:normalize)
```


دهنت سرویس ایمان


See https://3v4l.org/F7Ra8


results state should be `:errored` not `:canceled`.
probalbly you can do it ugly way:
```
     build.cancel
     build.state = :errored
     build.save!
```
and nice way:
* extend `lib/travis/model/build/states.rb` by event :error and define `error` method, which do something similar as above.
* here in `#run` just call `build.error!`


uma diferença de caractere


:+1: nên học về các callback như vậy.
Nhưng trong trường hợp này có thể tìm hiểu thêm
`restrict_with_error` trong http://edgeguides.rubyonrails.org/association_basics.html


갓준철...


covered TFS and P4 in the [next commit](https://github.com/jyotisingh/gocd/commit/fed000e7455b930a92ee8c2ef16538eef461594a) (branch -> https://github.com/jyotisingh/gocd/tree/rekey_cipher)


I don't see any issue here


@liach Thanks! Will change this soon.


This by itself is not so useful; try to log the information with it :)


Don't forget how important is proper indentation :smile_cat:


remove


I used Netbeans (8.0.2) for development.


@yyenigun This client has no pooling or timeout. I think it may cause tcp port exhaustion. I have a singleton, pooling HttpClient (Apache) object in my project. I want to inject it here instead of using default http client. Is there a plan to change this class in the future? Regards.


@jensens that's what I meant for fixing or reverting, re-running  a job is also part of fixing :-)

And it seems it is fixed.
Thanks!


I think you need to add a null check here. As it stands, this will throw a NPE if `highlightedListener` is not set.


thanks for this commit :+1:


Could enclose the whole section start from line 172 in a `with NamedTemporaryFile() as temp_json_file_handle:` clause and then the explicit unlink isn't needed either.


@slavaGanzin I've tried to address your feedback in PR #189 .

I've moved all the uses of `ConfigManager` into `KernelManager`. Before this PR, `Hydrogen` and  `KernelManager` interacted with `ConfigManager`. Now only `KernelManager` does. Since `ConfigManager` only keeps info about kernels, I think this is best.


Isn't it possible that MITK_BINARY_DIR also isn't canonical and the comparison still fails in a few cases?


You can also contribute to the project. Just add timeout and pooling support and send a pull request. We will review then.


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


"available __in__ ..." !


I am not sure is this script as safe as it should be. Modes are given based on ident and "wildcard" hostmask. I think this causes all connections coming from the same ISP to have same hostmask. Ident can also be changed easily by setting ENV IRCUSER or from irssi config. This means the script might give modes to a wrong person.

The problem would be solved by using full hostmask instead of *.example.mask.com.

I'm not 100% sure if this is the case so please correct me if I'm wrong.


CR done in the rebased branch https://github.com/bydga/mesos/commits/dockerfix-rebased. Please continue any discussions there.


Sure. Do we want the file to be unlinked if the test has failed?


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


std::numeric_limits<ElemType>::min(); returns the lowest positive for floating point types.


Wrong source file here: should be w_setenv.


Hi,

Is there any documentation to install this module?

I want to change the avatar, but this module don´t display in jitsi.

Thanks.


kfree(taskComm)


I think so. We can always log the content of the file: pytest will show us that output for failed tests.


@sarahgriffis  Can you please tell me the reason to make it static? We should not hard code this and let it dynamically see what server is actually running on.


human_readable_time  -> time


нужно внести этот файл в .gitignore


new comment


I'll add tests sometime next week and make another PR


Language should be lt instead of lv.


У меня 2, не знаю почему здесь 4)


<%= form_for @petition do |f| %> вместо <%= form_for Petition.new do |f| %>


@sarahgriffis  gulp-nodemon is used for the development purpose. We won't be able to auto start the server during dev if it is not there. Any specific reason to remove it ?


don't use goto excessively.
You can easily write this code without it.


можно, скоро залью исправления


@slavaGanzin I will revert this PR after merging #189 so that I don't have to deal with merge conflicts.


need to INIT_LIST_HEAD(&tmp->lh);


Nope, I can't make it without unlink - without ```delete=False``` it gets deleted as soon as we close it (next line). This makes context manager way pointless imho


хорошо


Please add to this one, after test is added I will merge.
THnx


( prototype issue about setting the opacity : https://github.com/sstephenson/prototype/issues/145 )


If you want to rephrase this, I'd suggest rearranging a little. Perhaps:

"""
We use the origin of the URL here since content sandboxed into a unique origin (e.g. `<iframe sandbox src="http://localhost/">`) would be treated as non-trustworthy by the algorithm in [[#is-origin-trustworthy]]. Since sandboxing is a strict reduction in the content's capabilities, and therefore in the risk it poses, we look at the origin of its URL to determine whether we would have considered it trustworthy before the sandbox was applied.
"""

WDYT?


I am looking for IT-recruiters asap! Are you living in / or around Amsterdam?

Ideally 4-5 years recent IT Tech recruitment (agency) experience
Fluent English, an additional European language would be a benefit
Experience working in a fast moving and high paced environment
Pro-active, dynamic, result driven
Ability to multi task and work towards deadlines
Excellent communication skills

http://www.yoores.com/werken-bij/65-test/280-it-tech-recruiter-international-ned


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


The changes in the readme file are not completely correct. After using these setting I noticed I am missing the correct conversion for 44100 files. The correct sink would be:

output = audioresample ! audioconvert ! audioconvert ! audio/x-raw,rate=44100,channels=2,format=S16LE ! wavenc ! filesink location=/tmp/snapfifo

This will correctly convert 48000 to 44100 sample rates while still keeping the 44100 ones.


remove tags


fix , in join


BTW (just for the record)..  this is incorrect construction, you cannot "if ... endif" within the block, the right way is:
```
libblkid_la_LDFLAGS = $(SOLIB_LDFLAGS)
if HAVE_VSCRIPT
libblkid_la_LDFLAGS += $(VSCRIPT_LDFLAGS),$(top_srcdir)/libblkid/src/libblkid.sym
endif
libblkid_la_LDFLAGS += -version-info $(LIBBLKID_VERSION_INFO)
```
automake really don't like  "\ endif" and if used then endif is within the final Makefile.

I'll fix it.


list_entry form is more preferred then container_of


You haven't implemented kmalloc and create_stack_entry error handling with rollback here.
See http://mit.spbau.ru/sewiki/index.php/Linux_kernel_2016#.D0.9E.D0.B1.D1.80.D0.B0.D0.B1.D0.BE.D1.82.D0.BA.D0.B0_.D0.BE.D1.88.D0.B8.D0.B1.D0.BE.D0.BA


What is the benefit of generated functions here, rather than using parametric functions? e.g. instead of deciding if `locs <: AbstractVector{Bool}` in the generated function body, just doing that by multiple dispatch?


The underlying client can't be made generic, especially when you return more than one type of objects, which is what we do here


Is this change related to #65?


while(!stack_empty(&data_stack)) is better here.


Iterable


same


default scope is anti pattern. will be better if you forget about this feature


wrong alignment


list_entry or list_first_entry is more preferred here


Ich gehe mal davon aus, dass die 14 für Domäne 14 steht. Muss dann oben der Prefix nicht auch 14 sein damit man die Statuspage aus dem netzt heraus aufrufen kann?


It looks like now the new url is also broken
```
Downloading http://faculty.cse.tamu.edu/davis/SuiteSparse/SuiteSparse-4.0.2.tar.gz...
[ERROR] urllib failed to download (reason: [Errno 110] Connection timed out): http://faculty.cse.tamu.edu/davis/SuiteSparse/SuiteSparse-4.0.2.tar.gz
```


s.u.


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


[Wiki](http://uo.com/wiki/ultima-online-wiki/items/random-loot-generator/)

>
> Disadvantages
>
> These appear on items randomly, items that are free of disadvantages, ie ‘clean’ can be generated. Weak items are frequently ‘clean’, powerful items are very rarely ‘clean’.


lil typo here


:smile:


Thanks @AlbertoFuente


Really? Like this:

```
with NamedTemporaryFile() as temp_json_file_handle:
    command = []
    ....
    temp_json_file_handle.write(build_json['contents'])
    temp_json_file_handle.flush()
    logger.info("Contents: %r" % build_json['contents'])
    ...
    with pytest.raises(...):
        self.exec_cli(command)
```

The temporary file only gets deleted once we leave that `with` clause, right at the end.


I have to convert the map here to a list because in python2 map returns a list whereas in python3 it returns an iterable object.


It's very similar, but note the use of the resultTransformer in this implementation - this is what made the code so simple here.
I think if at all, this implementation needs to be pulled out.


Старайся избегать использования файла component.php вообще. Это лишнее


@jnicklas I don't get it, why you need `!!` here? `_pundit_policy_scoped` is already boolean.


Shouldn't we clear the lists here too?

```
Timeline.Clear ();
Periods.Clear ();
Timers.Clear ();
```


building of JS on server side is common practice too. That is right. But I strongly recommend you do not this.
JS should be placed only in two places in rails app - `app/assets` (if asset will be compiled) and `public/vendors` (if it's precompiled or plain asset)

I can show you how to replace this approach with better way


Add store settings (platform and storefront) #340


Note to myself: Wasn't meant to be deleted :-/


oui !


for kibana-linux v.4.4.0


Indeed! 6ce13be ;)


I am using the PKGBUILD on https://aur.archlinux.org/packages/ros-indigo-image-view/

I can reproduce every time with `rm -rf src/; makepkg -f`.

If I re-run make without a clean it will of course work, because now the file is present. But when I `pgrep python` I can see that cmake continues before the python process spawned by that generate function. (At least so it seems.) But when I get home I will try to add some debug prints in the `generate_dynamic_reconfigure_options`  function


В классах совсем не обязательно проставлять эту проверку. Эта проверка нужна только для того, чтобы никакой код не отработал при прямом обращении к файлу из браузера. В твоем случае итак ничего не будет отработано, т.к. в этом файле только лишь определение класса. Используй этот заголовок только в файлах, которые доступы через браузер и в которых содержится какая-то логика, относящаяся к битриксу. Как правило, это файлы component.php, и все файлы шаблона компонента (template.php, result_modifier.php и component_epilog.php)


Ah, indeed, I forgot about ```flush```, pushing a new commit


Same as the other comment: shouldn't we clear the `List`?


What about checking the length of the stdin if you don't use '-', and
throwing help?
On Mar 10, 2016 6:48 AM, "Jon Pipitone" <notifications@github.com> wrote:

> > Shouldn't we still allow '-'?
>
> Do you feel strongly about it?
>
> On making the change, the only thing I *don't* like about not using - is
> that if you just run qbatch without any arguments there is no help
> printed. :-) I think that's a usability fail so I'm considering
> reverting it... :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pipitone/qbatch/commit/b382079ccf757230a3f2d21e7b3e096e741ba02e#commitcomment-16614317>
> .
>


oui


@aleffert calling searchThreads(text)  to fetch results for updated query


what about other Wrapper Classes,such as Double,Integer, and so on.


Для чего нужна эта переменная?


It looks like `move` and `rename` are mutually exclusive from `couch_file` perspective. This fact makes it possible to write `couch_server:delete_file/3` as follows:
```
delete_file(RootDir, FullFilePath, Options) ->
    IsCompaction = lists:member(compaction, Options),
    case config:get_boolean("couchdb", "rename_on_delete", false) of
        true when isCompaction ->
            couch_file:delete(RootDir, FullFilePath, [rename | Options]);
       _ ->
            couch_file:delete(RootDir, FullFilePath, Options)
end
```


iteration (macumber) - i386-Windows-7-VisualStudio-12: OK (2370 of 2397 tests passed, 0 test warnings)

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


Сразу видно, что скопировано из интернетов. Попробуй осознать, что именно делает этот код, и приведи имена переменных в camelCase как везде


I think you heard about conventions over configurations. DHH said many years ago - you needn't configuration files except `database.yml`

But look at this line. It's line of configuration. I can show you many places where you should configure your rails app.

I want to say - in real live rails is full of configurations. That is why you should find good way to manage your configurations. I can show you how to do that and forget about any problems with configurations of rails


не выполниться условие что только авторизованный пользователь может создавать петиции
нужно добавить строку before_action :authorize, only: [:new, :create]


Thank you very much!


almost :)


Shame :bell:! Shame :bell:! (this breaks cura :()


patch-1 (pachi) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2290 of 2365 tests passed, 0 test warnings)

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


Yes, it is in 0.22.0-rc now


mm


title missed. modified docker run command.


otherwise good


thanks i'll making there


search nie powinien być na POST


A couple of methods for activating key filters and navigation hierarchies for a list - both are overloaded (one for string-based internal names / titles for the fields, and another for a field objects collection).
The method requires Web object's Metadata Navigation and Filtering feature activated - web.ActivateFeature(new Guid("7201d6a4-a5d3-49a1-8c19-19c4bac6e668") for your convenience.


Hi @mgruner
This was commited only to master. Can you also cherry-pick it to rel_5-0? There is no localized error handling in rel_5-0, because missing the Translate() function from .tt files!


プレイヤーのHP表示 #333


I feel pretty strongly about having explicit semantics.

We should definitely avoid anti usability like no help.

Hopefully we can get both.
On Mar 10, 2016 7:58 AM, "Gabriel A. Devenyi" <gdevenyi@gmail.com> wrote:

> What about checking the length of the stdin if you don't use '-', and
> throwing help?
> On Mar 10, 2016 6:48 AM, "Jon Pipitone" <notifications@github.com> wrote:
>
>> > Shouldn't we still allow '-'?
>>
>> Do you feel strongly about it?
>>
>> On making the change, the only thing I *don't* like about not using - is
>> that if you just run qbatch without any arguments there is no help
>> printed. :-) I think that's a usability fail so I'm considering
>> reverting it... :-)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/pipitone/qbatch/commit/b382079ccf757230a3f2d21e7b3e096e741ba02e#commitcomment-16614317>
>> .
>>
>


Since we are parsing (and have a token location), can we make these ParsingExceptions?


I don't think this can be improved as logic of this loop differs everytime. We can't really skip that.


This is going to break every old link in the forums.


patch-1 (pachi) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


TeamCity Selenium POM :: CI [Build 70](https://teamcity.sponte.uk/viewLog.html?buildId=1084&buildTypeId=SeleniumPom_Ci) is now running


히이.. :+1:


Yeah, you're right. Sometimes we meet inner classes with such names.

On 03-10 (05:00), Wang Shidong wrote:
>    what about other Wrapper Classes,such as Double,Integer, and so on.
>
>    —
>    Reply to this email directly or [1]view it on GitHub.
>
>    Reverse link: [2]unknown
>
> References
>
>    Visible links
>    1. https://github.com/artur-shaik/vim-javacomplete2/commit/05a92e90b24591b49dc08a145617cb9c8b5e20f8#commitcomment-16615694
>    2. https://github.com/artur-shaik/vim-javacomplete2/commit/05a92e90b24591b49dc08a145617cb9c8b5e20f8#commitcomment-16615694

--
Best regards,
Artur Shaikhullin


TeamCity Selenium POM :: CI [Build 70](https://teamcity.sponte.uk/viewLog.html?buildId=1084&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 45 Build time: 00:00:58


You could change the return type of `update_mtime` to be `ok | {error, {Reason :: term(), FilePath}}`. Then you would write
```
case file:rename(Filepath, DelFile) of
    ok when Rename ->
       ok = update_mtime(DelFile),
       {ok, {renamed, FilePath}};
```
Yes it would badmatch on error but since the file name is included in the error term it would be very easy to debug if we need to. Also it seems that matching on result is already used in this PR [for example here](https://github.com/apache/couchdb-couch/pull/141/files?diff=unified#diff-f2555f39d773922624eef36b4516c459R297).

Would this work?


It should be using the pcre library bundled with the R install at `$HOME/R-bin/lib` not the system one, so it shouldn't have affected things. But clearly something has gone awry, perhaps I need to explicitly enable those flags in the PCRE configure script.


Super, thanks! :+1:


[R/get_nltt_values.R:50:48:](https://github.com/richelbilderbeek/ribir/blob/66b59ffcc78464570d98bbae3c036eb48a8ef7e2/R/get_nltt_values.R#L50) *style:* **Commas should always have a space after.**
```r
    m[row_from_index:row_to_index, 2] <- seq(0,1,dt)
                                               ^
```
[R/get_nltt_values.R:50:50:](https://github.com/richelbilderbeek/ribir/blob/66b59ffcc78464570d98bbae3c036eb48a8ef7e2/R/get_nltt_values.R#L50) *style:* **Commas should always have a space after.**
```r
    m[row_from_index:row_to_index, 2] <- seq(0,1,dt)
                                                 ^
```
[R/get_nltt_values.R:54:25:](https://github.com/richelbilderbeek/ribir/blob/66b59ffcc78464570d98bbae3c036eb48a8ef7e2/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                        ^
```
[R/get_nltt_values.R:54:29:](https://github.com/richelbilderbeek/ribir/blob/66b59ffcc78464570d98bbae3c036eb48a8ef7e2/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                            ^
```


fek off


That seems reasonable to me but I'm not that much into build process and gulp so I may not see a better solution here. Piotr and luxas probably will know more about that and we can discuss further in the PR.


My bad, added it to the repo

Sent from my iPhone

> On Mar 10, 2016, at 05:14, Sean Smith <notifications@github.com> wrote:
>
> we seem to be missing lib.speech
>
> —
> Reply to this email directly or view it on GitHub.
>


Yep, same thoughts.


этот код должен вызываться в контроллере


lgtm


Looks perfect :)


How about:

"""
      Note: We use the origin of the URL here since sandboxed content (e.g.
      `<iframe sandbox src="http://localhost/">`) is treated as being from a
      unique origin, which would be treated as non-trustworthy by
      [[#is-origin-trustworthy]]. Since sandboxing is a strict reduction in
      the content's capabilities, and therefore in the risk it poses, we look
      at the origin of its URL to determine whether we would have considered
      it trustworthy if it had not been sandboxed.
 """


what does it mean


Great, thank you. I'll put in the pull request later today I think :)


[R/get_nltt_values.R:54:25:](https://github.com/richelbilderbeek/ribir/blob/ab5ed3e9a12669134d3502c9ca019a00725d8559/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                        ^
```
[R/get_nltt_values.R:54:29:](https://github.com/richelbilderbeek/ribir/blob/ab5ed3e9a12669134d3502c9ca019a00725d8559/R/get_nltt_values.R#L54) *style:* **Commas should always have a space after.**
```r
  colnames(z) <- c("ID","t","nltt")
                            ^
```


Ah here is the issue, I compiled PCRE with UTF-8 support, but without unicode properties https://travis-ci.org/jimhester/r-source/builds/109656773#L2360. Will fix, thanks for letting me know the issue!


iteration (macumber) - Win64-Windows-7-VisualStudio-12: OK (2370 of 2397 tests passed, 0 test warnings)

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


@billfitzgerald0120 This change is not needed.  `security_groups` is a multi-select dialog field, unlike the other resources listed, and the name reflects that difference.

The key names are defined in the dialogs (for example [miq_provision_azure_dialogs_template.yaml#L170](https://github.com/ManageIQ/manageiq/blob/master/product/dialogs/miq_dialogs/miq_provision_azure_dialogs_template.yaml#L170)) and loaded by [cloud_manager/provision/options_helper.rb#L11](https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/cloud_manager/provision/options_helper.rb#L11) when provisioning the instance.

The second value in the array is the key used to set the selected value in the options hash for the task which is returned by [miq_provision_mixin.rb#L110](https://github.com/billfitzgerald0120/manageiq/blob/a2cc7bb6251e9f3fffb2b7bf7bd0be97bb8b7936/app/models/mixins/miq_provision_mixin.rb#L110) and used here [miq_provision_mixin.rb#L124](https://github.com/billfitzgerald0120/manageiq/blob/a2cc7bb6251e9f3fffb2b7bf7bd0be97bb8b7936/app/models/mixins/miq_provision_mixin.rb#L124).


Wouldn't it be more accurate to replace the `result.getSuperClass() == null` with a comparison to a `ResolvedJavaType` object corresponding to `java.lang.Object`?


;0

On 10 March 2016 at 13:13, JosephCowin <notifications@github.com> wrote:

> fek off
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OrrySwift/Normiemon/commit/26bdd528f242e07bb5aee9d61037e67f2c46e40d#commitcomment-16615979>
> .
>


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


It was erroring out until I changed it. It will have to be run on 8081 for eb.
```> node server.js
/var/app/current/server.js:10
        console.log('Server is listening on ' + server.info.port);
    at /var/app/current/server.js:10:60
    at /var/app/current/node_modules/hapi/lib/server.js:276:20
    at internals.Server._invoke (/var/app/current/node_modules/hapi/lib/server.js:344:16)
    at /var/app/current/node_modules/hapi/lib/server.js:268:14
    at Server.finalize (/var/app/current/node_modules/hapi/lib/connection.js:176:9)
    at Server.g (events.js:260:16)
    at Server.emit (events.js:166:7)
npm ERR! paydash@1.0.0 start: `node server.js`
npm ERR! Failed at the paydash@1.0.0 start script 'node server.js'.
npm ERR!     node server.js
> node server.js
/var/app/current/server.js:10
        console.log('Server is listening on ' + server.info.port);
    at /var/app/current/server.js:10:60
    at /var/app/current/node_modules/hapi/lib/server.js:276:20
    at internals.Server._invoke (/var/app/current/node_modules/hapi/lib/server.js:344:16)
    at /var/app/current/node_modules/hapi/lib/server.js:268:14
    at Server.finalize (/var/app/current/node_modules/hapi/lib/connection.js:176:9)
    at Server.g (events.js:260:16)
    at Server.emit (events.js:166:7)
npm ERR! paydash@1.0.0 start: `node server.js`
npm ERR! Failed at the paydash@1.0.0 start script 'node server.js'.
npm ERR!     node server.js```


review code


Így igaz! Még Arch-Linux is a belle-siphez külön letölti a fordításához a 3.4 -et.
Visszacsinálom, és kösz!


It prevents me to create a new SO


I meant to write "[planner]" :)


this error is mostly about iterator bounds, it2 actually is still same itr copy and when base m_GroupMembers tries to delete it2, it alters whole container and its itr iterator.

don't remember in which standart it actually came but safe way to erase iterator is like I described - itr = containter.erase(itr); This call alters and updates iterator in safe way


wut


indeed


What you mean, @mythi?


I think it'd make more sense to call it "Custom Hooks" rather than "Plugin API", as Oxide provides the Plugin API, not plugins, so it would be a bit of a confusion.


test


How is this different than SectionTagName?


Here we would need `interfacesB[j]`.


Yes, npm isn't able to install a dependency of gulp-nodemon and it was causing EB to fail.

```    inflating: /tmp/deployment/application/tasks/nodemon.js
  > fsevents@1.0.5 install /tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents
  gyp ERR! stack Error: EACCES: permission denied, mkdir '/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/build'
  gyp ERR! command "/opt/elasticbeanstalk/node-install/node-v4.2.3-linux-x64/bin/node" "/opt/elasticbeanstalk/node-install/node-v4.2.3-linux-x64/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--module=/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v46-linux-x64/fse.node" "--module_name=fse" "--module_path=/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v46-linux-x64"
  gyp ERR! cwd /tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents
  node-pre-gyp ERR! stack Error: Failed to execute '/opt/elasticbeanstalk/node-install/node-v4.2.3-linux-x64/bin/node /opt/elasticbeanstalk/node-install/node-v4.2.3-linux-x64/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v46-linux-x64/fse.node --module_name=fse --module_path=/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v46-linux-x64' (1)
  node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
  node-pre-gyp ERR! command "/opt/elasticbeanstalk/node-install/node-v4.2.3-linux-x64/bin/node" "/tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
  node-pre-gyp ERR! cwd /tmp/deployment/application/node_modules/gulp-nodemon/node_modules/nodemon/node_modules/chokidar/node_modules/fsevents```


- http://136246-9-0-e4232b-all.runbot.odoo.com/
- Sales > Quotations > Create
- Add a Customer
- Try to save: "The following fields are invalid: order reference"


`defaultl`
\o/


We already have the minix.py file for cluster. There should be the cluster commands that need separate implementation. We have only keys and scan currently there. We can add any additional command there. All the rest of commands would be delegated to pool or connection by getattr.


:+1:

Why embed the file though? I would just add it as a dependency.


Grafiken in fig:polygon-oriented müssen auch noch angepasst werden


Denis Ledoux !


I've briefly checked the code and it looks really nice. I'll also try to test it later if I find some time :)


Find a path to escape from Maze


We're still missing libs :)
I guess lib was getting ignored by this https://github.com/imprint75/speechy/blob/master/.gitignore#L18?


I'm trying to repeat a failure now -- I did see that TestJournalPlayer.PrefetchMultipleTags was 100% failing on master so I pushed 9cf46b8 to correct it.


awesome


c rigolow


Interesting - I think fundamentally using `inherits()` with S4 objects is a bad idea.


There were 14 merges added in order to push 1 commit. These should have been rebased before the Hudson job was started. The cause is probably that the used branch was not reset to origin/master before the changes were performed. Always use git rebase instead of git merge (git pull --rebase when pulling)


I may need to keep it for a while longer :))


I'm sure I cargo-culted this code from somewhere. Change seems fine, though. Not sure what it was doing before.


f7b15b8e4f467325dae0c29553e9c05f82d87d67

C'est assez rigolo que le runbot n'en ai rien eu a foutre :D.


If you want to check the localization, please check out HEAD of this branch (not this commit). You'll need to do `npm install` too, to pick up the XtbGenerator and some gulp libraries.

And yeah, I used the [Quick Language Switcher](https://chrome.google.com/webstore/detail/quick-language-switcher/pmjbhfmaphnpbehdanbjphdcniaelfie?hl=en) Chrome extension to test the localization.


This seems like it could be very expensive as soon as the interface hierarchy gets larger. When the same interface is implemented multiple times by the same super interface (e.g. a diamond), we are also re-doing work that could be cached.

Also it seems to me like it would not necessarily pick the *lowest* upper bound, but instead *some* upper bound. But I might be missing something...

Perhaps a better way would be to build an associative map of all the transitive super-interfaces of `a`, and then do a traversal of transitive super-interfaces of `b`. This would allow us to traverse all possible upper bounds, and then decide what the result of `meetTypes` is, for some definition of lowest upper bound.


C'est probablement pas un flux standard de vouloir créer un SO. Pas besoin de tester ça.


You can add fields and methods to enums; see the Planet example.


Good catch. I've added a test that would trigger this error and fixed it.


我还是觉得英文两端还是加空格的好，看过许多翻译的项目都是这么做的。


1915_WeatherFile2 (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


@bsnizek Det her er rimeligt lamt, men det virker. Kan du se en smartere måde at gøre det på?


@sindresorhus the reason is that `ember-cli` doesn't want to import from `node_modules` :sob: even with `ember-browserify` I've been working on this and once fix this will add as a dependency.


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


bgpdump uses brackets for AS_CONFED_SET and parenthesis for AS_CONFED_SEQUENCE, we should follow their convention.


これ、わざわざ変数にしなくていいですよ。このクラスの中だったら、 スコープの中で arel_table[:status] と呼べます。scopeはクラスメソッドに変換されるからです。


Alternatively, we could just give up after some threshold.


yes


There were 14 merges including 2723104 added in order to push 1 commit. These should have been rebased before the Hudson job was started. The cause is probably that the used branch was not reset to origin/master before the changes were performed. Always use git rebase instead of git merge (git pull --rebase when pulling)


Yes, agreed. In general, it would be helpful to cache the result of meet types; at least per compilation.

It is a best effort towards lowest bound. In some cases the lowest bound is even undecidable.


@dakull I think "performance" stats like this are too negligible to take into account. Several times the same argument has been made for string style. (Even I argued for single-quotes for the speed, but now see that there is no real difference.) And when it comes to percent-literals, typing the array as `['a', 'b', 'c']` is really a moot point.


just pushed some fixes


Awesomeness!  GL at LTC.


Removed double quote at the end of variable $temppatient_name


I originally had called it the "Inter-Plugin API", as that best describes what it does - it's an API for plugins to call each other. It's a little verbose for my liking, that's why I shortened it to "Plugin API"...

Thoughts?


aa


Is it wrong that I think this is awful?  An enumeration, to me, means a set
of allowed values. Maybe if we want to stretch it to a set of allowed
structs, I can see that. However, a set of allowed computations seems a
step too far.

I guess if it's idiomatic java,  I'll hold my nose and restructure it this
way.

Jordan Thayer
http://www.jordanthayer.com

On Thu, Mar 10, 2016 at 8:26 AM, Steve McCoy <notifications@github.com>
wrote:

> You can add fields and methods to enums; see the Planet example.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/velour/wrecked/commit/1f681172c728bfc1d11f4936cffe0fb95971713d#commitcomment-16616238>
> .
>


and please update the commit message title

```
common/config_opts.h: add osd_crush_initial_weight
```
would suffice.


You're not wrong.


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


There's a small typo in the passed argument, `$exmapleName` instead of `$exampleName`. Unless we're really taking about ex maple trees here :grin:


It was print "INF" at the top of calendar


I've created issue #118 to register these suggestions.


I put back a couple of the unit tests in 3ebe3a5d265e72c3d673ad6f35ff84f5697b35ea after updating them to use implicit collection support. I think those couple of unit tests are still useful.


I would replace isNeitherNullNorShutdown + shutdown by a simplier method ensureShutdown()


fajnie by było usunąć ten kod z kontrolera i przenieść do osobnej klasy zwłaszcza że niżej jest powtórzony


@sindresorhus I created related issue https://github.com/imanhodjaev/ember-cli-ascii-spin/issues/1 will ping you once fixed.

Thank you, `cli-spinners` is awesome.


这边不要  remove所有Message，这个Handler不是我们要维护的。View自己会维护。


Sounds good!


@bsnizek For hvert tree item, som er et search match, løber jeg hele vejen op til root item. Det er ret lamt. Kan du se en smartere vej?


This is important.


No need for the `elseif`; the previous statement is an `if` that `returns` so the code will only reach this in the "else" case of the previous if.


Which Apache version? Using data that can be manipulated by the client (in this case HTTP headers) to identify security related stuff is in general dangerous. There are also other headers they may be used in some environments like "HTTP_X_FORWARDED_PROTO" to point out that HTTPS is in use, but i think its better to do this vhost configuration

https always on:
```
<VirtualHost *:443>
  SetEnv HTTPS on
```

or conditional based on a trusted header from the proxy server:
```
<VirtualHost *:80>
  SetEnvIf x-forwarded-proto https HTTPS=on
```

or if that is not possible then in the bootstrap e.g. config.inc.php
```
$_SERVER['HTTPS']=$_SERVER['HTTP_MY_TRUSTED_HEADER']
```

i suggest to make it configurable in the backend on which header the shop should trust that it is HTTPS, but for the default settings we should use the most secure settings which is to trust only $_SERVER['HTTPS']


ეს აქ რო შეიტანე ამ მეთოდს ყოველი ასოს ჩაწერაზე ვიძახებ და ბევრი ცვლადი არ შეიქმნება?


#include <linux/threads.h>
PID_MAX_DEFAULT


1915_WeatherFile2 (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


What's the font you used in this logo?


minDelay or fixedDelay?


Did you never pull request this?


@koalo , the moteProbe is thread rather than the handler of serial port, which mean you can't close it like this.


The time taken might be fairly long either way, and it still isn't clear to me if it is just reporting more test files than are actually run. If you don't also filter by filename, then this code would load the other *.test.xml files, but load them with no rules. I think that might list the files even if no rules within them were actually run?


magic numbers?


I did not change this on purpose.
It's caused by Qt designer due to UI elements have different size on different platforms.
Let me know if you want to keep its original value.


Shouldn't that be TYPE_AUDIO instead TYPE_TEXT?


აქ სტრინგები რატო წაუშალე ყველას?


You probably want network-online.target, not network.target.  See https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/ for more information.


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


fasze momcere.nu mspamav.

On Thu, 10 Mar 2016, 13:35 giusha, <notifications@github.com> wrote:

> აქ სტრინგები რატო წაუშალე ყველას?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/giusha/Dictionary/commit/219d33a6a6c248b564f5f455e2b7c2448cf1fda3#commitcomment-16616472>
> .
>


121x, in my computer. 247x is the improvement of ruby code.


Not exactly. We are always renaming when `rename_on_delete` is set, but moving only on move\compaction and how we renaming depends on if we are moving, i.e. the above would bring us back to the compaction files left in shards dirs.

I guess we can just make is into `{move, Boolean}` with Boolean set by `lists:member/2` in `couch_server`


pep8に準拠させました。


Does this even compile O.o?


Because before my change around the line https://github.com/bydga/mesos/commit/acf79781c04ad9309083dc39131e2c8305331431#diff-ce6f8d2809238b2d8ae03074344bd413R235  - where I improved setting the `killed = true` flag - only setting it when the task is really killed by SIGKILL - the control flow always jumped to the ``` else if (killed) {
              state = TASK_KILLED;
          }```  condition.

Now, when I'm properly setting the killed flag - that condition is missed and this one is fulfilled, because the run process is not ready - is killed by this time. But the task is definitely not failed (we wanted to terminate it, right..?) - its just "finished".

To me, it seems like this like has been there coded wrongly since the first mesos docker support. But because the flow always ended at the TASK_KILLED part, this error-behavior never occured.. :)


@cgiffard: I tried this after a long time, but it doesn't work. So I decided to try phantomjs-prebuilt and got the same results. Something else must broke this.


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


おお、ちょっと私の指摘がまずかった。

- 基本はwhere(status: xxx) 的な記述
- ダメだったらarel_table
- どうしても困ったら文字列

な順がいいです。ここの場合には

```rb
	scope :should_work_on, ->{ where.not(status: statuses[:done]) } # where.not できる
	scope :rescently_done, ->{ done.where(arel_table[:updated_at].gt(3.days.ago)) } # enumでscopeとして done ができている
```

と書けます。あと、私の場合には `{}` に文章を一行でつなげる際には前後に1文字空けています。そういうプロジェクトもこれまで多かったので参考にしてください。


Can you check the version of your pymysql?

On March 10, 2016 4:28:56 AM tstadler <notifications@github.com> wrote:

> Did not fix my problem with comiting blobs
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/datajoint/datajoint-python/commit/12085b8aa45a1dc7f7309bd9d973dc9431880878#commitcomment-16612834


ts - от timestamp?


But calling close does nothing else than setting self.goOn to false so the while loop ends. Shouldn't the thread terminate itself after that?