pyPESTO icon indicating copy to clipboard operation
pyPESTO copied to clipboard

Update `test_thermodynamic_integration`

Open PaulJonasJost opened this issue 1 year ago • 2 comments

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

PaulJonasJost avatar May 03 '24 08:05 PaulJonasJost

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
@@             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.

codecov-commenter avatar May 03 '24 08:05 codecov-commenter

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.

arrjon avatar May 06 '24 09:05 arrjon