On Wed, 29 Sep 2010 12:57:54 -0700 (PDT) Jean-Pierre Flori <jpfl...@gmail.com> wrote:
> On 29 sep, 20:28, Burcin Erocal <bur...@erocal.org> wrote: > > On Wed, 29 Sep 2010 09:48:25 -0700 (PDT) > > > > Jean-Pierre Flori <jpfl...@gmail.com> wrote: > > > Maybe it is a good thing to keep the same order as ginac > > > internally and your more usual ordering for printing. > > > > It is good to keep the ginac ordering internally. The user friendly > > ordering is more expensive so it slows down computations, even when > > you don't print intermediate expressions. > My bad, I didn't read your previous post correctly, I thought that > patch reverted the change you made in the ordering, but in fact it put > these changes in separate functions and use them only for printing... > That is why I tried to fix the bug in the code without your patch... > So I guess I should work WITH the patch ! Yes, please work with the patch. :) <snip> > > > But if you'd better use the above patch, that won't be necessary. > > > > I would like to replace the orders completely. I wanted to attack > > this problem for a long time. Now that we started, we might as > > well finish. :) > So once more I'm confused.. > What do you mean by replace the orders completely ? > Having separate functions for internal use and printing as with what > your patch does ? <snip> I want to continue with the patch, to go back to using the original comparison functions from ginac and use the new ones only for printing. > > > > Can you help put the patch in usable shape? At the moment the > > ordering it defines is completely different from the one we use in > > Sage. This must be because I changed some signs while I was copying > > code. > I'll help with pleasure. Just want to be sure to take the right > direction ! Thank you. Please don't hesitate to ask if you need any help. Cheers, Burcin -- To post to this group, send email to sage-support@googlegroups.com To unsubscribe from this group, send email to sage-support+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-support URL: http://www.sagemath.org