Skip to content

NEP29: Set minimum required version to NumPy 1.23+ - #2991

Merged
seisman merged 1 commit into
mainfrom
nep29/numpy-1.23
Jan 15, 2024
Merged

NEP29: Set minimum required version to NumPy 1.23+#2991
seisman merged 1 commit into
mainfrom
nep29/numpy-1.23

Conversation

@seisman

Copy link
Copy Markdown
Member

Description of proposed changes

Following NEP29 policy where NumPy 1.22 is to be dropped on or after 1 Jan 2024 (in a minor version increment, i.e. for PyGMT v0.11.0). Bumps minimum supported NumPy version to 1.23 in the pyproject.toml, requirements.txt and environment.yml files. Also update installation documentation to mention NumPy 1.22+ requirement.

This is in line with PyGMT's policy on NEP29 at https://www.pygmt.org/v0.10.0/maintenance.html#dependencies-policy.

Supersedes #2586

- python=3.12
- gmt=6.5.0
- ghostscript=10.02.1
- numpy>=1.22

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Removing the numpy version information from this file, so that we have one fewer file to update next time.

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.

Ok, using >=1.22 might mean the latest numpy version is pulled in anyways.

@seismanseisman added the deprecation Deprecating a feature label Jan 12, 2024
@seismanseisman added this to the 0.11.0 milestone Jan 12, 2024
@seismanseisman added the needs review This PR has higher priority and needs review. label Jan 12, 2024
- python=3.12
- gmt=6.5.0
- ghostscript=10.02.1
- numpy>=1.22

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.

Ok, using >=1.22 might mean the latest numpy version is pulled in anyways.

@seisman
seisman merged commit 621d6da into mainJan 15, 2024
@seisman
seisman deleted the nep29/numpy-1.23 branch January 15, 2024 03:28
@seismanseisman removed the needs review This PR has higher priority and needs review. label Jan 16, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecationDeprecating a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@seisman@weiji14