On Fri, Mar 27, 2009 at 4:15 AM, Martin Albrecht <m...@informatik.uni-bremen.de> wrote: > > Hi there, > > the patch at > > http://trac.sagemath.org/sage_trac/ticket/5576 > > changes the behaviour of the subs() function for mq.MPolynomialSystem. Instead > of modifying self it now returns a new mq.MPolynomialSystem. This matches the > behaviour of other subs() functions and not doing it was a stupid design > decision due to me. > > However, this introduces interesting problems. For instance, -long would run > out of RAM (fixed in the latest version of the patch) because I forgot to > replace a line F.subs(s) with F = F.subs(s). > > Bottomline: The calling convention didn't change but the behaviour of the > function changed. How do we do this properly? We can't just deprecate it. > > I guess not too many people use that class anyway but still, should we have > some protocol for handling this? >
I have no good ideas except clearly documenting this in the docstring and release notes. -- William --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
