#11521: Use weak references to cache homsets
--------------------------------------------------+-------------------------
       Reporter:  jpflori                         |         Owner:  robertwb    
                 
           Type:  defect                          |        Status:  
positive_review              
       Priority:  major                           |     Milestone:  sage-5.5    
                 
      Component:  coercion                        |    Resolution:              
                 
       Keywords:  sd35                            |   Work issues:              
                 
Report Upstream:  N/A                             |     Reviewers:  Jean-Pierre 
Flori, Nils Bruin
        Authors:  Simon King                      |     Merged in:              
                 
   Dependencies:  #12969; to be merged with #715  |      Stopgaps:              
                 
--------------------------------------------------+-------------------------

Comment (by jdemeyer):

 Sorry to spoil the party, but with sage-5.4.rc0 + #715 + #11521, I get
 {{{
 sage -t  "devel/sage/sage/libs/singular/ring.pyx"
 **********************************************************************
 File
 "/release/merger/sage-5.5.beta0/devel/sage/sage/libs/singular/ring.pyx",
 line 490:
     sage: ring_ptr in ring_refcount_dict
 Expected:
     False
 Got:
     True
 **********************************************************************
 1 items had failures:
    1 of  22 in __main__.example_8
 ***Test Failed*** 1 failures.
 For whitespace errors, see the file
 /home/jdemeyer/.sage//tmp/ring_18750.py
          [3.4 s]
 sage -t
 "devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx"
 Exception AttributeError:
 AttributeError('PolynomialRing_field_with_category' object has no
 attribute '_modulus',) in  ignored
 Exception AttributeError:
 AttributeError('PolynomialRing_field_with_category' object has no
 attribute '_modulus',) in  ignored
 **********************************************************************
 File
 
"/release/merger/sage-5.5.beta0/devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx",
 line 423:
     sage: len(ring_refcount_dict) == n
 Expected:
     True
 Got:
     False
 **********************************************************************
 1 items had failures:
    1 of  19 in __main__.example_4
 ***Test Failed*** 1 failures.
 For whitespace errors, see the file
 /home/jdemeyer/.sage//tmp/multi_polynomial_libsingular_19160.py
          [5.8 s]
 }}}
 So perhaps this does require #13447?

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