Skip to content

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented Dec 5, 2024

Simplify creating a PEtab SbmlModel from antimony files / strings.

Replace simplesbml by antimony in tests.

@codecov-commenter
Copy link

codecov-commenter commented Dec 5, 2024

Codecov Report

Attention: Patch coverage is 56.25000% with 14 lines in your changes missing coverage. Please review.

Project coverage is 74.24%. Comparing base (980926f) to head (e8a57b1).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
petab/v1/models/sbml_model.py 56.25% 8 Missing and 6 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #331      +/-   ##
===========================================
- Coverage    74.36%   74.24%   -0.13%     
===========================================
  Files           53       53              
  Lines         5138     5168      +30     
  Branches       903      910       +7     
===========================================
+ Hits          3821     3837      +16     
- Misses         977      984       +7     
- Partials       340      347       +7     

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

@dweindl dweindl changed the title Add ? SbmlModel.from_antimony` Add SbmlModel.from_antimony Dec 5, 2024
@dweindl dweindl marked this pull request as ready for review December 5, 2024 09:10
@dweindl dweindl requested a review from a team as a code owner December 5, 2024 09:10
@dweindl dweindl self-assigned this Dec 5, 2024
@dweindl dweindl force-pushed the ant branch 2 times, most recently from 0839523 to af542d9 Compare December 5, 2024 14:50
Copy link
Member

@dilpath dilpath left a comment

Choose a reason for hiding this comment

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

Nice, add support for antimony to the docs and spec in v2?

Comment on lines 361 to 370
model = SbmlModel.from_antimony(
"fixedParameter1 = 0.5\n"
"fixedParameter2 = 1.0\n"
"dynamicParameter1 = 0.0\n"
"observableParameter1_obs1 = 0.0\n"
"observableParameter2_obs1 = 0.0\n"
"observableParameter1_obs2 = 0.0"
)
Copy link
Member

Choose a reason for hiding this comment

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

Use """ instead of \n? Or does antimony interpret indentation?

Copy link
Member Author

Choose a reason for hiding this comment

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

Makes sense. No, it doesn't.

@dweindl
Copy link
Member Author

dweindl commented Dec 9, 2024

Nice, add support for antimony to the docs and spec in v2?

Undecided. Convenient, but not really a strong enough reason to deviate from SBML and make things less portable.

Simplify creating a PEtab SbmlModel from antimony files or strings.

Replace simplesbml by antimony in tests.
@dweindl dweindl merged commit 45a3371 into PEtab-dev:develop Dec 9, 2024
7 checks passed
@dweindl dweindl deleted the ant branch December 9, 2024 14:22
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.

3 participants