-
Notifications
You must be signed in to change notification settings - Fork 891
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
Clean up _copy_type_metadata #11156
Clean up _copy_type_metadata #11156
Conversation
Codecov Report
@@ Coverage Diff @@
## branch-22.08 #11156 +/- ##
===============================================
Coverage ? 86.32%
===============================================
Files ? 144
Lines ? 22742
Branches ? 0
===============================================
Hits ? 19631
Misses ? 3111
Partials ? 0 Continue to review full report at Codecov.
|
Co-authored-by: Charles Blackmon-Luca <20627856+charlesbluca@users.noreply.github.com>
Co-authored-by: Charles Blackmon-Luca <20627856+charlesbluca@users.noreply.github.com>
rerun tests |
rerun tests |
1 similar comment
rerun tests |
@gpucibot merge |
1 similar comment
@gpucibot merge |
This PR moves index-related logic from
Frame._copy_type_metadata
into theIndexedFrame
override. It also removes all reference to the index parameter from the corresponding methods of various index types.