Skip to content

Temporarily restrict NumPy version in pyproject.toml#6210

Merged
tkoyama010 merged 2 commits intopyvista:mainfrom
adeak:maint/numpy_less_than_2
Jun 6, 2024
Merged

Temporarily restrict NumPy version in pyproject.toml#6210
tkoyama010 merged 2 commits intopyvista:mainfrom
adeak:maint/numpy_less_than_2

Conversation

@adeak
Copy link
Copy Markdown
Member

@adeak adeak commented Jun 5, 2024

We hope to fix all compatibility issues in #6156, but NumPy 2.0 is expected in 10 days and we should probably make sure that main doesn't break when NumPy 2.0 is released. Even though we'll only release when compatibility issues are fixed, excluding a dependency version that's known broken (just a bit) is good practice.

My plan/proposal is:

  1. merge this PR quickly to protect main from the NumPy 2.0 release until we're compatible
  2. remove the version restriction in Ensure compatibility with NumPy 2.0 #6156
  3. probably make the NumPy nightly build in our CI matrix mandatory, at least until 2.0 is released.

@adeak adeak added the maintenance Low-impact maintenance activity label Jun 5, 2024
@adeak adeak mentioned this pull request Jun 5, 2024
5 tasks
Copy link
Copy Markdown
Member

@tkoyama010 tkoyama010 left a comment

Choose a reason for hiding this comment

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

LGTM. I will merge this since we need to release as soon as possible.

@tkoyama010
Copy link
Copy Markdown
Member

I will create a patch release on this.

@tkoyama010 tkoyama010 enabled auto-merge (squash) June 5, 2024 22:13
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.09%. Comparing base (b6d92a8) to head (119d9ea).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6210   +/-   ##
=======================================
  Coverage   97.09%   97.09%           
=======================================
  Files         141      141           
  Lines       25932    25932           
=======================================
  Hits        25178    25178           
  Misses        754      754           

@tkoyama010 tkoyama010 disabled auto-merge June 6, 2024 00:35
@tkoyama010 tkoyama010 enabled auto-merge (squash) June 6, 2024 00:35
@tkoyama010 tkoyama010 merged commit 26856ce into pyvista:main Jun 6, 2024
tkoyama010 pushed a commit that referenced this pull request Jun 6, 2024
Co-authored-by: pyvista-bot <53544751+pyvista-bot@users.noreply.github.com>
@adeak adeak deleted the maint/numpy_less_than_2 branch June 6, 2024 04:22
@adeak
Copy link
Copy Markdown
Member Author

adeak commented Jun 6, 2024

Thanks, @tkoyama010!

adeak added a commit to adeak/pyvista that referenced this pull request Jun 6, 2024
Co-authored-by: pyvista-bot <53544751+pyvista-bot@users.noreply.github.com>
@banesullivan banesullivan mentioned this pull request Jul 7, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Low-impact maintenance activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants