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.