#11900: Serious regression caused by #9138
-------------------------------------------------+--------------------------
   Reporter:  SimonKing                          |          Owner:  tbd         
                 
       Type:  defect                             |         Status:  
needs_review                 
   Priority:  critical                           |      Milestone:  sage-4.8    
                 
  Component:  performance                        |       Keywords:  categories 
regression        
Work_issues:  Segmentation faults                |       Upstream:  N/A         
                 
   Reviewer:  Jeroen Demeyer, Nicolas M. Thiéry  |         Author:  Simon King, 
Nicolas M. Thiéry
     Merged:                                     |   Dependencies:  #9138 
#11911                 
-------------------------------------------------+--------------------------
Changes (by SimonKing):

  * status:  needs_work => needs_review


Comment:

 Also when having applied your patch as well, at least one of the tests in
 which you have observed a segfault works fine:
 {{{
 king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.8.alpha3/devel/sage$
 ../../sage -t sage/structure/element.pyx
 sage -t  "devel/sage-main/sage/structure/element.pyx"
          [3.5 s]

 ----------------------------------------------------------------------
 All tests passed!
 Total time for all tests: 3.5 seconds
 }}}

 Actually note that with your patch it is only 3.5 seconds, while with my
 patch it is only 5.5 seconds.

 I am now running all long tests. I have read your patch, and it looks very
 reasonable. I'd appreciate if you could re-run the tests with
 sage-4.8.alpha3.

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