#12257: 1.0*pi should *not* be pi
----------------------------+------------------------
       Reporter:  was       |        Owner:  burcin
           Type:  defect    |       Status:  new
       Priority:  minor     |    Milestone:  sage-6.4
      Component:  calculus  |   Resolution:
       Keywords:  pynac     |    Merged in:
        Authors:            |    Reviewers:
Report Upstream:  N/A       |  Work issues:
         Branch:            |       Commit:
   Dependencies:            |     Stopgaps:
----------------------------+------------------------

Comment (by rws):

 Replying to [comment:8 kcrisman]:
 > Here is another idea.  In
 [http://www.ginac.de/reference/mul_8cpp_source.html#l00442 the Ginac
 source], we have
 > {{{
 > 00464     } else if (seq_size==1 && overall_coeff.is_equal(_ex1)) {
 > 00465         // *(x;1) -> x
 > 00466         return recombine_pair_to_ex(*(seq.begin()));
 > }}}
 This is currently in Pynac:
 https://github.com/pynac/pynac/blob/master/ginac/mul.cpp#L741
 > and that could be somehow changed.  And the other issue is
 [http://www.ginac.de/reference/inifcns__trans_8cpp_source.html#l00303
 here]
 > {{{
 > 00319         if (z.is_equal(*_num0_p))  // sin(0)       -> 0
 > 00320             return _ex0;
 > }}}
 And this is currently
 https://github.com/pynac/pynac/blob/master/ginac/inifcns_trans.cpp#L430

 The Pynac ticket for this on github is
 https://github.com/pynac/pynac/issues/4

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