On 10/4/07, Robert Bradshaw <[EMAIL PROTECTED]> wrote: > On Oct 4, 2007, at 1:35 PM, Joel B. Mohler wrote: > > > On Thursday 04 October 2007 15:38, Robert Bradshaw wrote: > >> Craig's reply is a good summary of what I was going to say. I'm > >> getting an > >> > >> abort: unknown parent e91367eb18c1! > > > > Well, it looks like William succeeded in looking at it as the patch is > > changed. > > Maybe I just didn't pull recently enough... (did an -upgrade though). > > > Craig mentioned the call to MatrixSpace is because multiplication > > of a matrix > > with QQ entries happens in the ZZ space. I see that now, but it > > wasn't the > > only call to MatrixSpace. That is, multiplication of a matrix with ZZ > > entries also called MatrixSpace (forgetting that sometimes we are > > square and > > can just use my existing parent). > > Yes. Hopefully your patch fixes this. > > > I'm glad the matrix coercion code changed! It truly looked a bit > > frightening. > > Yeah, it was trying to handle each all sorts of things on a case-by- > case basis between vectors and matrices... Now it's completely out of > element.pyx, and is much shorter and easier to understand > (implemented via Actions, where matrices act on vectors, etc.)
Very nice. FYI: I've applied your new coercion / inplace patch, Cython package, and strassen fixes for Sage-2.8.6. -- William --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/ -~----------~----~----~----~------~----~------~--~---
