-
Notifications
You must be signed in to change notification settings - Fork 175
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
feat(python): support merge_columns via Ray integration #2278
base: main
Are you sure you want to change the base?
Conversation
V2 does not support I will resume this work once v2 |
@eddyxu IMO I think we should use the same API as |
be03a92
to
a6a4837
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2278 +/- ##
=======================================
Coverage 79.82% 79.83%
=======================================
Files 202 202
Lines 55462 55462
Branches 55462 55462
=======================================
+ Hits 44274 44276 +2
Misses 8621 8621
+ Partials 2567 2565 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Expose
merge_columns
via Ray integration, and add one step to Commit the merge OP.