[inst/site/classifier_test.R:9:1:](https://github.com/jMotif/jmotif-R/blob/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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/53b344e6045f6ddd42205632befa9a00ae12027c/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)
                                              ^
```


++ great


Force Permissive SELinux


q


:+1: :articulated_lorry: :computer: ** ** _ _ ` ` [google](http://google.com/)


LGTM.


Can you tell what does the base image openzipkin/jre-full:1.8.0_60 have?
I want marge the content to my base image.


That one's coming from FTBLib, not me.


same here


tet


We won't, but that's a really good point. We don't have any procedure built-in that takes arguments yet, WDYT about updating this example when we do?


Can we get a release of this?


Missing a close paren in this paragraph


Looks great so far.


already set to true by `super.initialize( true );`


I just want to add that I am trying to test the " Pull a request " item for commenting and discussing.


looks really strange to call super.initialize in the middle of child initialize call


zzz


lkjnljn


It was originally uppercase, but @Mats-SX told me the opposite, y'all need to hash this out before I change this again


TeamCity pull requests :: netty [Build 1138](https://garage.netty.io/teamcity/viewLog.html?buildId=5177&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4911, ignored: 23 Build time: 00:36:29


I don't think this will work. Consider the domains:
 - `www3.exmaple.com`
 - `beta.exmaple.com`
 - `exmaple.com`

All are equally "external" as `www.exmaple.com`


`_initializer ?= _loadManifest();`


Nope :laughing:


Lets let it pickle for a bit, I must admit I'm a fan, but happy to iterate on it. I like how it clarifies that this thing is context per invocation, which I think highlights that the class is not going to be around, it puts focus on the procedure methods, if that makes sense..


Why remove this header section? Seems sort of weird that all the other sections have headers.


comment_controllerの中では、2か所しか使っていないと思う＆引数が違うのでまとめなくて良い気がするのですが、、、。その代わりに@comments などを代入しているところをmethod化しようかなと思います。


I think this needs to be `css/` not `../css/`.


この文は、正直言って、どういう場合の話をしているのか、さっぱりわからなかった上、ifが2つ出てくる悪文で解釈に困ったのですが、stillについては、その後に比較級の形容詞がないので、「依然として」という意味に解釈するのが自然だと思います。
括弧の前で「～について考慮した方が良い」みたいなことを言っているので、括弧内では「（考慮しても）それでも(=still)、if以下のような場合にはproblematicだよ」ということなのだと理解しています。
そもそも、変更が"more or less equivalent"だとか"not also idempotent"とか、どういう場合を指しているのかまったく想像できないのですが…


Not sure I agree with this change at this point. I think having them downloadable from the TripleA client is useful. Remember many map makers aren't developers.


LGTM.


probando


probando a ver que pasa


Nope. `../css` is correct because of the hoop-jumping we're doing around URLs (specifically /data) and the nginx config. It's a nuisance to have to toggle between `../css` for production and `css` for local testing but I just crack it up to the many small injustices of life...


Demos as an Art

A little more than two years ago I came across a very small computer program that amazed me It was very small as computer programs go but it did something I have never seen before it used a mathematical formula to create a stunning moving display and played music in the background

After a while of looking around I found more of the same type of programs most created by different people, all of which had a common purpose of presenting the user with computer generated art - some by mathematical formulae, some by conventional means, but most a combination of the two. Each one had music composed especially for that program. They were all works of art, a new form of art. I found out that to make such a program one had to have some ideas for something to be represented by mathematical expressions, then express them all in a computer program, that was fast enough to do many complex calculations on an average user’s machine. The latter part turned out to be a lot harder than it sounded. As fast as computers are today, they are not fast enough, for there is always something which requires more and more calculations. Many tricks have to be implemented to make a program run the fastest possible, some of them being, ironically, to write it in the “early” computer languages which dealt more with computer instructions than with the structure of the program itself. Thus the more complex your goal, the simpler means you have to employ to reach it.

Because of this it is very hard to create a fast and small program. The ammount of time and effort spent on writing it can be amazingly high. I, myself, spent two days once writing a program that consisted of about 200 “letters” of computer istructions - a few lines - all generated from a few pages of the program that I wrote, and re-wrote, and re-wrote ...

These programs carry an unassuming name: “demos” , short for “demonstrations”, but they do so much more than that implies. Most of them push computers to the “limit”, doing what was earlier thought of as impossible or at least required a super-computer, but most importantly they define a new art form. This special mode of art requires many new and exciting factors, such as musical composition, art merged with science (a seemingly incompatible mixture), excellent computer programming skills, but most important - teamwork. For a “demo” is composed by a team of people, for no other reason than to show others like them what they can do, and of course to compete with them, for rivalry is an integral part of any human, if we choose to acknowledge that or not.

I am a part of a few of these teams (none of them very famous of course), but I am very proud to be playing some role in that world. Some of the people on those teams I know personally, some I have never met in person. It is very strange and exciting at the same time.

The effects of my discovery of this small world of people who work together around the world for no other reason than to impress each other had far reaching consequences on my further course of action. I decided to produce some of these programs on my own, and was forced to explore many areas by my decision. I was consulting with my friend (an artist), on some ideas, the colors, the overall mood I want to set. I had another friend of mine (a musician) compose something on computer (which he never did before, but discovered it a valuable resource after being convinced by me to try it), and learned more about my computer, which tricks work, and which damage it, which are considered “elegant”, and that a good thing to keep in mind when its 3 o’clock in the morning and you are still trying to correct a persistent error, is that if you punch your computer - it usually doesn’t help you or it. (And of course all the virtues and pitfalls of coffee).

This completely new domain of computer programming was like a whole new universe of possibilities goals and problems opening up for me I have met many people, talked to them about the subject and came to know a new society, into which it is easier to be accepted than one might think but very hard to get one’s bearings and to achieve in it Most people who participate in it are friendly and help others with their problems knowing that others will help them if they ever run into something they have to find out Even though there is little direct communication it is done by making your work available to others and others to you resulting in a stunningly large collection This community is not old  less than 10 years - yet there are people who are no longer in it but remembered and revered by all It is made up of mostly students - in schools and universities - the only ones with enough free time on their hands to participate Therefore it is kind of a life one might live coming in at the beginning learning contributing leaving and being remembered if you invented something new and important or were just plainly a good person

The demo scene (as it is referred to by its regulars), is a very dynamic, and independent place if you chose to call it that for it does not physically exist anywhere or even logically since it is a collection of individuals and their work. It has most of the properties of a real life community minus the “real part It has legacies of gathering (actual physical ones at that) struggles fights (not usually physical), competition, love hate band all of the other human emotions and actions but all expressed in the form of computer art

I have read some books that are composed in a letter format all the events are described in the form of letters from one person to another and back Now imagine a collection of letters not from one person to another and back, but many from many different people not specifically addressed to anyone Add illustrations and more things which could have many allegorical meaningsbAll of the demos are letters from people A way to express their concepts of the state of the world as a whole and of computer development at a given point in particular Just as we might look at paintings from the Renaissance and the 19th century and contrast them to see what changes and views the society held at given times so may be done to demos,of course the time scale compressed manifold

For example the simplest form of history that can be observed here is the evolution of computers themselves.A demo made in ‘93 would basically consist of two-dimensional math and some basic three-dimensioanl models Now only 3 years later a complex “world consisting of many three-dimensional objects and amazingly realistic can be seen regularly This is also observed in other media types like the movies Jurassic Park could not have been made in ‘90, beacuse there simply weren’t computers powerful enough to create realistic moving pictures of dinosaurs

The whole medium is new and emerging just as computers now are and is a brand new way of something that has not even been imagined before  the merge of science and art For centuries scientists and artists were two opposing camps the former exploring the world seeking logical explanations for everything The latter however did not concern themselves with logic but believed in human emotionsin the elements of the spiritual rather than the physical Now technology has finally started to catch up to art if only marginally Demo’s combine science with making raw mathematical expressions take a form. Math is used to describe objects their properties and create something that looks real, or can look like something not possible with any other means

I think that the existence of this “underground” demo scene is an interesting fact by itself. Participation requires a lot of contribution, and, it would seem, not much return. So why do people support it? Beacuse it is the ability to create something, to produce, which is so needed by humans. You are not something, until you have created something, to justify your existence, or just to prove to yourself you exist

"Demos as an Art."

Important Note: If you'd like to save a copy of the paper on your computer, you can COPY and PASTE it into your word processor. Please, follow these steps to do that in Windows:
1. Select the text of the paper with the mouse and press Ctrl+C.
2. Open your word processor and press Ctrl+V.


增加时间轴显示列表+MVP


If you'd like, you might want to fix the typo in the commit message: s/dynmic/dynamic/


It looks like the old code had a more descriptive Arg_WrongType message for this case.  Any reason not to include that?


Technically this is also a change in behavior, in that we now may throw this exception before we throw the null exception for the value.


減便が何を言っているのか一応本家に聞いてみます。返事があるかどうかわかりませんが...


@TheHoward , I'm aware of this issue in Python 3. Python 2 ignores `b` or `B` as bytes literals. But in Python 3, output of `encode('utf-8')` is accepted as bytes instead str type with `b'` as literal.

Due to this issue, this change is not yet included in the PyPI release.


null aware


done!


Object type should not be considered specialization?


NOTE RE: Changing the code in the Execute def seems to work, but this looks funny to me...
Ex:.. Too many pointers pointing to another pointer. local optimization with something more descriptive than `window` would help readability as to what object I should be expecting.
Most of the docstrings should have a return type doc(Ex: like sphinx) too would help.
```
        # installer = self.window.GetDetailsItem()
        installer = self.window.data[self.window.detailsItem]
```


Maybe I was just i bit unclear from the beginning. I'm not talking about performance as such. Obviously knowing that
>This point is actually relevant to the rest of the parser.

But more about code bloating. ~5 to ~11 lines of `(ruleNode.whatever !== whatever) ` is not something one could expect from such a trivial thing as "Am I valid here?" :)


Why has SSL support been reverted?


Let's not.


Why did you remove this?


90% of this commit... let's just not do that mkay.


TeamCity Waterfall :: Build [Build 53](https://tc.demonwav.com/viewLog.html?buildId=243&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 50 Build time: 00:00:59


一旦mergeします


Good idea. I've truncated to 5, which is a little bit conservative ([around 1m](http://gis.stackexchange.com/questions/8650/how-to-measure-the-accuracy-of-latitude-and-longitude))


Also, compactors are all purged


hey @david-truong @gamerson why did you change this to _package_ ?


Start using try catch block so as to throw and catch error for better debugging.
in this case php mail function can fail to send email and there is nothing to handle that case/error/exception.
http://php.net/manual/en/language.exceptions.php
http://www.w3schools.com/php/php_exception.asp


到时候从单机到OL需要的改动你得告诉我啊


I think `additionalRegistrationAttributes` would better reflect the nature of these properties instead of the rather unspecific `other`


Oubliez vos années de Java et faites du C#

```cs
public int YearBegin;
public readonly List<StudyYear> StudyYears;

public double Value {
    get {
        return 0;
    }
}

public double Average {
    get {
        return StudyYears.Average(sy => sy.Average);
    }
}
```

* https://msdn.microsoft.com/en-us/library/dw8e0z9z%28v=vs.90%29.aspx
* https://msdn.microsoft.com/library/bb910354%28v=vs.90%29.aspx


Sorry for the confusion, people. I indeed push to git@github.com:systemed/potlatch2.git. So if @nikerabbit also gets push rights for that repo, all should be well.


还有，GUI好看吗？: )


minor change to the DNSResolver for linux - ordered alphabetically.


These codes are commented so remove it. unnecessary commented codes should not be present in the code.
and that is why we use SCM(source code management) so as to get the previous code whenever required.


Please do feel free to change the copyright holders :wink:
(Same in LICENSE.md) :+1:


Please align to the beginning character of `titleL10nId` or use `var` for each variable.


Older versions crash on GTX 980 under RDP too. Probable cause: NVidia driver update


TeamCity Civic Assembly :: Build backend + frontend [Build 42](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=567&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


TeamCity Civic Assembly :: Build backend + frontend [Build 42](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=567&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1; exit code 123 Build time: 00:00:32


i find next 2 static methods surprising, would prefer to have have `store.getRecord(id, mode)` that will create newRecord and `store.getRecord(id)` for NORMAL mode access


should include rubinius as well.


Hi, I added the code for LPAD, could you merge it?
Thank you


@zhkzyth you are right on the first question. There is a chance that make the `self.on_result` method to be called twice but it does't matter, because in most cases it could be written with `gen.Task` as:

```python
response = yield gen.Task(tasks.func.apply_async, args)
```

The `callback` is the `set_result` wrapper function, so the future will just reslove only once.

```python
def Task(func, *args, **kwargs):
    future = Future()
    def handle_exception(typ, value, tb):
        if future.done():
            return False
        future.set_exc_info((typ, value, tb))
        return True
    def set_result(result):
        if future.done():
            return
        future.set_result(result)
    with stack_context.ExceptionStackContext(handle_exception):
        func(*args, callback=_argument_adapter(set_result), **kwargs)
    return future
```

Yes, of course, it is better that add a condition detection to avoid repeated calls(I forgot it!).

Note that there are no issues if the `self.client.get` is complete before waking up the subscriber, but not the opposite, because the `self.on_result` would call `unsubscribe_channel` and the both futures would be resolved by the same IOLoop.

About the second question, as you can see, this may be a bad practice. Currently, you can only deal with it yourself. In fact, I don't want to fix it. The `expires` argument is the value of the `CELERY_RESULT_PERSISTENT` configuration parameter. I think it is not accurate enough.

```python
if callback:
    self.consumer.wait_for(task_id,
                           partial(self.on_result, task_id, callback),
                           expires=self.prepare_expires(type=int),
                           persistent=self.app.conf.CELERY_RESULT_PERSISTENT)
```

There may be a better way to remove the timeout fucntion or use a custom configuration parameter(I did it in totoro).

```python
……

wait_timeout = int(properties.pop('wait_timeout', 86400)) * 1000

……

if wait_timeout:
            timeout = self.io_loop.add_timeout(
                timedelta(milliseconds=wait_timeout),
                partial(consume_delegate.on_message, WaitForResultTimeoutError(wait_timeout)))
```

In addition, I suggest [using redis-py instead of tornado-redis](https://github.com/leporo/tornado-redis) to `get` the result and deleting the result from the redis after the `on_result` call.


TeamCity Civic Assembly :: Build backend + frontend [Build 43](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=569&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


This addition does not generate a duplicate hierarchy ?


TeamCity Civic Assembly :: Build backend + frontend [Build 43](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=569&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1; exit code 123 Build time: 00:00:32


thanks @mr-salty, you provide an excellent support


Gut! Vielen Dank!


It does.

On 29 Jan 2016, at 11:06, Ric Klaren <notifications@github.com<mailto:notifications@github.com>> wrote:

This addition does not generate a duplicate hierarchy ?

—
Reply to this email directly or view it on GitHub<https://github.com/xebialabs/xl-testview-reference-project/commit/e3e5f109df5aa00d05e5bfa58a04ea80884ba8c4#commitcomment-15757122>.


Well, maybe, but I think boot interfaces shouldn't be managed by it. Also, if I am not wrong, NM doesn't manage bonding very well yet, that's why I usually configure bonding manually.
This patch worked fine for me, the alternative is to completely disable NM.


this test does not do what it should.
it is just a duplicate of "test_dashboard_regenerate_pdf"


Small typo here, should be "createJavaInstallationPro**b**e"


*bold* - I like it.


@lcalvet @simogasp You can use the Pose composition function directly here:
https://github.com/openMVG/openMVG/blob/master/src/openMVG/geometry/pose3.hpp#L43


This should not be possible. If rowidx is not null/undefined, rowidx[0] *should* be an animation id.

This is not a major issue, but I think that throwing an exception if animation is undefined is correct.


Re: default: It does already, in the line below?


What does `chalk.inverse` do?


This fixes mismatch errors?


One of the points with how it works now is to have limited number of priorities to make the ui and setup smoother more understandable and in most cases having more that 5-10 priorities doesn't make much sense.

The plugin will not change to have unlimited priorities but a fixed number as it is today.

If you have a special case like this maybe the Priority Sorter is not your best option, building your own QueueSorter that just sorts on the things you want is probably a much better idea.


Shouldn't there be a $(html).find('.cartodb-popup-content')) somewhere here?


Nice !


TeamCity Civic Assembly :: Build backend + frontend [Build 44](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=571&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Oh my... @scossar can you have a look?


[R/boot_jaatha.R:40:1:](https://github.com/statgenlmu/jaatha/blob/f12cae5fcad4d6ca51c86fed4a128362c2595e5f/R/boot_jaatha.R#L40) *style:* **lines should not be more than 80 characters.**
```r
                          file = tempfile(paste0("boot_log_", Sys.getpid(), "_")),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fit_glm.R:24:1:](https://github.com/statgenlmu/jaatha/blob/f12cae5fcad4d6ca51c86fed4a128362c2595e5f/R/fit_glm.R#L24) *style:* **lines should not be more than 80 characters.**
```r
                     control = list(maxit = 100))[c("coefficients", "converged")]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/initialization.R:105:1:](https://github.com/statgenlmu/jaatha/blob/f12cae5fcad4d6ca51c86fed4a128362c2595e5f/R/initialization.R#L105) *style:* **lines should not be more than 80 characters.**
```r
    block_widths <- utils::head(seq(1, block_width, length.out = n_steps + 1), -1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Shouldn't you add a comment line on my example to say why the `else` part has been added?


This breaks for me on a Touchable.

<img width="602" alt="screen shot 2016-01-29 at 11 14 05" src="https://cloud.githubusercontent.com/assets/211411/12672842/6fc62492-c679-11e5-8581-4bd82af43ddc.png">

Hi. I'm not sure if this part should diverge from Animated code. I think the mapping to CSS should only be done in setNativeProps side.
see my comment here https://github.com/taobaofed/react-web/issues/44#issuecomment-176310054


i'm sorry, didn't had the time to translate the names xD


test


Sure


Nice! I totally missed this opportunity offered by my "stupid but works" token-splitting approach.


thanks, it will save many headaches!


@twangboy Just a nitpick on the Git messages: currently this one is actually a logic error. And it is also better if you could squash some commits, like fixing syntax after adding something. :wink:

Also, I suggest to read this: http://chris.beams.io/posts/git-commit/


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 82](https://teamcity.labkey.org/viewLog.html?buildId=299902&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 15 (2 new), passed: 31 Build time: 01:10:56
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

##### there are 5 more failed tests, see build details
```


Maybe you could add a unit test : input is a list of elements where some are identicals


Why is this class in app/Visualization/UI ? I think it should live under app/Data somewhere as it doesn't actually build any UI, it fetches data from a CartoDB server (even if it's HTML data)...


The `else` part is used to have the sensor keep it's last state if your newest icloud update doesn't have any battery % in it (which happens sometimes). Otherwise the sensor will be blank.


Small typo. `frontend`


I'd log it in `execute`


Thanks


On line 44 you wrote forntend instead of frontend


I'm just curious why don't you use `~> 0.8.6' instead of `'>= 0.8.6', '<= 0.9'` in the gemspec file? In my interpretation they mean the same in this case. Do I miss something?


@gammell Why is this margin not used anymore? Is there an overload for operators > and <? Otherwise how do we ensure robust double comparison?


(And thanks for writing the docs!)


This email is always sent to admin though, customers don't get new order emails.


There is a spell mistake that ''ASII" in line 67 may be wrong. Did you mean "ASCII"?


Can you check if we should use \dump_state or \\dump_state ?


yw :)


Thx for fixinf this annoying bug.


Typo fixed


Personally, I don't like abbreviations in names, because it’s hard to spot when they're ambiguous, so I’d rename this to `metadataElement`


Oh, I see in the source of your comment that it is 2 backslashes.


variable y is useless and is not declared


Remove `currently only String and Boolean properties are supported` from the method documentation comment.


TeamCity Civic Assembly :: Build backend + frontend [Build 44](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=571&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 119 (87 new), passed: 53, ignored: 4 Build time: 00:09:29
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1454062657205: <no details avaliable>

Document tests: Should be able to revert from new version of Document: <no details avaliable>

Document tests: Should be able to propose Document to review: <no details avaliable>

Document tests: Should be able to publish proposed Document: <no details avaliable>

Document tests: Should be able to unlock and revert to draft locked proposed Document: <no details avaliable>

Document tests: Should be able to propose reverted Document to review: <no details avaliable>

Document tests: Should be able to decline proposed Document: <no details avaliable>

Document tests: Should be able to propose, publish and expire Document: <no details avaliable>

Media type tests: Should navigate to Media Types page: <no details avaliable>

User profile tests: Should navigate to user profile page: <no details avaliable>

User profile tests: Should not change user profile info: <no details avaliable>

##### there are 109 more failed tests, see build details
```


> I get all L1 has parent as L0 and L2 has parent as L1. Is this not universal?

Actually no, sorry...this is true only for strict hierarchies. Most countries follow this, but a few have exceptions, including the Philippines, which is an area covered by SAMBRO.
I hadn't even considered this issue before though here (my bad!), but it is another one that needs addressing.

It certainly can be possible that there are the same names in multiple places (e.g. very common in the many PH L4s)
Only L0s are guaranteed to be unique.
What you're trying to do here would resolve this but you've not implemented it correctly. You need the name clause in each level of the queries!

e.g.
query_l1 = (ltable.name.belongs(L1)) & \
                  (ltable.level == "L1") & \
                  (ltable.parent.belongs(location_id))


In this case it can't be, because I have to control the _counter value. It can't be increase or decrease without a condition.


No other review comments. Tested locally and it works as expected.


7ff1515　で解決できたと思います。
同じスレッドからの多重ロックを許可するmutexを使いました。


yes, thanks!


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


you're right, UTs are missing


Me and @boggle hashed it out, and agree that this should be called `ProcReadOnly` (which I guess is called TitleCase). The rule is 'TitleCase for `object`; camelCase if it's a function'.


@kfriedberger  Hi Karlheinz,

one of your commits is breaking tests and checkstyle, could you please take a look and fix?


Who uses tabs anyway!? Great commit!


Can be simplified to :
int ret = ::pread(_fd, buffer, sizeof(buffer), 0);
no?


nice catch, out of curiosity, what's the run time for dialyzing everything?


whats is this ?


Hm, is this one a mistake? It's already added in the CCFLAGS for debug builds and is probably not wanted on release builds, and here it's actually added to the LINKFLAGS, not sure it that works at all.

BTW the osx build seems to mix at will CCFLAGS and CCPFLAGS, not sure if there's a reason cor that.


Can be simplified to :
return _board_voltage_pin->voltage_average();
no?


It is logged in execute down the control flow path (line 395 now)


Can be simplified to :
return _servorail_pin->voltage_average();
no?


ok


I do not say that these two lines are wrong, but up to now we did not have these two lines setting the short version string and setting the cf bundle version. What is the story behind these two new lines?


NULL or nullptr?


There seems to be a problem here. I think `last_line` should be replaced by `first_line` here and in the line below.


Procedures is still in an early stage, and I think you have very few error messages like this written so far. I'd like to suggest that you dedicate one class for containing all of the error messages (through static methods or fields or whatever), making it possible to easily unify language, and to reuse error messages in a non-duplicated manner. What do you think about that idea, in general?


regarding the line break in the log message: we should take into account that line breaks in log messages affects the grepabilty of log files.


Hey Stef -- looks good, but did you mean to delete this line?


CodenvyStackLibraryFilter -> CheStackLibraryFilter


i can confirm its working on samsung galaxy tab 4 (msm8226), really good work!


I'd specify that it's about `inputs.isIncremental` otherwise (IMHO) it can mislead


Muito bem, mano. Me ouve e um dia tu vende esse Zumbi e aí vai chorar lembrado das minhas dicas. Hehehehehe


Exception is swallowed here and code execution continues. Is this intened?


I think only one '\'
from man rigctld:
       Example (Perl): `print $socket "\\dump_caps\n";' to see what the radio's backend can do
       (N.B.:  In Perl and many other languages a '\' will need to be escaped with a preceding '\' so that even though two backslash characters appear in
       the code, only one will be passed to rigctld.  This is a possible bug, beware!).


Debería ser h2


Вот так отлично работает:

modules.define('sitemap', ['i-bem__dom', 'search'], function(provide, BEMDOM, Search) {

provide(BEMDOM.decl(this.name, {
    onSetMod: {
        js: {
            inited: function() {
                var _this = this;
                Search.on({ modName: 'opened', modVal: '*' }, function(e) {
                    _this.toggleMod('opened');
                });
            }
        }
    }
}));

});


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 350](http://ci.infinispan.org/viewLog.html?buildId=34569&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 9e23e9747b4aaa97f81d96ebb7f37b1a6efa3061
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:08:53


I would even introduce a separate variable for `polyStart[...]` everywhere, but I guess this is just a matter of preferences.


I'm not sure, but I think at the time of writing the order was set on the Assay or Sample object (so the assay.samples list would always be sorted). However, I can't find the code in the current domain objects, so I think sorting here would be a good thing to do.


Related to above comment: it'll make writing tests slightly easier too, when asserting on the message itself :)


If they're empty, we should set to empty.


Done for UJF.
FYI, I think I added a new type of data set which could probably be factored-out if several sites want to: data of experiments to be published. I'm willing to state that we plan to adhere to the reproducible research standards, by
1/ publishing all datasets used to draw figures (benchmarks, etc)
2/ and all scripts, and description of software stack (including revision numbers)  and of architecture to allow reproduction of these data.


hehe, Oui, j'ai pas pensé à utiliser les propriétés pour autre chose que les champs. Je débute...

Merci pour la remarque.


ah ok. i'll go with long-sha, i don't have a preference personally


I believe you'd need to adjust line 650 too, to
`Bm(signed char a, signed char b) : l(a), s(b) {}`


That's a leftover from dev code, I'll clean it up, sorry !


Этот глобальный on() мне не нравится. Получается, что наш компонент лезет в DOM, о котором не знает. Он знает только свой DOM. А это рутовый элемент и все глубже. Выше он не видит. А тут по сути навешивается хендлер на весь документ. Нужно юзать рутовый элемент:

rootEl.find('.close').on(...)

Ну тут не вийде так як ти казав "rootEl.find('.close').on(...)" бо елемента '.close' ще нема в домі. Тому я просто додав пошук його через рутовий елемент


wtf


@AByzhynar See 8326b49


test


  for dir_path in /proc/*; do
        case "$dir_path" in
            ('/proc/*[!0-9]*')   continue;;
        esac


Nice to hear :)
I should have a solution for the possible lingering boot.img out soon as well (after I finish merging multirom into official twrp).


era só porque havia quem estivesse com duvidas porque pensava que tínhamos de ser nós a por os dois últimos valores do ficheiro


looks quite strange, to many manipulations with dynamicStore to make dynamicStore happy


Why, why, why BC?