search

ICB-DCM / pyPESTO

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

Update `test_thermodynamic_integration` #1385

Closed PaulJonasJost closed 1 month ago

PaulJonasJost commented 2 months ago

adjusted test_thermodynamic_integration, by lowering the number of chains by factor of 5 and only running 10 chains, as the comparison of more chains is not something that we should have to test imo.

In local tests (Mac) it went from ~200sek to 20

codecov-commenter commented 2 months ago

Codecov Report

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

Project coverage is 84.48%. Comparing base (d5fde03) to head (14af5f2).

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1385 +/- ## =========================================== - Coverage 84.51% 84.48% -0.03% =========================================== Files 157 157 Lines 12956 12959 +3 =========================================== - Hits 10950 10949 -1 - Misses 2006 2010 +4 ```

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

arrjon commented 2 months ago

True, this is not neceassary as a test. It was included to check convergence of the evidence estimation. Okay for me to remove it to speed up testing time.