#9016: make morphisms hashable
---------------------------------+------------------------------------------
       Reporter:  burcin         |         Owner:  AlexGhitza     
           Type:  defect         |        Status:  needs_review   
       Priority:  major          |     Milestone:  sage-5.6       
      Component:  categories     |    Resolution:                 
       Keywords:                 |   Work issues:                 
Report Upstream:  N/A            |     Reviewers:  Robert Bradshaw
        Authors:  Burcin Erocal  |     Merged in:                 
   Dependencies:                 |      Stopgaps:                 
---------------------------------+------------------------------------------

Comment (by tscrim):

 Looks much better now. Two more minor things (sorry for not noticing this
 earlier):

 - Move the `from copy import copy` from the header of `morphism.pyx` to
   {{{
   if not im_gens.is_immutable():
       from copy import copy
       im_gens = copy(im_gens)
       im_gens.set_immutable()
   }}}
   (or perhaps a `lazy_import`)
   This should make the startup plugin happy.

 - Could you add a (short) description such as {{{Return the hash of
 ``self``}}} to each `__hash__()` function.

 After this, that should be it. Thank you.

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