mllg / batchtools

Tools for computation on batch systems
https://mllg.github.io/batchtools/
GNU Lesser General Public License v3.0
172 stars 51 forks source link

JOSS manuscript: Incorrect statement about NEWS #76

Closed HenrikBengtsson closed 7 years ago

HenrikBengtsson commented 7 years ago

In https://github.com/openjournals/joss-reviews/files/639511/10.21105.joss.00135.pdf:

but the NEWS.md file now refers to the vignette:

batchtools 0.9.0 Initial CRAN release. See this vignette for a brief comparison with BatchJobs/BatchExperiments.

The manuscript should be updated to refer to the vignette(s).

BTW / FYI, although CRAN renders NEWS.md on their package page, it is not recognized by utils::news(), e.g.

> news(package = "batchtools")
NULL

and the HTML help (as in help.start()) doesn't list any NEWS. In other words, I think by using NEWS.md the NEWS information is only available on CRAN and GitHub and file.show(system.file("NEWS.md", package = "batchtools")). Should probably be reported / fixed in R itself, but I wanted to let you know.

mllg commented 7 years ago

I've updated the reference in a1c8bdd88949d3416226c16e829293b462301311. Will look into news() (which I never used before).

mllg commented 7 years ago

Talked to Uwe about this. We agreed that this is really something to be fixed by R core. He'll look into it.