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

Basic iteration functionality for products of projective spaces #19557

Closed
sagetrac-gjorgenson mannequin opened this issue Nov 9, 2015 · 17 comments
Closed

Basic iteration functionality for products of projective spaces #19557

sagetrac-gjorgenson mannequin opened this issue Nov 9, 2015 · 17 comments

Comments

@sagetrac-gjorgenson
Copy link
Mannequin

sagetrac-gjorgenson mannequin commented Nov 9, 2015

Implement nth_iterate and orbit functions for morphisms/points of products of projective spaces. Also implement nth_iterate_map for morphisms.

CC: @bhutz

Component: algebraic geometry

Author: Grayson Jorgenson

Branch/Commit: fded196

Reviewer: Ben Hutz

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

@sagetrac-gjorgenson sagetrac-gjorgenson mannequin added this to the sage-6.10 milestone Nov 9, 2015
@sagetrac-gjorgenson
Copy link
Mannequin Author

sagetrac-gjorgenson mannequin commented Nov 10, 2015

Branch: u/gjorgenson/ticket/19557

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2015

Commit: 3831d5a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2015

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

3831d5a19557: implementation of the iteration functionality

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2015

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

920d99719557: added example and fixed some spacing

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2015

Changed commit from 3831d5a to 920d997

@bhutz
Copy link

bhutz commented Nov 17, 2015

comment:5

Need to fix the merge issue.

From looking through the commits I see a couple things to fix:

in nth_iterate:
these checks are done in __call__ with better coercion, so not needed here.
if self.codomain()!=f.domain():
raise TypeError("Point is not defined over domain of function")

use .is_endomorphism() instead:
if f.domain() != f.codomain():
raise TypeError("Domain and Codomain of function not equal")

in nth_iterate_map
use is_endomorphism() check

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 19, 2015

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

1a1317219557: Merged with new beta to fix merge conflict.
559811d19557: changed checks

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 19, 2015

Changed commit from 920d997 to 559811d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 19, 2015

Changed commit from 559811d to 37c4e69

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 19, 2015

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

f114f1b19557: attempted conflict fix
37c4e6919557: second merge attempt

@bhutz
Copy link

bhutz commented Nov 19, 2015

comment:9

That all checks out fine. Just one thing: I was wrong about __call__ doing the coercion checking. That is true for projective spaces, but not for products. I think you should put that check back in as some weird behavior can occur.

Go ahead and put that back in, then I'll mark this positive.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 21, 2015

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

fded19619557: added check back in

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 21, 2015

Changed commit from 37c4e69 to fded196

@sagetrac-gjorgenson
Copy link
Mannequin Author

sagetrac-gjorgenson mannequin commented Nov 21, 2015

comment:11

Alright, I added the check into orbit as well.

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

comment:13

reviewer name

@bhutz
Copy link

bhutz commented Nov 23, 2015

Reviewer: Ben Hutz

@vbraun
Copy link
Member

vbraun commented Nov 24, 2015

Changed branch from u/gjorgenson/ticket/19557 to fded196

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