ICB-DCM / pyPESTO

python Parameter EStimation TOolbox
https://pypesto.readthedocs.io
BSD 3-Clause "New" or "Revised" License
216 stars 47 forks source link

Remove CESSOptimizer #1320

Closed dweindl closed 5 months ago

dweindl commented 6 months ago

SacessOptimizer is superior to CESSOptimizer and I wouldn't recommend using the latter. I think it's best to just remove it. No value, just extra maintenance effort. I don't think anybody was using it yet anyways, so this shouldn't actually be a breaking change for anybody. If not, please let me know.

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.30%. Comparing base (48ad07b) to head (5eddac0).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1320 +/- ## =========================================== - Coverage 84.40% 84.30% -0.10% =========================================== Files 153 152 -1 Lines 12533 12432 -101 =========================================== - Hits 10578 10481 -97 + Misses 1955 1951 -4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.