-
Notifications
You must be signed in to change notification settings - Fork 15
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
implement modified diebold mariano test #22
implement modified diebold mariano test #22
Conversation
I've migrated across the DM test stat code that Rob wrote with some small updates to make it consistent with Note that I've added statsmodel as a required python package. I still need to write up a tutorial notebook. |
Notebook tutorial added. |
ce3fbae
to
1f15b00
Compare
1f15b00
to
5a80823
Compare
f5fe3ae
to
7fbb5c3
Compare
FYI @tennlee , I've updated the Github actions here to also run unit tests on the develop branch (they were only running on main) |
f613f50
to
2b0a6e1
Compare
@tennlee Can this be merged? |
- Stripped to barestbones - Run all tests, all still succeed
Bring back Python 3.10 and 3.11 Leave Python 3.9 in the mix for the future
e4d911c
to
b944d7c
Compare
Thanks for that review @tennlee, I've made some updates based on your feedback |
* implement modified diebold mariano test * added dm test stat tutorial notebook * Implement the `acovf` function from `statsmodels` * updated github actions to run tests on develop branch * Add copyright from statsmodel * Improve keyword matching in exclude statements for pyproject.toml * added warning for dm tests if there are NaNs --------- Co-authored-by: nicholas loveday <nicholas.loveday@bom.gov.au> Co-authored-by: Harrison Cook <harrison.cook@bom.gov.au> Co-authored-by: Tennessee Leeuwenburg <tennessee.leeuwenburg@bom.gov.au>
No description provided.