#8807: Adding support for morphisms to the category framework
---------------------------+------------------------------------------------
   Reporter:  SimonKing    |       Owner:  Simon King                   
       Type:  enhancement  |      Status:  needs_work                   
   Priority:  major        |   Milestone:  sage-4.6.1                   
  Component:  categories   |    Keywords:  morphisms functors categories
     Author:  Simon King   |    Upstream:  N/A                          
   Reviewer:               |      Merged:                               
Work_issues:               |  
---------------------------+------------------------------------------------

Comment(by SimonKing):

 Replying to [comment:9 cremona]:
 > ...
 > Here are the minor issues in docstrings:
 >
 > line 44: "one should implement two methods" -- do you mean three?

 Corrected.

 > _apply_functor_to_morphism: first line of docstring is a copy from the
 previous function but should presumably be "Apply the functor to a
 morphism between ... something"

 Corrected.

 > In the new {{{__call__}}} function, I would like to see a test of the
 branch which raises a {{{TypeError}}} ""%s is ill-defined, ..."

 Done. I define a class that behaves like the matrix constructor used to.

 > Is the spelling of "{{{CompositConstructionFunctor}}}" intentional?
 Should it not be "{{{CompositeConstructionFunctor}}}"?

 That spelling is old, so, I won't touch it.

 I am running `sage -tp 3 sage` right now, but I am replacing the old patch
 in a minute.

 Cheers,
 Simon

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