#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, #14159        |      Stopgaps:                 
 
-----------------------------------------------+----------------------------

Comment (by SimonKing):

 I hesitate to do things like
 {{{
 from sage.misc.superseded import deprecated_function_alias
 from sage.misc.fast_methods import FastFashable_class as FH_class
 FastHashable_class = deprecated_function_alias(14054,FH_class)
 }}}
 in sage.categories.category_singleton:

 - It adds an unneeded import statement
 - `FastHashable_class` is a base class, and hence one would like to
 inherit from it. But doing the above, `FastHashable_class` would not be a
 class but a function in sage.categories.category_singleton.
 - In its now deprecated use, it was necessary to ''cimport'' (not just
 import!) the class. That will be impossible, no matter what way of
 deprecation warning we use, because deprecation warnings won't work at
 compile time.

 Is there a "lazy import statement with deprecation"? Then, one would not
 actually import `FastHashable_class` in category_singleton, but `from
 sage.categories.category_singleton import FastHashable_class` would result
 in a deprecation warning. And we can't cope with the compile time cimport
 problem anyway.

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