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

Upgrade to Treelite 1.1.0 #3708

Merged
merged 5 commits into from
Apr 6, 2021
Merged

Conversation

hcho3
Copy link
Contributor

@hcho3 hcho3 commented Apr 3, 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.

@hcho3 hcho3 requested a review from a team as a code owner April 3, 2021 02:39
@github-actions github-actions bot added conda conda issue gpuCI gpuCI issue labels Apr 3, 2021
ci/gpu/build.sh Outdated Show resolved Hide resolved
@hcho3 hcho3 added breaking Breaking change improvement Improvement / enhancement to an existing function non-breaking Non-breaking change and removed breaking Breaking change labels Apr 3, 2021
@hcho3 hcho3 mentioned this pull request Apr 3, 2021
@hcho3
Copy link
Contributor Author

hcho3 commented Apr 3, 2021

Rerun tests

1 similar comment
@hcho3
Copy link
Contributor Author

hcho3 commented Apr 3, 2021

Rerun tests

ci/gpu/build.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@JohnZed JohnZed left a comment

Choose a reason for hiding this comment

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

Didn't click approve yet because I know you're still going to update after integration, but looks good to me.

@hcho3
Copy link
Contributor Author

hcho3 commented Apr 5, 2021

rapidsai/integration#240 is now merged, so updating this PR one last time.

@github-actions github-actions bot removed the gpuCI gpuCI issue label Apr 5, 2021
@codecov-io
Copy link

Codecov Report

Merging #3708 (62582b6) into branch-0.19 (fd9ec89) will increase coverage by 2.21%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.19    #3708      +/-   ##
===============================================
+ Coverage        80.70%   82.92%   +2.21%     
===============================================
  Files              227      227              
  Lines            17615    17591      -24     
===============================================
+ Hits             14217    14587     +370     
+ Misses            3398     3004     -394     
Flag Coverage Δ
dask 45.31% <ø> (+0.32%) ⬆️
non-dask 74.95% <ø> (+2.03%) ⬆️

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

Impacted Files Coverage Δ
python/cuml/__init__.py 95.58% <ø> (+0.20%) ⬆️
...cuml/_thirdparty/sklearn/preprocessing/__init__.py 100.00% <ø> (ø)
...on/cuml/_thirdparty/sklearn/preprocessing/_data.py 64.27% <ø> (+1.16%) ⬆️
...hirdparty/sklearn/preprocessing/_discretization.py 83.33% <ø> (-0.88%) ⬇️
...l/_thirdparty/sklearn/preprocessing/_imputation.py 85.54% <ø> (+22.74%) ⬆️
python/cuml/_thirdparty/sklearn/utils/extmath.py 56.89% <ø> (ø)
...cuml/_thirdparty/sklearn/utils/skl_dependencies.py 79.54% <ø> (+25.62%) ⬆️
...ython/cuml/_thirdparty/sklearn/utils/validation.py 18.41% <ø> (-4.04%) ⬇️
python/cuml/cluster/__init__.py 100.00% <ø> (ø)
python/cuml/cluster/agglomerative.pyx 96.47% <ø> (ø)
... and 152 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 1554f14...62582b6. Read the comment docs.

@hcho3 hcho3 added the 5 - Ready to Merge Testing and reviews complete, ready to merge label Apr 5, 2021
@JohnZed
Copy link
Contributor

JohnZed commented Apr 6, 2021

@gpucibot merge

@dantegd
Copy link
Member

dantegd commented Apr 6, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 5475bcd into rapidsai:branch-0.19 Apr 6, 2021
@hcho3 hcho3 deleted the upgrade_treelite branch April 6, 2021 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to Merge Testing and reviews complete, ready to merge conda conda issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants