-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
parts of the random API missing from intersphinx #14359
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
Comments
okay there's quite a few other issues in the |
gh-14360 doesn't address everything yet, I just discovered that all |
Left to decide: there's still two namespaces left that look private:
Left to do:
|
I'm not sure these are used anymore after moving to SeedSeq. They might be removable. The only advantage is that random_entropy ready system entropy to an ndarray. It predates Python 3.x only and so was needed since there wasn't a good way to universally ready entropy. IIRC entropy is read from |
Edit: use |
With that one change, then it could be removed. |
|
I think SS is the right idea here. Empty to not reproducible so might as well DRY here and remove entropy. |
Is there a way to check that gh-14948 fixes scipy/scipy#10545? |
discussed with @mattip: this will fix it, let's just merge and then update SciPy after to use inter-sphinx links for the random functions again. The Cython changes are the more interesting ones in this PR. |
Going to close this as the fix has a 1.18 milestone. |
detected in a couple of doc build issues/PRs in SciPy now, see scipy/scipy#10540 (comment)
The
numpy.random
refactor somehow has caused entries in the intersphinx numpy output to go missing. at leastnumpy.random.RandomState
andnumpy.random.weibull
, so probably the whole older API is missing (and perhaps the new one as well).The tracking issue for the workaround in SciPy (explicitly referencing
random.mtrand.<object>
is scipy/scipy#10545The text was updated successfully, but these errors were encountered: