[inst/site/classifier_test.R:9:1:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L9) *style:* **lines should not be more than 100 characters.**
```r
cylinder <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 1,], w, p, a, "exact", 0.01)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/classifier_test.R:9:82:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L9) *style:* **Commas should always have a space after.**
```r
cylinder <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 1,], w, p, a, "exact", 0.01)
                                                                                ^
```
[inst/site/classifier_test.R:10:1:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L10) *style:* **lines should not be more than 100 characters.**
```r
bell <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 2,], w, p, a, "exact", 0.01)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/classifier_test.R:10:78:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L10) *style:* **Commas should always have a space after.**
```r
bell <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 2,], w, p, a, "exact", 0.01)
                                                                             ^
```
[inst/site/classifier_test.R:11:1:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L11) *style:* **lines should not be more than 100 characters.**
```r
funnel <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 3,], w, p, a, "exact", 0.01)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/classifier_test.R:11:80:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L11) *style:* **Commas should always have a space after.**
```r
funnel <- manyseries_to_wordbag(CBF[["data_train"]][CBF[["labels_train"]] == 3,], w, p, a, "exact", 0.01)
                                                                               ^
```
[inst/site/classifier_test.R:15:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L15) *style:* **Use <-, not =, for assignment.**
```r
tfidf = bags_to_tfidf( list("cylinder" = cylinder, "bell" = bell, "funnel" = funnel) )
      ^
```
[inst/site/classifier_test.R:19:18:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L19) *style:* **Use <-, not =, for assignment.**
```r
labels_predicted = rep(-1, length(CBF[["labels_test"]]))
                 ^
```
[inst/site/classifier_test.R:20:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L20) *style:* **Use <-, not =, for assignment.**
```r
labels_test = CBF[["labels_test"]]
            ^
```
[inst/site/classifier_test.R:21:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L21) *style:* **Use <-, not =, for assignment.**
```r
data_test = CBF[["data_test"]]
          ^
```
[inst/site/classifier_test.R:22:33:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L22) *style:* **Commas should always have a space after.**
```r
for (i in c(1:length(data_test[,1]))) {
                                ^
```
[inst/site/classifier_test.R:23:10:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L23) *style:* **Use <-, not =, for assignment.**
```r
  series = data_test[i,]
         ^
```
[inst/site/classifier_test.R:23:24:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L23) *style:* **Commas should always have a space after.**
```r
  series = data_test[i,]
                       ^
```
[inst/site/classifier_test.R:24:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L24) *style:* **Use <-, not =, for assignment.**
```r
  bag = series_to_wordbag(series, w, p, a, "exact", 0.01)
      ^
```
[inst/site/classifier_test.R:25:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L25) *style:* **Use <-, not =, for assignment.**
```r
  cosines = cosine_sim(list("bag"=bag, "tfidf" = tfidf))
          ^
```
[inst/site/classifier_test.R:25:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L25) *style:* **Put spaces around all infix operators.**
```r
  cosines = cosine_sim(list("bag"=bag, "tfidf" = tfidf))
                                ~^~
```
[inst/site/classifier_test.R:26:23:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L26) *style:* **Use <-, not =, for assignment.**
```r
  labels_predicted[i] = which(cosines$cosines == max(cosines$cosines))
                      ^
```
[inst/site/classifier_test.R:31:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L31) *style:* **Use <-, not =, for assignment.**
```r
error = length(which((labels_test != labels_predicted))) / length(labels_test)
      ^
```
[inst/site/classifier_test.R:31:22:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L31) *style:* **Place a space before left parenthesis, except in a function call.**
```r
error = length(which((labels_test != labels_predicted))) / length(labels_test)
                     ^
```
[inst/site/classifier_test.R:36:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/classifier_test.R#L36) *style:* **Place a space before left parenthesis, except in a function call.**
```r
which((labels_test != labels_predicted))
      ^
```
[inst/site/density_curve_test.R:7:3:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L7) *style:* **Use <-, not =, for assignment.**
```r
df=data.frame(time=c(1:length(ecg)),value=ecg)
  ^
```
[inst/site/density_curve_test.R:7:3:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L7) *style:* **Put spaces around all infix operators.**
```r
df=data.frame(time=c(1:length(ecg)),value=ecg)
 ~^~
```
[inst/site/density_curve_test.R:7:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L7) *style:* **Put spaces around all infix operators.**
```r
df=data.frame(time=c(1:length(ecg)),value=ecg)
                 ~^~
```
[inst/site/density_curve_test.R:7:37:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L7) *style:* **Commas should always have a space after.**
```r
df=data.frame(time=c(1:length(ecg)),value=ecg)
                                    ^
```
[inst/site/density_curve_test.R:7:42:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L7) *style:* **Put spaces around all infix operators.**
```r
df=data.frame(time=c(1:length(ecg)),value=ecg)
                                        ~^~
```
[inst/site/density_curve_test.R:8:51:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L8) *style:* **Put spaces around all infix operators.**
```r
p1 <- ggplot(df, aes(time, value)) + geom_line(lwd=1.1,color="blue1") + theme_classic() +
                                                 ~^~
```
[inst/site/density_curve_test.R:8:56:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L8) *style:* **Commas should always have a space after.**
```r
p1 <- ggplot(df, aes(time, value)) + geom_line(lwd=1.1,color="blue1") + theme_classic() +
                                                       ^
```
[inst/site/density_curve_test.R:8:61:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L8) *style:* **Put spaces around all infix operators.**
```r
p1 <- ggplot(df, aes(time, value)) + geom_line(lwd=1.1,color="blue1") + theme_classic() +
                                                           ~^~
```
[inst/site/density_curve_test.R:11:40:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L11) *style:* **Commas should always have a space after.**
```r
        axis.title.x = element_blank(),axis.title.y=element_blank(),
                                       ^
```
[inst/site/density_curve_test.R:11:52:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L11) *style:* **Put spaces around all infix operators.**
```r
        axis.title.x = element_blank(),axis.title.y=element_blank(),
                                                  ~^~
```
[inst/site/density_curve_test.R:12:21:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L12) *style:* **Put spaces around all infix operators.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                   ~^~
```
[inst/site/density_curve_test.R:12:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L12) *style:* **Commas should always have a space after.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                                     ^
```
[inst/site/density_curve_test.R:12:49:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L12) *style:* **Put spaces around all infix operators.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                                               ~^~
```
[inst/site/density_curve_test.R:13:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
               ~^~
```
[inst/site/density_curve_test.R:13:29:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Commas should always have a space after.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                            ^
```
[inst/site/density_curve_test.R:13:31:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Commas should always have a space after.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                              ^
```
[inst/site/density_curve_test.R:13:40:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Commas should always have a space after.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                                       ^
```
[inst/site/density_curve_test.R:13:47:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Commas should always have a space after.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                                              ^
```
[inst/site/density_curve_test.R:13:50:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                                                ~^~
```
[inst/site/density_curve_test.R:13:53:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Commas should always have a space after.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                                                    ^
```
[inst/site/density_curve_test.R:13:56:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L13) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=df[390:510,],aes(time,value),lwd=2,col="red")
                                                      ~^~
```
[inst/site/density_curve_test.R:15:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L15) *style:* **Use <-, not =, for assignment.**
```r
w=100
 ^
```
[inst/site/density_curve_test.R:15:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L15) *style:* **Put spaces around all infix operators.**
```r
w=100
~^~
```
[inst/site/density_curve_test.R:16:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L16) *style:* **Use <-, not =, for assignment.**
```r
p=8
 ^
```
[inst/site/density_curve_test.R:16:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L16) *style:* **Put spaces around all infix operators.**
```r
p=8
~^~
```
[inst/site/density_curve_test.R:17:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L17) *style:* **Use <-, not =, for assignment.**
```r
a=8
 ^
```
[inst/site/density_curve_test.R:17:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L17) *style:* **Put spaces around all infix operators.**
```r
a=8
~^~
```
[inst/site/density_curve_test.R:20:35:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L20) *style:* **Put spaces around all infix operators.**
```r
ecg_str <- paste(ecg_sax, collapse=" ")
                                 ~^~
```
[inst/site/density_curve_test.R:23:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L23) *style:* **Use <-, not =, for assignment.**
```r
density_curve = rep(0,length(ecg))
              ^
```
[inst/site/density_curve_test.R:23:23:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L23) *style:* **Commas should always have a space after.**
```r
density_curve = rep(0,length(ecg))
                      ^
```
[inst/site/density_curve_test.R:24:4:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L24) *style:* **Place a space before left parenthesis, except in a function call.**
```r
for(i in 2:length(ecg_grammar)){
   ^
```
[inst/site/density_curve_test.R:25:8:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L25) *style:* **Use <-, not =, for assignment.**
```r
  rule = ecg_grammar[[i]]
       ^
```
[inst/site/density_curve_test.R:26:6:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L26) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(j in 1:length(rule$rule_interval_starts)){
     ^
```
[inst/site/density_curve_test.R:27:8:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L27) *style:* **Use <-, not =, for assignment.**
```r
    xs = rule$rule_interval_starts[j]
       ^
```
[inst/site/density_curve_test.R:28:8:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L28) *style:* **Use <-, not =, for assignment.**
```r
    xe = rule$rule_interval_ends[j] + w
       ^
```
[inst/site/density_curve_test.R:33:20:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L33) *style:* **Put spaces around all infix operators.**
```r
which(density_curve==min(density_curve))
                  ~^~~
```
[inst/site/density_curve_test.R:34:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Use <-, not =, for assignment.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
           ^
```
[inst/site/density_curve_test.R:34:26:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Put spaces around all infix operators.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                        ~^~
```
[inst/site/density_curve_test.R:34:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Commas should always have a space after.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                                     ^
```
[inst/site/density_curve_test.R:34:39:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Put spaces around all infix operators.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                                     ~^~
```
[inst/site/density_curve_test.R:34:46:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Commas should always have a space after.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                                             ^
```
[inst/site/density_curve_test.R:34:46:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Place a space before left parenthesis, except in a function call.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                                             ^
```
[inst/site/density_curve_test.R:34:50:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L34) *style:* **Put spaces around all infix operators.**
```r
min_values = data.frame(x=c(444:476),y=rep(0,(476-443)))
                                                ~^~
```
[inst/site/density_curve_test.R:36:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L36) *style:* **Use <-, not =, for assignment.**
```r
density_df=data.frame(time=c(1:length(density_curve)),value=density_curve)
          ^
```
[inst/site/density_curve_test.R:36:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L36) *style:* **Put spaces around all infix operators.**
```r
density_df=data.frame(time=c(1:length(density_curve)),value=density_curve)
         ~^~
```
[inst/site/density_curve_test.R:36:27:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L36) *style:* **Put spaces around all infix operators.**
```r
density_df=data.frame(time=c(1:length(density_curve)),value=density_curve)
                         ~^~
```
[inst/site/density_curve_test.R:36:55:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L36) *style:* **Commas should always have a space after.**
```r
density_df=data.frame(time=c(1:length(density_curve)),value=density_curve)
                                                      ^
```
[inst/site/density_curve_test.R:36:60:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L36) *style:* **Put spaces around all infix operators.**
```r
density_df=data.frame(time=c(1:length(density_curve)),value=density_curve)
                                                          ~^~
```
[inst/site/density_curve_test.R:37:20:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L37) *style:* **Commas should always have a space after.**
```r
shade <- rbind(c(0,0), density_df, c(2229,0))
                   ^
```
[inst/site/density_curve_test.R:37:43:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L37) *style:* **Commas should always have a space after.**
```r
shade <- rbind(c(0,0), density_df, c(2229,0))
                                          ^
```
[inst/site/density_curve_test.R:38:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L38) *style:* **Put spaces around all infix operators.**
```r
names(shade)<-c("x","y")
           ~^~~
```
[inst/site/density_curve_test.R:38:21:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L38) *style:* **Commas should always have a space after.**
```r
names(shade)<-c("x","y")
                    ^
```
[inst/site/density_curve_test.R:39:31:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L39) *style:* **Put spaces around all infix operators.**
```r
p2 <- ggplot(density_df, aes(x=time,y=value)) +
                             ~^~
```
[inst/site/density_curve_test.R:39:37:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L39) *style:* **Commas should always have a space after.**
```r
p2 <- ggplot(density_df, aes(x=time,y=value)) +
                                    ^
```
[inst/site/density_curve_test.R:39:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L39) *style:* **Put spaces around all infix operators.**
```r
p2 <- ggplot(density_df, aes(x=time,y=value)) +
                                    ~^~
```
[inst/site/density_curve_test.R:40:16:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L40) *style:* **Put spaces around all infix operators.**
```r
  geom_line(col="cyan2") + theme_classic() +
              ~^~
```
[inst/site/density_curve_test.R:41:45:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L41) *style:* **Put spaces around all infix operators.**
```r
  geom_polygon(data = shade, aes(x, y), fill="cyan", alpha=0.5) +
                                           ~^~
```
[inst/site/density_curve_test.R:41:59:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L41) *style:* **Put spaces around all infix operators.**
```r
  geom_polygon(data = shade, aes(x, y), fill="cyan", alpha=0.5) +
                                                         ~^~
```
[inst/site/density_curve_test.R:43:1:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L43) *style:* **lines should not be more than 100 characters.**
```r
  theme(plot.title = element_text(size = rel(1.5)), axis.title.x = element_blank(),axis.title.y=element_blank(),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/density_curve_test.R:43:84:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L43) *style:* **Commas should always have a space after.**
```r
  theme(plot.title = element_text(size = rel(1.5)), axis.title.x = element_blank(),axis.title.y=element_blank(),
                                                                                ^
```
[inst/site/density_curve_test.R:43:96:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L43) *style:* **Put spaces around all infix operators.**
```r
  theme(plot.title = element_text(size = rel(1.5)), axis.title.x = element_blank(),axis.title.y=element_blank(),
                                                                                ~^~
```
[inst/site/density_curve_test.R:44:21:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L44) *style:* **Put spaces around all infix operators.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                   ~^~
```
[inst/site/density_curve_test.R:44:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L44) *style:* **Commas should always have a space after.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                                     ^
```
[inst/site/density_curve_test.R:44:49:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L44) *style:* **Put spaces around all infix operators.**
```r
        axis.ticks.y=element_blank(),axis.text.y=element_blank()) +
                                               ~^~
```
[inst/site/density_curve_test.R:45:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
               ~^~
```
[inst/site/density_curve_test.R:45:29:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Commas should always have a space after.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                            ^
```
[inst/site/density_curve_test.R:45:35:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Commas should always have a space after.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                                  ^
```
[inst/site/density_curve_test.R:45:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Commas should always have a space after.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                                     ^
```
[inst/site/density_curve_test.R:45:41:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                                       ~^~
```
[inst/site/density_curve_test.R:45:44:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Commas should always have a space after.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                                           ^
```
[inst/site/density_curve_test.R:45:47:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L45) *style:* **Put spaces around all infix operators.**
```r
  geom_line(data=min_values,aes(x,y),lwd=2,col="red")
                                             ~^~
```
[inst/site/density_curve_test.R:48:26:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/density_curve_test.R#L48) *style:* **Put spaces around all infix operators.**
```r
grid.arrange(p1, p2, ncol=1)
                        ~^~
```
[inst/site/direct_optimizer_test.R:9:5:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L9) *style:* **Use <-, not =, for assignment.**
```r
  w = round(x[1], digits = 0)
    ^
```
[inst/site/direct_optimizer_test.R:10:5:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L10) *style:* **Use <-, not =, for assignment.**
```r
  p = round(x[2], digits = 0)
    ^
```
[inst/site/direct_optimizer_test.R:11:5:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L11) *style:* **Use <-, not =, for assignment.**
```r
  a = round(x[3], digits = 0)
    ^
```
[inst/site/direct_optimizer_test.R:13:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L13) *warning:* **no visible binding for global variable ‘train_labels’**
```r
  m <- length(train_labels)
              ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:14:22:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L14) *warning:* **no visible binding for global variable ‘train_labels’**
```r
  c <- length(unique(train_labels))
                     ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:16:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L16) *warning:* **no visible global function definition for ‘cvFolds’**
```r
  folds <- cvFolds(m, K = nfolds, type = "random")
           ^~~~~~~
```
[inst/site/direct_optimizer_test.R:16:27:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L16) *warning:* **no visible binding for global variable ‘nfolds’, Did you mean 'folds'?**
```r
  folds <- cvFolds(m, K = nfolds, type = "random")
                          ^~~~~~
```
[inst/site/direct_optimizer_test.R:24:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *warning:* **no visible global function definition for ‘alply’, Did you mean 'apply'?**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
            ^~~~~
```
[inst/site/direct_optimizer_test.R:24:26:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *warning:* **no visible binding for global variable ‘train_labels’**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
                         ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:24:40:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *style:* **Commas should always have a space after.**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
                                       ^
```
[inst/site/direct_optimizer_test.R:24:42:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *style:* **Commas should always have a space after.**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
                                         ^
```
[inst/site/direct_optimizer_test.R:24:53:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
                                                    ^
```
[inst/site/direct_optimizer_test.R:24:55:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L24) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
    bags <- alply(unique(train_labels),1,function(x){x})
                                                      ^
```
[inst/site/direct_optimizer_test.R:26:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L26) *warning:* **no visible binding for global variable ‘train_labels’**
```r
      ll <- which(train_labels[set_train] == unique(train_labels)[j])
                  ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:26:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L26) *warning:* **no visible binding for global variable ‘train_labels’**
```r
      ll <- which(train_labels[set_train] == unique(train_labels)[j])
                  ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:28:71:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L28) *style:* **Commas should always have a space after.**
```r
                  manyseries_to_wordbag( (train_data[set_train, ])[ll,], w, p, a, "exact", 0.01)
                                                                      ^
```
[inst/site/direct_optimizer_test.R:30:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L30) *style:* **Use <-, not =, for assignment.**
```r
    tfidf = bags_to_tfidf( bags )
          ^
```
[inst/site/direct_optimizer_test.R:30:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L30) *warning:* **no visible global function definition for ‘bags_to_tfidf’**
```r
    tfidf = bags_to_tfidf( bags )
            ^~~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:32:22:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L32) *style:* **Use <-, not =, for assignment.**
```r
    labels_predicted = rep(-1, length(set_test))
                     ^
```
[inst/site/direct_optimizer_test.R:33:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L33) *style:* **Use <-, not =, for assignment.**
```r
    labels_test = train_labels[set_test]
                ^
```
[inst/site/direct_optimizer_test.R:33:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L33) *warning:* **no visible binding for global variable ‘train_labels’**
```r
    labels_test = train_labels[set_test]
                  ^~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:34:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L34) *style:* **Use <-, not =, for assignment.**
```r
    data_test = train_data[set_test,]
              ^
```
[inst/site/direct_optimizer_test.R:34:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L34) *warning:* **no visible binding for global variable ‘train_data’**
```r
    data_test = train_data[set_test,]
                ^~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:34:37:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L34) *style:* **Commas should always have a space after.**
```r
    data_test = train_data[set_test,]
                                    ^
```
[inst/site/direct_optimizer_test.R:37:10:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L37) *style:* **Use <-, not =, for assignment.**
```r
      bag=NA
         ^
```
[inst/site/direct_optimizer_test.R:37:10:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L37) *style:* **Put spaces around all infix operators.**
```r
      bag=NA
        ~^~
```
[inst/site/direct_optimizer_test.R:38:35:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L38) *style:* **Put spaces around all infix operators.**
```r
      if (length(labels_predicted)>1) {
                                 ~^~
```
[inst/site/direct_optimizer_test.R:39:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L39) *style:* **Use <-, not =, for assignment.**
```r
        bag = series_to_wordbag(data_test[j,], w, p, a, "exact", 0.01)
            ^
```
[inst/site/direct_optimizer_test.R:39:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L39) *warning:* **no visible global function definition for ‘series_to_wordbag’**
```r
        bag = series_to_wordbag(data_test[j,], w, p, a, "exact", 0.01)
              ^~~~~~~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:39:45:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L39) *style:* **Commas should always have a space after.**
```r
        bag = series_to_wordbag(data_test[j,], w, p, a, "exact", 0.01)
                                            ^
```
[inst/site/direct_optimizer_test.R:41:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L41) *style:* **Use <-, not =, for assignment.**
```r
        bag = series_to_wordbag(data_test, w, p, a, "exact", 0.01)
            ^
```
[inst/site/direct_optimizer_test.R:41:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L41) *warning:* **no visible global function definition for ‘series_to_wordbag’**
```r
        bag = series_to_wordbag(data_test, w, p, a, "exact", 0.01)
              ^~~~~~~~~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:43:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L43) *style:* **Use <-, not =, for assignment.**
```r
      cosines = cosine_sim(list("bag" = bag, "tfidf" = tfidf))
              ^
```
[inst/site/direct_optimizer_test.R:43:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L43) *warning:* **no visible global function definition for ‘cosine_sim’**
```r
      cosines = cosine_sim(list("bag" = bag, "tfidf" = tfidf))
                ^~~~~~~~~~
```
[inst/site/direct_optimizer_test.R:45:28:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L45) *style:* **Use <-, not =, for assignment.**
```r
       labels_predicted[j] = which(cosines$cosines == max(cosines$cosines))
                           ^
```
[inst/site/direct_optimizer_test.R:51:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L51) *style:* **Use <-, not =, for assignment.**
```r
    error = length(which((labels_test != labels_predicted))) / length(labels_test)
          ^
```
[inst/site/direct_optimizer_test.R:51:26:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L51) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    error = length(which((labels_test != labels_predicted))) / length(labels_test)
                         ^
```
[inst/site/direct_optimizer_test.R:56:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L56) *style:* **Use <-, not =, for assignment.**
```r
  err = mean(laply(errors,function(x){x}))
      ^
```
[inst/site/direct_optimizer_test.R:56:14:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L56) *warning:* **no visible global function definition for ‘laply’, Did you mean 'lapply'?**
```r
  err = mean(laply(errors,function(x){x}))
             ^~~~~
```
[inst/site/direct_optimizer_test.R:56:27:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L56) *style:* **Commas should always have a space after.**
```r
  err = mean(laply(errors,function(x){x}))
                          ^
```
[inst/site/direct_optimizer_test.R:56:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L56) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  err = mean(laply(errors,function(x){x}))
                                     ^
```
[inst/site/direct_optimizer_test.R:56:40:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L56) *style:* **Closing curly-braces should always be on their own line, unless it's followed by an else.**
```r
  err = mean(laply(errors,function(x){x}))
                                       ^
```
[inst/site/direct_optimizer_test.R:58:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L58) *style:* **Commas should always have a space after.**
```r
  print(paste(w,p,a, " -> ", err))
                ^
```
[inst/site/direct_optimizer_test.R:58:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L58) *style:* **Commas should always have a space after.**
```r
  print(paste(w,p,a, " -> ", err))
                  ^
```
[inst/site/direct_optimizer_test.R:66:8:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L66) *style:* **Use <-, not =, for assignment.**
```r
nfolds = 30
       ^
```
[inst/site/direct_optimizer_test.R:67:32:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L67) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(120,60,12)),
                               ^
```
[inst/site/direct_optimizer_test.R:67:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L67) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(120,60,12)),
                                 ^
```
[inst/site/direct_optimizer_test.R:67:49:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L67) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(120,60,12)),
                                                ^
```
[inst/site/direct_optimizer_test.R:67:52:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L67) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(120,60,12)),
                                                   ^
```
[inst/site/direct_optimizer_test.R:72:8:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L72) *style:* **Use <-, not =, for assignment.**
```r
nfolds = 50
       ^
```
[inst/site/direct_optimizer_test.R:73:32:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L73) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(140,50,12)),
                               ^
```
[inst/site/direct_optimizer_test.R:73:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L73) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(140,50,12)),
                                 ^
```
[inst/site/direct_optimizer_test.R:73:49:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L73) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(140,50,12)),
                                                ^
```
[inst/site/direct_optimizer_test.R:73:52:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L73) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(140,50,12)),
                                                   ^
```
[inst/site/direct_optimizer_test.R:77:6:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L77) *style:* **Use <-, not =, for assignment.**
```r
data = read.table("../../sax-vsm_classic.git/src/resources/data/Beef/Beef_TRAIN")
     ^
```
[inst/site/direct_optimizer_test.R:78:14:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L78) *style:* **Use <-, not =, for assignment.**
```r
train_labels = unlist(data[,1])
             ^
```
[inst/site/direct_optimizer_test.R:78:29:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L78) *style:* **Commas should always have a space after.**
```r
train_labels = unlist(data[,1])
                            ^
```
[inst/site/direct_optimizer_test.R:79:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L79) *style:* **Use <-, not =, for assignment.**
```r
train_data = matrix(unlist(data[,-1]), nrow = length(train_labels))
           ^
```
[inst/site/direct_optimizer_test.R:79:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L79) *style:* **Commas should always have a space after.**
```r
train_data = matrix(unlist(data[,-1]), nrow = length(train_labels))
                                 ^
```
[inst/site/direct_optimizer_test.R:80:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L80) *style:* **Use <-, not =, for assignment.**
```r
nfolds=30
      ^
```
[inst/site/direct_optimizer_test.R:80:7:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L80) *style:* **Put spaces around all infix operators.**
```r
nfolds=30
     ~^~
```
[inst/site/direct_optimizer_test.R:81:32:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L81) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(470,100,16)),
                               ^
```
[inst/site/direct_optimizer_test.R:81:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L81) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(470,100,16)),
                                 ^
```
[inst/site/direct_optimizer_test.R:81:49:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L81) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(470,100,16)),
                                                ^
```
[inst/site/direct_optimizer_test.R:81:53:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L81) *style:* **Commas should always have a space after.**
```r
S <- directL(cverror, rep(c(10,2,2)), rep(c(470,100,16)),
                                                    ^
```
[inst/site/direct_optimizer_test.R:85:3:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/direct_optimizer_test.R#L85) *style:* **Commented code should be removed.**
```r
# nloptr.print.options()
  ^~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/makedata.R:1:2:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L1) *style:* **Commented code should be removed.**
```r
## makes a gun-point dataset
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/site/makedata.R:6:14:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L6) *style:* **Use <-, not =, for assignment.**
```r
labels_train = unlist(dtrain[,1])
             ^
```
[inst/site/makedata.R:6:31:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L6) *style:* **Commas should always have a space after.**
```r
labels_train = unlist(dtrain[,1])
                              ^
```
[inst/site/makedata.R:7:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L7) *style:* **Use <-, not =, for assignment.**
```r
labels_test = unlist(dtest[,1])
            ^
```
[inst/site/makedata.R:7:29:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L7) *style:* **Commas should always have a space after.**
```r
labels_test = unlist(dtest[,1])
                            ^
```
[inst/site/makedata.R:9:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L9) *style:* **Use <-, not =, for assignment.**
```r
data_train = matrix(unlist(dtrain[,-1]), nrow = length(labels_train))
           ^
```
[inst/site/makedata.R:9:36:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L9) *style:* **Commas should always have a space after.**
```r
data_train = matrix(unlist(dtrain[,-1]), nrow = length(labels_train))
                                   ^
```
[inst/site/makedata.R:10:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L10) *style:* **Use <-, not =, for assignment.**
```r
data_test = matrix(unlist(dtest[,-1]), nrow = length(labels_test))
          ^
```
[inst/site/makedata.R:10:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L10) *style:* **Commas should always have a space after.**
```r
data_test = matrix(unlist(dtest[,-1]), nrow = length(labels_test))
                                 ^
```
[inst/site/makedata.R:12:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L12) *style:* **Use <-, not =, for assignment.**
```r
Gun_Point = list("labels_train" = labels_train, "data_train" = data_train,
          ^
```
[inst/site/makedata.R:22:14:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L22) *style:* **Use <-, not =, for assignment.**
```r
labels_train = unlist(dtrain[,1])
             ^
```
[inst/site/makedata.R:22:31:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L22) *style:* **Commas should always have a space after.**
```r
labels_train = unlist(dtrain[,1])
                              ^
```
[inst/site/makedata.R:23:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L23) *style:* **Use <-, not =, for assignment.**
```r
labels_test = unlist(dtest[,1])
            ^
```
[inst/site/makedata.R:23:29:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L23) *style:* **Commas should always have a space after.**
```r
labels_test = unlist(dtest[,1])
                            ^
```
[inst/site/makedata.R:25:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L25) *style:* **Use <-, not =, for assignment.**
```r
data_train = matrix(unlist(dtrain[,-1]), nrow = length(labels_train))
           ^
```
[inst/site/makedata.R:25:36:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L25) *style:* **Commas should always have a space after.**
```r
data_train = matrix(unlist(dtrain[,-1]), nrow = length(labels_train))
                                   ^
```
[inst/site/makedata.R:26:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L26) *style:* **Use <-, not =, for assignment.**
```r
data_test = matrix(unlist(dtest[,-1]), nrow = length(labels_test))
          ^
```
[inst/site/makedata.R:26:34:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L26) *style:* **Commas should always have a space after.**
```r
data_test = matrix(unlist(dtest[,-1]), nrow = length(labels_test))
                                 ^
```
[inst/site/makedata.R:28:5:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L28) *style:* **Use <-, not =, for assignment.**
```r
CBF = list("labels_train" = labels_train, "data_train" = data_train,
    ^
```
[inst/site/makedata.R:35:72:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/makedata.R#L35) *style:* **Put spaces around all infix operators.**
```r
ecg0606 <- read.table("inst/test_data/ecg0606_1.csv", as.is = T, header=F)$V1
                                                                      ~^~
```
[inst/site/readme_examples.R:10:16:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L10) *style:* **Put spaces around all infix operators.**
```r
plot(x, y, type="l", col="blue",
              ~^~
```
[inst/site/readme_examples.R:10:25:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L10) *style:* **Put spaces around all infix operators.**
```r
plot(x, y, type="l", col="blue",
                       ~^~
```
[inst/site/readme_examples.R:11:10:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L11) *style:* **Put spaces around all infix operators.**
```r
     main="A scaled sine wave with a random noise and its z-normalization")
        ~^~
```
[inst/site/readme_examples.R:14:9:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L14) *style:* **Put spaces around all infix operators.**
```r
abline(h=c(1,-1), lty = 2, col = "gray50")
       ~^~
```
[inst/site/readme_examples.R:14:14:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L14) *style:* **Commas should always have a space after.**
```r
abline(h=c(1,-1), lty = 2, col = "gray50")
             ^
```
[inst/site/readme_examples.R:15:36:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L15) *style:* **Commas should always have a space after.**
```r
legend(0, -4, c("scaled sine wave","z-normalized wave"),
                                   ^
```
[inst/site/readme_examples.R:16:18:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L16) *style:* **Commas should always have a space after.**
```r
       lty = c(1,1), lwd = c(1,1), col = c("blue","red"), cex = 0.8)
                 ^
```
[inst/site/readme_examples.R:16:32:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L16) *style:* **Commas should always have a space after.**
```r
       lty = c(1,1), lwd = c(1,1), col = c("blue","red"), cex = 0.8)
                               ^
```
[inst/site/readme_examples.R:16:51:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L16) *style:* **Commas should always have a space after.**
```r
       lty = c(1,1), lwd = c(1,1), col = c("blue","red"), cex = 0.8)
                                                  ^
```
[inst/site/readme_examples.R:24:16:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Commas should always have a space after.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
               ^
```
[inst/site/readme_examples.R:24:17:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Put spaces around all infix operators.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
               ~^~
```
[inst/site/readme_examples.R:24:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Put spaces around all infix operators.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                 ~^~
```
[inst/site/readme_examples.R:24:22:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Commas should always have a space after.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                     ^
```
[inst/site/readme_examples.R:24:23:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Put spaces around all infix operators.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                     ~^~
```
[inst/site/readme_examples.R:24:25:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Put spaces around all infix operators.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                       ~^~
```
[inst/site/readme_examples.R:24:27:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Put spaces around all infix operators.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                         ~^~
```
[inst/site/readme_examples.R:24:30:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L24) *style:* **Commas should always have a space after.**
```r
abline(v = c(1,1+7/3,1+7/3*2,8), lty = 3, lwd = 2, col = "gray50")
                             ^
```
[inst/site/readme_examples.R:28:12:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Commas should always have a space after.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
           ^
```
[inst/site/readme_examples.R:28:22:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Commas should always have a space after.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                     ^
```
[inst/site/readme_examples.R:28:23:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Put spaces around all infix operators.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                     ~^~
```
[inst/site/readme_examples.R:28:25:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Put spaces around all infix operators.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                       ~^~
```
[inst/site/readme_examples.R:28:28:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Commas should always have a space after.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                           ^
```
[inst/site/readme_examples.R:28:38:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Commas should always have a space after.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                                     ^
```
[inst/site/readme_examples.R:28:41:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Put spaces around all infix operators.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                                       ~^~
```
[inst/site/readme_examples.R:28:44:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Commas should always have a space after.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                                           ^
```
[inst/site/readme_examples.R:28:47:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L28) *style:* **Put spaces around all infix operators.**
```r
segments(1,y_paa3[1],1+7/3,y_paa3[1],lwd=1,col="red")
                                             ~^~
```
[inst/site/readme_examples.R:29:9:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
       ~^~
```
[inst/site/readme_examples.R:29:11:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
         ~^~
```
[inst/site/readme_examples.R:29:13:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
           ~^~
```
[inst/site/readme_examples.R:29:15:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
             ~^~
```
[inst/site/readme_examples.R:29:18:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Commas should always have a space after.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                 ^
```
[inst/site/readme_examples.R:29:19:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                 ~^~
```
[inst/site/readme_examples.R:29:30:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Commas should always have a space after.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                             ^
```
[inst/site/readme_examples.R:29:33:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                               ~^~
```
[inst/site/readme_examples.R:29:40:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Commas should always have a space after.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                                       ^
```
[inst/site/readme_examples.R:29:43:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Put spaces around all infix operators.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                                         ~^~
```
[inst/site/readme_examples.R:29:47:](https://github.com/jMotif/jmotif-R/blob/3bf2bbfe52aa8cda076a40c319b76bd3eb8ddd22/inst/site/readme_examples.R#L29) *style:* **Commas should always have a space after.**
```r
points(x=1+7/3/2,y=y_paa3[1],col="red",pch=23,lwd=5)
                                              ^
```


this is giving:
AttributeError: module 'dask.array' has no attribute 'from_sequence'


Good!!!


oh I see this should just be dask.bag instead of dask.array


we dont need these for commented out lines, do we?


Interesting; this was tested before, and if it passes it seems it must be related to helper threads being more accurate than before (e.g., compare http://tests.stockfishchess.org/tests/view/5698ef7e0ebc594a0871d892 and http://tests.stockfishchess.org/tests/view/563bd1b10ebc59227df16fc8 ).  Back then, the "depth filter" corrected for crappiness of helper threads to some extent.  There were many attempts to adjust this logic..


Apologies. Hard to test without access to a Windows machine.


@pferrel add this missing is necessary to enter the user previously created


Убрать скобки?


Remove the first Type production? It seems redundant with last one now since custom mods, constraint, and byref are optional.


TypeError: from_pandas() missing 1 required positional argument: 'npartitions'


Может это под default лучше убрать?


@AliceAponasko What the difference in this case?


BTW, how it works previously is that it just directly uses the column id.  This is something not originally intended.  I was thinking if I should put that back as a fallback but decided not to unless we run into further issues.


Crashes here after authorizing a Twitter account on iPad. Could make this an .Alert, or configure popoverPresentationController.


:+1: thanks, marc!


Looks good!


keep at v1.1.0 to match current SpaDES-module versions


@briansmith It's not exactly a trivial script (though of course it pales besides `configure`), so duplicating it in at least two scripting languages sounds like a hassle. Also we already require Python for several other things, possibly including LLVM (not quite sure if this is avoidable). Furthermore, people who install CMake and a C++ compiler (both unavoidable prerequisites), even if just to build from source (developers will often want to use one of the many Python scripts in `src/etc`), are probably power user enough to either have Python already or be able to install it easily. In summary, I don't really see building without Python as a worthwhile or easily achievable goal.


Added because I was pretty sure it's missing accidentally. However you're right it's not good to muddle the changes, I'll make a separate PR. Here's the OpenBSD commit for reference https://github.com/aliclark/rust/commit/cfd2a5cfa148b1c927571cc2e9f06e242831b696


Issue 14 complete
and Issue 13 complete

Adding more issues now.


MX priority. Since both have the same priority, clients should randomly choose which server to try first.


@fjumawan I went ahead and changed all post tips to have this styling (ex: `tip`) so if you wanted to continue using this you can.


Thanks


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


on the prior commit matt mentioned showing outputs here

also his ideas on using the ipython sphinx directive and hosting data files on s3 are worth considering, although they would take significant work. at the very least we'd better get with kerry and grab time from me or someone to test all the quickstarts once before we publish them, using data files as needed to make it work. even if we don't have them on s3 for users we could at least put the small test data files in a subdirectory of docs here on github.

then once matt's back from vacation he can look over it and merge it in.


Nitpick: trailing space added here by accident.


Hmm. Type can also have CustomMods* so it's a bit ambiguous grammar-wise.


//I don't know what to do IJZM


Is there a test case that changes based on this?


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


@clebertsuconic @normanmaurer

I'm always worried about commenting on public PRs.  I know I'm not a jerk but I always worry about being an accidental one.

It seems to me you're targeting making a more efficient ``FileRegion`` but it's not clear how much more efficient this can be considering that mutex that is in there.  I also don't believe its performance can be radically different than ``sendfile`` for a competing use case.

To me it seems ``libaio``'s top use case is to get data in and out of your process and a disk using a zero-copy DMA transfer.  If ``libaio`` is to be integrated into Netty, it should be for this use case and thus be part of the event loop, for an example, see

https://github.com/scylladb/seastar/blob/master/core/reactor.cc

Seastar has incredible performance and this is how it does it, there is no fundamental difference in the approach taken to files and sockets.  Both types are asynchronous, event driven and owned by a single thread.

It also fits better into Netty's treading model as it's possible to have one IO queue per thread instead of a shared one that could potentially have contention on that lock.  On Linux, at least on XFS, this would make it possible to have true asynchronous file I/O, but wouldn't that imply a Netty ``Channel`` for files would be needed?

I fear such a model would also mean even more work, since if Netty provides a ``Channel`` that represents a file, than a channel implementation for asynchronous I/O on other supported operating systems would be needed and yet another that falls back to blocking I/O.

However, I still see some major issues here, namely that all ``ByteBuf``s would need to be 512 bytes aligned, otherwise the user would be responsible for determining if it's safe to pass a ``ByteBuf`` versus another to the same API.

This also begs the question is Netty just for network I/O or [more] (http://netty.io/wiki/using-as-a-generic-library.html)


Nitpick: Switch cases should not be indented.


Trailing dot is missing for both entries; these hostnames will be treated as subdomains, e.g. `mxa.mailgun.org.jenkins.io`.


Thanks. Tested the patch on Arch Linux and it works great.


But Jenkins looks like failed !!!


OK, makes sense to me, thanks.


@borshop: retry


You don't have to refer to self from the main thread. I think the code looks cleaner.


@nickdodd25 I was running it on nginx but the following gist (or something along these lines) would probably work if you throw it in a .htaccess https://gist.github.com/JakeBooher/5cb6c7eaeaf35c772221


Hah, I was just about to push the same thing -- cheers!


Thank you for discovering what the setup commands do!


Can we rip out PolymorphicFunctionType completely, or is it still used in the AST?


Under review.


I wasn't sure if I should call this encounterRole or encounterRoleUuid.


Что-то не понимаю, это как*


Build of six hours.


PlantCompBaseClassAgain (mbadams5) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 329 test warnings)

 * 331 tests had: AUD diffs.
 * 329 tests had: BND diffs.

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


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


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


Thank you! I don't know how I missed this earlier - I've been very busy with PhD work.


Thanks @sleitner

I didn't feel too bad about pushing log file name changes directly instead of PR.


No need to include the header an foot inside the block. They are already including outside of the block via base.html

Essentially adding a header and footer between a header and footer.


I think SIL parsing is the big thing that's still in terms of PolymorphicFunctionType / GenericParamList.

Also it's not clear to me if serialization and name mangling are using interface types or not.


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/1ead83b2b6f8d51e29e2fa0b09b6c2b4bbea6dee


merging basho/riak_kv/at/finalisation = 1ead83b2 into borshop-integration-1327-at/finalisation


basho/riak_kv/at/finalisation = 1ead83b2 merged ok, testing candidate = 4436431f


what is happening here?


Also crashes here on iPad when asked to confirm disconnect.


Verified working on Linux.


TYPO


:eyes: :hammer:


에러객체는 new될 때 내부에서 스택을 생성함으로 부담을 줄이기위해


I've added a note to the bug to ask the original committer (still with the project) to verify the lineage.


After this change, the collapsing part is gone from researchinitiative extraction processing workflow. Shouldn't we collapse the results produced by `referenceextraction_researchinitiative_wos` and `referenceextraction_researchinitiative` workflow nodes? I'm thinking about a case when both of these nodes produce the same "document->research initiative context" link but with different confidence levels. In such case, we should probably collapse these links to a single one with confidence level equal to maximum of these two.

BTW, without your description in the commit, it would not be obvious what is in the change introduced in Oozie workflow - it's good that you put it in there! In order to see where the change is, I decided to run [vipe](https://github.com/openaire/vipe) on the old and new workflow and compare (this didn't work at first - I need to introduce a patch to vipe code).


I think we can agree that the purpose of this test is not "what does `new Buffer(10737418.235)` do?" though now that we're aware of the difference between node running on V8 and chakracore, and the apparent misalignment between V8 and ES6, it does sort of imply there's a new issue that should be filed about that whole thing. @Fishrock123 seems to think that the existing behavior will have to be endorsed by adding an explicit floor in Buffer; and yes, floor rather than round seems to be the current behavior (but both will test the correct thing, that writing too much to a stream is a detected error):

With either outcome of that issue, the change to conditionally round size will either become redundant on chakracore, or newly required on V8, so this test would need to be revisited at that time.  That is my main motivation for suggesting a change now that affects all engines, and that stops incidentally testing what the Buffer constructor does with non-Integers; that would be done by a new test added after it's decided how to handle non-integer arguments to Buffer.

(no, I didn't file an issue as I don't think I have a good enough understanding of the technical issues to file the best possible issue)

@kunalspathak, I understand your motivation to make sure that you aren't changing what the tests test on V8, and if that is the right motivation overall then I'm wrong to ask you to approach it differently for this one test only.


test


fixes https://github.com/libreoss/liberator-api/issues/125


ok, cool, thanks


Unreachable, right?


nice


This is a great tutorial! I love being able to see the individual commit changes per step.

I accidentally placed line 2 inside the if statement and took me a while to realise why I wasn't getting the expected output. It might be helpful to show the whole file in the tutorial (since it's so short), if this happens to more people.


Can this migrate to BetterMimaPlugin?


972154-NewSimpleChilledCeiling (RKStrand) - i386-Windows-7-VisualStudio-12: OK (2074 of 2074 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.

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


I don't think this does what you think it does in yaml.


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


PlantCompBaseClassAgain (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.

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


ok

Btw I also created https://github.com/typesafehub/migration-manager/issues/101


which tests are fixed by this? instead of doing it in this top level include (where it's quite invasive) can we do it only in the test assembly that needs it?


please, could you read the whitespaces, so that it looks more readable … this adds a `<br />`


Why is gitignore included?


same-nameDB.openDB()  this function will open the same name DB if this DB already existed,or it will create a new DB.


Line 16 on aggregate, shouldn't this be a method now? `fields: {}`


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


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


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


Solid edit.  Thanks @kalendar!


Yes it should. Going to change that tomorrow and review this again after some sleep


If we do that, how an admin user can modify the scope of a bookmark ("all" vs "limited to one user") ?


The default of ``true`` is the current behavior, right?


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


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


No problem!  Thanks for doing this.


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


Please undo this change.


Thanks, but I wanted the flexibility that argparser provides.


Sorry I meant, only hide when selecting another building that has crafting, so only 1 UI is visible


Made more generic.


I don't like that we have to reparse the configuration on every request. Can you parse each configuration once into a zero-terminated ``uint16_t`` array?


fixed


Looks great! Thanks for adding this.


I think that this would be clearer if we just did:

```C
if (s->current.retry_type == SIMPLE_RETRY) {
} else if (s->current.retry_type == DEAD_SERVER_RETRY) {
} else if (s->current.attempts < s->http_config_param->parent_connect_attempts) {
} ...
```


There's some new chages


Move the declaration of ``server_response`` here, since it is only used in this scope.


I would better change this line to `if (currCellAvailable <= 0.0)` as floating points are not easy to substract to exactly zero due to rounding errors.


@Yive Alright, sounds good.


While std::string tags for device creation makes some sense to me, making subtags into std::strings would mean having to deal with the fact that null pointers can no longer be used, and the default string constructor results in an unexpectedly valid DEVICE_SELF subtag. However, I have no idea why moving *all* device parameters to std::string was considered a good idea.

(Then again, my opinion is that it might be better to define device names by a required override, much as was done before 50767de7072d0a751840cd794af21da52e02c487. However, my plan would be to cache a std::string return value at config-complete time; this would allow device_t::m_name to be properly privatized while letting drivers and other devices vary their names depending on their specific configuration.)

I wonder: what was the problem that actually motivated these recent reverts?


Will allow undo after <kbd>OK</kbd> is pressed


Nice :+1:


the tab could be converted to whitespace but it does not break with python 3 so it is just cosmetic.


Since you can't have simple retry and dead server retry enabled at the same time, we should simplify this down to 2 configurations; one for the retry policy and one for the response codes.

``proxy.config.http.parent_origin.retry_enabled`` would accept values 0 (off), 1 (simple) and 2 (dead server).

Initially I didn't like the ``parent_origin`` nomenclature but it is consistent with our use of ``parent_proxy``, so not I think it is just fine :)


@obastemur Oops, run into duplicated work... I just checked in a change today: https://github.com/Microsoft/ChakraCore/blob/linux/lib/common/CommonPal.h#L90


It seems that the vni info is lost if I use the the vxlan-gpe extension, is there any steps i missed?
Host1
ip addr add dev eth0 192.168.50.22/24
ip link add vxlan0 type vxlan external gpel3 remote 192.168.50.21
ip link set dev vxlan0 up
ip addr add dev vxlan0 172.168.100.22/24
ip route add 172.168.100.0/24 dev vxlan0 encap ip id 10 dst 192.168.50.21

Host2
ip addr add dev eth0 192.168.50.21/24
ip link add vxlan0 type vxlan external gpel3 remote 192.168.50.22
ip link set dev vxlan0 up
ip addr add dev vxlan0 172.168.100.21/24
ip route add 172.168.100.0/24 dev vxlan0 encap ip id 10 dst 192.168.50.21


112158233_PVwithStorageSystem (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 0 test warnings)

 * 9 tests had: AUD diffs.

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


__API Description file syntax error on line 102__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Quoting @bohoomil in google plus.

> You can ignore it as it's only needed for coding. I should have put it in a different branch: sorry for unintended confusion. ;-)

https://plus.google.com/+bohoomil-infinality-bundle/posts/TsVuTBo6EoB


> Note: Collection functions work on arrays, objects, and array-like objects such as arguments, NodeList and similar. But it works by duck-typing, so avoid passing objects with a numeric length property. It's also good to note that an each loop cannot be broken out of — to break, use _.find instead.


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


Complete changelog:
* InfoPanel is reset when GameLogic is reset
* Added target types
* Added `sell` method and button
* Hovering over a buy button while buying another object no longer changes the info panel
* Changed InfoPanel from a `JEditorPane` to many `JLabel`s
* Added icon for goal
* Added German language support
* Minor language corrections for Spanish


Interesting question, I don't know if compatibility mode messes up the WordPress admin screens.

If so, I think it is worth keeping globally but if not then we could make this change.

A couple of notes, about your examples:

1.  I'm pretty sure the first example will not work in PHP < 5.3, so I'd prefer setting up a separate function
2.  While I respect the brevity of

```
is_admin() || add_action( 'send_headers', 'bsg_add_header_xua' , 1);
```

I've found being verbose and explicit is helpful in reviewing foreign code so I'd prefer.

```
if ( is_admin() ) {
add_action( 'send_headers', 'bsg_add_header_xua' , 1);
}
```


this was breaking with the latest nltk


good job!


112158233_PVwithStorageSystem (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 0 test warnings)

 * 9 tests had: AUD diffs.

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


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


"unused variable `stat`"
8 needs to be `qlen`


I would suggest processing input on medium priority. That way long computational low-prio stuff don't get in the way of input, and thus we get a more responsive system.


"unused variable `stat`" :)


great to hear this


Looks good too.

github_api_token should probably be added to defaults/main.yml. (see http://docs.ansible.com/ansible/playbooks_roles.html#role-default-variables and roles/*/defaults/main.yml)


@davelab6 Not sure how to handle this field which is not part of the Proto Message, but was stored in the JSON before...


Yeah this is actually pointed at the wrong version. master has a _-prefixed version and also fixes the prints to work with python 3. I may move it to the test directory though. First, dinner.


good to see the ugly hack will go away


Thanks!  I had a feeling I'd messed up somewhere…  I need to get into the habit of being more thorough with testing.


What


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


I don't think you need this import now (Text.PrettyPrint.Boxes), as you're not directly using anything from it.


Mine fails on compile with that set to false.


@rdoc101 please find the jpg version of the icons here


이 while 문은 뭐하는 짓이지 ? 없어도 될거같은데


just making a note here that I should probably make this more efficient with vector and ostringstream


I think originally I was trying for a more explicit statement, but now I'm not as convinced that my version is all that much better...


What about regular comments, like /* ... */  ?


Yeah, because this metadata field was so often out of sync between the public repo and serving infrastructure, it is no longer being maintained in the public repo.

Therefore this check needs to be inferred; if the family exists in the Developer API then it is "External". If the family exists in the CSS API when directly called there (eg 'http://fonts.googleapis.com/css?family=FamilySans&subset=all' returns CSS with TTF urls) then it is "Internal". If the CSS API returns a 400 error, it is "Sandbox"


> touch $(DESTDIR)/var/run/st2auth.sock and `%attr(644, nginx, nginx) %{_localstatedir}/run/st2auth.sock` shouldn't be part of packaging logic, since socket should appear only by and only when service starts. Also, `/run` and `/var/run` doesn't persist across reboots.

Doesn't matter, the file is a %ghost. Now that I think about it, we should probably remove `%attr` since it achieves nothing.

Frankly, at that point I want to make something, anything that works end to end. Then, once we have it, we can tweak it as long as we like. Full disclosure, I have no idea what "standard" means in that context, I'm like this boy from the joke: "I'm not the real welder, just found the mask on construction site".


`vector` is most likely actually measurable.

I don't know that the string addition is really all that bad since the updates are in-place. I suspect the stream might actually be worse (virtual function overhead).


Added Rule #3


please add missing files


/me note to myself: write unit test


please remove this chore test


PlantCompBaseClassAgain (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2052 of 2056 tests passed, 325 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: Table small diffs.

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


how could render call result in exception?


Yeah, we have measured differences for vector vs deque doing this exact same thing in other parts of the codebase. While the += updates are in place, I wonder if `(word + "_")` is doing bad things. For the stream, I will test it, but I can see your point.


Thank you so much for the feedback, Stephen! :)

Cheers,

Rita (JJ)


Looks like this should be e_sg, (sg effi) not h_sg (sg hits).


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


ok


Удобство чтения тут основное преимущество.
Мне доходчивее написано, что есть trashdirInput->text(), чётко обособлены пробелами title, oldDirectory, newDirectory. Я дальше "=" читать не стану. И так понятно, что там будет.


Could you format it according yii2 coding standards?


Dont understand


There is no need for this since we have symlinks in init


This is wrong! You are duplicating the driver!


aight


Great, I will remove it.


bergos, never do this!


I know it's AHRS from the library, but perhaps a more descriptive name would help with debugging/ quick-glances?


@m-kuhn working great.

Applying a gradient fill of type radial with coord mode set to viewport is _real_ nice :)
![radial-gradient png](https://cloud.githubusercontent.com/assets/1728657/12502209/9694f53e-c0f8-11e5-93f9-c5ecbdcab6aa.jpg)


done


Yeah, I know but we have a lot of diffs between pinmux and pinctrl. I will try to find other solution then.


Plz remove the "remove" button~


....D大这里为啥revert掉了呀，我这手机还跟着你加了这个的，没FC呀。。


@m-kuhn, could this possibly have caused the following regression: http://hub.qgis.org/issues/14158 ?


This would be slightly faster if you simply updated the existing build rather than delete and then insert a new one. Plus, the id won't change so potentially clients could be smart about merging changes into an existing object rather than see this as potentially a new build to handle.


maybe we can do a fast Merkle Tree, and the hash function is multiplying the siblings?

LimeWire used [Tiger Tree](https://www.wikiwand.com/en/Merkle_tree#/Tiger_tree_hash), which was basically a Merkle Tree and a [Tiger hash function](https://www.wikiwand.com/en/Tiger_(cryptography)) which was way more complex than a multiplication but it was very secure and fast.

It's important that we fix this as the bloom filter was used to count the number of successful downloads


Yet another reason not to delete Builds....you are holding onto an id here.


@m4rw3r Thanks for this release! I just updated my code to use the new operators (mostly `<|>` and `<*`), and the parser looks even better.


I know this is prototype code, but we might want to check for null here, so that this method isn't called if we failed to setup the AHRS above.


It means "fixed-size, versioned, and copyable using primitive operations". @rjmccall and I were talking about this earlier today and decided to go for principle of least surprise here: if you mark something `fixed_layout`, you probably have an idea of what you want it to do. I don't think there's actually a correctness issue, though, because of the requirement that you only use versioned structs and enums.

The one exception here is non-class-bound protocols, which may require arbitrary copying work. I suppose we could ban those too.


I agree. I think we should be using ControllerAdvice to handle exceptions/validation issues globally so we are not littering controllers with checks and try..catch.


The panic is caused by `self.attrs.borrow()` in the `if let`, and the else uses `self.attrs.borrow_mut()`.


Minor detail....copy/paste error here. Should reference TestResult instead of Build.


Hmm, ok. Might be better if this was a separate performance annotation. I guess for enums, we want something like this too, since it does feel semantically different from @closed.

In any case, I'd suggest making this a requirement for *explicit* @_fixed_layout declarations only. That is, if everything in a framework is fixed_layout by default (because its a private framework you're building non-resiliently), things should still work.


Same feedback here as given for the BuildService. Recommend updating existing data rather than delete/insert.


Do not commit hs_err_pid24234.log


So do you want me to create the patch and submit or you want Antti
Palosaari to do it as he is the Maintainer of Airspy in mainline kernel?


I think they do!!!! Change from browser


Want this to be part of the file


This doesn't fully fix the leak, since Dispose isn't being called.


That's done to add "React Transform" plugin only for builds in a "watch" mode, you can still build the project regularly for both production and development environments without injecting "React Transform" stuff by running either `npm run build` or `npm run build -- --release`.


For the second question: a closed enum has semantics already of being closed. It doesn't inherently promise anything about layout or copying, but because all of the cases are fixed we can just apply the rules recursively. I forgot to say that, though; it should be "a closed enum that is versioned with only fixed-layout-compatible payloads".

Even then, though, it still means checking the tag and then choosing how to copy the thing. Maybe you're right that "fixed-size" is a good enough constraint. @rjmccall?


I'm sorry, I searched but not sure how to do that. Does it mean that I need to declare errCbs as:
```
errCbs chan func()
```
And then here I would do:
```
nc.errCbs <- func(errCb ErrHandler, sub *Subscription, err error) func() {
    return func() {
        errCb(nc, sub, err)
    }
} (nc.Opts.AsyncErrorCB, s, ErrSlowConsumer)
```
which means that when reading from the channel, it would then be:
```
ecb := <- eh:
   ecb()
```
I don't think I got this right, but if I did, then why use 2 channels instead of 1 and do the same for ConnHandler? The advantage is that then Conn and Err handlers will have total ordering.


For enums we don't care because `@closed` gives us enough information to infer whether it's fixed-layout-compatible, and the additional performance assertions cover enforcing trivial copies or a max size. I guess we could add "this has *some* max size", which would match this property.


I think this just shows we would decouple 'fixed-size and trivial to copy for some version of trivial' from 'i'm not going to add and remove stored properties / cases'. ;-)


This is Jiaixng Liu, a student of OS class.


Will @inlineable properties have the same restrictions wrt their types being fixed-size?


Para modificar las vistas de devise se ejecutan el comando `rails g devise:views `, luego se realizan los cambios pertinentes


What if we say the accessors are @inlineable? So if they're present they have to be there from the beginning of time, and you can't change the body in a way that would break existing clients that inlined it.


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2049 of 2056 tests passed, 476 test warnings)

 * 547 tests had: AUD diffs.
 * 477 tests had: RDD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.
 * 2 tests had: MTD diffs.

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


No i hadent. Fixed it is now.


done


forgot to remove this thanks for catch. logic is implemented.


we already do in similar cases, do you have a specific instance in mind?


Thanks for that catch. Fixed


Nice


`.alwaysHit` isn't controlled by the accuracy event.

I suggested moving `ModifyMove` after the Protean check, which I believe is better.


Fixed


updated


Don't worry :)


Hi All, I just want to make sure this is correct and my build/system didn't miss something and remove something accidentally.


adding pages to shower_tray


carousel pic's header and info need to be pulled from database.


just breaking things


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


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


@maning why remove the building,yes here?


This is a great idea. It does the same thing as before, but it enforces mac address checking not only for assigned IP addresses, but those handed out automatically by the DNS server.