#13107: python long division causing infinite loop
------------------------------------------------+---------------------------
Reporter: titusn | Owner: burcin
Type: defect | Status:
positive_review
Priority: critical | Milestone: sage-5.3
Component: symbolics | Resolution:
Keywords: pynac division | Work issues:
Report Upstream: N/A | Reviewers: Burcin
Erocal, Jean-Pierre Flori
Authors: Titus Nicolae, Burcin Erocal | Merged in:
Dependencies: #13316 | Stopgaps:
------------------------------------------------+---------------------------
Changes (by jpflori):
* status: needs_review => positive_review
Old description:
> {{{
>
> sage: ~SR(2L)
>
> local/bin/sage-sage: line 303: 9165 Segmentation fault sage-ipython
> "$@" -i
>
> }}}
> {{{~SR(2L)}}}is evaluated to {{{ SR(2L)^(-1) }}}[[BR]]
> because of a faulty simplification in the power::eval function it
> generates power objects recursively.
>
> A Pynac update fixing that is available in #13316.
> Here are the corresponding doctests.
>
> Apply:
> * [attachment:power+review.patch]
New description:
{{{
sage: ~SR(2L)
local/bin/sage-sage: line 303: 9165 Segmentation fault sage-ipython
"$@" -i
}}}
{{{~SR(2L)}}}is evaluated to {{{ SR(2L)^(-1) }}}[[BR]]
because of a faulty simplification in the power::eval function it
generates power objects recursively.
A Pynac update fixing that is available in #13316.
Here are the corresponding doctests.
Apply:
* [attachment:power+review.2.patch]
--
Comment:
I've rebased on top of 5.2 and removed the ALLOW_TAB stuff together with
the 3 corresponding tabs from expression.pyx.
As far as I'm concerned, the original patch together with this similarly
minor modifications gets positive review.
Someone now needs to review the similarly simple #13316 which I have now
tested to work on top of 5.2.
Only apply [attachment:power+eval.2.patch]
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13107#comment:5>
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.
