Closed fabian-s closed 7 years ago
Thanks for your suggestions. I add later some comments on the other changes you suggest, and discuss here below why I am skeptical about 'roxygenizing' the Rd files and namespace, and I prefer keeping these as they are. Please let me know how you want to proceed.
Regarding the other changes:
Not gonna argue about roxygen with you, I think it's a huge mistake to not do this but it's your package and your funeral. Re. the code formatting you'll have a really hard time to disentangle my style changes from the actually changed code... Sorry about that, but, I'm sorry to say, the style you used is quite idiosyncratic and it's hard to collaborate on code that's hard to read because it's not formatted in a standard way...
Which other error/warnings were generated? Are they those reported in the CRAN check, with some --no-stop-on-test-error messages
I don't remember -- does it matter? Main thing is at the package now R CMD check
s without errors/warnings, right?
Summary of changes:
See comments under TODOs in dlnm-package.R, l. 81 for what's still missing:
exphist
,exphistint
is different - why? can this be changed?Rnw
-sources are missing"fun"
-string inonebasis
,checkonebasis
instead of handing over a proper function invites nasty scoping issues that seem to break examples in R CMD check, so I set those to ' \dontrun' . They still work fine in the console.