Skip to content

Return amici.Model from sbml2amici, pysb2amici#2971

Merged
dweindl merged 1 commit intoAMICI-dev:mainfrom
dweindl:return_model
Oct 31, 2025
Merged

Return amici.Model from sbml2amici, pysb2amici#2971
dweindl merged 1 commit intoAMICI-dev:mainfrom
dweindl:return_model

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented Oct 8, 2025

The import functions sbml2amici, pysb2amici, and antimony2amici now return an instance of the generated model class if called with compile=True (default) avoiding the need for a separate call to amici.import_model_module.

Closes #2970.

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 75.79%. Comparing base (25adebf) to head (f4d46c1).
⚠️ Report is 101 commits behind head on main.

Files with missing lines Patch % Lines
python/sdist/amici/pysb_import.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2971      +/-   ##
==========================================
- Coverage   76.48%   75.79%   -0.70%     
==========================================
  Files         308      308              
  Lines       19540    19542       +2     
  Branches     1500     1500              
==========================================
- Hits        14945    14811     -134     
- Misses       4582     4718     +136     
  Partials       13       13              
Flag Coverage Δ
cpp 72.88% <50.00%> (-0.01%) ⬇️
cpp_python 38.26% <0.00%> (-0.01%) ⬇️
petab 38.86% <50.00%> (+<0.01%) ⬆️
python 69.62% <50.00%> (-0.01%) ⬇️
sbmlsuite-jax ?

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

Files with missing lines Coverage Δ
python/sdist/amici/sbml_import.py 81.41% <100.00%> (-12.52%) ⬇️
python/sdist/amici/pysb_import.py 94.47% <0.00%> (-0.30%) ⬇️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dweindl dweindl self-assigned this Oct 9, 2025
The import function `sbml2amici`, `pysb2amici`, and `antimony2amici` now
return an instance of the generated model class if called with `compile=True`
(default) avoiding the need for a separate call to `amici.import_model_module`.

Closes AMICI-dev#2970.
@dweindl dweindl marked this pull request as ready for review October 29, 2025 15:47
@dweindl dweindl requested a review from a team as a code owner October 29, 2025 15:47
@dweindl dweindl added this pull request to the merge queue Oct 31, 2025
Merged via the queue into AMICI-dev:main with commit 4e87d9c Oct 31, 2025
22 of 24 checks passed
@dweindl dweindl deleted the return_model branch October 31, 2025 13:01
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.

Return amici.Model from sbml2amici, pysb2amici

2 participants