Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

MNT: Numpy 2.0 removals from ndarray class#26762

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
timhoffm merged 1 commit intomatplotlib:mainfromksunden:np_20_ptp_newbyteorder
Sep 14, 2023

Conversation

ksunden
Copy link
Member

PR summary

xrefnumpy/numpy#24682

Only a handful of lines, andmostly in tests/relatively unused utility functions likergb_to_hsv, but still something that should be addressed.

PR checklist

xrefnumpy/numpy#24682Only a handful of lines, and _mostly_ in tests/relatively unused utility functions like rgb_to_hsv, but still something that should be addressed.
Copy link
Member

@QuLogicQuLogic left a comment

Choose a reason for hiding this comment

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

I'm not sure if it matters too much, but the upstream PR does do theview beforebyteswap.

@ksunden
Copy link
MemberAuthor

I'm pretty sure those are equivalent; byteswap will incur a copy (unless inplace=True) and view is just a view, but whether the copy comes before or after the view doesn't really matter.

@timhoffmtimhoffm added this to thev3.8.0 milestoneSep 14, 2023
@timhoffmtimhoffm merged commit4c73219 intomatplotlib:mainSep 14, 2023
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull requestSep 14, 2023
@ksundenksunden mentioned this pull requestSep 15, 2023
5 tasks
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@QuLogicQuLogicQuLogic approved these changes

@timhoffmtimhoffmtimhoffm approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
v3.8.0
Development

Successfully merging this pull request may close these issues.

3 participants
@ksunden@QuLogic@timhoffm

[8]ページ先頭

©2009-2025 Movatter.jp