if you override the `findAll` on the store, you won't need to do this, and the request will run over the socket not the http


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


合并


Hi @VoyagerOne

I think you should also update the move in the thread-local countermoves

thisThread->counterMoves.update(pos.piece_on(prevSq), prevSq, move);

Only then the update is 'complete', not?


The previous version of ASQ transport, aligned to the core v5 had a setting making JSON default serializer. From the API point of view, was it possible to override that and use XML even if ASQ set the default serializer to JSON?

If not, I can ensure backward compatibility with the current approach, if yes, and the default serializer could be changed I need to rewrite this.


Hi !

Why this such useful content from this commit is not in the [GETTING_STARTED](http://www.rubydoc.info/gems/factory_girl/file/GETTING_STARTED.md#Associations) anymore ?


adf


А это не NoopOutputStream получился, у тебя уже вроде есть NoopOutputStream, только он NULL_OS называется.


Ah yeah, good point. I'll change it and re-submit.  But if USE_BOOST_FOR_CXX11 is false, this implies that -std=c++11 is required?


thats not what i meant, this line should be only executed when creg is enabled:

https://github.com/spring/spring/blob/develop/test/CMakeLists.txt#L282

the same for the LoadSave test.

-> don't compile it


@leplatrem : done that


By @ namespace, do you mean the account address? Then it would require an additional mapping between accounts and connection strings, wouldn't it?


that is a lot of CSS, especially when considering there's inline styles in the HTML. Is this normal for HTML emails?


OK thanks.

On Fri, Jan 29, 2016 at 12:46 PM, Karlheinz Friedberger <
notifications@github.com> wrote:

> @cheshire <https://github.com/cheshire>
> the unit-test was already broken by your commit one day earlier.
> now both problems are fixed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sosy-lab/java-smt/commit/340b1f501a79e38049bd12f02f486e2a86621ba5#commitcomment-15759758>
> .
>


This needs correction. Try submitting it with age = "aaa" or any non numeric number. And it's better experience if you actually show what the error is.


This is a beautiful thing.


Yah, good idea. Added an .npmrc to the project so this will hopefully never happen again!


Fixed all remarks but there is some synchronization work to be done for
the `Option<Child>`.

On 01/29/2016 02:55 PM, Lukas Kalbertodt wrote:
> ... change this |wait_with_output()| to |wait()| to avoid capturing
> stdout. Should work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ivegotasthma/cargo-watch/commit/a4c4a114e8745e1105f95e0a901f032c2e474648#commitcomment-15762176>.
>


Thanks.


Good question. I think calling new Finder would be better in that case. :)


合并


I did find a few interesting cases here missing unit tests.
The most important would be the `view` property.
On the `if(this !== _itemMeta)` cases these are quite easy to add to the respective attribute regions below.
```js
Item.meta.label = undefined;
// The default value is still there (did not delete)
expect(Item.meta.label).toBe(null);
```


@ahshahid is this really a Spark bug? We need to rationalize all Spark layer changes and open a Spark JIRA with PR is this is so.


Why do some have `Backtrace.is_important e; raise e` while others have `Backtrace.reraise e`?


the missing boost_python-${PYTHON_VERSION_MAJOR}.${PYTHON_VERSION_MINOR} here introduces a regression on Gentoo


If you're annoyed that I used my own fork for pg-escape, it's because pg-escape doesn't work with capitalised identifiers! See this PR I made that hasn't been merged: https://github.com/segmentio/pg-escape/pull/12. So be careful that the session table/schema isn't capitalised if you do this. Good luck!


He was right, comment 2 line update Armor and Damage. Then mob can hit larger than 1 dmg.


I don't see why this happens, then.

If you are disagreeing, I have to tell you, again, that all ammo count is done server-side. Any ammo count of any player that client does is purely cosmetic.


We had talked about the _subClassed classSpec case and it still think it is ok to ignore that one :-)


TeamCity Civic Assembly :: Build backend + frontend [Build 45](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=592&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 84 (2 new), passed: 88, ignored: 4 Build time: 00:08:23
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1454076907238: <no details avaliable>

Venue tests: Venue type: Should Add an Venue: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

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


same problem as in line 97ff here


You mean `BH_UINT8` and the lot?


@natim something like that?


the wording is not right, I mean, you don't overwrite the meta object, you just set its properties


Этот метод вызывается jersey, а исключение из writer-а мы вытаскиваем после завершения вызова jersey. Этот IoExceptionCatchingOutputStream нужен, чтобы закрыть Jersey глаза на то, что были какие-то проблемы с отдачей ответа клиенту, т.к. поведение jersey при этом нам не нравилось. Данный дополнительный catch - для ситуации, когда вообще не удается получить OutputStream из HttpServletResponse, но при этом Jersey все равно уже закончил запрос и собрался писать ответ, так что оверхед остается только на сериализацию в пустоту.


Okay, I'll turn it back


On second thought there is a problem with that suggestion.  USE_BOOST_FOR_CXX11 is only defined at Siconos compile time, it is not defined by installed headers.  The important point about this patch is that the choice fo C++11 or Boost headers affects the ABI.  Therefore it's necessary for the choice to be available in SiconosConfig.h, so that code using Siconos compiles with the correct SP:: pointers.  Shall we export USE_BOOST_FOR_CXX11 into SiconosConfig.h?


Нужно указать ключевые слова и фразы через запятую, например:
браузерная игра, banana wars, банановые войны


http://example.com is not a valid site to POST to.  note that it does nothing with the data you sent it.


It looks like there is starting to be some common functionality in these tasks that interact with the API (abort if no network, etc.). Would it make sense to break that out into an intermediate base class like `DriverApiAsyncTask` and then have our tasks inherit from that?


Please read back your commit before committing :)


The logical could be much simpler, knowing that collapseThreads is already false by default.


just to make sure Radek @pazdera , kano-wifi-gui runs under sudo right? will track_action pull the interactive username?


what is not being sent?


yes, it would, I'll check to see if there is an issue for that in ASQ transport


@organisciak : How is this different from the command-line call `'bookworm query '{"jsonkeys":["etc"]…}'`, which also shortcuts the server?


@ahshahid also in future, all spark layer additions need a SnappyData modification header comment in the file. Correcting that for the files I am fixing for scalastyle bugs we introduced.


Should be 10, no?


Для надёжности всегда целесообразно проверять наличие метки входа, например "Выход".
Если её нет, значит авторизоваться не удалось и нужно искать и парсить ошибку.


Could we make this a bit more explicit, e.g. by calling it `run_helper_subcommand()` and making the subcommand a separate argument? As it is right now it sounds as if it would do the same as `run_helper` just with parameters, but it actually behaves different.


This is so that CAS gets to know the language?


Am I missing something? I absolutely cannot find out what you mean. As far as I can see you point to a line of code which I removed with this commit `zr_draw_vertex_color col = zr_color32(color);` which was transferred down lower to be able to modify it.


Does this require that node be the default one in the PATH? I ask because in PATH I have node v5 but beaker requires v0.12 so I wanted to set a custom path for beaker.


[Issue 156](https://bitbucket.org/ompl/ompl/issues/156)


How did this happen? :P


yes, default could be overriden


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (958 of 958 tests passed, 0 test warnings)

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


ah OK


Nope ;) The line `if (!list) return;` is not necessary, because the next line says `if (!list || points_count < 2) return;`.


sdfsdsdf


I'd move this up to 151 so the validation conditions for the record are prominent.


@ornicar Yep, that's ok because it's impossible to attach any scripts (JS is forbidden at all) or external CSS to email letters so you should include all of them right into the body of email.
You can check any other templated email in your inbox and see it.


NOICE!


.exists()


please add to this:
`https://api.ircbd.homeoffice.gov.uk https://api-ircbd-uat.notprod.homeoffice.gov.uk https://api-ircbd-int.notprod.homeoffice.gov.uk https://api-ircbd-dev.notprod.homeoffice.gov.uk`

And remove rawgit.com since I don't think we're using that anymore? - we shouldn't if we are


please add to this:
`wss://api.ircbd.homeoffice.gov.uk wss://api-ircbd-uat.notprod.homeoffice.gov.uk wss://api-ircbd-int.notprod.homeoffice.gov.uk wss://api-ircbd-dev.notprod.homeoffice.gov.uk`


NOICE AGAIN!


I put +1 on this and what about also to add support for custom estension like:

```php
     if (isset($config['view_helpers'])) {
            foreach ($config['view_helpers'] as $className) {
                $engine->loadExtension(new $className);
            }
      }
```


The last two methods (read, write) are they really necessary? In which case do you think this will be useful for the SSL implementation? If I understand correctly it will read/write bytes and tell you how many they processed, it implicitly means there is a buffer behind this socket, however there isn't such method to read or write into this buffer. Anyway, such names are really misleading.

By the way, same remark for the read method, why not sticking with the C standard read/write methods?
```C++
int read(char*);
int read(void*, size_t);
int write(const char*);
int write(const void*, size_t);
```

Last thing you should be concerned about is Microsoft ```std::string``` constructor implementation is clobbering ```errno```.

Finally, you should be careful with the UML diagram on the website because it doesn't match some files on this GitHub repository.

Cordially,
payet-s


i believe beaker's requirement is for building beaker.
for running the plugin, i am pretty sure the newer versions are good (i have heard but have not tried myself).
in any case you can use the prefs file to set a path just for finding the plugin binary:
https://github.com/twosigma/beaker-notebook/wiki/Language-Preferences


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


a better question might be, why is there no client id?


client id is a required piece of data and the admin console doesn't let you create one without one.


Hey Chris,
I was able to build the DARC module on my LINUX server under Gnuradio 3.7.2.1. The blocks are available, but I do not understand how to use them. Do you have a Gnuradio project example ?
I would like to test it with the example files from munich:
https://apollo.open-resource.org/mission:log:2014:08:08:darc-side-of-munich-hunting-fm-broadcasts-for-bus-and-tram-display-information-on-90-mhz

Although trying to:
root@HP-Z820:py# python darc_udp.py pipe_01_bp.dump
Traceback (most recent call last):
  File "darc_udp.py", line 10, in <module>
    from bitstring import Bits
ImportError: No module named bitstring

does not work...
Can you help ?

Klaus


Yes you're right


It would - thanks :+1:


Shouldn't this be in `SpecialCases`?


Thank you :bow:


You're right! It was never a problem because all lines have the same size but that was a stupid mistake of looking too much at line 172 while typing this...


Sorry about that Peter. I see that. I'll change this back to reflect that. Thanks again! @peterborkuti


SET(NO_CREG TRUE CACHE BOOL "" FORCE) should change the option

options are cached in CMakeCache.txt, when the default is changed it doesn't change the cache / used

(please test locally first, i'm not 100% sure about the syntax)


primul comit


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 136](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=729&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:37:49


Guys, API modification in a minor revision is gross, isn't it?


Nice Boat


Its well spelled congrats :+1:


thanks, although I don't really want to change the cache, this branch is mostly to get an automatic build without creg to check that everything is ok


Ну не знаю.
Во-первых, своим from ты мне мозг подвозорвал. From - это обычно static метод, который конструирует instance класса, в котором он находится. А у тебя не так. У тебя скорее не from, а getValue.
Во-вторых, ты написал *Property классы, которые всего-то заменяют заменяют строчки:
```java
parseInt(properites.getProperty("outputBufferSize", "65536"))
```
сравни со своими
```
final IntProperty OUTPUT_BUFFER_SIZE = new IntProperty("outputBufferSize", 65536);
...
OUTPUT_BUFFER_SIZE.from(properties)
...
плюс еще и класс IntProperty
```
У тебя:
- больше сущностей
- больше кода
- задание дефолта и использование значения разнесены по разным местам, так что на гитхабе придется прыгать

В чем профит-то?


Whow. What have happened to plain old `static const uint64_t stackLimit = 1024`?


If you want to get rid of that space limitation you can easily do this by using something similar to this:

```bash
a=("foo bar" "blub blub" "foo bar")
declare -a deduped
while IFS= read -r -d $'\0' line; do
	deduped+=("$line")
done < <(printf "%s\0" "${a[@]}" | sort -zu)
for i in "${deduped[@]}"; do
	echo "$i"
done
````


clang++ uses the same switch as GCC, so your code should work. However I'd rather have a good logic sooner rather than later. Quick patches have a history of exceeding their live expectancies in Siconos ;). For what's worth, cmake has some knowledge about C/C++ standards (look for CMAKE_CXX11_STANDARD_COMPILE_OPTION and others in /usr/share/cmake*/Modules/Compiler). It's not that comprehensive so I'm not sure it's worth using it right now.


I hate the idea of changing ```async``` to ```Promise``` but I agree the store needs to better test the count.


it's probably better to use native ability to build systemd-enabled packages. it should be  dh $@ --with systemd. it's done by sed call for mongo builds. the complete list of available addons may be found in dh -l output (dh_systemd addon is required)


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 265](http://ci.infinispan.org/viewLog.html?buildId=34712&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 01582fca79a75e550f577c2123d6de885aac0c93
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 1


Try to remove old code whenever possible.


なるほど、内部に参照カウンタを持ったmutexがあるのか


C++ 17 will allow static_asserts with only the condition. Clang might already handle that, and it will be the much more common use case, so we must handle it as well. We should check if there is a second argument, and if not then we should put the first one, without a comma expression.


any reason to have embedded calls? dh_systemd_enable and dh_systemd_start are separate build stages


Yummy lambdas!! :)


LGTM


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 235](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=730&buildTypeId=XbimEssentials_XbimEssentials) is now running


do we have any update-alternatives calls to manage config files?


Patch already sent upstream... waiting for feedback/acceptance
http://lists.freedesktop.org/archives/dri-devel/2016-January/099708.html


@sasairc

> コメント行の扱いはどうなりますか？

↓みたいなことであれば「なし」だと思っています:

```text
speaker:唯<TAB>quote:全然反省してない…
# かわいい
speaker:ゆずこ<TAB>quote:マジカルって つけると なんでも グレードアップ するよね
speaker:唯<TAB>quote:電波だとのせられねーぞ？
```

注釈を残したい場合は`note:〜`列を追加してください:

```text
speaker:唯<TAB>quote:全然反省してない…
speaker:ゆずこ<TAB>quote:マジカルって つけると なんでも グレードアップ するよね<TAB>note:かわいい
speaker:唯<TAB>quote:電波だとのせられねーぞ？
```

（そもそもレファレンス実装もまだなんだけどね）


then

unset(NO_CREG CACHE)

maybe :)


Squash this into https://github.com/Woile/marionettejs.com/commit/8079542a029ec3a0e3e417664881a4579ce80515 please :smile:


that's impressive.


Sorry it's taken me so long to notice all this, not been playing factorio lately (obv) but would've at least merged the PRs more promptly if I hadn't somehow left github notifying me on the wrong, outdated email address...


How can I switch this setting? I tried adding `keepEmptyPriority: true` in the `"imdone-atom":` section in the Atom's `config.cson`, but it doesn't influence anything: when I drag items on the board, they get the priority number that I don't want to use at all.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 235](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=730&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:31


Oh no. Do I still have time to update my translations?

On 1/29/16, alex-crystal <notifications@github.com> wrote:
> It is perfectly! Will wait!
>
> Crystal
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BearWare/TeamTalk5/commit/0b2b8035b679a504142349cf6462ca37c1827da3#commitcomment-15756919


Ну, ниже тоже jersey специфики нет, тоже можно переименовать.


Lets break from liferay conventions.  Because bnd conventions is that
symbolic name == directory name unless they specifically set it.  So that
is my vote.  Actually I wish the Liferay gradle plugin didn't fail if you
don't have a symbolic name.  Because if you make symbolicName ==
packageName then that require every "blade create " command to specify a
-packagename. And a lot of people trying it out the first time wont do that.

On Fri, Jan 29, 2016 at 3:45 AM, David Truong <notifications@github.com>
wrote:

> The symbolic name is the package name in Liferay conventions. I don't think
> it should be name as least not the way we have set name
> On Fri, Jan 29, 2016 at 3:15 AM Andy Wu <notifications@github.com> wrote:
>
> > sure
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/david-truong/liferay-blade-tools/commit/72bcfc407e288d38f423d0326b6aa6033f99d0d3#commitcomment-15751036
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/david-truong/liferay-blade-tools/commit/72bcfc407e288d38f423d0326b6aa6033f99d0d3#commitcomment-15756646>
> .
>

--
Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com


Alright, I'll take a look at that, and find something tested against both g++ and clang++.  It would be better to use the CMake feature if possible, since it's more likely to work on other compilers as well.  I'll note that getting C++11 stuff working reliably appears to be a prerequisite to linking with Siconos from the Gazebo code base, that's the main reason I'm working on this.  Perhaps the CMake option you propose would allow to avoid the indirect try_run() approach for setting SICONOS_CXXVERSION.. I'll take a look.


@mkanoor You are right :+1:


I'm not disagreeing you fucking arrogant cunt. I'm asking.

Fuck off.


Forgotten `it.only`, please remove.


Good catch. USE_BOOST_FOR_CXX11 has to be defined in SiconosConfig.h for this to work. I added this extra control since I was not sure that we could replace the boost part with the std library without any major hassle. This was more a defensive programming move. Also I think if for whatever reasons someone wants to use the boost stuff (because of another lib for instance) and compile in C++11, it could work. People would have to put that flag in their CXXFLAGS manually. A cmake option could be easily added though.


What about:
```
        it("preserves the default value", function() {
          var FirstDerivative  = Item.extend({meta: {label: "Foo"}});
          var SecondDerivative = FirstDerivative.extend({meta: {label: "Bar"}});
          SecondDerivative.meta.label = undefined;
          // The default value is still there (did not delete)
          expect(SecondDerivative.meta.label).toBe("Foo");
        });
```


Is `<version.org.keycloak.secretstore>` still in use somewhere at all? In case it is, `-SNAPSHOT` does not sound right.


Never mind, I see you have reverted it in next commit. I'll delete my comments.


The whole message ie (name, email, subject and message)


@mkanoor I mean something like this:
```
@object = obj.kind_of?(ar_klass) ? self.class.filter_objects(obj) : ar_method { self.class.find_ar_object_by_id(obj.to_i) }


Is this not manages somewhere already? Otherwise, it should go to hawkular-parent.


Is this typo error?


Nitpick: No need to pass `null` as the first argument, `validateAndCallConnector()` is good enough.


CI 历史 ： https://ci.appveyor.com/project/qinwf53234/jiebar/history

CI pass: [`Add: suggest rmarkdown master 367d4166`](https://ci.appveyor.com/project/qinwf53234/jiebar/build/1.0.164) 七天前，v2 代码

这个还是旧版的 cppjieba 代码，测试没有问题。

CI fail: [`update: pass R CMD check on windows update-cpp 1ab6c2fe`](https://ci.appveyor.com/project/qinwf53234/jiebar/build/1.0.165) 2天前，v4 代码

从这个开始，用了 v4.4 的代码，就开始报下面的错误了。

```
Error: 2016-01-28 10:03:43 ../inst/include/lib/HMMModel.hpp:60 FATAL exp: [LoadEmitProb(line, emitProbB)] false.
Execution halted
```

这个报错是因为 getline 文本没有问题，但是英文平台下 split 不正确，分割后的字符串可能会 Decode 失败，返回 false 。 比如我找到的系统词典里一行 `一万张 3 m` ，在英文平台里用新 split 后 ，Decode 会失败。

之前都在处理其他平台的问题，因为本地 windows check 都是通过的，所以没有注意看 Windows CI，所以一直都是红上来，失败了很多次，没有修复。还有我log 的等级之前设的也是 fatal ，所以 decode error 可能在这里没看到。如果报 decode error 会因为读取的词条比较多，会瞬间出很多 error 信息。

发现这个问题以后，我重新拿 v2 的代码 CI pass: [`
add: roxygen2 test-appveyor 7fcd17c7`](https://ci.appveyor.com/project/qinwf53234/jiebar/build/1.0.164)  跑了一遍 CI 确认是有前后差异后。

对比前后代码运行结果差异，用旧的 split 替换新的 split 得到的就是最新的这个 CI 结果了  [`fix: windows char split bug
master 02c31111`](https://ci.appveyor.com/project/qinwf53234/jiebar/build/1.0.174)


Are there any activity ongoing to get away from those hard coded port assignments? I think I saw that somewhere...
Should be done via the port mapper instead. A Feature Request this is.
/A


I think this test case should be preserved, it verifies context properties.


persistence not persistance.
Also, I would rename serialize to toString()


Please note we should use JSESSIONID cookie name
If it conflict with other issues we should other name that more appropriate (e.g. INTITA_SESSION)


You can make those changes *either* using the GUI client on your local machine *or* by directly editing [the file within the browser](https://github.com/programminghistorian/ph-submissions/blob/gh-pages/lessons/intro-to-ar-development.md), and adding the commit message at the bottom of the page. (Click the "pencil" icon next to the trash can to edit.) [Add your author information](https://github.com/programminghistorian/ph-submissions/blob/gh-pages/_data/authors.yml) to the `authors.yml` file in the data directory, following the format that you see in the other examples there.


here you are just reading anyway so there shouldnt be any issues at all??


We discussed this offline. Even though it would be easy to copy this, the dep is also pretty simple (just includes a bunch of simple collector definitions) so we'll bring it in unless someone has a problem with it.


npm knows about paths so you can simplify to mocha ./tests/


this throws if a client looses connection, which can be business as usual


I'm not aware of any such system currently in place for this.


Excellent.  Thanks Tomasz! @FadeMind


```
host: mysql
```


licence needs to be MIT


В jetty профита не вышло, а grizzly до этого изменения был нечитаем. Пока обдумываю, у всех перепробованных вариантов нашлись минусы.


Sure is, and a rather random one. I'll fix it next time I push some changes.


h


@844196
岡（ちー）の下。

C89用のパーサあったけれど機能モリモリで遅そうだったので、必要になった時に車輪を泥で再発明する予定です。


I think we should use `replaceById` instead of `prototype.replaceAttributes`, as the former is more performant.


forgot to add local changes before doing `git commit`?


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


Why would all seeks in live be treated as seeks to 0?  What is your model for Live/DVR seeking?


AFAICT, the expected context is the same for both "atomic implementations" and "non-atomic implementations". Is that correct? In which case this test can be simplified by removing `if(dataSource.connector.replaceOrCreate)`.


Habe Prof Jeromin gefragt und er hat gemeint: A^2 macht keinen Sinn, da es sich hier nur um eine Punktemenge handelt. Bei V macht hingegen die Dimension Sinn.


in case libasync and libevent support the waitForData i think we should move to that


I figured out that it can be added in `.imdone/config.json`. Could you add it as a global switch to the plugin settings?


Hi!
Good catch:)


Actually, I think we still need the switch, because the non-atomic implementation should report `ctx.isNewInstance: false`, while the non-atomic implementation cannot tell whether we will call CREATE or REPLACE at the end.


The usage of a global index counter shouldn’t have any effect to the test suite. This looks like there is something odd in your changes.


Please fix the test case and the implementation, so that `replaceById` sets `ctx.isNewInstance:false`.


ok


Ah duh. OK I fixed it. Will upload the changes combined with some other changes.


@igor-ivanov Why do you need to modify `pmix_hash_fetch()` to support PMIX_RANK_UNDEF if you introduce `pmix_hash_fetch_by_key()`?


This looks suspicious to me, as it's effectively reverting an earlier fix. Is it intended?


AFAICT, this is removing "validates model after `before save` hook on replace" with no replacement, is that intentional?


What's wrong with Saturdays?


Oh, gosh, is it possible for our touchscreen?


No, codes should be different. Better put that thought up for review with others :)


please comment here https://github.com/searls/jasmine-rails/pull/176
to discuss your concrete problem


I think this should have `ctx.isNewInstance:false` when the connector does not provide atomic implementation.


我感觉这个应该是 windows 系统本身处理字符串的 bug，就像以前 IE 5、6 不遵守 web 标准一样。一般用中文系统的人一般不会遇到。

R 接口考虑需要兼容一下英文系统，是因为 R 包仓库用英文系统跑测试，四大类操作系统平台，不同编译器各种 check，R 包仓库的管理员还偶尔抽几个包来跑内存泄漏测试，反正就是各种测测测了。。。:cry: 测试不通过不给发布。


Missed that. Fixed in the next commit...


you probably want to `stub` rather than `spy`, since you don't care about it actually being able to connect to location


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


s/has/as/ I assume


I think this change will not work with Symfony smaller than 3?


ack


<idiomatic rant>
1. Why? It is just another local variable, after all...
2. A matter of personal style. I'm used to use weak conditions. Makes it less prone to obscuring errors.
3. It is used only once... Is it worth it?
4. We are using `push` _like_ a static function, so it seems cleared to my to reference `Array` instead of repeating `errors`.
</idiomatic rant>

We need to clarify the intended requisites. I would agree with you with stopping at the first fail, but then returning a boolean would be enough. If `validate` returns an array of `Error`, then I'm expecting it to aggregate all the invalid cases. And, of course, to be able pin point them (that information is missing).

In the sandbox it would be nice to display a list of errors, so the developer understands what is wrong.


I suppose this is just `click.js` and `ClickOptions`, like as `move.js` and `scroll.js`.
`mouse-action` has this postfix because we can't name it just `mouse.js` and `MouseOptions`


AFAICT both branches assert the same, please merge/unify.


though there's no fixes note


What change? xD


@igor-ivanov I thought we agreed to return the first occurence of the key. But it seems that here you will look for all key instances.


Yep, sorry. Last thing done yesterday, kind in a hurry. Will complement it.


don't forget remove it before the pull request =)


this isn't naming your module, this is requiring pace recursively


Comment by me


@igor-ivanov Why do you need to pull this constructor here?


Should there not be a check that this is a media type that we accept? There's nothing in the spec that says this has to be an image.


TeamCity Civic Assembly :: Build backend + frontend [Build 46](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=594&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 79 (1 new), passed: 93, ignored: 4 Build time: 00:10:37
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1454079206394: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

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


LGTM


Just being lazy. :-P Will improve this.


One could be reset the counter between compiling different model, but for what it's worth.


I see, thanks for the note!


**Line is longer than allowed. (81 > 80)**
*LineLengthBear, severity 1, section `python`.* *Related to file `coalib/tests/results/SourcePositionTest.py`, line 26 column 81 to line 26 column 82*.


This is weird. `options.notify:false` instructs the method to not trigger any notification hooks, thus this entry should not be created. I do realize `updateOrCreate` does create two entries. I am proposing the following:

 - fix the new replace* methods to honour `options.notify:false`
 - fill a new GH issue for LB3.0 to fix `updateOrCreate` (or the methods it calls) to honour `options.notify:false` too.


Great, will change.


Yep.


Is automationSettings.CLICK_STEP_DELAY used just here? If so we can get rid of this and define it in this file


Working but showing wrong orientation in preview


-> #45


@qinwf 我大概猜测了一下，很可能是这个trim函数里面包含了 isspace这个函数（isspace在windows上面的实现和linux上面似乎不太一样），可能导致问题，你能否尝试把 split 函数里面的两个trim函数注释掉再跑测试试看看？非常感谢。


As you noted, `QuaternionFromAxisAngle(float angle, Vector3 axis)` doesn't look coherent, I'll review it!


It seems like you forgot to push `Atom.jl`, since it doesn't recognise `ping` on master.


Ok, so the regarding two flags USE_CXX11 (or USE_STD if you prefer), and USE_BOOST_FOR_CXX11, the story will be:

case USE_BOOST_FOR_CXX11=ON, USE_STD=ON:
  * compile with -std=c++11
  * SiconosConfig.h features SICONOS_CXXVERSION=201103L
  * namespace std11=boost
  * code compiled against Siconos must define USE_BOOST_FOR_CXX11 to link correctly

case USE_BOOST_FOR_CXX11=OFF, USE_STD=ON:
  * compile with -std=c++11
  * SiconosConfig.h features SICONOS_CXXVERSION=201103L
  * namespace std11=std
  * code compiled against Siconos links correctly automatically

case USE_BOOST_FOR_CXX11=ON or OFF, USE_STD=OFF:
  * compile without -std=c++11
  * SiconosConfig.h features SICONOS_CXXVERSION=199711L
  * namespace std11=boost
  * code compiled against Siconos links correctly automatically

to cover the first case, perhaps we could add something to siconosConfig.cmake to add a -DUSE_BOOST_FOR_CXX11 if it was defined during the build.


I already said that in a bug i filed in this repo ...

Please, before asking, take a look at the diff. The coordinates changed. I tested it with some debug messages before - and the coordinates before were simply wrong.


@qinwf 这里还有一个trim


You can make the param optional like def get_net(gpu_num=0): so that callers can omit it and it will use that default value.


Could you try and see whether I need to add one or two backslashes in the code above? I don't have the hamlib tools installed at the moment.


You're just incrementing the local var here?


As I see you've moved this to domUtils


This has no sence?¿ There are already ShowCursor() and HideCursor() with DESKTOP and RPI support... no? @raysan5 Now there are thow functions which makes same thing; correct me if I'm wrong


That is to say, we could add another output variable in siconosConfig.cmake called "siconos_CXXFLAGS"


Whoops, fixed


does this need to be global since it comes from the post?


TODO: update from legacy LocalZoneSID terminology


Added additional comments, but am not seeing commit.


@gnn : I had to disable this test because it did not work any more and I do not know how to fix it.