Skip to content

Extended PEtab test suite / cleanup PEtab state reinitialization#1344

Closed
dweindl wants to merge 8 commits intodevelopfrom
petab_tests_develop
Closed

Extended PEtab test suite / cleanup PEtab state reinitialization#1344
dweindl wants to merge 8 commits intodevelopfrom
petab_tests_develop

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

@sonarqubecloud
Copy link

@sonarqubecloud
Copy link

@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