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

BUG: Fix subspace_angles for complex values #10833

Merged
merged 1 commit into from Sep 18, 2019
Merged

Conversation

larsoner
Copy link
Member

subspace_angles was giving the wrong result for complex values because we weren't using the conjugate transpose.

@larsoner larsoner added this to the 1.4.0 milestone Sep 17, 2019
@larsoner larsoner added backport-candidate This fix should be ported by a maintainer to previous SciPy versions. defect A clear bug or issue that prevents SciPy from being installed or used as expected scipy.linalg labels Sep 17, 2019
@@ -47,6 +47,9 @@ lambda expressions for them.

`scipy.linalg` improvements
---------------------------
The function ``scipy.linalg.subspace_angles(A, B)`` now gives correct
Copy link
Member

Choose a reason for hiding this comment

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

Don’t we want to add this to the 1.4 wiki page instead of here?

Copy link
Member Author

Choose a reason for hiding this comment

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

So far I've put things both places, didn't know which was preferred. I'm happy to put it there once this is merged so it's in both, or remove it here

Copy link
Member

Choose a reason for hiding this comment

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

IIRC the wiki is preferred since we added it because we got tired of dealing with all the merge conflicts for the version in the repo. If that's the case we should probably remove the version in the repo until 1.4 is actually released. (@rgommers thoughts?)

Copy link
Member

Choose a reason for hiding this comment

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

yep, wiki is preferred. don't bother to remove it here though if it's ready to merge. in that case just leave it out of the wiki. the main purpose of the wiki page is to easily collect content and then merge it into this file in one go without causing merge conflicts.

@person142 person142 merged commit 7005fe4 into scipy:master Sep 18, 2019
@person142
Copy link
Member

Per

don't bother to remove it here though if it's ready to merge

merging then. Thanks @larsoner.

@larsoner larsoner deleted the subspace branch September 18, 2019 15:54
@tylerjereddy tylerjereddy modified the milestones: 1.4.0, 1.3.2 Oct 31, 2019
@tylerjereddy tylerjereddy removed the backport-candidate This fix should be ported by a maintainer to previous SciPy versions. label Nov 29, 2019
tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request Jan 20, 2020
* update SciPy 1.2.3 LTS release notes
after backporting scipygh-10882, scipygh-11199,
scipygh-10961, and scipygh-10833.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect A clear bug or issue that prevents SciPy from being installed or used as expected scipy.linalg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants