[R/FDAClassifPreprocWrapper.R:12:29:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAClassifPreprocWrapper.R#L12) *style:* **Commas should always have a space after.**
```r
  requirePackages("stringi",default.method = "load")
                            ^
```
[R/FDAClassifPreprocWrapper.R:16:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAClassifPreprocWrapper.R#L16) *style:* **Variable or function name should be dotted.case.**
```r
    taskTs = makeFDAClassifTask(data = data, target = target, positive = args$positive)
    ^~~~~~
```
[R/FDAClassifPreprocWrapper.R:17:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAClassifPreprocWrapper.R#L17) *style:* **Variable or function name should be dotted.case.**
```r
    taskFa = convertFDATaskToNormalTask(task = taskTs)
    ^~~~~~
```
[R/FDA_feal_twoStepAIC.R:1:1:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L1) *style:* **Variable or function name should be dotted.case.**
```r
MAGICNumber4TWOSTEPAIC = 163
^~~~~~~~~~~~~~~~~~~~~~
```
[R/FDA_feal_twoStepAIC.R:21:35:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L21) *style:* **Commas should always have a space after.**
```r
  filename = paste("steplm1", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                  ^
```
[R/FDA_feal_twoStepAIC.R:21:78:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L21) *style:* **Commas should never have a space before.**
```r
  filename = paste("steplm1", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                                                            ~^
```
[R/FDA_feal_twoStepAIC.R:21:83:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L21) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  filename = paste("steplm1", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                                                                ~^~
```
[R/FDA_feal_twoStepAIC.R:35:35:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L35) *style:* **Commas should always have a space after.**
```r
  filename = paste("steplm2", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                  ^
```
[R/FDA_feal_twoStepAIC.R:35:78:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L35) *style:* **Commas should never have a space before.**
```r
  filename = paste("steplm2", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                                                            ~^
```
[R/FDA_feal_twoStepAIC.R:35:83:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDA_feal_twoStepAIC.R#L35) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  filename = paste("steplm2", "-",format(Sys.time(), "%b%d%H%M%S"), ".RData" , sep="")
                                                                                ~^~
```
[R/FDAconvertTask.R:34:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAconvertTask.R#L34) *style:* **Variable or function name should be dotted.case.**
```r
    modelSh = do.call(getFDAShapeletFeatures, new.args)
    ^~~~~~~
```
[R/FDAconvertTask.R:88:40:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAconvertTask.R#L88) *style:* **Only use double-quotes.**
```r
  colnames(newdata) = c(target, paste0('V', 1:(ncol(newdata) - 1) ))
                                       ^~~
```
[R/FDAconvertTask.R:106:40:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAconvertTask.R#L106) *style:* **Only use double-quotes.**
```r
  colnames(newdata) = c(target, paste0('V', 1:(ncol(newdata) - 1) ))
                                       ^~~
```
[R/FDAfeatures_FDboost.R:24:111:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L24) *style:* **Commas should never have a space before.**
```r
getFDAFDboostFeatures = function(data, target, have.target = TRUE, include.target = FALSE, bsignal.knots =10L ,bsignal.df = 3L) {
                                                                                ~^
```
[R/FDAfeatures_FDboost.R:24:112:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L24) *style:* **Commas should always have a space after.**
```r
getFDAFDboostFeatures = function(data, target, have.target = TRUE, include.target = FALSE, bsignal.knots =10L ,bsignal.df = 3L) {
                                                                                ^
```
[R/FDAfeatures_FDboost.R:47:52:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L47) *warning:* **Avoid 1:ncol(...) expressions, use seq_len.**
```r
  blrn = FDboost::bsignal(x = as.matrix(data), s = 1:ncol(data), knots = bsignal.knots , degree = bsignal.df)
                                                   ^
```
[R/FDAfeatures_FDboost.R:47:88:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L47) *style:* **Commas should never have a space before.**
```r
  blrn = FDboost::bsignal(x = as.matrix(data), s = 1:ncol(data), knots = bsignal.knots , degree = bsignal.df)
                                                                                ~^
```
[R/FDAfeatures_FDboost.R:48:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L48) *style:* **Variable or function name should be dotted.case.**
```r
  features_bsignal = mboost::extract(object = blrn, what = "design")  # get the design matrix from base learner
  ^~~~~~~~~~~~~~~~
```
[R/FDAfeatures_FDboost.R:49:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_FDboost.R#L49) *style:* **Variable or function name should be dotted.case.**
```r
  d_fdboost = as.data.frame(features_bsignal)
  ^~~~~~~~~
```
[R/FDAfeatures_fourier.R:44:27:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fourier.R#L44) *style:* **Commas should always have a space after.**
```r
  ffttrafo = t(apply(data,1, fft))
                          ^
```
[R/FDAfeatures_fourier.R:48:23:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fourier.R#L48) *style:* **Variable or function name should be dotted.case.**
```r
         amplitude = {fftPA = getFourierAmplitude(fourier.comp = ffttrafo)},
                      ^~~~~
```
[R/FDAfeatures_fourier.R:49:19:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fourier.R#L49) *style:* **Variable or function name should be dotted.case.**
```r
         phase = {fftPA = getFourierPhase(fourier.comp = ffttrafo)}
                  ^~~~~
```
[R/FDAfeatures_fourier.R:53:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fourier.R#L53) *style:* **Variable or function name should be dotted.case.**
```r
    fftPA = as.data.frame(matrix(fftPA, nrow = 1))
    ^~~~~
```
[R/FDAfeatures_fourier.R:55:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fourier.R#L55) *style:* **Variable or function name should be dotted.case.**
```r
    fftPA = as.data.frame(fftPA)
    ^~~~~
```
[R/FDAfeatures_fpca.R:40:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fpca.R#L40) *style:* **Variable or function name should be dotted.case.**
```r
  features_fpca = rst$scores
  ^~~~~~~~~~~~~
```
[R/FDAfeatures_fpca.R:41:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fpca.R#L41) *style:* **Variable or function name should be dotted.case.**
```r
  d_fpca = as.data.frame(features_fpca)
  ^~~~~~
```
[R/FDAfeatures_fpca.R:42:30:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fpca.R#L42) *style:* **Commas should always have a space after.**
```r
  names(d_fpca) = paste0("V",1:ncol(d_fpca))
                             ^
```
[R/FDAfeatures_fpca.R:42:30:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_fpca.R#L42) *warning:* **Avoid 1:ncol(...) expressions, use seq_len.**
```r
  names(d_fpca) = paste0("V",1:ncol(d_fpca))
                             ^
```
[R/FDAfeatures_multiResNaive.R:46:6:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_multiResNaive.R#L46) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(fdn in names(fd.features)){
     ^
```
[R/FDAfeatures_multiResNaive.R:71:36:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_multiResNaive.R#L71) *style:* **Trailing whitespace is superfluous.**
```r
  feat.list = vector("list", n.obs)
                                   ^~
```
[R/FDAfeatures_wavelets.R:39:5:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_wavelets.R#L39) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(inherits(data, "matrix"))
    ^
```
[R/FDAfeatures_wavelets.R:51:18:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_wavelets.R#L51) *style:* **Commas should always have a space after.**
```r
    a = t(data[i,])
                 ^
```
[R/FDAfeatures_wavelets.R:53:42:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/FDAfeatures_wavelets.R#L53) *style:* **Commas should always have a space after.**
```r
    wtdata = rbind(wtdata, unlist(c(wt@W,wt@V[[wt@level]])))
                                         ^
```
[R/RLearner_FDAclassif_kernel.R:30:83:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAclassif_kernel.R#L30) *style:* **Commas should always have a space after.**
```r
  learned.model = fda.usc::classif.kernel(group = glearn, fdataobj = data.fdclass,...)
                                                                                ^
```
[R/RLearner_FDAclassif_np.R:31:79:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAclassif_np.R#L31) *style:* **Commas should always have a space after.**
```r
  learned.model = fda.usc::classif.np(group = glearn, fdataobj = data.fdclass,...)
                                                                              ^
```
[R/RLearner_FDAregr_fgam.R:19:18:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L19) *warning:* **Unneded concatenation of a constant. Remove the "c" call.**
```r
    properties = c("numerics"),
                 ^
```
[R/RLearner_FDAregr_fgam.R:49:20:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L49) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
    mat.list[[fdn]]=  as.matrix(d[, tdesc$fd.features[[fdn]], drop = FALSE])
                  ~^
```
[R/RLearner_FDAregr_fgam.R:70:6:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L70) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in 1:num4channel){
     ^
```
[R/RLearner_FDAregr_fgam.R:71:34:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L71) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
    list4mat[[name4channel[[i]]]]=  as.matrix(subset(df, select = channel.list[[i]]))
                                ~^
```
[R/RLearner_FDAregr_fgam.R:78:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L78) *style:* **Variable or function name should be dotted.case.**
```r
  mextra_para  = list(...)
  ^~~~~~~~~~~
```
[R/RLearner_FDAregr_fgam.R:81:67:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/R/RLearner_FDAregr_fgam.R#L81) *style:* **Only use double-quotes.**
```r
  pred = predict(.model$learner.model, newdata = list4mat, type = 'response')
                                                                  ^~~~~~~~~~
```
[tests/testthat/test_FDA_FPCA.R:5:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_FPCA.R#L5) *style:* **Variable or function name should be dotted.case.**
```r
  taskTs = makeFDAClassifTask(data = gp, target = "X1", positive = "1")
  ^~~~~~
```
[tests/testthat/test_FDA_FPCA.R:6:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_FPCA.R#L6) *style:* **Variable or function name should be dotted.case.**
```r
  refData = getTaskData(taskTs, target.extra = TRUE)
  ^~~~~~~
```
[tests/testthat/test_FDA_FPCA.R:9:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_FPCA.R#L9) *style:* **Variable or function name should be dotted.case.**
```r
  f_df = getFDAFPCAFeatures(data = refData$data, target = "X1", have.target = TRUE, include.target = FALSE)
  ^~~~
```
[tests/testthat/test_FDA_Fourier.R:6:61:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L6) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1= as.factor(c(-1,1,1,-1, 1)))
                                                           ~^
```
[tests/testthat/test_FDA_Fourier.R:6:78:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1= as.factor(c(-1,1,1,-1, 1)))
                                                                             ^
```
[tests/testthat/test_FDA_Fourier.R:6:80:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1= as.factor(c(-1,1,1,-1, 1)))
                                                                               ^
```
[tests/testthat/test_FDA_Fourier.R:6:82:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1= as.factor(c(-1,1,1,-1, 1)))
                                                                                ^
```
[tests/testthat/test_FDA_Fourier.R:8:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L8) *style:* **Variable or function name should be dotted.case.**
```r
  taskTs = makeFDAClassifTask(data = gp, target = "X1", positive = "1")
  ^~~~~~
```
[tests/testthat/test_FDA_Fourier.R:9:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L9) *style:* **Variable or function name should be dotted.case.**
```r
  refData = getTaskData(taskTs, target.extra = TRUE)
  ^~~~~~~
```
[tests/testthat/test_FDA_Fourier.R:11:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L11) *style:* **Variable or function name should be dotted.case.**
```r
  taskFa = convertFDATaskToNormalTask(task = taskTs, method = "fourier", pars = list(fft.coeff = "amplitude"))
  ^~~~~~
```
[tests/testthat/test_FDA_Fourier.R:12:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L12) *style:* **Variable or function name should be dotted.case.**
```r
  faData = getTaskData(taskFa, target.extra = TRUE)
  ^~~~~~
```
[tests/testthat/test_FDA_Fourier.R:13:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L13) *style:* **Variable or function name should be dotted.case.**
```r
  taskFp = convertFDATaskToNormalTask(task = taskTs, method = "fourier", pars = list(fft.coeff = "phase"))
  ^~~~~~
```
[tests/testthat/test_FDA_Fourier.R:14:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L14) *style:* **Variable or function name should be dotted.case.**
```r
  fpData = getTaskData(taskFp, target.extra = TRUE)
  ^~~~~~
```
[tests/testthat/test_FDA_Fourier.R:20:35:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L20) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  expect_true(all(fpData$data < pi/2 & fpData$data > -pi/2))
                                 ~^~
```
[tests/testthat/test_FDA_Fourier.R:20:57:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Fourier.R#L20) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  expect_true(all(fpData$data < pi/2 & fpData$data > -pi/2))
                                                       ~^~
```
[tests/testthat/test_FDA_PreprocWrapper.R:4:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_PreprocWrapper.R#L4) *style:* **Variable or function name should be dotted.case.**
```r
  taskFa = convertFDATaskToNormalTask(task = gunpoint.task, method = "fourier",
  ^~~~~~
```
[tests/testthat/test_FDA_PreprocWrapper.R:6:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_PreprocWrapper.R#L6) *style:* **Variable or function name should be dotted.case.**
```r
  taskFp = convertFDATaskToNormalTask(task = gunpoint.task, method = "fourier",
  ^~~~~~
```
[tests/testthat/test_FDA_PreprocWrapper.R:11:60:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_PreprocWrapper.R#L11) *style:* **Only use double-quotes.**
```r
  lrn = makeFDAClassifPreprocWrapper(learner = makeLearner('classif.ranger'))
                                                           ^~~~~~~~~~~~~~~~
```
[tests/testthat/test_FDA_Wavelets.R:6:22:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                    ~^
```
[tests/testthat/test_FDA_Wavelets.R:6:39:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                                      ^
```
[tests/testthat/test_FDA_Wavelets.R:6:41:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                                        ^
```
[tests/testthat/test_FDA_Wavelets.R:6:43:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                                          ^
```
[tests/testthat/test_FDA_Wavelets.R:6:51:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Commas should never have a space before.**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                                                 ~^
```
[tests/testthat/test_FDA_Wavelets.R:6:52:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L6) *style:* **Commas should always have a space after.**
```r
  gp = data.frame( X1= as.factor(c(-1,1,1,-1, 1)) ,v1 = 1:5, v2 = 12:16, v3 = 3:7, v4 = 4:8)
                                                   ^
```
[tests/testthat/test_FDA_Wavelets.R:8:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L8) *style:* **Variable or function name should be dotted.case.**
```r
  taskTs = makeFDAClassifTask(data = gp, target = "X1", positive = "1")
  ^~~~~~
```
[tests/testthat/test_FDA_Wavelets.R:9:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L9) *style:* **Variable or function name should be dotted.case.**
```r
  taskW = convertFDATaskToNormalTask(task = taskTs, method = "wavelets", pars = list(filter = "haar"))
  ^~~~~
```
[tests/testthat/test_FDA_Wavelets.R:10:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L10) *style:* **Variable or function name should be dotted.case.**
```r
  wData = getTaskData(taskW, target.extra = TRUE)
  ^~~~~
```
[tests/testthat/test_FDA_Wavelets.R:12:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L12) *style:* **Variable or function name should be dotted.case.**
```r
  taskWb = convertFDATaskToNormalTask(task = taskTs, method = "wavelets", pars = list( filter = "d4", boundary = "reflection"))
  ^~~~~~
```
[tests/testthat/test_FDA_Wavelets.R:13:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L13) *style:* **Variable or function name should be dotted.case.**
```r
  wbData = getTaskData(taskWb, target.extra = TRUE)
  ^~~~~~
```
[tests/testthat/test_FDA_Wavelets.R:19:17:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L19) *style:* **Commas should always have a space after.**
```r
    a = t(raw[i,])
                ^
```
[tests/testthat/test_FDA_Wavelets.R:22:34:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L22) *style:* **Commas should always have a space after.**
```r
    bb = rbind(bb, unlist(c(wt@W,wt@V[[wt@level]])))
                                 ^
```
[tests/testthat/test_FDA_Wavelets.R:27:17:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L27) *style:* **Commas should always have a space after.**
```r
    a = t(raw[i,])
                ^
```
[tests/testthat/test_FDA_Wavelets.R:29:34:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_Wavelets.R#L29) *style:* **Commas should always have a space after.**
```r
    dd = rbind(dd, unlist(c(wt@W,wt@V[[wt@level]])))
                                 ^
```
[tests/testthat/test_FDA_base_nonFDALrnOnFDATask.R:5:10:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_base_nonFDALrnOnFDATask.R#L5) *style:* **Commas should always have a space after.**
```r
  data[1,1] = NA
         ^
```
[tests/testthat/test_FDA_classif_glm.R:13:18:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_glm.R#L13) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  dat = list("df"=dataf,"x"=mlearn)
                ~^~
```
[tests/testthat/test_FDA_classif_glm.R:13:25:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_glm.R#L13) *style:* **Commas should always have a space after.**
```r
  dat = list("df"=dataf,"x"=mlearn)
                        ^
```
[tests/testthat/test_FDA_classif_glm.R:13:28:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_glm.R#L13) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  dat = list("df"=dataf,"x"=mlearn)
                          ~^~
```
[tests/testthat/test_FDA_classif_glm.R:19:20:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_glm.R#L19) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  newdat = list("x"=mtest)
                  ~^~
```
[tests/testthat/test_FDA_classif_glm.R:23:7:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_glm.R#L23) *style:* **Commas should always have a space after.**
```r
  ph[,"label"] = glearn
      ^
```
[tests/testthat/test_FDA_classif_np.R:21:7:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_np.R#L21) *style:* **Commas should always have a space after.**
```r
  ph[,"label"] = glearn
      ^
```
[tests/testthat/test_FDA_classif_shapelets.R:12:54:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_shapelets.R#L12) *style:* **Commas should always have a space after.**
```r
  m = shapeletLib::learnShapeletModel(data = gp[1:50,-1], label = as.factor(gp[1:50,1]))
                                                     ^
```
[tests/testthat/test_FDA_classif_shapelets.R:12:85:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_shapelets.R#L12) *style:* **Commas should always have a space after.**
```r
  m = shapeletLib::learnShapeletModel(data = gp[1:50,-1], label = as.factor(gp[1:50,1]))
                                                                                ^
```
[tests/testthat/test_FDA_classif_shapelets.R:13:58:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_shapelets.R#L13) *style:* **Commas should always have a space after.**
```r
  p1 = predict(object = m, newdata = as.matrix(gp[51:200,-1]))
                                                         ^
```
[tests/testthat/test_FDA_classif_shapelets.R:14:14:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_classif_shapelets.R#L14) *style:* **Commas should always have a space after.**
```r
  levs = c(2,1)
             ^
```
[tests/testthat/test_FDA_featExtract_FDboost.R:4:79:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L4) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1 = as.factor(c(-1,1,1,-1, 1)))
                                                                              ^
```
[tests/testthat/test_FDA_featExtract_FDboost.R:4:81:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L4) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1 = as.factor(c(-1,1,1,-1, 1)))
                                                                                ^
```
[tests/testthat/test_FDA_featExtract_FDboost.R:4:83:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L4) *style:* **Commas should always have a space after.**
```r
  gp = data.frame(v1 = 1:5, v2 = 2:6, v3 = 3:7, v4 = 4:8, X1 = as.factor(c(-1,1,1,-1, 1)))
                                                                                ^
```
[tests/testthat/test_FDA_featExtract_FDboost.R:5:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L5) *style:* **Variable or function name should be dotted.case.**
```r
  taskTs = makeFDAClassifTask(data = gp, target = "X1", positive = "1")
  ^~~~~~
```
[tests/testthat/test_FDA_featExtract_FDboost.R:6:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L6) *style:* **Variable or function name should be dotted.case.**
```r
  refData = getTaskData(taskTs, target.extra = TRUE)
  ^~~~~~~
```
[tests/testthat/test_FDA_featExtract_FDboost.R:11:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featExtract_FDboost.R#L11) *style:* **Variable or function name should be dotted.case.**
```r
  f_df = getFDAFDboostFeatures(data = refData$data, target = "X1", have.target = TRUE,
  ^~~~
```
[tests/testthat/test_FDA_featextraction_base.R:5:118:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L5) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
  d = extractFDAFeatures(data = gp$data, target = getTaskTargetNames(gunpoint.task), method = "wavelets", args = list( filter = "d4", boundary = "reflection"))
                                                                                ^
```
[tests/testthat/test_FDA_featextraction_base.R:21:127:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L21) *style:* **Commas should never have a space before.**
```r
  d = extractMultiFDAFeatures(data = fuel$data, target = getTaskTargetNames(fuelsubset.task), fd.features = tdesc$fd.features , method = "wavelets", args = list( filter = "d4", boundary = "reflection"))
                                                                                ~^
```
[tests/testthat/test_FDA_featextraction_base.R:23:127:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L23) *style:* **Commas should never have a space before.**
```r
  d = extractMultiFDAFeatures(data = fuel$data, target = getTaskTargetNames(fuelsubset.task), fd.features = tdesc$fd.features , method = "fourier", args = list())
                                                                                ~^
```
[tests/testthat/test_FDA_featextraction_base.R:25:127:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L25) *style:* **Commas should never have a space before.**
```r
  d = extractMultiFDAFeatures(data = fuel$data, target = getTaskTargetNames(fuelsubset.task), fd.features = tdesc$fd.features , method = "multiRes", args = list())
                                                                                ~^
```
[tests/testthat/test_FDA_featextraction_base.R:27:127:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L27) *style:* **Commas should never have a space before.**
```r
  d = extractMultiFDAFeatures(data = fuel$data, target = getTaskTargetNames(fuelsubset.task), fd.features = tdesc$fd.features , method = "bsignal", args = list())
                                                                                ~^
```
[tests/testthat/test_FDA_featextraction_base.R:29:127:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featextraction_base.R#L29) *style:* **Commas should never have a space before.**
```r
  d = extractMultiFDAFeatures(data = fuel$data, target = getTaskTargetNames(fuelsubset.task), fd.features = tdesc$fd.features , method = "fpca", args = list())
                                                                                ~^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:3:1:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L3) *style:* **Variable or function name should be functionCamel.case.**
```r
makeData_art_10 = function(n){
^~~~~~~~~~~~~~~
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:16:62:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L16) *style:* **Trailing whitespace is superfluous.**
```r
  l4c$y = 3 * l4c$x2 - 4 * l4c$x8 + 5 * l4c$x9 + 3 * rnorm(n)
                                                             ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:23:1:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L23) *style:* **Variable or function name should be functionCamel.case.**
```r
makeData_art_10_2 = function(n){
^~~~~~~~~~~~~~~~~
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:36:98:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L36) *style:* **Trailing whitespace is superfluous.**
```r
  l4c$y = 3 * (l4c$x2 * l4c$x3 ) * - 4 * (l4c$x8 * l4c$x7) + 5 * (l4c$x9 * l4c$x1) + 3 * rnorm(n)
                                                                                ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:47:78:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L47) *style:* **Trailing whitespace is superfluous.**
```r
  mmodel = selectForward2Step(d = regr.train, target = "medv", muffle = TRUE)
                                                                             ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:52:33:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L52) *style:* **Commas should always have a space after.**
```r
test_that("regr_twoStepAIC_art",{
                                ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:54:24:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L54) *style:* **Commas should always have a space after.**
```r
  regr.train = df[1:40,] # 4 times the feature number
                       ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:55:27:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L55) *style:* **Commas should always have a space after.**
```r
  regr.test = df[501:1000,]
                          ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:56:75:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L56) *style:* **Trailing whitespace is superfluous.**
```r
  mmodel = selectForward2Step(d = regr.train, target = "y", muffle = TRUE)
                                                                          ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:60:35:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L60) *style:* **Commas should always have a space after.**
```r
test_that("regr_twoStepAIC_art_2",{
                                  ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:62:24:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L62) *style:* **Commas should always have a space after.**
```r
  regr.train = df[1:40,]
                       ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:63:27:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L63) *style:* **Commas should always have a space after.**
```r
  regr.test = df[501:1000,]
                          ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:64:75:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L64) *style:* **Trailing whitespace is superfluous.**
```r
  mmodel = selectForward2Step(d = regr.train, target = "y", muffle = TRUE)
                                                                          ^
```
[tests/testthat/test_FDA_featsel_twoStepAIC.R:66:1:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_featsel_twoStepAIC.R#L66) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[tests/testthat/test_FDA_multiResNaive.R:6:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L6) *style:* **Variable or function name should be dotted.case.**
```r
  TL  = 200 # length of each time serie instance
  ^~
```
[tests/testthat/test_FDA_multiResNaive.R:7:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L7) *style:* **Variable or function name should be dotted.case.**
```r
  Ts = replicate(I, rnorm(TL))
  ^~
```
[tests/testthat/test_FDA_multiResNaive.R:9:36:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L9) *style:* **Remove spaces before the left parenthesis in a function call.**
```r
  ngp = extractFDAMultiResFeatures (data = gp, curve.lens = c(100, 100), res.level = 3, shift = 0.5)
                                   ^
```
[tests/testthat/test_FDA_multiResNaive.R:17:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L17) *style:* **Variable or function name should be dotted.case.**
```r
  TL  = 200 # length of each time serie instance
  ^~
```
[tests/testthat/test_FDA_multiResNaive.R:18:3:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L18) *style:* **Variable or function name should be dotted.case.**
```r
  Ts = replicate(I, rnorm(TL))
  ^~
```
[tests/testthat/test_FDA_multiResNaive.R:28:41:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L28) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  dfn = getMultiFDAMultiResFeatures(data= df, fd.features = fdf)
                                       ~^
```
[tests/testthat/test_FDA_multiResNaive.R:37:40:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L37) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  dfn = extractFDAMultiResFeatures(data= df, curve.lens = rep(180,2))
                                      ~^
```
[tests/testthat/test_FDA_multiResNaive.R:37:67:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_FDA_multiResNaive.R#L37) *style:* **Commas should always have a space after.**
```r
  dfn = extractFDAMultiResFeatures(data= df, curve.lens = rep(180,2))
                                                                  ^
```
[tests/testthat/test_Fda_convert_task.R:6:38:](https://github.com/mlr-org/mlr/blob/913a451a5e782b66625294e99fa5ba8a35bedbbd/tests/testthat/test_Fda_convert_task.R#L6) *style:* **Put spaces around all infix operators (except exponentiation).**
```r
  expect_true(nrow(getTaskData(task))== dim(fuel$data)[1])
                                    ~^
```


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 4.0.7.30871e20f7fd6053cc3800e60d7a8d7ecd89f7be](http://build.palaso.org/viewLog.html?buildId=88397&buildTypeId=Libpalaso_PalasoPrecise64masterContinuous) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:31


iya mam, ini kan juga masih di repo mu, tapi kan kemaren aku ada mlakukan perubahan tuh di file config.php nya, apa itu ndk papa?


On weekend i will create a version for PHP 5.5.21 and send you the link to download.


TeamCity DevOpsFinal :: Build [Build 28](http://localhost:9494/viewLog.html?buildId=54&buildTypeId=DevOpsFinal_Build) is now running


TeamCity DevOpsFinal :: Build [Build 28](http://localhost:9494/viewLog.html?buildId=54&buildTypeId=DevOpsFinal_Build) outcome was **SUCCESS**
Summary: Tests passed: 25 Build time: 00:00:17


:-) Mislim da nešto nedostaje u commitu...


TeamCity libpalaso :: palaso-precise64-libpalaso-3.1 Continuous [Build 3.1.235.30871e20f7fd6053cc3800e60d7a8d7ecd89f7be](http://build.palaso.org/viewLog.html?buildId=88400&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:36


да, все верно.
accepted


Maybe can you send a pull request... ?


closes #209


Hi Vinod

Did these changes to Allow Listing VM using Filters. viz: Datacenter, Cluster and Folder.

Change includes modifying the Helper Classes for Cluster and Folder to get the List of Clusters and Folders without relying on the Datacenter


I'm not a fan of this.


@has2k1 👍


@Seldaek this btw is also an issue for VCS based repository types where options are not passed to the remote filesystem. I am currently facing this issue with `gitlab` repositories in the context of Satis where I need to disable `ssl/verify_peer` because of a self-signed certifacte which is currently not possible. Would it make sense passing repository options there, too?


Radio sam hard reset glave i očito zeznuo u editoru, ali napravi li se diff sa prethodnim commitom trebala bi se vidjeti razlika u tome što je zamijenjen u 43. liniji "-2" koji sam previdio sa "EXIT_WARNING".
Mea culpa :D


I kind of disagree with some changes here especially when it comes to equality and the positioning of ended memberships, the intended behavior before was as follows:
- Voided memberships are dead last, followed by ended ones.


itu gak apa apa, krna udah di masukin ke daftar ignore (file: `.gitignore`)


how do we create games


`nn.Conv2d` should be created with `bias=not batch_norm`


"The synchronized version" doesn't really tell us anything ;)

Would be a bit more explicit: the implementation of `next()` and `seek(...)` are marked as `synchronized`.


I'd think this should call `OpenmrsUtil.compareWithNullAsGreatest()` as before because if cohortId is null, this is a new cohort that is going to get assigned a new DB PK value which is incremental so it's always going to be greater and not less.


I think it's to big copy & paste.
Not sure if there is a better way to do it though :/


@Emroni Doesn't the dist file needs to be minified?


extra space here


and here


What specifically don't you like?


Also Random card generation out of the deck, instead of completely random cards.
And end of game detection when cards run out


Zadnja 2 commita slobodno možete obrisati, tek sada vidim grešku..
Mea culpa.


が、また復活した。


where is this function used ?


there should be a division by 0 check here


not completed.


this should probably go to the component test spec, no?


It's actually redundant to have the public modifier on an interface method since all interface methods are public by default.


incomplete.


You don't need fold_case for ids.


Yeah, I thought briefly about wrapping all the `it()` statements in a function, but there were 3 that won't work on the directive - so then each `it()` statement would be a separate function.  It would be more DRY but perhaps more brittle.


Implement properly.


AFN2017 (jasondegraw) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.png)


Great, thanks for addressing my feedback!


Good catch on these!


Yeah and I don't think he understands how undesirable or impossible it is to slap Json.NET attributes on constructors. All configuration should be possible from an external assembly without having to own or modify the type itself.


You went ham on this! Thank you....  I'm going to study the logic for a bit now. Really I don't think I have much I can offer you, but if there ever is, let me know. Thanks again Josh


Fixes #60


Huh, was this giving you an error? Technically I think this would have worked since parsed is never reassigned... only its properties are modified.


done. but multiselection does not seem to work in qml

On Tue, 25 Apr 2017 at 13:50 Olivier Roulet-Dubonnet <
olivier.roulet@gmail.com> wrote:

> I will fix this
>
> On Tue, 25 Apr 2017 at 13:48 Michael Fuchs <notifications@github.com>
> wrote:
>
>> @oroulet <https://github.com/oroulet> could you please investigate why
>> this crashes?
>>
>> —
>> You are receiving this because you were mentioned.
>>
>>
>> Reply to this email directly, view it on GitHub
>> <https://github.com/QTodoTxt/QTodoTxt2/commit/aee0e0d5db62172a8baf29bc0adab261167a5174#commitcomment-21894713>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/ACcfznCLOOZwxWWxJh0tdAOxgBcrPwS8ks5rzd12gaJpZM4NHVqG>
>> .
>>
>


AFN2017 (jasondegraw) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.png)


I didn't think we needed to expose this


I don't know a good way to get a handle on a plain element.  This is the best I could come up with [used here](#diff-299c86566fae5edd07a93042029f4e18R96).


nit: Beginning test names with "should" creates an arms race where soon every test will have to start with "should".


May I ask why you ended up reverting this one


you did it already. so lets try. I never managed to make submodules work

On Tue, 25 Apr 2017 at 17:53 Michael Fuchs <notifications@github.com> wrote:

> What about forking the Images repo as "Theme" and using it as submodule
> in QTodoTxt2?
>
> Am 25.04.2017 um 14:26 schrieb oroulet:
> > icons and compile script are in images repository in
> > https://github.com/QTodoTxt/Images Then the current process is to copy
> the
> > qrc file in qtodotxt repository
> >
> > On Tue, 25 Apr 2017 at 14:23 Michael Fuchs <notifications@github.com>
> wrote:
> >
> > > Ok.
> > > @icons <https://github.com/icons>: sure I will try to set the icons
> like
> > > they where in QTodoTxt. I have at least two potential solutions for
> this.
> > > Will Have to play around with them.
> > > Two questions concerning this:
> > >
> > > 1. Did you compile the qrc files yourself?
> > > 2. If yes, are you willing to share the icons in this repo?
> > > It would be easier for me to play around with the icons and themeing
> > > stuff. Later we can put them back into qrc's if we want to.
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > >
> > <
> https://github.com/QTodoTxt/QTodoTxt2/commit/81941ebac78d45b9e68b83ec0ff5f9b56c8d2dd7#commitcomment-21895108
> >,
> > > or mute the thread
> > >
> > <
> https://github.com/notifications/unsubscribe-auth/ACcfzrlKZCOcTFmqhIvIgpReoj9RsmhAks5rzeWpgaJpZM4NFuDY
> >
> > > .
> > >
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly, view it on GitHub
> > <
> https://github.com/QTodoTxt/QTodoTxt2/commit/81941ebac78d45b9e68b83ec0ff5f9b56c8d2dd7#commitcomment-21895146>,
>
> > or mute the thread
> > <
> https://github.com/notifications/unsubscribe-auth/ACw_MaGEPaZUaO1ALJ6yvCZ4pghzDWVuks5rzeZwgaJpZM4NFuDY
> >.
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/QTodoTxt/QTodoTxt2/commit/81941ebac78d45b9e68b83ec0ff5f9b56c8d2dd7#commitcomment-21898907>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACcfzs1EYFj__ftVwM3Kmdf7W4pvfeYvks5rzhbcgaJpZM4NFuDY>
> .
>


Alpine 3.5 packages php 7.0.16, this basically reopens #130 and #155. I've been spending at least 3 hours troubleshooting this. I've run out of motivation to run nextcloud with this. Are there any workarounds/fixes you can propose before I finally lose my mind?


application/*
assets/*


We're special casing in code things for specific providers. Same with the filter-replace code below. This info should be stored in the preprint_provider whenever possible, and should use a simple translation if not (for example, this one could be "remove all alpha characters and lowecase". Then we have to get sales and product to not promise things outside of these boundaries.


many thanks for your effort!!!


nit: No need for the "done" callback because there aren't any async methods here.


AFN2017 (jasondegraw) - x86_64-MacOS-10.10-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.png)


I think you mean Chrome :P


checking for config is redundant, you have to merge it with the default config.


For 'if' + 'else' construct it is preferable to avoid negation in the condition.


I'm getting a different cert for that IP, MIIIGjCCBwKgAwIBAgIIJgfaZudcgc4wDQYJKoZIh.... right now.


pls remove


<details>

<summary>Submitting the task to TaskCluster failed. Details</summary>

```js

unknown escape sequence at line 20, column 41:
     ...   - "set ChocolateyInstall=%CD%\chocolatey"
                                         ^
```

</details>


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.8.3b5c304a9f4820b9e0e04271e17e702dc8f4f185](http://build.palaso.org/viewLog.html?buildId=88407&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Compilation error: SIL.Windows.Forms\SIL.Windows.Forms.csproj (new); compilation error: SIL.DblBundle\SIL.DblBundle.csproj (new) Build time: 00:00:36


#2520943-WindowsCalc-Engine_Integration (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (1198 of 1772 tests passed, 0 test warnings)

Failures:

integration Test Summary
 * Passed: 1
 * Failed: 574

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-67.61%25-red.png) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-36.71%25-red.png)


$ is redundant?


Nice! Thanks man!!


Yeah, unused.


#2520943-WindowsCalc-Engine_Integration (vidanovic) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.png)


I think tests like that should be in the directive's spec file


A `TupleTag` is a purely local name; it is never appropriate for a runner to expect that the `TupleTag` of an expanded `PInput` or `POutput` is consistent across any other instance, just that any expansion of the same `PInput` or `POutput` is consistent between calls to `#expand()`. Runners should use the `PValue` as the identity of the thing that needs to be related.


Please send a pull request, we will not be using xml for this project but if you commit this as a list I can update the sql database to match it.


Ok, I'll move it there.


Lucas Added Credits Class


non on a besoin de cette condition car dans le page.php on a parfois pas cette partie du tableau (pas de tiny ni 1 ni 0 c'est d'ailleurs écrit dans la doc du fichier) donc on en a besoin pour éviter les erreurs php ....


You are mutating the object passed from the user here, this is not a good practice, please use as following:
```
const _config = {...{}, ...config};
```


Ref: https://github.com/curl/curl/pull/1445


add line break before return statement pls


AFN2017 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.png)


Test comment


okok mam, nah untuk logo sama foto yang mau di tampilin di homenya udah ku send via email mam, check ya.
untuk fotonya itu mam di crop aja mam, di pas pas in aja biar rapih gitu hehe, trs penempatannya di bawah selamat datang itu aja, untuk logonya di tempatin di samping kirinya selamat datang itu aja, ok mam? nnti kalo udah tak check lagi, thank


и правда)


Property binding in 2 ways.


You don't need to do this.  These are for building out the ref folder for netstandard1.x versions so that they can contain some set of package-based references.  We only do this if we need to build some netstandard1.x lib and that needs to reference one of these.


oiya sekalian untuk form login nya itu mam agak di keatasin aja ya hehe, soalnya disini ndk keliatan, pokoknya yang tak omongin tadilah yang pas di kampus


remove spaces after `{` and before `}` (there are few more places you used this style of code, pls remove there as well)


Well, there is [ContractResolver](http://www.newtonsoft.com/json/help/html/ContractResolver.htm) for situations where you have no other choice. But I would argue that immutable types and multiple constructors is not something out of ordinary that should require extreme measures.

And again, right now, the most troublesome are unintended consequences of introducing additional constructors to an immutable struct. It can silently bring everything down and detecting the root cause is far from obvious.


You'll need to obscure the RID-specific runtime assets under runtimes/win/lib/netstandard1.6.  You can do that like this: https://github.com/dotnet/corefx/blob/72002dbb7e3e657adaeb002feca0399d9667a54b/src/System.Reflection.DispatchProxy/pkg/System.Reflection.DispatchProxy.pkgproj#L18


`memo.select(&@user_filters.fetch(key))` сочетание &@ всё ещё пугает мои глаза и мозг


oh ini udah di change ya mam logo box nya? ok thank mam


@smadha
> Can you please manual merge instead of me updating PR?

Do you mean undo/edit these changes directly on master branch? I am not sure if that's a good idea!

>I'll have to merge this PR in my branch and it'll make commit history look weird, what say?

Since these changes are now in master, rebasing your PR to the latest commit on master is recommended!

>  it'll make commit history look weird,

Nope, it doesn't... Merging with upstream master doesn't mess up your PR log


This note belongs with the bmp280 option.


The space in FocUS Archive is the problem.  I added a translation that splits out the whitespace in addition to lowercasing?


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181740 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181740") to [prod](https://console.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181748 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181748") to [prod](https://console.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170425-181751 "deployed-2-env-Prod-20170425-181751") to [Prod](https://console.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181801 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181801") to [prod](https://console.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testrepo-traceability](https://github.com/idsdev/testrepo-traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181811 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170425-181811") to [prod](https://console.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "ibm:yp:us-south:user@ibm.com:prod"), including this commit


AFN2017 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.png)


brrti nanti tinggal tak
**

 **git pull master origin** lagi nih?


I would really make this a top-level class to keep the nesting manageable and the flow of typeProcess understandable.


The `in` seems unnecessary.


donc la on va planter(php) sur toutes les pages qui n'ont pas tiny ??


I would add that this is the checker that generated the error message and that the compound checker that contains this checker then uses this to sort the error messages.


Das funktioniert scheinbar nicht so gut. Wenn man von der Startseite aus einfach auf das Logo klickt, dann ändert die URL auf `/sprache/katalog/version` und das root-Element verschwindet. Nach zwei weiteren Klicks aus diesem Zustand auf das Logo erscheinen Fehler in der Konsole.

Das Problem ist, dass die `MainComponent `nicht auf das root-Element weiterleitet. Denn die Parameter der Route ändern ihren Wert ja nicht. Entsprechend wird das `subscribe` in Zeile 48 von `MainComponent` nicht ausgeführt. Ich habe leider keine schnelle Lösung gefunden. @sserena oder @brnsnt: Seht ihr da vielleicht eine gute Lösung für dieses Problem?


Ok, I think we don't need all of this test. It will be a pain in the ass to maintain.
All the day-calendar functionality is already tested.

Just add few tests to see that the popup is attached correctly to the custom input.


There are no changes for Lesson-8. All changes for Lesson-8 is covered in the changes implemented for Lesson-7


Esto es un comentario


Good spotting! Yes that is a hipster bug, but am too buzy not caring about whitespace to fix it.

But newlines is part of how hipster works, or something like that, you can see from the comment that i intended to add \n "to every line but the last" but then for some reason changed to adding it on every line.


`react-test-renderer` is making movements in the  React community, eh?


same as bellow.


Ok.  It did help me catch a few bugs in my directive code though.  The `theme` and `onOpenDelay` weren't working correctly.  I'll leave those tests in.


Do we really want to make this inbox?  I know we'll need to carry an impl but I would hope that we could avoid exposing any ref that throws on some of the NETCoreApp platforms.

Compare this to OpenSSL, where we carry the impl but don't expose the ref: https://github.com/dotnet/corefx/blob/ac09a797d09502ebab61f7167bb856846602e1e6/src/System.Security.Cryptography.OpenSsl/dir.props#L8

/cc @weshaggard @bartonjs


Cool! thanks


Instead, please add CNG to https://github.com/dotnet/corefx/blob/master/external/harvestPackages/harvestPackages.netstandard1.6.depproj


Can't connect to the internet.


The following doc(s) have been deployed to the [laurenkell](https://laurenkell-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/switching-dns](https://laurenkell-static-docs.pantheonsite.io/docs/switching-dns)


This kind of thing makes me sad. :(


No comentes pelotudeces


@TDMidkwhatisxd In no way does that mean that the software is ok. Just because they are a part of the organization does not mean that they will make any effort to fix the software. ATM they have not made any effort, and I do not think that they plan to.


Please add the css classes rather than replacing existing ones


Belum de, nambahin logo mungkin besok. Itu baru posisi nya doang.


please put whees\ back in the index file before committing


@pierre Looks like when we use the same `order_id` for querying that payment. The on-demand report api will return the successful authorization when we intended to query an unknown capture. Same happens for refund.


please do another merger from master to your fork


ne halwa chala bagundi ra


TeamCity DevOpsFinal :: Build [Build 29](http://localhost:9494/viewLog.html?buildId=58&buildTypeId=DevOpsFinal_Build) is now running


I'm pretty sure I wrote it, but never tested it. Then forgot about it.


TeamCity DevOpsFinal :: Build [Build 29](http://localhost:9494/viewLog.html?buildId=58&buildTypeId=DevOpsFinal_Build) outcome was **SUCCESS**
Summary: Tests passed: 25 Build time: 00:00:19


![](https://github.trello.services/images/mini-trello-icon.png) [6502 Compiler Instructions](https://trello.com/c/AsoldMaG/204-6502-compiler-instructions)


### Presentation Slides


Hey @craig,

What are you seeing? Are Google pushing new certs quicker than we can keep track of? :-\


Additionally, we need to think about how this affects metasmoke autoflagging. My gut says it could inflate weight - right now a manually reported post will never trigger flags because 100<180. On the other hand, this should only come into play if we actually missed something.


This file contains a lot of unnecessary whitespace conversions from tab to spaces. I'd suggest switching to tabbed indentation altogether...
For IntelliJ this is done via: File ->Settings ->Editor ->Code Style -> Java -> Tabs and Indents: [x] Use Tab character


It's not very important here, but I would remove set completely if it becomes empty after `delete`.


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/fd18c224-02cc-44bb-add5-d687f18d5023 "weather-app"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/b3ff289f-cd59-4cef-ade4-33e0cc1cd438 "weather-app") deployed [weather-app](https://github.com/robinbobbitt/weather-app/releases/tag/deploy-ibm-yp-us-south-ryehle-us.ibm.com-staging-20170425-183243 "deploy-ibm-yp-us-south-ryehle-us.ibm.com-staging-20170425-183243") to [staging](https://console.ng.bluemix.net/apps/989e27ab-e25b-4454-8405-9ffda1404641 "ibm:yp:us-south:ryehle@us.ibm.com:staging"), including this commit


You so smart Graceface


I see some redundancy here. `diffHMWithHSKeys` deletes all deleted proposals from index and `resultActiveProposals` does it too. I would remove `diffHMWithHSKeys` because `pmDelActivePropsIdx` will be removed soon.


fgdfgdf


Thanks baammaridi!


https://github.com/ZeroK-RTS/Zero-K/issues/2171 related?


yes, can be closed


@crunchyNYC
Thanks!

Make sure you take care of yourself...
I really appreciate your contributions.


Hi,
Hum, not that easy if you are totally new to C++.
Let me give you some starting points, but you will need to do some additional reading :

**install opencv**
You will need to install opencv on your computer.
The easiest way is to download it from here : http://opencv.org/releases.html
The more difficult way(although worthwhile) is to compile it by yourself using cmake (see for example this tutorial http://perso.uclouvain.be/allan.barrea/opencv/opencv.html)

** Configure the opencv path inside CMakeLists.txt**
You need to edit this file and modify the line number 4:
`set(opencv_3_1_path` add_path_to_your_opencv_specific_install_if_needed)`

Just enter the correct path for your opencv installation. Beware : replace backslashes (\) with forward slashes (/) inside your path.

**cmake**
This project uses cmake : cmake is a utillity that will generate a visual studio project for this code.

Method 1 : install cmake (cmake.org), and follow the instructions here : https://cmake.org/runningcmake/

Method 2 : luckily for you, Visual Studio 2017 has just added direct support for cmake. You might be able to open this folder without installing cmake, directly for MSVC. See https://www.youtube.com/watch?v=_lKxJjV8r3Y


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/fd18c224-02cc-44bb-add5-d687f18d5023 "weather-app"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/b3ff289f-cd59-4cef-ade4-33e0cc1cd438 "weather-app") deployed [weather-app](https://github.com/robinbobbitt/weather-app/releases/tag/deploy-ibm-yp-us-south-ryehle-us.ibm.com-prod-20170425-183753 "deploy-ibm-yp-us-south-ryehle-us.ibm.com-prod-20170425-183753") to [prod](https://console.ng.bluemix.net/apps/63cc1e3f-9eb6-4aec-b5ba-64d75dee1a22 "ibm:yp:us-south:ryehle@us.ibm.com:prod"), including this commit


Thanks! :+1:


Thanks it worked :)


What it is this commit??


the md5-hash changes because we're minifying first and then aggregating minified code, vs before aggregating unminified code first and minifying second.


@klaxa Have you tried? If it's the case, I would contact the maintainer.
I was tired of fixing bugs so I did this migration. But if it's still buggy, I will make my own image nginx/php image.


Same! I dropped everything to come check!


thanks!


@miguelcleon seems like the blank contained information about roles.. I am getting errors:
```
could not execute query: ERROR:  role "czimea" does not exist
    Command was: REVOKE ALL ON SCHEMA odm2extra FROM PUBLIC;
```


I also changed the way substitution works: we now perform substitution on the postfix expression, with a modification to the shunting yard algorithm to accept the token "x" as a number. This is because when we actually implement the trapezium rule, we'll need to substitute in a lot of different values. Doing it before it goes into postfix means running shunting yard on every substitution.


Aw I wanted to see what emo hadley looked like.

I was literally about to send a PR to fix this ;)


It doesn't seem to me like it should be 'inbox', but it is part of the netcore.app closure so I included it in this branch per yours/jeremy's comments on the [main pr](https://github.com/dotnet/corefx/pull/18689#discussion_r112585931) where I had removed it from Inbox.

Imo it would be better to remove it if we can.


Ok