aesara-devs / aesara

Aesara is a Python library for defining, optimizing, and efficiently evaluating mathematical expressions involving multi-dimensional arrays.
https://aesara.readthedocs.io
Other
1.17k stars 156 forks source link

Replace deprecated `np.product` with `np.prod` #1502

Closed brandonwillard closed 1 year ago

brandonwillard commented 1 year ago

This PR also fixes a typing issue caused by unnecessary staticmethods on the Ops in nlinalg.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1502 (0ad16ce) into main (2368ed3) will decrease coverage by 0.01%. The diff coverage is 100.00%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/aesara-devs/aesara/pull/1502/graphs/tree.svg?width=650&height=150&src=pr&token=2HNzVWyxrA&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs)](https://app.codecov.io/gh/aesara-devs/aesara/pull/1502?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs) ```diff @@ Coverage Diff @@ ## main #1502 +/- ## ========================================== - Coverage 74.96% 74.96% -0.01% ========================================== Files 176 176 Lines 49500 49494 -6 Branches 11986 11986 ========================================== - Hits 37110 37104 -6 Misses 10089 10089 Partials 2301 2301 ``` | [Impacted Files](https://app.codecov.io/gh/aesara-devs/aesara/pull/1502?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs) | Coverage Δ | | |---|---|---| | [aesara/scalar/basic.py](https://app.codecov.io/gh/aesara-devs/aesara/pull/1502?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs#diff-YWVzYXJhL3NjYWxhci9iYXNpYy5weQ==) | `79.17% <100.00%> (ø)` | | | [aesara/tensor/nlinalg.py](https://app.codecov.io/gh/aesara-devs/aesara/pull/1502?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs#diff-YWVzYXJhL3RlbnNvci9ubGluYWxnLnB5) | `98.54% <100.00%> (-0.03%)` | :arrow_down: |