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

Update to XGBoost 1.4.0rc1 #3699

Merged
merged 1 commit into from
Apr 7, 2021
Merged

Conversation

hcho3
Copy link
Contributor

@hcho3 hcho3 commented Apr 1, 2021

Closes #3695

@hcho3 hcho3 requested a review from a team as a code owner April 1, 2021 18:09
@github-actions github-actions bot added the gpuCI gpuCI issue label Apr 1, 2021
Copy link
Member

@ajschmidt8 ajschmidt8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The integration version below will need to be updated before this PR can be approved.

https://github.com/rapidsai/integration/blob/branch-0.19/conda/recipes/versions.yaml#L9-L12

@JohnZed
Copy link
Contributor

JohnZed commented Apr 1, 2021

rerun tests

@JohnZed JohnZed added non-breaking Non-breaking change tests Unit testing for project labels Apr 1, 2021
@hcho3 hcho3 added the improvement Improvement / enhancement to an existing function label Apr 1, 2021
@hcho3
Copy link
Contributor Author

hcho3 commented Apr 2, 2021

Just filed rapidsai/integration#236

@hcho3
Copy link
Contributor Author

hcho3 commented Apr 2, 2021

Blocked by dmlc/treelite#259

@hcho3 hcho3 mentioned this pull request Apr 3, 2021
@hcho3
Copy link
Contributor Author

hcho3 commented Apr 3, 2021

Depends on #3708

@JohnZed JohnZed added this to PR-WIP in v0.19 Release via automation Apr 5, 2021
rapids-bot bot pushed a commit that referenced this pull request Apr 6, 2021
This unblocks #3699, as the current version of Treelite (1.0.0) cannot handle model JSON files generated by XGBoost 1.4.0. The new Treelite release has the following improvements:

* Support JSON model files from XGBoost 1.4.0 
* Support DART models

I'll also file a pull request to `integration`.

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - John Zedlewski (https://github.com/JohnZed)
  - Ray Douglass (https://github.com/raydouglass)

URL: #3708
@dantegd
Copy link
Member

dantegd commented Apr 6, 2021

rerun tests

@codecov-io
Copy link

Codecov Report

Merging #3699 (de129fc) into branch-0.19 (fd9ec89) will increase coverage by 2.31%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.19    #3699      +/-   ##
===============================================
+ Coverage        80.70%   83.02%   +2.31%     
===============================================
  Files              227      227              
  Lines            17615    17712      +97     
===============================================
+ Hits             14217    14706     +489     
+ Misses            3398     3006     -392     
Flag Coverage Δ
dask 45.62% <100.00%> (+0.63%) ⬆️
non-dask 75.11% <100.00%> (+2.19%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...cuml/_thirdparty/sklearn/preprocessing/__init__.py 100.00% <ø> (ø)
...on/cuml/_thirdparty/sklearn/preprocessing/_data.py 64.40% <ø> (+1.29%) ⬆️
...hirdparty/sklearn/preprocessing/_discretization.py 83.33% <ø> (-0.88%) ⬇️
...l/_thirdparty/sklearn/preprocessing/_imputation.py 85.54% <ø> (+22.74%) ⬆️
...cuml/_thirdparty/sklearn/utils/skl_dependencies.py 80.00% <ø> (+26.07%) ⬆️
python/cuml/cluster/__init__.py 100.00% <ø> (ø)
python/cuml/cluster/agglomerative.pyx 96.47% <ø> (ø)
python/cuml/cluster/dbscan.pyx 98.23% <ø> (-1.77%) ⬇️
python/cuml/cluster/kmeans.pyx 92.03% <ø> (+0.07%) ⬆️
python/cuml/common/array_sparse.py 96.29% <ø> (+1.95%) ⬆️
... and 127 more

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 7cfb577...de129fc. Read the comment docs.

@JohnZed
Copy link
Contributor

JohnZed commented Apr 6, 2021

@gpucibot merge

v0.19 Release automation moved this from PR-WIP to PR-Reviewer approved Apr 7, 2021
@rapids-bot rapids-bot bot merged commit 25f7b5a into rapidsai:branch-0.19 Apr 7, 2021
v0.19 Release automation moved this from PR-Reviewer approved to Done Apr 7, 2021
@hcho3 hcho3 deleted the new_xgboost branch April 7, 2021 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gpuCI gpuCI issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change tests Unit testing for project
Projects
No open projects
v0.19 Release
  
Done
Development

Successfully merging this pull request may close these issues.

[FEA] Update CI to new XGBoost 1.4.0rc package
5 participants