Open HenrZu opened 1 month ago
Attention: Patch coverage is 98.90110%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 96.17%. Comparing base (
9ee0b56
) to head (5b81f94
). Report is 3 commits behind head on main.:exclamation: Current head 5b81f94 differs from pull request most recent head 7c48cd4
Please upload reports for the commit 7c48cd4 to get more accurate results.
Files | Patch % | Lines |
---|---|---|
cpp/models/ode_secirvvs/parameters_io.h | 98.18% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I have compared the runtime of the new extrapolation function with the old one. The results are almost identical. The result shown are for the secir model.
<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns="http://www.w3.org/TR/REC-html40">
Num Days| 1 | 5 | 10 | 20 -- | -- | -- | -- | -- New Version | 30,3303 | 42,7880 | 58,3960 | 90,1510 Old Version | 29,3939 | 42,1209 | 57,7696 | 89,6838
Changes and Information
Please briefly list the changes (main added features, changed items, or corrected bugs) made:
export_input_data_county_timeseries
is similar to the functionality inread_input_data_county
.If need be, add additional information and what the reviewer should look out for in particular:
-
Merge Request - Guideline Checklist
Please check our git workflow. Use the draft feature if the Pull Request is not yet ready to review.
Checks by code author
Checks by code reviewer(s)