Skip to content

Conversation

eddiebergman
Copy link
Contributor

Some tests were failing due to timeout's. Upon looking at the run time of other tests, they seem to take between 40-45 minutes.

This increase in test time is likely due to tests introduced from recent PR's. The variability of timing is likely outside of our control but a 15 minute buffer should be enough.

Future work might be needed in reducing the time needed for test completion but for now I think this is an acceptable solution.

@eddiebergman eddiebergman requested a review from mfeurer September 8, 2021 14:34
@codecov
Copy link

codecov bot commented Sep 8, 2021

Codecov Report

Merging #1239 (6c2f0d0) into development (0f03651) will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #1239      +/-   ##
===============================================
+ Coverage        88.22%   88.29%   +0.07%     
===============================================
  Files              139      139              
  Lines            11034    11034              
===============================================
+ Hits              9735     9743       +8     
+ Misses            1299     1291       -8     
Impacted Files Coverage Δ
autosklearn/util/logging_.py 88.96% <0.00%> (+0.68%) ⬆️
...learn/pipeline/components/regression/libsvm_svr.py 91.46% <0.00%> (+1.21%) ⬆️
...ipeline/components/regression/gradient_boosting.py 93.26% <0.00%> (+1.92%) ⬆️
...mponents/feature_preprocessing/nystroem_sampler.py 91.17% <0.00%> (+5.88%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f03651...6c2f0d0. Read the comment docs.

@mfeurer mfeurer merged commit f40beda into development Sep 10, 2021
@mfeurer mfeurer deleted the extend_tests_timeouts branch September 10, 2021 12:54
github-actions bot pushed a commit that referenced this pull request Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants