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

EpiNow2: Stan code review #43

Open
benjaminortizulloa opened this issue Nov 26, 2020 · 0 comments
Open

EpiNow2: Stan code review #43

benjaminortizulloa opened this issue Nov 26, 2020 · 0 comments

Comments

@benjaminortizulloa
Copy link
Collaborator

benjaminortizulloa commented Nov 26, 2020

please edit in outline from the previous task (no local copy - oops)

EpiNow2 makes use of stan to define and fit its model. This code is complex and requires independent review. There may also be areas that could benefit from optimisation or simple model changes that may drastically increase performance. This task could extend to recommendations on model structure or be limited to the functionality of the code.

  • Review all code in the inst/stan folder based on stan best practice and suggest improvements
  • Review interface R code (mainly in R/estimate_infections.R) and suggest improvements
  • Check testing of stan code (tests/testthat/test-stan......R) and suggest improvements
  • Review prior usage and initialisation
  • Review overall model structure

[impact: please edit in from previous task (no local copy -oops)]
[originally proposed by @seabbs]
[suggested repo: https://github.com/epiforecasts/EpiNow2 | @seabbs]
[additional notes: ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant