[R/approxSSM.R:67:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L67) *style:* **Variable and function names should be all lowercase.**
```r
approxSSM <-
^~~~~~~~~
```
[R/approxSSM.R:67:13:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L67) *style:* **Trailing whitespace is superfluous.**
```r
approxSSM <-
            ^
```
[R/approxSSM.R:69:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L69) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:70:15:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L70) *style:* **Put spaces around all infix operators.**
```r
    if(maxiter<1)
             ~^~
```
[R/approxSSM.R:71:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L71) *style:* **Trailing whitespace is superfluous.**
```r
      stop("Argument maxiter must a positive integer. ")
                                                        ^~
```
[R/approxSSM.R:72:11:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L72) *style:* **Put spaces around all infix operators.**
```r
    if(tol<0)
         ~^~
```
[R/approxSSM.R:74:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L74) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:76:5:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L76) *warning:* **no visible global function definition for ‘is.SSModel’**
```r
    is.SSModel(model, na.check = TRUE, return.logical = FALSE)
    ^~~~~~~~~~
```
[R/approxSSM.R:77:47:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L77) *style:* **Trailing whitespace is superfluous.**
```r
    if (all(model$distribution == "gaussian"))
                                              ^
```
[R/approxSSM.R:78:68:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L78) *style:* **Trailing whitespace is superfluous.**
```r
      stop("Model is completely Gaussian, nothing to approximate.")
                                                                   ^~~~
```
[R/approxSSM.R:79:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L79) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:86:37:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L86) *style:* **Trailing whitespace is superfluous.**
```r
    storage.mode(ymiss) <- "integer"
                                    ^~~~
```
[R/approxSSM.R:87:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L87) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:90:16:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L90) *warning:* **no visible global function definition for ‘init_theta’**
```r
      theta <- init_theta(model$y, model$u, model$distribution)
               ^~~~~~~~~~
```
[R/approxSSM.R:92:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L92) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:94:52:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L94) *style:* **Trailing whitespace is superfluous.**
```r
                   table = c("gaussian", "poisson",
                                                   ^
```
[R/approxSSM.R:96:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L96) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/approxSSM.R:98:11:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L98) *style:* **Trailing whitespace is superfluous.**
```r
    out <-
          ^
```
[R/approxSSM.R:99:66:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L99) *style:* **Trailing whitespace is superfluous.**
```r
      .Fortran(fapprox, NAOK = TRUE, model$y, ymiss, tv, model$Z,
                                                                 ^
```
[R/approxSSM.R:100:73:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L100) *style:* **Trailing whitespace is superfluous.**
```r
               model$T, model$R, Htilde = array(0, c(p, p, n)), model$Q,
                                                                        ^
```
[R/approxSSM.R:101:54:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L101) *style:* **Trailing whitespace is superfluous.**
```r
               model$a1, model$P1, model$P1inf, p, n,
                                                     ^
```
[R/approxSSM.R:102:45:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L102) *style:* **Trailing whitespace is superfluous.**
```r
               m, k, theta = theta, model$u,
                                            ^
```
[R/approxSSM.R:103:55:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L103) *style:* **Trailing whitespace is superfluous.**
```r
               ytilde = array(0, dim = c(n, p)), dist,
                                                      ^
```
[R/approxSSM.R:104:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L104) *style:* **Trailing whitespace is superfluous.**
```r
               maxiter = as.integer(maxiter), model$tol,
                                                        ^
```
[R/approxSSM.R:105:45:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L105) *style:* **Trailing whitespace is superfluous.**
```r
               as.integer(sum(model$P1inf)),
                                            ^
```
[R/approxSSM.R:107:16:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L107) *style:* **Put spaces around all infix operators.**
```r
    if(out$info!=0){
              ~^~~
```
[R/approxSSM.R:107:20:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L107) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if(out$info!=0){
                   ^
```
[R/approxSSM.R:107:21:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L107) *style:* **Trailing whitespace is superfluous.**
```r
    if(out$info!=0){
                    ^~~~~~~~
```
[R/approxSSM.R:111:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L111) *style:* **lines should not be more than 80 characters.**
```r
                     "1" = "Gaussian approximation failed due to non-finite value in linear predictor.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/approxSSM.R:112:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L112) *style:* **lines should not be more than 80 characters.**
```r
                     "2" = "Gaussian approximation failed due to non-finite value of p(theta|y)."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/approxSSM.R:113:9:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L113) *style:* **Trailing whitespace is superfluous.**
```r
      ))
        ^
```
[R/approxSSM.R:114:18:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L114) *style:* **Put spaces around all infix operators.**
```r
      if(out$info!=3) return(-.Machine$double.xmax^0.75)
                ~^~~
```
[R/approxSSM.R:114:51:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L114) *style:* **Put spaces around all infix operators.**
```r
      if(out$info!=3) return(-.Machine$double.xmax^0.75)
                                                 ~^~
```
[R/approxSSM.R:116:16:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L116) *style:* **Put spaces around all infix operators.**
```r
    if(out$info!=0){
              ~^~~
```
[R/approxSSM.R:117:19:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L117) *style:* **Put spaces around all infix operators.**
```r
      if (out$info==1) {
                 ~^~~
```
[R/approxSSM.R:118:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L118) *style:* **lines should not be more than 80 characters.**
```r
        stop("Non-finite value of likelihood or linear predictor in approximation algorithm.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/approxSSM.R:120:18:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L120) *style:* **Put spaces around all infix operators.**
```r
      if(out$info==2){
                ~^~~
```
[R/approxSSM.R:121:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L121) *style:* **lines should not be more than 80 characters.**
```r
        warning(paste("Maximum number of iterations reached, latest difference was", signif(out$diff,3)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/approxSSM.R:122:8:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L122) *style:* **Trailing whitespace is superfluous.**
```r
      }
       ^~~
```
[R/approxSSM.R:124:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L124) *style:* **Trailing whitespace is superfluous.**
```r

^~~
```
[R/approxSSM.R:134:4:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/approxSSM.R#L134) *style:* **Trailing whitespace is superfluous.**
```r
  }
   ^
```
[R/artransform.R:13:15:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/artransform.R#L13) *style:* **Trailing whitespace is superfluous.**
```r
artransform <-
              ^
```
[R/artransform.R:17:60:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/artransform.R#L17) *style:* **Trailing whitespace is superfluous.**
```r
    .Fortran("fartransform", as.double(param), phi = param,
                                                           ^
```
[R/artransform.R:19:2:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/artransform.R#L19) *style:* **Trailing whitespace is superfluous.**
```r
}
 ^
```
[R/checkModel.R:26:14:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L26) *style:* **Trailing whitespace is superfluous.**
```r
is.SSModel <-
             ^
```
[R/checkModel.R:28:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L28) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/checkModel.R:29:8:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L29) *style:* **Put spaces around all infix operators.**
```r
    tol<-1e7
      ~^~~
```
[R/checkModel.R:35:25:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L35) *style:* **Trailing whitespace is superfluous.**
```r
    one <- as.integer(1)
                        ^~~~
```
[R/checkModel.R:37:42:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L37) *style:* **Trailing whitespace is superfluous.**
```r
      x <- inherits(object, "SSModel") &&
                                         ^
```
[R/checkModel.R:38:69:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L38) *style:* **Trailing whitespace is superfluous.**
```r
        all(c("y", "Z", "H", "T", "R", "Q", "a1","P1", "P1inf", "u",
                                                                    ^
```
[R/checkModel.R:39:68:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L39) *style:* **Trailing whitespace is superfluous.**
```r
              "distribution", "tol", "call") %in% names(object)) &&
                                                                   ^
```
[R/checkModel.R:40:37:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
        all(object$distribution %in%
                                    ^
```
[R/checkModel.R:41:60:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L41) *style:* **Trailing whitespace is superfluous.**
```r
              c("gaussian", "poisson", "binomial", "gamma",
                                                           ^
```
[R/checkModel.R:45:51:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L45) *style:* **Trailing whitespace is superfluous.**
```r
        (identical(dim(object$Z), c(p, m, one)) ||
                                                  ^
```
[R/checkModel.R:47:43:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L47) *style:* **Trailing whitespace is superfluous.**
```r
        (identical(object$H, "Omitted") ||
                                          ^
```
[R/checkModel.R:50:51:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L50) *style:* **Trailing whitespace is superfluous.**
```r
        (identical(dim(object$T), c(m, m, one)) ||
                                                  ^
```
[R/checkModel.R:52:51:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L52) *style:* **Trailing whitespace is superfluous.**
```r
        (identical(dim(object$R), c(m, k, one)) ||
                                                  ^
```
[R/checkModel.R:54:51:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L54) *style:* **Trailing whitespace is superfluous.**
```r
        (identical(dim(object$Q), c(k, k, one)) ||
                                                  ^
```
[R/checkModel.R:59:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L59) *style:* **lines should not be more than 80 characters.**
```r
        (identical(object$u, "Omitted") || identical(dim(object$u), dim(object$y))) &&
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:60:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L60) *style:* **lines should not be more than 80 characters.**
```r
        all(diag(object$P1inf)%in%c(0,1)) && all(object$P1inf[col(diag(m))!=row(diag(m))]==0)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:60:31:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L60) *style:* **Put spaces around all infix operators.**
```r
        all(diag(object$P1inf)%in%c(0,1)) && all(object$P1inf[col(diag(m))!=row(diag(m))]==0)
                             ~^~~~~
```
[R/checkModel.R:60:75:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L60) *style:* **Put spaces around all infix operators.**
```r
        all(diag(object$P1inf)%in%c(0,1)) && all(object$P1inf[col(diag(m))!=row(diag(m))]==0)
                                                                         ~^~~
```
[R/checkModel.R:60:90:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L60) *style:* **Put spaces around all infix operators.**
```r
        all(diag(object$P1inf)%in%c(0,1)) && all(object$P1inf[col(diag(m))!=row(diag(m))]==0)
                                                                                ~^~~
```
[R/checkModel.R:61:20:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L61) *style:* **Trailing whitespace is superfluous.**
```r
      if (na.check)
                   ^
```
[R/checkModel.R:62:18:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L62) *style:* **Trailing whitespace is superfluous.**
```r
        x <- x &&
                 ^~~~~~~~~~
```
[R/checkModel.R:63:26:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L63) *style:* **Commas should never have a space before.**
```r
             'gamma' and ,'negative binomial'.")
                        ~^
```
[R/checkModel.R:63:26:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L63) *style:* **Commas should always have a space after.**
```r
             'gamma' and ,'negative binomial'.")
                         ^
```
[R/checkModel.R:64:72:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L64) *style:* **Trailing whitespace is superfluous.**
```r
          any(is.na(object[[x]])) || any(is.infinite(object[[x]])))) &&
                                                                       ^
```
[R/checkModel.R:65:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L65) *style:* **lines should not be more than 80 characters.**
```r
        max(object$Q)<=tol && ifelse(identical(object$u, "Omitted"), max(object$H)<=tol,TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:65:22:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L65) *style:* **Put spaces around all infix operators.**
```r
        max(object$Q)<=tol && ifelse(identical(object$u, "Omitted"), max(object$H)<=tol,TRUE)
                    ~^~~
```
[R/checkModel.R:65:83:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L65) *style:* **Put spaces around all infix operators.**
```r
        max(object$Q)<=tol && ifelse(identical(object$u, "Omitted"), max(object$H)<=tol,TRUE)
                                                                                ~^~~
```
[R/checkModel.R:68:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L68) *style:* **Trailing whitespace is superfluous.**
```r
      if (!inherits(object, "SSModel"))
                                       ^
```
[R/checkModel.R:70:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L70) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:72:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L72) *style:* **lines should not be more than 80 characters.**
```r
        stop("Storage mode of some of the model attributes 'p', 'k', 'm', 'n', 'tv'
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:72:84:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L72) *style:* **Trailing whitespace is superfluous.**
```r
        stop("Storage mode of some of the model attributes 'p', 'k', 'm', 'n', 'tv'
                                                                                ^
```
[R/checkModel.R:74:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L74) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:75:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L75) *style:* **Trailing whitespace is superfluous.**
```r
      components <- c("y", "Z", "H", "T", "R", "Q", "a1", "P1", "P1inf", "u",
                                                                             ^
```
[R/checkModel.R:76:53:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L76) *style:* **Trailing whitespace is superfluous.**
```r
                      "distribution", "tol", "call")
                                                    ^~~~~~
```
[R/checkModel.R:77:47:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L77) *style:* **Trailing whitespace is superfluous.**
```r
      if (!all(components %in% names(object)))
                                              ^
```
[R/checkModel.R:78:69:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L78) *style:* **Trailing whitespace is superfluous.**
```r
        stop(paste("Model is not a proper object of class 'SSModel'.
                                                                    ^
```
[R/checkModel.R:79:56:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L79) *style:* **Trailing whitespace is superfluous.**
```r
                   Following components are missing: ",
                                                       ^
```
[R/checkModel.R:80:71:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L80) *style:* **Trailing whitespace is superfluous.**
```r
                   paste(components[!(components %in% names(object))],
                                                                      ^
```
[R/checkModel.R:82:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L82) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:83:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L83) *style:* **Trailing whitespace is superfluous.**
```r
      if (!all(object$distribution %in%
                                       ^
```
[R/checkModel.R:84:63:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L84) *style:* **Trailing whitespace is superfluous.**
```r
                 c("gaussian", "poisson", "binomial", "gamma",
                                                              ^
```
[R/checkModel.R:85:42:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L85) *style:* **Trailing whitespace is superfluous.**
```r
                   "negative binomial")))
                                         ^
```
[R/checkModel.R:86:67:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L86) *style:* **Trailing whitespace is superfluous.**
```r
        stop("The distributions of the observations are not valid.
                                                                  ^
```
[R/checkModel.R:87:69:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L87) *style:* **Trailing whitespace is superfluous.**
```r
             Possible choices are 'gaussian', 'poisson', 'binomial',
                                                                    ^
```
[R/checkModel.R:89:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L89) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:90:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L90) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:92:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L92) *style:* **Trailing whitespace is superfluous.**
```r
              (identical(dim(object$Z), c(p, m, one)) ||
                                                        ^
```
[R/checkModel.R:94:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L94) *style:* **lines should not be more than 80 characters.**
```r
              (identical(object$H, "Omitted") || identical(dim(object$H), c(p, p, one)) ||
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:96:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L96) *style:* **Trailing whitespace is superfluous.**
```r
              (identical(dim(object$T), c(m, m, one)) ||
                                                        ^
```
[R/checkModel.R:98:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L98) *style:* **Trailing whitespace is superfluous.**
```r
              (identical(dim(object$R), c(m, k, one)) ||
                                                        ^
```
[R/checkModel.R:100:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L100) *style:* **Trailing whitespace is superfluous.**
```r
              (identical(dim(object$Q), c(k, k, one)) ||
                                                        ^
```
[R/checkModel.R:105:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L105) *style:* **lines should not be more than 80 characters.**
```r
              (identical(object$u, "Omitted") || identical(dim(object$u), dim(object$y)))))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:105:92:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L105) *style:* **Trailing whitespace is superfluous.**
```r
              (identical(object$u, "Omitted") || identical(dim(object$u), dim(object$y)))))
                                                                                ^
```
[R/checkModel.R:106:61:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L106) *style:* **Trailing whitespace is superfluous.**
```r
      stop("Model is not a proper object of class 'SSModel'.
                                                            ^
```
[R/checkModel.R:108:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L108) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:109:30:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L109) *style:* **Trailing whitespace is superfluous.**
```r
      if (na.check == TRUE &&
                             ^
```
[R/checkModel.R:110:73:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L110) *style:* **Trailing whitespace is superfluous.**
```r
            (any(sapply(c("H", "u", "T", "R", "Q", "a1", "P1", "P1inf"),
                                                                        ^
```
[R/checkModel.R:111:63:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L111) *style:* **Trailing whitespace is superfluous.**
```r
                        function(x) any(is.na(object[[x]])) ||
                                                              ^
```
[R/checkModel.R:113:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L113) *style:* **lines should not be more than 80 characters.**
```r
               max(object$Q)>tol || ifelse(identical(object$u, "Omitted"), max(object$H)>tol,FALSE)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:113:29:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L113) *style:* **Put spaces around all infix operators.**
```r
               max(object$Q)>tol || ifelse(identical(object$u, "Omitted"), max(object$H)>tol,FALSE)))
                           ~^~
```
[R/checkModel.R:113:89:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L113) *style:* **Put spaces around all infix operators.**
```r
               max(object$Q)>tol || ifelse(identical(object$u, "Omitted"), max(object$H)>tol,FALSE)))
                                                                                ~^~
```
[R/checkModel.R:114:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L114) *style:* **lines should not be more than 80 characters.**
```r
        stop(paste("System matrices (excluding Z) contain NA or infinite values, or covariance matrices contain values larger than",tol))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:115:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L115) *style:* **Trailing whitespace is superfluous.**
```r

^~~~~~
```
[R/checkModel.R:116:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L116) *style:* **lines should not be more than 80 characters.**
```r
      if(!all(diag(object$P1inf)%in%c(0,1)) || !all(object$P1inf[col(diag(m))!=row(diag(m))]==0))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:116:33:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L116) *style:* **Put spaces around all infix operators.**
```r
      if(!all(diag(object$P1inf)%in%c(0,1)) || !all(object$P1inf[col(diag(m))!=row(diag(m))]==0))
                               ~^~~~~
```
[R/checkModel.R:116:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L116) *style:* **Put spaces around all infix operators.**
```r
      if(!all(diag(object$P1inf)%in%c(0,1)) || !all(object$P1inf[col(diag(m))!=row(diag(m))]==0))
                                                                            ~^~~
```
[R/checkModel.R:116:93:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L116) *style:* **Put spaces around all infix operators.**
```r
      if(!all(diag(object$P1inf)%in%c(0,1)) || !all(object$P1inf[col(diag(m))!=row(diag(m))]==0))
                                                                                ~^~~
```
[R/checkModel.R:117:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L117) *style:* **lines should not be more than 80 characters.**
```r
        stop("Matrix P1inf is not a diagonal matrix with zeros and ones on diagonal.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkModel.R:119:4:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/checkModel.R#L119) *style:* **Trailing whitespace is superfluous.**
```r
  }
   ^
```
[R/coef.KFS.R:23:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/coef.KFS.R#L23) *style:* **lines should not be more than 80 characters.**
```r
        } else stop("Input does not contain smoothed estimates for states, rerun KFS with state smoothing.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/coef.KFS.R:27:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/coef.KFS.R#L27) *style:* **lines should not be more than 80 characters.**
```r
        } else stop("Input does not contain filtered estimates for states, rerun KFS with state filtering.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/coef.KFS.R:30:41:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/coef.KFS.R#L30) *style:* **Trailing whitespace is superfluous.**
```r
    if (!is.null(start) && start == end)
                                        ^
```
[R/coef.KFS.R:33:2:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/coef.KFS.R#L33) *style:* **Trailing whitespace is superfluous.**
```r
}
 ^
```
[R/deviance.KFS.R:13:16:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/deviance.KFS.R#L13) *style:* **Trailing whitespace is superfluous.**
```r
deviance.KFS <-
               ^
```
[R/deviance.KFS.R:16:45:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/deviance.KFS.R#L16) *style:* **Put spaces around all infix operators.**
```r
    sum(residuals(object, type = "deviance")^2, na.rm = TRUE)
                                           ~^~
```
[R/deviance.KFS.R:17:2:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/deviance.KFS.R#L17) *style:* **Trailing whitespace is superfluous.**
```r
}
 ^
```
[R/extract.SSModel.R:46:17:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L46) *style:* **Trailing whitespace is superfluous.**
```r
`[<-.SSModel` <-
                ^
```
[R/extract.SSModel.R:48:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L48) *style:* **Trailing whitespace is superfluous.**
```r
    element <- match.arg(arg = element,
                                       ^
```
[R/extract.SSModel.R:49:79:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L49) *style:* **Trailing whitespace is superfluous.**
```r
                         choices = c("y", "Z", "H", "T", "R", "Q", "a1", "P1",
                                                                              ^
```
[R/extract.SSModel.R:57:60:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L57) *style:* **Trailing whitespace is superfluous.**
```r
          if (min(states) < 1 | max(states) > attr(x, "m"))
                                                           ^
```
[R/extract.SSModel.R:58:75:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L58) *style:* **Trailing whitespace is superfluous.**
```r
            stop("Vector states should contain the indices or names of the
                                                                          ^
```
[R/extract.SSModel.R:61:44:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L61) *style:* **Trailing whitespace is superfluous.**
```r
          states <- match.arg(arg = states,
                                           ^
```
[R/extract.SSModel.R:62:77:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L62) *style:* **Trailing whitespace is superfluous.**
```r
                              choices = c("all", "arima", "custom", "cycle",
                                                                            ^
```
[R/extract.SSModel.R:63:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L63) *style:* **Trailing whitespace is superfluous.**
```r
                                          "seasonal", "trend", "regression"),
                                                                             ^
```
[R/extract.SSModel.R:77:56:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L77) *style:* **Trailing whitespace is superfluous.**
```r
          if (min(etas) < 1 | max(etas) > attr(x, "k"))
                                                       ^
```
[R/extract.SSModel.R:78:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L78) *style:* **Trailing whitespace is superfluous.**
```r
            stop("Vector etas should contain the indices or names of the etas
                                                                             ^
```
[R/extract.SSModel.R:81:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L81) *style:* **Trailing whitespace is superfluous.**
```r
          etas <- match.arg(arg = etas,
                                       ^
```
[R/extract.SSModel.R:82:75:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L82) *style:* **Trailing whitespace is superfluous.**
```r
                            choices = c("all", "arima", "custom", "cycle",
                                                                          ^
```
[R/extract.SSModel.R:83:76:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L83) *style:* **Trailing whitespace is superfluous.**
```r
                                        "seasonal", "trend", "regression"),
                                                                           ^
```
[R/extract.SSModel.R:94:53:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L94) *style:* **Trailing whitespace is superfluous.**
```r
      } else if (!all(series %in% (1:attr(x, "p"))))
                                                    ^
```
[R/extract.SSModel.R:95:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L95) *style:* **Trailing whitespace is superfluous.**
```r
        stop("Argument series must have values between 1 to p, where p is the
                                                                             ^
```
[R/extract.SSModel.R:99:65:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L99) *style:* **Trailing whitespace is superfluous.**
```r
      switch(element, y = , u = x[[element]][, series] <- value,
                                                                ^
```
[R/extract.SSModel.R:100:58:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L100) *style:* **Trailing whitespace is superfluous.**
```r
             Z = x[[element]][series, states, ] <- value,
                                                         ^
```
[R/extract.SSModel.R:101:58:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L101) *style:* **Trailing whitespace is superfluous.**
```r
             H = x[[element]][series, series, ] <- value,
                                                         ^
```
[R/extract.SSModel.R:102:58:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L102) *style:* **Trailing whitespace is superfluous.**
```r
             T = x[[element]][states, states, ] <- value,
                                                         ^
```
[R/extract.SSModel.R:103:56:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L103) *style:* **Trailing whitespace is superfluous.**
```r
             R = x[[element]][states, etas, ] <- value,
                                                       ^
```
[R/extract.SSModel.R:104:54:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L104) *style:* **Trailing whitespace is superfluous.**
```r
             Q = x[[element]][etas, etas, ] <- value,
                                                     ^
```
[R/extract.SSModel.R:105:52:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L105) *style:* **Trailing whitespace is superfluous.**
```r
             a1 = x[[element]][states, 1] <- value,
                                                   ^
```
[R/extract.SSModel.R:106:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L106) *style:* **Trailing whitespace is superfluous.**
```r
             P1 = x[[element]][states, states] <- value,
                                                        ^
```
[R/extract.SSModel.R:109:70:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L109) *style:* **Trailing whitespace is superfluous.**
```r
      switch(element, y = , u = x[[element]][times, series] <- value,
                                                                     ^
```
[R/extract.SSModel.R:110:63:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L110) *style:* **Trailing whitespace is superfluous.**
```r
             Z = x[[element]][series, states, times] <- value,
                                                              ^
```
[R/extract.SSModel.R:111:63:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L111) *style:* **Trailing whitespace is superfluous.**
```r
             H = x[[element]][series, series, times] <- value,
                                                              ^
```
[R/extract.SSModel.R:112:63:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L112) *style:* **Trailing whitespace is superfluous.**
```r
             T = x[[element]][states, states, times] <- value,
                                                              ^
```
[R/extract.SSModel.R:113:61:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L113) *style:* **Trailing whitespace is superfluous.**
```r
             R = x[[element]][states, etas, times] <- value,
                                                            ^
```
[R/extract.SSModel.R:114:59:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L114) *style:* **Trailing whitespace is superfluous.**
```r
             Q = x[[element]][etas, etas, times] <- value,
                                                          ^
```
[R/extract.SSModel.R:115:52:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L115) *style:* **Trailing whitespace is superfluous.**
```r
             a1 = x[[element]][states, 1] <- value,
                                                   ^
```
[R/extract.SSModel.R:116:57:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L116) *style:* **Trailing whitespace is superfluous.**
```r
             P1 = x[[element]][states, states] <- value,
                                                        ^
```
[R/extract.SSModel.R:123:15:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L123) *style:* **Trailing whitespace is superfluous.**
```r
`[.SSModel` <-
              ^
```
[R/extract.SSModel.R:125:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L125) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/extract.SSModel.R:126:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L126) *style:* **Trailing whitespace is superfluous.**
```r
    element <- match.arg(arg = element,
                                       ^
```
[R/extract.SSModel.R:127:79:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L127) *style:* **Trailing whitespace is superfluous.**
```r
                         choices = c("y", "Z", "H", "T", "R", "Q", "a1", "P1",
                                                                              ^
```
[R/extract.SSModel.R:129:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L129) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/extract.SSModel.R:136:60:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L136) *style:* **Trailing whitespace is superfluous.**
```r
          if (min(states) < 1 | max(states) > attr(x, "m"))
                                                           ^
```
[R/extract.SSModel.R:137:75:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L137) *style:* **Trailing whitespace is superfluous.**
```r
            stop("Vector states should contain the indices or types of the
                                                                          ^
```
[R/extract.SSModel.R:140:44:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L140) *style:* **Trailing whitespace is superfluous.**
```r
          states <- match.arg(arg = states,
                                           ^
```
[R/extract.SSModel.R:141:77:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L141) *style:* **Trailing whitespace is superfluous.**
```r
                              choices = c("all", "arima", "custom", "cycle",
                                                                            ^
```
[R/extract.SSModel.R:142:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L142) *style:* **Trailing whitespace is superfluous.**
```r
                                          "seasonal", "trend", "regression"),
                                                                             ^
```
[R/extract.SSModel.R:156:56:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L156) *style:* **Trailing whitespace is superfluous.**
```r
          if (min(etas) < 1 | max(etas) > attr(x, "k"))
                                                       ^
```
[R/extract.SSModel.R:157:78:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L157) *style:* **Trailing whitespace is superfluous.**
```r
            stop("Vector etas should contain the indices or types of the etas
                                                                             ^
```
[R/extract.SSModel.R:160:40:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L160) *style:* **Trailing whitespace is superfluous.**
```r
          etas <- match.arg(arg = etas,
                                       ^
```
[R/extract.SSModel.R:161:66:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L161) *style:* **Trailing whitespace is superfluous.**
```r
                            choices = c("all", "arima", "custom",
                                                                 ^
```
[R/extract.SSModel.R:162:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L162) *style:* **lines should not be more than 80 characters.**
```r
                                                    "cycle", "seasonal", "trend", "regression"), several.ok = TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:172:53:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L172) *style:* **Trailing whitespace is superfluous.**
```r
      } else if (!all(series %in% (1:attr(x, "p"))))
                                                    ^
```
[R/extract.SSModel.R:173:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L173) *style:* **lines should not be more than 80 characters.**
```r
        stop("Argument series must have values between 1 to p, where p is the number of time series in model. ")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:176:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L176) *style:* **lines should not be more than 80 characters.**
```r
      switch(element, y = , u = x[[element]][, series, drop = drop], Z = x[[element]][series,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:176:94:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L176) *style:* **Trailing whitespace is superfluous.**
```r
      switch(element, y = , u = x[[element]][, series, drop = drop], Z = x[[element]][series,
                                                                                ^
```
[R/extract.SSModel.R:177:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L177) *style:* **lines should not be more than 80 characters.**
```r
                                                                                states, , drop = drop], H = x[[element]][series, series, , drop = drop],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:177:160:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L177) *style:* **Trailing whitespace is superfluous.**
```r
                                                                                states, , drop = drop], H = x[[element]][series, series, , drop = drop],
                                                                                ^
```
[R/extract.SSModel.R:178:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L178) *style:* **lines should not be more than 80 characters.**
```r
             T = x[[element]][states, states, , drop = drop], R = x[[element]][states,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:178:87:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L178) *style:* **Trailing whitespace is superfluous.**
```r
             T = x[[element]][states, states, , drop = drop], R = x[[element]][states,
                                                                                ^
```
[R/extract.SSModel.R:179:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L179) *style:* **lines should not be more than 80 characters.**
```r
                                                                                etas, , drop = drop], Q = x[[element]][etas, etas, , drop = drop],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:179:147:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L179) *style:* **Trailing whitespace is superfluous.**
```r
                                                                                etas, , drop = drop], Q = x[[element]][etas, etas, , drop = drop],
                                                                                ^
```
[R/extract.SSModel.R:180:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L180) *style:* **lines should not be more than 80 characters.**
```r
             a1 = x[[element]][states, 1, drop = drop], P1 = x[[element]][states,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:180:82:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L180) *style:* **Trailing whitespace is superfluous.**
```r
             a1 = x[[element]][states, 1, drop = drop], P1 = x[[element]][states,
                                                                                ^
```
[R/extract.SSModel.R:181:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L181) *style:* **lines should not be more than 80 characters.**
```r
                                                                           states, drop = drop], P1inf = x[[element]][states, states, drop = drop],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:181:148:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L181) *style:* **Trailing whitespace is superfluous.**
```r
                                                                           states, drop = drop], P1inf = x[[element]][states, states, drop = drop],
                                                                                ^
```
[R/extract.SSModel.R:184:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L184) *style:* **lines should not be more than 80 characters.**
```r
      switch(element, y = , u = x[[element]][times, series, drop = drop], Z = x[[element]][series,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:184:99:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L184) *style:* **Trailing whitespace is superfluous.**
```r
      switch(element, y = , u = x[[element]][times, series, drop = drop], Z = x[[element]][series,
                                                                                ^
```
[R/extract.SSModel.R:185:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L185) *style:* **lines should not be more than 80 characters.**
```r
                                                                                states, times, drop = drop], H = x[[element]][series, series, times,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:185:161:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L185) *style:* **Trailing whitespace is superfluous.**
```r
                                                                                states, times, drop = drop], H = x[[element]][series, series, times,
                                                                                ^
```
[R/extract.SSModel.R:186:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L186) *style:* **lines should not be more than 80 characters.**
```r
                                                                                drop = drop], T = x[[element]][states, states, times, drop = drop],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:186:207:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L186) *style:* **Trailing whitespace is superfluous.**
```r
                                                                                drop = drop], T = x[[element]][states, states, times, drop = drop],
                                                                                ^
```
[R/extract.SSModel.R:187:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L187) *style:* **lines should not be more than 80 characters.**
```r
             R = x[[element]][states, etas, times, drop = drop], Q = x[[element]][etas,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:187:88:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L187) *style:* **Trailing whitespace is superfluous.**
```r
             R = x[[element]][states, etas, times, drop = drop], Q = x[[element]][etas,
                                                                                ^
```
[R/extract.SSModel.R:188:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L188) *style:* **lines should not be more than 80 characters.**
```r
                                                                                etas, times, drop = drop], a1 = x[[element]][states, 1, drop = drop],
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:188:153:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L188) *style:* **Trailing whitespace is superfluous.**
```r
                                                                                etas, times, drop = drop], a1 = x[[element]][states, 1, drop = drop],
                                                                                ^
```
[R/extract.SSModel.R:189:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L189) *style:* **lines should not be more than 80 characters.**
```r
             P1 = x[[element]][states, states, drop = drop], P1inf = x[[element]][states,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:189:90:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L189) *style:* **Trailing whitespace is superfluous.**
```r
             P1 = x[[element]][states, states, drop = drop], P1inf = x[[element]][states,
                                                                                ^
```
[R/extract.SSModel.R:190:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L190) *style:* **lines should not be more than 80 characters.**
```r
                                                                                states, drop = drop], )
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/extract.SSModel.R:192:4:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/extract.SSModel.R#L192) *style:* **Trailing whitespace is superfluous.**
```r
  }
   ^
```
[R/fitSSM.R:76:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L76) *style:* **Variable and function names should be all lowercase.**
```r
fitSSM <-
^~~~~~
```
[R/fitSSM.R:76:10:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L76) *style:* **Trailing whitespace is superfluous.**
```r
fitSSM <-
         ^
```
[R/fitSSM.R:78:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L78) *style:* **Trailing whitespace is superfluous.**
```r

^~~
```
[R/fitSSM.R:80:1:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L80) *style:* **lines should not be more than 80 characters.**
```r
      # use default updating function, only for time invariant covariance matrices
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/fitSSM.R:81:7:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L81) *style:* **Variable and function names should be all lowercase.**
```r
      estH <- !identical(model$H, "Omitted") && any(is.na(model$H))
      ^~~~
```
[R/fitSSM.R:82:7:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L82) *style:* **Variable and function names should be all lowercase.**
```r
      estQ <- any(is.na(model$Q))
      ^~~~
```
[R/fitSSM.R:83:27:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L83) *style:* **Put spaces around all infix operators.**
```r
      if ((dim(model$H)[3]>1 && estH || (dim(model$Q)[3] > 1) && estQ))
                         ~^~
```
[R/fitSSM.R:83:33:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L83) *style:* **Variable and function names should be all lowercase.**
```r
      if ((dim(model$H)[3]>1 && estH || (dim(model$Q)[3] > 1) && estQ))
                                ^~~~
```
[R/fitSSM.R:83:66:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L83) *style:* **Variable and function names should be all lowercase.**
```r
      if ((dim(model$H)[3]>1 && estH || (dim(model$Q)[3] > 1) && estQ))
                                                                 ^~~~
```
[R/fitSSM.R:83:72:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L83) *style:* **Trailing whitespace is superfluous.**
```r
      if ((dim(model$H)[3]>1 && estH || (dim(model$Q)[3] > 1) && estQ))
                                                                       ^
```
[R/fitSSM.R:84:74:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L84) *style:* **Trailing whitespace is superfluous.**
```r
        stop("No model updating function supplied, but cannot use default
                                                                         ^
```
[R/fitSSM.R:87:12:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
        if(estQ){
           ^~~~
```
[R/fitSSM.R:89:11:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L89) *style:* **Variable and function names should be all lowercase.**
```r
          naQd <- which(is.na(diag(Q)))
          ^~~~
```
[R/fitSSM.R:90:11:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L90) *style:* **Variable and function names should be all lowercase.**
```r
          naQnd <- which(upper.tri(Q[naQd, naQd]) & is.na(Q[naQd, naQd]))
          ^~~~~
```
[R/fitSSM.R:90:38:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L90) *style:* **Variable and function names should be all lowercase.**
```r
          naQnd <- which(upper.tri(Q[naQd, naQd]) & is.na(Q[naQd, naQd]))
                                     ^~~~
```
[R/fitSSM.R:90:44:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L90) *style:* **Variable and function names should be all lowercase.**
```r
          naQnd <- which(upper.tri(Q[naQd, naQd]) & is.na(Q[naQd, naQd]))
                                           ^~~~
```
[R/fitSSM.R:90:61:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L90) *style:* **Variable and function names should be all lowercase.**
```r
          naQnd <- which(upper.tri(Q[naQd, naQd]) & is.na(Q[naQd, naQd]))
                                                            ^~~~
```
[R/fitSSM.R:90:67:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L90) *style:* **Variable and function names should be all lowercase.**
```r
          naQnd <- which(upper.tri(Q[naQd, naQd]) & is.na(Q[naQd, naQd]))
                                                                  ^~~~
```
[R/fitSSM.R:91:13:](https://github.com/helske/KFAS/blob/7051296d8b58fc8f8e68d3c439ccb2bffce7985c/R/fitSSM.R#L91) *style:* **Variable and function names should be all lowercase.**
```r
          Q[naQd, naQd][lower.tri(Q[naQd, naQd])] <- 0
            ^~~~
```


pouzivas spatne konce radku (viz git diff b983f19b7) + tady je divny whitespace :)


Yes it would be nice to "annotate" them somehow too.


メソッド切り分け出来る気がします。


Big change, do that in a separated patch.


Fair point. How about moving this test into neo4j-integrationtests?


why is there an underscore in IPAddress ?


These .pyt's will be much nicer to configuration manage in Github - are there any 10.X vs. Pro issues with these (since I don't see a _10.3 version like the other toolboxes)?


@lassewesth Better?


QObject:: doesn´t belong in text string


Very minor nitpick, typo `logick`/`logic`.


I think there was a reason for that. I don't quite remember now.


@kslong @smangham @Higginbottom

Before, the Log_parallel messages (messages logging various MPI processes) wrote to screen for every thread. This was a bit silly because if you run with 100 threads you get 100 threads telling you they are doing OK and clogging up your output.

Now, only thread 0 writes to screen, but the others save to their individual diag files. This means you can collect all the messages together by doing

```
grep Para: diag_root/*.diag
```

Originally, I wanted all of these to write to a log_parallel file. this didn't work becuase you get muddled output when threads attempt to flush memory at the same time (at least I assume that's why). General question hereabout how to generally write out information from multiple threads to a single source of data (a database structure can be used- that seems unnecessary here but could be useful for e.g. reverb mapping info). One can also just use post processing scripts to collect multiple files- that's the easiest option.


This should show up if you add `'sphinx.ext.mathjax'` to this list https://github.com/snoplusuk/echidna/blob/master/docs/conf.py#L31


Not that I know of. And Slim Auth won't really come into play until your users log in after they've registered. Just make sure to use [`password_hash`](http://php.net/manual/en/function.password-hash.php) to create their password and assign a role to new users and you should be fine :-)


It's been a while and I don't remember why I had `reboot` here.


Recursing back through the command here is very confusing.


This is of course better, although not supported in all kickstarts.


This is what I noted while manually testing this.

* Ctrl-C has to be hit twice to quit the CLI (probably Aesh issue)

* Line editing (probably Aesh issue):
make a narrow terminal width, so that you can easily enter lines that are longer than the terminal width. Enter such a line and try hitting backspace until you end up with a line shorter than the terminal width, keep backspacing, you'll notice a character left at the end of the line.
There are other issues related to lines spanning more than one line, but that's difficult to describe in English :) I have a screenshot, just in case. But if you play with it, I guess you'll see issues.

* Authentication
- Ctrl-C doesn't work when entering authentication data (e.g. username);
- every button has to be hit twice to have the expected effect (actually it doesn't matter which button is hit first, the first one is simply ignored);
- try entering a username. Only every second character shows up;
- try backspacing a username, same thing, every second hit deletes a character;
- same for hitting Enter at the end of the line.


тут не вызывался оператор для GeometryForm... В нём, в свою очередь, ещё какие-то ошибки... посмотришь?


Unfortunately not.


This seems fine, although it'd be better to remove it entirely, rather than commenting it out, even for consistency in the RHEL7 kickstart below.


Code is well organized:
- SoftMax classifier is added to the Classify module
- NeuralNetworks module implements the NN model based on the Stanford NN tutorial, and NNClassifier is added to the Classify module
- DeepLearning module implements Sparse Autoencoder and StackedSA
- Test code using both small and big/wide datasets is provided for all functions, and Maryam has demonstrated that her implementations scale nicely

I think this code commit can be merged


I believe the lists:flatten is not required here. Just change the spec to return an iolist.


@franticpsyx Claudio added this a minor version ago I believe @claudiosmweb? And he added it to the wrong script. select2 should not be localised, the scripts that implement select2 should.


Note that it wasn't necessary before, as it was done on the line before the `if` block, thus affecting both `imageAsPNG` and `imageAsTikZ`. Thus `imageAsTikZ` is called exactly the same way as it was before my changes. I tried to keep the changes as minimal as possible, but adapting the TikZ output by calling `flipImageIfAxesReversed()` probably makes sense.


Then please add a full-fledged Boost.Config(.Predef?) check for
"# pragma pop_macro/push_macro" instead of the simple checks for gcc and clang. A quick search indicates it might be available with at least gcc-4.4 but older gccs are still partieally supported. That is one reason for this patch to add more brokenness. At least until I have some information that this will work elsewhere (think of gcc-4.2.1 on OSX or the various clang versions emulating it).

And me thinks that using Q_SLOTS and Q_SIGNALS is the right way, too. And you surely know that "emit" expands to nothing ;-) And at least Qt-4.8.6 and everything from Qt-5.x upward should have "clean" headers not using those dreaded "keywords".

For Qt-3 and earlier 4.x versions you are out of luck, but those should be ported to Qt-5 anyway.
We also have a macro based solution to avoid the "signals" macro destroying Boost.Signals, but this has caused much headache here, too. I'm not maintaining foreach and using c++11 range-based for instead, so I'm quite a bit biased.


Take a look at mk kernel thread - I'm sure that mkaluza wrote command to get *.cpio.gz ramdisk. If you not sure, try to ask someone in his thread - I think that someone can help.


This was a long hesitation. The thing is i considered easier to manipulate int than strings in c (+ performance reason). Nevertheless, you are true that it is debatable.


The behavior changes here seem fine, although when we mess with these settings we tend to find corner cases and bug reports, from my experience :) Looks good.


True. I just changed the existing test, which used a global rng.


Looks fine.


@grifferz this method seems to be unused since e97d94d6 was merged.  That commit changed it to no longer call announce_win and instead implements the disp_nick logic inline.


Mach Dir das Leben doch einfacher und ersetze owibranding.py durch eine verkürzte Version, die zur build time mit den entsprechenden Werten als Konstanten gefüllt wird.
owibranding.py in der Form ist eher für die Lutscher von OpenPLi und VTI.


I still don't understand why you insist on reinventing the printable_attr_value, returning strings(lists) or binaries should not matter for this method.


Cuidado, pois essa construção `foo && bar || baz` não é um substituto a `if/then/else`.

Digamos que o primeiro teste de `testa_data` seja OK, então o segundo comando será executado. Digamos que deu erro nesse segundo comando, talvez porque o conteúdo da opção `-f` esteja errado ou `$2` tem um formato diferente. Neste caso, o terceiro comando **também** será executado! Ou seja, nesta situação hipotética, todos os três comandos do `foo && bar || baz` serão executados.

Usar `&&` e `||` é seguro se for um **ou** outro, mas os dois juntos tem esse tipo de pegadinha, aí é melhor usar o bom e velho `if/then/else` mesmo.


now you can directly call `stack.resources`, it returns a `Resources` collection object


Epic. At first it wasn't clear to me about what this is, but it's obvious - and from the architect perspective, that's an improvement (generally I just use a plane for representing stairs, until the stairs finally gets detail-modeled ... though most of the time it's hard to tell which stairs one should model and having steps on the plane/quad definitely is a nice generic symbol for a stair).


Either mutate the method receiver (please don't) or return back the server setup (yay).


Don't exit from the method call.  It prevents us from unit testing this method.


Note: `expr` is part of the Gnu coreutils and it seems to support \s in the regexp part of a `expr STRING : REGEXP` construct to mean any space character. I hope this works on Mac OS X too.


I really only had a very short look. If you decide on introducing `flipImageIfAxesReversed()` there as well, then you can rather move it out of `imageAsPNG` so that it is not duplicated.


You need to append the new columns/fields to the end of the last. ALTER TABLE doesn't support inserting them somewhere in between.


Oops!


Bad indentation (same three lines down).


No curly braces on single line bodies.


Ah, e só pra deixar claro, a função agora funciona normalmente no BSD:

```console
$ zzdatafmt -f SEMANA hoje
Terça-feira
$
```


No need to remember both as you will always only fill one of them.


No curly braces.


this is temporary, until firefox 37 rolls out. see http://stackoverflow.com/questions/27132992/mediaelementaudiosourcenode-prevents-video-playback-in-firefox/27163238#27163238


@yehudasa In the updated version of *set_copy_attrs* there are three cases in the switch statement.
In the first case depending on intra_region one attribute of src_attrs can be overridden and all the rest left untouched.
In the second case intra_region parameter is totally ignored.
In the third case intra_region is checked to find out if ACL from *src_attrs* has to be overridden by ACL from *attrs*.
I don't understand how 'return on intra_region'  will help to get the conditions out of the switch statement.


@haonanlu this is the ro_list you need to add after the start request.


Whoa! Darn. I only compile-tested. Please note that the convention in Git is to use `!...` instead of `... == 0` in such cases, I will amend my commits.


@haonanlu this is the ro_list carried in the commit message.


Damn, this shouldn't happen…
I'm going to have to take a look at the pre-commit hook script apparently.


Published to http://ui.edev.desire2learn.com/45_efe5e15e19eca4c328080f1602c6a00c1e44a338


> @ardumont: Cool stuff!

Thanks!

> Can you please split this into two separate pull requests:
> 1. For tests (they should work without cask as we have another workaround for missing ert/cl-lib).

# TL; DR

The `er/cl-lib` is not the reason of this PR.
The reason was to fix the el-mock or more generally all `dev` dependencies needed for tests purposes.

# More details

I initially proposed this [to make sure I did not break the initial code I extracted into a function](https://github.com/ardumont/haskell-mode/commit/966dbe7de47ad3ee57f5f123b9f65ec1c39834fb#diff-26b50d5bad8251212c2d68099413ddfcR1037)  before adding the [new functionality I was aiming at (`haskell-process-wrapper-function`)](https://github.com/haskell/haskell-mode/pull/370).

The production code proposed was merged but without the tests.

This PR kept existing with the original code (the one not fully merged).

I rebased last week to keep it clean and more readable.

And now, we no longer see [the hack it was supposed to fix](https://github.com/ardumont/haskell-mode/commit/966dbe7de47ad3ee57f5f123b9f65ec1c39834fb#diff-02f6e744abd167f18eab5b363acb6f5bR9).

> Can you please split this into two separate pull requests:

Yes, I can.

Be aware then that:

- I'll have to reintroduce the initial hack to have the el-mock dependencies satisfied ([it's cask which deals with it](https://github.com/ardumont/haskell-mode/commit/ec29dfbcde3682d429cdc3ae737e889b7ee42de4#diff-64ba2c4df6c907da3a5f3860e9f6913eR11)).

- which has impact on the overall time of running tests.

(But maybe, you'll have a better implementation.)

Cheers,


@mikejolley

>select2 should not be localised, the scripts that implement select2 should.

Yup, of course. I was simply using wc_select_params in my own scripts to get these translated strings at the moment... didn't really question the fact that they were added to the wrong script, but you are right.

So I guess I need to update my extensions :)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/909
success: http://build.servo.org/builders/linux2/builds/907
success: http://build.servo.org/builders/android/builds/945
success: http://build.servo.org/builders/mac1/builds/1124
success: http://build.servo.org/builders/mac2/builds/1124
success: http://build.servo.org/builders/gonk/builds/95


It hasn't man - just the CEO recruitment pages xxx

Are you still with F&C?

On Tue, Mar 3, 2015 at 2:17 PM, Harry Fox <notifications@github.com> wrote:

> Sorry to see the farm has gone
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WalworthGardenFarm/WalworthGardenFarm.github.io/commit/fc2d701eaf57afda60679bafb440001588abc3a8#commitcomment-10007276>
> .
>


Can all of this go on one line? That and put it on the `StrArrayConverter` object and add it into the header.


Hello Ferenc!

It seems that these chanes are already performed in https://github.com/pear/pear-core/compare/incompatible-context


Fixes #400


Sorry xD


Yeah, sorry that comment was so vague. @Phrohdoh explained it. And while I suppose the minelayer will never have a second ammo pool, it mught be nice to check for consistency with other traits.
On the other hand the minelayer has its own trait which noone else uses...
But I still vote for consistency with the rest.


@barbalberto By bisecting I found out that this commit is the reason why the ``dev::PolyDriverTest`` is failing with a segfault, see https://travis-ci.org/robotology/yarp/builds/52288310


Yes, this would be better. Actually there are also 039 and 684 that do not seem to be standard. I would good to change these too.


Since it [gets initialized to `null`](https://github.com/skroutz/analytics.js/blob/master/src/actions_manager.coffee#L14), you should not worry about `undefined`.


Неплохо бы разделять методы пустыми строчками


这是啥


Se crea carpeta de css


@itamarnet agora com essa mudança os testes que estavam ok aqui, quebraram.

Aqui a zzstr2hexa retorna os dados em uma linha, normal. Aí no teu ela retorna sem a quebra de linha final?

```console
$ zzstr2hexa binario
62 69 6e 61 72 69 6f
$
```


Good catch, it can be done by adding minlength to the input:
`
tep_draw_input_field('firstname', NULL, 'required aria-required="true" id="inputFirstName" placeholder="' . ENTRY_FIRST_NAME . '" minlength="'. ENTRY_FIRST_NAME_MIN_LENGTH .'"');
`


Yeah, I can put it all on one line. I'll change the name to `AllocStrArray`.


fast-forwarding master to auto = 417a932e


See www.greenbuttondata.org/developer for technical data, apis, and test
tools.

HTH,
Marty

On Tue, Mar 3, 2015 at 4:12 AM, charaninduri <notifications@github.com>
wrote:

> Is there any documentation for download my data for greenbutton.
> Are there are any REST api to download green button data.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/energyos/OpenESPI-GreenButton-API-Documentation/commit/f152545b9763b686c77d1e2cfb1b814ee9059c0f#commitcomment-10001793>
> .
>


I did not write these but @abhiataero did ... I think. He might be able to check on these


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1123 of 1123 tests passed)

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


Je mettrai plutôt `song/search` comme URL ici.


This is a question for @abhiataero. Abhijit?


I did, it is pull request #4371.


1540218-SlinkyGroundHX (mmatts) - x86_64-MacOS-10.9-clang: OK (1623 of 1623 tests passed)

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


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1123 of 1123 tests passed)

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


@khelek если так можно, то поправлю)


strings.xml?


Thanks for finding this bug! I had no idea what was going on. Would it be possible to do a release so that I can require this feature in my composer.json?


No worries Claudi! It's :ok:


Isso mesmo!
Os testes estavam falhando por conta da ausência da quebra de linha.


please store Domain.first to some var, so we don't repeat the call (I know AR has cache but still, it's not DRY)


Not that I know of. Though I don't think these were made for 10.3. @nataliecampos would be able to try them out.


please store Domain.order(:id).last.id to some var, so we don't repeat the call (I know AR has cache but still, it's not DRY)


Test commit


Why are we explicitly setting the color for strong elements?


Don't need menus


All of our scripts could probably use an update for 2015.


#Added find title tab


I know I just saw that in passing. If there are too many lines to change, let's not do that in that PR.


this not work if the ldap username is in form of xx.yyy. I create a ldap user named odoo.as but it does not work. I remove .encode('utf-8'), it works again


They didn't before, so... no?


Yeah, the comment should say "rename" :D


Moved!


Why do you expect an array? The response of this request with given one service_request_id should serve one or zero objects, for my opinion.


I could do that, but then the line gets over the limit (80 chars by default) ... but we can simply change the limit to a larger number.


Please add destructor as well as other implementations discussed during the class.


This line should be next to the one asserting on the presence of `value_of_pension_pots ` ?


Note: you would normally only update if you made an actual change/update to the script - that would extend your copyright.


No you are right. I'll fix it as soon as I'll get in (in like half an hour).


If you have time, you can add [these 3 variables](https://github.com/ThemeFuse/Unyson-Shortcodes-Extension/blob/348836de832fe9d3a78fae1cdca3c2bf9cd4412a/includes/class-fw-shortcode.php#L145-L147) comments in all shortcode views


Wouldn't be better to implementing this having two fields, one for TemporaryFolder rule and one for SystemProperty  rules rule and the propagate the before and after to each of them?


Ok (mais bien song sans 's', pour éviter une collision possible avec un slug d'artiste)
[j'ai une branche various_fix qui traite cela pour `songs/random`]


See PR https://github.com/PowerShellOrg/cWebAdministration/pull/10


It says "10.1SP1" in the header below - so maybe verify and update that also if necessary


Thanks!


occhi alle parentesi


why did you choose 5000 chars for comment? It would be much faster to input only 100-200 characters (even that much is not really important)


Not sure about commenting this one... We'll have to check that with real SCORM cases.


## io.js a più di 10000 stelle su GitHub
Metterei "supera le 10000 stelle", mi sembra rensa meglio break


Done in 8e5d42a390c0d5cf75658dd16c6e59ccf0a42d70


neexistuju zle konce riadkov jednoducho su len konce riadkov :D Navise toto bude ten najmensi problem s ktorym sa v mojom kode stretnes :D


I think this case is unnecessary, because this functionality would be checked a lot in following tests. If they fail with notifications = user is not subscribed. This test case only checks UI change (subscribe/unsubscribe button display)


@gordonwoodhull I typically treat it as a separation of concerns; jshint does the error checking and jscs is for style persistence.  What's the jshint option anyway?  If it's [camelcase](http://jshint.com/docs/options/#camelcase), that option is now deprecated.


// Program 3
// Description: Product program
// Name:  Samuel Roitblat

#include <iostream>
#include <fstream>
#include <string>
#include <iomanip>
using namespace std;

struct Division
{
string divisionName;
int q1Sales;
int q2Sales;
int q3Sales;
int q4Sales;
};

struct Corporation
{
int totalQ1Sales;
int totalQ2Sales;
int totalQ3Sales;
int totalQ4Sales;
int numberOfDivisions;
};

void getDivision(Division*, ifstream &);
void printDivision(Division*);
void addDivision(Division*, Corporation*);
void printCorpSummary(Corporation*, Division*);

int main()
{
        ifstream infile;
        infile.open("prog3.txt");

        if (!infile)
        {
                cout << "ERROR - no such file exists!\n";
        }

        Corporation* ptrTotalSales = new Corporation;
        Division* ptrDivisionInfo = new Division;

cout << "----------------------------------------------\n";


call `stack.resources`. it returns a `Resources` collection object.


I think algorithm W is a viable way to move forward. It is well documented, proven to work and I think we can adapt it to our case. If you want to get on board the algorithm W train I think it's best if you pull dev-tc into liam-tc and continue there, as me and Johan are right in the middle of trying to solve some problems in the dev-tc branch.
We are working from a tutorial paper by Martin Grabmüller, as stated in the comments in the code, which can serve as an introduction. You can find it here: https://github.com/wh5a/Algorithm-W-Step-By-Step
I do not use facebook, maybe keep any discussions/questions in the group?


Попробую.


To me it looks like it's in the PR -- the commit is listed in
"conversation", and the comment appears in the "files changed".

The "shippable" check fails. Not sure why - when I click the link I'm
brought to a Shippable UI that looks like marketing and doesn't really tell
me anything.

On Tue, Mar 3, 2015 at 4:36 AM, David Oppenheimer <notifications@github.com>
wrote:

> LGTM, thanks.
>
> But I think you didn't create a PR -- the email I got just asked me to
> comment on the commit. I think you need to add this to the PR where you
> added the test.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jdef/kubernetes/commit/c0c1ef727f5373bffb88d07b72b962368cc3b535#commitcomment-10002160>
> .
>

--
James DeFelice
585.241.9488 (voice)
650.649.6071 (fax)


yes, we should add a TODO to update fog


user is not subbed to branch


I can take care of the Spanish translation if needed.


It should be `typeof obj2`


@benbc Beautiful!


I removed note.content because it was preventing excepts from showing, and instead showed the full content--remove note.content if you are having issues


shall we rename the method to something like `all_servers` since it only filters `OS::Nova::Server`?


y u no alpha order your dependencies?


@benbc Liking it!


Why are we setting the color explicitly?


What? you trust clustering logic?
Well, the scenario I gave you is a real cluster legal state change. It always (or at least should always) happens in `ClusterTopologyChangesIT#slaveShouldServeTxsAfterMasterLostQuorumWentToPendingAndThenQuorumWasRestored`.

In previous implementation ( before a pr about epoch ), the slaves will go back to `PENDING` whenever there is a new election, but the current implementation will stay in the `SLAVE` if the master id is not changed. So if you want to keep this `if`, we might pull updates when master is in `TO_MASTER`. Thus I would suggest to make sure question 1 is answered properly or if you already have a test for that?

Then for the question 2, it is just a wonder how this `lastUpdateTime` is used. https://github.com/neo4j/neo4j/blob/master/enterprise/ha/src/main/java/org/neo4j/kernel/ha/UpdatePuller.java#L323. I would guess the `InvalidEpochException` will be finally handled in
```
        catch ( ComException e )
        {
            cappedLogger.event( Pair.of( "Pull updates by " + this + " failed due to network error.", e ) );
        }
```
Then, should we just leave update puller retrying until the cluster detects the error and recovers itself?


If i get it right, this also fixes a bug of double "maybe first querying item", because it also happens in get_last_uid(). If that is right, should we reflect it in the commit message? Probably a bit over cautious, but, well, i noticed it.


Don't we generally want emphasized text to appear differently somehow? Usually `font-style: italic` or something similar


Erm ... I didn't even plan a german translation, I guess it
isn't necessary in general :)

Mit freundlichen Grüßen | Best regards
Daniel Gimmer

/*DG Media* - Beratung | Fotografie | Design | Produktion
Hauptstraße 235 | 73111 Lauterstein
Fon +49 160 8346199 | info@dg-media.com
www.dg-media.com | www.facebook.com/dgmedia2012

/
Am 03.03.2015 um 16:09 schrieb Guardsoul:
>
> I can take care of the Spanish translation if needed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Realm667/WolfenDoom/commit/46330ca06d4d3462724980f1ec307bca830f18a4#commitcomment-10008476>.
>


Just Topics.createCodeReview(codeReview)


Can we not use [`write_all`](http://doc.rust-lang.org/std/io/trait.Write.html#method.write_all) for this instead of a loop?


Maybe I'm reading it too strictly.  But to me "Currently the only way.." is inferring that sometime in the future there may be other ways.  All I'm asking is that it be phrased in the present, we have one way, its a command line, we don't really care to communicate that there might be other ways.  So my suggested change deal with the present.


GasHumidifier (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


GasHumidifier (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


So this is probably an odd place to put this but I'm getting a 404 error whenever I try to view this PR (#5794?) :confused:.


GasHumidifier (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


First of all, could you be so kind next time to put this extended explanation right into the commit message and not into the comment?
People have been migrating from SourceForge to Launchpad and then from Launchpad to GitHub. And I am pretty sure that GitHub is not the last thing in this chain. The problem with these migration is that you loose some of the information on the previous location. If there is a way to store something right inside of the git repo, so much the better.

Now about that ``x-terminal-emulator`` change. As far as I remember terminator was very dumb about the -e flag. While other terminals were treating all arguments after the -e flag as parameters (e.g. ``-e man conky``), terminator was splitting the next parameter by spaces (e.g. ``-e 'man conky'``). This has an obvious problem with parameters that contain spaces.
That bug in terminator was one of the reasons why ``x-terminal-emulator`` could not be used. Was it fixed?


GasHumidifier (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Curious why, if jRuby is required, this was removed?


I think it's ok, hard to test right now I vote merge it in


Такая сигнатура может ввести пользователя в панику. "Я просто хочу узнать, есть ли элемент в хеш-таблице, какой ещё ListElement?" :)


@benbc Create a variable or constant with a good name and a literal value, as documentation?


Agggh! I missed this additional patch "Make length calculation work right" on common/tree.c

That explains it!

Mar  3 16:09:45 xx dhcpd: Solicit message from fe80::16cc:20ff:fe8a:15c4 port 546, transaction ID 0x49453F00
Mar  3 16:09:45 xx dhcpd: Picking pool prefix 2001:db8:7d92:f00::/56
Mar  3 16:09:45 xx dhcpd: Advertise PD: address 2001:db8:7d92:f00::/56 to client with duid 00:03:00:01:14:cc:20:8a:15:c4 iaid = 1 valid for 60 seconds
Mar  3 16:09:45 xx dhcpd: Sending Advertise to fe80::16cc:20ff:fe8a:15c4 port 546
Mar  3 16:09:46 xx dhcpd: Request message from fe80::16cc:20ff:fe8a:15c4 port 546, transaction ID 0xE66F0A00
Mar  3 16:09:46 xx dhcpd: Reply PD: address 2001:db8:7d92:f00::/56 to client with duid 00:03:00:01:14:cc:20:8a:15:c4 iaid = 1 valid for 60 seconds
Mar  3 16:09:46 xx dhcpd: execute_statement argv[0] = /usr/local/sbin/prefix-delegation-routing.rb
Mar  3 16:09:46 xx dhcpd: execute_statement argv[1] = add
Mar  3 16:09:46 xx dhcpd: execute_statement argv[2] = 0:0:0:1:0:0:0:0:0:0:0:0:0:1a:0:19:0:0:0:0:0:0:0:0:38:20:1:d:b8:7d:92:f:0:0:0:0:0:0:0:0:0
Mar  3 16:09:46 xx dhcpd: execute_statement argv[3] = fe80::16cc:20ff:fe8a:15c4
Mar  3 16:09:46 xx dhcpd: execute_statement argv[4] = eth0
Mar  3 16:09:46 xx dhcpd: Sending Reply to fe80::16cc:20ff:fe8a:15c4 port 546

route -6
Kernel IPv6 routing table
Destination                    Next Hop                   Flag Met Ref Use If
2001:db8:7d92:f00::/56         fe80::16cc:20ff:fe8a:15c4  UG   1024 0     0 eth0


This constructor can not fail. If configPath or criterions were provided it would no longer be the case (duplicated criterion, null arguments...)


``vary_on_headers`` and ``vary_on_cookies`` are missing on generating cache_key. Please see https://github.com/metglobal/django-easy-cms/blob/master/easy_cms/templatetags/cms.py#L30.

Actually you can write a utility function to generate cache_key.


\o/


Pull requests are always worth it.

I don't think there's an _unsigned anywhere (I can't see it). The string/number type system is old and works for simple config cases. it does not work for encoding data, nor APIs, so we've developed a more robust one in zsock_send/recv and zproto.

The V2 codec does not provide any API for working with hash table values. It's redundant; we have that in zhash already, or can add it on top. The sheer size of the V1 API was a serious problem for something meant to be "simple and fast".

Now to your question. Sending a hash is a decent idea. I'd call it "headers" maybe, to match the HTTP sense. It's extensible and self-describing. This is what it was made for. However I'd write my own methods to work with the hash, and leave that out of zproto.

As for parsing complex data and getting complex results, IMO it's better to return 0/-1 and then provide extended error information via another class method, e.g. _error (). The trouble with variable error return codes is that it makes applications very fragile.  So we simply don't do that.


Me and Björn will keep working on this, we've started to get our heads around what needs to be done. I agree that the google group should suffice for type checking discussions etc.


89435026-initialize-SysTotCoolCap (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1122 of 1122 tests passed)

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


It is always necessary, how to find "this"/"self" otherwise ?


1: M101 signature is missing


@prudhomm Warning, this attach probably not work! You must create a NullSpace with a specific backend, in order to have a petsc vector, not ublas vector. Look here https://github.com/feelpp/feelpp/blob/develop/feel/feelalg/backend.hpp#l1011, you need to copy the NullSpace with UblasVector into a NullSpace with Petsc Vector


Just a color usually isn't enough to accessibly distinguish a link. We probably want an underline too, and I'd like us to use the fancy underlines.


If invalidated after the function return it can not be used by the client to find the error which is it's purpose :/


a better way to do this is:

        private ReturnStatement ParseReturnStatement()
        {
            Expression argument = null;

            ExpectKeyword("return");

            if (!_state.InFunctionBody && Equals(this._lookahead.Value, ";"))
            {
                ThrowErrorTolerant(Token.Empty, Messages.IllegalReturn);
            }


We probably want to mix with black (shade) the color, not darken it.


This is hilarious.  I didn't realize that would ping you!  Sorry!


Actually, I think an even cleaner way would be to pass `alpha` as an additional parameter to `flipImageIfAxesReversed()` and handle all the flipping at once. Let me know if I should make that change


This should be defaulted to '.*' so that it works out of the box, but can be configured to filter if necessary.


indeed


I see everything. EVERYTHING.


Thanks for the review @dawagner


petty


initializer list


fix #138


looks good, thank you.


isCommandSupported


Branch name is required, because we don't have the assurance that commentator would have permissions to leave code review comments inside another branch and that there are no 500 pages with sticked topics


Looks like a good start.


Maybe you could add a comment section outlining the type checking process you are trying to implement? Or if you are working from some source, reference that source...
I see parts of a type checker, but it is difficult to see how they will fit together.


well, i was hoping to get some information on compiler support once the testsuite cycle. fwiw apple-gcc-4.2 does accepts this code.

as for keywords: the problem is not the use of `foreach`, but the compilation will fail with awkward errors, once including `qglobal.h` and say `boost/flyweight.hpp` in the same translation unit. i'm also using range-based for and c++11, but boost.flyweight (and possibly other boost libraries) do not use range-based for loops, but BOOST_FOREACH.


The name gnu++0x is deprecated, better use gnu++11.


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


too long comment


We're going to want to manage our theme-able colors as a map and not as variables (I'll be working on this as part of #67), but at the very least, these should go into something like a `_theme.scss` file and the IBM colors should probably be renamed to like `_ibm-colors.scss`.


again, comment too long


I recover the majority of time for integration, but there are still a few lost milliseconds (around 10 millisecs). I guess they can be attributed to the timing step.
Yes we can externalize this step to further reduce the computation time.


test again


You don't need put a new line here.


@Antiavanti why copy?


Why did you remove this dependency?


There is a bug. Should be:
`self.longPressLocation = CGPointMake(gestureRecognizer.view.frame.origin.x + (gestureRecognizer.view.frame.size.width / 2), gestureRecognizer.view.frame.origin.y + gestureRecognizer.view.layer.borderWidth);`


Like a composite rule? The problem is that system property rule needs the root folder of the temporary folder, which is not available when the systemProperty is created.