Skip to content

Conversation

seberg
Copy link
Member

@seberg seberg commented Nov 21, 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.

@@ -153,13 +152,13 @@ def _DType_reduce(DType):


def __getattr__(name):
# Deprecated 2021-10-20, NumPy 1.22
if name == "machar":
# Deprecated 2022-11-21, NumPy 1.24.
Copy link
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
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
Copy link
Member

charris commented Nov 22, 2022

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

@seberg
Copy link
Member Author

seberg commented Nov 22, 2022

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
Copy link
Member Author

seberg commented Nov 22, 2022

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

@charris
Copy link
Member

charris commented Nov 22, 2022

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
Copy link
Member

charris commented Nov 22, 2022

My guess is a caching problem.

@seberg
Copy link
Member Author

seberg commented Nov 22, 2022

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
Copy link
Member

charris commented Nov 22, 2022

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