#3889: extend parameter for number field sqrt method
-----------------------------+----------------------------------------------
   Reporter:  jbmohler       |          Owner:  davidloeffler
       Type:  enhancement    |         Status:  new          
   Priority:  major          |      Milestone:  sage-4.7.2   
  Component:  number fields  |       Keywords:               
Work_issues:                 |       Upstream:  N/A          
   Reviewer:                 |         Author:               
     Merged:                 |   Dependencies:               
-----------------------------+----------------------------------------------
Changes (by jdemeyer):

  * upstream:  => N/A


Old description:

> Number field element sqrt should support the extend parameter in analogy
> with other sqrt functions.
>
> {{{
> sage: ZZ(4).sqrt(extend=False)
> 2
> sage: CyclotomicField(4)(4).sqrt(extend=False)
> ...
> TypeError: 'extend' is an invalid keyword argument for this function
> }}}
>
> If it would even have the parameter and raise a NotImplementedError if
> extend==True, that would aid in writing generic code for the present.

New description:

 Number field element sqrt should support the extend parameter in analogy
 with other sqrt functions.

 {{{
 sage: ZZ(4).sqrt(extend=False)
 2
 sage: CyclotomicField(4)(4).sqrt(extend=False)
 ...
 TypeError: sqrt() got an unexpected keyword argument 'extend'
 }}}

 If it would even have the parameter and raise a NotImplementedError if
 extend==True, that would aid in writing generic code for the present.

--

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3889#comment:3>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to