[R/plotting-colours.R:89:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L89) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:89:32:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L89) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:89:35:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L89) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:89:41:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L89) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:106:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L106) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:106:32:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L106) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:106:35:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L106) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:106:41:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L106) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:131:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L131) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:131:32:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L131) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:131:35:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L131) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:131:41:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L131) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:284:75:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L284) *style:* **Put spaces around all infix operators.**
```r
    isFac <- any(raster::is.factor(grobToPlot)) & all(na.omit(grobToPlot[]%%1)==0)
                                                                         ~^~~
```
[R/plotting-colours.R:284:79:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-colours.R#L284) *style:* **Put spaces around all infix operators.**
```r
    isFac <- any(raster::is.factor(grobToPlot)) & all(na.omit(grobToPlot[]%%1)==0)
                                                                             ~^~~
```
[R/plotting-helpers.R:1023:44:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1023) *style:* **Commas should always have a space after.**
```r
          grobToPlot <- grobToPlot[polySeq,]
                                           ^
```
[R/plotting-helpers.R:1028:38:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1028) *style:* **Commas should always have a space after.**
```r
        grobToPlot <- grobToPlot[a[1,],]
                                     ^
```
[R/plotting-helpers.R:1028:40:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1028) *style:* **Commas should always have a space after.**
```r
        grobToPlot <- grobToPlot[a[1,],]
                                       ^
```
[R/plotting-helpers.R:1291:41:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1291) *style:* **Put spaces around all infix operators.**
```r
            if (all(na.omit(grobToPlot[]%%1)==0)) {
                                       ~^~~
```
[R/plotting-helpers.R:1291:45:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1291) *style:* **Put spaces around all infix operators.**
```r
            if (all(na.omit(grobToPlot[]%%1)==0)) {
                                           ~^~~
```
[R/plotting-helpers.R:1380:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L1380) *style:* **Lines should not be more than 100 characters.**
```r
    grid.rect(x = 0, height = unit(1 + needsNewTitle * inherits(grobToPlot, "Raster") * 0.20 / (nRows / 2), "npc"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2511:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2511) *style:* **Lines should not be more than 100 characters.**
```r
#'          jitter(sort(rbeta(nPoints, betaPar, betaPar) * (maxY - minY) + minY, decreasing = TRUE)),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2589:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2589) *style:* **Variable or function name should be lowerCamelCase.**
```r
thin.SpatialPolygons <- function(x, tolerance = NULL, returnDataFrame = FALSE, minCoordsToThin = 0,
^~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2610:8:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2610) *style:* **Commented code should be removed.**
```r
      #xyOrd[["out"]] <- xyOrd[["out"]][thinRes, ]# thin line
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2633:50:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2633) *style:* **Commas should always have a space after.**
```r
            dat <- x@data[as.numeric(names1) + 1,]
                                                 ^
```
[R/plotting-helpers.R:2634:14:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2634) *style:* **Commented code should be removed.**
```r
            #row.names(dat) <- as.character(seq_len(length(xyOrd)))
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2666:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2666) *style:* **Variable or function name should be lowerCamelCase.**
```r
thin.default <- function(x, tolerance, returnDataFrame, minCoordsToThin, ...) {
^~~~~~~~~~~~
```
[R/plotting-helpers.R:2701:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2701) *style:* **Variable or function name should be lowerCamelCase.**
```r
  IDs <- tryCatch(unlist(lapply(ordSeq, function(xx) {
  ^~~
```
[R/plotting-helpers.R:2726:7:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2726) *style:* **Commented code should be removed.**
```r
    # Polygons <- rep(rep(seq(numPolygons), numPolygon), idLength$V1) # sequential numbering
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2727:5:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2727) *style:* **Variable or function name should be lowerCamelCase.**
```r
    Polygons <- rep(rep(IDs, numPolygon), idLength$V1) # actual ID labelling
    ^~~~~~~~
```
[R/plotting-helpers.R:2728:5:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2728) *style:* **Variable or function name should be lowerCamelCase.**
```r
    Polygon <- rep(unlist(lapply(numPolygon, seq)), idLength$V1)
    ^~~~~~~
```
[R/plotting-helpers.R:2735:36:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2735) *style:* **Commas should always have a space after.**
```r
    return(data.frame(lat = xyOrd[,1], long = xyOrd[,2], order = orders,
                                   ^
```
[R/plotting-helpers.R:2735:54:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2735) *style:* **Commas should always have a space after.**
```r
    return(data.frame(lat = xyOrd[,1], long = xyOrd[,2], order = orders,
                                                     ^
```
[R/plotting-helpers.R:2737:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2737) *style:* **Lines should not be more than 100 characters.**
```r
                      #group = paste0(as.character(Polygons), ".", as.character(Polygon)))) # the actual fortify
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2740:40:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2740) *style:* **Commas should always have a space after.**
```r
    out <- setDT(data.frame(x = xyOrd[,1], y = xyOrd[,2], groups = groups))
                                       ^
```
[R/plotting-helpers.R:2740:55:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2740) *style:* **Commas should always have a space after.**
```r
    out <- setDT(data.frame(x = xyOrd[,1], y = xyOrd[,2], groups = groups))
                                                      ^
```
[R/plotting-helpers.R:2755:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2755) *style:* **Lines should not be more than 100 characters.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2755:34:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2755) *style:* **Put spaces around all infix operators.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
                                ~^~
```
[R/plotting-helpers.R:2755:35:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2755) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
                                  ^
```
[R/plotting-helpers.R:2778:26:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-helpers.R#L2778) *style:* **Commas should always have a space after.**
```r
          totalAvailable," polygons in this view. See options('quickPlot.maxNumPolygons')")
                         ^
```
[R/plotting-other.R:202:48:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L202) *style:* **Put spaces around all infix operators.**
```r
  if (any(raster::is.factor(ras1)) & all(ras1[]%%1==0)) {
                                              ~^~~
```
[R/plotting-other.R:202:51:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L202) *style:* **Put spaces around all infix operators.**
```r
  if (any(raster::is.factor(ras1)) & all(ras1[]%%1==0)) {
                                                 ~^~~
```
[R/plotting-other.R:251:30:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L251) *style:* **Commas should always have a space after.**
```r
    if (sum(corners$coords[1,] - corners$coords[2,]) %==% 0) {
                             ^
```
[R/plotting-other.R:251:51:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L251) *style:* **Commas should always have a space after.**
```r
    if (sum(corners$coords[1,] - corners$coords[2,]) %==% 0) {
                                                  ^
```
[R/plotting-other.R:335:27:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L335) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (length(row) == 0) { # above the plot
                          ^
```
[R/plotting-other.R:338:30:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting-other.R#L338) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (length(column) == 0) { # above the plot
                             ^
```
[R/plotting.R:290:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting.R#L290) *style:* **Lines should not be more than 100 characters.**
```r
    isDoCall <- grepl("^do.call", scalls) & grepl("\\<Plot\\>", scalls) & !grepl("test_that", scalls)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:308:6:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting.R#L308) *style:* **Commented code should be removed.**
```r
    #whFrame <- grep(scalls, pattern = "^Plot|^quickPlot::Plot")
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:318:8:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting.R#L318) *style:* **Commented code should be removed.**
```r
      #dotObjs <- mget(dotNames[!noName], envir = plotFrame, inherits = TRUE)
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:322:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/R/plotting.R#L322) *style:* **Lines should not be more than 100 characters.**
```r
        if (is.null(title)) plotArgs$title <- ifelse(nzchar(names(dotObjs2)), names(dotObjs2), dotNames)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-clickExtent.R:11:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-clickExtent.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Sr1 <- Polygon(coords1)
  ^~~
```
[tests/testthat/test-clickExtent.R:12:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-clickExtent.R#L12) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Srs1 <- Polygons(list(Sr1), "s1")
  ^~~~
```
[tests/testthat/test-clickExtent.R:14:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-clickExtent.R#L14) *style:* **Lines should not be more than 100 characters.**
```r
  crs(shpEcozone) <- "+init=epsg:4326 +proj=longlat +datum=WGS84 +no_defs +ellps=WGS84 +towgs84=0,0,0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-clickExtent.R:24:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-clickExtent.R#L24) *style:* **Lines should not be more than 100 characters.**
```r
  bb <- raster::shapefile(file.path(cacheRepo, "EC_borealdisturbance_polygonal_2008_2010_FINAL_ALBERS.shp"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-examples.R:11:15:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-examples.R#L11) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
    setwd(owd)}
              ^
```
[tests/testthat/test-Plot-do-call.R:8:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L8) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Sr1 <- Polygon(coords1)
  ^~~
```
[tests/testthat/test-Plot-do-call.R:9:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L9) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Srs1 <- Polygons(list(Sr1), "s1")
  ^~~~
```
[tests/testthat/test-Plot-do-call.R:11:1:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L11) *style:* **Lines should not be more than 100 characters.**
```r
  crs(shpEcozone) <- "+init=epsg:4326 +proj=longlat +datum=WGS84 +no_defs +ellps=WGS84 +towgs84=0,0,0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-Plot-do-call.R:14:24:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L14) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                       ^
```
[tests/testthat/test-Plot-do-call.R:14:27:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L14) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                          ^
```
[tests/testthat/test-Plot-do-call.R:14:29:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-do-call.R#L14) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                            ^
```
[tests/testthat/test-Plot-nonVisual.R:65:21:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-nonVisual.R#L65) *style:* **Commented code should be removed.**
```r
  library(raster); #on.exit(detach("package:raster"), add = TRUE)
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-Plot-nonVisual.R:70:3:](https://github.com/PredictiveEcology/quickPlot/blob/6c78480b7a80dea97a0077fc8cf8a34d37c42b81/tests/testthat/test-Plot-nonVisual.R#L70) *style:* **Variable or function name should be lowerCamelCase.**
```r
  N <- ncol * nrow
  ^
```


ok


#6712-Zone-Coil-Autosizing-does-not-include-impact-of-AirTerminalSingleDuctMixer (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1684 of 1684 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-40.18%25-green.png)


Created multidev environment [sfgov#pr-88](https://dashboard.pantheon.io/sites/91d50373-c4cf-40e4-a646-cb73e16a140c#pr-88).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-88-sfgov.pantheonsite.io/)


The following doc(s) have been deployed to the [3274varss](https://3274varss-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/style-guide](https://3274varss-static-docs.pantheonsite.io/docs/style-guide)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/ecd9ffc76d6e7b19ac5fc7f27cb57dfba4210d88/source/docs/assets/css/main.css)


Why don't we need/want a version here?


@greyson-signal Do you consider dead code/unused return values something worth an issue or PR?


cake build and pack task copy this library, yet it will not be bundled in nuget fot netstandard1.6 ?


Sean, this commit causes a significant amount of corruption because you are using an out of date development environment. I think I'm just going to back it out. Sorry!


I'm not sure if this trick still works on in 8.4.2. I'm getting lots of errors like:

```
/nix/store/j7ik31915nyi96n559r4z8x389w3n5lw-android-ndk-r17/libexec/android-ndk-r17/toolchains/aarch64-linux-android-4.9/prebuilt/linux-x86_64/bin/../lib/gcc/aarch64-linux-android/4.9.x/../../../../aarch64-linux-android/bin/ld: /nix/store/nv96l9d9a96ra2yvwczng5vmh1bf7nll-jsaddle-dom-0.9.0.0-aarch64-unknown-linux-android/lib/ghc-8.4.2/jsaddle-dom-0.9.2.0/libHSjsaddle-dom-0.9.2.0-Gj9mXJoK54x4vXQlcc3ajc.a(Document.o): relocation R_AARCH64_ADR_PREL_PG_HI21 against external symbol `stg_upd_frame_info' can not be used when making a shared object; recompile with -fPIC
```

in cross


correcty


[R/plotting-colours.R:89:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L89) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:89:32:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L89) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:89:35:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L89) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:89:41:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L89) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:106:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L106) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:106:32:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L106) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:106:35:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L106) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:106:41:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L106) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:131:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L131) *style:* **Lines should not be more than 100 characters.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-colours.R:131:32:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L131) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                              ~^~~
```
[R/plotting-colours.R:131:35:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L131) *style:* **Put spaces around all infix operators.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                 ~^~~
```
[R/plotting-colours.R:131:41:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L131) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
      if (all(na.omit(object[])%%1==0)) { # some factor rasters are actually real number -- makes no sense
                                        ^
```
[R/plotting-colours.R:284:75:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L284) *style:* **Put spaces around all infix operators.**
```r
    isFac <- any(raster::is.factor(grobToPlot)) & all(na.omit(grobToPlot[]%%1)==0)
                                                                         ~^~~
```
[R/plotting-colours.R:284:79:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-colours.R#L284) *style:* **Put spaces around all infix operators.**
```r
    isFac <- any(raster::is.factor(grobToPlot)) & all(na.omit(grobToPlot[]%%1)==0)
                                                                             ~^~~
```
[R/plotting-helpers.R:1023:44:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1023) *style:* **Commas should always have a space after.**
```r
          grobToPlot <- grobToPlot[polySeq,]
                                           ^
```
[R/plotting-helpers.R:1028:38:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1028) *style:* **Commas should always have a space after.**
```r
        grobToPlot <- grobToPlot[a[1,],]
                                     ^
```
[R/plotting-helpers.R:1028:40:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1028) *style:* **Commas should always have a space after.**
```r
        grobToPlot <- grobToPlot[a[1,],]
                                       ^
```
[R/plotting-helpers.R:1291:41:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1291) *style:* **Put spaces around all infix operators.**
```r
            if (all(na.omit(grobToPlot[]%%1)==0)) {
                                       ~^~~
```
[R/plotting-helpers.R:1291:45:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1291) *style:* **Put spaces around all infix operators.**
```r
            if (all(na.omit(grobToPlot[]%%1)==0)) {
                                           ~^~~
```
[R/plotting-helpers.R:1380:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L1380) *style:* **Lines should not be more than 100 characters.**
```r
    grid.rect(x = 0, height = unit(1 + needsNewTitle * inherits(grobToPlot, "Raster") * 0.20 / (nRows / 2), "npc"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2511:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2511) *style:* **Lines should not be more than 100 characters.**
```r
#'          jitter(sort(rbeta(nPoints, betaPar, betaPar) * (maxY - minY) + minY, decreasing = TRUE)),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2589:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2589) *style:* **Variable or function name should be lowerCamelCase.**
```r
thin.SpatialPolygons <- function(x, tolerance = NULL, returnDataFrame = FALSE, minCoordsToThin = 0,
^~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2610:8:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2610) *style:* **Commented code should be removed.**
```r
      #xyOrd[["out"]] <- xyOrd[["out"]][thinRes, ]# thin line
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2633:50:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2633) *style:* **Commas should always have a space after.**
```r
            dat <- x@data[as.numeric(names1) + 1,]
                                                 ^
```
[R/plotting-helpers.R:2634:14:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2634) *style:* **Commented code should be removed.**
```r
            #row.names(dat) <- as.character(seq_len(length(xyOrd)))
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2666:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2666) *style:* **Variable or function name should be lowerCamelCase.**
```r
thin.default <- function(x, tolerance, returnDataFrame, minCoordsToThin, ...) {
^~~~~~~~~~~~
```
[R/plotting-helpers.R:2701:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2701) *style:* **Variable or function name should be lowerCamelCase.**
```r
  IDs <- tryCatch(unlist(lapply(ordSeq, function(xx) {
  ^~~
```
[R/plotting-helpers.R:2726:7:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2726) *style:* **Commented code should be removed.**
```r
    # Polygons <- rep(rep(seq(numPolygons), numPolygon), idLength$V1) # sequential numbering
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2727:5:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2727) *style:* **Variable or function name should be lowerCamelCase.**
```r
    Polygons <- rep(rep(IDs, numPolygon), idLength$V1) # actual ID labelling
    ^~~~~~~~
```
[R/plotting-helpers.R:2728:5:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2728) *style:* **Variable or function name should be lowerCamelCase.**
```r
    Polygon <- rep(unlist(lapply(numPolygon, seq)), idLength$V1)
    ^~~~~~~
```
[R/plotting-helpers.R:2735:36:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2735) *style:* **Commas should always have a space after.**
```r
    return(data.frame(lat = xyOrd[,1], long = xyOrd[,2], order = orders,
                                   ^
```
[R/plotting-helpers.R:2735:54:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2735) *style:* **Commas should always have a space after.**
```r
    return(data.frame(lat = xyOrd[,1], long = xyOrd[,2], order = orders,
                                                     ^
```
[R/plotting-helpers.R:2737:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2737) *style:* **Lines should not be more than 100 characters.**
```r
                      #group = paste0(as.character(Polygons), ".", as.character(Polygon)))) # the actual fortify
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2740:40:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2740) *style:* **Commas should always have a space after.**
```r
    out <- setDT(data.frame(x = xyOrd[,1], y = xyOrd[,2], groups = groups))
                                       ^
```
[R/plotting-helpers.R:2740:55:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2740) *style:* **Commas should always have a space after.**
```r
    out <- setDT(data.frame(x = xyOrd[,1], y = xyOrd[,2], groups = groups))
                                                      ^
```
[R/plotting-helpers.R:2755:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2755) *style:* **Lines should not be more than 100 characters.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting-helpers.R:2755:34:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2755) *style:* **Put spaces around all infix operators.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
                                ~^~
```
[R/plotting-helpers.R:2755:35:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2755) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    which(polygon@data$Shape_Area>(sort(polygon@data$Shape_Area, decreasing = TRUE)[maxNumPolygons]))
                                  ^
```
[R/plotting-helpers.R:2778:26:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-helpers.R#L2778) *style:* **Commas should always have a space after.**
```r
          totalAvailable," polygons in this view. See options('quickPlot.maxNumPolygons')")
                         ^
```
[R/plotting-other.R:202:48:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L202) *style:* **Put spaces around all infix operators.**
```r
  if (any(raster::is.factor(ras1)) & all(ras1[]%%1==0)) {
                                              ~^~~
```
[R/plotting-other.R:202:51:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L202) *style:* **Put spaces around all infix operators.**
```r
  if (any(raster::is.factor(ras1)) & all(ras1[]%%1==0)) {
                                                 ~^~~
```
[R/plotting-other.R:251:30:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L251) *style:* **Commas should always have a space after.**
```r
    if (sum(corners$coords[1,] - corners$coords[2,]) %==% 0) {
                             ^
```
[R/plotting-other.R:251:51:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L251) *style:* **Commas should always have a space after.**
```r
    if (sum(corners$coords[1,] - corners$coords[2,]) %==% 0) {
                                                  ^
```
[R/plotting-other.R:335:27:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L335) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (length(row) == 0) { # above the plot
                          ^
```
[R/plotting-other.R:338:30:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting-other.R#L338) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (length(column) == 0) { # above the plot
                             ^
```
[R/plotting.R:290:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting.R#L290) *style:* **Lines should not be more than 100 characters.**
```r
    isDoCall <- grepl("^do.call", scalls) & grepl("\\<Plot\\>", scalls) & !grepl("test_that", scalls)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:308:6:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting.R#L308) *style:* **Commented code should be removed.**
```r
    #whFrame <- grep(scalls, pattern = "^Plot|^quickPlot::Plot")
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:318:8:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting.R#L318) *style:* **Commented code should be removed.**
```r
      #dotObjs <- mget(dotNames[!noName], envir = plotFrame, inherits = TRUE)
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/plotting.R:322:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/R/plotting.R#L322) *style:* **Lines should not be more than 100 characters.**
```r
        if (is.null(title)) plotArgs$title <- ifelse(nzchar(names(dotObjs2)), names(dotObjs2), dotNames)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-clickExtent.R:11:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-clickExtent.R#L11) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Sr1 <- Polygon(coords1)
  ^~~
```
[tests/testthat/test-clickExtent.R:12:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-clickExtent.R#L12) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Srs1 <- Polygons(list(Sr1), "s1")
  ^~~~
```
[tests/testthat/test-clickExtent.R:14:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-clickExtent.R#L14) *style:* **Lines should not be more than 100 characters.**
```r
  crs(shpEcozone) <- "+init=epsg:4326 +proj=longlat +datum=WGS84 +no_defs +ellps=WGS84 +towgs84=0,0,0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-clickExtent.R:24:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-clickExtent.R#L24) *style:* **Lines should not be more than 100 characters.**
```r
  bb <- raster::shapefile(file.path(cacheRepo, "EC_borealdisturbance_polygonal_2008_2010_FINAL_ALBERS.shp"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-examples.R:11:15:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-examples.R#L11) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
    setwd(owd)}
              ^
```
[tests/testthat/test-Plot-do-call.R:8:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L8) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Sr1 <- Polygon(coords1)
  ^~~
```
[tests/testthat/test-Plot-do-call.R:9:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L9) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Srs1 <- Polygons(list(Sr1), "s1")
  ^~~~
```
[tests/testthat/test-Plot-do-call.R:11:1:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L11) *style:* **Lines should not be more than 100 characters.**
```r
  crs(shpEcozone) <- "+init=epsg:4326 +proj=longlat +datum=WGS84 +no_defs +ellps=WGS84 +towgs84=0,0,0"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-Plot-do-call.R:13:24:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L13) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                       ^
```
[tests/testthat/test-Plot-do-call.R:13:27:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L13) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                          ^
```
[tests/testthat/test-Plot-do-call.R:13:29:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-do-call.R#L13) *style:* **Commas should always have a space after.**
```r
  r <- raster(extent(0,10,0,10), res = 1, vals = 1:100)
                            ^
```
[tests/testthat/test-Plot-nonVisual.R:65:21:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-nonVisual.R#L65) *style:* **Commented code should be removed.**
```r
  library(raster); #on.exit(detach("package:raster"), add = TRUE)
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[tests/testthat/test-Plot-nonVisual.R:70:3:](https://github.com/PredictiveEcology/quickPlot/blob/e79bb2335c3b22633e0118f1b2580ed481f2eac6/tests/testthat/test-Plot-nonVisual.R#L70) *style:* **Variable or function name should be lowerCamelCase.**
```r
  N <- ncol * nrow
  ^
```


Puzzle `1168-428c4b61` disappeared from [`src/test/java/com/zerocracy/tk/PingTest.java`](https://github.com/zerocracy/farm/blob/master/src/test/java/com/zerocracy/tk/PingTest.java), that's why I closed #1201. Please, remember that the puzzle was not necessarily removed in this particular commit. Maybe it happened earlier, but we discovered this fact only now.


no


Sure but breakage usually offers some value! This is just renaming something for the sake of renaming it :( Sad times for anyone who had issues with this like we did.


Just to beat this horse further: why not write the migration as Python code instead of SQL then, to let Django do its own cascaded deletes? E.g. https://github.com/CJWorkbench/cjworkbench/blob/364f949a8475d25e486f613bd4ad4b583a57eeca/server/migrations/0082_storedobject_read.py#L7


Any specific reason to use Tensorflow slim ?
I was thinking of training with keras or Tensorflow


might add the rest later, rn it's just a couple hooks (also don't blame me for the lack of information, blame whmcs)


ffdfdfdfdfdfdfdfdf


#6712-Zone-Coil-Autosizing-does-not-include-impact-of-AirTerminalSingleDuctMixer (rraustad) - x86_64-MacOS-10.9-clang: OK (2920 of 2932 tests passed, 9 test warnings)

Messages:

 * 8 tests had: AUD diffs.
 * 12 tests had: EIO diffs.
 * 10 tests had: ESO big diffs.
 * 10 tests had: Table big diffs.
 * 5 tests had: ERR diffs.
 * 5 tests had: MTR big diffs.
 * 2 tests had: ESO small diffs.

Failures:

regression Test Summary
 * Passed: 611
 * Failed: 12

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


there's a bug in those code preventing the navbar toggler from being displayed on small screens


@bbaldino Why we need member.identity here (as a param to MUC_MEMBER_JOINED), I don't see it being used anywhere, or am I missing something?


Why do you need this struct?
Use GString instead.


[R/cat_names.R:5:18:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L5) *style:* **Commas should always have a space after.**
```r
  cat(paste(",'",names(x), "'",'\n', sep = ''))
                 ^
```
[R/cat_names.R:5:32:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L5) *style:* **Commas should always have a space after.**
```r
  cat(paste(",'",names(x), "'",'\n', sep = ''))
                               ^
```
[R/cat_names.R:5:32:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L5) *style:* **Only use double-quotes.**
```r
  cat(paste(",'",names(x), "'",'\n', sep = ''))
                               ^~~~
```
[R/cat_names.R:5:44:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L5) *style:* **Only use double-quotes.**
```r
  cat(paste(",'",names(x), "'",'\n', sep = ''))
                                           ^~
```
[R/cat_names.R:7:1:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L7) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/cat_names.R:8:1:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L8) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/cat_names.R:9:1:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cat_names.R#L9) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/clusDropVar2.R:21:13:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L21) *style:* **Commas should never have a space before.**
```r
  vte = c() ,
           ~^
```
[R/clusDropVar2.R:41:29:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L41) *style:* **Commas should always have a space after.**
```r
        ddm1 <- dudi.mix(x[,vtu1[-i]], add.square = FALSE, scannf = F, nf = ncol(x))
                            ^
```
[R/clusDropVar2.R:51:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L51) *style:* **Commas should always have a space after.**
```r
          pam1 <- clara(ddm1$li[,1:length(ddm1$eig[ddm1$eig > 1])], k = j, samples = 50)
                                 ^
```
[R/clusDropVar2.R:57:44:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L57) *style:* **Commas should always have a space after.**
```r
            print(paste("i: ", i, " | k: ",j," | sil: ",sil1, " | var: ",vtu1[i], sep = ""));flush.console()
                                           ^
```
[R/clusDropVar2.R:57:46:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L57) *style:* **Commas should always have a space after.**
```r
            print(paste("i: ", i, " | k: ",j," | sil: ",sil1, " | var: ",vtu1[i], sep = ""));flush.console()
                                             ^
```
[R/clusDropVar2.R:57:57:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L57) *style:* **Commas should always have a space after.**
```r
            print(paste("i: ", i, " | k: ",j," | sil: ",sil1, " | var: ",vtu1[i], sep = ""));flush.console()
                                                        ^
```
[R/clusDropVar2.R:57:74:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L57) *style:* **Commas should always have a space after.**
```r
            print(paste("i: ", i, " | k: ",j," | sil: ",sil1, " | var: ",vtu1[i], sep = ""));flush.console()
                                                                         ^
```
[R/clusDropVar2.R:71:109:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L71) *style:* **Commas should always have a space after.**
```r
      print(paste("Dropping var: ", varToDrop, " | Best Score : ", max(scores1Df$score), " | vtu1 length: ",length(vtu1) , sep = ""));flush.console()
                                                                                ^
```
[R/clusDropVar2.R:71:122:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/clusDropVar2.R#L71) *style:* **Commas should never have a space before.**
```r
      print(paste("Dropping var: ", varToDrop, " | Best Score : ", max(scores1Df$score), " | vtu1 length: ",length(vtu1) , sep = ""));flush.console()
                                                                                ~^
```
[R/cnn.R:13:19:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cnn.R#L13) *style:* **Commas should always have a space after.**
```r
    print(paste(i,": ", colnames(dataset)[i], " | ", class(dataset[,i]), sep=""))
                  ^
```
[R/cnn.R:13:69:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cnn.R#L13) *style:* **Commas should always have a space after.**
```r
    print(paste(i,": ", colnames(dataset)[i], " | ", class(dataset[,i]), sep=""))
                                                                    ^
```
[R/cnn.R:13:77:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/cnn.R#L13) *style:* **Put spaces around all infix operators.**
```r
    print(paste(i,": ", colnames(dataset)[i], " | ", class(dataset[,i]), sep=""))
                                                                           ~^~
```
[R/hiFreqFact.R:13:27:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/hiFreqFact.R#L13) *style:* **Commas should always have a space after.**
```r
  rt1 <- rt1[rt1$Freq > y,]
                          ^
```
[R/hiFreqFact.R:14:32:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/hiFreqFact.R#L14) *style:* **Only use double-quotes.**
```r
  x <- ifelse(x %in% rt1$x, x, 'other')
                               ^~~~~~~
```
[R/ls.objects.R:26:49:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/ls.objects.R#L26) *style:* **Put spaces around all infix operators.**
```r
    out <- out[order(out[[order.by]], decreasing=decreasing), ]
                                               ~^~
```
[R/lsos.R:18:26:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/lsos.R#L18) *style:* **Put spaces around all infix operators.**
```r
  lsObjects(..., order.by="Size", decreasing=TRUE, head=TRUE, n=n)
                        ~^~
```
[R/lsos.R:18:45:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/lsos.R#L18) *style:* **Put spaces around all infix operators.**
```r
  lsObjects(..., order.by="Size", decreasing=TRUE, head=TRUE, n=n)
                                           ~^~
```
[R/lsos.R:18:56:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/lsos.R#L18) *style:* **Put spaces around all infix operators.**
```r
  lsObjects(..., order.by="Size", decreasing=TRUE, head=TRUE, n=n)
                                                      ~^~
```
[R/lsos.R:18:64:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/lsos.R#L18) *style:* **Put spaces around all infix operators.**
```r
  lsObjects(..., order.by="Size", decreasing=TRUE, head=TRUE, n=n)
                                                              ~^~
```
[R/naHandler.R:31:1:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/naHandler.R#L31) *style:* **Trailing blank lines are superfluous.**
```r

^
```
[R/profileDat.R:11:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L11) *style:* **Variable and function names should not use underscores.**
```r
  pro_df <- data.frame(row_num = 1:ncol(x), col_name = names(x))
  ^~~~~~
```
[R/profileDat.R:12:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L12) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$row_cnt <- nrow(x)
  ^~~~~~
```
[R/profileDat.R:12:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L12) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$row_cnt <- nrow(x)
         ^~~~~~~
```
[R/profileDat.R:14:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L14) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$uniq_val <- sapply(x, function(x) length(unique(x)))
  ^~~~~~
```
[R/profileDat.R:14:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L14) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$uniq_val <- sapply(x, function(x) length(unique(x)))
         ^~~~~~~~
```
[R/profileDat.R:16:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L16) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_cnt <- sapply(x, function(x) sum(is.na(x)))
  ^~~~~~
```
[R/profileDat.R:16:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L16) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_cnt <- sapply(x, function(x) sum(is.na(x)))
         ^~~~~~
```
[R/profileDat.R:18:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L18) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_cnt <- sapply(x, function(x) length(x[!is.na(x) & as.character(x) == '']))
  ^~~~~~
```
[R/profileDat.R:18:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L18) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_cnt <- sapply(x, function(x) length(x[!is.na(x) & as.character(x) == '']))
         ^~~~~~~~~
```
[R/profileDat.R:18:85:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L18) *style:* **Only use double-quotes.**
```r
  pro_df$blank_cnt <- sapply(x, function(x) length(x[!is.na(x) & as.character(x) == '']))
                                                                                ^~
```
[R/profileDat.R:19:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L19) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$top_val <- sapply(x, function(x) rankTab(x)[1,'x'])
  ^~~~~~
```
[R/profileDat.R:19:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L19) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$top_val <- sapply(x, function(x) rankTab(x)[1,'x'])
         ^~~~~~~
```
[R/profileDat.R:19:56:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L19) *style:* **Commas should always have a space after.**
```r
  pro_df$top_val <- sapply(x, function(x) rankTab(x)[1,'x'])
                                                       ^
```
[R/profileDat.R:19:56:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L19) *style:* **Only use double-quotes.**
```r
  pro_df$top_val <- sapply(x, function(x) rankTab(x)[1,'x'])
                                                       ^~~
```
[R/profileDat.R:21:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
  ^~~~~~
```
[R/profileDat.R:21:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
         ^~~~~~
```
[R/profileDat.R:21:26:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
                         ^~~~~~
```
[R/profileDat.R:21:33:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
                                ^~~~~~
```
[R/profileDat.R:21:42:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
                                         ^~~~~~
```
[R/profileDat.R:21:49:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L21) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$na_pct <- round(pro_df$na_cnt / pro_df$row_cnt, 4)
                                                ^~~~~~~
```
[R/profileDat.R:22:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
  ^~~~~~
```
[R/profileDat.R:22:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
         ^~~~~~~~~
```
[R/profileDat.R:22:29:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
                            ^~~~~~
```
[R/profileDat.R:22:36:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
                                   ^~~~~~~~~
```
[R/profileDat.R:22:48:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
                                               ^~~~~~
```
[R/profileDat.R:22:55:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L22) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$blank_pct <- round(pro_df$blank_cnt / pro_df$row_cnt, 4)
                                                      ^~~~~~~
```
[R/profileDat.R:24:3:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L24) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$top_5_val_pct <- round(sapply(x, function(x) sum(sum(rankTab(x)[1:5, "Freq"], na.rm = TRUE)) / length(x)), 4)
  ^~~~~~
```
[R/profileDat.R:24:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L24) *style:* **Variable and function names should not use underscores.**
```r
  pro_df$top_5_val_pct <- round(sapply(x, function(x) sum(sum(rankTab(x)[1:5, "Freq"], na.rm = TRUE)) / length(x)), 4)
         ^~~~~~~~~~~~~
```
[R/profileDat.R:27:10:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/profileDat.R#L27) *style:* **Variable and function names should not use underscores.**
```r
  return(pro_df)
         ^~~~~~
```
[R/rankTab.R:12:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/rankTab.R#L12) *style:* **Commas should always have a space after.**
```r
  tab1 <- tab1[order(-tab1$Freq),]
                                 ^
```
[R/setTypes.R:15:39:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L15) *style:* **Only use double-quotes.**
```r
    df1 <- data.frame(var = i, type = '', stringsAsFactors = F)
                                      ^~
```
[R/setTypes.R:17:30:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L17) *style:* **Commas should always have a space after.**
```r
    print(as.character(x[1:5,i]));flush.console()
                             ^
```
[R/setTypes.R:19:11:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L19) *style:* **Commas should always have a space after.**
```r
    df1[1,'type'] <- readline('Type - f / n / c?')
          ^
```
[R/setTypes.R:19:11:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L19) *style:* **Only use double-quotes.**
```r
    df1[1,'type'] <- readline('Type - f / n / c?')
          ^~~~~~
```
[R/setTypes.R:19:31:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L19) *style:* **Only use double-quotes.**
```r
    df1[1,'type'] <- readline('Type - f / n / c?')
                              ^~~~~~~~~~~~~~~~~~~
```
[R/setTypes.R:25:30:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L25) *style:* **Only use double-quotes.**
```r
  charVar <- df2[df2$type == 'c','var']
                             ^~~
```
[R/setTypes.R:25:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L25) *style:* **Commas should always have a space after.**
```r
  charVar <- df2[df2$type == 'c','var']
                                 ^
```
[R/setTypes.R:25:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L25) *style:* **Only use double-quotes.**
```r
  charVar <- df2[df2$type == 'c','var']
                                 ^~~~~
```
[R/setTypes.R:26:29:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L26) *style:* **Only use double-quotes.**
```r
  numVar <- df2[df2$type == 'n','var']
                            ^~~
```
[R/setTypes.R:26:33:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L26) *style:* **Commas should always have a space after.**
```r
  numVar <- df2[df2$type == 'n','var']
                                ^
```
[R/setTypes.R:26:33:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L26) *style:* **Only use double-quotes.**
```r
  numVar <- df2[df2$type == 'n','var']
                                ^~~~~
```
[R/setTypes.R:27:30:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L27) *style:* **Only use double-quotes.**
```r
  factVar <- df2[df2$type == 'f','var']
                             ^~~
```
[R/setTypes.R:27:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L27) *style:* **Commas should always have a space after.**
```r
  factVar <- df2[df2$type == 'f','var']
                                 ^
```
[R/setTypes.R:27:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L27) *style:* **Only use double-quotes.**
```r
  factVar <- df2[df2$type == 'f','var']
                                 ^~~~~
```
[R/setTypes.R:29:6:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L29) *style:* **Commas should always have a space after.**
```r
  x[,charVar] <- as.data.frame(lapply(x[,charVar], as.character))
     ^
```
[R/setTypes.R:29:42:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L29) *style:* **Commas should always have a space after.**
```r
  x[,charVar] <- as.data.frame(lapply(x[,charVar], as.character))
                                         ^
```
[R/setTypes.R:30:6:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L30) *style:* **Commas should always have a space after.**
```r
  x[,numVar] <- as.data.frame(lapply(x[,numVar], function(x) return(as.numeric(as.character(x)))))
     ^
```
[R/setTypes.R:30:41:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L30) *style:* **Commas should always have a space after.**
```r
  x[,numVar] <- as.data.frame(lapply(x[,numVar], function(x) return(as.numeric(as.character(x)))))
                                        ^
```
[R/setTypes.R:31:6:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L31) *style:* **Commas should always have a space after.**
```r
  x[,factVar] <- as.data.frame(lapply(x[,factVar], as.factor))
     ^
```
[R/setTypes.R:31:42:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/R/setTypes.R#L31) *style:* **Commas should always have a space after.**
```r
  x[,factVar] <- as.data.frame(lapply(x[,factVar], as.factor))
                                         ^
```
[tests/testthat/test-clusDropVar2.R:7:81:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-clusDropVar2.R#L7) *style:* **Put spaces around all infix operators.**
```r
  d <- clusDropVar(iris, 3:5, threshold = .6, vte = "Species", xLoopsBeforePrint=1)
                                                                               ~^~
```
[tests/testthat/test-lint.R:5:5:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L5) *style:* **Variable and function names should not use underscores.**
```r
    my_linters <- list(
    ^~~~~~~~~~
```
[tests/testthat/test-lint.R:6:28:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L6) *style:* **Put spaces around all infix operators.**
```r
      absolute_paths_linter=absolute_paths_linter,
                          ~^~
```
[tests/testthat/test-lint.R:7:24:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L7) *style:* **Put spaces around all infix operators.**
```r
      assignment_linter=assignment_linter,
                      ~^~
```
[tests/testthat/test-lint.R:8:26:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L8) *style:* **Put spaces around all infix operators.**
```r
      closed_curly_linter=closed_curly_linter,
                        ~^~
```
[tests/testthat/test-lint.R:9:20:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L9) *style:* **Put spaces around all infix operators.**
```r
      commas_linter=commas_linter,
                  ~^~
```
[tests/testthat/test-lint.R:11:26:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L11) *style:* **Put spaces around all infix operators.**
```r
      infix_spaces_linter=infix_spaces_linter,
                        ~^~
```
[tests/testthat/test-lint.R:12:25:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L12) *style:* **Put spaces around all infix operators.**
```r
      line_length_linter=line_length_linter,
                       ~^~
```
[tests/testthat/test-lint.R:13:20:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L13) *style:* **Put spaces around all infix operators.**
```r
      no_tab_linter=no_tab_linter,
                  ~^~
```
[tests/testthat/test-lint.R:14:26:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L14) *style:* **Put spaces around all infix operators.**
```r
      object_usage_linter=object_usage_linter,
                        ~^~
```
[tests/testthat/test-lint.R:15:24:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L15) *style:* **Put spaces around all infix operators.**
```r
      snake_case_linter=snake_case_linter,
                      ~^~
```
[tests/testthat/test-lint.R:16:27:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L16) *style:* **Put spaces around all infix operators.**
```r
      multiple_dots_linter=multiple_dots_linter,
                         ~^~
```
[tests/testthat/test-lint.R:17:27:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L17) *style:* **Put spaces around all infix operators.**
```r
      object_length_linter=object_length_linter,
                         ~^~
```
[tests/testthat/test-lint.R:18:24:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L18) *style:* **Put spaces around all infix operators.**
```r
      open_curly_linter=open_curly_linter,
                      ~^~
```
[tests/testthat/test-lint.R:19:27:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L19) *style:* **Put spaces around all infix operators.**
```r
      single_quotes_linter=single_quotes_linter,
                         ~^~
```
[tests/testthat/test-lint.R:20:27:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L20) *style:* **Put spaces around all infix operators.**
```r
      spaces_inside_linter=spaces_inside_linter,
                         ~^~
```
[tests/testthat/test-lint.R:22:34:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L22) *style:* **Put spaces around all infix operators.**
```r
      trailing_blank_lines_linter=trailing_blank_lines_linter,
                                ~^~
```
[tests/testthat/test-lint.R:23:33:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L23) *style:* **Put spaces around all infix operators.**
```r
      trailing_whitespace_linter=trailing_whitespace_linter
                               ~^~
```
[tests/testthat/test-lint.R:25:36:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L25) *style:* **Put spaces around all infix operators.**
```r
    lintr::expect_lint_free(linters=my_linters)
                                  ~^~
```
[tests/testthat/test-lint.R:25:37:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-lint.R#L25) *style:* **Variable and function names should not use underscores.**
```r
    lintr::expect_lint_free(linters=my_linters)
                                    ^~~~~~~~~~
```
[tests/testthat/test-naHandler.R:15:7:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-naHandler.R#L15) *style:* **Commas should always have a space after.**
```r
  x[1,] <- NA
      ^
```
[tests/testthat/test-naHandler.R:17:18:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-naHandler.R#L17) *style:* **Only use double-quotes.**
```r
  expect_is(nas, 'list')
                 ^~~~~~
```
[tests/testthat/test-rankTab.R:10:28:](https://github.com/barrenWuffet/convPkg/blob/da69ba82efe04d9d88e65e1a4f8feb110b2e729c/tests/testthat/test-rankTab.R#L10) *style:* **Commas should always have a space after.**
```r
  expect_equal(dim(d), c(3,2))
                           ^
```


Release is tagged. I remembered to build the windows binaries with directdraw this time. :-)


That's WAY better than what's currently there. Can't someone update the
tuning branch?

On Thu, Jun 21, 2018 at 3:11 PM, Rocky640 <notifications@github.com> wrote:

> By default , small values will not change much. Positive values will not
> allowed to be negative.
> And value 0 will not be tuned at all.
>
> One way to "fix" this is to hack the current default_range
>
> inline Range default_range(double v) {
>   return v > 0 ? Range(0, 2 * v) : Range(2 * v, 0);
> }
>
> to something else. Many solutions are possible, here is one
>
> inline Range default_range(double v) {
>   int mindelta = 10; //always make sure we have at least +-10
>   return v > 0 ? Range(std::min(0, v-mindelta),std::max(2*v, v+mindelta))
>                        : Range(std::min(2*v, v-mindelta),std::max(0, v+mindelta));
> }
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/protonspring/Stockfish/commit/33a1bae3bd5c33434e4fcf8aa13014d019fccbd4#commitcomment-29456079>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AcVnSFK950KM3n7PV2mBUv016uhxzJPZks5t_AwGgaJpZM4UysV->
> .
>

--
Michael T. Whiteley
mobile: 801.707.6886


haha, don't worry. fixed :)


Do not use `printf` here (we have `msg_debug`, `msg_error`, etc...).

(I think, this message is not needed here.)

And one more interesting thing: when you use printf for debugging, sometimes using fprintf to stderr could be a better choice as stderr on most platforms not buffering and prints your message immediately.


Well Done


Hello jason, I'm looking to use skggm for implementing some papers that require quic. I would also like to be able to use it with sklearn 0.19, and am happy to help with any testing you might need. Is there any way you can move the requirement to >=0.18 ?


what if you configure syslog-ng with _two_ (or even more) destination?


The following doc(s) have been deployed to the [3274varss](https://3274varss-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/style-guide](https://3274varss-static-docs.pantheonsite.io/docs/style-guide)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/ecd9ffc76d6e7b19ac5fc7f27cb57dfba4210d88/source/docs/assets/css/main.css)


This test has been committed with CRLF newlines.


use glib string operations instead (and use `gchar` instead of `char`, this is a syslog-ng convention)


![](https://github.trello.services/images/mini-trello-icon.png) [07 - Tela de sucesso ao realizar uma compra](https://trello.com/c/NWak9fm0/121-07-tela-de-sucesso-ao-realizar-uma-compra)


git add --all

~$git commit -m "Initial commit"

~$git push -u origin master


What do you think about just a flat range of +/- 20?:

return Range(v-20,v+20);

On Thu, Jun 21, 2018 at 3:49 PM, Michael Whiteley <mike@whiteley.org> wrote:

> That's WAY better than what's currently there. Can't someone update the
> tuning branch?
>
> On Thu, Jun 21, 2018 at 3:11 PM, Rocky640 <notifications@github.com>
> wrote:
>
>> By default , small values will not change much. Positive values will not
>> allowed to be negative.
>> And value 0 will not be tuned at all.
>>
>> One way to "fix" this is to hack the current default_range
>>
>> inline Range default_range(double v) {
>>   return v > 0 ? Range(0, 2 * v) : Range(2 * v, 0);
>> }
>>
>> to something else. Many solutions are possible, here is one
>>
>> inline Range default_range(double v) {
>>   int mindelta = 10; //always make sure we have at least +-10
>>   return v > 0 ? Range(std::min(0, v-mindelta),std::max(2*v, v+mindelta))
>>                        : Range(std::min(2*v, v-mindelta),std::max(0, v+mindelta));
>> }
>>
>> —
>> You are receiving this because you authored the thread.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/protonspring/Stockfish/commit/33a1bae3bd5c33434e4fcf8aa13014d019fccbd4#commitcomment-29456079>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/AcVnSFK950KM3n7PV2mBUv016uhxzJPZks5t_AwGgaJpZM4UysV->
>> .
>>
>
>
>
> --
> Michael T. Whiteley
> mobile: 801.707.6886
>

--
Michael T. Whiteley
mobile: 801.707.6886


naming: give a more expressive name to this function


`self.locations = []` can be removed from `_basic_login()`, it is already in `installation()`, and that is the best place for it.

Or have I missed something?


#158531941-VerticalVenetianBlindCoefficients (vidanovic) - Win64-Windows-7-VisualStudio-14: OK (2933 of 2933 tests passed, 13 test warnings)

Messages:

 * 11 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: SSZ small diffs.
 * 1 test had: ZSZ small diffs.

![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)


TeamCity Glyssen :: Glyssen pull requests - untagged [Build *.*.30.0a038451fddc9fa5cd2a1f1b79097deb2f5eab01](http://build.palaso.org/viewLog.html?buildId=123142&buildTypeId=Glyssen_GlyssenPullRequestsUntagged) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:31


![](https://github.trello.services/images/mini-trello-icon.png) [FE : Create login screen](https://trello.com/c/9fVQQbRA/9-fe-create-login-screen)


When a function don’t have parameters, use a `void` in the formal parameter list.
`void init(void)`.


yeah i guess this is kinda sneaky.  jibri injects javascript code into meet when it joins and it adds a listener for this event.  When it fires, jibri's handler uses the member.identity field.


Не рекомендовал бы использовать модуль Message внутри наших "внутренних" классов. В классе интерфэйс - ок. Но все остальные классы у нас отвечают за работу с данными и лучше от методов в этих классах получать данные/true/false/nil, и потом уже в клиентском коде (вызывающем метод) - выводить текст, если wagon_add(wagon) не вернет правдивого значения


Apology to interrupt you but I think you are looking at a commit which has
been done 14 days ago.

Here is the latest one -
https://github.com/videetssinghai/syslog-ng/commit/d33f06529d54051675928afcdfda2a83834b3e6c

On Fri, Jun 22, 2018 at 3:22 AM, Budai Laszlo <notifications@github.com>
wrote:

> use glib string operations instead (and use gchar instead of char, this
> is a syslog-ng convention)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/videetssinghai/syslog-ng/commit/875d132fdb8b64479081faaff449b5c429b1269a#commitcomment-29456460>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AMFlFIGy8PHLytucE5uj3u5edqij5vGRks5t_BWlgaJpZM4Uy0Ts>
> .
>


Using global init/deinit is not something good.


![](https://github.trello.services/images/mini-trello-icon.png) [36 - Tela de nenhum produto cadastrado (Produtor)](https://trello.com/c/FqjYuxb6/183-36-tela-de-nenhum-produto-cadastrado-produtor)


Looks great!


remove every comment that contains code: a comment should be a comment and code should be code, avoid mixing them in final code.


![](https://github.trello.services/images/mini-trello-icon.png) [Commit?](https://trello.com/c/p2JzU7jx/1-commit)


Pure briliance


Как я уже писал [тут](https://github.com/denio-rus/thinknetica-ruby/commit/2a5baf157f723567dbd7f78cb29c8b8caa048ff6#diff-59cabc3304dab06a6f3f40ec139ffae0L26)


![](https://github.trello.services/images/mini-trello-icon.png) [Furniture buying period](https://trello.com/c/NpHnEL88/3-furniture-buying-period)


MemoryStruct as a name is too generic. And there is `GString`...


TeamCity Glyssen :: Glyssen pull requests [Build *.*.717.0a038451fddc9fa5cd2a1f1b79097deb2f5eab01](http://build.palaso.org/viewLog.html?buildId=123141&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1844, ignored: 18 Build time: 00:07:53


The following doc(s) have been deployed to the [3274varss](https://3274varss-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/style-guide](https://3274varss-static-docs.pantheonsite.io/docs/style-guide)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/ecd9ffc76d6e7b19ac5fc7f27cb57dfba4210d88/source/docs/assets/css/main.css)


Thanks – and sorry! Should be fixed with 23ca95aa3e8b138572717c012e65ec657340fefe.


You still have a global shared state. What will happen when you have two or more active dst in a cfg?


Run `make style-format` on your code. Before that install `astyle`.


Because when the model code changes, the migrations break -- breaking new installs (and unit/integration tests).

I was affected by this problem today, in another migration. I created `server_wfmodule.has_unseen_notification`, and in dev mode everything was fine. But when I went to run unit tests, this migration refused to run: https://github.com/CJWorkbench/cjworkbench/blob/85e3798759a9804aed2e16eb3b4071eec5fff6d6/server/migrations/0093_storedobject_to_uploadedfile.py -- I got a uselessly-cryptic SQL error, so it took me 20min to figure out what was happening: Django was trying to select all WfModule columns in Migration 93 -- but that column isn't there yet.

The theory goes, if you make migrations depend on the code that depends on the migrations, you're doomed.

For migration 93, I just commented out the code: I assume everybody who needs to run it already has.

For an open-source project, it's extra-important to adhere to the theory. Otherwise, occasional developers' databases will break.

Maybe the Migration-93 approach is what we should use for the next month or two. In retrospect, the pain I experienced with this particular RunSQL migration may be greater than the sum total of pain everybody would experience had I used RunPython.

Longer-term, we should certainly never make migrations run model code. That _always_ breaks. The better approach is to use simpler data models. This migration would be a breeze if:

* we weren't using django-polymorphic
* parameterval weren't its own table
* server_wfmodule.workflow_id were NOT NULL (that is, commands could not own quasi-orphan WfModules)
* StoredObjects on disk were in a directory specific to the workflow

I view the first three points as bad schema design. Files on disk will always be painful to migrate, so we should make a sincere effort to avoid changing file paths and whatnot. (In Overview we haven't changed blob paths in years; we did change our search-index path, but we modeled it as database cache instead of a database dependency, so the migration was easy.)


Thanks! I forgot that Python handles basename slightly different to the linux shell....


Should you mark the node as unvisited here? This should still work, and you could remove the ugly `dest_node->type == TEST` check below.


Мне нравится идея с модулями (хотя честно говоря не все менторы ее одобряют), только ты включил модуль на "верхнем" уровне. И теперь он доступен по всей программе. (я понимаю, что именно с этой целью ты и включил его на верхнем уровне) Но исходя из моего комента выше, модуль следует включить именно в класс Interface. И обращаться к нему кстати можно не используя константу, а в данно случае просто так
```
else
  wrong_choise
end
```
Правда, учитывая этот комент, может понадобиться к имени метода добавить слово message