Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2101 +/- ##
==========================================
- Coverage 80.31% 79.66% -0.65%
==========================================
Files 330 330
Lines 22641 22660 +19
Branches 1532 1533 +1
==========================================
- Hits 18184 18052 -132
- Misses 4447 4598 +151
Partials 10 10
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
b0a00e9 to
74b9d3d
Compare
|
Current state: |
I remember that we had issues with IC calculation for models where there shouldn't have been problems, even for simulation only. |
After a closer look: For simulation-only, we are fine for all currently tested models. For forward sensitivities, we have a variety of issues:
|
Fixes import of some SBML test cases with algebraic rules (e.g. `00661`). Related to AMICI-dev#2101.
For future reference: The problem was using the wrong mass matrix. In matlab-imported models, only |
|


Support for AlgebraicRules was added in #2017. Sensitivity analysis is not yet supported.
This implements the missing
Model_DAE::fdxdotdpandModel_DAE::fsxdot, and imports all SBML test suite models with sensitivity code.Fixes #2102.