[test.R:1:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L1) *style:* **Commented code should be removed.**
```r
#install.packages("devtools")
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[test.R:2:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L2) *style:* **Commented code should be removed.**
```r
#library(devtools)
 ^~~~~~~~~~~~~~~~~
```
[test.R:3:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L3) *style:* **Commented code should be removed.**
```r
#devtools::use_travis()
 ^~~~~~~~~~~~~~~~~~~~~~
```
[test.R:6:57:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L6) *style:* **Commas should always have a space after.**
```r
  packagestoinstall_split <- strsplit(packagestoinstall,", ")[[1]]
                                                        ^
```
[test.R:8:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L8) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!packagename %in% rownames(installed.packages())){
      ^
```
[test.R:12:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L12) *style:* **Commas should always have a space after.**
```r
      print(paste(packagename,"already installed"))
                              ^
```
[test.R:14:40:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L14) *style:* **Put spaces around all infix operators.**
```r
    library(packagename, character.only=TRUE)
                                      ~^~
```
[test.R:25:4:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L25) *style:* **Put spaces around all infix operators.**
```r
  x+y
  ~^~
```
[test.R:27:6:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/2201bf0234e09255cfdd433c65e16ed54961874f/test.R#L27) *style:* **Commas should always have a space after.**
```r
f1(1,2)
     ^
```


[cm_v1.R:1:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
require(rpart)
              ^
```
[cm_v1.R:1:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:2:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
     ~^~~
```
[cm_v1.R:2:22:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                    ~^~
```
[cm_v1.R:2:60:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                          ~^~
```
[cm_v1.R:2:70:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                                    ~^~
```
[cm_v1.R:5:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L5) *style:* **Trailing whitespace is superfluous.**
```r
require(rpart)
              ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L6) *style:* **Use <-, not =, for assignment.**
```r
mydata$Industry=as.factor(mydata$Industry)
               ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L6) *style:* **Put spaces around all infix operators.**
```r
mydata$Industry=as.factor(mydata$Industry)
              ~^~
```
[cm_v1.R:6:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L6) *style:* **Trailing whitespace is superfluous.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:7:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/cm_v1.R#L7) *style:* **Use <-, not =, for assignment.**
```r
contrasts(mydata$Industry) = contr.treatment(213)
                           ^
```


[data_exploration-12-5.R:9:23:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L9) *style:* **Put spaces around all infix operators.**
```r
hist(RETMONTH, breaks =60)
                      ^~
```
[data_exploration-12-5.R:11:35:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L11) *style:* **Put spaces around all infix operators.**
```r
(re_up <- length(RETMONTH[RETMONTH<0]))
                                 ~^~
```
[data_exploration-12-5.R:12:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L12) *style:* **Put spaces around all infix operators.**
```r
(re_down <- length(RETMONTH[RETMONTH>0]))
                                   ~^~
```
[data_exploration-12-5.R:12:42:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
(re_down <- length(RETMONTH[RETMONTH>0]))
                                         ^
```
[data_exploration-12-5.R:14:6:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L14) *style:* **Put spaces around all infix operators.**
```r
re_up/(re_up+re_down)
    ~^~
```
[data_exploration-12-5.R:14:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L14) *style:* **Place a space before left parenthesis, except in a function call.**
```r
re_up/(re_up+re_down)
      ^
```
[data_exploration-12-5.R:14:13:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L14) *style:* **Put spaces around all infix operators.**
```r
re_up/(re_up+re_down)
           ~^~
```
[data_exploration-12-5.R:16:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L16) *style:* **Put spaces around all infix operators.**
```r
up_down <- ifelse(RETMONTH >0, 1, 0)
                           ^~
```
[data_exploration-12-5.R:27:26:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L27) *style:* **Commas should always have a space after.**
```r
uniqueness <- apply(data,2,numb_unique)
                         ^
```
[data_exploration-12-5.R:27:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L27) *style:* **Commas should always have a space after.**
```r
uniqueness <- apply(data,2,numb_unique)
                           ^
```
[data_exploration-12-5.R:29:29:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L29) *style:* **Put spaces around all infix operators.**
```r
length(uniqueness[uniqueness<50])
                           ~^~
```
[data_exploration-12-5.R:31:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L31) *style:* **Put spaces around all infix operators.**
```r
which(uniqueness <50)
                 ^~
```
[data_exploration-12-5.R:35:32:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L35) *style:* **Only use double-quotes.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                               ^~~
```
[data_exploration-12-5.R:35:40:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L35) *style:* **Put spaces around all infix operators.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                                      ~^
```
[data_exploration-12-5.R:35:42:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L35) *style:* **Only use double-quotes.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                                         ^~~~~
```
[data_exploration-12-5.R:36:9:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L36) *style:* **Put spaces around all infix operators.**
```r
abline(h= mean(RETMONTH[n_low:n]), col = "blue", lwd = 2)
       ~^
```
[data_exploration-12-5.R:38:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L38) *style:* **Put spaces around all infix operators.**
```r
MSE_i <- (RETMONTH[n_low:n]-mean(RETMONTH[n_low:n]))^2
                          ~^~
```
[data_exploration-12-5.R:38:53:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L38) *style:* **Put spaces around all infix operators.**
```r
MSE_i <- (RETMONTH[n_low:n]-mean(RETMONTH[n_low:n]))^2
                                                   ~^~
```
[data_exploration-12-5.R:44:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L44) *style:* **Put spaces around all infix operators.**
```r
n_low <- n-1000
         ~^~
```
[data_exploration-12-5.R:44:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L44) *style:* **Trailing whitespace is superfluous.**
```r
n_low <- n-1000
               ^
```
[data_exploration-12-5.R:65:21:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L65) *style:* **Commas should always have a space after.**
```r
xs <- pca$x[n:n_low,1:9]
                    ^
```
[data_exploration-12-5.R:71:30:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L71) *style:* **Commas should always have a space after.**
```r
data2 <- data_no_ind[n_low:n,]
                             ^
```
[data_exploration-12-5.R:79:26:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L79) *style:* **Put spaces around all infix operators.**
```r
fit <- glm(updown~., data= data3, family = "binomial")
                        ~^
```
[data_exploration-12-5.R:83:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L83) *style:* **Put spaces around all infix operators.**
```r
cv_est <- cv.glm(data3, fit, K= 10)$delta[1]
                             ~^
```
[data_exploration-12-5.R:85:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L85) *style:* **Put spaces around all infix operators.**
```r
1-cv_est
~^~
```
[data_exploration-12-5.R:86:1:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/data_exploration-12-5.R#L86) *style:* **Trailing blank lines are superfluous.**
```r

^
```


[GPUAcceleratedClustering.R:4:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L4) *style:* **Only use double-quotes.**
```r
install.packages('rpud')
                 ^~~~~~
```
[GPUAcceleratedClustering.R:11:17:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
  set.seed(seed)
                ^
```
[GPUAcceleratedClustering.R:12:32:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L12) *style:* **Put spaces around all infix operators.**
```r
  matrix(rnorm(dim * num), nrow=num)
                              ~^~
```
[GPUAcceleratedClustering.R:21:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L21) *style:* **Variable and function names should be all lowercase.**
```r
  dgpu <- rpuDist(matrix)
          ^~~~~~~
```
[GPUAcceleratedClustering.R:21:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L21) *warning:* **no visible global function definition for ‘rpuDist’**
```r
  dgpu <- rpuDist(matrix)
          ^~~~~~~
```
[GPUAcceleratedClustering.R:22:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L22) *style:* **Variable and function names should be all lowercase.**
```r
  rpuHclust(dgpu)
  ^~~~~~~~~
```
[GPUAcceleratedClustering.R:22:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L22) *warning:* **no visible global function definition for ‘rpuHclust’**
```r
  rpuHclust(dgpu)
  ^~~~~~~~~
```
[GPUAcceleratedClustering.R:35:29:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L35) *style:* **Put spaces around all infix operators.**
```r
print(system.time(cpuhclust <-run_cpu(m)))
                            ^~~
```
[GPUAcceleratedClustering.R:43:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/GPUAcceleratedClustering.R#L43) *style:* **Commas should always have a space after.**
```r
plot(gpuhclust,hang = -1)
               ^
```


[NeuralNet.R:6:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L6) *style:* **Only use double-quotes.**
```r
install.packages('neuralnet')
                 ^~~~~~~~~~~
```
[NeuralNet.R:12:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L12) *style:* **Use <-, not =, for assignment.**
```r
stockdata = read.csv("mf850-finalproject-data.csv")
          ^
```
[NeuralNet.R:13:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L13) *style:* **Use <-, not =, for assignment.**
```r
stockdata$Date = NULL
               ^
```
[NeuralNet.R:14:20:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L14) *style:* **Use <-, not =, for assignment.**
```r
stockdata$Industry = NULL
                   ^
```
[NeuralNet.R:15:14:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L15) *style:* **Use <-, not =, for assignment.**
```r
scaled_Stock = scale(stockdata)
             ^
```
[NeuralNet.R:16:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L16) *style:* **Use <-, not =, for assignment.**
```r
y = stockdata$RETMONTH_SPX
  ^
```
[NeuralNet.R:17:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L17) *style:* **Use <-, not =, for assignment.**
```r
x = stockdata
  ^
```
[NeuralNet.R:18:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L18) *style:* **Use <-, not =, for assignment.**
```r
x$RETMONTH_SPX = NULL
               ^
```
[NeuralNet.R:24:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L24) *style:* **Commas should always have a space after.**
```r
trainingdata <- cbind(traininginput,trainingoutput)
                                    ^
```
[NeuralNet.R:25:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L25) *style:* **Commas should always have a space after.**
```r
colnames(trainingdata) <- c("Input","Output")
                                    ^
```
[NeuralNet.R:31:36:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L31) *style:* **Commas should always have a space after.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                   ^
```
[NeuralNet.R:31:56:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L31) *style:* **Put spaces around all infix operators.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                                      ~^~
```
[NeuralNet.R:31:71:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L31) *style:* **Put spaces around all infix operators.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                                                     ~^~
```
[NeuralNet.R:38:27:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L38) *style:* **Place a space before left parenthesis, except in a function call.**
```r
testdata <- as.data.frame((1:10)^2) #Generate some squared numbers
                          ^
```
[NeuralNet.R:38:33:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L38) *style:* **Put spaces around all infix operators.**
```r
testdata <- as.data.frame((1:10)^2) #Generate some squared numbers
                               ~^~
```
[NeuralNet.R:48:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L48) *style:* **Commas should always have a space after.**
```r
cleanoutput <- cbind(testdata,sqrt(testdata),
                              ^
```
[NeuralNet.R:50:36:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L50) *style:* **Commas should always have a space after.**
```r
colnames(cleanoutput) <- c("Input","Expected Output","Neural Net Output")
                                   ^
```
[NeuralNet.R:50:54:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/NeuralNet.R#L50) *style:* **Commas should always have a space after.**
```r
colnames(cleanoutput) <- c("Input","Expected Output","Neural Net Output")
                                                     ^
```


[test.R:1:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L1) *style:* **Commented code should be removed.**
```r
#install.packages("devtools")
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[test.R:2:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L2) *style:* **Commented code should be removed.**
```r
#library(devtools)
 ^~~~~~~~~~~~~~~~~
```
[test.R:3:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L3) *style:* **Commented code should be removed.**
```r
#devtools::use_travis()
 ^~~~~~~~~~~~~~~~~~~~~~
```
[test.R:6:57:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L6) *style:* **Commas should always have a space after.**
```r
  packagestoinstall_split <- strsplit(packagestoinstall,", ")[[1]]
                                                        ^
```
[test.R:8:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L8) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!packagename %in% rownames(installed.packages())){
      ^
```
[test.R:12:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L12) *style:* **Commas should always have a space after.**
```r
      print(paste(packagename,"already installed"))
                              ^
```
[test.R:14:40:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L14) *style:* **Put spaces around all infix operators.**
```r
    library(packagename, character.only=TRUE)
                                      ~^~
```
[test.R:25:4:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L25) *style:* **Put spaces around all infix operators.**
```r
  x+y
  ~^~
```
[test.R:27:6:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/6716eb22d0dc851f76e534559b74e685bfbc377f/test.R#L27) *style:* **Commas should always have a space after.**
```r
f1(1,2)
     ^
```


good


是否是漏上传了一个common工程？com.ctrip.infosec.common，
下载下来的代码报错，缺少jar


해당하는 액티비티에 configChanges="orientation|keyboardHidden|screenSize" 한줄 추가 해주면
화면회전시 oncreate 재호출을 방지해준다.


Good catch!


lihaile


이 화면을 구성하는 xml 파일에
<com.google.android.youtube.player.YouTubePlayerView
    android:id="@+id/youtubeplayer"
    android:layout_width="fill_parent"
    android:layout_height="fill_parent">
</com.google.android.youtube.player.YouTubePlayerView> 필요하며

이 구성품을 쓰기위해서는 그 액티비티가 YoutubeBaseActivity를 상속받아야 한다.


Yes, because pull from the bottom will let me load more items, paginaiton like
http://forums.crackberry.com/attachments/blackberry-passport-f434/328790d1421900337t-cb-pull-up-load-more-using-capacitive-keyboard-img_20150122_071452.png


Thank you, i did this for the C side way back and never implemented the others.


delete


Documentation looks much prettier when you are using **Mardown**

[https://guides.github.com/features/mastering-markdown/](https://guides.github.com/features/mastering-markdown/ )


@geofftrojans I think we are okay. We will probably relook at this whole user thing.  I actually am mixing metaphors a bit, I think.  We started with users more like personas, and my user entry is more of a role.  So, what I mean is that one way to do UX is to think of personas accessing the site.  But when we get down to users in a more programmatic sense, then we will need to think of roles.  Even then, user is wrong...  it should be "authenticated user" probably... but "authenticated user" wouldn't register, "anonymous user" would register.  So, I will have to split these concepts out.  But I think we are good for now.  This is doing what I hoped it would.  You have come up with a bunch of good ideas.  I will add an "anonymous user" to the mix.  I will try to think of a few more tasks for these users.  Then I will make a big master list of the tasks, and start to think of how they fit into roles and then we can work on what roles we want to have on the site, which will have us start moving to think about our security model.


Any interest in making a pull request for this fix?


@NetanelFelcher : when you mention an issue, please say why you mentiong it. Cryptic comments are not a good idea at all.

#6

@orimarco @idabran @zvili @yh-github @AdiOmari


Can we get some versionadded annotations on this?


get


test


Can we please change this back? I don't care what the linter says about this, but how it was is a lot cleaner than what is was changed to. The whole idea here is to display a "matrix" how the args are created and in which order the values are picked.


As said above, the index should be between 0 and 1.
Your implementation is quadratic even though the time required should be linear.

@AdiOmari


Not sure which errors would come through here, never saw one. Perhaps errors when you can't reach the host?


请问下这个工程是什么工程？如何才能正常跑起来？


This we should just change to `callback = callback || (() => {})` as it's a no-op.


Good!


Hi Dave

Thank you very much!

Just a little comment: starting from
https://new.galaxyproject.org/GalaxyUpdates/2016_12/  , I follow the
link: https://new.galaxyproject.org/Community/GalaxyAdmins/Meetups/2016_12_15/index.md
  and end up with : '404 Not Found'

The link should go
to:https://new.galaxyproject.org/Community/GalaxyAdmins/Meetups/2016_12_15/
 shouldn't-it?

Regards, Hans-Rudolf

On Tue, Dec 6, 2016 at 10:27 PM, Dave Clements <notifications@github.com> wrote:
> December GalaxyAdmins Announcement is live
>
> Attention: @hrhotz, @bgruening, @bebatut, @dannon
>
> --
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub:
> https://github.com/galaxyproject/galaxy-site/commit/0d4842b914c5727535fb7a52d40a1661e536b412


@Danack This causes error in PHP 7.0.13 (VC14):
```
C:\php-sdk\php70dev>nmake php_imagick.dll | grep error

Microsoft (R) Program Maintenance Utility Version 14.00.24210.0
Copyright (C) Microsoft Corporation.  All rights reserved.

NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\cl.exe"' : return code '0x2'
Stop.
ext\imagick\imagick_class.c(12532): error C2059: syntax error: '<parameter-list>'
ext\imagick\imagick_class.c(12538): error C2106: '=': left operand must be l-value
ext\imagick\imagick_class.c(12542): error C2223: left of '->severity' must point to struct/union
ext\imagick\imagick_class.c(12543): error C2223: left of '->reason' must point to struct/union
ext\imagick\imagick_class.c(12544): error C2106: '=': left operand must be l-value
ext\imagick\imagick_class.c(12548): error C2106: '=': left operand must be l-value
```


I suspect this is more in line with what @kimikul had in mind now that I have some inkling of what they are expecting.  Unfortunately, I don't have the luxury of spending a lot of time on non-paying coding as my financial situation is critical and I'm just cranking stuff out - working 4 projects right now 1 laravel/php, 1 opencart php, 1 sabre-io php/angular, and 1 small ios maintenance project.  I kind if figured my job history would count for something and at this phase in my career - this kind of thing is pretty much bullshit.  I wonder if the CFO had to take a test on basic arithmetic?


I needed a way to change the password with a script and used the first trick I could find:
Create hash with openssl passwd and put that to /etc/shadow.

And yes, I know that's not a safe way to do it because it uses md5.


Oh,,, and one ruby on rails project - so 5.  Would be great to have only one thing to work on so I could focus.


I'd really recommend you to invert the config and only include what you _really_ want to ship and what is used in deployments/installs.

examples, documentation, tests, (dev)config files, development files or stuff like
```
-rw-r--r--   1 XXX  XXX     0B Oct  7 06:28 .node-xmlhttprequest-sync-2758
-rw-r--r--   1 XXX  XXX   1.3K Nov 25 23:32 workspace.json
```
(yes that was in one of your publishes)

Really shouldn't be published.

I believe a config like so:

```
/*
!/dist
!/resources
```
would be better, you might have a couple of more files you need to ship for the package to be functional, I've not dug into it.

NPM will automatically include the `license.txt` and the `readme.md` so no reason to add those.


I do. It lets me know to which part of the code a function belongs, and
distinguish different functions with identical names.
Could you add a flag somewhere that would allow this?

On Wed, Dec 7, 2016, 04:28 Evan Hempel <notifications@github.com> wrote:

> I didn't see filenames listed in other flamegraph implementations, and in
> my use didn't find any time where I needed it to distinguish between
> identically named functions.
>
> Have you had a need for it?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/evanhempel/python-flamegraph/commit/7ddcc7246aae4ddccafc27d8685670ababe9a722#commitcomment-20097574>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AALBR569fClnUP4n_bcet_xfODr7gKgjks5rFhm9gaJpZM4LFiJH>
> .
>


@fritx :laughing:


	//networkPlugin.sendData('/api/users/1/templates/', {'subject' : '���', 'contents' : '����'}, "POST", function(result) {
if(result.data>0){
   성공
}
}


Afaik wpcom might still reject php7 specific code on commit, for now.


Hello Darkness my old Friend


Needs fix in https://github.com/libretro/libretro-super/blob/master/dist/info/2048_libretro.info
Might be a good idea to compare the other cores and see if we have more license issues.


I'm using the version before you removed the commits without issues. But if
I ever update pvparena now, how to reset their speed?

Op wo 7 dec. 2016 00:53 schreef slipcor <notifications@github.com>:

> but this is the reason why I removed the commits, BECAUSE the speed was
> messed up. Now I do not mess with speed and everything is fine again :)
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/slipcor/pvparena/commit/0fb1730ce1b3188ef56000afeac6531151e448b9#commitcomment-20096245>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AFZthzZSaaM9nwMFJh2pt_b1kDVMbpOQks5rFfVtgaJpZM4LEhst>
> .
>


Local LTC-Test
Score of pgo_7f167a2 vs pgobase: 120 - 125 - 757  [0.498] 1002 ELO difference: -1.73 +/- 10.63
SPRT: llr -0.719, lbound -2.94, ubound 2.94


Finish Homework 9


It was not my intention to copy from your work. After having a look at other review I felt the need to include more content.
The readme-section is taken from the coursera-page - see here: https://www.coursera.org/learn/r-packages/peer/25XGp/documenting-code
Hence, I do not intend to change the readme-file.
Concerning the vignette file: I think I did not put enough effort into this one. I will update this one asap. I hope that all problems will be solved with the new commit.


:+1 sorry!


Fixed all the build errors we had previously by updating travis.yml to use jdk8


Зачем вот этот setState ?
Зачем вообще update() который обновляет стейт, который ты нигде не используешь ?


nice 搞起搞起


OK, I've started forking upstream repos to kodi-game and creating a single `kodi-fixes` branch. As I go through your updates and my own testing, I commit each fix I make. Once we're through the cores, we'll have a long list of repos with upstream fixes, and we can go down the list and open a single PR for all the cores in one go.


вот это setFlag: PropTypes.func }; не очень красиво смотриться.
дужка должна быть на новой строке.


дужечки


@florianrusch Take a look at our discussion on #22
In c11b98d190e4ef6bad767ac39c3fe9c793c4eddb I added the DashBoardController as an attribute of the dashboard.fxml
When you generate the Controller from the fxml it automatically becomes an attribute, so you don't have to set it with the fxmlLoader.


travis is not passing because on the checks on the filesizes. its showing "wrong" sizes because of commit 3683698f4acdfa5b5918f1c95fa18372e395a6e8, take a look @soywiz


Здесь то надо было метод distance дернуть
p1.distance(p2);


сделай так что бы не пришлось постоянно писать this.props.xxxx


Or we could even create a separate org for the forked libretro repos with upstream patches. It's a complexity-organization tradeoff, though i don't really care which way we tip


ok, will do - thank you.


Вот только в России этот регион называется «Республика Крым». Поправьте, если ошибаюсь.


In getting started https://deployer.org/docs/getting-started


везде ты пишешь как func = () => {
                                    }
в этом не так, почему ?


just comment over chetan edits


Then there should be another function that behaves as you mention. The function copyDirTo is used in many other places and it behaves as I explained.


Awesome to see fixing this bug involves a code cleanup like this :)


The quit is when cache expires or update-lock is released after update or patch. The update lock is currently set to 10 seconds by default. Normal updates are done in a lot less. The lock is set for any collection and has no configuration.


дужка


весь этот конструктор он не нужен.
стейт можно инициализировать и не в конструкторе, убери его.


I thought you were talking about the Java variable name. Changing the environment variable name from `http_proxy` to `httpProxy` might not be a good idea because `http_proxy` is the de facto standard, so most user behind a proxy will already have this variable set (and often additionally `HTTP_PROXY`)


NoNo


yes sir


Hello, World!

I am brand new to coding, with very little coding knowledge to start with, but I am excited nonetheless to start this new journey.


What I've done so far was just forking repos in my useraccount, and PRing directly from there. no strong opinion on this, all 3 options are fine by me. but we should not let these patches age to long in our own repos as upstream acceptance was really fast. In one case it was even faster than I could push the patch file.


@spastorino any reason to this reverting? I think the `reverse` way is more difficult to either read or figure out the purpose without comments.


Does this still have the bug we identified?


1. DOM元素的几何属性变化

　　当DOM元素的几何属性变化时，渲染树中的相关节点就会失效，浏览器会根据DOM元素的变化重新构建渲染树中失效的节点。之后，会根据新的渲染树重新绘制这部分页面。而且，当前元素的重排也许会带来相关元素的重排。例如，容器节点的渲染树改变时，会触发子节点的重新计算，也会触发其后续兄弟节点的重排，祖先节点需要重新计算子节点的尺寸也会产生重排。最后，每个元素都将发生重绘。可见，重排一定会引起浏览器的重绘，一个元素的重排通常会带来一系列的反应，甚至触发整个文档的重排和重绘，性能代价是高昂的。

　　2. DOM树的结构变化

　　当DOM树的结构变化时，例如节点的增减、移动等，也会触发重排。浏览器引擎布局的过程，类似于树的前序遍历，是一个从上到下从左到右的过程。通常在这个过程中，当前元素不会再影响其前面已经遍历过的元素。所以，如果在body最前面插入一个元素，会导致整个文档的重新渲染，而在其后插入一个元素，则不会影响到前面的元素。

　　3. 获取某些属性

　　浏览器引擎可能会针对重排做了优化。比如Opera，它会等到有足够数量的变化发生，或者等到一定的时间，或者等一个线程结束，再一起处理，这样就只发生一次重排。但除了渲染树的直接变化，当获取一些属性时，浏览器为取得正确的值也会触发重排。这样就使得浏览器的优化失效了。这些属性包括：offsetTop、offsetLeft、 offsetWidth、offsetHeight、scrollTop、scrollLeft、scrollWidth、scrollHeight、clientTop、clientLeft、clientWidth、clientHeight、getComputedStyle() (currentStyle in IE)。所以，在多次使用这些值时应进行缓存。

　　此外，改变元素的一些样式，调整浏览器窗口大小等等也都将触发重排。

　　开发中，比较好的实践是尽量减少重排次数和缩小重排的影响范围。例如：

　　1. 将多次改变样式属性的操作合并成一次操作。例如，

　　JS:

var changeDiv = document.getElementById(‘changeDiv’);
changeDiv.style.color = ‘#093′;
changeDiv.style.background = ‘#eee’;
changeDiv.style.height = ’200px’;

　　可以合并为：
　　CSS:

div.changeDiv {
background: #eee;
color: #093;
height: 200px;
}

　　JS:

document.getElementById(‘changeDiv’).className = ‘changeDiv’;

　　2. 将需要多次重排的元素，position属性设为absolute或fixed，这样此元素就脱离了文档流，它的变化不会影响到其他元素。例如有动画效果的元素就最好设置为绝对定位。

　　3. 在内存中多次操作节点，完成后再添加到文档中去。例如要异步获取表格数据，渲染到页面。可以先取得数据后在内存中构建整个表格的html片段，再一次性添加到文档中去，而不是循环添加每一行。

　　4. 由于display属性为none的元素不在渲染树中，对隐藏的元素操作不会引发其他元素的重排。如果要对一个元素进行复杂的操作时，可以先隐藏它，操作完成后再显示。这样只在隐藏和显示时触发2次重排。

　　5. 在需要经常获取那些引起浏览器重排的属性值时，要缓存到变量。


I'm not concerned about the quality for now. I just want to avoid copyright issues.


Use the current palette for colors. I would try [QGuiApplication::palette().window()](http://doc.qt.io/qt-5/qpalette.html#window) here.


If nothing is selected then add the row at the end of the list, i.e. set `row = query_table->rowCount() - 1`.

Note that you need `row + 1` in the following lines. It makes sense to do the increment already in initialization (and adjust the above 'if' statement accordingly).


Similar to `addRow()`, delete the last row here.


QPalette::window()


The creation of the QueryOperation can be factored out to a separate method `std::unique_ptr<QueryOperation> makeQuery() const`. The invalid-query case would be need to be signaled in a different way, of course.


I was tempted to suggest moving this to a method in QueryOperation (for reuse and testing). But while MapPart doesn't (yet) provide direct access to the underlying collection of objects, it does provide its own type of visitor algorithm: `MapPart::existsObject`, `MapPart::applyOnAllObjects` and `MapPart::applyOnMatchingObjects`.
So my suggestion is to implement a `bool QueryOperation::operator()(Object* object) const` instead of `evaluate` so that the loop can be rewritten by a lambda operation plus the QueryOperation as condition.
Then rewrite the remainder with `MapPart::existsObject` and `MapPart::applyOnMatchingObjects`, using a lambda function operation which adds the matching objects to the selection without creating the intermediate list first.

I can do this after the merge, or push a commit to your branch for review once you created a pull request.
I would probaby move QueryOperation to a separate file in /src/ then (no GUI dependencies, could get a unit test).


I wonder if this is neccessary here. The tools should watch for selection changes, and trigger switching IIRC.


Requires a plural translation:
`tr("%n object(s) selected", "", matches.size())`

`matches.size()` might need to become `map->selectedObjects().size()`.

The plural translation can go directly to the `setText()` in the next line.


I would use a single return statement:
~~~
return tags.contains(key_arg)
       && tags.value(key_arg) == value_arg;
~~~

Similar for the following lines.


Single statement, too:
~~~
return left_arg->evaluate(obj) || right_arg->evaluate(obj);


Nhóm gồm 5 thành viên


good point, this makes much sense


lepiej sobie przypisać $('body') do jakiejś zmiennej, jeżeli będzie wykorzystywana więcej niż 1 raz


:+1: In total, I like this work very much.


- Huỳnh Thanh Thảo Linh


ㅁㄴㅇㄹ


dđ


Yes, I think 64 bits should be enough.


why did you revert this? does it not work?


@mocchira LGTM, thanks


There are a lack of subquestion for array text and array number question here :) : subquestion1 and subquestion2 see https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/viewHelper.php#L105


真愛就值得珍惜，支持修改民法972，支持婚姻平權。


либо пишешь setState в одну строчку, либо в три. но ни как не в две.


Adding implement of minhash, will produce .sig file for each .fna read, can use distance.h to calculate


ok


public void createContact(ContactData contactData) {


OMG ur so smart, plz teach me! I trade knowledge for detective squirrel :squirrel:


Have you finished it?can you send it to me? 2920215396@qq.com


@at3no コミット粒度とメッセージ、どちらも良いですね！素晴らしいです！！


sprawdź czy name nie jest pusty


Why not just `quorum_after_change = raft::calc_quorum(status.progress.len())`？


Hi guys - one thing to bear in mind as discussed with Tom is the way client side routing works.  When you move from one page to another on the client the URL is re-written e.g. petabencana.id/cards,  petabencana.id/map etc however if you were to enter this in the URL bar in your browser or hit refresh in any page other than home you would get a 404 error, the reason being that the “server” (CloudFront) only knows how to render index.html, everything else is happening in Javascript.  The way I have overcome this with other SPA applications I have worked on (using React) is to setup a custom CloudFront error handler to redirect all 404s to index.html and return the status as 200, the router *should* then be smart enough to know that the re-direct came from, say petabencana.id/cards, re-write the URL and present the correct page.  Will need testing with aurelia router as my experience here is limited to the couple of days I spent on the PoC.

This is a screenshot from one of my apps.

Mark

>


- [ ] why RelativeLayout - > LinearLayout


Moving target, will try to find a long term solution


j


<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:text="@string/text_Spots" **使用alt+enter快速添加**
..:textSize="24sp"/>**要用sp**


Dear GitHub:
    This is my first using


We can put a version in our mirror


### Lipsum Really?

So we _really_ were not able to find a better filler text?


Yes, but if we don't do it in time we lose the file (which is what happened here - I went to update the mirror but imagemagick had already bumped to a new version and removed the version we are using) so if there's a complete archive available that would be a better option.


can*


Bitmaps: https://lmgtfy.com/?q=adafruit+gfx+bitmap

The SD card is on the same SPI bus and has it's own chip select, you use it like any other SD card on the Arduino.


Sean C.Campbell  Merchant Coinbase Bloomberg

On Dec 7, 2016, at 12:59 AM, sean campbell <seanc11.sc@outlook.com> wrote:

I am interested in 11001 N 50th St, Scottsdale, AZ 85254.35994-128205 INSTORE FINANCING PDF CERTIFICATE

image1.PNG

Sean C.Campbell  Merchant Coinbase Bloomberg 35994-128205 INSTORE FINANCING PDF CERTIFICATE

Sean C.Campbell  Merchant Coinbase MO7 Customer Posting 3$660M.Millions USD PDF Payments.pdfText File 7braintree.txt

Sean C.Campbell  Merchant Coinbase Bloomberg


'dirName' is not clear. change to 'outputDirName'.


ini buat apa om @arissupriy ?


If you do not understand, it does not mean that it is not necessary to anybody.


Great


这里包含了增删改查的所有操作和路由（接口）


Perfect dude! Works exactly like i used to know rbenv & buby-build!! Thanks!


test comments on commit


this assumes that there are no '\n' withouth a file name. the code will not work if the input looks something like:
file1.txt \n
\n
file2.txt \n
\n


@nkzawa Check out the logs in your terminal when deploying. Once the site has finished uploading, you'll see "Running on http://localhost:<port". That makes no sense because it's neither running on localhost, nor is it running on the port that's shown. It will simply be exposed on the URL that the user already received in the beginning right after the project started uploading.

Therefore, showing this message is basically a useless duplicate. Added the same for [list](https://github.com/zeit/list/blob/master/bin/list#L311). 😊


All letters in downcase please ;)
spawn-ressources-in-chest


# Header**wehoghwe**
_italic_


this should be exported to a function 'getNumberOfFileToTest()'


I made a stupid copy paste error apparently. I get 21.73% instead of 23.73%. With the same population data as me, that video gets 21.79%, so the video's config is not optimal (at least not with the 2015 population data, maybe it was optimal in 2011). Just goes to show :) Fixing


need to write the declerations for these variables in the beginning, not the middle. but still better to create them in a different function


Was the recovery of this file intentional? It isn't used anywhere and was renamed to `5.nix` in bf570a494abfbf093afe90f19b50c33a86023498


The problem is that you have specified the directory "$YOUR_OPENCV_CONTRIB_PATH/opencv_contrib-3.1.0/modules" as the directory where OpenCV is installed. In Unix systems the environment variables used in shell scripting are names in capital letters preceded by the '$' sign, as in this case; the meaning is that you have to set the directory name of where *you* installed the OpenCV contrib.
For example, if you have installed the contribs in the directory "/home/ubuntu/opencv_contrib-3.1.0", you have to launch cmake with: -D OPENCV_EXTRA_MODULES_PATH=/home/ubuntu/opencv_contrib-3.1.0/modules


That's cool, I just got started with mina, and I needed this feature urgently for a project that I was migrating from capistrano. Your version looks much more sturdy meanwhile :)


wow 😄


The logo looks blurry


Hi Gabriel, thanks for the answer!  that's right, the 2.7 python is under discussion.
@RalfHauenschild  gets a key error, because Pool does not support context management, as I understand.
It seems that in this case the old line is more robust `pool = multiprocessing.Pool(int(options.np))`

[Here](http://stackoverflow.com/a/25968716) is a similar problem about usage of `Pool` and `with` statement.

> New in version 3.3: Connection objects now support the context management protocol – see Context Manager Types. __enter__() returns the connection object, and __exit__() calls close().


@wgordon17, a mere consistency concern that was brought to my attention...


again - need to write the declerations for these variables in the beginning, not the middle. but still better to create them in a different function to initiate *FilesToTestLengthArray


# H1
## H2##
### H3
**bold**
_italic_

> insert a quote

> insert a quote
![Uploading DingTalk20161207161627.png…]()
![Uploading DingTalk20161207161627.png…]()

> `import java`
`import com.alibaba.android.rimet`
`public static void main`
[This is a link Test](www.baidu.com)

- bulleted list
- bulleted list

1. numbered list

2.numbered list

- [ ] task list


noted


表とか


It introduced a behavior change which caused the following test to fail:
https://github.com/zammad/zammad/blob/develop/test/unit/cti_caller_id_test.rb#L208

Previous and now current behavior:
Go through both levels and get entries only for the current level. If nothing was found check every other entry.

PR behavior:
Get all entries ordered by the levels.

JFI: The debug code will get removed with my next pushes. Oops 🤓


@smeyer-ilias, What do you think about these changes? We won't need to force additions in this case anymore. But we need to take care of this, if we merge to the core, since .gitignore must not be changed there.


This is because #merge involves a checkout and if you checkout origing/<mergeDestinationBranch> you will be checking out a detached HEAD pointing to the latest commit in a remote branch, which is not useful for merging. Instead, if you provide just the #basename (i.e. <mergeDestinationBranch> without 'origin/'), the backend will always checkout a local branch (creating it if it does not exist previously), which will be suitable for merging onto it.


Das würde ich ungern so machen.


![Uploading DingTalk20161207161627.png…]()


I think it's
List(Daily = "...")
Rather than
"List(daily= ...)"


Fix of #431


I think you can add the phar as a library in phpstorm.


public void createContact(ContactData contactData)


fillContactForm(contactData);


'outputLogFile' should be 'fileTestOutputLogPath'


this should definitely go to an outside function


Right! 30f1b52


Right, but it shouldn't have much impact. The clean-up is not stored, just applied before query type detection.


not work

sample:

[[if? &is=`1+1` &math=`on`]]     //return 2   not 1+1
[[if? &is=`$_GET['test']` &math=`on`]]     //return value of $_GET['test'];
[[if? &is=`[+ditto_iteration+]+1:%:3` &then=`<div class="clear"></div>` &math=`on`]]


@jjethwa Could you please elaborate this change? I don't understand why this is needed.

Quoting [Docker's best practices](https://docs.docker.com/engine/userguide/eng-image/dockerfile_best-practices/#volume):
> The VOLUME instruction should be used to expose any database storage area, configuration storage, or files/folders created by your docker container. You are strongly encouraged to use VOLUME for any mutable and/or user-serviceable parts of your image.


You have to, if you want to give him some attributes, like the `exportModel`


add 'numOfRunningProcess' to the for loop above:
if (retVal == 0){...}
else {
...
numOfRunningProcess++;
}


Андрей Николаевич, я тоже вместо `"Здравствуй, мир!"` всегда так пишу. Это намного короче, и клавиши в одном ряду справа налево, пальцы сами набирают.


Hi, i see that you added https support. Would you consider make a pull request to upstream?


Yeah, sorry about that...


я отписалась от тебя! кышкыш)


фу, херня твой ридми


I agree but it shows that the server is ready too.
Additionally, we don't want to add this check for each platform like the following

```js
if (!process.env.NOW && !process.env.FOO && !process.env.BAR) {
  console.log(`> Ready on http://localhost:${argv.port}`)
}
```

Let's consider an alternative way.


The two missing commits (see previous) have be synchronized finally with this commit. My warning in the commit message can be ignored.


change it to self.apikey = apikey , otherwise "global name string is not defined"


사용한 library
build.gradle 파일에  이 네 줄이 들어가있으며
 compile files('libs/commons-codec-1.9.jar')
 compile files('libs/commons-logging-1.2.jar')
 compile files('libs/httpcore-4.2.4.jar')
 compile files('libs/httpclient-4.2.5.jar')

Project Structure
 - Dependencies 안에 + 해주면 된다.

사용한 라이브러리
httpcore-4.2.4.jar
httpclient-4.2.5.jar
commons-codec-1.9.jar
commons-logging-1.2.jar


That's a good looking effect, I like it :+1:

Could you had a "delay" parameter to summon's API ? That's useful for me in [Dungeon Mercenary](http://www.schplaf.org/hgames) and I think it is useful generally speaking. For example to throw a potion, I would enqueue a sliding effect and a summon event, with the summon event delayed by the slide's duration.


You filed a CQ for this... right?
Please refrain to commit code before filing a CQ, as I don't think you would want the git history to be rewritten should there be an issue with a dependency.
/cc @ctron


interface all the things? :)


a


Hello, world3!


It shouldn't try to parse username from within password.
That's a hack for PASS command, for clients which don't let users to configure username which is sent in USER.
SASL PLAIN has a special field for username.


Modules should be able to add new mechanisms (e.g. certauth for EXTERNAL). Though for now let's keep it simple.


good idea, I created #20804 so we don't forget about this


As I understand it, `*` should be checked in the other branch, `if (m_bSaslAuthenticating) {`, and to do `m_bSaslAuthenticating = false;` in it.


#pull


not performMoveOf()
it's getPredictMoveOf(board) function


Actually, I don't think we need this part. Subscriptions added using this.subscribe (as opposed to mx.data.subscribe) are unsubscribed on destroy automatically. That's the idea ;)


Looks good!
Is a trap limited to function scope?


Error Message:
Error: /Stage[main]/Tftp/Service[tftpd-hpa]: Provider upstart is not functional on this host
OS: Ubuntu 16.04.1 LTS

puppet --version
4.8.1


Nueva poesia


Looks like this has been fixed differently elsewhere: https://github.com/liris/websocket-client/pull/286/files


![](https://github.trello.services/images/mini-trello-icon.png) [Bug 1012586 - [doc] Upgrade chapter is wrong](https://trello.com/c/xSmGntgH/2243-bug-1012586-doc-upgrade-chapter-is-wrong)


I didn't finish the review yet. This function confuses me. I need to think more about it.


Nice! I've been using this for almost a year now, but never PR'd it. Glad to see an official version. Yours is definitely more thorough.

https://github.com/forerunnergames/peril/blob/develop/client/src/main/java/com/forerunnergames/peril/client/io/ShaderProgramLoader.java


If you look at the png image, you can see it's a stupid copy paste image, because half of 43.46 is 21.73, not 23.73 :)


In C++11, `noexcept` is generally recommended over `throw()`.


eg → http://lydie-chen.strikingly.com/


i have tested it and it appears so


Done!


Thanks @bgruening !


Nope, at the moment this class is purely used internally by the `Compiler` and these classes are event instantiated inline: https://github.com/shopware/shopware/commit/051434594076e8da31dd0050f62f7665d85c5349#diff-d6a9b5e7180afa2e033f1eaa53721fd5R122

This was a internal refactoring and was not meant to open more extension points.


mohli bychom z toho odstranit toho usera a dát to do promennych travise?


if(false!=str&&null!str)
{
    System.out.println("success for date!");
}


Automatic build report for commit 9da950b136919935e7168dcc5bfe2e40dddeb3cc:

[![Unit Tests](https://img.shields.io/badge/unittests-97%20/%2097-brightgreen.svg)](#)
[![PyLint](https://img.shields.io/badge/pylint-9.91-brightgreen.svg)](#)
[![Documentation](https://img.shields.io/badge/doc-passed-brightgreen.svg)](#)


Reverted master to this commit.
This is a stable version.


Done in the context of your comment in https://github.com/stuclem/vic/commit/726675e60a83953cbecc246338602547e9ab93f1.


Fixes #59


@nkzawa What do you suggest? Is there a general way to find out if the application is running in production? 🤔


Created multidev environment [ci-107-ci-example-d8-composer](https://dashboard.pantheon.io/sites/c401fd14-f745-4e51-9af2-f30b45146a0c#ci-107). [[Visit ci-107 Site](http://ci-107-ci-example-d8-composer.pantheonsite.io/)]


#59 ControllerProxy is not visible due to security restrictions, thats turned off in the development environment


Well, that's fun. Now it's in there ;-). I'll remove it for the next release


I'm sorry I'm not familiar with CoffeeScript :(


It won't be unsafe - it will just throw an exception http://docs.cms.orckestra.com/FAQ/Developer?q=What+does+%22ThreadDataManager+hasn%27t+been+initialized...%22+exception+mean%3F . If you think of DataConnection as of something that creates a context for the data layer to work, that's it is not a side effect but an intended effect :)


@JaimeIvanCervantes , I originally thought it would be cleaner to have this code in opObjectExtraction and make the opRegionFeatures just a machine to compute features, without any knowledge of defaults. I still believe it's better from the design point of view, but it looks like we've been building around the old bad design for so long that changing it now causes too many unpredictable side effects. If you are sure feature computation and table export both work for object classification, let's take your variant and move on. Thanks a lot for looking into it so deeply!


:thumbsup:  :tada: :taco: :beers:


NOW THIS 👍 🎉 🌮 🍻


FIRST COMMIT


was creating conflict due to order of include - it is added in RPCclient.h before DED


Final commit needed to close #131.


请问您的这个代码该怎么运行啊，需要安装什么东西吗，我直接VS打开是不能运行的，报了很多错误，求解


Doesn't hurt ;)

On 7 Dec 2016 10:12, "Jelte Lagendijk" <notifications@github.com> wrote:

> Well, that's fun. Now it's in there ;-). I'll remove it for the next
> release
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/mendix/AppStoreWidgetBoilerplate/commit/cc73bcfc00c059a496f1e92aebffe3802416c784#commitcomment-20100232>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AD6Yj1mow5v15DRrQqOChFGTscnh76Hsks5rFnh_gaJpZM4LGVsq>
> .
>


Should check if the navigation is backward. Maybe somebody clicked a button on the details page and tried to navigate out.


Do you know why it is failing on OSX?


@SkiTiSu No problem. Thank you for finding a bug 👍


The CreateUiDefinition still outputs the location. Should be removed there as well.
https://github.com/Azure/azure-quickstart-templates/blob/master/marketplace-samples/simple-windows-vm/createUiDefinition.json#L107


TeamCity Mobile :: Android :: Android SDK [Build 734](http://teamcity.hq.judo/viewLog.html?buildId=76908&buildTypeId=JudoAndroid_AndroidGradle) is now running


TeamCity Mobile :: Android :: Android SDK [Build 734](http://teamcity.hq.judo/viewLog.html?buildId=76908&buildTypeId=JudoAndroid_AndroidGradle) outcome was **FAILURE**
Summary: Gradle exception (new); exit code 1 Build time: 00:00:44


@omelnyk This might need a brief comment above this line?


Done!


test


I added `zbuff*` to `libzstd` in Visual Studio. Now it works fine in `Debug` but I noticed that `ZSTDLIB_API` is missing for all functions what causes that these functions will not be exported to Windows LIB/DLL.


**test by sahshi**
****
how are you

>

>

> [`ffsfsff`](url)


Salut bande tarés c'est super le matin


* inject.js


here you have a link to the commit :
https://git.busybox.net/buildroot/commit/?id=4bca88c839e54a9140c06010666f5e034127b8b1


У тебя же класс, зачем тебе глобальные переменные? Перенеси их в класс и сделай инстанс-переменными, и инициализацию вынеси в конструктор


please add to readme about how to use or run the this script.


兼容主流浏览器，原生js封装模块
技术：使用html/css和javaScript实现网页布局和特效。
功能：轮播，tab切换、运动框架返回顶部、产品的动态放大、 分页、图片产品刷新等。
 - 效果截图：
 - ![image](https://github.com/136shine/miguo/raw/master/imgs/miguo.png)


Ты можешь использовать делегат System.Action если тебе нужен пустой делегат - зачем много типов создавать?


Надо было пойти дальше и всю логику инкапсулировать в класс. У класса будет один публичный метод (выводящий меню), а все остальные приватные.


список - перелік


Что это?!


Сама идея вноса общих действий в класс не совсем верная. Класс - это сущность, которая логически объединяет данные и методы их обработки. У тебя же данные вне класса, а методы используются в другом месте.


Для чего?


error in sax see attached file


Весь код внутри when нужно вынести в отдельные приватные методы, а здесь их просто вызывать


TeamCity Mobile :: Android :: Android SDK [Build 735](http://teamcity.hq.judo/viewLog.html?buildId=76911&buildTypeId=JudoAndroid_AndroidGradle) is now running


well done


Duly noted and fixed.


TeamCity Mobile :: Android :: Android SDK [Build 735](http://teamcity.hq.judo/viewLog.html?buildId=76911&buildTypeId=JudoAndroid_AndroidGradle) outcome was **FAILURE**
Summary: Gradle exception; exit code 1 Build time: 00:00:34


Должно быть просто wsgi:application
Во всех статьях видел.

Ты уверен??!


Зачем тут эта строка? Они ничего не делает и не возвращает. Удалив ее не изменится абсолюно ничего.


Having issues with GitHub not being able to push to 'master'


Я бы рекомендовал вынести все проверки в приватные методы и здесь их просто вызвать. Код станет чище и понятнее.


И опять бессмысленная строка


Мы последние две строки добавляем в Dockerfile ....

---> RUN echo "${django_allowed_host}\n${email}" >> ssl/CM-ssl.ini


Используй elsif


Okey... Yes boss...

On Dec 7, 2016 2:57 PM, "navyasrib" <notifications@github.com> wrote:

> please add to readme about how to use or run the this script.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/navyasrib/nave/commit/ac7d8b2ebd0f3228fd24192152ad188f8ae4e8ad#commitcomment-20100440>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AOjeMfTC1KalL_jvJgRKNQtgbpMU9Bssks5rFnv0gaJpZM4LGXlh>
> .
>


зачем убрал(((


ma se non ci metto le parentesi graffe il linter mi da il seguente errore:
>  78:14  error  Arrow function should not return assignment  no-return-assign


Code & Supporting files (if any) for my Data Science Experiments


hi, errors for russia :
2016-12-07 10:21:28     skip TagRemove_Roundabout_Ref
2016-12-07 10:21:28     init TagWatchFrViPofm (node, way, relation)
2016-12-07 10:21:28   Analysing file /home/osmose/work/extracts/russia_moscow.osm.pbf
((1000L, {u'shop': u'supermarket', u'comment': u'\u041e\u041e\u041e "\u0410\u0421\u041f-\u0413\u0420\u0423\u041f\u041f", \u0418\u041d\u041d 7735108290, \u0434\u043e 22:45 c 2014-05-23', u'name': u'\u0414\u0435\u0441\u044f\u0442\u043e\u0447\u043a\u0430', u'opening_hours': u'08:30-22:45'}, (37.207464800000004, 55.9950817)), {'lat': 55.9950817, 'tag': {u'shop': u'supermarket', u'comment': u'\u041e\u041e\u041e "\u0410\u0421\u041f-\u0413\u0420\u0423\u041f\u041f", \u0418\u041d\u041d 7735108290, \u0434\u043e 22:45 c 2014-05-23', u'name': u'\u0414\u0435\u0441\u044f\u0442\u043e\u0447\u043a\u0430', u'opening_hours': u'08:30-22:45'}, 'lon': 37.207464800000004, 'id': 1000L})
Traceback (most recent call last):
  File "/home/osmose/osmose-backend/modules/OsmPbf.py", line 92, in NodeParse
    self._output.NodeCreate(data)
  File "./analysers/analyser_sax.py", line 142, in NodeCreate
    res = meth(data, tags)
  File "/home/osmose/osmose-backend/plugins/Name_Script.py", line 181, in node
    print(s)
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-8: ordinal not in range(128)

.......

((60295631L, {u'name:en': u'Oktyabrskaya', u'name': u'\u041e\u043a\u0442\u044f\u0431\u0440\u044c\u0441\u043a\u0430\u044f', u'colour': u'orange', u'network': u'\u041c\u043e\u0441\u043a\u043e\u0432\u0441\u043a\u0438\u0439 \u043c\u0435\u0442\u0440\u043e\u043f\u043e\u043b\u0438\u0442\u0435\u043d', u'operator': u'\u0413\u0423\u041f \xab\u041c\u043e\u0441\u043a\u043e\u0432\u0441\u043a\u0438\u0439 \u043c\u0435\u0442\u0440\u043e\u043f\u043e\u043b\u0438\u0442\u0435\u043d\xbb', u'railway': u'subway_entrance', u'description': u'\u041a\u0430\u043b\u0443\u0436\u0441\u043a\u043e-\u0420\u0438\u0436\u0441\u043a\u0430\u044f \u043b\u0438\u043d\u0438\u044f'}, (37.61267410000003, 55.731279300000026)), {'lat': 55.731279300000026, 'tag': {u'name:en': u'Oktyabrskaya', u'name': u'\u041e\u043a\u0442\u044f\u0431\u0440\u044c\u0441\u043a\u0430\u044f', u'colour': u'orange', u'network': u'\u041c\u043e\u0441\u043a\u043e\u0432\u0441\u043a\u0438\u0439 \u043c\u0435\u0442\u0440\u043e\u043f\u043e\u043b\u0438\u0442\u0435\u043d', u'operator': u'\u0413\u0423\u041f \xab\u041c\u043e\u0441\u043a\u043e\u0432\u0441\u043a\u0438\u0439 \u043c\u0435\u0442\u0440\u043e\u043f\u043e\u043b\u0438\u0442\u0435\u043d\xbb', u'railway': u'subway_entrance', u'description': u'\u041a\u0430\u043b\u0443\u0436\u0441\u043a\u043e-\u0420\u0438\u0436\u0441\u043a\u0430\u044f \u043b\u0438\u043d\u0438\u044f'}, 'lon': 37.61267410000003, 'id': 60295631L})
Traceback (most recent call last):
  File "/home/osmose/osmose-backend/modules/OsmPbf.py", line 92, in NodeParse
    self._output.NodeCreate(data)
  File "./analysers/analyser_sax.py", line 142, in NodeCreate
    res = meth(data, tags)
  File "/home/osmose/osmose-backend/plugins/Name_Script.py", line 181, in node
    print(s)
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-10: ordinal not in range(128)

2016-12-07 10:21:37   Closing reader and parser
2016-12-07 10:21:37   error on analyse...
2016-12-07 10:21:37     Traceback (most recent call last):
2016-12-07 10:21:37       File "./osmose_run.py", line 621, in run
2016-12-07 10:21:37         analyser_obj.analyser()
2016-12-07 10:21:37       File "./analysers/analyser_sax.py", line 53, in analyser
2016-12-07 10:21:37         self._run_analyse()
2016-12-07 10:21:37       File "./analysers/analyser_sax.py", line 439, in _run_analyse
2016-12-07 10:21:37         self.parser.CopyTo(self)
2016-12-07 10:21:37       File "/home/osmose/osmose-backend/modules/OsmPbf.py", line 54, in CopyTo
2016-12-07 10:21:37         raise Exception()
2016-12-07 10:21:37     Exception
2016-12-07 10:21:37


danke. der Augenkrebs war stark.


Is the sql feature meant to be incorporated, and how well is it functioning?


We have to write full Readme or documentation. There we will add it. This is a simple script for only us.
the usage is as follows

sh nave.sh [file name]


give a descriptive error with context information. don't depend on e.getMessage(). fix this in all places.


@ximex  Ok, didn't know that.. Still maybe using `/* ... */` would be better because it is more explicit.


ah, ho capito: effettivamente non stai tornando nessun valore, quindi ha senso avere le parentesi


Ты неправильно используешь неравенство. Для "не равно" нужно использовать оператор `!=`, а тут у тебя получается другая логика: сначала ты применяешь отрицание к `carriage.type`, что дает тебе `false` (т.к. если type возвращает любое, отличное от `nil` или `false` значение, то оно тракутется как `true`, а `!true` вернет `false`), а затем сравниваешь. В итоге, у тебя получается `false == self.type`, что в большинстве случаев будет давать `false`


I don't know whether it's always equivalent, in fact I was going to open an issue to ask you, but there's also:

    matchstr(getline('.')[:col('.')-2], '\S\+$')

It seems equivalent, but I'm not sure.


even through the logical block size remains 512, for most modern hdd devices and sdd devices, the physical block size is 4k not 512 (so-called 512e disk). I think there will be a lot of advantage if you move to 4k block size (such as avoiding double reading in os level).


Опять неверное сравнение. Пройдись по коду, поищи еще такие вещи и исправь их на `!=`


Кстати, для удаления такая проверка не нужна.


If the offset value no longer exists we need to recalculate.


Не стоит мешать 2 действия в один метод. Раздели на 2 отдельных метода, а проверки вынеси в приватные методы, чтобы не дублировать


TeamCity Mobile :: Android :: Android SDK [Build 736](http://teamcity.hq.judo/viewLog.html?buildId=76917&buildTypeId=JudoAndroid_AndroidGradle) is now running


Error: Cannot access private property Symfony\Bridge\Twig\Extension\FormExtension::$renderer
500 Internal Server Error - FatalErrorException

in symfony 3.2


无用


@programmin1 This works fine for me in IE 10: http://jsbin.com/dimowe/edit?html,js,console

If you can tweak this jsbin until it shows the issue you have, please open a new issue with the link to this jsbin.


Please see #580 . Thanks Eric.
Apologies for the snails pace. 🐌


Это излишняя проверка. Мы можем передать nil и это будет трактоваться как false, или наоборот другое значение, рассчитывая, что оно будет трактоваться как true. В руби не строгая типизация, не нужно ее привностить такими вот проверками, там, где это не важно. Если переданный объект умеет себя вести так, как ожидается, то нам не важно, какого он класса (в большинстве случаев). Это очень важная концепция, стоит ей проникнуться. Т.е. принимающему коду в большинстве случаев без разницы, какие объекты ему передали, если они работают как ожидается. Если нет, то это проблема вызывающего кода.


I'm not supporting using my repository as a Bower source anymore. There's a pre-built copy at https://github.com/components/codemirror , though that also doesn't seem to have bower.json anymore. Open an issue there if you need it.


![view](https://cloud.githubusercontent.com/assets/19756758/20962405/221fe652-bca4-11e6-8a7b-9acbe9b6b98c.jpg)

截图来一张看看


Вот такое смешение 2-х действий в одном методе ведет к разрастанию метода. Держи ориентиром, что 1 метод должен быть не более 10 строк (в идеале - 5-7). И старайся все вспомогательно выносить в приватные методы.


Зачем это проверять здесь? Ты уже проверил это, в методе, устанавливающем маршрут, далее надо доверять собственному коду.


Если тут другой смысл - проверить, есть ли маршрут, то тогда лучше проверить так: `if self.route`. Потому что если там будет nil, то проверка не пройдет. А если другой объект и у него нет метода list, то это не проблема данного метода, это проблема того, какие связи были установлены между объектами. Инями словами, это проблема вызывающего кода, а не вызываемого.


oh yeah I added the font mechanism to the window itself


I have updated the comment, is it clearer now? @yossigil


这是我的代码


So just missing to create a tar of it? :)


Bonjour PandiPanda69

Je ne sais pas comment vous contacter j'aimerais bien vous parler en priver svp


TeamCity Mobile :: Android :: Android SDK [Build 736](http://teamcity.hq.judo/viewLog.html?buildId=76917&buildTypeId=JudoAndroid_AndroidGradle) outcome was **SUCCESS**
Summary: Tests passed: 58 Build time: 00:07:32


Should this one be applicable for perf_event_open02 testcase ?

perf_event_open02.c
...
	for (i = 0; i < MAX_CTRS; i++) {
		fdarry[i] = perf_event_open(&hw_event, 0, -1, -1, 0);
		if (fdarry[i] == -1) {
			if (errno == ENOENT) {
				tst_brkm(TCONF, cleanup,
				         "PERF_COUNT_HW_INSTRUCTIONS not supported");
			}
...

The inconsistency made perf_event_open01 failed (returned EOPNOTSUPP) and perf_event_open02 passed on my machine. I would expect same results of these testcases if perf_event_open() failed and return same errno ? Is it right ?


Надо в конструкторе вызывать этот метод


@xqbumu see https://github.com/sonata-project/SonataAdminBundle/pull/4216


Isn't that handled by the s3 plugin? That's what I thought last night 😄


Этот метод отличается от того что в пассажирском поезде по факту только классом вагона который мы можем добавить. Поэтому можно определить константу `WAGON_TYPE = CargoWagon`, а сам метод оставить полностью в базовом классе.


good catch


Why don't you use the removeSanitizeSeed method in that line?


How do you know this is the correct rotation? Is there a source or is it a guess?


Moror og se. tror vi har løst det ganske likt. jeg gadd ikke bruke klasse, fordi det føles deilig!


@fritx TF App


Be careful with the "run :local" call in my version. I think it somehow sticks and also sets the environment for the subsequent calls to local. If one of them is the deploy cleanup task, it will delete stuff in your local working directory :-/ I was surprised by this behavior of "run :local" and opened the issue https://github.com/mina-deploy/mina/issues/489


`undefined == null // true`


Checkmarx scan results 07 Dec 2016 09:53 UTC +00:00

High
Medium
Low
Info

New
3
4
20
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

c140ff008ddc5acd057241d76b0c3552375841d8	@web-flow


Good job.

I think you did not use MAVEN properly. It could have taken care of JUNIT.

Less important: the code below is bloated for no reason. Please talk to someone who is more fluent with Java.
```java
		 +	private ArrayList<tableTuple> list;
 +	private int numOfTuples;
 +
 +	public DataList(){
 +		this.list= new ArrayList<>();
 +		this.numOfTuples=0;
 +	}
```


I just did. The only exception is the `Transform2D` class that defines an array of a fixed size of 6. Here, I think it makes sense to stick to an array representation.


@Simn is there a reason you used single-quote here besides saving a couple escaping keystrokes? :) Because I later added `Ast.s_escape f` there and now someone complains about `"'"` not being escaped (because s_escape doesn't do that), so I'm thinking about fixing it with making it use double-quote instead.


Since we need this function for one of the malones alert, we are adding it to riemann-boshrelease so as not to duplicate in the properties file and also make is shareable with other teams in case they have similar requirements.


0b61091e715b90a8f8251951ccba0490accdd6ad
@web-flow 0b61091e715b90a8f8251951ccba0490accdd6ad
@alpharomeo13 0b61091e715b90a8f8251951ccba0490accdd6ad


We figured out after this commit that there was a problem with the using of MAVEN and we fixed it in later commits.


This gives a compiler error:

```
miner.cpp: In function ‘CBlockTemplate* CreateNewBlock(const CScript&)’:
miner.cpp:102:113: warning: large integer implicitly truncated to unsigned type [-Woverflow]
 stTracker resourceTracker(std::numeric_limits<size_t>::max(), std::numeric_limits<size_t>::max());
                                                                                ^
miner.cpp:102:113: warning: large integer implicitly truncated to unsigned type [-Woverflow]
```


I don't remember.


Maybe the deprecation message should also mention `Access.key!/1` for cases when you don't need a default value?


Alright, I just hope it doesn't matter for Closure and other weird JS tooling.


Oh, I see! I see it was presumptuous of me to make this commit then, so (1) I wish to express my apologies for not asking you about it in the first place, and (2) thank you very much for correcting my misunderstanding. :)


@michalmuskala will do.


@julien-roux: R version downgraded, is that an error, or thanks to more correct object initialization?


@javache is this now called yoga?


[data_exploration-12-5.R:9:23:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L9) *style:* **Put spaces around all infix operators.**
```r
hist(RETMONTH, breaks =60)
                      ^~
```
[data_exploration-12-5.R:11:35:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L11) *style:* **Put spaces around all infix operators.**
```r
(re_up <- length(RETMONTH[RETMONTH<0]))
                                 ~^~
```
[data_exploration-12-5.R:12:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L12) *style:* **Put spaces around all infix operators.**
```r
(re_down <- length(RETMONTH[RETMONTH>0]))
                                   ~^~
```
[data_exploration-12-5.R:12:42:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
(re_down <- length(RETMONTH[RETMONTH>0]))
                                         ^
```
[data_exploration-12-5.R:14:6:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L14) *style:* **Put spaces around all infix operators.**
```r
re_up/(re_up+re_down)
    ~^~
```
[data_exploration-12-5.R:14:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L14) *style:* **Place a space before left parenthesis, except in a function call.**
```r
re_up/(re_up+re_down)
      ^
```
[data_exploration-12-5.R:14:13:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L14) *style:* **Put spaces around all infix operators.**
```r
re_up/(re_up+re_down)
           ~^~
```
[data_exploration-12-5.R:16:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L16) *style:* **Put spaces around all infix operators.**
```r
up_down <- ifelse(RETMONTH >0, 1, 0)
                           ^~
```
[data_exploration-12-5.R:27:26:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L27) *style:* **Commas should always have a space after.**
```r
uniqueness <- apply(data,2,numb_unique)
                         ^
```
[data_exploration-12-5.R:27:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L27) *style:* **Commas should always have a space after.**
```r
uniqueness <- apply(data,2,numb_unique)
                           ^
```
[data_exploration-12-5.R:29:29:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L29) *style:* **Put spaces around all infix operators.**
```r
length(uniqueness[uniqueness<50])
                           ~^~
```
[data_exploration-12-5.R:31:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L31) *style:* **Put spaces around all infix operators.**
```r
which(uniqueness <50)
                 ^~
```
[data_exploration-12-5.R:35:32:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L35) *style:* **Only use double-quotes.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                               ^~~
```
[data_exploration-12-5.R:35:40:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L35) *style:* **Put spaces around all infix operators.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                                      ~^
```
[data_exploration-12-5.R:35:42:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L35) *style:* **Only use double-quotes.**
```r
plot(RETMONTH[n_low:n], type = 'l', col= 'red', lwd = 2)
                                         ^~~~~
```
[data_exploration-12-5.R:36:9:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L36) *style:* **Put spaces around all infix operators.**
```r
abline(h= mean(RETMONTH[n_low:n]), col = "blue", lwd = 2)
       ~^
```
[data_exploration-12-5.R:38:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L38) *style:* **Put spaces around all infix operators.**
```r
MSE_i <- (RETMONTH[n_low:n]-mean(RETMONTH[n_low:n]))^2
                          ~^~
```
[data_exploration-12-5.R:38:53:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L38) *style:* **Put spaces around all infix operators.**
```r
MSE_i <- (RETMONTH[n_low:n]-mean(RETMONTH[n_low:n]))^2
                                                   ~^~
```
[data_exploration-12-5.R:44:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L44) *style:* **Put spaces around all infix operators.**
```r
n_low <- n-1000
         ~^~
```
[data_exploration-12-5.R:44:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L44) *style:* **Trailing whitespace is superfluous.**
```r
n_low <- n-1000
               ^
```
[data_exploration-12-5.R:65:21:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L65) *style:* **Commas should always have a space after.**
```r
xs <- pca$x[n:n_low,1:9]
                    ^
```
[data_exploration-12-5.R:71:30:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L71) *style:* **Commas should always have a space after.**
```r
data2 <- data_no_ind[n_low:n,]
                             ^
```
[data_exploration-12-5.R:79:26:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L79) *style:* **Put spaces around all infix operators.**
```r
fit <- glm(updown~., data= data3, family = "binomial")
                        ~^
```
[data_exploration-12-5.R:83:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L83) *style:* **Put spaces around all infix operators.**
```r
cv_est <- cv.glm(data3, fit, K= 10)$delta[1]
                             ~^
```
[data_exploration-12-5.R:85:2:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L85) *style:* **Put spaces around all infix operators.**
```r
1-cv_est
~^~
```
[data_exploration-12-5.R:86:1:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/data_exploration-12-5.R#L86) *style:* **Trailing blank lines are superfluous.**
```r

^
```


[cm_v1.R:1:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
require(rpart)
              ^
```
[cm_v1.R:1:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:2:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
     ~^~~
```
[cm_v1.R:2:22:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                    ~^~
```
[cm_v1.R:2:60:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                          ~^~
```
[cm_v1.R:2:70:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                                    ~^~
```
[cm_v1.R:5:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L5) *style:* **Trailing whitespace is superfluous.**
```r
require(rpart)
              ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L6) *style:* **Use <-, not =, for assignment.**
```r
mydata$Industry=as.factor(mydata$Industry)
               ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L6) *style:* **Put spaces around all infix operators.**
```r
mydata$Industry=as.factor(mydata$Industry)
              ~^~
```
[cm_v1.R:6:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L6) *style:* **Trailing whitespace is superfluous.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:7:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/cm_v1.R#L7) *style:* **Use <-, not =, for assignment.**
```r
contrasts(mydata$Industry) = contr.treatment(213)
                           ^
```


[cm_v1.R:1:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
require(rpart)
              ^
```
[cm_v1.R:1:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L1) *style:* **Use two spaces to indent, never tabs.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:2:7:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
     ~^~~
```
[cm_v1.R:2:22:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                    ~^~
```
[cm_v1.R:2:60:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                          ~^~
```
[cm_v1.R:2:70:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L2) *style:* **Put spaces around all infix operators.**
```r
mydata<-read.csv(file="mf850-finalproject-data.csv", header=TRUE, sep=",")
                                                                    ~^~
```
[cm_v1.R:5:15:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L5) *style:* **Trailing whitespace is superfluous.**
```r
require(rpart)
              ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L6) *style:* **Use <-, not =, for assignment.**
```r
mydata$Industry=as.factor(mydata$Industry)
               ^
```
[cm_v1.R:6:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L6) *style:* **Put spaces around all infix operators.**
```r
mydata$Industry=as.factor(mydata$Industry)
              ~^~
```
[cm_v1.R:6:43:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L6) *style:* **Trailing whitespace is superfluous.**
```r
mydata$Industry=as.factor(mydata$Industry)
                                          ^
```
[cm_v1.R:7:28:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/81e372380a4e8ae2a3a57fc7d3a3e6bfd6b197c9/cm_v1.R#L7) *style:* **Use <-, not =, for assignment.**
```r
contrasts(mydata$Industry) = contr.treatment(213)
                           ^
```


[GPUAcceleratedClustering.R:4:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L4) *style:* **Only use double-quotes.**
```r
install.packages('rpud')
                 ^~~~~~
```
[GPUAcceleratedClustering.R:11:17:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
  set.seed(seed)
                ^
```
[GPUAcceleratedClustering.R:12:32:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L12) *style:* **Put spaces around all infix operators.**
```r
  matrix(rnorm(dim * num), nrow=num)
                              ~^~
```
[GPUAcceleratedClustering.R:21:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L21) *style:* **Variable and function names should be all lowercase.**
```r
  dgpu <- rpuDist(matrix)
          ^~~~~~~
```
[GPUAcceleratedClustering.R:21:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L21) *warning:* **no visible global function definition for ‘rpuDist’**
```r
  dgpu <- rpuDist(matrix)
          ^~~~~~~
```
[GPUAcceleratedClustering.R:22:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L22) *style:* **Variable and function names should be all lowercase.**
```r
  rpuHclust(dgpu)
  ^~~~~~~~~
```
[GPUAcceleratedClustering.R:22:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L22) *warning:* **no visible global function definition for ‘rpuHclust’**
```r
  rpuHclust(dgpu)
  ^~~~~~~~~
```
[GPUAcceleratedClustering.R:35:29:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L35) *style:* **Put spaces around all infix operators.**
```r
print(system.time(cpuhclust <-run_cpu(m)))
                            ^~~
```
[GPUAcceleratedClustering.R:43:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/GPUAcceleratedClustering.R#L43) *style:* **Commas should always have a space after.**
```r
plot(gpuhclust,hang = -1)
               ^
```


[NeuralNet.R:6:18:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L6) *style:* **Only use double-quotes.**
```r
install.packages('neuralnet')
                 ^~~~~~~~~~~
```
[NeuralNet.R:12:11:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L12) *style:* **Use <-, not =, for assignment.**
```r
stockdata = read.csv("mf850-finalproject-data.csv")
          ^
```
[NeuralNet.R:13:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L13) *style:* **Use <-, not =, for assignment.**
```r
stockdata$Date = NULL
               ^
```
[NeuralNet.R:14:20:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L14) *style:* **Use <-, not =, for assignment.**
```r
stockdata$Industry = NULL
                   ^
```
[NeuralNet.R:15:14:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L15) *style:* **Use <-, not =, for assignment.**
```r
scaled_Stock = scale(stockdata)
             ^
```
[NeuralNet.R:16:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L16) *style:* **Use <-, not =, for assignment.**
```r
y = stockdata$RETMONTH_SPX
  ^
```
[NeuralNet.R:17:3:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L17) *style:* **Use <-, not =, for assignment.**
```r
x = stockdata
  ^
```
[NeuralNet.R:18:16:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L18) *style:* **Use <-, not =, for assignment.**
```r
x$RETMONTH_SPX = NULL
               ^
```
[NeuralNet.R:24:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L24) *style:* **Commas should always have a space after.**
```r
trainingdata <- cbind(traininginput,trainingoutput)
                                    ^
```
[NeuralNet.R:25:37:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L25) *style:* **Commas should always have a space after.**
```r
colnames(trainingdata) <- c("Input","Output")
                                    ^
```
[NeuralNet.R:31:36:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L31) *style:* **Commas should always have a space after.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                   ^
```
[NeuralNet.R:31:56:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L31) *style:* **Put spaces around all infix operators.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                                      ~^~
```
[NeuralNet.R:31:71:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L31) *style:* **Put spaces around all infix operators.**
```r
net.sqrt <- neuralnet(Output~Input,trainingdata, hidden=100, threshold=0.01)
                                                                     ~^~
```
[NeuralNet.R:38:27:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L38) *style:* **Place a space before left parenthesis, except in a function call.**
```r
testdata <- as.data.frame((1:10)^2) #Generate some squared numbers
                          ^
```
[NeuralNet.R:38:33:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L38) *style:* **Put spaces around all infix operators.**
```r
testdata <- as.data.frame((1:10)^2) #Generate some squared numbers
                               ~^~
```
[NeuralNet.R:48:31:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L48) *style:* **Commas should always have a space after.**
```r
cleanoutput <- cbind(testdata,sqrt(testdata),
                              ^
```
[NeuralNet.R:50:36:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L50) *style:* **Commas should always have a space after.**
```r
colnames(cleanoutput) <- c("Input","Expected Output","Neural Net Output")
                                   ^
```
[NeuralNet.R:50:54:](https://github.com/myellen/MF850_Computational_Finance_Final/blob/640cc60fafeeaa91bcc557bbd0d0b23d03791e3d/NeuralNet.R#L50) *style:* **Commas should always have a space after.**
```r
colnames(cleanoutput) <- c("Input","Expected Output","Neural Net Output")
                                                     ^
```