Skip to content

DEP: Finalize MachAr and machar deprecations - #22638

Merged
charris merged 1 commit into
numpy:mainfrom
seberg:machar-depr
Nov 22, 2022
Merged

charris merged 1 commit into
numpy:mainfrom
seberg:machar-depr

Conversation

@seberg

@seberg seberg commented Nov 21, 2022

Copy link
Copy Markdown
Member

This removes the attributes on finfo and the "public" module. It also deprecates np.core.MachAr. We should be able to get away with just deleting it, but there seems little reason to not just deprecate it for now.

Comment thread numpy/core/__init__.py Outdated
def __getattr__(name):
# Deprecated 2021-10-20, NumPy 1.22
if name == "machar":
# Deprecated 2022-11-21, NumPy 1.24.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Needs to be NumPY 1.25 now :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Hah, fine, maybe makes life easier for Numba (or it doesn't because for them the other one matters)...

@charris charris added this to the 1.25.0 release milestone Nov 22, 2022
This removes the attributes on finfo and the "public" module.  It also
deprecates `np.core.MachAr`.  We should be able to get away with just
deleting it, but there seems little reason to not just deprecate it for now.
@charris

charris commented Nov 22, 2022

Copy link
Copy Markdown
Member

Not sure what is going on with circleci here, but it looks related :)

@seberg

seberg commented Nov 22, 2022

Copy link
Copy Markdown
Member Author

The FutureWarnings are new (annoying but the doc build doesn't fail on them).

I think either the job ran at a weird time with respect to branching, or the update to 1.25 development, is missing to indlude the new release notes.

@seberg

seberg commented Nov 22, 2022

Copy link
Copy Markdown
Member Author

Ran through now, I hope it was just weird timing or caching with branching and updating that or so...

@charris

charris commented Nov 22, 2022

Copy link
Copy Markdown
Member

It is showing up in other PRs, so not a timing oddity. The release notes are correctly linked, the error message itself is emitted from numpy/__init__.py.

@charris

charris commented Nov 22, 2022

Copy link
Copy Markdown
Member

My guess is a caching problem.

@seberg

seberg commented Nov 22, 2022

Copy link
Copy Markdown
Member Author

Well, this is going through and I just restarted the last few jobs. Hopefully, that is all we see from this.

@charris
charris merged commit 02441cb into numpy:main Nov 22, 2022
@charris

charris commented Nov 22, 2022

Copy link
Copy Markdown
Member

Thanks Sebastian.

@seberg
seberg deleted the machar-depr branch November 22, 2022 18:11
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