#12588: abs(pi*I) should return pi
-------------------------------------+-------------------------------------
       Reporter:  dkrenn             |        Owner:  burcin
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.8
      Component:  symbolics          |   Resolution:
       Keywords:  absolute value,    |    Merged in:
  symbolic                           |    Reviewers:  Karl-Dieter Crisman
        Authors:  Ralf Stephan       |  Work issues:
Report Upstream:  Fixed upstream,    |       Commit:
  in a later stable release.         |  33e868c92e7cb2627739d717b79ad21854033beb
         Branch:                     |     Stopgaps:
  u/rws/abs_pi_i__should_return_pi   |
   Dependencies:  #18537             |
-------------------------------------+-------------------------------------

Comment (by dkrenn):

 Replying to [comment:18 rws]:
 > Replying to [comment:17 kcrisman]:
 > > {{{
 > > sage: abs(i*x)
 > > abs(I*x)
 > > }}}
 > That's because the product's factors are separated into real constants
 and the rest. If the rest is constant, take `abs` of it, else
 `abs().hold()`. Then multiply together. Here `I*x` has symbols so it is
 held.
 > [...] However I'd suggest not to hold up this ticket for this.

 Anyhow, since {{{abs(I)}}} already in current Sage gives {{{1}}}, the
 expression {{{abs(I*x)}}} should be simplified to {{{abs(x)}}}.

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