Skip to content

Conversation

@loicdiridollou
Copy link
Member

  • Closes #xxxx (Replace xxxx with the Github issue number)
  • Tests added (Please use assert_type() to assert the type of any return value)
  • If I used AI to develop this pull request, I prompted it to follow AGENTS.md.

@loicdiridollou
Copy link
Member Author

This is a subset of the changes proposed in #1568 (which did not raise any requests), added tests for some of the types.


if sys.version_info >= (3, 11):
def __init__(
self, values: AnyArrayLike, mask: np_1darray_bool, copy: bool = False
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
self, values: AnyArrayLike, mask: np_1darray_bool, copy: bool = False
self, values: AnyArrayLike, mask: np_ndarray_bool, copy: bool = False

Wouldn't nd instead of 1d work in all versions?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Arghhh I see the issue, I tried np_1darray_bool last time and not np_ndarray_bool

Copy link
Contributor

@cmp0xff cmp0xff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you @loicdiridollou !

@cmp0xff cmp0xff merged commit cc7f48f into pandas-dev:main Dec 23, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants