Skip to content

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

Merged
merged 5 commits into from
Jun 26, 2019

Conversation

jbrockmendel
Copy link
Member

Also some unrecheable try_cast code.

@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #27043 into master will increase coverage by 1.39%.
The diff coverage is 100%.

Impacted file tree graph

@@            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
Flag Coverage Δ
#multiple 90.68% <100%> (+0.03%) ⬆️
#single 41.88% <0%> (?)
Impacted Files Coverage Δ
pandas/core/internals/managers.py 96.86% <ø> (+0.76%) ⬆️
pandas/core/panel.py 17.8% <ø> (+0.19%) ⬆️
pandas/core/internals/blocks.py 95.14% <100%> (+0.52%) ⬆️
pandas/util/testing.py 90.94% <0%> (+0.1%) ⬆️
pandas/core/arrays/categorical.py 95.92% <0%> (+0.12%) ⬆️
pandas/core/indexes/datetimes.py 96.37% <0%> (+0.16%) ⬆️
pandas/io/formats/printing.py 86.72% <0%> (+0.94%) ⬆️
pandas/io/clipboard/clipboards.py 34.78% <0%> (+2.89%) ⬆️
pandas/core/computation/expr.py 97.8% <0%> (+3.02%) ⬆️
pandas/core/computation/common.py 89.47% <0%> (+5.26%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9182df...9432501. Read the comment docs.

@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #27043 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            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
Flag Coverage Δ
#multiple 90.68% <100%> (+0.03%) ⬆️
#single 41.85% <0%> (-0.06%) ⬇️
Impacted Files Coverage Δ
pandas/core/internals/managers.py 96.86% <ø> (+0.76%) ⬆️
pandas/core/panel.py 17.8% <ø> (+0.19%) ⬆️
pandas/core/internals/blocks.py 95.14% <100%> (+0.52%) ⬆️
pandas/io/gbq.py 88.88% <0%> (-11.12%) ⬇️
pandas/core/frame.py 96.89% <0%> (-0.12%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7f1d69...8fc4307. Read the comment docs.

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")
Copy link
Contributor

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.

Copy link
Member Author

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.

@jbrockmendel
Copy link
Member Author

CI fail looks like a conda problem

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Jun 26, 2019 via email

@gfyoung gfyoung added Clean Internals Related to non-user accessible pandas implementation labels Jun 26, 2019
@jreback jreback added this to the 0.25.0 milestone Jun 26, 2019
@jreback
Copy link
Contributor

jreback commented Jun 26, 2019

lgtm, pls merge master and ping on green.

@jbrockmendel
Copy link
Member Author

ping

@TomAugspurger TomAugspurger merged commit ff50b46 into pandas-dev:master Jun 26, 2019
@jbrockmendel jbrockmendel deleted the xs branch April 5, 2020 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean Internals Related to non-user accessible pandas implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants