ENH: Create ArrowDtype #46774
ENH: Create ArrowDtype #46774
Conversation
|
discussed on the March call, id like to migrate from BaseMaskedDtype/NumericDtype etc to just MaskedDtype. Similarly id suggest here just ArrowDtype. |
|
By "just ArrowDtype" you mean have |
Right, without all the subclasses |
mroeschke
changed the title
pandas/core/arrays/arrow/base.py
Outdated
| from pandas.core.arrays.arrow import ArrowExtensionArray | ||
|
|
||
|
|
||
| class ArrowDtype(StorageExtensionDtype): |
There was a problem hiding this comment.
why are the dtypes here? shouldn't these be in pandas/core/dtypes ?
There was a problem hiding this comment.
I was thinking of consolidating all arrow related functionality under /arrays/arrow/. Thoughts @jbrockmendel
There was a problem hiding this comment.
no objection. Might be worth keeping something in core.dtypes.dtypes for dependency-structure purposes
There was a problem hiding this comment.
ok fair, looking to keep these separate as much as possible, maybe rename this to _dtype.py at some point.
can we rename NumericDtype -> NumericMaskedDtype (alt maybe NumericNumpyDtype and rename BaseMaskedDtype) |
This PR will just add the base ArrowDtype for now |
Not activated yet. Opening to get any feedback about the approach cc @jbrockmendel