-
Notifications
You must be signed in to change notification settings - Fork 5
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
Rt inference #239
Rt inference #239
Conversation
Codecov Report
@@ Coverage Diff @@
## main #239 +/- ##
===========================================
Coverage 100.00% 100.00%
===========================================
Files 105 47 -58
Lines 3380 1122 -2258
===========================================
- Hits 3380 1122 -2258 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @laraherriott , Here are a few minor suggestions before merging this in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with Richard's suggestions. Here are a couple of things I would also suggest you delete / change in addition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy with this, and the more standard
Add Rt inference notebook |
Summary
Notebook to illustrate inference of Rt on the New Zealand output files.