Add SbmlModel.{to_antimony,to_sbml_str}

[dweindl]

Simplify conversion of SBML models to antimony and XML strings. Convenient in particular for debugging.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Project coverage is 74.59%. Comparing base (d5589c8) to head (805182a).

Files with missing lines	Patch %	Lines
petab/v1/models/sbml_model.py	84.61%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #371   +/-   ##
========================================
  Coverage    74.58%   74.59%           
========================================
  Files           58       58           
  Lines         6123     6133   +10     
  Branches      1061     1062    +1     
========================================
+ Hits          4567     4575    +8     
- Misses        1149     1150    +1     
- Partials       407      408    +1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[dilpath]

Looks fine to me. Should Model have to_str and from_str methods such that Problem.model_dump can serialize the model, too?

[matthiaskoenig]

Can the functions be from_antimony and to_antimony. We have other converters to convert to SBML formats which are also string based. I would be more explicit here.

[dilpath]

Can the functions be from_antimony and to_antimony. We have other converters to convert to SBML formats which are also string based. I would be more explicit here.

Agreed, and this is the current state in this PR too. My comment was rather about (de)serialization of SBML and antimony models, not interconversion.

Since the Model base class already has from_file and to_file, I guess to would suffice to have some from_str and to_str with the file contents as a string.

But this (de)serialization to string is not important to me -- just that it would make the pydantic model_dump in this other PR more comprehensive.
#372

[dweindl]

Should Model have to_str and from_str methods such that Problem.model_dump can serialize the model, too?

It could be convenient at times, but I don't see a strong need yet. At least, I wouldn't require its implementation for every model type.