Skip to content

Replace np.product with np.prod due to deprecation#385

Merged
btjanaka merged 2 commits intomasterfrom
fix/np-prod
Sep 22, 2023
Merged

Replace np.product with np.prod due to deprecation#385
btjanaka merged 2 commits intomasterfrom
fix/np-prod

Conversation

@btjanaka
Copy link
Copy Markdown
Member

@btjanaka btjanaka commented Sep 22, 2023

Description

np.product is deprecated in numpy 1.25.0 -- see numpy/numpy#23314

This PR replaces our calls to np.product with np.prod.

TODO

Questions

Status

  • I have read the guidelines in
    CONTRIBUTING.md
  • I have formatted my code using yapf
  • I have tested my code by running pytest
  • I have linted my code with pylint
  • I have added a one-line description of my change to the changelog in
    HISTORY.md
  • This PR is ready to go

@btjanaka btjanaka mentioned this pull request Sep 22, 2023
@btjanaka btjanaka merged commit 473f24b into master Sep 22, 2023
@btjanaka btjanaka deleted the fix/np-prod branch September 22, 2023 23:34
btjanaka added a commit that referenced this pull request Sep 25, 2023
<!-- Provide a brief description of the PR's purpose here. -->

`np.product` is deprecated in numpy 1.25.0 -- see
numpy/numpy#23314

This PR replaces our calls to `np.product` with `np.prod`.

<!-- Notable points that this PR has either accomplished or will
accomplish. -->

<!-- Any concerns or points of confusion? -->

- [x] I have read the guidelines in

[CONTRIBUTING.md](https://github.com/icaros-usc/pyribs/blob/master/CONTRIBUTING.md)
- [x] I have formatted my code using `yapf`
- [x] I have tested my code by running `pytest`
- [x] I have linted my code with `pylint`
- [x] I have added a one-line description of my change to the changelog
in
      `HISTORY.md`
- [x] This PR is ready to go
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.

1 participant