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.

Sign up for GitHub

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

Jump to bottom

standard smith normal form #785

Closed
syazdani77 mannequin opened this issue Oct 2, 2007 · 0 comments
Closed

standard smith normal form #785

syazdani77 mannequin opened this issue Oct 2, 2007 · 0 comments
Assignees
@syazdani77
Labels
c: linear algebra r: duplicate t: enhancement
Milestone
sage-2.8.6

Comments

@syazdani77
Copy link
Mannequin

syazdani77 mannequin commented Oct 2, 2007

The smith_form function for integer dense matrices are printed backward from the usual notation. This is because pari prints them backward.

The enclosed patch fixes this problem, by permuting the entries appropriately.

Component: linear algebra

Keywords: smith_form

Issue created by migration from https://trac.sagemath.org/ticket/785
@syazdani77 syazdani77 mannequin added this to the sage-2.8.6 milestone Oct 2, 2007
@syazdani77 syazdani77 mannequin self-assigned this Oct 2, 2007
@syazdani77 syazdani77 mannequin added r: duplicate and removed p: minor / 4 labels Oct 2, 2007
@syazdani77 syazdani77 mannequin closed this as completed Oct 2, 2007
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@syazdani77 syazdani77

Labels
c: linear algebra r: duplicate t: enhancement
Projects
None yet
Milestone
sage-2.8.6
Development

No branches or pull requests
0 participants