Skip to content

Bump Numpy minimal version to 1.20 #5902

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 2, 2024

Conversation

galagam
Copy link
Contributor

@galagam galagam commented Feb 2, 2024

Description

Fix Linux release pipeline due to usage of np.broadcast_shapes which was introduced in numpy==1.20.0

Motivation and Context

Fix #5900

onnx#5900
Fix Linux release pipeline due to usage of np.broadcast_shapes
which was introduced in numpy==1.20.0

Signed-off-by: Gal Hubara Agam <ghubaraagam@nvidia.com>
@galagam galagam requested a review from a team as a code owner February 2, 2024 17:02
Copy link

codecov bot commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3cd21a9) 56.65% compared to head (ba94e92) 56.65%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5902   +/-   ##
=======================================
  Coverage   56.65%   56.65%           
=======================================
  Files         506      506           
  Lines       30192    30192           
  Branches     4555     4555           
=======================================
  Hits        17104    17104           
  Misses      12266    12266           
  Partials      822      822           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
@justinchuby justinchuby enabled auto-merge February 2, 2024 18:00
@justinchuby justinchuby changed the title Bump Numpy minimal version to 1.20.0 (#5900) Bump Numpy minimal version to 1.20.3 Feb 2, 2024
@justinchuby justinchuby changed the title Bump Numpy minimal version to 1.20.3 Bump Numpy minimal version to 1.20 Feb 2, 2024
@justinchuby justinchuby disabled auto-merge February 2, 2024 18:05
@justinchuby justinchuby enabled auto-merge February 2, 2024 18:05
@justinchuby justinchuby added this pull request to the merge queue Feb 2, 2024
Merged via the queue into onnx:main with commit d08208f Feb 2, 2024
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
### Description
Fix Linux release pipeline due to usage of np.broadcast_shapes which was
introduced in numpy==1.20.0

### Motivation and Context

Fix onnx#5900

---------

Signed-off-by: Gal Hubara Agam <ghubaraagam@nvidia.com>
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Co-authored-by: Justin Chu <justinchuby@users.noreply.github.com>
Signed-off-by: Linsho Kaku <linsho@preferred.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

#5812 breaks Linux release pipeline
3 participants