-
Notifications
You must be signed in to change notification settings - Fork 10
Ndefries/epidatasets migration #382
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
Conversation
4815e37
to
b50d6df
Compare
@dajmcdon Any suggestions on the tests failures? There are also a bunch of warnings about adding new snapshots. The tests pass locally for me. |
How are you running them locally? You may have to push changes to those files. |
I'm doing |
Can you try |
|
This reverts commit 00d0dd7.
Co-authored-by: Daniel McDonald <[email protected]>
@brookslogan Documenting runs faster for me now. Is it working better for you? |
Checklist
Please:
dajmcdon.
DESCRIPTION
andNEWS.md
.Always increment the patch version number (the third number), unless you are
making a release PR from dev to main, in which case increment the minor
version number (the second number).
(backwards-incompatible changes to the documented interface) are noted.
Collect the changes under the next release number (e.g. if you are on
0.7.2, then write your changes under the 0.8 heading).
epiprocess
version in theDESCRIPTION
file ifepiprocess
soonepipredict
andepiprocess
Change explanations for reviewer
Moves included datasets to the
epidatasets
package. Some datasets are being used by epiprocess as well, so dataset names change. The data are reexported inepipredict
for easy access. Data contents mostly remain the same. The exception is that the Canadian graduate salary dataset now has values for 2017.