Skip to content
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

scipy.stats.invwishart does not check if scale matrix is symmetric #6472

Closed
idc9 opened this issue Aug 8, 2016 · 1 comment · Fixed by #16104
Closed

scipy.stats.invwishart does not check if scale matrix is symmetric #6472

idc9 opened this issue Aug 8, 2016 · 1 comment · Fixed by #16104
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.stats
Milestone

Comments

@idc9
Copy link

idc9 commented Aug 8, 2016

The scale matrix argument for scipy.stats.invwishart should be symmetric, but it does not currently throw an error or warning. See documentation http://docs.scipy.org/doc/scipy-0.16.1/reference/generated/scipy.stats.invwishart.html

the scale matrix, which must be symmetric

The following code runs fine but should throw an error

df = 2
scale = [[1, 1], [0, 1]]
stats.invwishart.rvs(df=df, scale=scale)

I'm using scipy version 0.17.0.

@ev-br
Copy link
Member

ev-br commented Sep 1, 2016

I think we should loosely follow lapack and simply document which triangle is being ignored (upper or lower). @ChadFulton?

@mdhaber mdhaber added defect A clear bug or issue that prevents SciPy from being installed or used as expected Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org and removed defect A clear bug or issue that prevents SciPy from being installed or used as expected labels Mar 16, 2022
@tupui tupui added this to the 1.9.0 milestone May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.stats
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants