Skip to content
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

DEPR: .as_blocks and .blocks #17302

Closed
jreback opened this issue Aug 21, 2017 · 3 comments · Fixed by #17656
Closed

DEPR: .as_blocks and .blocks #17302

jreback opened this issue Aug 21, 2017 · 3 comments · Fixed by #17656
Labels
API Design Deprecate Functionality to remove in pandas
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Aug 21, 2017

These shouldn't be public methods. Deprecate and make private (lead with _) for now.

@jreback jreback added API Design Deprecate Functionality to remove in pandas labels Aug 21, 2017
@jreback jreback added this to the 0.21.0 milestone Aug 21, 2017
@jorisvandenbossche
Copy link
Member

jorisvandenbossche commented Aug 22, 2017

+ 1

(as long as we keep the private access like df._data.blocks stable, but I don't think we have any plans or incentives to change that in 1.x)

Is it even necessary to make them private? Eg internally we have exact one usage of as_blocks, which I suppose can easily be changes to go through _data

cc @pandas-dev/pandas-core

@jbrockmendel
Copy link
Member

+1

Speaking of _data, is there (or could there be) an implicit convention that foo._data should always (with the possible exception of Index._data) point at a BlockManager?

@jreback
Copy link
Contributor Author

jreback commented Aug 23, 2017

Speaking of _data, is there (or could there be) an implicit convention that foo._data should always

its just the internal impl. For DataFrame/Series its a BlockManager. For Index its an ndarry (or another array-like type, e.g. Categorical).

jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit to jreback/pandas that referenced this issue Sep 24, 2017
jreback added a commit that referenced this issue Sep 25, 2017
alanbato pushed a commit to alanbato/pandas that referenced this issue Nov 10, 2017
No-Stream pushed a commit to No-Stream/pandas that referenced this issue Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Design Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants