#16721: implement gcd(a,b,hold=True)
-------------------------------+------------------------
       Reporter:  rws          |        Owner:
           Type:  enhancement  |       Status:  new
       Priority:  major        |    Milestone:  sage-6.3
      Component:  symbolics    |   Resolution:
       Keywords:               |    Merged in:
        Authors:               |    Reviewers:
Report Upstream:  N/A          |  Work issues:
         Branch:               |       Commit:
   Dependencies:               |     Stopgaps:  todo
-------------------------------+------------------------
Changes (by rws):

 * type:  defect => enhancement


Old description:

> Defect because of
> {{{
> sage: x=var('x')
> sage: gcd(x,4)
> 1
> sage: gcd(x,4,hold=True)
> ---------------------------------------------------------------------------
> Traceback (most recent call last)
> ...
> TypeError: gcd() takes no keyword arguments
> }}}
> See also #15497

New description:

 The implemented behaviour of `gcd` is consistent if its arguments are
 rational numbers or polynomials. Trying to give a symbolic variable will
 convert to polynomial which behaves unexpectedly:
 {{{
 sage: x=var('x')
 sage: gcd(x,4)
 1
 }}}
 The natural way to handle it is using the keyword `hold`:
 {{{
 sage: gcd(x,4,hold=True)
 ---------------------------------------------------------------------------
 Traceback (most recent call last)
 ...
 TypeError: gcd() takes no keyword arguments
 }}}
 As comment:18 shows this may not be easy and involve other changes.

 See also #15497

--

--
Ticket URL: <http://trac.sagemath.org/ticket/16721#comment:17>
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