[inst/examples/globe/server.R:3:27:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L3) *style:* **Put spaces around all infix operators.**
```r
data(world.cities, package="maps")
                         ~^~
```
[inst/examples/globe/server.R:5:23:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L5) *style:* **Put spaces around all infix operators.**
```r
earth_dark <- list(img=system.file("images/world.jpg",package="threejs"),
                     ~^~
```
[inst/examples/globe/server.R:5:55:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L5) *style:* **Commas should always have a space after.**
```r
earth_dark <- list(img=system.file("images/world.jpg",package="threejs"),
                                                      ^
```
[inst/examples/globe/server.R:5:62:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L5) *style:* **Put spaces around all infix operators.**
```r
earth_dark <- list(img=system.file("images/world.jpg",package="threejs"),
                                                            ~^~
```
[inst/examples/globe/server.R:6:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L6) *style:* **Put spaces around all infix operators.**
```r
                   bodycolor="#0011ff",
                           ~^~
```
[inst/examples/globe/server.R:7:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L7) *style:* **Put spaces around all infix operators.**
```r
                   emissive="#0011ff",
                          ~^~
```
[inst/examples/globe/server.R:8:30:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L8) *style:* **Put spaces around all infix operators.**
```r
                   lightcolor="#99ddff")
                            ~^~
```
[inst/examples/globe/server.R:14:42:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L14) *style:* **Commas should always have a space after.**
```r
     world.cities[order(world.cities$pop,decreasing=TRUE)[1:input$N],]
                                         ^
```
[inst/examples/globe/server.R:14:52:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
     world.cities[order(world.cities$pop,decreasing=TRUE)[1:input$N],]
                                                  ~^~
```
[inst/examples/globe/server.R:14:70:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L14) *style:* **Commas should always have a space after.**
```r
     world.cities[order(world.cities$pop,decreasing=TRUE)[1:input$N],]
                                                                     ^
```
[inst/examples/globe/server.R:20:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L20) *style:* **Put spaces around all infix operators.**
```r
    col <- rainbow(10, start=2.8 / 6,end=3.4 / 6)
                           ~^~
```
[inst/examples/globe/server.R:20:38:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L20) *style:* **Commas should always have a space after.**
```r
    col <- rainbow(10, start=2.8 / 6,end=3.4 / 6)
                                     ^
```
[inst/examples/globe/server.R:20:41:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L20) *style:* **Put spaces around all infix operators.**
```r
    col <- rainbow(10, start=2.8 / 6,end=3.4 / 6)
                                       ~^~
```
[inst/examples/globe/server.R:24:15:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L24) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
             ~^~
```
[inst/examples/globe/server.R:24:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L24) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
                          ~^~
```
[inst/examples/globe/server.R:24:40:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L24) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
                                      ~^~
```
[inst/examples/globe/server.R:30:26:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Commas should never have a space before.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                        ~^
```
[inst/examples/globe/server.R:30:36:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Put spaces around all infix operators.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                                  ~^~
```
[inst/examples/globe/server.R:30:55:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Put spaces around all infix operators.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                                                     ~^~
```
[inst/examples/globe/server.R:30:76:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Put spaces around all infix operators.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                                                                          ~^~
```
[inst/examples/globe/server.R:30:91:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Put spaces around all infix operators.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                                                                                ~^~
```
[inst/examples/globe/server.R:30:111:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L30) *style:* **Put spaces around all infix operators.**
```r
    args <- c(earth_dark , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=TRUE))
                                                                                ~^~
```
[inst/examples/globe/server.R:31:26:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/server.R#L31) *style:* **Put spaces around all infix operators.**
```r
    do.call(globejs, args=args)
                        ~^~
```
[inst/examples/globe/ui.R:8:57:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/globe/ui.R#L8) *style:* **Put spaces around all infix operators.**
```r
      sliderInput("N", "Number of cities to plot", value=5000, min = 100, max = 10000, step = 100),
                                                       ~^~
```
[inst/examples/scatterplot/server.R:15:40:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/inst/examples/scatterplot/server.R#L15) *style:* **Commas should always have a space after.**
```r
    color <- rep(rainbow(input$colors),length.out = nrow(data))
                                       ^
```
[R/globe.R:147:47:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L147) *style:* **Put spaces around all infix operators.**
```r
  img=system.file("images/world.jpg",  package="threejs"),
                                             ~^~
```
[R/globe.R:161:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L161) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(lat) || missing(long))
    ^
```
[R/globe.R:168:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L168) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/globe.R:171:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L171) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(j))
      ^
```
[R/globe.R:177:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L177) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/globe.R:180:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L180) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(j))
      ^
```
[R/globe.R:186:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L186) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0) bg <- substr(bg, 1, 7)
    ^
```
[R/globe.R:187:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L187) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(arcs))
    ^
```
[R/globe.R:197:22:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L197) *style:* **Put spaces around all infix operators.**
```r
  options <- list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                    ~^~
```
[R/globe.R:197:32:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L197) *style:* **Put spaces around all infix operators.**
```r
  options <- list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                              ~^~
```
[R/globe.R:197:44:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L197) *style:* **Put spaces around all infix operators.**
```r
  options <- list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                                          ~^~
```
[R/globe.R:197:63:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L197) *style:* **Put spaces around all infix operators.**
```r
  options <- list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                                                             ~^~
```
[R/globe.R:198:23:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L198) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                     ~^~
```
[R/globe.R:198:41:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L198) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                       ~^~
```
[R/globe.R:198:56:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L198) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                                      ~^~
```
[R/globe.R:198:65:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L198) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                                               ~^~
```
[R/globe.R:199:27:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L199) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                         ~^~
```
[R/globe.R:199:46:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L199) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                                            ~^~
```
[R/globe.R:199:66:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L199) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                                                                ~^~
```
[R/globe.R:201:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L201) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(additional_args) > 0) options <- c(options, additional_args)
    ^
```
[R/globe.R:203:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L203) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("bodycolor" %in% names(options))
    ^
```
[R/globe.R:205:20:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L205) *style:* **Commas should always have a space after.**
```r
    i <- grep("^#",options$bodycolor)
                   ^
```
[R/globe.R:206:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L206) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(length(i) > 0) options$bodycolor <- substr(options$bodycolor, 1, 7)
      ^
```
[R/globe.R:208:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L208) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("emissive" %in% names(options))
    ^
```
[R/globe.R:211:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L211) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(length(i) > 0) options$emissive <- substr(options$emissive, 1, 7)
      ^
```
[R/globe.R:213:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L213) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("lightcolor" %in% names(options))
    ^
```
[R/globe.R:216:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L216) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(length(i) > 0) options$lightcolor <- substr(options$lightcolor, 1, 7)
      ^
```
[R/globe.R:220:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L220) *warning:* **no visible global function definition for ‘texture’**
```r
  if (!is.list(img)) img <- texture(img)
                            ^~~~~~~
```
[R/globe.R:244:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L244) *warning:* **no visible global function definition for ‘shinyRenderWidget’**
```r
    shinyRenderWidget(expr, globeOutput, env, quoted = TRUE)
    ^~~~~~~~~~~~~~~~~
```
[R/globe.R:244:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/globe.R#L244) *warning:* **no visible binding for global variable ‘globeOutput’**
```r
    shinyRenderWidget(expr, globeOutput, env, quoted = TRUE)
                            ^~~~~~~~~~~
```
[R/graph.R:121:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L121) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.list(g) && "igraph" %in% class(g[[1]]))
    ^
```
[R/graph.R:124:31:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L124) *warning:* **no visible binding for global variable ‘as_edgelist’**
```r
    fromlist <- lapply(g[-1], as_edgelist)
                              ^~~~~~~~~~~
```
[R/graph.R:127:45:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L127) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
    lcollist <- lapply(g[-1], function(x) { ifel(is.null(E(x)$color), NA, E(x)$color) })
                                            ^~~~
```
[R/graph.R:127:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L127) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
    lcollist <- lapply(g[-1], function(x) { ifel(is.null(E(x)$color), NA, E(x)$color) })
                                                         ^
```
[R/graph.R:127:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L127) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
    lcollist <- lapply(g[-1], function(x) { ifel(is.null(E(x)$color), NA, E(x)$color) })
                                                         ^
```
[R/graph.R:128:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L128) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(missing(layout)) layout <- lapply(g, function(x) {ifel(is.null(x$layout), layout_with_fr(x, dim=3, niter=50), x$layout)})
      ^
```
[R/graph.R:128:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L128) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
    if(missing(layout)) layout <- lapply(g, function(x) {ifel(is.null(x$layout), layout_with_fr(x, dim=3, niter=50), x$layout)})
                                                         ^~~~
```
[R/graph.R:128:82:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L128) *warning:* **no visible global function definition for ‘layout_with_fr’**
```r
    if(missing(layout)) layout <- lapply(g, function(x) {ifel(is.null(x$layout), layout_with_fr(x, dim=3, niter=50), x$layout)})
                                                                                ^~~~~~~~~~~~~~
```
[R/graph.R:128:103:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L128) *style:* **Put spaces around all infix operators.**
```r
    if(missing(layout)) layout <- lapply(g, function(x) {ifel(is.null(x$layout), layout_with_fr(x, dim=3, niter=50), x$layout)})
                                                                                ~^~
```
[R/graph.R:128:112:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L128) *style:* **Put spaces around all infix operators.**
```r
    if(missing(layout)) layout <- lapply(g, function(x) {ifel(is.null(x$layout), layout_with_fr(x, dim=3, niter=50), x$layout)})
                                                                                ~^~
```
[R/graph.R:131:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L131) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!("igraph" %in% class(g))) stop("g must be an igraph object")
    ^
```
[R/graph.R:133:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L133) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(layout)) layout <- ifel(is.null(g$layout), function(g) layout_with_fr(g, dim=3, niter=50), g$layout)
    ^
```
[R/graph.R:133:33:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L133) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(layout)) layout <- ifel(is.null(g$layout), function(g) layout_with_fr(g, dim=3, niter=50), g$layout)
                                ^~~~
```
[R/graph.R:133:69:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L133) *warning:* **no visible global function definition for ‘layout_with_fr’**
```r
  if(missing(layout)) layout <- ifel(is.null(g$layout), function(g) layout_with_fr(g, dim=3, niter=50), g$layout)
                                                                    ^~~~~~~~~~~~~~
```
[R/graph.R:133:90:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L133) *style:* **Put spaces around all infix operators.**
```r
  if(missing(layout)) layout <- ifel(is.null(g$layout), function(g) layout_with_fr(g, dim=3, niter=50), g$layout)
                                                                                ~^~
```
[R/graph.R:133:99:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L133) *style:* **Put spaces around all infix operators.**
```r
  if(missing(layout)) layout <- ifel(is.null(g$layout), function(g) layout_with_fr(g, dim=3, niter=50), g$layout)
                                                                                ~^~
```
[R/graph.R:134:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(vertex.color)) vertex.color <- ifel(is.null(V(g)$color), "orange", V(g)$color)
    ^
```
[R/graph.R:134:45:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L134) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(vertex.color)) vertex.color <- ifel(is.null(V(g)$color), "orange", V(g)$color)
                                            ^~~~
```
[R/graph.R:134:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L134) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.color)) vertex.color <- ifel(is.null(V(g)$color), "orange", V(g)$color)
                                                         ^
```
[R/graph.R:134:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L134) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.color)) vertex.color <- ifel(is.null(V(g)$color), "orange", V(g)$color)
                                                         ^
```
[R/graph.R:135:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L135) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(vertex.size)) vertex.size <- ifel(is.null(V(g)$size), 2, V(g)$size / 7)
    ^
```
[R/graph.R:135:43:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L135) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(vertex.size)) vertex.size <- ifel(is.null(V(g)$size), 2, V(g)$size / 7)
                                          ^~~~
```
[R/graph.R:135:56:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L135) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.size)) vertex.size <- ifel(is.null(V(g)$size), 2, V(g)$size / 7)
                                                       ^
```
[R/graph.R:135:56:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L135) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.size)) vertex.size <- ifel(is.null(V(g)$size), 2, V(g)$size / 7)
                                                       ^
```
[R/graph.R:136:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L136) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(vertex.shape)) vertex.shape <- ifel(is.null(V(g)$shape), "circle", V(g)$shape)
    ^
```
[R/graph.R:136:45:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L136) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(vertex.shape)) vertex.shape <- ifel(is.null(V(g)$shape), "circle", V(g)$shape)
                                            ^~~~
```
[R/graph.R:136:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L136) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.shape)) vertex.shape <- ifel(is.null(V(g)$shape), "circle", V(g)$shape)
                                                         ^
```
[R/graph.R:136:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L136) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.shape)) vertex.shape <- ifel(is.null(V(g)$shape), "circle", V(g)$shape)
                                                         ^
```
[R/graph.R:137:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L137) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(vertex.label)) vertex.label <- ifel(is.null(V(g)$label), NA, V(g)$label)
    ^
```
[R/graph.R:137:45:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L137) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(vertex.label)) vertex.label <- ifel(is.null(V(g)$label), NA, V(g)$label)
                                            ^~~~
```
[R/graph.R:137:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L137) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.label)) vertex.label <- ifel(is.null(V(g)$label), NA, V(g)$label)
                                                         ^
```
[R/graph.R:137:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L137) *warning:* **no visible global function definition for ‘V’, Did you mean 'g'?**
```r
  if(missing(vertex.label)) vertex.label <- ifel(is.null(V(g)$label), NA, V(g)$label)
                                                         ^
```
[R/graph.R:138:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L138) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(edge.color)) edge.color <- ifel(is.null(E(g)$color), NA, E(g)$color)
    ^
```
[R/graph.R:138:41:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L138) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(edge.color)) edge.color <- ifel(is.null(E(g)$color), NA, E(g)$color)
                                        ^~~~
```
[R/graph.R:138:54:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L138) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
  if(missing(edge.color)) edge.color <- ifel(is.null(E(g)$color), NA, E(g)$color)
                                                     ^
```
[R/graph.R:138:54:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L138) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
  if(missing(edge.color)) edge.color <- ifel(is.null(E(g)$color), NA, E(g)$color)
                                                     ^
```
[R/graph.R:139:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L139) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(edge.width)) edge.width <- ifel(is.null(E(g)$width), 1, E(g)$width)
    ^
```
[R/graph.R:139:41:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L139) *warning:* **no visible global function definition for ‘ifel’, Did you mean 'file'?**
```r
  if(missing(edge.width)) edge.width <- ifel(is.null(E(g)$width), 1, E(g)$width)
                                        ^~~~
```
[R/graph.R:139:54:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L139) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
  if(missing(edge.width)) edge.width <- ifel(is.null(E(g)$width), 1, E(g)$width)
                                                     ^
```
[R/graph.R:139:54:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L139) *warning:* **no visible global function definition for ‘E’, Did you mean 'g'?**
```r
  if(missing(edge.width)) edge.width <- ifel(is.null(E(g)$width), 1, E(g)$width)
                                                     ^
```
[R/graph.R:140:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L140) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(edge.alpha))
    ^
```
[R/graph.R:142:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L142) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(length(E(g)) > 1000) edge.alpha <- 0.4
      ^
```
[R/graph.R:147:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L147) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.function(layout)) layout <- layout(g)
    ^
```
[R/graph.R:148:8:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L148) *warning:* **no visible global function definition for ‘as_edgelist’**
```r
  g <- as_edgelist(g)
       ^~~~~~~~~~~
```
[R/graph.R:152:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L152) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.list(layout))
    ^
```
[R/graph.R:155:33:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L155) *warning:* **no visible binding for global variable ‘norm_coords’**
```r
    layout <- Reduce(rbind, Map(norm_coords, layout))
                                ^~~~~~~~~~~
```
[R/graph.R:161:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L161) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("pie" %in% u) pch <- gsub("pie", "@", pch)
    ^
```
[R/graph.R:162:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L162) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("sphere" %in% u) pch <- gsub("sphere", "o", pch)
    ^
```
[R/graph.R:163:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L163) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("square" %in% u) pch <- gsub("square", ".", pch)
    ^
```
[R/graph.R:164:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L164) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("csquare" %in% u) pch <- gsub("csquare", ".", pch)
    ^
```
[R/graph.R:165:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L165) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("rectangle" %in% u) pch <- gsub("rectangle", ".", pch)
    ^
```
[R/graph.R:166:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L166) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("crectangle" %in% u) pch <- gsub("crectangle", ".", pch)
    ^
```
[R/graph.R:167:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L167) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("vrectangle" %in% u) pch <- gsub("vrectangle", ".", pch)
    ^
```
[R/graph.R:168:22:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L168) *style:* **Put spaces around all infix operators.**
```r
  options <- c(list(x=layout, pch=pch, size=vertex.size, color=vertex.color, from=g,
                    ~^~
```
[R/graph.R:168:34:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L168) *style:* **Put spaces around all infix operators.**
```r
  options <- c(list(x=layout, pch=pch, size=vertex.size, color=vertex.color, from=g,
                                ~^~
```
[R/graph.R:168:44:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L168) *style:* **Put spaces around all infix operators.**
```r
  options <- c(list(x=layout, pch=pch, size=vertex.size, color=vertex.color, from=g,
                                          ~^~
```
[R/graph.R:168:63:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L168) *style:* **Put spaces around all infix operators.**
```r
  options <- c(list(x=layout, pch=pch, size=vertex.size, color=vertex.color, from=g,
                                                             ~^~
```
[R/graph.R:168:82:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L168) *style:* **Put spaces around all infix operators.**
```r
  options <- c(list(x=layout, pch=pch, size=vertex.size, color=vertex.color, from=g,
                                                                                ~^~
```
[R/graph.R:169:22:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L169) *style:* **Put spaces around all infix operators.**
```r
                  lwd=edge.width, linealpha=edge.alpha, lcol=edge.color,
                    ~^~
```
[R/graph.R:169:44:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L169) *style:* **Put spaces around all infix operators.**
```r
                  lwd=edge.width, linealpha=edge.alpha, lcol=edge.color,
                                          ~^~
```
[R/graph.R:169:61:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L169) *style:* **Put spaces around all infix operators.**
```r
                  lwd=edge.width, linealpha=edge.alpha, lcol=edge.color,
                                                           ~^~
```
[R/graph.R:170:23:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
                  axis=FALSE, grid=FALSE, center=TRUE, bg=bg, main=main), list(...))
                     ~^~
```
[R/graph.R:170:35:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
                  axis=FALSE, grid=FALSE, center=TRUE, bg=bg, main=main), list(...))
                                 ~^~
```
[R/graph.R:170:49:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
                  axis=FALSE, grid=FALSE, center=TRUE, bg=bg, main=main), list(...))
                                               ~^~
```
[R/graph.R:170:58:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
                  axis=FALSE, grid=FALSE, center=TRUE, bg=bg, main=main), list(...))
                                                        ~^~
```
[R/graph.R:170:67:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
                  axis=FALSE, grid=FALSE, center=TRUE, bg=bg, main=main), list(...))
                                                                 ~^~
```
[R/graph.R:171:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L171) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!(length(vertex.label) == 1 && is.na(vertex.label))) options$labels <- vertex.label
    ^
```
[R/graph.R:172:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L172) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!(length(edge.color) == 1 && is.na(edge.color))) options$lcol <- edge.color
    ^
```
[R/graph.R:173:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L173) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.na(scenes)) options$scenes <- scenes
    ^
```
[R/graph.R:174:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L174) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(glist)
    ^
```
[R/graph.R:176:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if((length(fromlist) + 1) != scenes) stop("mismatching graph and layout lengths")
      ^
```
[R/graph.R:176:8:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if((length(fromlist) + 1) != scenes) stop("mismatching graph and layout lengths")
       ^
```
[R/graph.R:181:33:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/graph.R#L181) *style:* **Put spaces around all infix operators.**
```r
  do.call("scatterplot3js", args=options)
                               ~^~
```
[R/scatterplot.R:209:19:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L209) *style:* **Commas should always have a space after.**
```r
  num.ticks = c(6,6,6),
                  ^
```
[R/scatterplot.R:209:21:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L209) *style:* **Commas should always have a space after.**
```r
  num.ticks = c(6,6,6),
                    ^
```
[R/scatterplot.R:225:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L225) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.matrix(x))
      ^
```
[R/scatterplot.R:233:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L233) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(pch)) pch <- rep("o", nrow(x))
    ^
```
[R/scatterplot.R:234:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L234) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(pch) != nrow(x)) pch <- rep_len(pch, nrow(x))
    ^
```
[R/scatterplot.R:240:42:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L240) *style:* **Commas should always have a space after.**
```r
  color <- sub("^(#[[:xdigit:]]{6}+).*$","\\1", color, perl = TRUE)
                                         ^
```
[R/scatterplot.R:241:39:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L241) *style:* **Commas should always have a space after.**
```r
  bg <- sub("^(#[[:xdigit:]]{6}+).*$","\\1", bg, perl = TRUE)
                                      ^
```
[R/scatterplot.R:251:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L251) *style:* **Put spaces around all infix operators.**
```r
  x <- x[, c(1, 3, 2), drop=FALSE]
                          ~^~
```
[R/scatterplot.R:262:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L262) *style:* **Put spaces around all infix operators.**
```r
  mn <- apply(x[, 1:3, drop=FALSE], 2, min)
                          ~^~
```
[R/scatterplot.R:263:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L263) *style:* **Put spaces around all infix operators.**
```r
  mx <- apply(x[, 1:3, drop=FALSE], 2, max)
                          ~^~
```
[R/scatterplot.R:278:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L278) *style:* **Put spaces around all infix operators.**
```r
  x[, 1:3] <- (x[, 1:3, drop=FALSE] - rep(mn, each = n)) / (rep(mx - mn, each = n))
                           ~^~
```
[R/scatterplot.R:282:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L282) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("center" %in% names(options) && options$center) # not yet documented, useful for graph
    ^
```
[R/scatterplot.R:286:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L286) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("scenes" %in% names(options)) # XXX experimental
    ^
```
[R/scatterplot.R:289:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L289) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(N != floor(N)) stop("number of points must be a multiple of scenes")
      ^
```
[R/scatterplot.R:290:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L290) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if("fps" %in% names(options)) nframes <- options$fps * (options$scenes - 1)
      ^
```
[R/scatterplot.R:291:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L291) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if("fpl" %in% names(options)) nframes <- options$fpl * (options$scenes - 1)
      ^
```
[R/scatterplot.R:297:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L297) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!("linealpha" %in% names(options))) options$linealpha <- 1
    ^
```
[R/scatterplot.R:308:40:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L308) *style:* **Commas should always have a space after.**
```r
    num.ticks <- pmax(1, num.ticks[c(1,3,2)])
                                       ^
```
[R/scatterplot.R:308:42:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L308) *style:* **Commas should always have a space after.**
```r
    num.ticks <- pmax(1, num.ticks[c(1,3,2)])
                                         ^
```
[R/scatterplot.R:310:19:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L310) *style:* **Put spaces around all infix operators.**
```r
    t1 <- seq(from=mn[1], to=mx[1], length.out=num.ticks[1])
                 ~^~
```
[R/scatterplot.R:310:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L310) *style:* **Put spaces around all infix operators.**
```r
    t1 <- seq(from=mn[1], to=mx[1], length.out=num.ticks[1])
                           ~^~
```
[R/scatterplot.R:310:47:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L310) *style:* **Put spaces around all infix operators.**
```r
    t1 <- seq(from=mn[1], to=mx[1], length.out=num.ticks[1])
                                             ~^~
```
[R/scatterplot.R:312:19:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L312) *style:* **Put spaces around all infix operators.**
```r
    t2 <- seq(from=mn[2], to=mx[2], length.out=num.ticks[2])
                 ~^~
```
[R/scatterplot.R:312:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L312) *style:* **Put spaces around all infix operators.**
```r
    t2 <- seq(from=mn[2], to=mx[2], length.out=num.ticks[2])
                           ~^~
```
[R/scatterplot.R:312:47:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L312) *style:* **Put spaces around all infix operators.**
```r
    t2 <- seq(from=mn[2], to=mx[2], length.out=num.ticks[2])
                                             ~^~
```
[R/scatterplot.R:314:19:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L314) *style:* **Put spaces around all infix operators.**
```r
    t3 <- seq(from=mn[3], to=mx[3], length.out=num.ticks[3])
                 ~^~
```
[R/scatterplot.R:314:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L314) *style:* **Put spaces around all infix operators.**
```r
    t3 <- seq(from=mn[3], to=mx[3], length.out=num.ticks[3])
                           ~^~
```
[R/scatterplot.R:314:47:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L314) *style:* **Put spaces around all infix operators.**
```r
    t3 <- seq(from=mn[3], to=mx[3], length.out=num.ticks[3])
                                             ~^~
```
[R/scatterplot.R:316:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L316) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(flip.y) t3 <- t3[length(t3):1]
      ^
```
[R/scatterplot.R:322:9:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L322) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(any(i))
        ^
```
[R/scatterplot.R:332:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L332) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!is.null(x.ticklabs)) options$xticklab <- x.ticklabs
      ^
```
[R/scatterplot.R:333:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L333) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!is.null(y.ticklabs)) options$zticklab <- y.ticklabs
      ^
```
[R/scatterplot.R:334:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L334) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!is.null(z.ticklabs)) options$yticklab <- z.ticklabs
      ^
```
[R/scatterplot.R:341:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L341) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("from" %in% names(options))
    ^
```
[R/scatterplot.R:343:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L343) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.matrix(options$from) && ncol(options$from) == 2)
      ^
```
[R/scatterplot.R:345:48:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L345) *style:* **Commented code should be removed.**
```r
      options$to <- options$from[, 2] - 1    # zero-index
                                               ^~~~~~~~~~
```
[R/scatterplot.R:347:9:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L347) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(length(options$from) == 1)
        ^
```
[R/scatterplot.R:356:9:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L356) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(!("to" %in% names(options))) stop("both from and to must be specified")
        ^
```
[R/scatterplot.R:359:9:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L359) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(nl != length(options$to)) stop("from and to must be the same length")
        ^
```
[R/scatterplot.R:361:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L361) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!("lwd" %in% names(options))) options$lwd <- 1L
      ^
```
[R/scatterplot.R:363:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L363) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("fromlist" %in% names(options))
    ^
```
[R/scatterplot.R:365:7:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L365) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!("tolist" %in% names(options))) stop("both fromlist and tolist must be specified")
      ^
```
[R/scatterplot.R:366:68:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L366) *style:* **Commented code should be removed.**
```r
    options$fromlist <- Map(function(x) x - 1, options$fromlist) # zero-index
                                                                   ^~~~~~~~~~
```
[R/scatterplot.R:393:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L393) *warning:* **no visible global function definition for ‘shinyWidgetOutput’**
```r
    shinyWidgetOutput(outputId, "scatterplotThree", width, height, package = "threejs")
    ^~~~~~~~~~~~~~~~~
```
[R/scatterplot.R:400:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L400) *warning:* **no visible global function definition for ‘shinyRenderWidget’**
```r
    shinyRenderWidget(expr, scatterplotThreeOutput, env, quoted = TRUE)
    ^~~~~~~~~~~~~~~~~
```
[R/scatterplot.R:400:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L400) *warning:* **no visible binding for global variable ‘scatterplotThreeOutput’**
```r
    shinyRenderWidget(expr, scatterplotThreeOutput, env, quoted = TRUE)
                            ^~~~~~~~~~~~~~~~~~~~~~
```
[R/scatterplot.R:410:9:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L410) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(is.matrix(x))
        ^
```
[R/scatterplot.R:420:44:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L420) *style:* **Commas should always have a space after.**
```r
    color <- sub("^#([[:xdigit:]]{6}+).*$","\\1", color, perl = TRUE)
                                           ^
```
[R/scatterplot.R:423:30:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L423) *style:* **Put spaces around all infix operators.**
```r
    x <- x[, c(1, 3, 2), drop=FALSE]
                            ~^~
```
[R/scatterplot.R:425:31:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L425) *style:* **Put spaces around all infix operators.**
```r
    x[, 1:3] <- (x[, 1:3, drop=FALSE] - rep(options$mn, each = n)) / (rep(options$mx - options$mn, each = n))
                             ~^~
```
[R/scatterplot.R:432:34:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L432) *style:* **Commas should always have a space after.**
```r
    x <- as.vector(t(signif(data,signif)))
                                 ^
```
[R/scatterplot.R:437:90:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L437) *style:* **Put spaces around all infix operators.**
```r
      local_options$size <- c(rep(options$size, length.out = n_old), rep(size, length.out=n))
                                                                                ~^~
```
[R/scatterplot.R:442:61:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L442) *style:* **Put spaces around all infix operators.**
```r
      local_options$color <- c(rep(options$color, length.out=n_old), rep(color, length.out=n))
                                                           ~^~
```
[R/scatterplot.R:442:91:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L442) *style:* **Put spaces around all infix operators.**
```r
      local_options$color <- c(rep(options$color, length.out=n_old), rep(color, length.out=n))
                                                                                ~^~
```
[R/scatterplot.R:447:57:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L447) *style:* **Put spaces around all infix operators.**
```r
      local_options$pch <- c(rep(options$pch, length.out=n_old), rep(pch, length.out=n))
                                                       ~^~
```
[R/scatterplot.R:447:85:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/scatterplot.R#L447) *style:* **Put spaces around all infix operators.**
```r
      local_options$pch <- c(rep(options$pch, length.out=n_old), rep(pch, length.out=n))
                                                                                ~^~
```
[R/utilities.R:43:23:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L43) *style:* **Commas should always have a space after.**
```r
  ext <- gsub(".*\\.","", data)
                      ^
```
[R/utilities.R:44:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L44) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(grepl("^http", data))
    ^
```
[R/utilities.R:46:24:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L46) *style:* **Put spaces around all infix operators.**
```r
    u <- url(data, open="rb")
                      ~^~
```
[R/utilities.R:49:29:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L49) *style:* **Put spaces around all infix operators.**
```r
    writeBin(readBin(u, what="raw", n=10e6), data, useBytes=TRUE)
                           ~^~
```
[R/utilities.R:49:38:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L49) *style:* **Put spaces around all infix operators.**
```r
    writeBin(readBin(u, what="raw", n=10e6), data, useBytes=TRUE)
                                    ~^~
```
[R/utilities.R:49:60:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L49) *style:* **Put spaces around all infix operators.**
```r
    writeBin(readBin(u, what="raw", n=10e6), data, useBytes=TRUE)
                                                          ~^~
```
[R/utilities.R:53:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L53) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(nchar(ext) < 1) ext <- "png"
    ^
```
[R/utilities.R:54:10:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L54) *warning:* **no visible global function definition for ‘dataURI’**
```r
  img <- dataURI(file=data, mime=sprintf("image/%s", ext))
         ^~~~~~~
```
[R/utilities.R:54:22:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L54) *style:* **Put spaces around all infix operators.**
```r
  img <- dataURI(file=data, mime=sprintf("image/%s", ext))
                    ~^~
```
[R/utilities.R:54:33:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L54) *style:* **Put spaces around all infix operators.**
```r
  img <- dataURI(file=data, mime=sprintf("image/%s", ext))
                               ~^~
```
[R/utilities.R:55:11:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L55) *style:* **Put spaces around all infix operators.**
```r
  list(img=img, dataURI=TRUE)
         ~^~
```
[R/utilities.R:55:24:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L55) *style:* **Put spaces around all infix operators.**
```r
  list(img=img, dataURI=TRUE)
                      ~^~
```
[R/utilities.R:61:5:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/R/utilities.R#L61) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(isTRUE(a)) return(b)
    ^
```
[tests/threejs.R:4:26:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L4) *style:* **Put spaces around all infix operators.**
```r
i <- sample(3, N, replace=TRUE)
                        ~^~
```
[tests/threejs.R:5:31:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L5) *style:* **Put spaces around all infix operators.**
```r
x <- matrix(rnorm(N * 3), ncol=3)
                             ~^~
```
[tests/threejs.R:6:24:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L6) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x, color=rainbow(N), size=i, renderer="canvas")
                      ~^~
```
[tests/threejs.R:6:41:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L6) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x, color=rainbow(N), size=i, renderer="canvas")
                                       ~^~
```
[tests/threejs.R:6:53:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L6) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x, color=rainbow(N), size=i, renderer="canvas")
                                                   ~^~
```
[tests/threejs.R:11:18:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L11) *style:* **Commas should always have a space after.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)))
                 ^
```
[tests/threejs.R:11:20:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L11) *style:* **Commas should always have a space after.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)))
                   ^
```
[tests/threejs.R:11:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L11) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)))
                          ~^~
```
[tests/threejs.R:12:18:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L12) *style:* **Commas should always have a space after.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)), axisLabels=c("a", "b", "c"))
                 ^
```
[tests/threejs.R:12:20:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L12) *style:* **Commas should always have a space after.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)), axisLabels=c("a", "b", "c"))
                   ^
```
[tests/threejs.R:12:28:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L12) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)), axisLabels=c("a", "b", "c"))
                          ~^~
```
[tests/threejs.R:12:59:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L12) *style:* **Put spaces around all infix operators.**
```r
scatterplot3js(x,y,z, color=rainbow(length(z)), axisLabels=c("a", "b", "c"))
                                                         ~^~
```
[tests/threejs.R:15:22:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L15) *style:* **Put spaces around all infix operators.**
```r
data(flights, package="threejs")
                    ~^~
```
[tests/threejs.R:17:37:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L17) *style:* **Put spaces around all infix operators.**
```r
freq <- sort(table(dest), decreasing=TRUE)
                                   ~^~
```
[tests/threejs.R:22:12:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L22) *style:* **Put spaces around all infix operators.**
```r
globejs(lat=ll[, 1], long=ll[, 2], arcs=frequent_flights,
          ~^~
```
[tests/threejs.R:22:26:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L22) *style:* **Put spaces around all infix operators.**
```r
globejs(lat=ll[, 1], long=ll[, 2], arcs=frequent_flights,
                        ~^~
```
[tests/threejs.R:22:40:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L22) *style:* **Put spaces around all infix operators.**
```r
globejs(lat=ll[, 1], long=ll[, 2], arcs=frequent_flights,
                                      ~^~
```
[tests/threejs.R:23:19:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L23) *style:* **Put spaces around all infix operators.**
```r
        arcsHeight=0.3, arcsLwd=2, arcsColor="#ffff00", arcsOpacity=0.15,
                 ~^~
```
[tests/threejs.R:23:32:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L23) *style:* **Put spaces around all infix operators.**
```r
        arcsHeight=0.3, arcsLwd=2, arcsColor="#ffff00", arcsOpacity=0.15,
                              ~^~
```
[tests/threejs.R:23:45:](https://github.com/bwlewis/rthreejs/blob/c6fed672f9316a6a71c508c27cca3fd85372580c/tests/threejs.R#L23) *style:* **Put spaces around all infix operators.**
```r
        arcsHeight=0.3, arcsLwd=2, arcsColor="#ffff00", arcsOpacity=0.15,
                                           ~^~
```


Yeah. Once we have that many we'll have to rethink the nav all together


error, not sure how it snuck back in


testing that bug is tricky tho. As long as we don't run an embedded ES instance or something.


Yeah, then I undid it. Struggling with props vs state. Though thought of a way to get back to this set up by putting the filter function in the render method.


Uhh, broken indentation?


Changes to SHR and MITRE sections


using `normalize.css` we run back into the loading stylesheets problem again.

by including only once, we run back to the same problem of using `!important`


I would replace the "so please" with ", please"


The problem is likely caused by not incrementing `n_off` as originally, in the condition at line 1838


Agree 100%. Just done for the short term as we get initial edits/updates done prior to launch.


Any particular reason why we don't place these in a PythonWrappers sub namespace?


r+


Curious why these null checks are not needed?


Nice! Everyone loves to be able to set things like this.


@tisto Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.vocabularies/16/violations
```
plone/app/vocabularies/actions.py:52:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/security.py:73:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/security.py:132:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/security.py:146:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:98:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:149:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:189:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:240:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:307:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/testing.py:18:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/syndication.py:43:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/syndication.py:68:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/__init__.py:20:13: D001 found directlyProvides( replace it with zope.interface.provider
plone/app/vocabularies/editors.py:33:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:72:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:139:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:239:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/skins.py:72:1: E305 expected 2 blank lines after class or function definition, found 1
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Also indentation?


Also indentation again?


@herkulano thanks for clarifying. So no straightforward way to pack the ```yarn build``` to the image. And if we are doing this there then perhaps not such a big issue with keeping the ```git clone`` operation in there too as you point out it's not expensive.

I think there aren't any containers in docker-compose dependant on the Dashboard container so the start-up time of that is not such a big deal.


this should probably be just `response_headers.append((headername, webresp.msg.get(headername)))`


@asadsahi - Welcome to the `JavaScript Renaissance` fun time to be an engineer :)


This seems to also fix #122.


```java
this.multiQueryParams.put(key,previousValues);
```
I hope you dont need this statement, map keeps references and list is a reference which already got the elements added.


Can you please try again on your i386 machine? I think I've sorted this.
Please let me know if this now works!

*Gianluca Baio*

*Reader in Statistics & Health Economics*
Department of Statistical Science
University College London
1-19 Torrington Place, London WC1E 6BT, UK
Telephone +44(0)20 7679 1248 (internal: 41248)
Fax +44 (0)20 7383 4703
Website: http://www.homepages.ucl.ac.uk/~ucakgba
<http://www.homepages.ucl.ac.uk/~ucakgba/>
Blog: http://gianlubaio.blogspot.co.uk

On 12 January 2017 at 14:21, Gianluca Baio <g.baio@ucl.ac.uk> wrote:

> Good --- glad this works. I am not sure how to take care of the
> distinction between the binary files for the two architectures. I had a
> quick look around, but couldn't find indication of how to deal with this.
> Will keep looking --- nearly there! :-)
>
>
>
> *Gianluca Baio*
>
> *Reader in Statistics & Health Economics*
> Department of Statistical Science
> University College London
> 1-19 Torrington Place, London WC1E 6BT, UK
> Telephone +44(0)20 7679 1248 <020%207679%201248> (internal: 41248)
> Fax +44 (0)20 7383 4703 <020%207383%204703>
> Website: http://www.homepages.ucl.ac.uk/~ucakgba
> <http://www.homepages.ucl.ac.uk/~ucakgba/>
> Blog: http://gianlubaio.blogspot.co.uk
>
> On 12 January 2017 at 14:18, Peter Konings <notifications@github.com>
> wrote:
>
>> Sorry, my bad! I always forget that my work laptop is 32 bit - and I did
>> not understand that the warning message referred to that distinction. It
>> works like a charm under 64 bit.
>> There must be a way to make this work for both 32 and 64 bit systems, as
>> CRAN does this transparently.
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/giabaio/survHE/commit/5969685091f9fc887732b83b9d5cd647304adb4c#commitcomment-20452756>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/AIUayYmZErZgRru9XSZ8cCaxkZa1MnjCks5rRjZRgaJpZM4LhuEH>
>> .
>>
>
>


This is a very nice and useful implementation. Thanks a lot!


No. Do you recommend me to add this sub namespace?


I think there is a typo here:


TeamCity Win32 Bindings for Haskell :: Win32 Continuous builds [Build 24](http://exia:8888/viewLog.html?buildId=346&buildTypeId=Win32_Win32ContinuousBuilds) is now running


I stopped working on it because there was improper DB flagging of fished
up monsters at the time. That has since been fixed, I just haven't
gotten around to messing with it since.

I also have a minor packet adjustment from setzor that needs to be put in.

On 1/12/2017 9:11 AM, nyczducky wrote:
>
> Been on my TODO list to get some of this working but got swamped
> during holiday season. I can look through it too if you put up a PR. I
> was going to do some retail fishing packet captures especially around
> the mob fishing.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/DarkstarProject/darkstar/commit/7d82f314d2e7937738b843e458e36f7b95888c13#commitcomment-20452674>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AGPFUUaKro_2IhXjCyW-TyBXlFlTMgLLks5rRjStgaJpZM4Lhyd4>.
>


Comment


@timgraham Thanks for the hint. Please see [ticket #27730](https://code.djangoproject.com/ticket/27730#ticket).


Hi, Do you have a different library to recommend? Thanks


@afolarin I'm trying it out, hopefully, it will work 👍


Let's add it again with a bigger length. Will do that on next PR for json parsing


Typically you want the namespacing to match the project/directory structure. Since these files appear to be inside a PythonWrappers directory, I would place them in a PythonWrappers namespace as well.


bundle.zip


yeah this change alone did nothing.


According to the new design, We can remove most asserts. But for training cells, contributor may want to judge that the result is desirable. Currently this is achieved by adding some assertions to training or prediction cells. This can help ensure both the mxnet core function and the notebook itself are working correctly.

Figures are automatically ignored when testing. Detail instructions for contribute a notebook will be posted after automated testing script is completed.


#3 Tsk Tsk :V


What if you have multiple videos?


TeamCity Win32 Bindings for Haskell :: Win32 Continuous builds [Build 24](http://exia:8888/viewLog.html?buildId=346&buildTypeId=Win32_Win32ContinuousBuilds) outcome was **SUCCESS**
Summary: Running Build time: 00:07:21


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 977](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43945&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:13


137438445_Issue5133 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2334 of 2391 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-97.62%25-yellow.png)


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 978](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43949&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:16


@tisto Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.vocabularies/17/violations
```
plone/app/vocabularies/security.py:73:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/security.py:132:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/security.py:146:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:98:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:149:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:189:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:240:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/types.py:307:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:72:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:139:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/workflow.py:239:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/editors.py:33:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/__init__.py:20:13: D001 found directlyProvides( replace it with zope.interface.provider
plone/app/vocabularies/skins.py:72:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/testing.py:18:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/actions.py:52:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/syndication.py:43:1: E305 expected 2 blank lines after class or function definition, found 1
plone/app/vocabularies/syndication.py:68:1: E305 expected 2 blank lines after class or function definition, found 1
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


This is a bit of (probably unhelpful) snark. Most of the rhetoric seems to dismiss an understanding of `this` with "just use arrow functions instead".


@Scope(value = "request", proxyMode = ScopedProxyMode.TARGET_CLASS)
to be autowired in classes annotated with @Component


This should have a comma


put up image on website


Typo commit msg.

original commit msg like
```
Modify build tools, maven to gradle
```

new commit msg like
```
Change build tool, maven to gradle
```


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 979](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43953&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency failed: ... Lint Server (new)) Build time: 00:00:01


TeamCity Platform Services :: Environment Manager :: Test Client [Build 951](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43954&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency canceled: ... Test Lambda (new)) Build time: 00:00:00


137438445_Issue5133 (Nigusse) - 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.png)


TeamCity Platform Services :: Environment Manager :: Test Client [Build 949](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43946&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:50


that works, i slightly changed it by removing the style attribute as it's not necessary


TeamCity Platform Services :: Environment Manager :: Test Client [Build 950](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43950&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:48


BugIssue_5020 (Nigusse) - Win64-Windows-7-VisualStudio-14: OK (2285 of 2359 tests passed, 500 test warnings)

 * 515 tests had: AUD diffs.
 * 15 tests had: Table big diffs.

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


Project from Semester 1 for HCI and Web Dev.

Written in HTML and CSS


When history provider is set by this method, it has been initialize at [Engine.cs](https://github.com/QuantConnect/Lean/blob/master/Engine/Engine.cs#L127), thus a null exception would have thrown before.

I don't see why RunTimeError cannot be null, since there is a null check at [AlgorithmManager.cs](https://github.com/QuantConnect/Lean/blob/master/Engine/AlgorithmManager.cs#L367).


I went there


Joli le $(...)


`<iframe width="560" height="315" src="https://www.youtube.com/embed/7lkXS8jhEbc" frameborder="0" allowfullscreen></iframe>`


Hey, I don't like this image, can we change it?


Nice! I added your output as an example to the readme at https://github.com/sudomesh/sudowrt-firmware#the-easy-way  . Did you try running the image? On my machine is produced a bunch of output in the built_firmware directory as expected. Same for you?


Example of a comment:  Let's think about changing this image.


TeamCity Platform Services :: Environment Manager :: Test Client [Build 952](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43966&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency canceled: ... Test Lambda (new)) Build time: 00:00:00


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 980](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43965&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency failed: ... Lint Server (new)) Build time: 00:00:01


Hey?


I think we should too!


This is bad code. You fail.


LGTM, I have test in win7


This is fun!


@gcgallo thanks for the example


Is this really true?


Hey! My fork is actually not deprecated -- if you think these changes could be helpful to other users, would you mind opening an issue/PR?

Thanks!


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 981](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43969&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency failed: ... Lint Server (new)) Build time: 00:00:00


TeamCity Platform Services :: Environment Manager :: Test Client [Build 953](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43970&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency canceled: ... Test Lambda (new)) Build time: 00:00:00


project description


两处病句修改


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 982](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43957&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:22


It wasn't missing, but it was in the wrong place -- I think I had an out of date checkout, so I added the changelog under 1.3.3, which still said "(unreleased)" on my copy. Then @gforcada merged it to master and it ended up getting added to the changelog for the already-released 1.3.3 :(

Sorry for the confusion and extra work.


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 983](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43961&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:24


You forgot to remove comment.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-14: OK (1782 of 1782 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)


Nicer names than numbers would be good here as well.


Typically it is best to design a component in isolation. In other words, whether or not this component checks for null should not be dependent on the other components that currently use it. This assumes that we'll only ever use the method the way it is currently used. In addition, I don't think these guard clauses hurt anything, even if they will never throw.

I will typically check for null if the method doesn't immediately use the argument. In other words, if this method immediately accessed the null argument then that would throw an exception, but since we save it, the exception don't be thrown until someone else accesses the variable, thereby losing the original source of the null value.


@pva701 I suppose it's ok, but please take a look.


@MilhouseVH @sraue fyi, in case you still ship pvr client addons in repositories, this was broken for at least half an year (or maybe more, idk). ref https://github.com/xbmc/xbmc/pull/7783


TeamCity Platform Services :: Environment Manager :: Test Client [Build 954](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43958&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:38


In response to the indentation issues: I discovered hadn't enabled the "replace tabs with spaces" option in Notepad++ when making these changes.  GitHub looks like it uses a tab width of 8 while my editor uses 4, so the issue wasn't obvious while I was editing.  I fixed that in some of the files I changed, but evidently not all.  Also, the changes have already been merged to the experimental branch, so I don't think it's worthwhile for me to fix in my fork.  I'll let Blood Asp decide what to do about them when he gets to doing the merge/rebase/whatever to transfer the changes for the experimental branch to the unstable branch.


It seems that the `dist/` directory has not been rebuilt and committed for release v2.2.10.


Maybe the nut of the issue is that rather than paper over one's lack of understanding of JavaScript's scoping rules by `=>`'ing everywhere, take the half day required to learn how the language works first.


This accessor goes against the intended use of the `Cstring`. In fact, I designated this class to be solely used as arguments for the `log` functions, not to keep them as member variables. (I wish there was a way to limit C++ types to be used as temporaries only, or is there a way?)

Could you please reconsider your use of `Cstring` or explain the problem that the `Cstring` solves for you and which cannot be solved easily without it?


Put another way, you could argue that if `=>` helps you write less buggy code, then pause and consider learning scoping more rigorously first. If it helps you write more terse code and you like the style, go nuts


What makes you say that?

https://raw.githubusercontent.com/swagger-api/swagger-ui/master/dist/swagger-ui.js


Hm, we're changing this from a sliding time window (22 months before now) to a fixed time window.

So before, we always kept the last 22 months of stuff, basically.

Now, we will keep 22 months of stuff when the date is 22 months ago. Then a month later, we keep 23 months of stuff. Then a month after that, we keep 24 months of stuff, and so on.


Well, probably not ok.


some files in `dist/` and even in `src/` are dirty after `npm run build`.


You have a stale libmono-profile-aot.so somewhere, we don't use those dirs anymore.


TeamCity Platform Services :: Environment Manager :: Test Client [Build 955](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43962&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:06:31


Lol, just figured out i need a `not` somewhere


it'd be great if you were more specific.  The `swagger-ui.js` is really where everything happens.


@lkzhao Hellow Ikzaho. I think 'guard' have to be used to prevent against error throw or fatal error. then 'if' or 'if let' pattern is batter suited. How think about this ?


Isn't that the whole point of this post, though? The three topics critical for truly understanding JS are coercion, closures and `this`? So the thesis of this post is that people _should_ be diving deeper and that the "just use `=>`" rhetoric is harmful.


Adding a test comment


When I say "dirty" I mean that git flags them as modified.


Er, realloc ... ?


Mea culpa, I was really just responding to the comments not looking at the prose closely.


Is there a blog post or some note that provides background on the move from OwnCloud to NextCloud?


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 984](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43983&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:13


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-14: OK (1782 of 1782 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)


Thanks, we'll look into this and make any necessary changes!


which files?  What does git diff say?  Again, I'd like to help, but this isn't enough info.


So here, unlike in other code I've recently looked at, we're using the description to determine the format of the view, which seems reasonable - should we be doing that elsewhere?

Also, do we know that UNORM is the way to go here, or is SRGB sometimes the answer? Do we need to establish a convention? (For instance I'm pretty sure Oculus assumes you're handing over SRGB textures)  I'm not fully versed in d3d11 and color space interactions, or how mapping occurs (what happens when you have a wide-gamut HMD, beyond just SRGB, at some point? Adobe RGB isn't that unusual, though full coverage of Adobe RGB is.)


Yes, I am doing in a virtualenv, and I run ```python setup.py install``` afterwards. The problem is that I can import pymei in a python console, but I can not import it in my script.


I suspect you intended to unset the ENV after running? Otherwise you could use `before` instead.


Nice addition, but this should be on new minor like 2.2, not a patch according to semver.


我日，真的可以了


How to reproduce:
```
git checkout v2.2.10
rm -Rf node_modules
yarn  # or npm install
git status -s -uno
```
Result:
```
 M dist/css/print.css
 M dist/css/reset.css
 M dist/css/screen.css
 M dist/index.html
 M dist/swagger-ui.js
 M dist/swagger-ui.min.js
 M src/main/template/templates.js
```


@misferkhanPL You may want to have a look at http://docs.sonarqube.org/display/PLUG/Custom+Rules+for+PHP


@carito22 hay un filtro para esto - formatTime:'timeSchedule'?


Nit: `that I was most involved in` or `that I was most involved with`, or if you want to be very formal, `in which I was most involved`


And `git diff` gives???  For all I know, you're using windows and we have a `^M` issue.


Would be curious to find out what a `renderTextureResourceView->GetDesc()` call when passing nullptr would return - that would presumably tell us the difference. https://msdn.microsoft.com/en-us/library/windows/desktop/ff476629(v=vs.85).aspx


Thank you! :)


las líneas deben tener 80 caracteres de longitud o menos - puedes dividir esto en varias líneas para que sea legible?


@MilhouseVH if your pvr clients are part of the build, there is nothing to change, everything should work fine. this affects only pvr clients in repositories, once pvr client is updated, it may be automaticaly disabled because it failed to load. so the user has to restart kodi and re-enable the addon manualy!


看着还不错嘛


The animation range and time step are still hard-coded with values appropriate for the original KML demo file. We'll need to read these values dynamically from KML files at load time.


Hm, what about if we passed a time string as an argument to `call` instead? Would simplify the API and this test. And we could pass the ENV in `scheduler.rake` instead.


`respuesta` te parece buen nombre de variable?


ok


@lkzhao Hellow Ikzhao. I think 'guard' have to be used to prevent 'fatal error'. then 'if' or 'if let' pattern is batter suited. How think about this?


woops


TeamCity Platform Services :: Environment Manager :: Test Lambda [Build 985](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43987&buildTypeId=PlatformServices_EnvironmentManager_RunLambaTests) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:01:26


In summary, the outstanding comments were:
- improve performance by not checking more items in the queue than needed (stop if you find a request that is within max_queue_time, because all the items behind that are younger and thus also within max_queue_time)
- guarantee that items > max_queue_time don't get processed (check the condition whenever the server is picking an item to give to an available thread, don't rely on the patrol thread -- it can run at a more relaxed interval, it's mainly useful to evict items when the server is 100% stuck)


OK, since we didn't received a test that will show different - I reverted this commit by accepting @eka7a commit.


It's particulates.


TeamCity Platform Services :: Environment Manager :: Test Client [Build 956](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=43984&buildTypeId=PlatformServices_EnvironmentManager_TestClient) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 00:05:47


I'm on Linux.


Yes, got that - I've mentioned it to those that support the add-ons and repository. Maybe something we need to get in for the next beta.


OK I can't help you if you can't paste the output of what's different.  Please open an issue with some details.

For the release, I rebuild everything, commit the `dist` folder.  Likely you're using a different version of some node module, installed globally.  But as I've been saying, you're not giving me enough info to do anything helpful.


why?


@tkuther why do you embed the minified version? Piwigo already does the minification with the {combine_css} and it would make it easier to study the original CSS and review changes when updating the CSS from Bootswatch :-)


The fact that we return `time` to return a falsy value confused me a little.

Here's an alternative idea – what do you think?

```
raw_time = ENV["REMOVE_EVENTS_OLDER_THAN"]
raw_time && Time.parse(raw_time)
```

Or more explicit:

```
raw_time = ENV["REMOVE_EVENTS_OLDER_THAN"]
raw_time ? Time.parse(raw_time) : nil
```


might be nice to try installing HDF5 with chocolatey, i.e.
```
- cinst hdf5
```


Hello,
can I ask you a word on why this is necessary?
Seems risky to just strip out all the `throw new Error()` lines!

Thank you for any clarification.


62、62行
```ruby
```ruby


Typo?