On Sun, Mar 27, 2011 at 01:33:59AM -0700, Anne Schilling wrote:
> Thanks for the explanation! Everything seems to work fine now.
> I gave a positive review on trac for the added features, perhaps
> someone else could do a technical review of the patch.

I'll try to handle the technical review (though I might leave some
parts about the connection with Singular to another reviewer). I am
soooo glad someone is taking the time to pursue the categorification
of Sage's code!

To start with, a couple questions for Simon:

 - The ticket contains two fairly distinct sets of features:
    - (1) The categorification of quotient rings and the like
    - (2) The letterplace free algebra
   What about splitting it into two tickets?

 - At this point, some code for, e.g., constructing quotient rings
   was copied from Ring to Rings.ParentMethods. Would it be possible
   to instead move it there? This raises two subquestions:
    - Are there any rings left in Sage which are not in the Rings category?
    - In the moved/copied code, is there any piece that really has to
      be written in Cython for some of the Sage rings? (at first sight
      I would believe not; I don't expect myriads of quotient rings to
      be created per second).

Cheers,
                                Nicolas
--
Nicolas M. ThiƩry "Isil" <nthi...@users.sf.net>
http://Nicolas.Thiery.name/

-- 
You received this message because you are subscribed to the Google Groups 
"sage-combinat-devel" group.
To post to this group, send email to sage-combinat-devel@googlegroups.com.
To unsubscribe from this group, send email to 
sage-combinat-devel+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/sage-combinat-devel?hl=en.

Reply via email to