Skip to content

TYP: fix ravel_multi_index false rejections #29184

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 1 commit into from
Jun 11, 2025

Conversation

jorenham
Copy link
Member

Valid uses like in

# Compute the sample indices in the flattened histogram matrix.
# This raises an error if the array is too large.
xy = np.ravel_multi_index(Ncount, nbin)
will now be accepted by static type-checkers.

@charris charris merged commit 3d1cdb4 into numpy:main Jun 11, 2025
77 checks passed
@charris
Copy link
Member

charris commented Jun 11, 2025

Thanks Joren.

@jorenham jorenham added the 09 - Backport-Candidate PRs tagged should be backported label Jun 11, 2025
@jorenham jorenham deleted the typing/ravel_multi_index branch June 11, 2025 20:59
charris pushed a commit to charris/numpy that referenced this pull request Jun 12, 2025
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants