[inst/examples/globe/server.R:3:27:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L3) *style:* **Put spaces around all infix operators.**
```r
data(world.cities, package="maps")
                         ~^~
```
[inst/examples/globe/server.R:7:48:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L7) *style:* **Commas should always have a space after.**
```r
earth_dark   <- system.file("images/world.jpg",package="threejs")
                                               ^
```
[inst/examples/globe/server.R:7:55:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L7) *style:* **Put spaces around all infix operators.**
```r
earth_dark   <- system.file("images/world.jpg",package="threejs")
                                                     ~^~
```
[inst/examples/globe/server.R:8:42:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L8) *style:* **Commas should always have a space after.**
```r
moon    <- system.file("images/moon.jpg",package="threejs")
                                         ^
```
[inst/examples/globe/server.R:8:49:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L8) *style:* **Put spaces around all infix operators.**
```r
moon    <- system.file("images/moon.jpg",package="threejs")
                                               ~^~
```
[inst/examples/globe/server.R:9:42:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L9) *style:* **Commas should always have a space after.**
```r
mars    <- system.file("images/mars.jpg",package="threejs")
                                         ^
```
[inst/examples/globe/server.R:9:49:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L9) *style:* **Put spaces around all infix operators.**
```r
mars    <- system.file("images/mars.jpg",package="threejs")
                                               ~^~
```
[inst/examples/globe/server.R:10:45:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L10) *style:* **Commas should always have a space after.**
```r
jupiter <- system.file("images/jupiter.jpg",package="threejs")
                                            ^
```
[inst/examples/globe/server.R:10:52:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L10) *style:* **Put spaces around all infix operators.**
```r
jupiter <- system.file("images/jupiter.jpg",package="threejs")
                                                  ~^~
```
[inst/examples/globe/server.R:12:15:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Put spaces around all infix operators.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
             ~^~
```
[inst/examples/globe/server.R:12:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Put spaces around all infix operators.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                      ~^~
```
[inst/examples/globe/server.R:12:36:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Commas should always have a space after.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                   ^
```
[inst/examples/globe/server.R:12:45:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Put spaces around all infix operators.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                           ~^~
```
[inst/examples/globe/server.R:12:56:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Commas should always have a space after.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                                       ^
```
[inst/examples/globe/server.R:12:64:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Put spaces around all infix operators.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                                              ~^~
```
[inst/examples/globe/server.R:12:75:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Commas should always have a space after.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                                                          ^
```
[inst/examples/globe/server.R:12:85:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L12) *style:* **Put spaces around all infix operators.**
```r
    earth_dark=list(img=earth_dark,bodycolor="#0011ff",emissive="#0011ff",lightcolor="#99ddff"),
                                                                                ~^~
```
[inst/examples/globe/server.R:13:9:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
       ~^~
```
[inst/examples/globe/server.R:13:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
                ~^~
```
[inst/examples/globe/server.R:13:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Commas should always have a space after.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
                       ^
```
[inst/examples/globe/server.R:13:33:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
                               ~^~
```
[inst/examples/globe/server.R:13:53:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
                                                   ~^~
```
[inst/examples/globe/server.R:13:75:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    moon=list(img=moon,bodycolor="#555555", emissive="#444444", lightcolor="#555555"),
                                                                         ~^~
```
[inst/examples/globe/server.R:14:9:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
       ~^~
```
[inst/examples/globe/server.R:14:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
                ~^~
```
[inst/examples/globe/server.R:14:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Commas should always have a space after.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
                       ^
```
[inst/examples/globe/server.R:14:33:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
                               ~^~
```
[inst/examples/globe/server.R:14:53:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
                                                   ~^~
```
[inst/examples/globe/server.R:14:75:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    mars=list(img=mars,bodycolor="#aaaaaa", emissive="#000000", lightcolor="#aaaaaa"),
                                                                         ~^~
```
[inst/examples/globe/server.R:15:12:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Put spaces around all infix operators.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
          ~^~
```
[inst/examples/globe/server.R:15:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Put spaces around all infix operators.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
                   ~^~
```
[inst/examples/globe/server.R:15:30:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Commas should always have a space after.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
                             ^
```
[inst/examples/globe/server.R:15:39:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Put spaces around all infix operators.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
                                     ~^~
```
[inst/examples/globe/server.R:15:59:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Put spaces around all infix operators.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
                                                         ~^~
```
[inst/examples/globe/server.R:15:81:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L15) *style:* **Put spaces around all infix operators.**
```r
    jupiter=list(img=jupiter,bodycolor="#222222", emissive="#000000", lightcolor="#aaaaaa")
                                                                               ~^~
```
[inst/examples/globe/server.R:23:42:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L23) *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:23:52:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L23) *style:* **Put spaces around all infix operators.**
```r
     world.cities[order(world.cities$pop,decreasing=TRUE)[1:input$N],]
                                                  ~^~
```
[inst/examples/globe/server.R:23:70:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L23) *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:29:29:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L29) *style:* **Put spaces around all infix operators.**
```r
    col <- rainbow(10, start=2.8 / 6,end=3.4 / 6)
                           ~^~
```
[inst/examples/globe/server.R:29:38:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L29) *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:29:41:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L29) *style:* **Put spaces around all infix operators.**
```r
    col <- rainbow(10, start=2.8 / 6,end=3.4 / 6)
                                       ~^~
```
[inst/examples/globe/server.R:33:15:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L33) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
             ~^~
```
[inst/examples/globe/server.R:33:28:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L33) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
                          ~^~
```
[inst/examples/globe/server.R:33:40:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L33) *style:* **Put spaces around all infix operators.**
```r
    list(value=value, color=col, cities=cities)
                                      ~^~
```
[inst/examples/globe/server.R:40:32:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Commas should never have a space before.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                              ~^
```
[inst/examples/globe/server.R:40:42:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Put spaces around all infix operators.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                                        ~^~
```
[inst/examples/globe/server.R:40:61:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Put spaces around all infix operators.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                                                           ~^~
```
[inst/examples/globe/server.R:40:82:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Put spaces around all infix operators.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                                                                                ~^~
```
[inst/examples/globe/server.R:40:97:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Put spaces around all infix operators.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                                                                                ~^~
```
[inst/examples/globe/server.R:40:117:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L40) *style:* **Put spaces around all infix operators.**
```r
    args <- c(col[[input$map]] , list(lat=v$cities$lat, long=v$cities$long, value=v$value, color=v$color, atmosphere=atmo))
                                                                                ~^~
```
[inst/examples/globe/server.R:41:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/server.R#L41) *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/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/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/globe/ui.R:9:31:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/ui.R#L9) *style:* **Put spaces around all infix operators.**
```r
      selectInput("map", label="Surface map",
                             ~^~
```
[inst/examples/globe/ui.R:11:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/ui.R#L11) *style:* **Put spaces around all infix operators.**
```r
           list("Earth"="earth_dark",
                      ~^~
```
[inst/examples/globe/ui.R:12:23:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/ui.R#L12) *style:* **Put spaces around all infix operators.**
```r
                "Moon"="moon",
                     ~^~
```
[inst/examples/globe/ui.R:13:23:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/ui.R#L13) *style:* **Put spaces around all infix operators.**
```r
                "Mars"="mars",
                     ~^~
```
[inst/examples/globe/ui.R:14:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/globe/ui.R#L14) *style:* **Put spaces around all infix operators.**
```r
                "Jupiter"="jupiter"),
                        ~^~
```
[inst/examples/scatterplot/server.R:5:3:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L5) *style:* **Place a space before left parenthesis, except in a function call.**
```r
if(!exists("example_data")) example_data <- matrix(runif(50 * 3), ncol=3)
  ^
```
[inst/examples/scatterplot/server.R:5:71:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L5) *style:* **Put spaces around all infix operators.**
```r
if(!exists("example_data")) example_data <- matrix(runif(50 * 3), ncol=3)
                                                                     ~^~
```
[inst/examples/scatterplot/server.R:11:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L11) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(num.ticks == 0) num.ticks <- NULL
      ^
```
[inst/examples/scatterplot/server.R:12:37:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L12) *style:* **Commas should always have a space after.**
```r
    else num.ticks <- rep(num.ticks,3)
                                    ^
```
[inst/examples/scatterplot/server.R:13:40:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L13) *style:* **Commas should always have a space after.**
```r
    color <- rep(rainbow(input$colors),length.out=nrow(example_data))
                                       ^
```
[inst/examples/scatterplot/server.R:13:50:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L13) *style:* **Put spaces around all infix operators.**
```r
    color <- rep(rainbow(input$colors),length.out=nrow(example_data))
                                                ~^~
```
[inst/examples/scatterplot/server.R:14:57:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L14) *style:* **Put spaces around all infix operators.**
```r
    sizes <- rep(c(0.5, 1, 2)[1:input$sizes], length.out=nrow(example_data))
                                                       ~^~
```
[inst/examples/scatterplot/server.R:15:61:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L15) *style:* **Commas should always have a space after.**
```r
    labs <- sprintf("x=%.2f, y=%.2f, z=%.2f", example_data[,1], example_data[,2], example_data[,3])
                                                            ^
```
[inst/examples/scatterplot/server.R:15:79:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L15) *style:* **Commas should always have a space after.**
```r
    labs <- sprintf("x=%.2f, y=%.2f, z=%.2f", example_data[,1], example_data[,2], example_data[,3])
                                                                              ^
```
[inst/examples/scatterplot/server.R:15:97:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L15) *style:* **Commas should always have a space after.**
```r
    labs <- sprintf("x=%.2f, y=%.2f, z=%.2f", example_data[,1], example_data[,2], example_data[,3])
                                                                                ^
```
[inst/examples/scatterplot/server.R:16:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L16) *style:* **Put spaces around all infix operators.**
```r
    scatterplot3js(x=example_data,
                   ~^~
```
[inst/examples/scatterplot/server.R:17:30:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L17) *style:* **Put spaces around all infix operators.**
```r
                    num.ticks=num.ticks,
                            ~^~
```
[inst/examples/scatterplot/server.R:18:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L18) *style:* **Put spaces around all infix operators.**
```r
                    color=color,
                        ~^~
```
[inst/examples/scatterplot/server.R:19:25:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L19) *style:* **Put spaces around all infix operators.**
```r
                    size=sizes,
                       ~^~
```
[inst/examples/scatterplot/server.R:20:27:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L20) *style:* **Put spaces around all infix operators.**
```r
                    labels=labs,
                         ~^~
```
[inst/examples/scatterplot/server.R:21:33:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L21) *style:* **Put spaces around all infix operators.**
```r
                    label.margin="80px 10px 10px 10px",
                               ~^~
```
[inst/examples/scatterplot/server.R:22:29:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L22) *style:* **Put spaces around all infix operators.**
```r
                    renderer=input$renderer,
                           ~^~
```
[inst/examples/scatterplot/server.R:23:25:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/server.R#L23) *style:* **Put spaces around all infix operators.**
```r
                    grid=input$grid)
                       ~^~
```
[inst/examples/scatterplot/ui.R:13:36:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/ui.R#L13) *style:* **Put spaces around all infix operators.**
```r
      selectInput("renderer", label="Rendering method",
                                  ~^~
```
[inst/examples/scatterplot/ui.R:14:31:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/ui.R#L14) *style:* **Put spaces around all infix operators.**
```r
         choices = list("Auto"="auto", "Canvas"="canvas", "WebGL"="webgl"),
                             ~^~
```
[inst/examples/scatterplot/ui.R:14:48:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/ui.R#L14) *style:* **Put spaces around all infix operators.**
```r
         choices = list("Auto"="auto", "Canvas"="canvas", "WebGL"="webgl"),
                                              ~^~
```
[inst/examples/scatterplot/ui.R:14:66:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/inst/examples/scatterplot/ui.R#L14) *style:* **Put spaces around all infix operators.**
```r
         choices = list("Auto"="auto", "Canvas"="canvas", "WebGL"="webgl"),
                                                                ~^~
```
[R/globe.R:162:47:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L162) *style:* **Put spaces around all infix operators.**
```r
  img=system.file("images/world.jpg",  package="threejs"),
                                             ~^~
```
[R/globe.R:176:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(lat) || missing(long))
    ^
```
[R/globe.R:183:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L183) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/globe.R:186:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L186) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(j))
      ^
```
[R/globe.R:192:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L192) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/globe.R:195:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L195) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(j))
      ^
```
[R/globe.R:201:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L201) *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:202:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L202) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(arcs))
    ^
```
[R/globe.R:212:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L212) *style:* **Put spaces around all infix operators.**
```r
  options = list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                   ~^~
```
[R/globe.R:212:31:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L212) *style:* **Put spaces around all infix operators.**
```r
  options = list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                             ~^~
```
[R/globe.R:212:43:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L212) *style:* **Put spaces around all infix operators.**
```r
  options = list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                                         ~^~
```
[R/globe.R:212:62:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L212) *style:* **Put spaces around all infix operators.**
```r
  options = list(lat=lat, long=long, color=color, arcsOpacity=arcsOpacity,
                                                            ~^~
```
[R/globe.R:213:23:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L213) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                     ~^~
```
[R/globe.R:213:41:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L213) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                       ~^~
```
[R/globe.R:213:56:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L213) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                                      ~^~
```
[R/globe.R:213:65:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L213) *style:* **Put spaces around all infix operators.**
```r
                 value=value, atmosphere=atmosphere, bg=bg, arcs=arcs,
                                                               ~^~
```
[R/globe.R:214:27:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L214) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                         ~^~
```
[R/globe.R:214:46:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L214) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                                            ~^~
```
[R/globe.R:214:66:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L214) *style:* **Put spaces around all infix operators.**
```r
                 arcsColor=arcsColor, arcsLwd=arcsLwd, arcsHeight=arcsHeight)
                                                                ~^~
```
[R/globe.R:216:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L216) *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:218:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L218) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("bodycolor" %in% names(options))
    ^
```
[R/globe.R:220:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L220) *style:* **Commas should always have a space after.**
```r
    i = grep("^#",options$bodycolor)
                  ^
```
[R/globe.R:221:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L221) *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:221:68:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L221) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$bodycolor = substr(options$bodycolor,1,7)
                                                                   ^
```
[R/globe.R:221:70:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L221) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$bodycolor = substr(options$bodycolor,1,7)
                                                                     ^
```
[R/globe.R:223:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L223) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("emissive" %in% names(options))
    ^
```
[R/globe.R:225:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L225) *style:* **Commas should always have a space after.**
```r
    i = grep("^#",options$emissive)
                  ^
```
[R/globe.R:226:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L226) *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:226:66:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L226) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$emissive = substr(options$emissive,1,7)
                                                                 ^
```
[R/globe.R:226:68:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L226) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$emissive = substr(options$emissive,1,7)
                                                                   ^
```
[R/globe.R:228:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L228) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("lightcolor" %in% names(options))
    ^
```
[R/globe.R:230:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L230) *style:* **Commas should always have a space after.**
```r
    i = grep("^#",options$lightcolor)
                  ^
```
[R/globe.R:231:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L231) *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:231:70:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L231) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$lightcolor = substr(options$lightcolor,1,7)
                                                                     ^
```
[R/globe.R:231:72:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L231) *style:* **Commas should always have a space after.**
```r
    if(length(i) > 0) options$lightcolor = substr(options$lightcolor,1,7)
                                                                       ^
```
[R/globe.R:235:28:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L235) *warning:* **no visible global function definition for ‘texture’**
```r
  if (!is.list(img)) img = texture(img)
                           ^~~~~~~
```
[R/globe.R:259:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L259) *warning:* **no visible global function definition for ‘shinyRenderWidget’**
```r
    shinyRenderWidget(expr, globeOutput, env, quoted = TRUE)
    ^~~~~~~~~~~~~~~~~
```
[R/globe.R:259:29:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/globe.R#L259) *warning:* **no visible binding for global variable ‘globeOutput’**
```r
    shinyRenderWidget(expr, globeOutput, env, quoted = TRUE)
                            ^~~~~~~~~~~
```
[R/graph.R:68:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L68) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.data.frame(edges))
    ^
```
[R/graph.R:70:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L70) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(edges$size)) edges$size = 1
    ^
```
[R/graph.R:71:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L71) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(edges$color)) edges$color = "lightgray"
    ^
```
[R/graph.R:72:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L72) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!all(c("from", "size", "to", "color") %in% names(edges)))
    ^
```
[R/graph.R:76:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L76) *style:* **Put spaces around all infix operators.**
```r
    nodes = data.frame(id=unique(c(edges$from, edges$to)), size=1, color="orange", stringsAsFactors=FALSE)
                        ~^~
```
[R/graph.R:76:64:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L76) *style:* **Put spaces around all infix operators.**
```r
    nodes = data.frame(id=unique(c(edges$from, edges$to)), size=1, color="orange", stringsAsFactors=FALSE)
                                                              ~^~
```
[R/graph.R:76:73:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L76) *style:* **Put spaces around all infix operators.**
```r
    nodes = data.frame(id=unique(c(edges$from, edges$to)), size=1, color="orange", stringsAsFactors=FALSE)
                                                                       ~^~
```
[R/graph.R:76:100:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L76) *style:* **Put spaces around all infix operators.**
```r
    nodes = data.frame(id=unique(c(edges$from, edges$to)), size=1, color="orange", stringsAsFactors=FALSE)
                                                                                ~^~
```
[R/graph.R:79:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L79) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(nodes$label)) nodes$label = nodes$id
    ^
```
[R/graph.R:80:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L80) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(nodes$size)) nodes$size = 1
    ^
```
[R/graph.R:81:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L81) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(nodes$color)) nodes$color = "orange"
    ^
```
[R/graph.R:82:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L82) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!all(c("id", "size", "label", "color") %in% names(nodes)))
    ^
```
[R/graph.R:85:47:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L85) *style:* **Put spaces around all infix operators.**
```r
  stroke = switch(as.character(stroke), "TRUE"="black", "white")
                                             ~^~
```
[R/graph.R:88:17:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L88) *style:* **Put spaces around all infix operators.**
```r
  x = list(nodes=nodes,
               ~^~
```
[R/graph.R:89:17:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L89) *style:* **Put spaces around all infix operators.**
```r
           edges=edges,
               ~^~
```
[R/graph.R:90:16:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L90) *style:* **Put spaces around all infix operators.**
```r
           main=main,
              ~^~
```
[R/graph.R:91:14:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L91) *style:* **Put spaces around all infix operators.**
```r
           bg=bg,
            ~^~
```
[R/graph.R:92:14:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L92) *style:* **Put spaces around all infix operators.**
```r
           fg=fg,
            ~^~
```
[R/graph.R:93:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L93) *style:* **Put spaces around all infix operators.**
```r
           showLabels=showLabels,
                    ~^~
```
[R/graph.R:94:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L94) *style:* **Put spaces around all infix operators.**
```r
           attraction=attraction,
                    ~^~
```
[R/graph.R:95:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L95) *style:* **Put spaces around all infix operators.**
```r
           repulsion=repulsion,
                   ~^~
```
[R/graph.R:96:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L96) *style:* **Put spaces around all infix operators.**
```r
           iterations=max_iterations,
                    ~^~
```
[R/graph.R:97:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L97) *style:* **Put spaces around all infix operators.**
```r
           curvature=curvature,
                   ~^~
```
[R/graph.R:98:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L98) *style:* **Put spaces around all infix operators.**
```r
           opacity=opacity,
                 ~^~
```
[R/graph.R:99:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L99) *style:* **Put spaces around all infix operators.**
```r
           stroke=stroke)
                ~^~
```
[R/graph.R:102:35:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L102) *style:* **Put spaces around all infix operators.**
```r
          x = toJSON(x, auto_unbox=TRUE),
                                 ~^~
```
[R/graph.R:123:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L123) *warning:* **no visible global function definition for ‘shinyRenderWidget’**
```r
    shinyRenderWidget(expr, graphOutput, env, quoted = TRUE)
    ^~~~~~~~~~~~~~~~~
```
[R/graph.R:123:29:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L123) *warning:* **no visible binding for global variable ‘graphOutput’**
```r
    shinyRenderWidget(expr, graphOutput, env, quoted = TRUE)
                            ^~~~~~~~~~~
```
[R/graph.R:149:1:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L149) *style:* **lines should not be more than 130 characters.**
```r
  if (!is.data.frame(edges) || !(c("from", "to") %in% names(edges))) stop("edges must be a data frame with 'from' and 'to' columns")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/graph.R:152:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L152) *style:* **Put spaces around all infix operators.**
```r
    nodes = data.frame(id=unique(c(edges$from, edges$to)))
                        ~^~
```
[R/graph.R:157:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L157) *style:* **Commas should always have a space after.**
```r
    idx = edges[,"from"] > edges[,"to"]
                 ^
```
[R/graph.R:157:35:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L157) *style:* **Commas should always have a space after.**
```r
    idx = edges[,"from"] > edges[,"to"]
                                  ^
```
[R/graph.R:158:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L158) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(idx))
      ^
```
[R/graph.R:169:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L169) *style:* **Commas should always have a space after.**
```r
  names(id) = nodes[,"id"]
                     ^
```
[R/graph.R:170:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *warning:* **no visible global function definition for ‘sparseMatrix’**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
      ^~~~~~~~~~~~
```
[R/graph.R:170:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
                   ~^~
```
[R/graph.R:170:53:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
                                                   ~^~
```
[R/graph.R:170:83:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
                                                                                ~^~
```
[R/graph.R:170:91:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
                                                                                ~^~
```
[R/graph.R:170:110:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L170) *style:* **Put spaces around all infix operators.**
```r
  M = sparseMatrix(i=id[as.character(edges$from)], j=id[as.character(edges$to)], x=x, dims=c(N, N), symmetric=symmetric)
                                                                                ~^~
```
[R/graph.R:171:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L171) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.null(nodes$label)) colnames(M) = rownames(M) = nodes$label
    ^
```
[R/graph.R:189:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L189) *warning:* **no visible global function definition for ‘Matrix’, Did you mean 'matrix'?**
```r
  M = Matrix(M)
      ^~~~~~
```
[R/graph.R:190:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L190) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!any(grepl("CMatrix", class(M)))) stop("M must be a matrix or CsparseMatrix object")
    ^
```
[R/graph.R:193:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L193) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if("x" %in% slotNames(M)) size = as.numeric(M@x)
    ^
```
[R/graph.R:193:15:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L193) *warning:* **no visible global function definition for ‘slotNames’, Did you mean 'setNames'?**
```r
  if("x" %in% slotNames(M)) size = as.numeric(M@x)
              ^~~~~~~~~
```
[R/graph.R:194:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L194) *style:* **Put spaces around all infix operators.**
```r
  nodes = data.frame(id=1:n, label=1:n, size=1, color="orange")
                      ~^~
```
[R/graph.R:194:35:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L194) *style:* **Put spaces around all infix operators.**
```r
  nodes = data.frame(id=1:n, label=1:n, size=1, color="orange")
                                 ~^~
```
[R/graph.R:194:45:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L194) *style:* **Put spaces around all infix operators.**
```r
  nodes = data.frame(id=1:n, label=1:n, size=1, color="orange")
                                           ~^~
```
[R/graph.R:194:54:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L194) *style:* **Put spaces around all infix operators.**
```r
  nodes = data.frame(id=1:n, label=1:n, size=1, color="orange")
                                                    ~^~
```
[R/graph.R:195:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L195) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.null(colnames(M))) nodes$label = colnames(M)
    ^
```
[R/graph.R:197:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L197) *style:* **Put spaces around all infix operators.**
```r
  edges = data.frame(from=M@i + 1, to=rep(seq_along(dp), dp), size=size, color="lightgray")
                        ~^~
```
[R/graph.R:197:38:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L197) *style:* **Put spaces around all infix operators.**
```r
  edges = data.frame(from=M@i + 1, to=rep(seq_along(dp), dp), size=size, color="lightgray")
                                    ~^~
```
[R/graph.R:197:67:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L197) *style:* **Put spaces around all infix operators.**
```r
  edges = data.frame(from=M@i + 1, to=rep(seq_along(dp), dp), size=size, color="lightgray")
                                                                 ~^~
```
[R/graph.R:197:79:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L197) *style:* **Put spaces around all infix operators.**
```r
  edges = data.frame(from=M@i + 1, to=rep(seq_along(dp), dp), size=size, color="lightgray")
                                                                             ~^~
```
[R/graph.R:198:13:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L198) *style:* **Put spaces around all infix operators.**
```r
  list(nodes=nodes, edges=edges)
           ~^~
```
[R/graph.R:198:26:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/graph.R#L198) *style:* **Put spaces around all infix operators.**
```r
  list(nodes=nodes, edges=edges)
                        ~^~
```
[R/scatterplot.R:148:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L148) *style:* **Commas should always have a space after.**
```r
  num.ticks = c(6,6,6),
                  ^
```
[R/scatterplot.R:148:21:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L148) *style:* **Commas should always have a space after.**
```r
  num.ticks = c(6,6,6),
                    ^
```
[R/scatterplot.R:159:23:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L159) *style:* **Commas should always have a space after.**
```r
  renderer = c("auto","canvas","webgl"),
                      ^
```
[R/scatterplot.R:159:32:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L159) *style:* **Commas should always have a space after.**
```r
  renderer = c("auto","canvas","webgl"),
                               ^
```
[R/scatterplot.R:165:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L165) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!missing(y) && !missing(z))
    ^
```
[R/scatterplot.R:167:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L167) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.matrix(x))
      ^
```
[R/scatterplot.R:169:16:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L169) *style:* **Put spaces around all infix operators.**
```r
    x = cbind(x=x,y=y,z=z)
              ~^~
```
[R/scatterplot.R:169:19:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L169) *style:* **Commas should always have a space after.**
```r
    x = cbind(x=x,y=y,z=z)
                  ^
```
[R/scatterplot.R:169:20:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L169) *style:* **Put spaces around all infix operators.**
```r
    x = cbind(x=x,y=y,z=z)
                  ~^~
```
[R/scatterplot.R:169:23:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L169) *style:* **Commas should always have a space after.**
```r
    x = cbind(x=x,y=y,z=z)
                      ^
```
[R/scatterplot.R:169:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L169) *style:* **Put spaces around all infix operators.**
```r
    x = cbind(x=x,y=y,z=z)
                      ~^~
```
[R/scatterplot.R:171:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L171) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(ncol(x) != 3) stop("x must be a three column matrix")
    ^
```
[R/scatterplot.R:172:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L172) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.data.frame(x)) x = as.matrix(x)
    ^
```
[R/scatterplot.R:173:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L173) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.matrix(x)) stop("x must be a three column matrix")
    ^
```
[R/scatterplot.R:175:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L175) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(pch)) pch = NULL
    ^
```
[R/scatterplot.R:176:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(missing(renderer) && nrow(x) > 10000)
    ^
```
[R/scatterplot.R:186:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L186) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/scatterplot.R:189:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L189) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(any(j))
      ^
```
[R/scatterplot.R:191:40:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L191) *style:* **Commas should always have a space after.**
```r
      color[i][j] = substr(color[i][j],1,7)
                                       ^
```
[R/scatterplot.R:191:42:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L191) *style:* **Commas should always have a space after.**
```r
      color[i][j] = substr(color[i][j],1,7)
                                         ^
```
[R/scatterplot.R:194:17:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L194) *style:* **Commas should always have a space after.**
```r
  i = grep("^#",bg)
                ^
```
[R/scatterplot.R:195:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L195) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0)
    ^
```
[R/scatterplot.R:197:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L197) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(nchar(bg) > 7)
      ^
```
[R/scatterplot.R:199:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L199) *style:* **Commas should always have a space after.**
```r
      bg = substr(bg,1,7)
                     ^
```
[R/scatterplot.R:199:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L199) *style:* **Commas should always have a space after.**
```r
      bg = substr(bg,1,7)
                       ^
```
[R/scatterplot.R:205:49:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L205) *style:* **Commas should always have a space after.**
```r
  options = options[!(names(options) %in% c("x","y","z","i","j"))]
                                                ^
```
[R/scatterplot.R:205:53:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L205) *style:* **Commas should always have a space after.**
```r
  options = options[!(names(options) %in% c("x","y","z","i","j"))]
                                                    ^
```
[R/scatterplot.R:205:57:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L205) *style:* **Commas should always have a space after.**
```r
  options = options[!(names(options) %in% c("x","y","z","i","j"))]
                                                        ^
```
[R/scatterplot.R:205:61:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L205) *style:* **Commas should always have a space after.**
```r
  options = options[!(names(options) %in% c("x","y","z","i","j"))]
                                                            ^
```
[R/scatterplot.R:207:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L207) *style:* **Commas should always have a space after.**
```r
  i = grep("\\.",names(options))
                 ^
```
[R/scatterplot.R:208:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L208) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(i) > 0) names(options)[i] = gsub("\\.", "", names(options)[i])
    ^
```
[R/scatterplot.R:211:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L211) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(length(colnames(x)) == 3 && is.null(options$axisLabels)) options$axisLabels = colnames(x)
    ^
```
[R/scatterplot.R:214:22:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L214) *style:* **Commas should always have a space after.**
```r
  x = matrix(x[, c(1,3,2)], ncol=3)
                     ^
```
[R/scatterplot.R:214:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L214) *style:* **Commas should always have a space after.**
```r
  x = matrix(x[, c(1,3,2)], ncol=3)
                       ^
```
[R/scatterplot.R:214:33:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L214) *style:* **Put spaces around all infix operators.**
```r
  x = matrix(x[, c(1,3,2)], ncol=3)
                               ~^~
```
[R/scatterplot.R:221:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L221) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!missing(xlim) && length(xlim) == 2)
    ^
```
[R/scatterplot.R:226:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L226) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!missing(ylim) && length(ylim) == 2)
    ^
```
[R/scatterplot.R:231:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L231) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!missing(zlim) && length(zlim) == 2)
    ^
```
[R/scatterplot.R:238:24:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L238) *style:* **Put spaces around all infix operators.**
```r
  x = (x - rep(mn, each=n)) / (rep(mx - mn, each=n))
                      ~^~
```
[R/scatterplot.R:238:49:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L238) *style:* **Put spaces around all infix operators.**
```r
  x = (x - rep(mn, each=n)) / (rep(mx - mn, each=n))
                                               ~^~
```
[R/scatterplot.R:239:18:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L239) *style:* **Commas should always have a space after.**
```r
  if (flip.y) x[,3] = 1 - x[,3]
                 ^
```
[R/scatterplot.R:239:30:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L239) *style:* **Commas should always have a space after.**
```r
  if (flip.y) x[,3] = 1 - x[,3]
                             ^
```
[R/scatterplot.R:245:28:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L245) *style:* **Commas should always have a space after.**
```r
  x = as.vector(t(signif(x,signif)))
                           ^
```
[R/scatterplot.R:248:5:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L248) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(!is.null(num.ticks))
    ^
```
[R/scatterplot.R:250:7:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L250) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(length(num.ticks) != 3) stop("num.ticks must have length 3")
      ^
```
[R/scatterplot.R:251:31:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L251) *style:* **Commas should always have a space after.**
```r
    num.ticks = num.ticks[c(1,3,2)]
                              ^
```
[R/scatterplot.R:251:33:](https://github.com/bwlewis/rthreejs/blob/8c2b47f29cf39b1be01ccd14db0eb6f5309298ad/R/scatterplot.R#L251) *style:* **Commas should always have a space after.**
```r
    num.ticks = num.ticks[c(1,3,2)]
                                ^
```


I'm honestly really not sure? Robert seemed to think None was sufficient for both ComputationStrategyRemoved and FieldNameUpdated.


ACHTUNG!
Das Modell wurde geändert.
Es gibt nun keine Unterscheidung der Course nach "Lecture" und "ExerciseCourse" mehr. Stattdessen gibt es ein Attribut in der Klasse Course: ECourseType mit den Werten "LECTURE" und "EXERCISE". Grund ist die Persistierung: Das Laden von Kursen ist mit diesem Modell viel leichter.


nice.


Please is it possible to follow semver, Should be a 0.4.0.


is drawRadius centered around 0,1 though?


looking good.


Убрать `console.log`


я просто пью вино самодельное. поэтому сейчас я могу рассуждать только на таких вот уровнях =)


yes, all culling uses drawMidPos...


Ack


why use regexes for simple string replace?


AFAIK the order in enumerate tuples is the other way around: (index, element)


same here as above


Seems like using snapshots is a reasonable idea, but the initialization needs to use any present `:data` option, as this is how you get a graph projection (and generally initialize a Graph as part of a Repository). It should perhaps either raise an exception if `:data` isn't a `Hamster::Hash` or cast it to a `Hamster::Hash`


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


drawmidpos isn't (0,1) afaik, it's midpos (which is decided by other stuff) extrapolated for drawing frames between gameframes


Read where? ..


Doesn't seem this is actually testing the graph; just the list itself. Not sure why the original test doesn't work, as `g` should have just a single subject, so `g.subjects.first` should be the same as `ten.subject`.


Is the promotion still needed now that _map is declared immutable?


drawMidPos = drawPos + relMidPos, relMidPos defaults to (maxs + mins)*0.5


.updated_at should definitely be set in the `save` method (Not `Model.update`), if and only if the model has the column. Otherwise looks :+1:


You win.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.94](https://pixelated.design/build/viewLog.html?buildId=324&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.90](https://pixelated.design/build/viewLog.html?buildId=325&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 267 Build time: 00:00:17


Juste, pourquoi passes-tu toutes les variables de sorties en pointeurs ?


I would rather perfrom the `stable_partiontion` and then erase to group of iterators. i.e

`iter = std::stable_partition(vector.begin(), vector.end(), Predicate)`
`std::remove(vector.erase(iter, vector.end))`


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.37](https://pixelated.design/build/viewLog.html?buildId=326&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


Should this change be reflected in the corresponding SIP file?

I have the same issue in my PR #2491, so I am looking forward to your answer.


the default is irrelevant since devs can change it (since midpos is used for lots of other stuff in the engine)


missing || die


Sorry. The .gitignore file was sitting unsaved in Notepad++ :stuck_out_tongue_closed_eyes:


1) midpos != relmidpos
2) relmidpos can not be changed


Thanks Jow, all suggestions have been added. I could make things much cleaner if the model saved each user as a section Ideally I think there needs to be a users overview page that list
all non-root users. Within the users overview there should be a button
option to add a new user, and each existing user should have 2 button
options, "edit" and "remove". The add and edit button should redirect to
page for creating/editing  the user and the delete button  should remove
the user from the config and  the system.  It would also be extremely
helpful if the model saved each user as a section ... eg config user
'frank", this would allow each users options to be accessed by username ..
uci:get(users, username, option).

 I will admit, creating the interface is something I struggle with. If you
could provide an example of how to redirect to a separate model  to
add/edit the user... I would imagine for the edit user you would need to
provide the username as an argument  ?  I am not sure how to go about that
…


`g` has 20 statements and 10 subjects.

The test checks that building a new list from the graph yields a well structured list. It does, except that in the red line there's only a 1 in 10 chance that the created list has the correct head node.

The behavior we see in the failing spec is that the `RDF::List` created by `.new` is a sub-list of the one in Graph, beginning at some random point and containing the remainder.


5 volt beak to beak


did you read the code?
https://github.com/spring/spring/blob/61559fa24cb0dd1cb16f9be4b29d9fa6a0132126/rts/Rendering/UnitDrawer.cpp#L1442


added all suggestions and removed admin group ...

https://github.com/Hostle/luci/commit/ff01098f0aaaa210cd0e864ba039a63af0ba6b5a
https://github.com/Hostle/luci/commit/e6a854ccb08fb24eb8e47cbccbeb07f1846fb301


@eht16 @codebrainz Sorry, haven't checked since which version the function is available. Actually I could have used plain g_hash_table_lookup () - will fix.


More accurately, you __won__, like 6 months ago.


Yep. I'll come back to this.

I was a bit thrown off because my original commit + test run yielded fewer failing tests. I decided that was an improvement and pushed, but it turns out it was just random, since most of the order-dependent tests have about a 50/50 chance of failing.


C'est à moitié fini et ça casse les tests, mais comme je ne sais pas quand je prendrai le temps de finir, je publie ça quand même.


(u->midPos - u->pos) is *the same as* relMidPos.

When midPos is changed, relMidPos changes with it.


but drawRadius doesn't!


Closes #15609
Closes #16267


Maybe could travis be enabled for the plugins too to prevent guys like me doing stupid things?


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


Sure, of course, each list element has it's own subject :8ball: Maybe a query for subjects which aren't the object of any statements, unless we still emit `?lh rdf:type rdf:List .`


### ngRoute
Este va a manejar todo el ruteo en el browser. Esto quiere decir que en vez de estar matcheando las páginas desde el servidor, AngularJS va a cargar el template definido, compilarlo y ubicarlo dentro de un elemento específico del DOM. El servidor solo va a servir el template de forma estática pero no va a responder a los cambios en la URL. Esto convierte nuestro server Express en algo más parecido a una API.
Este módulo tiene dos modos URL: el modo *legacy* usando una parte hash para soportar exploradores antiguos y un modo HTML5 usando la API "Historia" soportada por los exploradores recientes.

El comportamiento por default de este módulo es usar la parte hash de la URL para el ruteo. Como se usa casi siempre para linkeos dentro de la página, cuando la parte hash cambia el browser no realizará una petición al server. Esto le permite a AngularJS soportar viejos browsers manteniendo un esquema de ruteo decente. Por lo tanto una ruta angular común sería **http://localhost:3000/#/example**.
Peeero, las aplicaciones single-page tienen un mayor problema. No son indexeables por motores de búsquedas y sufren de un pobre *SEO*. Para resolver este problema la mayoría de estos motores ofrecen a los desarrolladores marcar sus aplicaciones como single-page. De esta forma los mismos conocen este tipo de aplicaciones usando **AJAX** para renderizar nuevos *paths* y esperar por el resultado antes de que dejes la página. Para marcar las aplicaciones se necesita de un esquema de ruteo llamado **Hashbangs**. Estos son implementados usando un signo de admiración ` ! `. Un ejemplo sería: `http://localhost:3000/#!/example`. AngularJS soporta este esquema usando un bloque de configuración de módulo y el servicio `$locationProvider` de AngularJS.

Y hablando de servicios, AngularJS viene cargado de estos para abstraer las tareas de desarrollo más comunes. Los servicios más utilizados son:

- **$http**: Para manejar requests AJAX
- **$resource**: Para manejar APIs RESTful
- **$location**: Para manejar manipulaciones de URL.
- **$q**: Para manejar *promises*
- **$rootScope**: Retorna el objeto *scope* raiz.
- **$window**: Retorna el objeto *window*.

Por otro lado la creación de servicios custom es vital para una aplicación AngularJS (brinda una mejor testeabilidad por ejemplo). Para crearlos se realiza mediante uno de los tres métodos: `provider()`, `service()` y `factory()`.

- **provider()**: La forma más completa.
- **service()**: Para instanciar un nuevo objeto singleton. Debe usarse cuando definimos un servicio como *prototype*
- **factory()**: Para proveer el valor que retorna de la función invocada por el servicio. Debe usarse cuando queremos compartir objetos y data a través de toda la app


So what? Changing midPos before this commit would also mess up the culling.


Ah, I didn't say it's this commit's fault... :)


> In addition it doesn't scale

Of course, I agree on this. Mentioned it on the [issue's comment](http://jira.xwiki.org/browse/XWIKI-4906?focusedCommentId=89607&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-89607) as well.
This is just a first step in improving things and setting up what is needed for the generic solution.

> Also, in order to remove the spaghetti feeling

https://github.com/xwiki/xwiki-platform/commit/9e8f69b8398cddb0eaf987457ad2a9517e8d658a#diff-8601ad4357df8c1614efbeb593ca4237, one of the following commits, improves things slightly, but there is still the annoying problem of copy/pasted code due to the fact that oldcore's rename() method does not reuse refactoring's jobs (in synchronized mode). As far as I can see, the only impediment would be around [here](https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/doc/XWikiDocument.java#L6703), where the OldRendering component allows changing the default behavior if (for example, only example actually) the old 1.0 rendering extension is installed and wants to handle things differently than DefaultOldRendering already does.

If we really want to leave that pluggable stuff, we might move it to the refactoring module, but 2 almost identical rename implementations are hard to maintain and useless.


It shouldn't be hard to find the head node. The best check is probably a query for a subject which isn't an object of a `rest` statement.

I wasn't worried about writing that code because we already know the head node via `ten`. Does this make the test fragile in some way I'm missing?


why i can not find the new attribute when i compile the library ?
i already have the latest version


No problem, glad you could use the code too :)


on the superdirt side, this could be either a float or an int, so ```Num``` would be fine.


Oh right, forgot to mention in commit message - planting the seed for the jail system.


This commit breaks backward compatibility. Symfony 3.0 requires PHP >= 5.5.9, which means I can't use it with my hand-maintained version of PHP 5.5.8. Symfony 2.3 was fine with PHP >= 5.3.3. Please increase at least the minor version for fundamental changes like these.


I will setup a dev version tomorrow. I think I will change the application id to tachiyomi-dev to allow having the stable and dev version installed.


no need of `&&` characters


々test


Probably the best way of prevent this is using pushd and popd commands.

```
pushd xcache
popd || return 0;
```


there are 2 files here ".DS_Store" and "lean_fridge_app.db" that are data files and you shouldn't put them into github. in order to avoid git from suggesting you to add them to git, you can add them in a file called gitignore. read about it here: https://git-scm.com/docs/gitignore

so you can add that config in your computer and then from next time it will not Show you files with those extensions.


Might need to walk me through what these are for on Monday. Thanks!


Don't know if you got an alert, but I commented your commit to manage-this-node. At least in my version I had to change ``var sifsrc = selectedContent.children('iframe').src;`` to ``var sifsrc = selectedContent.children('iframe').attr('src');``. The original was never defined.

This resulted in every content-tab being reloaded on menu-click.
I don't know if it's acting the same way for you, but when I copied your code to Muximux it did - just in case you're having issues!


Yeah, saw that error too, fixed in commit 9f5e081


This line was very readable as was before. Also is out of the scope of this PR


@obazoud +1


this shouldn't be part of primary key i think


| yes, all culling is drawMidPos-based

erm no?
i.e. in the case of FlyingPieces it's not


hmm didnt we talk about reducing the scope such that you only have table (and one model) ?


This is awesome! Where did you find this info?


done. rebased on last commit.


ForgeMultipart will not update any more


変更部分について確認しました。特にコメントはありません。


Semantic markup: Gebruikt geen anchor tags als het geen link is. Gebruik <button>'s.


El problema es que no está enlazado el archivo script en el index


This depends on https://github.com/ruby-rdf/rdf-spec/pull/46. After https://github.com/ruby-rdf/rdf-spec/pull/47, this repository should be passing the full test suite.


Why is humidity ""%.1f C"?


Git commit


i noticed it happening again before i changed this, but I can't get it to happen 100% anymore


Thanks for catching this - fixed it at 548a76235db37db6740e306d7024cb229d3a75a3 .


why comment this out? don't you want to take the new count ?


can you have 2 rows with the same name? if not this field must be marked as unique


dene


I don't understand the full implication of re-using the same or starting a new dataloader either.  Since these events happen infrequently, none will not have great performance impact.  It is fine to leave it as none.


name this variable as "newItemName" or "new_item_name" instead of generic fieldcontent


`operator`での変換だと以下のコードがエラーになることが分かったので、変換を`operator`ではなく、コンストラクタで行う事にしました。
```cpp

namespace dxle
{
	class rgb;
	class dx_color;

	class rgb {
	public:
		explicit operator dx_color()const;
	};
	class dx_color {
	public:
		operator rgb()const { return rgb{}; }
	};
	rgb::operator dx_color()const { return dx_color{}; }
}
int main()
{
	using namespace dxle;
	rgb rgb1;
	dx_color dxcolor1;

	rgb1 = dxcolor1; //OK
	//dxcolor1 = a; //OK
	dxcolor1 = static_cast<dx_color>(rgb1); //OK

	rgb rgb2(dxcolor1); //OK
	dx_color dxcolor2(rgb1); //OK

	rgb rgb3{ dxcolor1 };//error
	dx_color dxcolor3{ rgb1 };//error
}
```


doesn't the total field need a value ?


I don't think you even need this datum line at all, unless i'm missing something. Which would mean that activateFunctions doesn't have to be more global, if you're feeling squirrelly about that.


boo this stinks robotbuilder is better


lines 85-87 are repeated from lines 76-78. by mistake?


Thank you for this!


resolves #40 and #39


I haven't played with this yet, what gets output?    Do both the temperature and humidity fields wind up in each (JSON) message?   If so, is it clear it's NaN, Null, or equivalent?


good catch


I don't think a snapshot is required here (or in `#count`) as `ss` serves that purpose once we enter the block.  `#query_pattern` has multiple calls to `@data`, so grabbing a reference to a specific structure when the method starts seems like a good call.


Olá Swift!


@mfpiccolo @samphippen @mcasper I pushed this directly to master as there's another PR blocked on this change, but please feel free to review.


Could you please make the constructors public. There is no way currently to use this class outside the library.


About the `% .1f` -- I don't know, the original code is `%3.1f` which makes sense for CSV only.
`data_make` and `data_acquired_handler` don't have access to `.fields` -- the format is encoded in the `data_make` call only.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1170](http://teamcity.monogame.net/viewLog.html?buildId=13645&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Note: This is effectively a stand-in for what would w/ higher ranked types be

```rust
T: for<U> SelectableExpression<U>,
```

Ultimately the like of HRTs doesn't matter as much for marker traits, as `()` is usually a reasonable stand in.


hmm, the db file name here doesn't match the one in line 10 of the other file .. by accident? you need only one db file right ?


+= with one replicated and one distributed argument


it was the " C" I was asking about - is that a literal or some bit of format?


You mean
```
C += A * B
```
?


1) 'tells'
2) code returns, it doesn't tell
3) strings can contain ❤ ☀ ☆ ☂ ☻ ♞ ☯ ☭ ☢ € also, not just numbers ... and that's ok, you don't have to specify that :)


l'anglais (sans A majuscule) : 2 occurences ligne 138
piètre (pas piêtre)
1 bon article pour comprendre l'intérêt de HSL : https://la-cascade.io/3-raisons-dutiliser-hsl-pour-vos-couleurs/


Well spotted. It should really be "%%".


format the code, please


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


This revision is clean.


Yes
On Jan 16, 2016 6:24 PM, "Justus Calvin" <notifications@github.com> wrote:

> You mean
>
> C += A * B
>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ValeevGroup/tiledarray/commit/4db07c05b50885c6bf0c0bba472b1cef50c001e5#commitcomment-15492600>
> .
>


Why Hash=512 ? Joona never said to use an absurdly oversized hash.
What if these tests fail, and you have to do them all over again with a correct hash size ? What a waste...


Can't you use diff merging? Then you never ever lose things like the ImagicalMine header.
I bet someone is adding them back and commit the files with the headers.. useless work and commits :)
Just a tip ofcourse. Makes commits more understandable. btw use better descriptions for commit titles. Then everyone understand what the commit is all about and it is easier to track a changelog (if you do that).


`si` seems to be undeclared; might have been thinking of just `mu`?


This should be in exceptions.py


Combine multiple types in a single isinstance call: `isinstance(e, (x, y, z))`


Copy & paste fail :D fixed


Definitely, and as much as we want people to contribute we also need to be able to understand what they're trying to contribute.


une explosion du nombre de sélecteurs (ou : une multiplication des sélecteurs)


Summa will definitely cause the argument tiles to be broadcast. It assumes that the input tile must always be broadcast. It might be worth while to abstract the broadcast functionality of Summa: dense, sparse, and replicated broad cast functors. This would make it easier to mix and match the argument dependent behavior that we want.


RuboCop was giving me some grief with Exception and wanted Standard Error. Pedantic but made rubocop happy.


istype(player.current, /mob/new_player)? All seems to work as expected in that case.

Also already a define isnewplayer(player.current) which presumably would work.


Unfortunately, the workaround of explicitly dlopening libblas to resolve missing symbols in libgsl doesn't seem to work on Android.


А почему в одну строчку все атрибуты не укажешь?


It's not working.

Running `VERBOSE=1 make install` gives:
```
<snipped>

error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: can't open file: /usr/local/openmm/lib/plugins/libOpenMMCUDA.dylib (No such file or directory)

<snipped>
```

(Incidentally, this error doesn't stop make from continuing.)

The error only occurs if I nuke `/usr/local/openmm` first. This make me think that `install_name_tool` is being run before the file installed to that location.


...suffisamment éloquentes pour qu'il ne soit pas nécessaire...


uhm, no


You basically removed the LowPolySystem-integration, _thanks_ :+1:


response code for new things created is usually 201:
http://www.restapitutorial.com/httpstatuscodes.html


i actually tried out your code on my machine and addding an item didn't work. does it work for you ?


Any reason why none of the classes are in packages?


makes sense.


These constructors should probably be public


michelNumberCode -> expectedMichelNumber


With the new "VerbatimInput" the book and code3 files will always be in alignment since the code3 copies are imported into the book.


code -> michelNumber


Any reason why these are not private?


Comments to describe what classes or methods do should be in JavaDoc style


Это не expected, мы здесь это не проверяем. Эта переменная нужна нам лишь для стаббинга метода и все. Сделай для нее inline.


Any reason these aren't private? You have no control over how subclasses manipulate these fields so, on the principle that you should use the most restrictive modifier possible, I would have thought private is more appropriate


Congratulations, no fuckups!


Why not raise an error here in this case?


Честно говоря, я бы ее заинлайнил.


Consider making some of the fields set here final. Not necessary but can act as a nice way of ensuring fields are valid.


Relevant: https://github.com/pebble/pebblejs/issues/127


ids -> seriesIds


Unneeded line spacing changes, but it works. urn:upnp-org:serviceId:SwitchPower1 is indeed the proper service id. +1. Please merge


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1170](http://teamcity.monogame.net/viewLog.html?buildId=13645&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:27:36


shlint: you don't need `$`.


It pointless to test the API by duplicating its exact implementation in the test.

A better test for this API would:
- On Windows, compare the returned value with COMPUTERNAME environment variable
- On Unix, compare the returned value with HOSTNAME environment variable


shlint: avoid backticks.


shlint_reuse: consider wrapping `zenity` into a function.


This probably doesn't belong in this class. I think it would be better to completely separate game logic from GUI, its more flexible and they are totally separate concerns.


![img5517b2269ea893 00868213n](https://cloud.githubusercontent.com/assets/14913140/12375119/80afb06e-bcd5-11e5-8364-4da88d51d24b.jpg)


Добавил комментарии для метода `findByMichelNumber()`, для остальных трех тоже самое.


BTW: This is violating the corefx coding guidelines to always reference platform libraries via x-plat shims.


Might be worthwhile actually figuring out the aim is here and whether `float64bits` is needed. All this line seems to be doing is seeing whether the `2^0` bit is flipped on (or, in other words, testing if `z` is odd or even). "May" be faster to just get the lower word and then do `&`. Granted, getting the lower word requires two lookups (1 to set the float64 view and 1 to access the uint32 view), but my hunch is still faster than repeated `string` concatenation, etc.


*all culling _where glitches would be visually obvious_ (i.e. not tiny fast-flying piece polygons)

happy now, or shall we split even more hairs?


P.S. Вроде бы все проревьювил, так что можешь исправлять.


avoid names like "my... "


hmm, where is this myItemQuery defined?


shlint: weird indirect expansion at `${!ans}`


👌

On Sat, Jan 16, 2016 at 1:54 PM, andrew-rgr <notifications@github.com>
wrote:

> typo fix. thanks @peterood!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/redgreenrepeat/redgreenrepeat.github.io/commit/e42abed23939969c7d4c83f492f43e81a6afa5b0
>


OK


this should be named just `item_id` since thats what this represents right ?


I'm getting a crash on "free(bad);" :(


shlint-name: RC stands for run-command. Consider cfg.
shlint-sec: use `declare -p $1` or `${1}=$(printf '%q' "$2")` for correct escaping.


Its not explicitly clear that difficulty == maximum number of goes. Maybe rename variable?


shlint: what? regex?


shlint-ext-prog: Consider using `/proc` here to remove external calls.


I think this is an abuse of Exceptions as this isn't an exceptional case, the method when operating correctly just notifies its caller if the game is over or not. Maybe its better to return ```true``` if the game is over and ```false``` if the game is still ongoing?


shlint-quote: mkdir arg not quoted.


God that API really looks ugly in ES6. Would much more prefer something like `Highcharts.use(exporting)`


i think delete implementation is a bit incomplete


quot.


Clean, simple and readable code. I know it is just a beginning but still :heart_eyes:

I am proud!
![lee](https://cloud.githubusercontent.com/assets/7466903/12375136/b181fd62-bcb4-11e5-962f-f08fb284f59b.gif)


Hello Luis. Thanks for your work. However, this is not the proper way to solve the problem.
The way of working with map/reduce is filtering all the elements you're interesting in within the javascript function. So the filter you use on db.users.inline_map_reduce must be done inside spendingMap().

I'm uploading the correct way to work with this.


Or have an enum


Hey Sam. I like that solution!
While we're at this submission, can I ask a favour? would you mind terribly reorganising this whole article?

Please look at [this example](https://github.com/FreeCodeCamp/wiki/blob/master/Bonfire-Find-The-Longest-Word-in-a-String.md) for reference of what I mean... The solutions together, and then all the explanations together afterwards. Pretty please?


@mrbig Yes, it was intentional because we by necessity^ either broke the external ADC reading or the internal VDD reading. It was deemed that not supporting the external ADC by default was the lesser evil.

Existing installations should be unaffected as the init data block is already present. For new builds one can either define the appropriate `ESP_INIT_DATA_ENABLE_READVDD33`, `ESP_INIT_DATA_ENABLE_READADC` or `ESP_INIT_DATA_FIXED_VDD33_VALUE` as desired, or explicitly construct + write the init data (e.g. using the offical Espressif flash tool).

We apologize the inconvenience.

^) It's an SDK and/or hardware limitation that makes it not possible to support both at the same time.


what happens if someone adds an item that already exist? what is the desired behavior?


copyright-chk: non-gpl-compliant copyright info. Use `# Copyright (C) Cee TeeEchi (cth451) <cth451@gmail.com>`.


I don't think it should be the Game State's responsibility to do this. I think a Game Logic class should implement this method and hold a reference to Game State instead.


If I do not use regex to remove space, then later I must use regex to match with possible spaces.


Again, not responsibility of State.


Same as below. `main script for EarthLiveShell`.


Introduces another typo :)


Thank you for pointing this out. The original ES6 source had it returning the value of next() but evidently I didn't rewrite that bit correctly

For reference, this was the original ES6 source:

```javascript
import isFunction from 'lodash/lang/isFunction';

export default deps => () => next => action =>
  next(isFunction(action) ? action(deps) : action);
```


I'd rather use `Start EarthLive`.


`earthlive-gtk.sh`.


Play until what? Method name could be clearer


Yep. These two should be cammel casified shouldn't they?


Unnecessary `$` again.


Can use foreach loop


Use @Override annotations


You are correct, but it is minor. If someone cleans it up the new code should use defines. I mainly used this code when I worked on the demod. It is of limited use now.


It would be clearer to say g != '0'


Well using com->T in that location never worked for me but the 0 also didn't work.  Thank you for clarifying that, I will try the new unit when I get back.  I've never used C before so this is all very new to me.


Could you use a source which always points to the most current version, e.g. https://raw.githubusercontent.com/justjohn/twig.js/master/twig.min.js?


Yes, `opt*` should be used whenever possible.


@jboulder That did not happen when the exception was some pyon deserialization error.


hahaha you caught me. i started fixing up my blog on a plane a few weeks ago, got tired, and fell asleep. needed to commit this act of blasphemy in order to get my latest blog post out the door :)


I'm quite sure that this is not an appropriate way to handle this, but it is far better than just silently swallowing the error and therefore wasting the time of whoever is trying to debug it.


PAT_fix (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (2128 of 2384 tests passed, 0 test warnings)

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


bah this commit didnt do what i said it did


be sure you update Printer::cleanNozzle with 0 and 1 accordingly


Use 0  and 1 for T0 and T1 instead of 1 and 2


Good catch!


@koistya Implementing built-in APIs in JavaScript itself is something we've been thinking about recently and have considered in the past.  It is in C++ mostly due to inertia.  Having them in JavaScript would afford us nice benefits such as automatic optimization via the JIT, e.g. inlining, make it faster to implement the APIs and more accessible for the community to do so.

That said we have no yet done a deep dive to explore what sorts of issues might crop up in moving APIs to JavaScript.  I expect there will be gotchas presenting hurdles to overcome before we can realize the benefits.  Definitely an idea we'd like to investigate further.


Do you know when the arrival and departure happen (cf. the months you mention in first paragraph)?


Lands with multiple land types (eg Overgrown Tomb) have problems due to order of basic land types, fix implemented. Commit incoming.


"towards" used twice in close proximity


@Nadahar I think we have more urgent things to work on, and probably always will :)


Does this FIXME just want a way to check in the `ObligationForest` whether or not an obligation is present within the tree containing the current obligation, and that it's also not in the backtrace?

Would it then be sufficient here and wherever else to just add an iterator that can walk subtrees in the forest and use the resulting walk here in conjunction with the backtrace? Or some partially persistent ('cuz snapshotting) lookup structure?

(I can chuck whatever into the snapshotting branch I have)


Op


Opip


Good


never submit uselesscode


is this file being used?


might be simpler to rename `read_body`?


@Syphurith It means "response end".


Interesting considerations, @TobiasLounsbury. I hadn't thought of all of this.

Part of the complication here is that Joomla!'s handling of permissions that were created by a CiviCRM extension is broken. If I use an access argument "register to volunteer" in the menu XML, it will be rejected every time in a Joomla! environment.

Our workaround for the Joomla! permissions problem in the Volunteer 1.x series was to check permissions using `CRM_Volunteer_Permission::check()`, which basically says, "If this CiviCRM instance can't handle extension permissions, give the green light any time a CiviVolunteer permission is checked." This was okay(ish) when we had only a few permissions and they didn't grant users much access to the system. For CiviVolunteer 2.x, where there are a lot more permissions and they grant considerably more access to the system, `CRM_Volunteer_Permission::check()` needs to be more nuanced, the Joomla! integration needs to be fixed, or people just shouldn't use CiviVolunteer 2.x on Joomla!. But I digress...

Unless I'm mistaken, none of the menu items edited in the commit appear in CiviCRM's main menu, so the show/hide benefit you pointed out here probably isn't relevant.

However, digging around some of the core menu XML, I discovered that you can [pass your own callable](https://github.com/civicrm/civicrm-core/blob/4.6.11/CRM/Core/xml/Menu/Contact.xml#L7) into an item's `access_callback` property. If this works as it appears to, I could use `CRM_Volunteer_Permission::check()` in the menu layer as well as the form and BAO layers where it has traditionally been used, like so:

```xml
   <item>
      <path>civicrm/volunteer/roster</path>
      <page_callback>CRM_Volunteer_Page_Roster</page_callback>
      <title>Roster</title>
     <!-- Access is further restricted in CRM_Volunteer_BAO_Project::retrieve() -->
     <access_callback>CRM_Volunteer_Permission::check</access_callback>
     <access_arguments>access CiviCRM</access_arguments>
     <is_public>true</is_public>
    </item>
```

This seems like a decent middle ground and worth trying out, don't you think?


ほげほげ


`enum class`? With member names such as `SUCCESS` there's a good chance that an unscoped enum will result in conflicts sooner or later.


I make no claims to be a documentation expert, this just seemed like the least verbose way to explain it.


yes.


Great. will try to follow the object oriented way. Thanks. :+1:


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


I think in this case it is redundant.


Looks like this will solve issue #4977

I'm confused by the tagging. Is this included in 18.0-rc or 15.0-rc?
![image](https://cloud.githubusercontent.com/assets/1640318/12370254/cb52bf70-bbbe-11e5-8228-f28cd534c8eb.png)


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


Looks like the message was accidentally changed from `Inlinee is vararg` to `Inlinee has no body`.