-
Notifications
You must be signed in to change notification settings - Fork 16
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
677-computation-of-the-effective-reproduction-number-in-the-secir-model #698
Closed
johapau
wants to merge
54
commits into
main
from
677-computation-of-the-effective-reproduction-number-in-the-secir-model
Closed
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
3285184
Added reproduction number files to the branch
johapau 6cca6ba
Tested the secir reproduction number calculation
johapau 46af975
Upper-lowercase changes
johapau 58048f2
changed tests, added namespace
johapau af35572
changed secir-model.h and included methods to compute reproduction
johapau 7fc41d5
added test for secir-reprod numbers
johapau 1674b76
changed model.h
johapau a2867ab
reproduction number test added
johapau 7851f20
Merge branch '677-computation-of-the-effective-reproduction-number-in…
johapau 500e389
nothing major
johapau 7792f5b
First calculations to find the reproduction number
johapau 382843c
temporary save
johapau 5da4e8f
stash
johapau d6718c5
stashed
johapau fec6ee5
stash
johapau 1bea8dd
stash
johapau 8ac1b6e
changes
johapau 4d3767d
finished calculation
johapau f18362b
adjustment
johapau 21ee673
documentation
johapau 5036a2e
changes
johapau a7b0b80
Still have to correct errors in testing
johapau 1f4d801
tests finished
johapau 2e81e62
reversed changes to ode_secir
johapau d57f941
changes
johapau a61aa7a
formatting
johapau 1e372aa
CI changes
johapau e2a36a7
test results calculated by hand
johapau f5dadb7
tests modified
johapau 3d26c4d
output
johapau ea45b2c
output all
johapau b41708b
new tests
johapau 09a7674
changes to indices
johapau 9716861
Index check
johapau 5a43fdf
changes
johapau f5993fb
Merge github.com:DLR-SC/memilio into 677-computation-of-the-effective…
johapau 6f0aa91
integrated new compartments
johapau 281eac3
removed comment
johapau 125f7bb
remove iomanip
johapau 37e4123
unnecessary includes
johapau eb860ce
tests added
johapau 531dfdc
added one test
johapau 0ff5c48
changes
johapau c7e863d
considered case of empty agegroups
johapau 964135b
changes to total population
johapau 2cfeeb9
total population zero handling modified
johapau 0d66010
total population zero
johapau ba8c7c2
changes seasonality
johapau 3b039a0
tests modified
johapau 92215e8
additional test
johapau 0c79824
better invertibility checking
johapau b0b451c
fixed CI error
johapau e33b287
work in progress
johapau 200c9b7
changes undone, only time independent
johapau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
The diff you're trying to view is too large. We only load the first 3000 changed files.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 know/assume that this contains the derivative of the ODE so a lot of code-doubling happens. Generally it is just not a very good a idea to introduce redudant code parts. We should maybe think of how only one responsible code part returns for instance the mechanism of X and then this is used for get_derivatives and get_reproduction_number.