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

Improve implementation of reordering within estimate_joint #118

Open
sangeetabhatia03 opened this issue May 21, 2021 · 0 comments
Open

Improve implementation of reordering within estimate_joint #118

sangeetabhatia03 opened this issue May 21, 2021 · 0 comments

Comments

@sangeetabhatia03
Copy link
Collaborator

Current implementation works but is quite clunky. I think we could improve it using something like the following:

  • reorder_incid (Reorder based on transmissibility, returns say the index of the max transmissible variant and reordered incidence)
  • all operations are now on reordered incid
  • then reorder_results, with the index from step 1

This also has the advantage that we can reformat results in the lasts tep to look exactly like results from EpiEstim.

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

No branches or pull requests

1 participant