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
MAINT: SciPy 1.11.1 backports #18779
MAINT: SciPy 1.11.1 backports #18779
Conversation
…nputs (scipy#18754) Also ensure `test_factorial{,2,k}_array_corner_cases` covers 0-dim. inputs correctly Unfortunately, `np.array` will ignore an explicit `ndim=0` request and happily give back a 1-dimensional result without complaint: ``` >>> np.array([1], ndmin=0, dtype=np.int64).ndim 1 ``` This invalidated the original assumptions for these tests. Additionally, we need to take care not to iterate over 0-dim arrays (but also keep the tests with two elements in `content` correct), that n.ravel().ndim == 1 even if n.ndim == 0, and that assert_equal will not distinguish scalars from 0-dim arrays with the same value.
* Fixes scipy#18765 * the offending unuran source file was removed over at scipy/unuran#9, and this PR updates us to point to latest `main` version of submodule (caution: this pulls in a few more commits from `main` we didn't have in original `1.11.0` release) * as noted over there, the (SciPy) testsuite appears to pass just fine when this file is purged, likely because we have `HAVE_DECL_LOG1P` always set, since we require C99, and therefore unuran always uses the version from `math.h` instead of in-house anyway
Just to comment - it would be great if #18644 could make it into the next patch release as well. |
The only CI failure is or was a known issue on
I agree, this situation is a bit different in the sense that I'm moving a bit faster than usual and only added backports that are already merged. Apologies, but it'll surely be in |
450d8aa
into
scipy:maintenance/1.11.x
This release is happening pretty quickly to deal with a licensing issue.
Backports included:
factorial(..., exact=False)
for 0-dim array inputs #18754And updated release notes accordingly. The testsuite seems fine locally, didn't check doc build.