ENH: stats: add random variable infrastructure #110
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mdhaber
force-pushed
the
rv_infrastructure
branch
from
b406dc1
to
07b7286
Compare
[skip ci]
…ocs for ilogcdf/ilogccdf
* Correct todense documentation * Fix docstring for sparse matrices * Added optional parameters to todense method * Fix infinite recursion Co-authored-by: Dan Schult <[email protected]> * Remove array/matrix from array-only docstring --------- Co-authored-by: Dan Schult <[email protected]> Co-authored-by: CJ Carey <[email protected]>
Co-authored-by: Gregory Lee <[email protected]>
…sparray (scipy#20889) * make test_base uniformly use @ over * for matmul unless testing it * flesh out inplace matmul tests * fix test of dense @= due to numpy change in 1.25 * include review suggestions
Remove inadvertently copy and pasted duplicate lines Fix non-working clang-17 install Add missing sudo in clang-17 install Add missing requirement Fix invocation of python -m build Remove incorrect use of false in Clang-17 workflow Remove unnecessary parts of clang-17 CI job
BLD/DEV: special: Fix warning due to mixed initializers
Some assertions have atol/rtol configurable, and one assertion had them hardcoded, and that was causing tolerance problems in a Debian build with reference LAPACK. closes scipy#20911
This is a follow-up to scipygh-20477, where HiGHS wasn't touched on purpose to avoid a merge conflict in another PR. Minor useful side benefit: it shrinks the size of `_highs_wrapper.so` by 0.4% Closes scipygh-20256
* ENH: stats: add array API support to combine_pvalues
DOC: update doctests to satisfy scipy-doctests==1.2.0 (the 1.2.0 version is not released yet, so testing this locally)
Allow the filter-warnings context manager to not receive a (doc)test. This is useful to filter out DeprecationWarnings which are emitted during test discovery. Which is needed, e.g. for numpy 1/2 transition where things deprecated in numpy 2 emit warnings on import.
…recation-message
to after checking if they are `None`.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@tupui
rv_infrastructureis the branch you should base your Array API work on. However, I see that there is some work inrv_continuousthat was intended to make it into SciPy already - all the stuff in_zeros.pyandtest_zeros.py. Here's what I'd suggest:_distribution_infrastructure.pyandtest_continuous.pyfromrv_continuous.Meanwhile, I'll submit a PR to SciPy main with the changes toDone._zeros.pyandtest_zeros.py. Sorry those didn't make it before you reviewed the bracketing PR; apparently I had refactored for clarity in this branch.After that merges, we'll rebase this branch on SciPyDone.main. We'll also make sure to remove the duplicate hypothesis configuration.rv_continuousinto your Array API branch again because it will have changed, but there shouldn't be any difficult merge conflicts. You should just be able to usetheirsfor all files except the ones you've modified.How does that sound?