Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the pending future from ensemble builder to the history #1046

Merged
merged 1 commit into from Jan 5, 2021

Conversation

franchuterivera
Copy link
Contributor

In case there are pending ensemble iterations of ensemble builder when smac is complete, we wait for the ensemble future to finish.

We should add this ensemble iteration to the ensemble history as it can produce a new better performing model.

@codecov
Copy link

codecov bot commented Dec 30, 2020

Codecov Report

Merging #1046 (c6dd8c3) into master (f255a0f) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1046      +/-   ##
==========================================
- Coverage   85.49%   85.45%   -0.04%     
==========================================
  Files         127      127              
  Lines       10139    10178      +39     
==========================================
+ Hits         8668     8698      +30     
- Misses       1471     1480       +9     
Impacted Files Coverage Δ
autosklearn/automl.py 84.69% <100.00%> (-0.33%) ⬇️
autosklearn/util/backend.py 74.73% <0.00%> (-1.43%) ⬇️
...ature_preprocessing/select_rates_classification.py 85.91% <0.00%> (ø)
autosklearn/util/logging_.py 88.09% <0.00%> (+0.79%) ⬆️
...ine/components/classification/gradient_boosting.py 92.17% <0.00%> (+0.86%) ⬆️
autosklearn/util/dependencies.py 93.22% <0.00%> (+1.69%) ⬆️

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 f255a0f...c6dd8c3. Read the comment docs.

@mfeurer mfeurer changed the base branch from master to development January 5, 2021 08:29
@mfeurer mfeurer merged commit 64b8b9c into automl:development Jan 5, 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.

None yet

2 participants