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

Revert "Use CPU input for test_boost_from_prediction. (#6818)" #6858

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

trivialfis
Copy link
Member

This reverts commit 74f3a2f.

@trivialfis
Copy link
Member Author

@hcho3 Can we merge this one, and upgrade to 0.19 after its release? (instead of nightly)

@codecov-commenter
Copy link

codecov-commenter commented Apr 20, 2021

Codecov Report

Merging #6858 (db462ec) into master (b9a4f33) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6858      +/-   ##
==========================================
+ Coverage   81.94%   81.98%   +0.03%     
==========================================
  Files          13       13              
  Lines        3889     3902      +13     
==========================================
+ Hits         3187     3199      +12     
- Misses        702      703       +1     
Impacted Files Coverage Δ
python-package/xgboost/compat.py 72.38% <0.00%> (-0.17%) ⬇️
python-package/xgboost/core.py 83.19% <0.00%> (ø)
python-package/xgboost/dask.py 81.89% <0.00%> (+0.02%) ⬆️
python-package/xgboost/callback.py 81.86% <0.00%> (+0.04%) ⬆️
python-package/xgboost/sklearn.py 90.52% <0.00%> (+0.16%) ⬆️

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 b9a4f33...db462ec. Read the comment docs.

@trivialfis trivialfis merged commit bec2b4f into dmlc:master Apr 20, 2021
@trivialfis trivialfis deleted the try-revert-cpu-input branch April 20, 2021 06:54
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.

3 participants