-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Remove FrozenNDArray #29335
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
Remove FrozenNDArray #29335
Conversation
@jbrockmendel this is partially a culprit for the circular references you would get when trying to import
But PandasObject is not defined by the time step 4 comes around, hence the circular import In a follow up could maybe question if FrozenList really needs to inherit from PandasObject, but will leave that to a separate discussion |
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.
I really thought these changes would have been trivial, but I am somewhat confused by the many test failures that this removal triggered...
Not having looked at the test failures yet, two thoughts: 1) im on board for removing this if we can and 2) not needing to subclass PandasObject is also plausible |
Closing to clear queue but plan on coming back to this |
This seems to have caused the problem described here: which I'm now also experiencing. I dumped a MultiIndex dataframe containing ndarrays to a pickle on disk under pandas 0.25.x in Python 3.6, and now I'm getting:
when trying to load it in pandas 1.0.3 (still on Python 3.6). Any suggestions/workarounds? Should I instead open up a new issue? This solved issue seems related, but is for Python 2.7: This comment in the original rationale for getting rid of |
Can you open a dedicated issue for that? I think need to define some pickle combat for that particular use case but can discuss on its own thread |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff