Quantco / metalearners

MetaLearners for CATE estimation
https://metalearners.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
34 stars 4 forks source link

Increase time limit grid search example #68

Closed FrancescMartiEscofetQC closed 4 months ago

FrancescMartiEscofetQC commented 4 months ago

In some commits readthedocs failed, see here or here. In both cases pushing an empty commit to trigger rebuilding the docs solved it. We can't check the reason why it failed because it's dumped on a file but as it is not a consistent failure my hypothesis is that it is due to reaching the maximum execution time. This PR increases the timeout for the grid search example which is the one that failed.

PD: I opened an issue to be able to show the traceback in case the execution of a notebook fails. If that is fixed we could set nb_execution_show_tb = True in conf.py and we would see the error on the output of ReadtheDocs.

Checklist

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 94.88%. Comparing base (851c1ca) to head (3745318). Report is 21 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #68 +/- ## ======================================= Coverage 94.88% 94.88% ======================================= Files 15 15 Lines 1544 1544 ======================================= Hits 1465 1465 Misses 79 79 ```

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