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

implement a symplectic form for finding symplectic bases #2664

Closed
ncalexan mannequin opened this issue Mar 25, 2008 · 4 comments
Closed

implement a symplectic form for finding symplectic bases #2664

ncalexan mannequin opened this issue Mar 25, 2008 · 4 comments

Comments

@ncalexan
Copy link
Mannequin

ncalexan mannequin commented Mar 25, 2008

Implement a symplectic_form for computing symplectic bases for alternating, anti-symmetric matrices. This is essential for computing with polarized Abelian varieties.

The attached patch computes symplectic forms over fields and the integers.

CC: @ncalexan

Component: linear algebra

Keywords: symplectic basis form integer matrix

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

@ncalexan ncalexan mannequin added this to the sage-2.11 milestone Mar 25, 2008
@ncalexan ncalexan mannequin assigned williamstein Mar 25, 2008
@ncalexan ncalexan mannequin added the s: needs review label Mar 25, 2008
@ncalexan
Copy link
Mannequin Author

ncalexan mannequin commented Mar 25, 2008

Attachment: 2664-ncalexan-symplectic-form-1.patch.gz

@craigcitro
Copy link
Member

comment:1

Patch looks great! There are one or two tiny things I'd like to see fixed:

  • I don't like the line alternating_form = symplectic_form, for the same reasons as my similar comment on clean and better document is_totally_real(), add is_totally_imaginary() to NumberField_generic #2707.

  • Should the new matrix/symplectic_basis.py file have a copyright notice at the top?

  • Given that there's this nice code for computing symplectic bases, it might be nice for someone to write a SymplecticSpace class that inherits from some kind of vector space class, and uses this code behind the scenes where necessary. (Do we already have an inner product space class?) I'm not saying you should implement this right now, of course! But maybe file a trac ticket? Would you use this in what you're doing with this code? (I don't personally have any need of it right now, but it seems like it would be useful in general.)

@craigcitro craigcitro changed the title implement a symplectic form for finding symplectic bases [with referee report] implement a symplectic form for finding symplectic bases Mar 28, 2008
@ncalexan
Copy link
Mannequin Author

ncalexan mannequin commented Mar 28, 2008

Attachment: 2664-ncalexan-symplectic-form-2.patch.gz

@craigcitro craigcitro changed the title [with referee report] implement a symplectic form for finding symplectic bases implement a symplectic form for finding symplectic bases Mar 28, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 28, 2008

comment:3

Merged in Sage 2.11.alpha2

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

2 participants