Skip to content

Move prob_arx() in advanced vignette over to epipredict #54

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

Closed
ryantibs opened this issue Mar 4, 2022 · 1 comment · Fixed by #538
Closed

Move prob_arx() in advanced vignette over to epipredict #54

ryantibs opened this issue Mar 4, 2022 · 1 comment · Fixed by #538
Assignees
Labels
documentation Improvements or additions to documentation P2 low priority

Comments

@ryantibs
Copy link
Member

ryantibs commented Mar 4, 2022

We should move the prob_arx() function from the advanced vignette over to the epipredict. I moved it over to using list args in preparation for that. And, when the latter package is up and running, we probably just want to call an appropriate forecaster from epipredict for the example in the advanced vignette, because that in-vignette definition of prob_arx() is kind of unreasonably long and daunting.

@dshemetov dshemetov added the documentation Improvements or additions to documentation label Mar 9, 2022
@brookslogan brookslogan added P2 low priority blocked-by-internal Can't/shouldn't proceed w/o internal action --- link to relevant Issue, rm label when complete labels May 24, 2022
@brookslogan brookslogan removed the blocked-by-internal Can't/shouldn't proceed w/o internal action --- link to relevant Issue, rm label when complete label Jun 28, 2023
@brookslogan
Copy link
Contributor

Remaining work here is to use epipredict forecasters in examples, or maybe to simplify prob_arx by taking away a bunch of options and using any completion utilities we implement.

@brookslogan brookslogan self-assigned this May 3, 2024
@dshemetov dshemetov self-assigned this Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation P2 low priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants