#16986: Enhance rational_preimages and all_rational_preimages to work over 
Number
Fields
-------------------------------------+-------------------------------------
       Reporter:  jdefaria           |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.4
      Component:  algebraic          |   Resolution:
  geometry                           |    Merged in:
       Keywords:                     |    Reviewers:  Ben Hutz
        Authors:  Joao de Faria      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  928941e6a3df966c89ae97c7bf0250bd049e264c
  u/jdefaria/ticket/16986            |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by bhutz):

 * status:  needs_review => needs_work


Comment:

 Actually, this example is not fixed

 {{{
 P.<x,y>=ProjectiveSpace(CC,1)
 H=End(P)
 f=H([x^2-2*y^2,y^2])
 f.rational_preimages(P([0,1]))
 }}}
 but the other one is.

 Also some minor things:
 - Both docstrings still says requires QQ and endomorphism of projective
 space. Neither are still true.
 - There are two lines with terminating white space 2519, 2550.
 - Coordinates is spelled wrong on 2604.

--
Ticket URL: <http://trac.sagemath.org/ticket/16986#comment:16>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to