-
Notifications
You must be signed in to change notification settings - Fork 513
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
Migrate to Treelite 4.0 #5701
Migrate to Treelite 4.0 #5701
Conversation
rerun tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the changes I'm seeing here! They really show how many improvements Treelite 4.0 brings to the table. I posted one question before approval, but overall things look great.
Given how significant this change is, we might want to do a test of building the Triton backend for FIL either against this branch or immediately after it is merged, but I'm guessing there will be no or minimal issues.
dcf7ae0
to
0eff650
Compare
a07e299
to
8838cb6
Compare
/merge |
1ee99d7
into
rapidsai:branch-24.02
Test cuML with Treelite 4.0 RC1.
This PR should be merged only when Treelite 4.0 release is finalized.
Supersedes #5650