#13107: python long division causing infinite loop
------------------------------------------------+---------------------------
       Reporter:  titusn                        |         Owner:  burcin        
                  
           Type:  defect                        |        Status:  needs_review  
                  
       Priority:  critical                      |     Milestone:  sage-5.4      
                  
      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_work => needs_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.2.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:trac_13107.patch]

--

Comment:

 Updated patch on top of 5.4. beta0.
 The failing hunks were formatting issues <hich have been corrected in
 another ticket in the meantime of getting this merged.

 5.4.beta1 should be ok with the updated patch.
 If someone confirms that, let's merge this.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13107#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 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