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

change ring does not work for affine morphisms #21285

Closed
bhutz opened this issue Aug 18, 2016 · 8 comments
Closed

change ring does not work for affine morphisms #21285

bhutz opened this issue Aug 18, 2016 · 8 comments
Labels
c: algebraic geometry p: minor / 4 t: bug
Milestone
sage-7.4

Comments

@bhutz
Copy link

bhutz commented Aug 18, 2016

the following does not work

 K.<a> = QuadraticField(-1)
 A.<x,y> = AffineSpace(K, 2)
 H = End(A)
 phi = H([x/y, y])
 emb = K.embeddings(QQbar)[0]
 phi.change_ring(emb)

since the coordiante ring is a fraction field

CC: @sagetrac-gjorgenson

Component: algebraic geometry

Author: Ben Hutz

Branch/Commit: 50d05de

Reviewer: Grayson Jorgenson

Issue created by migration from https://trac.sagemath.org/ticket/21285
@bhutz bhutz added this to the sage-7.4 milestone Aug 18, 2016
@bhutz
Copy link
Author

bhutz commented Aug 18, 2016

Branch: u/bhutz/t/21285

@bhutz
Copy link
Author

bhutz commented Aug 18, 2016

Changed branch from u/bhutz/t/21285 to none

@bhutz
Copy link
Author

bhutz commented Aug 18, 2016

New commits:

50d05de21285: error in change_ring for affine morphisms

@bhutz
Copy link
Author

bhutz commented Aug 18, 2016

Branch: u/bhutz/t/21285

@bhutz
Copy link
Author

bhutz commented Aug 18, 2016

Commit: 50d05de

@sagetrac-gjorgenson
Copy link
Mannequin

sagetrac-gjorgenson mannequin commented Aug 18, 2016

comment:4

Thanks for fixing this. Looks good to me.

@sagetrac-gjorgenson
Copy link
Mannequin

sagetrac-gjorgenson mannequin commented Aug 18, 2016

Reviewer: Grayson Jorgenson

@vbraun
Copy link
Member

vbraun commented Aug 21, 2016

Changed branch from u/bhutz/t/21285 to 50d05de

@vbraun vbraun closed this as completed in e4a5ac4 Aug 21, 2016
@sagetrac-gjorgenson sagetrac-gjorgenson mannequin mentioned this issue Sep 7, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebraic geometry p: minor / 4 t: bug
Projects
None yet
Milestone
sage-7.4
Development

No branches or pull requests
2 participants
@vbraun @bhutz