Skip to content

Update _svds.py removing no longer necessary QR for LOBPCG output #17651

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

Merged
merged 2 commits into from
Dec 25, 2022
Merged

Update _svds.py removing no longer necessary QR for LOBPCG output #17651

merged 2 commits into from
Dec 25, 2022

Conversation

lobpcg
Copy link
Contributor

@lobpcg lobpcg commented Dec 23, 2022

QR of lobpcg eigenvectors should be no longer needed since gh-16320 is merged and can be removed still passing all tests

Some issues with formatting using the quotes in the docstring needs fixing

Reference issue

N/A

What does this implement/fix?

QR of lobpcg eigenvectors is no longer needed after gh-16320 is merged and thus removed still passing all tests.

All issues with formatting using the quotes in the docstring are fixed.

Additional info?

There is a file _svds_doc.py in the same directory, I am unsure what for and how related to the docstring in _svds.py that I am lightly editing in this PR.

QR of lobpcg eigenvectors should be no longer needed after gh-16320 is merged and can be removed still passing all tests
@lobpcg lobpcg marked this pull request as draft December 23, 2022 20:15
editing of apparently typos in formatting quotes
@lobpcg lobpcg marked this pull request as ready for review December 23, 2022 20:32
@j-bowhay j-bowhay added enhancement A new feature or improvement scipy.sparse.linalg labels Dec 23, 2022
@lobpcg
Copy link
Contributor Author

lobpcg commented Dec 23, 2022

The failure looks unrelated, so ready for review and merge.
Only a one-line deletion plus a few formatting fixes. @melissawm ?

Copy link
Member

@tupui tupui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are green in it goes. Thanks @lobpcg

@tupui tupui added this to the 1.11.0 milestone Dec 25, 2022
@tupui tupui merged commit c137672 into scipy:main Dec 25, 2022
@lobpcg lobpcg deleted the patch-1 branch December 25, 2022 19:36
windows-server-2003 pushed a commit to windows-server-2003/scipy that referenced this pull request Jan 9, 2023
…put (scipy#17651)

* Update _svds.py

QR of lobpcg eigenvectors should be no longer needed after scipygh-16320 is merged and can be removed still passing all tests

* formatting quotes in docstring 

editing of apparently typos in formatting quotes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature or improvement scipy.sparse.linalg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants