-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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
REF: back DatetimeBlock, TimedeltaBlock by DTA/TDA #40456
Conversation
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.
also pls rebase
rebased + green. between this and #40477 we're within striking distance of ripping out a couple more Block subclasses |
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.
this looks pretty good, just a small comment
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.
just a question. what's failing on master?
@@ -9541,6 +9541,9 @@ def func(values: np.ndarray): | |||
|
|||
def blk_func(values, axis=1): | |||
if isinstance(values, ExtensionArray): | |||
if values.ndim == 2: |
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.
isn't this just axis=axis
?
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.
sure. next pass
havent looked this morning, but last night it was something td64/dt64-casting related merged into npdev |
@@ -1172,6 +1174,7 @@ def _interpolate_with_fill( | |||
limit_area=limit_area, | |||
) | |||
|
|||
values = maybe_coerce_values(values) |
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.
prob can have maybe_coerce_values do downcasting (maybe)
Broken off from #40149. Like #40149, this backs DTBlock/TDBlock by (sometimes 2D) DTA/TDA. Unlike #40149, this does not make the DTA backing DatetimeTZBlock sometimes-2D.