Skip to content
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

[Feat] Built-in parameter distributions and estimates from literature #132

Closed
6 tasks done
quantum9Innovation opened this issue Dec 29, 2021 · 3 comments · Fixed by #136
Closed
6 tasks done

[Feat] Built-in parameter distributions and estimates from literature #132

quantum9Innovation opened this issue Dec 29, 2021 · 3 comments · Fixed by #136
Assignees
Labels
feat 🚀 New feature or request good first issue 👍 Good for newcomers high-priority 🔼 This is important
Milestone

Comments

@quantum9Innovation
Copy link
Member

quantum9Innovation commented Dec 29, 2021

Is your feature request related to a problem? Please describe.
This feature is not related to any problem, but it is inspired by epiforcasts/EpiNow2

Describe the solution you'd like
Epispot should include a handful of parameter estimates from literature to aid with the creation of new models. Additionally, epispot should come with built in random modeling of various parameter distributions which can either be selected manually or automatically in the case that the user has not passed in a function for a specific parameter.

Describe alternatives you've considered
The alternative of not including this would complicate model creation greatly as there are often many, many parameters (exponentially many so as more compartments are added). Therefore, it would be helpful to have some kind of "filler" parameter distribution for unlabeled parameters. Additionally, by adding literature estimates, we can fit COVID-19 trends better using vetted sources rather than tweaking parameters to find an exact match for the data.

Additional context
No testing is expected to happen before this code is pushed to production

Current progress:

@quantum9Innovation
Copy link
Member Author

/claim

@github-actions
Copy link

Thank you @quantum9Innovation for claiming this issue!
Please reference this issue when you submit your PR and make sure to follow the contributing guidelines.

@quantum9Innovation
Copy link
Member Author

Ready for the PR (all remaining items we aim to finish by the release of v3, though these can always be postponed to minor updates instead)

quantum9Innovation added a commit that referenced this issue Dec 29, 2021
Epispot now supports built-in parameter distributions, see:
#132
for more.
@quantum9Innovation quantum9Innovation linked a pull request Dec 29, 2021 that will close this issue
Development v2 automation moved this from To do to Done Jan 3, 2022
Operation: spaghetti automation moved this from To do to Done Jan 3, 2022
quantum9Innovation added a commit that referenced this issue Jan 3, 2022
* Add `params.py`
Epispot now supports built-in parameter distributions, see:
#132
for more.

* Fix spelling

* Fix spelling

* Fix spelling
@quantum9Innovation quantum9Innovation unpinned this issue Aug 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 🚀 New feature or request good first issue 👍 Good for newcomers high-priority 🔼 This is important
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant