[inst/doc/trees_with_root_edge.R:27:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/trees_with_root_edge.R#L27) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[inst/doc/vignette_get_average_nltt.R:100:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_average_nltt.R#L100) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[inst/doc/vignette_get_nltt_values.R:152:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L152) *style:* **lines should not be more than 80 characters.**
```r
qplot(t, nltt, data = df, geom = "blank", ylim = c(0, 1), main = "Average nLTT plot of phylogenies") +
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/doc/vignette_get_nltt_values.R:152:103:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L152) *style:* **Trailing whitespace is superfluous.**
```r
qplot(t, nltt, data = df, geom = "blank", ylim = c(0, 1), main = "Average nLTT plot of phylogenies") +
                                                                                ^
```
[inst/doc/vignette_get_nltt_values.R:164:53:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L164) *style:* **Trailing whitespace is superfluous.**
```r
phylogenies <- c(phylogeny1, phylogeny2, phylogeny3,
                                                    ^
```
[inst/doc/vignette_get_nltt_values.R:178:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L178) *style:* **lines should not be more than 80 characters.**
```r
qplot(t, nltt, data = df, geom = "blank", ylim = c(0, 1), main = "Average nLTT plot of phylogenies") +
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/doc/vignette_get_nltt_values.R:178:103:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L178) *style:* **Trailing whitespace is superfluous.**
```r
qplot(t, nltt, data = df, geom = "blank", ylim = c(0, 1), main = "Average nLTT plot of phylogenies") +
                                                                                ^
```
[inst/doc/vignette_get_nltt_values.R:180:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_get_nltt_values.R#L180) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[inst/doc/vignette_nltt_statistic.R:21:22:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L21) *style:* **Trailing whitespace is superfluous.**
```r
  ape::rcoal(n = 10),
                     ^
```
[inst/doc/vignette_nltt_statistic.R:22:22:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L22) *style:* **Trailing whitespace is superfluous.**
```r
  ape::rcoal(n = 10),
                     ^
```
[inst/doc/vignette_nltt_statistic.R:23:16:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L23) *style:* **Trailing whitespace is superfluous.**
```r
  species_tree,
               ^
```
[inst/doc/vignette_nltt_statistic.R:39:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L39) *style:* **lines should not be more than 80 characters.**
```r
rect(par("usr")[1],par("usr")[3],par("usr")[2],par("usr")[4], col = colors()[240])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/doc/vignette_nltt_statistic.R:39:20:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L39) *style:* **Commas should always have a space after.**
```r
rect(par("usr")[1],par("usr")[3],par("usr")[2],par("usr")[4], col = colors()[240])
                   ^
```
[inst/doc/vignette_nltt_statistic.R:39:34:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L39) *style:* **Commas should always have a space after.**
```r
rect(par("usr")[1],par("usr")[3],par("usr")[2],par("usr")[4], col = colors()[240])
                                 ^
```
[inst/doc/vignette_nltt_statistic.R:39:48:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L39) *style:* **Commas should always have a space after.**
```r
rect(par("usr")[1],par("usr")[3],par("usr")[2],par("usr")[4], col = colors()[240])
                                               ^
```
[inst/doc/vignette_nltt_statistic.R:45:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L45) *style:* **lines should not be more than 80 characters.**
```r
  nLTT::nltt_lines(p, lwd = 3, lty = 2, col = grDevices::heat.colors(n = 4)[col_index])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/doc/vignette_nltt_statistic.R:54:27:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L54) *style:* **Variable and function names should be all lowercase.**
```r
  nltt_stats_exact[i]  <- nLTTstat_exact(species_tree, p)
                          ^~~~~~~~~~~~~~
```
[inst/doc/vignette_nltt_statistic.R:55:27:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L55) *style:* **Variable and function names should be all lowercase.**
```r
  nltt_stats_approx[i] <- nLTTstat(species_tree, p)
                          ^~~~~~~~
```
[inst/doc/vignette_nltt_statistic.R:62:38:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L62) *style:* **Trailing whitespace is superfluous.**
```r
  nltt_stat_exact = nltt_stats_exact,
                                     ^
```
[inst/doc/vignette_nltt_statistic.R:69:14:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L69) *style:* **Trailing whitespace is superfluous.**
```r
  nltt_stats,
             ^
```
[inst/doc/vignette_nltt_statistic.R:70:21:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L70) *style:* **Trailing whitespace is superfluous.**
```r
  id.vars = c("id"),
                    ^
```
[inst/doc/vignette_nltt_statistic.R:76:13:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L76) *style:* **Trailing whitespace is superfluous.**
```r
  df$method,
            ^
```
[inst/doc/vignette_nltt_statistic.R:82:13:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L82) *style:* **Trailing whitespace is superfluous.**
```r
  data = df,
            ^
```
[inst/doc/vignette_nltt_statistic.R:89:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_nltt_statistic.R#L89) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[inst/doc/vignette_stretch_nltt_matrix.R:91:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/inst/doc/vignette_stretch_nltt_matrix.R#L91) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[tests/testthat/test_calculate_weight.R:2:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L2) *style:* **Use two spaces to indent, never tabs.**
```r
	weights <- rep(1, 2);
^
```
[tests/testthat/test_calculate_weight.R:3:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L3) *style:* **Use two spaces to indent, never tabs.**
```r
	particles <- list();
^
```
[tests/testthat/test_calculate_weight.R:4:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L4) *style:* **Use two spaces to indent, never tabs.**
```r
	for(i in 1:2) {
^
```
[tests/testthat/test_calculate_weight.R:5:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L5) *style:* **Use two spaces to indent, never tabs.**
```r
		particles[[i]] <- c(10, 10)
^~
```
[tests/testthat/test_calculate_weight.R:5:17:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L5) *style:* **Put spaces around all infix operators.**
```r
	weights <- rep(1, 2);
               ~^
```
[tests/testthat/test_calculate_weight.R:6:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L6) *style:* **Use two spaces to indent, never tabs.**
```r
	}
^
```
[tests/testthat/test_calculate_weight.R:6:19:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L6) *style:* **Put spaces around all infix operators.**
```r
	particles <- list();
                 ~^~~
```
[tests/testthat/test_calculate_weight.R:7:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L7) *style:* **Use two spaces to indent, never tabs.**
```r
	sigma <- 1;
^
```
[tests/testthat/test_calculate_weight.R:7:12:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L7) *style:* **Place a space before left parenthesis, except in a function call.**
```r
	for(i in 1:2) {
           ^
```
[tests/testthat/test_calculate_weight.R:8:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L8) *style:* **Use two spaces to indent, never tabs.**
```r
	prior_density_function <- function(x) {
^
```
[tests/testthat/test_calculate_weight.R:9:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L9) *style:* **Use two spaces to indent, never tabs.**
```r
		dunif(x[1], min=0, max=1000) *
^~
```
[tests/testthat/test_calculate_weight.R:10:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L10) *style:* **Use two spaces to indent, never tabs.**
```r
		dunif(x[2], min=0, max=1000) }
^~
```
[tests/testthat/test_calculate_weight.R:11:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L11) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[tests/testthat/test_calculate_weight.R:11:32:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L11) *style:* **Put spaces around all infix operators.**
```r
	prior_density_function <- function(x) {
                              ~^~~
```
[tests/testthat/test_calculate_weight.R:12:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L12) *style:* **Use two spaces to indent, never tabs.**
```r
	expect_equal(
^
```
[tests/testthat/test_calculate_weight.R:13:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L13) *style:* **Use two spaces to indent, never tabs.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function), #log(5) - log(10) == -1 * (log(20) - log(10))
^~
```
[tests/testthat/test_calculate_weight.R:13:46:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L13) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
		dunif(x[2], min=0, max=1000) }
                                             ^
```
[tests/testthat/test_calculate_weight.R:14:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L14) *style:* **Use two spaces to indent, never tabs.**
```r
		calculate_weight(weights, particles, current=c(20, 20), sigma, prior_density_function), #so these should be equal, because jumping densities are symmetric (dnorm)
^~
```
[tests/testthat/test_calculate_weight.R:14:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L14) *style:* **Trailing whitespace is superfluous.**
```r

^
```
[tests/testthat/test_calculate_weight.R:15:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L15) *style:* **Use two spaces to indent, never tabs.**
```r
		tolerance = 0.0001
^~
```
[tests/testthat/test_calculate_weight.R:16:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L16) *style:* **Use two spaces to indent, never tabs.**
```r
	)
^
```
[tests/testthat/test_calculate_weight.R:16:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L16) *style:* **lines should not be more than 80 characters.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function), #log(5) - log(10) == -1 * (log(20) - log(10))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test_calculate_weight.R:16:61:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L16) *style:* **Put spaces around all infix operators.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function), #log(5) - log(10) == -1 * (log(20) - log(10))
                                                           ~^~
```
[tests/testthat/test_calculate_weight.R:16:90:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L16) *style:* **Commented code should be removed.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function), #log(5) - log(10) == -1 * (log(20) - log(10))
                                                                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test_calculate_weight.R:17:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L17) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[tests/testthat/test_calculate_weight.R:17:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L17) *style:* **lines should not be more than 80 characters.**
```r
		calculate_weight(weights, particles, current=c(20, 20), sigma, prior_density_function), #so these should be equal, because jumping densities are symmetric (dnorm)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test_calculate_weight.R:17:61:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L17) *style:* **Put spaces around all infix operators.**
```r
		calculate_weight(weights, particles, current=c(20, 20), sigma, prior_density_function), #so these should be equal, because jumping densities are symmetric (dnorm)
                                                           ~^~
```
[tests/testthat/test_calculate_weight.R:18:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L18) *style:* **Use two spaces to indent, never tabs.**
```r
	expect_equal(
^
```
[tests/testthat/test_calculate_weight.R:19:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L19) *style:* **Use two spaces to indent, never tabs.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function),
^~
```
[tests/testthat/test_calculate_weight.R:20:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L20) *style:* **Use two spaces to indent, never tabs.**
```r
		calculate_weight(weights, particles, current=c(5.0, 5.0), sigma, prior_density_function),
^~
```
[tests/testthat/test_calculate_weight.R:20:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L20) *style:* **Trailing whitespace is superfluous.**
```r

^
```
[tests/testthat/test_calculate_weight.R:21:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L21) *style:* **Use two spaces to indent, never tabs.**
```r
		tolerance = 0.0001
^~
```
[tests/testthat/test_calculate_weight.R:22:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L22) *style:* **Use two spaces to indent, never tabs.**
```r
	)
^
```
[tests/testthat/test_calculate_weight.R:22:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L22) *style:* **lines should not be more than 80 characters.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test_calculate_weight.R:22:61:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L22) *style:* **Put spaces around all infix operators.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function),
                                                           ~^~
```
[tests/testthat/test_calculate_weight.R:22:88:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L22) *style:* **Trailing whitespace is superfluous.**
```r
		calculate_weight(weights, particles, current=c(5, 5), sigma, prior_density_function),
                                                                                ^
```
[tests/testthat/test_calculate_weight.R:23:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L23) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[tests/testthat/test_calculate_weight.R:23:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L23) *style:* **lines should not be more than 80 characters.**
```r
		calculate_weight(weights, particles, current=c(5.0, 5.0), sigma, prior_density_function),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test_calculate_weight.R:23:61:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L23) *style:* **Put spaces around all infix operators.**
```r
		calculate_weight(weights, particles, current=c(5.0, 5.0), sigma, prior_density_function),
                                                            ^~
```
[tests/testthat/test_calculate_weight.R:24:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L24) *style:* **Use two spaces to indent, never tabs.**
```r

^
```
[tests/testthat/test_calculate_weight.R:26:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L26) *style:* **Trailing whitespace is superfluous.**
```r

^
```
[tests/testthat/test_calculate_weight.R:27:1:](https://github.com/richelbilderbeek/nLTT/blob/8163f359b50153aed2bfaa3333a54725e7e39ca2/tests/testthat/test_calculate_weight.R#L27) *style:* **Trailing whitespace is superfluous.**
```r

^
```


Should we still keep the line "(use `--strict` to show all issues, `--help` for options)." to signal user we do have "--strict" and "--help" option?


Go ahead and close this commit. I will open a new one when we get through this next round of changes. I just wanted you to have a general idea of what is being done. One of the things that I hand to change was the auto complete. It was out dated and no longer supported. So I put in a new one.  Also, I increase the size of the model to accommodate more features and make it easier to see and use.


Since this release, node-newrelic requires npm v3+!

I have the following error with npm v2.14 :
```
module.js:340
    throw err;
    ^

Error: Cannot find module 'core-util-is'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:289:25)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (/opt/applications/SncfFusionIdCab/build/node_modules/newrelic/node_modules/readable-stream/lib/_stream_readable.js:46:12)
```

It think it would be better to support at least node v4 & npm v2.


Do we need this as a separate method? I believe that we can still manage to have a single method for displaying a book and rely on partials. Alternatively, we could agree that we create a separate action for each book (as the landing pages may be entirely different) and then delete the generic `show` action.


Probably a good idea. :sweat_smile:


Well the udev rules might be part of upstream. Specific to all Linux
distros.


@olecrivain We have unpublished version 1.30.2, please run 1.30.1 for now and we'll have a new version posted soon!


nasser and hosseyn bolded..!


Why do you need this Select? you can just AddRange(result)


clear TCP/IP buffer.


put the name crawl instead of docCrawls


You shouldn't have to remove any `@Override`. You should be able to compile against the newest RN but run on older RN. This is how Android works as well: you always compile against the newest SDK but put runtime checks in to ensure compatibility with older versions. In this case that would be implementing the old callback to simply call the new one.


I wouldn't worry about it though, I'd just make it work with the latest RN only and if people aren't willing to upgrade they can use the older version of your library as well.


Why the old  frameCount here relevant?


wrong condition: docIPs can can be xxx.xxx.xxx and visitInfo.IPAddress is xxx.xxx.xxx.xxx


wrong condition: docIPs can can be xxx.xxx.xxx and visitInfor.IPAddress is xxx.xxx.xxx.xxx


Ok thanks :)


At glance, I don't see any problems - I guess you would have noticed those in your tests anyway. Running without prune target still works, I assume?

Let me know if you feel like this is ready for merge, I don't have any objections.


I'm still testing the actual algorithm on real-size models. The previous version was overshooting sometimes. I'll let you know when I think it is stable


I mainly asked for simpler solution than sudo/dialout and I think that upstream developers have the biggest knowledge.

esp-udev-rules is just suggestion/note if nobody has a better idea.


I frankly don't understand why you would need something like this...


And everyting goes on if I comment the line7 of schedule.php

> $event_common_db->execute_query('ATTACH DATABASE `data/event/marathon.db_` AS marathon');


The Legendary Voyager1!!!!111!!!!


minetest.get_node ↑ either returns a table or it crashes due to wrong argument

What is the meaning of testing if under exists?


What's the worst that could happen if a malicious site makes calls on this endpoint?


I'm actually working on providing rundeck to alpine linux. I wonder why a "ln"' in 2 steps instead of one; is there a reason I miss ?


It builds successfully too!


next


1、修复freeModbus对于发送和接收的超时时间导致9600波特率下出错
2、新增更合理的Print调试打印
===========下一步===========
1、准备修复freeModbus定时器和重组代码
2、对于读取逆变器的数据加以更好的处理


![Szeptember 14. - Szerda](http://ujlipot.vakvarju.com/data/napimenu/22.jpg?v=1473664774)
Nyírségi gombócleves
Rozmaringos csirkecomb velesült hagymás burgonyával
Francia mákos krémes


Hello. I'm wondering why -regex ".*.bat$$" and not -name "*.bat". Is there a good reason I'd miss ? (I'm working on providing rundeck to alpine linux now)


This line, the reference to scrollEnabled specifically, caused an app to be rejected by Apple last week. It got marked as using a Private API.

 I removed the if code block and rebuilt the module and resubmitted the app to Apple and it was approved yesterday.

I guess they don't like the conversion of a UIView (does not officially have scrollEnabled) to UIScrollView (does have the scrollEnabled property).


This looks like a backup file, and should be removable.


Sounds good. This is definitely going in the right direction and looking forward to the next round of changes.


This command didn't work for me: nothing produces the target/dependency directory.

Instead of trying further in this direction, you could just use the exec maven plugin: `mvn exec:java -Dexec.mainClass=org.flywaydb.sample.mongodb.Main -pl flyway-sample-mongodb`


Я смотрел на него, но не сообразил как его заставить работать в полуавтоматическом режиме..


Why is it plural?


The reason that we have to change the owner is so that if the process is started with another UID then they can control their console (that's why this code exists in the first place). I'm on the fence about how much we need to do here -- but not changing the group is "good enough" IMO (it handles all of the standard uses of `devpts` as well as all of the different mapping combinations you can have). Though I am, of course, open to discussion about nicer ways of solving this issue.


Папку .ideа удали из репозитория и добавь в .gitignore, чтобы она больше не попадала сюда. Эта папка хранит настройки RubyMine, а если говорить о командной разработке, то эти настройки могут быть разными у разных разработчиков. Поэтому все, что специфично для конкретного разработчика, по хорошему, не должно присутствовать в репо.


Create Project in yii11BKLCP59/yii11/app


@snicolet Your test has in fact another bench number than Marco's one and seem to resist much more at SPRT[-3,1]
Does this mean that we are actually calling SEE on illegal moves (not-evasion moves when we are in check)?


это весь патч ? мало изменений или так и должно быть


Same, always happen on me too. that script sometime added by itself -_-


this line is not needed. in fact if bluez5 is in DISTRO_FEATURES, bluez5 will be automatically added to the image.

what I had in mind was to install the BT firmware *only* if bluez5 is in DISTRO_FEATURES.


I don't know. I think @iurevych can know this. I've just modified because it's a second time when customer copies this code and gets error.


Amazing VoyagerOne! Congratulations!!


how i can build it in windows.'
can you please share the information


not reqd


mấy cái thư viện tự tạo thì nó add vào chỗ nào anh Sơn


amen


When running a nowcast over long time periods (7+ years), this change to vic.py causes the popen argument to be too long for my OS (14.04 on server with good specs) so I had to change it back. Example:

```
Traceback (most recent call last):
  File "./rheas", line 16, in <module>
    sys.exit(rheas.run())
  File "/home/ubuntu14/RHEAS/src/rheas.py", line 89, in run
    nowcast.execute(dbname, options)
  File "/home/ubuntu14/RHEAS/src/nowcast.py", line 193, in execute
    runVIC(dbname, options)
  File "/home/ubuntu14/RHEAS/src/nowcast.py", line 30, in runVIC
    runDeterministicVIC(dbname, options)
  File "/home/ubuntu14/RHEAS/src/nowcast.py", line 79, in runDeterministicVIC
    model.save(saveto, savevars)
  File "/home/ubuntu14/RHEAS/src/vic/vic.py", line 598, in save
    "date"], "{0}".format(var), initialize, skipsave=skipsave)
  File "/home/ubuntu14/RHEAS/src/vic/vic.py", line 556, in writeToDB
    proc = subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
  File "/usr/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1327, in _execute_child
    raise child_exception
OSError: [Errno 7] Argument list too long
```


I did like the concept of `advance_on_success` to keep the `transform` function simple. But I also used it in a slightly more sensible way in my Python version than just to return a copy of the state.

Basically, I always (well, for most transformations) search for the next state in the `advance` and `advance_on_success` functions. The transform function then just takes the state and applies the transformation without having to deduce the next state from the one passed into the function.

For instance, imagine a transformation that deletes `BAR` from the following input file:
```
EXAMPLE FOOBAR FOOBAR
```

In my Python version the steps would be:

1. `new` intitialises state with 8 (position of first `FOO`)
2. `transform` checks that state points to a valid position and does the transformation.
```
EXAMPLE BAR FOOBAR
```
3. `advance_on_success` advances state to 12 (position of next `FOO` after old state)
4. `transform` checks that state points to a valid position and does the transformation.

`advance` does basically the same thing but looks for the next position after "state + 1" to skip the unsuccessful state.

Does that make sense to decouple searching for the next possible state and applying the transformation?


Do:
```
self.filtered(lambda x: x.location_id in ps_locations).mapped('product_id').update_prestashop_qty()
```
Better code and less calls to the update method.


This is only needed internally for our cc file, right? Can we move it there?


You could try with `type="date"`


++, thanks


Correct.  It must not be exposed to the module authors.  It is an implementation detail meant to be unknown to module code.

Also it doesn't need to be typedef'ed and can instead simply be
```C++
struct napi_func_cb_info_wrapper {
  napi_value Arguments;
  napi_persistent ReturnValue;
};
```


NO es la 10 pliss


Review, finished. Great job! Try to finish CSS exercises for tomorrow.


Create domain model, repository, domain model validator for Activity


I think you are correct, but psql doesn't seem to throw an error. Looking at: https://github.com/Netflix/lemur/blob/master/lemur/tests/conf.py#L53. The password is `lemur` so that means one way or the other postgres is dropping single quotes. That's the only way I can think of that the tests are successfully running.


Need more detail.


Chnages to work Autosuggession in Matter and Document page.


> Where in the Desktop version of Github is the Pull Request command?

I've never seen functionality in a client app for git to help open Pull Requests. You'll need to do it directly on GitHub.

1. navigate to https://github.com/Esri/developer-support/pulls
2. click `New Pull Request`

Once you've 'opened a pull request' you'll choose `gh-pages` as the base branch and compare it to the branch where you committed your own changes, whether they've already been pushed to this repository or your own fork.

this is a workflow that allows people to *propose* merging new work, whether they have write permission to a codebase or not.


Git told me that all the lines changed, I'm not sure what happened. Forgot about these changes though.


fix html


@Legend286

I understand your frustration in regards to the compiling process of OpenTomb. The team have discussed improving the build system on various occasions. Due to limited resources and time, nothing has been done yet. The readme.md included in the repo has basic information on how to compile the engine. I'm fully aware this is no where near enough, is also outdated and should explicitly state which versions of the libraries should be used.

I'm going to see if I can pull someone from the team to update this information as I'm pre-occupied at the
moment. I'm also using custom hacks (undefining macros) to build the engine because I get problems myself. We currently use CMake to setup the dependencies and generate project files.

Bullet and FreeType is included so you don't need to compile those libraries.

You'll have to compile the following libraries and link them:
Lua 5.2
OpenAL
Libpng 1.2
SDL 2.0.3/2.0.4
Zlib

I'm going to lock this, further discussion should continue here: https://github.com/opentomb/OpenTomb/issues/383

Cheers.


@bcoe great, I also do this in more recent projects, along with https://github.com/semantic-release/semantic-release for versioning 👍


do you plan to upgrade to react router 4.0 as well? https://github.com/ReactTraining/react-router/tree/v4


aa


Yayayay! Thank you so much!


I dont suppose support for discovery.ca could also be added by any chance


This import is not used in the example


It will make the monster avoid the field harder, I don't think the monster should NOT avoid the tile if it is already affected by the condition, as I already saw strange behavior with TFS 1.x. Player would make use of a field to make the monster dance step over it. I also don't know why it should be so harded to avoided with a 18 multiplier if the monster is not immune/affected at all.


`insance` -> `instance`


ditto


You can simplify the code quite a bit by using the V8 types directly in this struct as well.  You can do this since it is only used within the implementation of the API and not exposed to the users of the API.

Also, wrapper is not really the right term since it isn't wrapping a `napi_func_cb_info`.  Instead it is providing the implementation, so a name like `napi_func_cb_info_impl` would be better.

So in addition to being moved to the .cc file, the struct should look like this:
```C++
struct napi_func_cb_info_impl {
  const v8::Arguments* Arguments;
  v8::Persistent<v8::Value>* ReturnValue;
};
```

Then you can simply copy the arguments and return value pointer from `v8impl::FunctionCallback` directly into the struct and you won't need the `JsFunctionCallbackInfoFromV8FunctionCallbackInfo` helper or the other conversion helpers at all.  Much simpler.


@olalonde nice, I've been wanting to take the leap and move towards a fully automated approach like that.


bbb


Ideally we would be pulling this back from the API on loading the site so we always have the latest data. I know that states are a lot more static, but for things like cities, counties, zip codes and schools etc we should try to make them all dynamic and not statically coded. OR If they are statically coded, move this to config.js or a constants.js file that we can have all constants in one place and not mixed in with all the js code. Make sense? If you would like I can create a constants service for this purpose and check it in tonight. Let me know.


Sorry to nitpick, but does this render to a link since there's no `for-label`? I also wonder if it is dangerous to add a `for-label` in this case...


I have done this as well, but let's refrain from checking in console.logs and commented out code.


TeamCity Umami :: WebClient :: Sites :: Micro Sites :: GC Microsite - Restaurant Search :: Pull Request Test [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=5216380&buildTypeId=Umami_WebClient_MicroSites_GcMicrositeRestaurantSearch_PullRequestTest) is now running


Is your bootstrap all being coded to have a fallback for mobile? col-md and col-xs...


no, that's fine... I went through the same thought process. It *doesn't* render as a link, and I decided I didn't want to wrestle with that import.


Should always use === instead of ==. Javascript best practice.  A lot of these things are caught by JSHINT. So please don't turn it off locally on your machine or check in code without doing "grunt build" to make sure it is all valid javascript.


I love what I'm seeing here because this would be very easy for me or someone else to modify and refine. 🎉


Any reason this doesn't follow the `cmd/mirrorissues` [pattern](https://godoc.org/golang.org/x/tools/cmd)? It might be harder to tell it's a command rather than a library at a glance this way.


轉換用 `Translator`，反查用 `Filter`。已經有根據字符集過濾結果的 `charset_filter`，建議本案命名爲 `codepoint_translator`、`codepoint_filter`。


woops, undo this.


这里的表格需要翻译，参照：https://github.com/laravel-china/laravel-docs/blob/5.2/testing.md#使用表单


Why make it blocking?


nvm..


TeamCity Umami :: WebClient :: Sites :: Micro Sites :: GC Microsite - Restaurant Search :: Pull Request Test [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=5216380&buildTypeId=Umami_WebClient_MicroSites_GcMicrositeRestaurantSearch_PullRequestTest) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:04:51


TeamCity Umami :: WebClient :: Sites :: Micro Sites :: GC Microsite - Restaurant Search :: Pull Request Test [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=5216440&buildTypeId=Umami_WebClient_MicroSites_GcMicrositeRestaurantSearch_PullRequestTest) is now running


Oh we also noticed that after doing this simplification, the ReturnValue field wouldn't need to be a pointer and you can use it directly instead of having a local variable.


如果你的表单包含 `上次文件` 的输入框类型


If one of the maintainers is an "Apple Developer" here's [a guide](https://successfulsoftware.net/2012/08/30/how-to-sign-your-mac-os-x-app-for-gatekeeper/) to sign the application.


TeamCity Umami :: WebClient :: Sites :: Micro Sites :: GC Microsite - Restaurant Search :: Pull Request Test [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=5216440&buildTypeId=Umami_WebClient_MicroSites_GcMicrositeRestaurantSearch_PullRequestTest) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:01:34


To open Matter Landing Page in the same tab in MatterDashboard


Je prefere avoir mon nom de famille


    Assert that the view has a given list of bound data.

翻译为：

    断言响应视图里存在输入数组里的所有数据


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (2334 of 2335 tests passed, 0 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


Tu pense que c'est plus loin mais t'es encore debutant


you probably meant items[i](str) ?

Same in the orMatcher() method above.


上面这个代码块里有三句注释，麻烦翻译一下。


以下の様にsimple_switch_13.SimpleSwitch13を継承し
switch_features_handlerとadd_flowを削除しました。

import simple_switch_13
class SimpleSwitch13(simple_switch_13.SimpleSwitch13):

Ryu-bookの内容に従いアプリケーションを実行したところ、
ポートステータスの管理部分にバグがありましたので、
別コミットにて修正します。


@treo I get that CMake has issues with that, but now libnd4j gets linked differently on Linux because it no longer links with $ORIGIN but with $$ORIGIN. We may have issues because of that, so please make it so that it still links with $ORIGIN on Linux, not $$ORIGIN, thank you.
```
$ readelf -d blasbuild/cpu/blas/libnd4j.so  | grep RPATH
 0x000000000000000f (RPATH)              Library rpath: [$$ORIGIN/]
```
This needs to be $ORIGIN, *not* $$ORIGIN.


@yyuho
メールでの伝え方が悪く、申し訳なかったのですが、
`var askUserName = function askUserName(){`
ではなく、
`function askUserName(){`
でも動くはずなので、試してみてください。

内部的には変わらないので変更しなくてもよいのですが、可読性としては後者のほうがスッキリします。


make love


@Shashank-In http://www.testingsecurity.com/whitelists_vs_blacklists


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2369 of 2370 tests passed, 0 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


@lestertj It's add your own name to the first line of the file leh


Extracting the sender from the first line would make it easier for the server, don't you think?

E.g. For usage in errorReply()


dfsdfsdfsdfsdffd


dfsdfsfdfsdfsf


This looks awesome~!
cool solution. Whats the problem? it ran fine on repl.it


I see in `napi_set_return_value` you assign a new persistent to the dereferenced pointer ReturnValue.  My understanding from looking at the v8 code is that `v8::Persistent<>` is copyable, so you should be able to avoid the pointer and simply copy the new value over the old value.

E.g.

```C++
void napi_set_return_value(...) {
   ...
   cbinfo.ReturnValue = v8::Persistent<v8::Value>::New(val);
}
```


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - 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-passing-green.svg)


u make very pretty grids man


Ok, i was confused, this code here then only prevents combining an unknown slab with itself, so probably isn't necessary.


I agree with you, the players would make the monster dance around fields...I don'n know the formula hehe

Thanks a lot dude! =)


@vangheem Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/46/violations
```
plone/namedfile/browser.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/copy.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/marshaler.py:8:1: C901 'If 8' is too complex (13)
plone/namedfile/scaling.py:248:1: C901 'ImageScaling.create' is too complex (13)
plone/namedfile/scaling.py:294:41: S001 found %r formatter
plone/namedfile/tests/base.py:54:16: P301 format call provides unused index (1)
plone/namedfile/tests/test_scaling.py:91:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:92:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:109:14: D001 found failIf( replace it with assertFalse
plone/namedfile/tests/test_scaling.py:158:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:159:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:166:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:167:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:174:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:175:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:215:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:216:70: S001 found % formatter
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Works fine, thank you!


@vangheem Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/47/violations
```
plone/namedfile/browser.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/scaling.py:173:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


python 多线程执行


I can also make the same change but if you would like to create a pull request, so you get credit for fixing the issue, that also is fine.


Projeto 1 - POO


Feel free to move it.


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1762 of 1763 tests passed, 0 test warnings)

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


Af hverju viljum við fela miniChart-ið í töflusýninni?


Bugfixes


@InfernoZeus @scowalt @NGMarmaduke
Thanks!


my only suggestion would change the solutioncount++ (s++) part out of ur print function
since print is doing its own thing id move it to the 8queens fxn so the code just reads more logically and u can see the steps laid out.
Example here.
https://repl.it/DaAS/0


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Yes. Will also drop redial for redux-taxi or another solution that doesn't depend on route-level data-fetching


Besta ternary lína EVER


上次，是上传文件吗？


Curious, why does this API require a string but Set and Get do not?


This appears to break building on OSX mavericks. I'm currently fighting with the homebrew. It can't find stdatomic.h

Its using a clang 6.x series compiler so I'm not sure why.


fafafadsfasfsafsdad


Okay, I think it's safe to leave it as is


makingPrettyTransition (mitchute) - i386-Windows-7-VisualStudio-12: OK (2328 of 2335 tests passed, 24 test warnings)

 * 10 tests had: AUD diffs.
 * 31 tests had: ERR diffs.
 * 7 tests had: Table big diffs.
 * 21 tests had: EIO diffs.

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


113836013-Swimming-Pool-Evaporation-and-Cover-Issues (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1190 of 1190 tests passed, 0 test warnings)

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


Huh, no way to see if an object is a symbol?  Looks like `v8::String` is the type used for Symbols.  Is there not a way to ask if a string is a symbol?


Probably not too important since this version of v8 was probably from before ES6 was published.


heyy check out the discord!


@jjskuld heyy check out the discord!


Created this file so that the Acheteurs section can have it's own header message in the event I want to change "L'equipe de courtiers...."


Looks pretty well to me yaw


I thought you already moved those to paths.hpp|cpp?


Ah, that was there before and I forgot to remove it.


Hidden message!: TABS not good in Git


Put this line back


Put this line back. It uses the site name and description in the `_config.yml` file.


 blockquote, div.yahoo_quoted { margin-left: 0 !important; border-left:1px #715FFA solid !important; padding-left:1ex !important; background-color:white !important; } Is working

Sent from Yahoo Mail for iPad

On Wednesday, September 14, 2016, 5:32 AM, Valentyn Kolesnikov <notifications@github.com> wrote:

You may tro import HttpClient from package com.github.javadev.undescriptive.client.

Kind regsrds, Valentyn

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


I get an error pertaining to this commit stating initializer for conditional binding must have optional type, not 'UIView. I included turbolinks using cocoapods. The error goes away if I undo this commit in my code, but when i build the app freezes when calling the visit url function. I am not sure if its because of this.


Is working now


El orden de inicialización de estos atributos debe ser el mismo que el que están declarados en el .h.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1780 of 1780 tests passed, 0 test warnings)

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


Esto no está muy bueno, hay un vector de vectores de CuadritoTetris. Me parece que es una estructura de datos complicada y poco conveniente.


this is where you make comment of each version changed


Poné el nombre de la imagen en una constante global.


Esto no tiene muy buena pinta, es una estructura muy complicada. Por favor, pensá otra opción.


Magic Number


I am idiot I think. I try this - http://pastebin.com/fF3kAmNk
And I have this - http://imgur.com/a/24RrI


bin/


う～ん...
このタイトルだけだと、とても広範囲にわたる修正をしているように読めませんか？
`lib: Fix for bug of port status`

また、コミット文では
　「このパッチでバグが組み込まれました」
とだけ説明していて、
　「どんなバグか？」
はほとんど説明できていないと思います。
Ryu-devel でやり取りした経歴も無いバグですので、
ある程度説明は必要かと思います。

どういうか、ですが、このパッチは大きく下記の２件ではないでしょうか？

1. Python2/3 互換の問題(MAC の比較)
2. PortStatus の問題(前回状態の保持)

```
stplib: Fix to compare MAC address and Bridge ID

cmp() func was introduced for Python 3 compatibility before,
but this implementation is not enough, because a MAC address
can not be compared with a Bridge ID (integer value) by com() func.

This patch fixes to convert the MAC address into an integer value
before comparing with Bridge ID and fixes this problem.
```

```
stplib: Fix to store current OFPPort status

Currently, stplib compares the status of OFPPortStatus message with
that of Datapath instance, but this comparison returns always "equal",
so stplib fails to get the correct port status.

This patch fixes to store the current port status and to compare the
new status with the stored status.
```


Why do we need to create a new HandleScope? Is it because the throw is global?


このインデント追加は余計ではないでしょうか？


Poner el código de esta funcion en el .cpp. Estás haciendo un poco mas que setear un valor. Por otro lado, por que un set devuelve un valor?


Removing the copyright notice is a violation of the MIT license.  Please add it back in!


No me convence mucho esta estructura de datos.


os_2_0_develop_linux (macumber) - 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)


Esto esta bueno, sin embargo, creo que puede ser mucho mas útil hacer que el Tetromino sea genérico, y tengas una implementación de cada tetromino por separado, y no un Tetromino que pueda ser todos a la vez. Esto es poco escalable.


destructor debe ser virtual


hmmm, won't this break stuff like `f(multiReturnCall())`?


nuevo hecho actualmente


Está bueno usar containers de stl, sin embargo, esto puede ser demasiado para Cuadrito Tetris. Probá usar arreglos comunes.


ここはsprintf使わないで教科書に従おう


Agregar destructor virtual


Destructor virtual


ここちょっと変


これは最後に書いて説明する
```
if (!isset($_SESSION['join'])) {
  header('Location: index.php');
  exit();
}
```


Great!


Magic Numbers


os_2_0_develop_linux (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


ok


```
Just for information, the source files for application of Ryu-Book
will Integrate as for OpenFlow1.3.
```
このコミットコメントは必要なのでしょうか？
下記のように英文法で怪しいところがありますが、
それを押してでも書きたかった内容は何でしょうか？

- 「ちなみに」という文言は「FYI」とよく表現されているようです。
- 「Integrate」の部分で、大文字始まりとなっていること、また、
他動詞なのに目的語（何を統合するか、の「何」）が無いです。


Why? Whatever happened to duck typing?


os_2.0_develop_signal_slot_bug (kbenne) - 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)


LOL. Here I am trying to work out why my patch wouldn't merge and it's because you've done it for me. My original patch was half-baked. This fixes it.  My version of _getTime (which I called _getTimeFields) returned an object so you could go: now = this._getTimeFields() and then use "this._month < now._month" without all the extra function calls.

I'm happy to patch that version back if anybody is interested. Here's my function:
DailyRotateFile.prototype._getTimeFields = function () {
  var now = new Date();
  return this.localTime ? {
    _year: now.getFullYear(),
    _month: now.getMonth(),
    _date: now.getDate(),
    _hour: now.getHours(),
    _minute: now.getMinutes(),
    _weekday: weekday[now.getDay()]
  } : {
    _year: now.getUTCFullYear(),
    _month: now.getUTCMonth(),
    _date: now.getUTCDate(),
    _hour: now.getUTCHours(),
    _minute: now.getUTCMinutes(),
    _weekday: weekday[now.getUTCDay()]
  };
};


:laugh:


This change is broken at least in RPi2 builds. Reverting to 6090a90 fixes.


1. The issue with the branch name was fixed.
2. You don't need bash with `fly hijack`, it just default to bash. Please just use `fly hijack <..args..> sh`.
3. Is the operation that expensive?

@xinzweb, please see above. ^^^


I don't want to open an issue because it looks like Facebook's fault but just wanted to mention this doesn't work anymore. It looks like Facebook ditched the pkg versions, and moved onto zips.


・$_REQUESTは、$_GET、$_POST、$_COOKIEの値が入っている
・$_GETでも$_REQUESTでもどちらでも


Why do we want to add the confirmed and lockable stuffs? Are we going to send email for the user account confirmation?

BTW: you have created an new task for email setup. it will be better for those changes into that task


Just a bit confused, you said that we wanted this password controller and i just reverted it....


Oh. #26.


Excelente!!! Muy bueno el código!!! Sin embargo me gustaría que revisemos la clase board que me parece que está muy sobre cargada de cosas, hay algunos problemas cohesión. La clase tetrimino también hay que repensarla. El main debería ser un poco mas pelado. He visto que no has usado herencia, ni polimorfismo. Tampoco has utilizado punteros (salvo los que sdl te impone) y referencias cuando creo que lo podrías haber hecho. En algunos casos has utilizado smart pointers, lo cual está bien, pero está mejor que te topes con los problemas comunes de manejo de punteros y puedas resolverlos. Es una buena oportunidad para trabajar con eso y experimentar problemas.

Mas allá de eso, está excelente!!! Adelante!!!


Yeah, my pull request for mysqli was a hack, but switching to 'dataProvider'  would probably break other db drivers usage in xmlschema03 platform attributes. There are at least 6 driver classes that have value 'native' for dataProvider. Other drivers use 'odbc' as dataProvider..

The naming 'databaseType', 'dataProvider'  and 'platform' and their values in the drivers feel a bit creepy to me.

E.g. databaseType is mysqli for adodb driver mysqli, but dataProvider is mysql. My tiny brain would like to switch the values, because the databaseType in my logical understanding is always mysql, but the driver differs..

This quick'ndirty script generates the table below showing the 'inconsistency' between different adodb drivers.

```php
<?php #error_reporting(E_ALL);
include("../adodb.inc.php"); $d=array();
if ($handle = opendir('../drivers')) {
        while (false !== ($driver = readdir($handle))) {
                $driver=substr($driver, 6); # strip 'adodb-'
                if( # only test adodbdrivers that do not simply die() on my environment
                   substr($driver,0,1)=='f'
                || substr($driver,0,3)=='acc'
                || substr($driver,0,1)=='b'
                || substr($driver,0,3)=='csv'
                || substr($driver,0,6)=='db2oci'
                || (substr($driver,0,1)=='i' && !substr($driver,0,10)=='informix72')
                || substr($driver,0,4)=='ldap'
                || substr($driver,0,1)=='o'
                || substr($driver,0,1)=='s'
                || substr($driver,0,1)=='t'
                || substr($driver,0,6)=='sqlite'
                || (substr($driver,0,5)=='mysql' && substr($driver,0,7)!='mysqlpo')
                #|| (substr($driver,0,5)=='mssql' && substr($driver,0,7)!='mssqlpo')
                || substr($driver,0,4)=='pdo.'
                || substr($driver,0,8)=='postgres'
                ){
                        $driver=substr($driver, 0, -8); # strip '.inc.php'
                        #ADOLoadCode($driver);
                        $c = ADONewConnection($driver);
                        $d[]=array(
                                'class'=>get_class($c),
                                'driver'=>$driver,
                                'databaseType'=>$c->databaseType,
                                'dataProvider'=>$c->dataProvider
                        );
                #echo '<pre>';
                #print_r($c);
                #die();
                }
        }
        closedir($handle);

        function drivercmp($a,$b){
                return strcmp($a['driver'], $b['driver']);
        }
        usort($d, 'drivercmp');
        $css='<style>
h2 {text-align:center;}
table {border-collapse:collapse;border:1px solid #ddd;margin-left:auto;margin-right:auto;}
table td, table th {padding:4px;border-bottom:1px solid #ccc;}
</style>
';
        $out.='<table><thead><tr>
<th>PHP class</th>
<th>driver<br> (adodb-*.inc.php)</th>
<th>databaseType</th>
<th>dataProvider</th>
```
generated with adodb 5.20.6:
![drivercompare_adodb5 20 6](https://cloud.githubusercontent.com/assets/1839154/18535646/4d108fc8-7af6-11e6-8090-c3bea69e81c0.png)


@SubJunk Maybe a stupid question, but do these 2 libraries, `org.apache.commons.lang3.StringUtils.*` and  `org.codehaus.plexus.util.StringUtils`, need to be present together ? One will not be sufficient ?


建立

安裝的git的Android NDK
$ export ANDROID_NDK=/path/to/your/android-ndk
$ ./FFmpeg-Android.sh
libffmpeg.so將建成 build/ffmpeg/{neon,armv7,vfp,armv6}/


This isn't exactly correct semantics.  In the newer V8, Escape would write a handle to the parent HandleScope but not close the current scope.  In the older V8, Close actually closes the current scope and makes the previous scope the new current one.  That means the user code can only call this method once, and only after it is done using any locals that were allocated on the top handle scope.

This is a point where our abstraction appears to not work across different VMs.  Something for us to ponder more about.


This is actually already fixed in the latest version of the code, this diff is from an earlier commit. See https://github.com/libretro/RetroArch/pull/3547/files


Deux repo sur github ? Pense à faire le ménage


Message de commit pas adapté et trop flou


Copié collé ? As-tu compris?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1007](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=13312&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 238, ignored: 3 Build time: 00:10:01


Pourquoi ? try again


Et du coup, le web ?


Please do not remove classes at LS core code. What about the people having copied a shipped template and having adjusted it?
Instead, please add additional classes and use more specific CSS definitions like .answertext .text-danger if existing styles need to be overwritten.


this service is tested to be somehow unstable, maybe it should be removed in releases.


copié collé ? Toutes les infos sont-elles vraiment utiles ?


let's rename it to LoadingIndicatoComponent


Plus de source !


lets use LoadingIndicatorComponent instead of shorthand


We have commons-io as a project-wide dependency.
Have a look at this method: https://commons.apache.org/proper/commons-io/javadocs/api-2.5/org/apache/commons/io/FileUtils.html#copyFile(java.io.File,%20java.io.OutputStream)


copié collé ?


Try Again !


tu ne dis pas ce que sont les bang ...


What happens if we can't find the file requested?


Let's try to put everything in order:

- An identifier is the name of a variable **or** a type.
- This looks like a hack since there is no way to maintain this, other than rewriting.
- We have similar problem with "Wring<x>" which changes to "x"
- In short, this needs a quick rewrite. feel free to open an issue (perfection), assign it to me or whoever, and erase these comments from the code.


Trivia: JavaScript in Moodle is now supposed to be camelCase, so this variable should be className (I know that isn't consistent with the existing variables, this code is a bit old).


Trivia: Extra space before || sign


CSS


mazi, the Denis work is for the next major release.


For me the branch seems OK, I think you can launch a SPRT[-3..1] test. Meanwhile I have put the priority of http://tests.stockfishchess.org/tests/view/57d872800ebc59030fbe52af at -5 (and it was about to fail anyway).


I did not try to make coverall to work again recently. I would find time to re-test it.. but time is missing.
Can you be more explicit about the problem you see on your side?
Perhaps it's  just a story of Github authorization for the third_party service.


I like it a lot!


Why you don't let it just fail with message pointing to HAL-1182 ?


On a related note (improving boot behaviour): https://github.com/BPI-SINOVOIP/BPI-M2P-bsp/commit/4ed67b2c9ebc81cf1a06bd9ca4e7d27cbec7a721


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1008](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=13316&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 237, ignored: 3 Build time: 00:09:54


TeamCity Branches Tests :: Git yaegor sandbox [Build 29](http://munit-050:8112/tc/viewLog.html?buildId=99043&buildTypeId=BranchesTests_GitYaegorSandbox) is now running


all that needs would be a tagged release i think. this change is the only one since 0.4 - i'd rather collect more things before doing the next release. if you need the fixed annotations to develop, you can git checkout master && git pull in the vendor directory as a workaround.

/cc @ChristianRiesen


ok, if you have tested it then ok


@ThisTestUser How does the new Gwen work?


@maziminke : here the .label .label-danger is really a bug for all templates.

With text-danger : we put the color red for all template. And remove the nowrap broken system.

You're right on one point : we must add an extra class (just for ls).

Something like that maybe:

````
<tr id="javatbd<?php echo $myfname;?>" class="subquestion-list questions-list array<?php echo $zebra; ?><?php if($error){ echo " has-error";} ?>" <?php echo $sDisplayStyle;?> role="radiogroup" aria-labelledby="answertext<?php echo $myfname;?>">
    <th class="answertext control-label <?php if($error){ echo " ls-error ls-error-mandatory";} ?>"  id="answertext<?php echo $myfname;?>">
        <?php echo $answertext; ?>
````

or:
````
<li id="javatbd<?php echo $myfname;?>" class="radio-item <?php if($error){ echo " has-error";} ?>" <?php echo $sDisplayStyle;?>>
<label class="control-label <?php if($error){ echo " ls-error ls-error-mandatory";} ?>"
````

for list of choices.


添田です。
ここの["$key"]は[$key]で良いですね。


typos with existing (2nd line) and intersection (last line)


Would this be backported to master_2 and 2.16?


A lot of those warnings are not a matter to me. Worse, some of them are really bad – like putting `[cfg(feature…)]` everywhere. Though, I didn’t know about `.cloned()`, I’ll add that. :) Have you tried to use luminance or ion yet? I saw you forking both the repos!


callbackをこのように書くのは、何かの言語の影響なのでしょうか？


It's working fine for me. I think your problem is not from this change.


八月十五


Oh, I see... OK! Right now looking at some real-time values in  /sys/class/power_supply/battery/current_now at 100% and right after that.


fixes #14


1. This bumps the Mednafen requirement to 0.9.39 without changing documentation and without changing error messages.
2. This breaks compatibility with Ubuntu 16.04 which has Mednafen 0.9.38.7 in stock.


:+1: Grazie!


I think it is.


Wauw dit kan beter..


Bennikrechts?


@tanwanirahul for now I do not think maintaining different branches in XGBooost branch is the adopted release strategy. If you like, you can maintain a version in a different repository


May I ask what's the rationale behind this choice? AFAIU `inline` variables don't really provide new functionality; they are purely a more convenient syntax for functionality that already existed. So, under that assumption:

 - the two branches of this `#if` are functionally-equivalent; and
 - the convenience of inline variables will never be enjoyed because we will be using the macro anyway.

So what's the gain?


lgtm


Took me some time to figure this out. According to their documentation a `binary` should work. So I tried using `buf` as is (since it works without options) but then it threw `500`. Converting it to `base64` fixed it though, but yeah, took me so long...


@Shougo Are you going to support Vim 8.0 async API in denite?


qqqqq


TeamCity Branches Tests :: Git yaegor sandbox [Build 29](http://munit-050:8112/tc/viewLog.html?buildId=99043&buildTypeId=BranchesTests_GitYaegorSandbox) outcome was **SUCCESS**
Summary: Running Build time: 0:10:3


TeamCity Branches Tests :: Git yaegor sandbox [Build 30](http://munit-050:8112/tc/viewLog.html?buildId=99044&buildTypeId=BranchesTests_GitYaegorSandbox) is now running


My Sensors 2.0 ESP8266 gateway with DHT22 temperature sensor and IoT integration


argumentsで、関数の引数の配列を引き取れるのは、初めて知りました！
この書き方は面白いですね。

上記だとargsとargumentsが同じ内容になる気がしましたがいかがでしょう？

余談ですが、JavaScriptの最近の仕様では、
`for (v of arr) {}`
で配列の値でループが回せるようになっています。

`in`の文法だとオブジェクトのprototypeもループで回ってしまうことがあるので注意が必要です。
http://qiita.com/C058/items/e1bcd1617188260a9443

あくまで余談ですので、`for ... of`には書き換えなくて結構です。


@PawelGutkowski, are you looking into Travis CI failures? Is there any way to get server logs and display them in Travis-CI results?


@ayende Why did you decide to use `XXHash32` here? Performance? I'm wondering if I should apply the same change in https://github.com/ayende/ravendb/blob/v4.0/src/Raven.Server/Documents/Indexes/MapReduce/ReduceKeyProcessor.cs


Thanks! :)


ova se probav da go menam vrakajki se nazad o 6toto ali nema opcija


Has this bug fix been included in 2.1.1? I can't see it in the change notes? http://devdocs.magento.com/guides/v2.1/release-notes/ReleaseNotes2.1.1CE.html


@flocke hello, Jakob, sorry for dusturbing you. A year ago you made script for changing selinux to permissive
I'm about this post: https://plus.google.com/+JakobNixdorf/posts/BQ7CzPNzZx6
Can you share this script again? Link is broken


質問です。
readline.promptの第一引数が`true`となるとどういう動作なるのでしょう？


I hadn't seen this comment was related to Stas' PR.

I have a few questions before applying this change:

1. How is this fixing the issue? Why did not use `to_i` cause the property to not be changing?
2. method `saveProperty` takes `Long` arguments, shouldn't it be `resource_id.to_l` (if it even exists)


Peer Review Performed. Looks Good.


Thanks, copy and paste error :(


Many thanks for writing these tests!


There are some logs accessible in sauce labs:
![image](https://cloud.githubusercontent.com/assets/17570261/18546666/663ff896-7b3f-11e6-948e-6cb5c422d007.png)
Only way to display server logs I see is to access openmrs.log in docker container in after-script, and display its content with `tail` or some other bash command...


I mentioned this on training.
I think `TravelTimePredictionModel` class should be Serializable.
This gives us another solution of this problem by implementing `PredictionModel` as Checkpointed


`babel-polyfill` should be in our dependencies, not in devDependencies.


Changed holding variable type to Object[] to avoid cast exception for any spark plugin.


Looks good


TeamCity Branches Tests :: Git yaegor sandbox [Build 30](http://munit-050:8112/tc/viewLog.html?buildId=99044&buildTypeId=BranchesTests_GitYaegorSandbox) outcome was **SUCCESS**
Summary: Running Build time: 0:10:1


It should be 4


Done :smiley:


firstcommit


Use deeper indentation for each component.
Such as
```
<div>
    some content
    <section>
        some other content
    </section>
</div>
```


will re upload


For coding make-up style you should run the Uncrustify. src/Tools/selxUncrustify.py


Good point!
done


I like the idea with after-script. Unfortunately, the stacktrace, which can be seen in the screenshot doesn't help me much.


TeamCity Branches Tests :: Git yaegor sandbox [Build 31](http://munit-050:8112/tc/viewLog.html?buildId=99046&buildTypeId=BranchesTests_GitYaegorSandbox) is now running


Thanks. Fixed.


Also p2_it is updated to p2_2_it every iteration


I drew stuff


I think this change means that using a partial path for a component causes errors, as subsequent components also try to use this partial path, e.g. https://github.com/BBC-News/election-2016-renderer/blob/master/src/components/london_banner/models/london_banner.rb#L12


Hmm ... Using the committed settings I always get the following error within the browser:

> Uncaught Error: Expected 'styles' to be an array of strings.

I use the SCSS files within the `app.component.ts` in the following way:
```
@Component({
  selector: 'app',
  encapsulation: ViewEncapsulation.None,
  styleUrls: [
    './app.style.scss'
  ],
  templateUrl: 'app.tpl.html'
})
```


Testing comment


I don't know why its doing it like this though... why is it not just checking the count of `SPRITE_LIST_NULL` and if its 0 return null?


If you want, I can try to setup after-script


It is ok for me, probably

Bitboard defensive = pieces(color_of(piece_on(s)));

will be slower than original version because it is executed always, while
the original version only in very rare cases. So I suggest to drop this to
have higher chances to pass the test.

On Thu, Sep 15, 2016 at 10:49 AM, pb00068 <notifications@github.com> wrote:

> @mcostalba <https://github.com/mcostalba> @snicolet
> <https://github.com/snicolet> I have prepared branch
> <https://github.com/pb00068/Stockfish/tree/correctpin3> for another
> bugfix-attempt. As suggested by Stephane it includes also the bug-fix
> proposed by Ronald to run both fixes within a single SPRT[-3,1] test. Can
> you please take a look at this and say if this should be the next bug-fix
> attempt (hopefully the last one) we should run? Thanks in advance
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/ad7d2bdeedd2b0021e459297f3ddeaf6f8e07d87#commitcomment-19032505>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABDGAVmFVgdTChWvqbi92nqHBD1fTQzCks5qqQaGgaJpZM4J6qB8>
> .
>


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/35436/testReport/) for this commit**


The conversion to int was taking a bit of time here, actually


Hello


who is "root"? I really want to know who I can complain to about shadowing the method parameter 'uid'! https://jira.bikalabs.com/browse/LIMS-2394


Fine!


添田です。
ちゃんとDBの型とかを調べるのは良いですね！僕らが教えない事も多いので、自分から積極的に取りにいってください！


As far as I [can see here](https://github.com/skellock/apisauce/blob/ede9c33c6b972330d14b432d8d9ef0341c2933b4/package.json#L32), `apisauce 0.3.0` depends on `ramda 0.21.0`.

So it's not possible to install `reactotron-apisauce 1.1.2`, is it? Because [at the moment](https://github.com/reactotron/reactotron/blob/f6cc33236fef68063850b17954751198678fd98f/packages/reactotron-apisauce/package.json#L57-L58) that requires both `apisauce 0.3.0` and `ramda 0.22.1`.

I'm in dependency hell right now and seek a way out :)


Actually, I usually don't complain - but I always do want to know.


Yes, please do.

@dkayiwa, @wluyima, do you recall any changes you merged in the last 12 hours, which could cause the above exception? It doesn't seem like upgrading coreapps could cause that.


<form action="welcome.php" method="post">
Name: <input type="text" name="name">

E-mail: <input type="text" name="email">

<input type="submit">
</form>


This approach is pretty inefficient for large layers - you are still iterating through everything. Wouldn't it be better to filter fids if the selection ids are used?


I'll take a look At the assembly but it probably is just an efficient way of doing that


You made these changes on the master branch. Was that intended?


I believe this is for legacy u-boot only, mainline u-boot doesn't reference this option anywhere.


You seem to mistake id with class.
A page should only unique id for every id.
So it's not appropriate to have an id to indicate left or light alignment. It should be class instead

Try removing html~ and css~ files. You should always check what's in staging before committing.

You can have id and class in one tag, and even several classes.
Make sure you add an indentation for every nested tag.
```
<div class="class1 class2" id = "id">
    some content
    <section>
        some other content
    </section>
    <b>last content</b>
</div>
```


messed


TeamCity Branches Tests :: Git yaegor sandbox [Build 31](http://munit-050:8112/tc/viewLog.html?buildId=99046&buildTypeId=BranchesTests_GitYaegorSandbox) outcome was **SUCCESS**
Summary: Running Build time: 0:10:2


@daniktheboss as you can see, without packets we can't do anything


i have it at medium


Crazy man ^^ is it angler blobs or regenerated bsp?


Is it supposed to have a comma here? @maxpeterson


Fixed.
But when compiling for linux I always get
minimize Bin/LINUX/Data/Dialogs/dlgAirspaceDetails.min.xml
  minimize Bin/LINUX/Data/Dialogs/dlgAirspace_L.min.xml
runtime error
xsltApplyStylesheet: saving to Bin/LINUX/Data/Dialogs/dlgAirspaceColours_L.min.xml may not be possible
make: *** [Bin/LINUX/Data/Dialogs/dlgAirspaceColours_L.min.xml] Error 9
make: *** Waiting for unfinished jobs....
  minimize Bin/LINUX/Data/Dialogs/dlgAirspacePatterns_L.min.xml

From: Bruno de Lacheisserie
Sent: Thursday, September 15, 2016 9:58 AM
To: LK8000/LK8000
Cc: Paolo Ventafridda ; Author
Subject: Re: [LK8000/LK8000] Fix minor unused variables out of testbench mode (96dd04a)

bug : no more notification about SerialPort Error with this change.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


> Which property are you talking about?
Sorry for that i missed a names

> I had problem with local consts (not all of them, however) where the build simply didn't run.
It's strange i never experienced this issue on iOS devices, AIR 3.4 or later


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/460/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=XW-2018).


Is crc hack enabled ?


@xeolabs Why is it forced to be 'true' here?


Thank you for the concern. Just double checked on npmjs, and the latest release is `0.3.3` so I think 3 is ok.
If another version is published, I'll update of course :)


TeamCity Branches Tests :: Git yaegor sandbox [Build 32](http://munit-050:8112/tc/viewLog.html?buildId=99047&buildTypeId=BranchesTests_GitYaegorSandbox) is now running


was me... I guess: root <root@lynn>
Apologies... for both things:
a) WTF I used root for this?
b) For shadowing "uid"


in parcial


fast-forwarding riak_ts-develop to borshop-integration-1478-fix-az-explain_column_type = c32b52de


There seems to be a discrepancy in indentation between the vars above and here. Spaces vs tabs?


What value does feature detection add? We don't really expect a version of Python 3 where there is a basestring, do we? I guess it eliminates a line of code, but I think the result is that this code becomes a little bit harder to read, as it's not clear from context whether Python 2 or Python 3 features the basestring.


One of these blanks could be removed


Could you post a screenshot of your config. And of the shadows issue.


One of these blanks could be removed. Assume this applies to all forward occurrences.


Thanks, fixed in ab6d796e08f2c0255f53aaeb49b4b7155c9b8e0f


I believe there's a dependency missing in the Makefile. creating lsp/git_describe.lsp by running `make lsp/git_describe.lsp` in src/ works. But that target isn't pulled in automatically when I run plain `make`, leading to an error.


komnentar na drugo update so (1 parent 4e0a0be ) bez komit


Only a suggestion, but what if there was a single "insert row" option under the right click menu.  You select that and a window would open up allowing you enter the number of rows to insert and two options to select if it's above or below.  This would help limit the size of the right click menu size, but not sure how complicated it is to implement pop up windows?


There's a typo here.


added comments


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/461/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=XW-2018).


ok


solved by cleani

From: Bruno de Lacheisserie
Sent: Thursday, September 15, 2016 9:58 AM
To: LK8000/LK8000
Cc: Paolo Ventafridda ; Author
Subject: Re: [LK8000/LK8000] Fix minor unused variables out of testbench mode (96dd04a)

bug : no more notification about SerialPort Error with this change.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


xsltApplyStylesheet happen only with parallel build, it's because 2 or more xsltproc process try to create same directory at same time

run "make" again without -j option


It's in the header file...


when I enter "A" in the prompt the program doesn't respond appropriately (not at all)...


The fact that you know depend on repositories in `ceylon.config` seems incompatible with compiling dependent projects. Let me explain: a project with a required project explicitly adds in its repositories thet output repository of the required project. Look at the end of `CeylonProject.createRepositoryManager()`:
```
        .systemRepo(systemRepository)
        .extraUserRepos(Arrays.asList(
            for (p in referencedCeylonProjects)
            javaString(p.ceylonModulesOutputDirectory.absolutePath)))
```

These additional required project output repos should not be added in the ceylon config file since it is a IDE-only thing. So when compiling in out-process the archives of the required projects will not be seen.


That's a bug.


In this commit, also changed header files to initialize members of several data structures


`FindUnique`? Name sounds weird.


Yes.
This is more intuitive and it doesn't require indenting the elses below with an inversed if statement


rather than keeping number in a global variable, you should return it from `askForNumber` and use its return value in `translateNumberToGerman`


??? Could you post me a gs dump ?


fixed


vector from a to b
vector from a to p


WIth the same CRC and and same GSdx settings I don't get the same issue. Do you have widescreen patch/cheats enabled?


inline function requires that


I believe `git rev-parse --abbrev-ref HEAD` will do exactly the same as this if/then/else.


Would make `Object.assign(this, data)` more sense?


there is a less clunky way of addressing the item in the array that matches the (number-1). no need for a loop.


TeamCity Branches Tests :: Git yaegor sandbox [Build 32](http://munit-050:8112/tc/viewLog.html?buildId=99047&buildTypeId=BranchesTests_GitYaegorSandbox) outcome was **SUCCESS**
Summary: Running Build time: 0:10:1


HAAAAAAAAAAAAANDS


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.168](http://build.clear-measure.com/viewLog.html?buildId=10453&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


And if you don't like the style, we can review the style settings in selxUncrustify.cfg


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/462/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=XW-2018).


sss


Wait a sec, the shadow is fine in the screenshot you posted...


插入bootstrap/base.html 中的head代码，首先先super（）保留基模板中定义的块的原始内容。


you are throwing a string here but should throw an error object with a message


I was already using repositories from the config, so nothing has really changed.


Hah, indeed it would! Didn't even think about that one.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.168](http://build.clear-measure.com/viewLog.html?buildId=10453&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 72, ignored: 2 Build time: 00:02:52


А разница только в потребляемой памяти?


Thanks for answering. I was just wondering the need for it in this specific _FileUtil.java_


These bl flags need naming too


interesting. why are you doing this instead of decreasing the number by one?


I think @taschini is mainly about removing the `#noqa` here.
Not sure if it is worth.


okay... then is all working okay no?


I'm not sure, but the comparison `>= -0.0` is IMHO not reliable across different compilers. I would suggest using C99 built-in function `signbit()`.


This could also be achieved without depending on `sed` like this: `version_release=${version_tag#v}; version_release=${version_release//_/.}` (none of the current tags contain underscores, though, so that the second half is superfluous, no?)


probably, but you should still test the use-case I explain here, since it seems there might be a bug preventing the compilation of dependent projects, which would be weird IMO.
This change will have made it harder to fix this dependent-projects bug.


it doesn't looks like this argument gets used.


<!DOCTYPE HTML>
<html>
<head>


And the `# pragma: no cover`, of course.


Yes it was. I was playing around with automatically building the documentation on readthedocs.io. The existence of the file should not affect anything else.


looks good to me!


In other places, you've used `ab`.


I left my comment in the wrong place: This could also be done via `git rev-parse --abbrev-ref HEAD`.


Does not play well with OSMC. I built this on Raspbian wheezy and the jessie version currently does NOT support Kodi using apt-get install kodi. So if you load this on OSMC, the graphics are corrupted and displays incorrectly. Might give a poor impression. So, load this on a Raspberry PI 2 or wait for Kodi to be supporte on the Jessie build for the Raspberry Pi 3.


i implicitly declared


I was wrong about the not none comparison, so this was  probably better before


fixed

It's only used to determine the support smoothing...


I don't understand.
The setting has been removed. Does the version upgrade system need a special case for that? Why can't it see for itself that the setting doesn't exist any more?


It would make racket-doc require web-server-doc which would force that package to go to a weird place in the dependency tree


@mullerlele Now fixed in master.


@fh1ch: why have you added this lines? which part of the code does now init the external database?

I am thinking about removing this part and modifying the lines below to work with external databases.

--

This comment is related to discussions on PR #697


> AFAIU inline variables don't really provide new functionality.

IIUC they do provide new functionality with respect to `static_const<T>::value` if the object being constructed doesn't have a `constexpr` constructor or has side-effects, but this is beyond the point since range-v3 doesn't use this.

This commit was one of the first usages of inline variables "in the real world", it discovered an incompatibility with C++<=14 which broke clang's initial implementation, which is now fixed and shipped in clang 3.9, and which will hopefully help in getting GCC and MSVC implementations to be correct as well. It also discovered the need to put the inline variable in an `inline namespace` which is... really unfortunate.

Was this worth it? It was worth it for me, since I learned something I did not know before and I am now able to use them in my own projects (without the macro) because clang supports them properly.

Was this worth it for range-v3? In retrospect, I don't think so. While there are gains on paper (each TU doesn't need to instantiate all those little `static_const<T>` anymore), I doubt they are measurable.


@taylorotwell I am curious what would be the benefit of importing full class instead of the assigned alias?


# first-time

my first repozitory on Git ... bla...bla...bla

I like :hacking: :sniffing: and :money:.


Or you can do: return data == "False"


it might have been easier to make an array here and to adjust the number before getting its value. but this can also be justified since it's more like a dictionary this way.


En quoi consiste cette mise à jour ?
Est-ce que les données des packages n'étaient pas valides/complètes ?


ну я не думаю что из-за увеличения значений дебаффов память будет потребляться больше)


thx u


I noticed this was removed, is it still OK to use this method?


If resource_id is passed as a string, then it will not be cast by rails. ".to_i" convert the string as an number (it will work for long)


return (data == "False")


强迫症犯了，必须整理文件夹！


This does not seem to be working, parameters.visuals are already defined at this point from default values


FYI: This broke serialize a few days ago, because a bool could not be serialized. Is fixed by now.


To illustrate:

    ──────v──────
    ──────v──────
    ────┐ ░ ┌────
    ──┐ │ ░ │ ┌──
    █ │ │ ░ │ │ █
    ──┘ │ ░ │ └──
    ────┘ ░ └────
    ──────^──────
    ──────^──────

░ = Possible room for support.


package.json is not updated on purescript version. It is still 0.8.4!!!
So currently master doesn't build :-(
I have tried the code with boilerplate from the latest pux starter app, https://github.com/alexmingoia/pux-starter-app
And it worked OK after the needed small adjustments.


this works, but doesn't make too much sense because you are throwing an error and then catching it in the same function. you could have just applied your error handling behavior where you threw the error.
if you read the exercise again, you'll see that the error handling should happen in `translateNumeberToGerman`


Yes there are fine. If you use the SW renderer (hit F9), you will see the difference.


yes i have add to new branch nextp3 too its now default in oe-core


@xeolabs Thank you :-)


TeamCity test :: test [Build 14](http://ci.lemmew.com:8111/viewLog.html?buildId=1466&buildTypeId=Test_Test) is now running


TeamCity test :: test [Build 14](http://ci.lemmew.com:8111/viewLog.html?buildId=1466&buildTypeId=Test_Test) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


In sai_thrift_create_virtual_router() fixed issue with passing Ipv4 attribute to Ipv6 function. We should pass vr_attribute2_value instead of vr_attribute1_value to enable V6 routing.


Do I understand correctly that your intention is to preempt any grading that is currently running, forget that gradingTask, create another and start grading a new answer?

1. Is this the desired behavior across all tasks that are graded with callbacks? E.g. python programming, etc.

2. I think the current gradingTask must be properly destroyed first, via destroyTask(), otherwise there may be some undefined behavior (factories must be disposed, and unload must be called). Should I add this?


looks good to me!
compare this one with the other exercise: you are putting the call of the potentially error-throwing function (`JSON.parse` in this case) in a `try` block.


Good job, Anatoliy


zmq_connect with sourceip:port for Ipv6 is not working. I am using the format

**zmq_connect(socket,"tcp://[::ffff:127.0.0.1]:5552;[::ffff:127.0.0.1]:5558");**

But this is not working, can u please tell why,


You don't need `project` as a separate parameter; you can use `originalClass.project`


do camera work now ?


@rkorytkowski I am working on printing logs, and build on my CI shows no errors, that's weird...


Больше, просто за счет хранения таблицы в памяти, и чуть большим временем использования cpu для перебора этой таблицы. Но эти величины будут мизерные. В филгере, факт, что больше строк, да и конфиг автооткрытия итемов поболе.


what is this


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/463/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=XW-2018).


Why is the `isLocal` check neeed? How can we get a local variable in the list of members to move?


Yay, this actually did it! Builds are now passing again :D


Ну, я вижу тут три варианта:
1 - Оставить все так, как есть (я имею ввиду про старые дебаффы).
2 - Удалить старые дебаффы и запостить их на форуме.
3 - Вернуть все старые дебаффы в конфиг сборки.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1009](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=13323&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Added, thanks!

https://github.com/ryanbr/fanboy-adblock/commit/cab06cc133e55659de4a6e9883024fa4ead355ef


Do we need also change
PREFERRED_VERSION_python-native ?= "2.7.11"
to
PREFERRED_VERSION_python-native ?= "2.7.12"
?


Does he have to pay the fine directly to you? :p


Nope, you even can't (No recipe for python-native 2.7.12).
Just python ?= "2.7.12


This is not a cloud-init template. It's a LXD template. The wording doesn't quite flow in the description.


What do you mean by your CI?


Typo on template.


Please add it to master comit. This isn't included at the 7.7.4


This includes the file in pretty much every page load in LOVD. Shall we check which functions are really used that often and put them in a more general place, like in `inc-lib-init.php` or perhaps as a method in `LOVD_Object` itself?


Maybe CannotLoadDefaultPolicyException is enough for class name and message can give more exact description of the problem we have


We should rename these functions, based on what they will now do:

- GenerateNetworkConfigV1
- GenerateEtcNetworkInterfaces

as that makes it more obvious what they will do.


Does ```generatedTemplate``` always have a trailing new-line?


Ok thanks.


I don't believe this is necessary anymore. Though we should check to see what KVM does.


@mcostalba But Stephane claimed this version to be faster, see comments on the first [commit](https://github.com/official-stockfish/Stockfish/commit/402bb47126391571944bcf3209365feac99d54a6)


Можно добавить различные бонусы для змейки и еды. Например, ускорение или замедление.


not rely ON DHCP.


Fiddle: https://jsfiddle.net/1ct0xb4r/15/


Can we explain the problem a little better?

Not all network segments will  have DHCP running on what is the eth0 subnet so we cannot use DHCP as a default configuration. (Or something better. :))


Capitalise ENI.


public static double distance(Point p2) {
    return Math.sqrt(Math.pow(p2.x - this.x, 2) + Math.pow(p2.y - this.y, 2));
}


Not sure I understand this comment.


Create the "network configuration", not cloud-int cfg.


I think that for Kotlin this needs to be tracked explicitly, not calculated from the directory. We don't always have Java's matching package/directory structure.


As "refactor" it won't show up in the changelog, so we have to manually add it when generating


Agreed.... oversaw the name to osname change!


Your site is not blank. Your apache dont display errors.

Have first line maincore.php in your site.
````
ini_set ('display_errors', 1);
````


Why has it tried to retrieve /etc/network/interfaces? I don't understand "get" here.


Taille des paquets ajoutée


"Willst du dich sich..." !?


+  public static double distance(Point p2) {
 +    return Math.sqrt(Math.pow(this.a - p2.a, 2) + Math.pow(this.b - p2.b, 2));
 +  }


Again, not sure I understand the reference to "get" and the path. Why is it dealing with "/var/lib/cloud/"?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1009](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=13323&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 237, ignored: 3 Build time: 00:12:31


Readability: New line after 'type Devices'.


@wsalesky I would like to push this typo correction to the master branch, but I will leave that to you if that's OK since I don't want to mess up the correct procedure.
cc: @davidamichelson


Not sure the "/" is harmless. We should use:

    https://golang.org/pkg/path/filepath/#Join


Use %q for paths.


These are not cloud-init templates. These are LXD templates.


You also need to change:

```
"pl.project13.scala" %% "sbt-jmh-extras"           % extrasV, // Apache v2
```

to:

```
"pl.project13.scala" %% "sbt-jmh-extras"           % extrasV, // Apache v2
```

Before you cut another release:

```diff
extras/src/main/java/pl/project13/scala/jmh/extras/profiler/FlightRecordingProfiler.java
+++ extras/src/main/java/pl/project13/scala/jmh/extras/profiler/FlightRecordingProfiler.java	(revision )
@@ -70,10 +70,6 @@

   @Override
   public Collection<String> addJVMOptions(BenchmarkParams params) {
-
-    startFlightRecordingOptions += "filename=" + jfrData;
-    flightRecorderOptions += "settings=" + params.getJvm().replace("bin/java", "lib/jfr/profile.jfc");
-
     return Arrays.asList(
       "-XX:+UnlockCommercialFeatures",
       "-XX:+FlightRecorder",
```


Question is answered by https://github.com/fossology/fossology/pull/697#issuecomment-247305754


Можно добавить генератор надежных (>= 20 символов) паролей.


Merged.


Maybe you just need to use: exec _ZN11HWCMediator7prepareEjPP22hwc_display_contents_1+620


Good catch, though I had wrapped it in enableDebug already but I must have missed it


Nice


Each html tags add prop like aria-label='some-recognized-aria'

Example:

<div aria-label='bookmark'>link </div>


The Init method is not virtual anymore therefore cannot be overriden and that broke the GettingStarted solution that relies on this.


I mean Travis CI on my fork: https://saucelabs.com/beta/builds/a6e22beb060c442081df07b3cd666738


Good :D With user builds, even with the adb fingerprint popup fix, i was noticing another little issue: it doesn't let you do "adb remount" even if you run "adb root", but root works if you go into adb shell.
With userdebug, this is fixed too.


Question: I'm not thrilled by this change at all, because it breaks the layering (it makes a model-level object aware of a controller-level object). I can see the reason of this change though, but I'm wondering if there would be a better way ?

Also, I'm doing some GitX work atm because I think I upgraded to git 2.10 recently, and the commit view was doing weird things (like staging a bunch of random files just after a commit is made). Does that ring a bell to you ?


We will in the future.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Cw9m-6i3R9SQvwhv_4jRQQ


There won't be support for the same reason as there won't be brim: it doesn't fit.

Note totally sure though. Will put it in the JIRA issue.


I also notice this, why the fix is still not deployed?


HW does not allow to rewrite other than LSB of MAC address, but test case tries to rewrite second byte.


TeamCity Todo :: Build [Build 100](http://109.103.226.38:8512/viewLog.html?buildId=38495&buildTypeId=Todo_Build) is now running


@Sami32 IDE tools like Eclipse check for those things and will issue a warning for unused imports. If they're both there they are probably both used.u


In fact it is not truly necessary to reimplement it because when calling the functions of the player it will then listen to the socket and wait to recv/send message. Maybe we should work as adapt the main_loop to have a more suitable handling.


TeamCity test :: test [Build 15](http://ci.lemmew.com:8111/viewLog.html?buildId=1467&buildTypeId=Test_Test) is now running


TeamCity Todo :: Build [Build 100](http://109.103.226.38:8512/viewLog.html?buildId=38495&buildTypeId=Todo_Build) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:00:08


TeamCity test :: test [Build 15](http://ci.lemmew.com:8111/viewLog.html?buildId=1467&buildTypeId=Test_Test) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


It started to work on openmrs as well: https://saucelabs.com/tests/c87203db207d4ed9873cc0f344f85ca5


Старые муторно ворочать, потому что http://shestak.org/forum/showpost.php?p=13035


TeamCity test :: test [Build 16](http://ci.lemmew.com:8111/viewLog.html?buildId=1468&buildTypeId=Test_Test) is now running


TeamCity test :: test [Build 16](http://ci.lemmew.com:8111/viewLog.html?buildId=1468&buildTypeId=Test_Test) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


For reference: https://github.com/Ultimaker/uranium/commit/a7b270c9


It probably can't hurt too much though..


nice, thanks bigpresh


Do we actually need a `runReadAction` here, given that this method runs in the EDT?


I just ran into an error compiling Clementine on Arch Linux, caused by `PROTOBUF_STATIC_LIBRARY` not being set. Simply reverting this commit fixed it, and gave me a fully working executable (as far as I can tell). What is the downside of not linking statically to libprotobuf?


not sure to understand why you need this, can't you get a cell by its pos (which could be consider as its id btw)


@mcostalba Running a normal bench I measure that there are snipers in 20% of cases.
In case of snipers !=0 the mean-value is 1.03 so I suggest to move the _defensive_  assignment into the loop of snipers.


Would be nice to undo the incomplete changes if we run into this situation.


@valegagge uhm one of these files is for iCubGenova05?


LoL. Yes :+1:


meow


BTW: its more like:
```
─────v─────────v─────
─────v─────────v─────
────┐ ┌───────┐ ┌────
──┐ │ │ ┌───┐ │ │ ┌──
█ │ │ │ │ ░ │ │ │ │ █
──┘ │ │ └───┘ │ │ └──
────┘ └───────┘ └────
─────^─────────^─────
─────^─────────^─────
```


Not related to this PR but: Why `div` is in `kotlinx.html.js` package and not in `kotlinx.html.div` ?


first: no need to break after a return
second: even if your inside a static method of a factory do the RoomFactory::getXxxRoom(), it makes the read easier.


TeamCity test :: test [Build 17](http://ci.lemmew.com:8111/viewLog.html?buildId=1469&buildTypeId=Test_Test) is now running