PEtab-dev / libpetab-python

Python package for working with PEtab files
https://libpetab-python.readthedocs.io
MIT License
14 stars 5 forks source link

Fix sympy symbol name clashes #202

Closed dweindl closed 1 year ago

dweindl commented 1 year ago

Make sure symbols like N, beta, ... are used as scalar symbols, and not as sympy functions with the same name.

See also https://github.com/ICB-DCM/pyPESTO/issues/1048

codecov-commenter commented 1 year ago

Codecov Report

Merging #202 (dda5034) into develop (bd0176f) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #202      +/-   ##
===========================================
+ Coverage    76.28%   76.31%   +0.02%     
===========================================
  Files           34       34              
  Lines         3159     3162       +3     
  Branches       765      765              
===========================================
+ Hits          2410     2413       +3     
  Misses         552      552              
  Partials       197      197              
Impacted Files Coverage Δ
petab/calculate.py 89.92% <100.00%> (+0.07%) :arrow_up:
petab/lint.py 72.28% <100.00%> (+0.07%) :arrow_up:
petab/observables.py 97.29% <100.00%> (+0.03%) :arrow_up: