On Sun, Feb 22, 2009 at 1:30 AM, Rob Beezer <goo...@beezer.cotse.net> wrote: > > I have been walking through some of the matrix code before adding some > enhancements. >
... > > Proposal: Place the guts of kernel computations for each > (specialized) class into a right_kernel() method, where it would seem > to naturally belong. Mostly these would replace kernel() methods that +1 > are computing left kernels. A call to kernel() or left_kernel() > should arrive at the top of the hierarchy where it would take a > transpose and call the (specialized) right_kernel(). So there > wouldn't be a change in behavior in routines currently calling kernel > () or left_kernel(), and we would retain Sage's preference for the > left by having the vanilla kernel() give back a left kernel. The > changes would be more efficient computationally and also make the code > more expressive of what is really happening when and where. > > Comments, suggestions, warnings? (1) Add very detailed comments to the docstring describing exactly what is going on. (2) Test before and after behavior/timings to be sure the patch does as you hope. > > > > --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to sage-devel-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---