-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
CLN: Remove no-longer-used BlockManager.xs #27043
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #27043 +/- ##
==========================================
+ Coverage 90.64% 92.04% +1.39%
==========================================
Files 180 180
Lines 50754 50713 -41
==========================================
+ Hits 46005 46677 +672
+ Misses 4749 4036 -713
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #27043 +/- ##
==========================================
+ Coverage 92.01% 92.04% +0.03%
==========================================
Files 180 180
Lines 50754 50713 -41
==========================================
- Hits 46699 46677 -22
+ Misses 4055 4036 -19
Continue to review full report at Codecov.
|
copy = new_data.is_mixed_type | ||
result._set_is_copy(self, copy=copy) | ||
return result | ||
raise NotImplementedError("Panel is removed in pandas 0.25.0") |
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.
Is this code hit? I'm confused by the reference to panel.
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.
We've removed most of the Panel tests, but not yet all of Panel itself. So this is just explicitly-disabling a method that would call BlockManager.xs, since we're on the road towards removing the class anyway.
CI fail looks like a conda problem |
#26595 fixes it.
…On Tue, Jun 25, 2019 at 10:36 PM jbrockmendel ***@***.***> wrote:
CI fail looks like a conda problem
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#27043?email_source=notifications&email_token=AAKAOIXBUYYTBAJZ5CLEXNDP4LP5XA5CNFSM4H3MG4HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSHAWY#issuecomment-505704539>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKAOIQYOAMY7S4DA6K3ALDP4LP5XANCNFSM4H3MG4HA>
.
|
lgtm, pls merge master and ping on green. |
ping |
Also some unrecheable try_cast code.