[R/get_fire_season.R:2:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/R/get_fire_season.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/get_percentile_raster.R:47:32:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/R/get_percentile_raster.R#L47) *style:* **Put spaces around all infix operators.**
```r
      quantile(x, probs = probs/100, na.rm = TRUE)
                              ~^~
```
[R/import_geff_data_from_tar.R:10:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/R/import_geff_data_from_tar.R#L10) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/plot_percentile_raster.R:10:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/R/plot_percentile_raster.R#L10) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/validate_fire_danger_levels.R:28:57:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/R/validate_fire_danger_levels.R#L28) *style:* **Trailing whitespace is superfluous.**
```r
#'   ct <- validate_fire_danger_levels(fire_index = FWI,
                                                        ^
```
[tests/testthat/test-decompress_gz.R:16:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-decompress_gz.R#L16) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:4:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L4) *style:* **Variable or function name should be snake_case.**
```r
dataDates <- seq.Date(from = as.Date('1980-01-01'),
^~~~~~~~~
```
[tests/testthat/test-get_fire_season.R:4:38:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L4) *style:* **Only use double-quotes.**
```r
dataDates <- seq.Date(from = as.Date('1980-01-01'),
                                     ^~~~~~~~~~~~
```
[tests/testthat/test-get_fire_season.R:5:36:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L5) *style:* **Only use double-quotes.**
```r
                      to = as.Date('2016-12-31'),
                                   ^~~~~~~~~~~~
```
[tests/testthat/test-get_fire_season.R:9:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L9) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:11:56:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L11) *style:* **Only use double-quotes.**
```r
  seasons <- get_fire_season(dates = dataDates, zone = 'north')
                                                       ^~~~~~~
```
[tests/testthat/test-get_fire_season.R:12:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L12) *style:* **Commented code should be removed.**
```r
  # View(data.frame("dates" = dataDates, "season" = seasons))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_fire_season.R:13:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:19:57:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L19) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
test_that("get_fire_season works for south hemisphere", {
                                                        ^
```
[tests/testthat/test-get_fire_season.R:19:58:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L19) *style:* **Trailing whitespace is superfluous.**
```r
test_that("get_fire_season works for south hemisphere", {
                                                         ^~
```
[tests/testthat/test-get_fire_season.R:20:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L20) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:22:56:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L22) *style:* **Only use double-quotes.**
```r
  seasons <- get_fire_season(dates = dataDates, zone = 'south')
                                                       ^~~~~~~
```
[tests/testthat/test-get_fire_season.R:23:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L23) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:27:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L27) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:30:48:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L30) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
test_that("get_fire_season works for tropics", {
                                               ^
```
[tests/testthat/test-get_fire_season.R:30:49:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L30) *style:* **Trailing whitespace is superfluous.**
```r
test_that("get_fire_season works for tropics", {
                                                ^~
```
[tests/testthat/test-get_fire_season.R:31:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L31) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:33:56:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L33) *style:* **Only use double-quotes.**
```r
  seasons <- get_fire_season(dates = dataDates, zone = 'tropics')
                                                       ^~~~~~~~~
```
[tests/testthat/test-get_fire_season.R:34:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L34) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_fire_season.R:38:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_fire_season.R#L38) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:4:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L4) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:5:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L5) *style:* **Variable or function name should be snake_case.**
```r
  GFED4BasisRegions <- get_gfed4(varname = "BasisRegions")
  ^~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:6:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:8:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L8) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:9:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L9) *style:* **Variable or function name should be snake_case.**
```r
  EURO <- get_gfed4(varname = "BasisRegions", region = "EURO")
  ^~~~
```
[tests/testthat/test-get_gfed4.R:10:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L10) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:12:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L12) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:16:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L16) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:17:12:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L17) *style:* **Variable or function name should be snake_case.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = NULL, end_date = NULL,
           ^~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:18:72:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L18) *style:* **Trailing whitespace is superfluous.**
```r
                                         temporal_resolution = "daily",
                                                                       ^
```
[tests/testthat/test-get_gfed4.R:20:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L20) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:22:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L22) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:26:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L26) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:27:12:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L27) *style:* **Variable or function name should be snake_case.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
           ^~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:27:68:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L27) *style:* **Trailing whitespace is superfluous.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
                                                                   ^
```
[tests/testthat/test-get_gfed4.R:29:68:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L29) *style:* **Trailing whitespace is superfluous.**
```r
                                        temporal_resolution = NULL,
                                                                   ^
```
[tests/testthat/test-get_gfed4.R:31:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L31) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:33:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L33) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:37:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L37) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:38:12:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L38) *style:* **Variable or function name should be snake_case.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "1900-01-01",
           ^~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:38:68:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L38) *style:* **Trailing whitespace is superfluous.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "1900-01-01",
                                                                   ^
```
[tests/testthat/test-get_gfed4.R:40:71:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
                                        temporal_resolution = "daily",
                                                                      ^
```
[tests/testthat/test-get_gfed4.R:42:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L42) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:44:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:48:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L48) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:49:12:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L49) *style:* **Variable or function name should be snake_case.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
           ^~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:49:68:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L49) *style:* **Trailing whitespace is superfluous.**
```r
  x <- try(DailyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
                                                                   ^
```
[tests/testthat/test-get_gfed4.R:51:71:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L51) *style:* **Trailing whitespace is superfluous.**
```r
                                        temporal_resolution = "daily",
                                                                      ^
```
[tests/testthat/test-get_gfed4.R:53:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L53) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:55:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L55) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:59:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L59) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:60:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L60) *style:* **Variable or function name should be snake_case.**
```r
  MonthlyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
  ^~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:60:61:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L60) *style:* **Trailing whitespace is superfluous.**
```r
  MonthlyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
                                                            ^
```
[tests/testthat/test-get_gfed4.R:62:66:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L62) *style:* **Trailing whitespace is superfluous.**
```r
                                 temporal_resolution = "monthly",
                                                                 ^
```
[tests/testthat/test-get_gfed4.R:64:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L64) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:67:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L67) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:71:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L71) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:72:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L72) *style:* **Variable or function name should be snake_case.**
```r
  DailyBurnedAreas <- get_gfed4(start_date = "2003-01-01",
  ^~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_gfed4.R:76:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L76) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_gfed4.R:79:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_gfed4.R#L79) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-get_perc_risk_index.R:6:40:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_perc_risk_index.R#L6) *style:* **Put spaces around all infix operators.**
```r
                                package="caliver"))
                                      ~^~
```
[tests/testthat/test-get_percentile_raster.R:1:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L1) *style:* **Commented code should be removed.**
```r
# context("get_percentile_raster")
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:2:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:4:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L4) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:7:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#                                          input_file_path = NULL), silent = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:8:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L8) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:10:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L10) *style:* **Commented code should be removed.**
```r
#   expect_equal(class(probsMaps), "try-error")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:11:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:13:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L13) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:15:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L15) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:20:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L20) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:22:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L22) *style:* **Commented code should be removed.**
```r
#   expect_equal(class(probsMaps), "try-error")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:23:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L23) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:25:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:27:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L27) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:28:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L28) *style:* **Commented code should be removed.**
```r
#   probsMaps1B <- get_percentile_raster(probs = 50, input_raster = s)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:29:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L29) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:31:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L31) *style:* **Commented code should be removed.**
```r
#   expect_equal("RasterLayer" %in% class(probsMaps1B), TRUE)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:32:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L32) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:34:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L34) *style:* **Commented code should be removed.**
```r
#   expect_equal(names(probsMaps1B), "FWI50")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:35:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L35) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:36:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L36) *style:* **Commented code should be removed.**
```r
#   mean50 <- round(raster::cellStats(probsMaps1B, stat = 'mean'), 0)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:37:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L37) *style:* **Commented code should be removed.**
```r
#   expect_equal(mean50, 7)
    ^~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:38:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L38) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:40:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:41:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L41) *style:* **Lines should not be more than 80 characters.**
```r
# test_that("get_percentile_raster works with multiple probs from raster file", {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:42:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L42) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:43:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L43) *style:* **Commented code should be removed.**
```r
#   probsMaps2B <- get_percentile_raster(probs = c(50, 75), input_raster = s)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:44:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L44) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:46:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L46) *style:* **Commented code should be removed.**
```r
#   expect_equal("RasterBrick" %in% class(probsMaps2B), TRUE)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:47:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L47) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:49:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L49) *style:* **Commented code should be removed.**
```r
#   expect_equal(names(probsMaps2B), c("FWI50", "FWI75"))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:50:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L50) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:51:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L51) *style:* **Commented code should be removed.**
```r
#   mean50 <- round(raster::cellStats(probsMaps2B$FWI50, stat = 'mean'), 0)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:52:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L52) *style:* **Commented code should be removed.**
```r
#   mean75 <- round(raster::cellStats(probsMaps2B$FWI75, stat = 'mean'), 0)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:53:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L53) *style:* **Commented code should be removed.**
```r
#   expect_equal(mean50, 7)
    ^~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:54:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L54) *style:* **Commented code should be removed.**
```r
#   expect_equal(mean75, 9)
    ^~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:55:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L55) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-get_percentile_raster.R:57:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L57) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-get_percentile_raster.R:58:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L58) *style:* **Commented code should be removed.**
```r
# # test_that("get_percentile_raster works with single prob from nc file", {
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:59:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L59) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:60:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L60) *style:* **Commented code should be removed.**
```r
# #   skip("skip temporarily for testing")
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:61:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L61) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^~~
```
[tests/testthat/test-get_percentile_raster.R:62:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L62) *style:* **Commented code should be removed.**
```r
# #   probsMaps1A <- get_percentile_raster(probs = 50,
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:63:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L63) *style:* **Commented code should be removed.**
```r
# #                                        input_file_path = geff5nc)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:64:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L64) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:66:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L66) *style:* **Commented code should be removed.**
```r
# #   expect_equal("RasterLayer" %in% class(probsMaps1A), TRUE)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:67:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L67) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:69:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L69) *style:* **Commented code should be removed.**
```r
# #   expect_equal(names(probsMaps1A), "FWI50")
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:70:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L70) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:71:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L71) *style:* **Commented code should be removed.**
```r
# #   mean50 <- round(raster::cellStats(probsMaps1A, stat = 'mean'), 0)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:72:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L72) *style:* **Commented code should be removed.**
```r
# #   expect_equal(mean50, 26)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:73:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L73) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:74:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L74) *style:* **Commented code should be removed.**
```r
# # })
  ^~~~
```
[tests/testthat/test-get_percentile_raster.R:75:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L75) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:76:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L76) *style:* **Commented code should be removed.**
```r
# # test_that("get_percentile_raster works with multiple probs from nc file", {
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:77:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L77) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:78:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L78) *style:* **Commented code should be removed.**
```r
# #   skip("skip temporarily for testing")
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:79:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L79) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^~~
```
[tests/testthat/test-get_percentile_raster.R:80:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L80) *style:* **Commented code should be removed.**
```r
# #   probsMaps2A <- get_percentile_raster(probs = c(50, 75),
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:81:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L81) *style:* **Commented code should be removed.**
```r
# #                                        input_file_path = geff5nc)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:82:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L82) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:84:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L84) *style:* **Commented code should be removed.**
```r
# #   expect_equal("RasterBrick" %in% class(probsMaps2A), TRUE)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:85:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L85) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:87:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L87) *style:* **Commented code should be removed.**
```r
# #   expect_equal(names(probsMaps2A), c("FWI50", "FWI75"))
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:88:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L88) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:89:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L89) *style:* **Commented code should be removed.**
```r
# #   mean50 <- round(raster::cellStats(probsMaps2A[[1]], stat = 'mean'), 0)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:90:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L90) *style:* **Commented code should be removed.**
```r
# #   mean75 <- round(raster::cellStats(probsMaps2A[[2]], stat = 'mean'), 0)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:91:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L91) *style:* **Commented code should be removed.**
```r
# #   expect_equal(mean50, 26)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:92:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L92) *style:* **Commented code should be removed.**
```r
# #   expect_equal(mean75, 30)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:93:4:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L93) *style:* **Trailing whitespace is superfluous.**
```r
# #
   ^
```
[tests/testthat/test-get_percentile_raster.R:94:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-get_percentile_raster.R#L94) *style:* **Commented code should be removed.**
```r
# # })
  ^~~~
```
[tests/testthat/test-import_geff_data_from_tar.R:5:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-import_geff_data_from_tar.R#L5) *style:* **Commented code should be removed.**
```r
  # s <- import_geff_data_from_tar(geff5tar)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-mask_crop_subset.R:4:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L4) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:6:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:12:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L12) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:18:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L18) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:24:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L24) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:30:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L30) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_crop_subset.R:36:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_crop_subset.R#L36) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:4:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L4) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:6:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:8:55:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L8) *style:* **Commas should always have a space after.**
```r
  expect_equal(round(raster::cellStats(xmasked, mean),3), 7.582)
                                                      ^
```
[tests/testthat/test-mask_with_fuelmodel.R:9:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L9) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:13:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:15:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L15) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:17:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L17) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-mask_with_fuelmodel.R:19:46:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L19) *style:* **Commas should always have a space after.**
```r
  expect_equal(round(raster::extent(ymasked),0)[1], -180)
                                             ^
```
[tests/testthat/test-mask_with_fuelmodel.R:20:46:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L20) *style:* **Commas should always have a space after.**
```r
  expect_equal(round(raster::extent(ymasked),0)[2], 180)
                                             ^
```
[tests/testthat/test-mask_with_fuelmodel.R:21:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-mask_with_fuelmodel.R#L21) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:5:61:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L5) *style:* **Commas should always have a space after.**
```r
test_that("plot_fire_pdf match expectations for no y-limit",{
                                                            ^
```
[tests/testthat/test-plot_fire_pdf.R:6:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L6) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:8:39:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L8) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                      ^
```
[tests/testthat/test-plot_fire_pdf.R:8:41:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L8) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                        ^
```
[tests/testthat/test-plot_fire_pdf.R:8:43:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L8) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                          ^
```
[tests/testthat/test-plot_fire_pdf.R:8:45:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L8) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                            ^
```
[tests/testthat/test-plot_fire_pdf.R:13:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L13) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:16:59:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L16) *style:* **Commas should always have a space after.**
```r
test_that("Plot layers match expectations without vLines",{
                                                          ^
```
[tests/testthat/test-plot_fire_pdf.R:17:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L17) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:19:39:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L19) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                      ^
```
[tests/testthat/test-plot_fire_pdf.R:19:41:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L19) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                        ^
```
[tests/testthat/test-plot_fire_pdf.R:19:43:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L19) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                          ^
```
[tests/testthat/test-plot_fire_pdf.R:19:45:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L19) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                            ^
```
[tests/testthat/test-plot_fire_pdf.R:21:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L21) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:26:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L26) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:29:56:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L29) *style:* **Commas should always have a space after.**
```r
test_that("Plot layers match expectations with vLines",{
                                                       ^
```
[tests/testthat/test-plot_fire_pdf.R:30:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L30) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:32:39:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L32) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                      ^
```
[tests/testthat/test-plot_fire_pdf.R:32:41:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L32) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                        ^
```
[tests/testthat/test-plot_fire_pdf.R:32:43:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L32) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                          ^
```
[tests/testthat/test-plot_fire_pdf.R:32:45:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L32) *style:* **Commas should always have a space after.**
```r
                     thresholds = c(1,2,3,4,5),
                                            ^
```
[tests/testthat/test-plot_fire_pdf.R:35:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L35) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_fire_pdf.R:40:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_fire_pdf.R#L40) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test-plot_obs_vs_forecast.R:1:3:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L1) *style:* **Commented code should be removed.**
```r
# context("plot_obs_vs_forecast")
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:2:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:4:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L4) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:5:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L5) *style:* **Commented code should be removed.**
```r
#   skip("skip temporarily for testing")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:6:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L6) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:7:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#   url <- "https://www.dropbox.com/s/939t7tipitme17o/CAMS_2015-01-01_2015-01-02_frpfire_rotated.nc?dl=0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:7:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L7) *style:* **Commented code should be removed.**
```r
#   url <- "https://www.dropbox.com/s/939t7tipitme17o/CAMS_2015-01-01_2015-01-02_frpfire_rotated.nc?dl=0"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:8:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L8) *style:* **Commented code should be removed.**
```r
#   input_file <- file.path(tempdir(), "cams.nc")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:9:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L9) *style:* **Commented code should be removed.**
```r
#   download.file(url = url, destfile = input_file)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:10:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L10) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:11:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L11) *style:* **Commented code should be removed.**
```r
#   geffDir <- file.path(tempdir(), "geff")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:12:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L12) *style:* **Commented code should be removed.**
```r
#   dir.create(geffDir)
    ^~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:13:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L13) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:14:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L14) *style:* **Lines should not be more than 80 characters.**
```r
#   testfileURL <- "https://www.dropbox.com/s/5isrbxg6qvrnxns/20150101_20150101_ecfire_fc_fwi_fwi.nc?dl=0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:14:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L14) *style:* **Commented code should be removed.**
```r
#   testfileURL <- "https://www.dropbox.com/s/5isrbxg6qvrnxns/20150101_20150101_ecfire_fc_fwi_fwi.nc?dl=0"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:15:37:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L15) *style:* **Trailing whitespace is superfluous.**
```r
#   download.file(url = testfileURL,
                                    ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:16:48:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L16) *style:* **Trailing whitespace is superfluous.**
```r
#                 destfile = file.path(geffDir,
                                               ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:17:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L17) *style:* **Lines should not be more than 80 characters.**
```r
#                                      "20150101_20150101_ecfire_fc_fwi_fwi.nc"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:18:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L18) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:19:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L19) *style:* **Lines should not be more than 80 characters.**
```r
#   testfileURL <- "https://www.dropbox.com/s/796cq51n2scddno/20150101_20150102_ecfire_fc_fwi_fwi.nc?dl=0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:19:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L19) *style:* **Commented code should be removed.**
```r
#   testfileURL <- "https://www.dropbox.com/s/796cq51n2scddno/20150101_20150102_ecfire_fc_fwi_fwi.nc?dl=0"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:20:37:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L20) *style:* **Trailing whitespace is superfluous.**
```r
#   download.file(url = testfileURL,
                                    ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:21:48:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L21) *style:* **Trailing whitespace is superfluous.**
```r
#                 destfile = file.path(geffDir,
                                               ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:22:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L22) *style:* **Lines should not be more than 80 characters.**
```r
#                                      "20150101_20150102_ecfire_fc_fwi_fwi.nc"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:23:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L23) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:24:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L24) *style:* **Lines should not be more than 80 characters.**
```r
#   testfileURL <- "https://www.dropbox.com/s/yr135dyqm8sv2ax/20150102_20150102_ecfire_fc_fwi_fwi.nc?dl=0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:24:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L24) *style:* **Commented code should be removed.**
```r
#   testfileURL <- "https://www.dropbox.com/s/yr135dyqm8sv2ax/20150102_20150102_ecfire_fc_fwi_fwi.nc?dl=0"
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:25:37:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
#   download.file(url = testfileURL,
                                    ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:26:48:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L26) *style:* **Trailing whitespace is superfluous.**
```r
#                 destfile = file.path(geffDir,
                                               ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:27:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L27) *style:* **Lines should not be more than 80 characters.**
```r
#                                      "20150102_20150102_ecfire_fc_fwi_fwi.nc"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:28:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L28) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:30:1:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L30) *style:* **Lines should not be more than 80 characters.**
```r
#                                   p = as(raster::extent(-9.2, -7.5, 38.8, 41.4),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:31:63:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
#                                          "SpatialPolygons"),
                                                              ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:32:43:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L32) *style:* **Trailing whitespace is superfluous.**
```r
#                          threshold = 14,
                                          ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:36:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L36) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^
```
[tests/testthat/test-plot_obs_vs_forecast.R:37:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L37) *style:* **Commented code should be removed.**
```r
#   expect_equal(length(my_plot$layers) == 2, TRUE)
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:38:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L38) *style:* **Commented code should be removed.**
```r
#   expect_identical(my_plot$labels$y, "Forecast date")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:39:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L39) *style:* **Commented code should be removed.**
```r
#   expect_identical(my_plot$labels$x, "Observation date")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:40:2:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
#
 ^~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:41:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L41) *style:* **Commented code should be removed.**
```r
#   # test <- file.path(tempdir(), "test.png")
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:42:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L42) *style:* **Commented code should be removed.**
```r
#   # png(filename = test)
    ^~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:44:5:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L44) *style:* **Commented code should be removed.**
```r
#   # dev.off()
    ^~~~~~~~~~~
```
[tests/testthat/test-plot_obs_vs_forecast.R:45:6:](https://github.com/ecmwf/caliver/blob/4753a8a8b4a0daf7efa2eca54c0c85572c8a82c9/tests/testthat/test-plot_obs_vs_forecast.R#L45) *style:* **Trailing whitespace is superfluous.**
```r
#   #
     ^
```


y


Stupid bad programs


@NycuRO One day, yes.


@edolstra Can we talk about it this? I have been working on a number of changes for cross, and been coordinating with our release managers @globin and @fpletz on how to get 17.09 *only* when they've been demonstrated *not* to cause regressions.

Also, this but one of many examples of you reverting some change where you missed discussion before merge. https://github.com/NixOS/nixpkgs/pull/28557 was first opened almost 2 weeks ago, and I pinged you for review---no one was trying to do things behind your back. Just saying something like "I think this change is a mistake, please properly motivate it to me within the day or I'll revert" would be a huge improvement. It still allows you quickly undo what you perceive as the damage done, while giving your contributors some opportunity to explain themselves.


@moozzyk I want to use websockets, but if it fails, at my company the proxy forbids websockets to azure.. it should fallback to long-polling. Should I build this myself? Try to connect websockets and on failure start connection using longpolling? Sorry for asking but couldn't find it anywhere.


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/6278fb66-bfc2-4ef8-9db7-03519553f58f "halbwissenbot"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/a5a7ad46-ada2-4b2e-9939-be85bd6d1ff0 "halbwissenbot") deployed [halbwissenbot](https://github.com/bahuma/halbwissenbot/releases/tag/deploy-ibm-yp-us-south-max.bachhuber-bahuma.io-bahuma-20170907-152754 "deploy-ibm-yp-us-south-max.bachhuber-bahuma.io-bahuma-20170907-152754") to [bahuma](https://console.bluemix.net/apps/2e32ffda-ed57-4dfe-8771-266a8514648d?env_id=ibm:yp:us-south "ibm:yp:us-south:max.bachhuber@bahuma.io:bahuma"), including this commit


It looks like maximum speed for the project to be reviewed should be 10MPH, but that might be out of scope of this commit. Can be introduced later.


Is there any reason to move those methods to be internal?
IMHO, those are very useful for server side development, I can easily generate a response from status code.

The other hand, the http_exception doesn't accept status_code as parameter, so my workaround currently is trying to pass status_code as well as corresponding reason phrase to http_exception.


Yes, `npm` will re-change it. This is an auto-generated file that npm will overwrite everytime. It would be good if we could avoid fighting it ☺


A Basic Program to Enter the Elements in a LinkedList


Good spot.... @bennuttall ?


why not just use jasmine spy here?


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-153135 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-153135") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


How do I use this?


Bluemix [toolchain](https://console-regional.ng.bluemix.net/devops/toolchains/b09e3ee0-bc62-4bd7-bcf6-7b033dada6af "TriAzurUE"): [Delivery Pipeline](https://console-regional.ng.bluemix.net/devops/pipelines/57bc70fc-3a85-4eb7-a477-d1a67ac25fb2 "TriAzurUE") deployed [TriAzur_code](https://github.com/Doncassino/TriAzur_code/releases/tag/deploy-ibm-yp-eu-gb-TriAzur-TriAzur-20170907-153203 "deploy-ibm-yp-eu-gb-TriAzur-TriAzur-20170907-153203") to [TriAzur](https://console.bluemix.net/apps/11b7850a-b04f-4546-975c-fb2662c615cd?env_id=ibm:yp:eu-gb "ibm:yp:eu-gb:TriAzur:TriAzur"), including this commit


Hm? What do you mean?


Sounds like a good idea.


@0xdeafcafe
```
(node:11251) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): AssertionError [ERR_ASSERTION]: No INPUT matching 'input[type=email]'
(node:11251) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
```


![](https://github.trello.services/images/mini-trello-icon.png) [block linearization and redundant jump removal](https://trello.com/c/jWzkYlBp/14-block-linearization-and-redundant-jump-removal)


Hi


Nada que ver, te recomiendo ver el $routeProvider.


#3 fixed with this commit.


Ok, I'll revert these commits and that should make Jenkins and npm happy again.

/cc @hhuijser


IDFEditorFixesFor8-8-0 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 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)


IDFEditorFixesFor8-8-0 (JasonGlazer) - 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)


It just blocks sending new remittances.


During backport to `1.7.x` the Lambda must be turned to the inline interface impl.


IDFEditorFixesFor8-8-0 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (0 of 0 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-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


# CONFIG_LEGACY_PTYS=y


IDFEditorFixesFor8-8-0 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (0 of 0 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-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


![](https://github.trello.services/images/mini-trello-icon.png) [Method syntax](https://trello.com/c/sQN7xQ17/9-method-syntax)


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/fatal-js-error-breaking-admin-upgrade-page/69559/4


IDFEditorFixesFor8-8-0 (JasonGlazer) - 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)


otra modificacion


Rename the first `vote` => `voteRecord` so we have `voteRecord.vote` ... it will be clearer, IMO.


Really?


@gpetruc is there a better way of doing this?

let me know what you think, i.e. does it make sense to implement simple operations on collections yielding floats/ints?
we could use it for  HT, counting softact jets etc..
I would even add a StringParser cut but I'm not sure how do do it beside adding an unused var in Identity class


Oops. As an aside since the owner is the only one who can withdraw, can I make the state change after the transfer. Even if transfer does something malicious it's not like the contract can lose any ether. Or is it best practice to make the state changes after in this case too


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154215 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154215") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154224 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154224") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Sorry: https://github.com/plesk/ext-uptime-robot/commit/18102d9da71b1f054661cd4b55f93e6cd2cddc01


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170907-154234 "deployed-2-env-Prod-20170907-154234") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154246 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170907-154246") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Really? Did it came from your own thoughts?


Cool. I was looking for a better way to do it!


Postv8.8IDDChanges (mjwitte) - 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)


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/blank-page-after-upgrade-and-javascript-error/69548/9


Killing those files manually is discouraged, use dispatcher  --osm-base --terminate to stop the dispatcher instead.


Enable_ChillerHeater_and_CentralHeatPump (jmarrec) - x86_64-MacOS-10.10-clang: Build Failed

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


Using -O3 is currently discouraged due to possible crashes (depends on compiler). Use -O2 instead.


I can read this dood


I don't think we need separate well-known URI's for refs and CAS.  You [mention][1] consumers wanting them for different purposes.  But consumers can fetch a single resource containing both and ignore the information they don't need.  Reasons to split into two URIs woulsbe:

* Different authors for the different information, where you could use file permissions on the server to group write access.  But I expect in most cases a single author/group will manage all well-known OCI information on a given server.  And whee you have different author groups, you can havethem write somethere else and have a single, trusted task collect their entries into the combined file.
* One and/or the other becomes so large that consumer's intersted in only one suffer from the wasted bandwidth.  But I expect both sets to be small and expect most consumers will care about both.

Do you see other reasons, or feel I'm guessing wrong when dismissing the ones I've listed?

[1]: https://github.com/xiekeyang/oci-discovery/pull/2#issuecomment-327718907


Prefer `||` to `or`. Yes, `or` reads better but its operator precedence differs and it'll [bite yer legs](http://www.virtuouscode.com/2010/08/02/using-and-and-or-in-ruby/) 🍗


Try as much as possible *not* to hard-code the redirect host destination...

use %{HTTP_HOST} instead


Try as much as possible not to hard-code the redirect host destination...

use %{HTTP_HOST} instead


Some babel JS transpiler hack :)


Unit test case needed.


Задание к уроку Основы Ruby. Часть 3-4/train.rb

Название переменной должно отражать ее суть, а тут сейчас путаница. Разве там хранится число вагонов?


Этот метод теперь нерабочий - поправь


presumably ``local_endpoint`` should be used to pick the listen socket and use its port, or one of its ports if there are multiple


Technologies used : ASP.Net MVC 5, Entity Framework, data store is Animal.mdf


I've been looking at writing an Obsidian Portal API wrapper for client-side JS, and I just wanted to say that "Mage Hand" is the best name for an OP API wrapper... I'd never considered looking through D&D spells for a name.

Great job!


@alanconway Why not build the Proton binding? It seemed to work fine for me, https://issues.apache.org/jira/browse/PROTON-1575. Go has a TSan of its own (go -race), maybe that can be incorporated when testing the Go binding.


ok as is, though i don't know how many  applications it will have in reality since usually HT, NJet, etc are computed after an analysis-dependent cleaning & selection, and except for SA jets we usually save the objects.
No need of string parser for filtering, as you can filter upstream.


Этот и предыдущий методы явно предназначены для внутреннего использования. Подумай какой модификатор доступа им нужен.


so, is this in need of some accommodation?


Задание к уроку Основы Ruby. Часть 3-4/main.rb

Снаружи класса используется только этот метод. Значит остальные надо скрыть - задай им модификатор доступа


Y bro i wanted to try revert :cry: :cry: :smile: :smile:


typo


Некрасиво же выглядит, ну. Либо `train.assign_route(route)`, либо `train.route = route`.


Local windows build is success. but it is failed in appveyor.

E:\github\bacardi>bacardi build
gyp info it worked if it ends with ok
gyp info using node-gyp@3.6.2
gyp info using node@8.4.0 | win32 | x64
gyp info spawn C:\Users\rucif\.windows-build-tools\python27\python.EXE
gyp info spawn args [ 'E:\\github\\bacardi\\node_modules\\node-gyp\\gyp\\gyp_main.py',
gyp info spawn args   'binding.gyp',
gyp info spawn args   '-f',
gyp info spawn args   'msvs',
gyp info spawn args   '-G',
gyp info spawn args   'msvs_version=2015',
gyp info spawn args   '-I',
gyp info spawn args   'E:\\github\\bacardi\\build\\config.gypi',
gyp info spawn args   '-I',
gyp info spawn args   'E:\\github\\bacardi\\node_modules\\node-gyp\\addon.gypi',
gyp info spawn args   '-I',
gyp info spawn args   'C:\\Users\\rucif\\.node-gyp\\8.4.0\\include\\node\\common.gypi',
gyp info spawn args   '-Dlibrary=shared_library',
gyp info spawn args   '-Dvisibility=default',
gyp info spawn args   '-Dnode_root_dir=C:\\Users\\rucif\\.node-gyp\\8.4.0',
gyp info spawn args   '-Dnode_gyp_dir=E:\\github\\bacardi\\node_modules\\node-gyp',
gyp info spawn args   '-Dnode_lib_file=C:\\Users\\rucif\\.node-gyp\\8.4.0\\<(target_arch)\\node.lib',
gyp info spawn args   '-Dmodule_root_dir=E:\\github\\bacardi',
gyp info spawn args   '-Dnode_engine=v8',
gyp info spawn args   '--depth=.',
gyp info spawn args   '--no-parallel',
gyp info spawn args   '--generator-output',
gyp info spawn args   'E:\\github\\bacardi\\build',
gyp info spawn args   '-Goutput_dir=.' ]
gyp info spawn C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\MSBuild.exe
gyp info spawn args [ 'build/binding.sln',
gyp info spawn args   '/clp:Verbosity=minimal',
gyp info spawn args   '/nologo',
gyp info spawn args   '/p:Configuration=Release;Platform=x64' ]
이 솔루션의 프로젝트를 한 번에 하나씩 빌드합니다. 병렬 빌드를 사용하려면 "/m" 스위치를 추가하세요.
  bacardi.vcxproj -> E:\github\bacardi\build\Release\\bacardi.node
  bacardi.vcxproj -> E:\github\bacardi\build\Release\bacardi.pdb (Full PDB)
  tsc
  idl
  Your environment has been set up for using Node.js 8.4.0 (x64) and npm.
gyp info ok

E:\github\bacardi>bacardi test
warning: LF will be replaced by CRLF in package.json.
The file will have its original line endings in your working directory.

  add function test
(node:46976) Warning: N-API is an experimental feature and could change at any time.
    √ should be equal with expected value

  sub function test
    √ should be equal with expected value

  mul function test
    √ should be equal with expected value

  div function test
    √ should be equal with expected value

  4 passing (28ms)

E:\github\bacardi>


Зря пробел убрал


If anyone lands here... :) refs https://github.com/laravel/framework/pull/18566#issuecomment-290482198 and following


IDFEditorFixesFor8-8-0 (JasonGlazer) - Win64-Windows-7-VisualStudio-14: OK (0 of 0 tests passed, 0 test warnings)

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


Je connaissais l'existance de json.stringify mais n'était pas sur de son utilisation dans mon cas précis. Par doute j'ai préféré le faire à la main.


? the JIRA is not flagged for backport.


well if we think is not needed than we could just write a producer for SA without complicating the global vars
we could in principle compute the Tk MET but then we would need to  loop once per variable we store (pt,phi etc..) that makes little sense... perhaps instead a dedicated class that does loops with predef operations on a CandidatePtr makes more sense?