Skip to content
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

Updated all np.product calls to np.prod #5493

Merged
merged 9 commits into from
Sep 21, 2023

Conversation

acchamber
Copy link
Contributor

@acchamber acchamber commented Sep 12, 2023

🚀 Pull Request

Description

Updated all np.product calls to np.prod, as they caused depreciation warnings and were easy to fix.


Consult Iris pull request check list

Copy link
Contributor

@ESadek-MO ESadek-MO left a comment

Choose a reason for hiding this comment

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

Needs a whatsnew, but otherwise seems good, and I can't find any np.products that you've missed!

docs/src/whatsnew/latest.rst Outdated Show resolved Hide resolved
docs/src/whatsnew/latest.rst Outdated Show resolved Hide resolved
docs/src/whatsnew/latest.rst Outdated Show resolved Hide resolved
acchamber and others added 2 commits September 15, 2023 16:11
Co-authored-by: Elias <110238618+ESadek-MO@users.noreply.github.com>
acchamber and others added 2 commits September 15, 2023 16:15
Co-authored-by: Elias <110238618+ESadek-MO@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (10cec69) 89.37% compared to head (0b23152) 89.37%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5493   +/-   ##
=======================================
  Coverage   89.37%   89.37%           
=======================================
  Files          89       89           
  Lines       22446    22446           
  Branches     5387     5387           
=======================================
  Hits        20061    20061           
  Misses       1639     1639           
  Partials      746      746           
Files Changed Coverage Δ
...is/fileformats/_structured_array_identification.py 91.39% <100.00%> (ø)
lib/iris/pandas.py 95.14% <100.00%> (ø)

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

@ESadek-MO ESadek-MO merged commit b6e39d7 into SciTools:main Sep 21, 2023
17 checks passed
tkknight added a commit to tkknight/iris that referenced this pull request Oct 26, 2023
* upstream/main:
  moved latest warning banner logic to conf.py (SciTools#5508)
  updated layout of top navbar (SciTools#5505)
  Oblique and Rotated Mercator (SciTools#5548)
  [pre-commit.ci] pre-commit autoupdate (SciTools#5549)
  [pre-commit.ci] pre-commit autoupdate (SciTools#5527)
  Bump scitools/workflows from 2023.09.1 to 2023.10.0 (SciTools#5540)
  nep29 drop table schedule numpy>1.21 (SciTools#5525)
  Updated environment lockfiles (SciTools#5545)
  Replaced `NotImplementedError` with `NotImplemented` (SciTools#5544)
  Gallery: show colour bar stealing space from multiple axes (SciTools#5537)
  Updated environment lockfiles (SciTools#5524)
  Set some memory benchmarks to on-demand to reduce noise. (SciTools#5481)
  updating docs and stale comment (SciTools#5522)
  Ensure removal of release candidate from What's New title. (SciTools#5526)
  Updated environment lockfiles (SciTools#5513)
  Docs page on filtering warnings (SciTools#5509)
  Replaced pkg_resources version parser with packager version parser. (SciTools#5511)
  Categorise warnings (SciTools#5498)
  Updated all np.product calls to np.prod (SciTools#5493)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants