-
Notifications
You must be signed in to change notification settings - Fork 28
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
version 3.11.7 Progress #177
Comments
need to recheck the test-MARSSkfss: with MARSS 3.11.7
same behavior occurs with MARSS 3.11.4 and 3.11.3. I think it is a numerical issue at these really small R values. |
Tests run on
On Posit.Cloud (linux) and M1 MacBook Air 2020. With R 4.2.3. Compared to MARSS 3.11.4 and some test compared on MARSS 3.11.3. |
I cannot replicate. I think the code was something like
But perhaps this is marssTMB specific? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This release is mainly concerned with updates to allow the marssTMB package to integrate with MARSS.
Fix the articles tab
Get the manual to build on MARSS-User-Guide repo
Run checks without building the vignettes
devtools::build()
it destroys theinst/doc
folder.Note, the Mac LaTeX installation doesn't install packages on the fly. You may need to install the needed packages. From terminal:
tlmgr install imakeindex
,tlmgr install collection-fontsrecommended
,tlmgr install footmisc
, tlmgr install appendix`, plus a few more in header.tex.Open terminal
Run version tests
Run this
vignettes/versiontest.R
Run checks on the built package
Run internal tests
tests/model.R
to set up models for testsdevtools::test()
but I find running the tests one by one in RStudio is easier to make sense of. Some tests will through errors, but all test should pass. Errors are correct if the test passes and result should be try-error.[ FAIL 0 | WARN 8 | SKIP 0 | PASS 57 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 76 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 18 ]
All pass on Linux, on M1 Mac the hard coded logLik are different up to delta 1 while on Linux they are the same.[ FAIL 0 | WARN 0 | SKIP 0 | PASS 45 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 68 ]
In one model, R is < 1e-13. On Linux, one logLik test on line 81 fails although the par are the same within machine tolerance. On M1 Mac, the same test passes.[ FAIL 0 | WARN 0 | SKIP 0 | PASS 160 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 120 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 84 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 5 ]
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 4403 ]
Rerun the plot testing Rmds in inst/testing-releases and make sure the plots look ok
Check the documentation pdfs
Compress documentation pdfs
Update the doc folder for GitHub. This has to be done manually by copying from
Prepare for release:
urlchecker::url_check()
Update the doc folder on GitHub. This has to be done manually
Re-read the submission guidelines to see if anything changed: https://cran.r-project.org/submit.html
Check that package builds on Windows
Upload package to CRAN
The items below are for cases where you are submitting the package files without any extra processing. I process my vignette PDFs separately, so the below won't work. But when submitting there is a link to check the tar.gz file on Windows.
devtools::check_win_devel()
rhub::check_for_cran()
revdepcheck::revdep_check(num_workers = 4)
Submit to CRAN:
usethis::use_version('patch')
devtools::submit_cran()
Wait for CRAN...
The text was updated successfully, but these errors were encountered: