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

projective automorphism group fails for homogenized maps #18399

Closed
bhutz opened this issue May 11, 2015 · 10 comments
Closed

projective automorphism group fails for homogenized maps #18399

bhutz opened this issue May 11, 2015 · 10 comments

Comments

@bhutz
Copy link

bhutz commented May 11, 2015

When the numerator is a constant, the .polynomial() conversion returns a different variable than the denominator. This is the expected behavior from the documentation of those functions. So this ticket will change how the conversion is done.

Example that fails

A.<z>=AffineSpace(QQ,1)
HA=End(A)
f=HA([1/z^7])
F=f.homogenize(1)
F.automorphism_group()

Component: algebraic geometry

Keywords: projective morphism

Author: Ben Hutz

Branch: 62ce7b2

Reviewer: Grayson Jorgenson

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

@bhutz bhutz added this to the sage-6.7 milestone May 11, 2015
@bhutz bhutz self-assigned this May 11, 2015
@bhutz
Copy link
Author

bhutz commented May 11, 2015

Branch: u/bhutz/ticket/18399

@bhutz
Copy link
Author

bhutz commented May 11, 2015

New commits:

6c04c1318399: fix bug in projective morphism automorphism group

@bhutz
Copy link
Author

bhutz commented May 11, 2015

Commit: 6c04c13

@bhutz

This comment has been minimized.

@sagetrac-gjorgenson
Copy link
Mannequin

sagetrac-gjorgenson mannequin commented May 14, 2015

comment:3

Everything looks good, but I think the ticket example should be added to the documentation as a test or example to demonstrate the fix.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 15, 2015

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

62ce7b218399: example added

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 15, 2015

Changed commit from 6c04c13 to 62ce7b2

@sagetrac-gjorgenson
Copy link
Mannequin

sagetrac-gjorgenson mannequin commented May 15, 2015

Reviewer: Grayson Jorgenson

@vbraun
Copy link
Member

vbraun commented May 18, 2015

Changed branch from u/bhutz/ticket/18399 to 62ce7b2

@bhutz
Copy link
Author

bhutz commented May 20, 2015

Changed commit from 62ce7b2 to none

@bhutz bhutz modified the milestones: sage-6.7, sage-6.8 May 20, 2015
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