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

Add check for orthogonal/unitary matrices #10863

Closed
rbeezer mannequin opened this issue Mar 1, 2011 · 8 comments
Closed

Add check for orthogonal/unitary matrices #10863

rbeezer mannequin opened this issue Mar 1, 2011 · 8 comments

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Mar 1, 2011

Adds .is_unitary() for matrices over general rings, plus an over-riding method for matrices over RDF and CDF.

Depends on:

  1. Add vector conjugation #10536

Apply:

  1. attachment: trac_10863-unitary-matrices.patch

Component: linear algebra

Author: Rob Beezer

Reviewer: Martin Raum

Merged: sage-4.7.alpha4

Issue created by migration from https://trac.sagemath.org/ticket/10863

@rbeezer rbeezer mannequin added c: linear algebra labels Mar 1, 2011
@rbeezer rbeezer mannequin assigned jasongrout and williamstein Mar 1, 2011
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Mar 1, 2011

Attachment: trac_10863-unitary-matrices.patch.gz

@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Mar 1, 2011

Author: Rob Beezer

@rbeezer

This comment has been minimized.

@rbeezer rbeezer mannequin added the s: needs review label Mar 1, 2011
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Mar 5, 2011

comment:2

See relevant discussion on #10848.

@sagetrac-mraum

This comment has been minimized.

@sagetrac-mraum
Copy link
Mannequin

sagetrac-mraum mannequin commented Mar 25, 2011

Reviewer: Martin Raum

@sagetrac-mraum
Copy link
Mannequin

sagetrac-mraum mannequin commented Mar 25, 2011

comment:3

Looks good to me; All tests pass.

@jdemeyer jdemeyer added this to the sage-4.7 milestone Mar 25, 2011
@jdemeyer
Copy link

jdemeyer commented Apr 5, 2011

Merged: sage-4.7.alpha4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants