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

indeterminacy locus for rational maps #21104

Closed
sagetrac-mmanes mannequin opened this issue Jul 27, 2016 · 13 comments
Closed

indeterminacy locus for rational maps #21104

sagetrac-mmanes mannequin opened this issue Jul 27, 2016 · 13 comments

Comments

@sagetrac-mmanes
Copy link
Mannequin

sagetrac-mmanes mannequin commented Jul 27, 2016

Create function to compute the indeterminancy locus of a rational map on projective space.

Component: algebraic geometry

Author: Michelle Manes

Branch/Commit: 8680cc0

Reviewer: Ben Hutz

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

@sagetrac-mmanes sagetrac-mmanes mannequin added this to the sage-7.3 milestone Jul 27, 2016
@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 28, 2016

@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 28, 2016

New commits:

61ce7d721104: define indeterminacy_locus function

@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 28, 2016

Author: Michelle Manes

@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 28, 2016

Commit: 61ce7d7

@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 28, 2016

comment:3

I want to add functionality for indeterminacy_points when the indeterminacy locus is a zero-dimensional subscheme.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 29, 2016

Changed commit from 61ce7d7 to 3802039

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 29, 2016

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

380203921104: added function for indeterminacy_points

@sagetrac-mmanes
Copy link
Mannequin Author

sagetrac-mmanes mannequin commented Jul 29, 2016

New commits:

380203921104: added function for indeterminacy_points

@bhutz
Copy link

bhutz commented Jul 29, 2016

comment:6

Everything works find, just a couple minor things

  • need to wrap long doc lines

  • do not need the line

F = self.base_ring()

  • output of indet_pts: spelling of indet.

  • double quote F everywhere it occurs in docs

  • if not F.is_field() should be inside F is not None if branch

  • need doctest to test that F works

Here are some interesting examples I was trying. They all work, so add which ever you think

%time
set_verbose(None)
P.<x,y,z>=ProjectiveSpace(QQ,2)
H=End(P)
f=H([x^2+y^2,y^2-z^2,x^2+z^2])
f.indeterminacy_points(F=QuadraticField(-1))
%time
set_verbose(None)
K.<t>=FunctionField(QQ)
P.<x,y,z>=ProjectiveSpace(K,2)
H=End(P)
f=H([x^2-t^2*y^2,y^2-z^2,x^2-t^2*z^2])
f.indeterminacy_points()
%time
set_verbose(None)
P.<x,y,z>=ProjectiveSpace(Qp(3),2)
H=End(P)
f=H([x^2-7*y^2,y^2-z^2,x^2-7*z^2])
f.indeterminacy_points()
  • you should add a warning/note of some kind in the docs about the fact that these functions do not normalize the coordinates for examples such as
P.<x,y,z>=ProjectiveSpace(QQ,2)
H=End(P)
f=H([x*x^2,x*y^2,x*z^2])
f.indeterminacy_locus()

@bhutz
Copy link

bhutz commented Jul 29, 2016

Reviewer: Ben Hutz

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 29, 2016

Changed commit from 3802039 to 8680cc0

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 29, 2016

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

8680cc021104: improved documentation and minor improvement to code

@vbraun
Copy link
Member

vbraun commented Aug 10, 2016

Changed branch from u/mmanes/indeterminacy_locus_for_rational_maps to 8680cc0

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