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

Additional method "matrix" / "_matrix_" for the element class of ClassicalMatrixLieAlgebra #25008

Closed
soehms opened this issue Mar 18, 2018 · 8 comments

Comments

@soehms
Copy link
Member

soehms commented Mar 18, 2018

A method to obtain the underlying matrix as element of the corresponding full matrix space analog to the similar method of MatrixGroupElement_generic should be added. This may be a shortcut for the following:

sage: LM = lie_algebras.sp(QQ, 4, representation='matrix')
sage: e1, e2, f1, f2, h1, h2 = LM.gens()
sage: mh1 = LM.associative_algebra()(dict(h1)); mh1
[ 1  0  0  0]
[ 0 -1  0  0]
[ 0  0 -1  0]
[ 0  0  0  1]
sage: mh1.parent()
Full MatrixSpace of 4 by 4 sparse matrices over Rational Field

CC: @tscrim

Component: algebra

Keywords: classical lie algebra matrix matrix

Author: Sebastian Oehms

Branch/Commit: f1d16d1

Reviewer: Travis Scrimshaw

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

@soehms soehms added this to the sage-8.2 milestone Mar 18, 2018
@soehms
Copy link
Member Author

soehms commented Mar 18, 2018

Branch: u/soehms/matrix_method-25008

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 19, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

089091asimplification

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 19, 2018

Commit: 089091a

@tscrim
Copy link
Collaborator

tscrim commented Mar 22, 2018

Changed commit from 089091a to f1d16d1

@tscrim
Copy link
Collaborator

tscrim commented Mar 22, 2018

comment:4

I made a few small fixes to the documentation and added another test. If my changes look good, then you can set a positive review.


New commits:

f1d16d1Small doc tweaks and adding another test.

@tscrim
Copy link
Collaborator

tscrim commented Mar 22, 2018

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Mar 22, 2018

@vbraun
Copy link
Member

vbraun commented May 12, 2018

Changed branch from public/lie_algebras/matrix_method-25008 to f1d16d1

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