Skip to content

BUG: Fix ExtensionArray binary op protocol #61990

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 16 commits into from
Aug 14, 2025

Conversation

tisjayy
Copy link
Contributor

@tisjayy tisjayy commented Jul 29, 2025

@tisjayy tisjayy closed this Jul 29, 2025
@tisjayy tisjayy reopened this Jul 29, 2025
@tisjayy tisjayy force-pushed the fix-ea-non-1d-ops branch from f9cb022 to 9496569 Compare July 29, 2025 05:21
@tisjayy

This comment was marked as resolved.

@tisjayy

This comment was marked as resolved.

@tisjayy tisjayy closed this Jul 29, 2025
@tisjayy tisjayy reopened this Jul 29, 2025
@tisjayy tisjayy closed this Jul 29, 2025
@tisjayy tisjayy reopened this Jul 29, 2025
@tisjayy

This comment was marked as resolved.

@tisjayy tisjayy changed the title BUG: NotImplemented for non-1D BooleanArray ops; add numeric EA shape error tests BUG: Fix ExtensionArray binary op protocol, return NotImplemented instead of raising NotImplementedError Jul 29, 2025
@tisjayy tisjayy changed the title BUG: Fix ExtensionArray binary op protocol, return NotImplemented instead of raising NotImplementedError BUG: Fix ExtensionArray binary op protocol Jul 29, 2025
@mroeschke mroeschke requested a review from jbrockmendel July 30, 2025 16:11
@mroeschke mroeschke added Numeric Operations Arithmetic, Comparison, and Logical operations ExtensionArray Extending pandas with custom dtypes or arrays. labels Jul 30, 2025
@tisjayy

This comment was marked as resolved.

@tisjayy

This comment was marked as resolved.

@tisjayy tisjayy requested review from arthurlw and jbrockmendel July 31, 2025 04:39
@tisjayy
Copy link
Contributor Author

tisjayy commented Aug 6, 2025

@jbrockmendel can you please review at your convenience? thanks!

@jbrockmendel
Copy link
Member

I'm out of town for a few days, will look at this when i get back.

@tisjayy
Copy link
Contributor Author

tisjayy commented Aug 6, 2025

pre-commit.ci autofix

@tisjayy
Copy link
Contributor Author

tisjayy commented Aug 8, 2025

@jbrockmendel changes you requested have been made. please take a look

@tisjayy
Copy link
Contributor Author

tisjayy commented Aug 13, 2025

Hi @mroeschke , I think this pr is ready, would you mind reviewing it if you get a chance?

@jbrockmendel jbrockmendel merged commit 23aae9f into pandas-dev:main Aug 14, 2025
38 checks passed
@jbrockmendel
Copy link
Member

thanks @tisjayy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ExtensionArray Extending pandas with custom dtypes or arrays. Numeric Operations Arithmetic, Comparison, and Logical operations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Operations not implemented for non-1D ExtensionArrays
4 participants