Skip to content
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

Extended PEtab test suite / cleanup PEtab state reinitialization #1344

Closed
wants to merge 8 commits into from

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Nov 30, 2020

To include cases from PEtab-dev/petab_test_suite#35

Related to #1319

@codecov
Copy link

codecov bot commented Nov 30, 2020

Codecov Report

Merging #1344 (5605966) into develop (b113544) will increase coverage by 9.24%.
The diff coverage is 75.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1344      +/-   ##
===========================================
+ Coverage    78.80%   88.04%   +9.24%     
===========================================
  Files           63        2      -61     
  Lines         9944     1405    -8539     
===========================================
- Hits          7836     1237    -6599     
+ Misses        2108      168    -1940     
Flag Coverage Δ
cpp ?
petab ?
python ?
sbmlsuite 88.04% <75.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
python/amici/ode_export.py 85.27% <75.00%> (-7.47%) ⬇️
python/amici/sbml_import.py 92.26% <0.00%> (-3.78%) ⬇️
src/edata.cpp
src/exception.cpp
include/amici/solver_cvodes.h
python/amici/petab_simulate.py
src/spline.cpp
src/cblas.cpp
src/vector.cpp
... and 49 more

@sonarcloud
Copy link

sonarcloud bot commented Dec 11, 2020

@sonarcloud
Copy link

sonarcloud bot commented Jan 26, 2021

@dweindl
Copy link
Member Author

dweindl commented Feb 12, 2021

Closing, as this isn't going to work as intended...

@dweindl dweindl closed this Feb 12, 2021
dweindl added a commit that referenced this pull request Feb 18, 2021
…fixes (#1417)

Adds possibility to provide state indices for selective reinitialization based on fixed parameters.

The previous `ExpData::reinitializeFixedParameterInitialStates` is still there, but will be removed in an upcoming version.


Addresses #1345, #1396, #1319

Supersedes #1344 

Include new test cases from PEtab-dev/petab_test_suite#35
@dweindl dweindl deleted the petab_tests_develop branch February 20, 2021 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants