#12116: perfect_power for integers
----------------------------------------------------------------+-----------
       Reporter:  roed                                          |         
Owner:  AlexGhitza    
           Type:  enhancement                                   |        
Status:  needs_review  
       Priority:  major                                         |     
Milestone:  sage-5.8      
      Component:  basic arithmetic                              |    
Resolution:                
       Keywords:                                                |   Work 
issues:                
Report Upstream:  Fixed upstream, but not in a stable release.  |     
Reviewers:  David Loeffler
        Authors:  David Roe                                     |     Merged 
in:                
   Dependencies:  #10596, #12363, #12638                        |      
Stopgaps:                
----------------------------------------------------------------+-----------

Comment (by kcrisman):

 I like this a lot.

 Silly questions:
  * `cdef int exp = 1` in the second hunk - since
 {{{
 sage: type(4.perfect_power()[0])
 sage.rings.integer.Integer
 }}}
   and you now reassign `exp` to that, does that line make any sense at
 all? It's certainly not an int.
  * Do we need to deprecate `base_exponent`?  It would be easy enough,
 basically same code as deprecating `is_power` except reverse the order of
 the output.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12116#comment:22>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to