-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possible improvements to example usage #26
Comments
Thanks for the suggestions, @johnlees. I have added the package tutorial and the iterated block particle filter tutorial as formal R package vignettes. These are now in the GitHub version of spatPomp and will be pushed to CRAN in the next upload. I have simply included the pdf tutorials as vignettes, to make sure they are found by people who look for help via vignettes on CRAN or through the package itself. There could be some advantage to putting the vignette source code into the package, but examples sufficient to demonstrate and validate the code are too computationally intensive for inclusion in the package. So, it seems appropriate to keep the source code for these vignettes in their own GitHub repos. You noted that some of the code for the tutorial is hidden. I have added links to the full R script, extracted from the Rnw source using In spatPomp, the R documentation examples are mostly not run, and the reader is referred to the tests for complete tested example code. This is done to save time, since it takes a nontrivial time to compile the spatPomp model objects. For examples, one would have to re-build the model objects many times. For the tests, many functions can be tested on each model object. Looking at your comments, I notice that |
Excellent, thank you for these comprehensive changes! I will update the JOSS review |
Current usage examples are detailed statistical PDFs, or source code. An improvement to users would be to consider adding one of these as a formal R package vignettes, at least for one of the listed resources. I think a simple example running the iterated block particle filter would be nice.
I was able to follow through the existing PDF (which has a lot of troubleshooting and plotting which is great), but some commands didn't work 'out of the box' e.g.:
And the doParallel setup was not explicit
The text was updated successfully, but these errors were encountered: