Skip to content

DOC: fix np.unique release notes [skip cirrus]#26369

Merged
seberg merged 1 commit intonumpy:mainfrom
andyfaff:doc
May 2, 2024
Merged

DOC: fix np.unique release notes [skip cirrus]#26369
seberg merged 1 commit intonumpy:mainfrom
andyfaff:doc

Conversation

@andyfaff
Copy link
Member

@andyfaff andyfaff commented May 1, 2024

Fixes an incorrect link in the numpy2.0 release notes to #25553.

Noticed when the changes introduced in that PR broke a downstream package. Caused a bit of head scratching.

Perhaps the np.unique docstring could do with being amended with a 'changed in 2.0' note, even if the original behaviour was considered a bug. The docstring doesn't indicate that anything was different between 1.26.4 and 2.0, so when my code broke I thought it was a bug introduced into numpy2.

@seberg seberg added the 09 - Backport-Candidate PRs tagged should be backported label May 2, 2024
@seberg
Copy link
Member

seberg commented May 2, 2024

Thanks @andyfaff

@seberg seberg merged commit 5b7b91c into numpy:main May 2, 2024
@charris
Copy link
Member

charris commented May 2, 2024

This should have been made against the maintenance/2.0.x branch. The notes will be forward ported after the release and overwrite main.

EDIT: the notes in the branch have already diverged from main.

@seberg
Copy link
Member

seberg commented May 2, 2024

Ouch, sorry, I was wondering about that after merging, but wasn't sure then :/.

@charris
Copy link
Member

charris commented May 2, 2024

Not to worry, I can fix it up.

@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label May 2, 2024
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.

3 participants