#14054: Cythoned UniqueRepresentation
-----------------------------------------------+----------------------------
       Reporter:  SimonKing                    |         Owner:  tbd            
 
           Type:  enhancement                  |        Status:  needs_review   
 
       Priority:  major                        |     Milestone:  sage-5.8       
 
      Component:  performance                  |    Resolution:                 
 
       Keywords:  cython UniqueRepresentation  |   Work issues:                 
 
Report Upstream:  N/A                          |     Reviewers:  Travis 
Scrimshaw
        Authors:  Simon King                   |     Merged in:                 
 
   Dependencies:  #14017, #6495                |      Stopgaps:                 
 
-----------------------------------------------+----------------------------

Comment (by SimonKing):

 Replying to [comment:25 tscrim]:
 > It must have something to testing things in the right order...I'm
 wondering if the problem lies with `free_module.tensor_constructor()`
 being a cached_method and something is being improperly stored...

 The failing assertion is
 {{{
       File "/mnt/storage2TB/patchbot/Sage/sage-5.8.beta0/local/lib/python
 /site-packages/sage/categories/modules_with_basis.py", line 1387, in
 __call__
         assert(x.parent() is self.domain())
 }}}

 I can hardly imagine that there is yet another premature deallocation via
 weak references in play. After all, both `x._parent` and `self._domain`
 are just plain strong references.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:26>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to