Skip to content

Update Treelite to 4.7.1 - #8522

Open
chyunsu3 wants to merge 3 commits into
NVIDIA:mainfrom
chyunsu3:update_treelite
Open

Update Treelite to 4.7.1#8522
chyunsu3 wants to merge 3 commits into
NVIDIA:mainfrom
chyunsu3:update_treelite

Conversation

@chyunsu3

Copy link
Copy Markdown
Contributor

Closes #8507, #8425
Requires conda-forge/treelite-feedstock#99

The new release of Treelite incoporates the following bug fixes:

@chyunsu3
chyunsu3 requested review from a team as code owners August 25, 2026 22:03
@chyunsu3
chyunsu3 requested a review from bdice August 25, 2026 22:03
@github-actions github-actions Bot added conda conda issue Cython / Python Cython or Python issue CMake CUDA/C++ labels Aug 25, 2026
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: d24c9607-b4c8-4329-ac76-6c3869a1b273

📥 Commits

Reviewing files that changed from the base of the PR and between c17776b and 7c5c060.

📒 Files selected for processing (9)
  • conda/environments/all_cuda-129_arch-aarch64.yaml
  • conda/environments/all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-133_arch-aarch64.yaml
  • conda/environments/all_cuda-133_arch-x86_64.yaml
  • conda/recipes/cuml/conda_build_config.yaml
  • conda/recipes/libcuml/conda_build_config.yaml
  • cpp/cmake/thirdparty/get_treelite.cmake
  • dependencies.yaml
  • python/cuml/pyproject.toml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the supported Treelite version to 4.7.1 or later, while retaining compatibility with versions below 5.0.0.
    • Applied the updated requirement across runtime, build, and installation configurations.
    • Refreshed the bundled Treelite source reference.

Walkthrough

Changes

The PR updates Treelite requirements from 4.7.0 to 4.7.1 across CMake, project metadata, Conda recipes, and CUDA environment files. The CMake configuration also updates the pinned Treelite commit and copyright notice.

Treelite dependency update

Layer / File(s) Summary
Treelite source pin
cpp/cmake/thirdparty/get_treelite.cmake
The configured Treelite version changes to 4.7.1 with a new pinned commit. The copyright notice covers 2021–2026 and includes affiliates.
Project dependency constraints
dependencies.yaml, python/cuml/pyproject.toml
Build and runtime requirements now require Treelite 4.7.1 or newer and retain the <5.0.0 upper bound.
Packaging and environment constraints
conda/recipes/*/conda_build_config.yaml, conda/environments/all_cuda-*
Conda recipes and CUDA environment definitions now require Treelite 4.7.1 and retain the <5.0.0 upper bound.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 7c5c0

This dependency update is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Suggested reviewers: bdice, jameslamb

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the main change: updating Treelite to version 4.7.1.
Description check ✅ Passed The description is related to the changes. It identifies the Treelite release, relevant bug fixes, linked issues, and the required conda-forge dependency update.
Linked Issues check ✅ Passed The PR updates Treelite to 4.7.1 across Conda, CMake, build, and Python dependency declarations. The stated scikit-learn exporter fix is relevant to the RandomForest ONNX failures tracked in [#8507].
Out of Scope Changes check ✅ Passed The changes are limited to updating Treelite dependency constraints, the pinned Treelite version, and its copyright year range. No unrelated code changes are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (9 skipped: 9 unsupported.)

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@jameslamb

Copy link
Copy Markdown
Member

I updated to latest main to start a new CI run for you, I know there have been some fixes merged in recent days.

@jameslamb
jameslamb removed the request for review from bdice August 28, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake conda conda issue CUDA/C++ Cython / Python Cython or Python 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.

[CI] Tracking RandomForest Failures

4 participants