Skip to content

Conversation

@annawendler
Copy link
Member

@annawendler annawendler commented Jul 17, 2023

Changes

  • In the simulation, we update the compartment sizes now using the flows calculated for the current time step.
  • In the initialization, we compute the compartment sizes using the sum formula from the flows as before.
  • The order of the member functions is changed to clarify which functions are used to compute the initial size of the compartments and which functions are used for later time steps.

Merge Request - GuideLine Checklist

  • Check our git workflow before opening a Pull request/Merge request.
  • Request a reviewer when your work is ready to review, before this please use the draft feature.

Checks by code author

  • Every addressed issue is linked (use the "Closes #ISSUE" keyword below)
  • New code adheres to coding guidelines
  • No large data files have been added (files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)
  • Tests are added for new functionality and a local test run was successful
  • Appropriate documentation for new functionality has been added (Doxygen in the code and Markdown files if necessary)
  • Proper attention to licenses, especially no new third-party software with conflicting license has been added

Checks by code reviewer(s)

  • Corresponding issue(s) is/are linked and addressed
  • Code is clean of development artifacts (no deactivated or commented code lines, no debugging printouts, etc.)
  • Appropriate unit tests have been added, CI passes and code coverage is acceptable (did not decrease)
  • No large data files added in the whole history of commits(files should in sum not exceed 100 KB, avoid PDFs, Word docs, etc.)

Additional context

Please list additional information or things a reviewer should look out for.

Closes #675

@annawendler annawendler self-assigned this Jul 17, 2023
@annawendler annawendler linked an issue Jul 17, 2023 that may be closed by this pull request
@annawendler annawendler marked this pull request as ready for review July 17, 2023 07:46
@lenaploetzke lenaploetzke self-assigned this Mar 20, 2024
@codecov
Copy link

codecov bot commented Mar 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.35%. Comparing base (79158d1) to head (9d5af9f).
Report is 3 commits behind head on main.

❗ Current head 9d5af9f differs from pull request most recent head 6151dd0. Consider uploading reports for the commit 6151dd0 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #702   +/-   ##
=======================================
  Coverage   96.34%   96.35%           
=======================================
  Files         129      129           
  Lines       10072    10087   +15     
=======================================
+ Hits         9704     9719   +15     
  Misses        368      368           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lenaploetzke
Copy link
Member

The pipeline currently failed only because of the lower patch coverage due to added lines. When PR #923 is merged, the code coverage will be at 100% also with the changes of this PR.

@lenaploetzke lenaploetzke requested a review from mknaranja March 22, 2024 10:41
@lenaploetzke lenaploetzke added status::in review This issues is in review. model::ide This issue concerns any kind of integro differential equations-based model. labels Mar 22, 2024
@mknaranja mknaranja changed the title 675 alternative for compartments of ide 675 alternative computation of compartments in IDE model Apr 25, 2024
Copy link
Member

@mknaranja mknaranja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @lenaploetzke for these changes. I mostly have small suggestions (which however might change code lines in many occasions). Could you check them?

lenaploetzke and others added 2 commits April 26, 2024 11:25
Co-authored-by: Martin J. Kühn <62713180+mknaranja@users.noreply.github.com>
@lenaploetzke lenaploetzke requested a review from mknaranja April 26, 2024 11:20
@mknaranja mknaranja merged commit 8d0d615 into main Apr 26, 2024
@mknaranja mknaranja deleted the 675-alternative-for-compartments-of-ide branch April 26, 2024 13:13
reneSchm pushed a commit that referenced this pull request May 24, 2024
In order to reduce simulation time, compartment values in IDE SECIR model are now computed 
via last value and flow value of the current time step instead of historic flow values only.
For the initialization, compartment values are computed using the sum formula from the flows as before.
The order of the member functions is changed to clarify which functions are used to compute the
initial size of the compartments and which functions are used for later time steps.

Co-authored-by: lenaploetzke <70579874+lenaploetzke@users.noreply.github.com>
Co-authored-by: Martin J. Kühn <62713180+mknaranja@users.noreply.github.com>
Co-authored-by: René Schmieding <rene.schmieding@helmholtz-hzi.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

model::ide This issue concerns any kind of integro differential equations-based model. status::in review This issues is in review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement alternative computation of compartments in IDE model

4 participants