#5756: [with patch; positive review] improve coverage of rings/morphism.pyx (and
fix 5 bugs in morphism.pyx)
-------------------------+--------------------------------------------------
Reporter: was | Owner: mabshoff
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.1
Component: doctest | Keywords:
-------------------------+--------------------------------------------------
Comment(by mabshoff):
Replying to [comment:6 robertwb]:
> I agree that the notation could be made more consistent, but this patch
simply documents what is there (which is good) and fixes some bugs.
Yeah, getting it in should be the main goal here and now.
> One thing I noticed, which is not just common to this patch, is that
when we return a list that we don't want people to change (e.g. im_gens)
we (hopefully) make a copy. This is why tuples were invited, shouldn't we
just be using those instead?
'''implemented'' instead of '''invited''' I assume?
Either way, can you please open a followup ticket so that this doesn't get
lost.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5756#comment:7>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
