On Mon, Dec 19, 2011 at 1:18 PM, Simon King <[email protected]> wrote: > Hi! > > I work on fixing a memory leak at #11521. My patch uses weak > references for the cache of homsets (in sage.categories.homset). > > Problem: > > There are two doc tests that are repeated in three different files, > and they test that you can actually do > sage: z=(2/3); Hom(z,8/1) > But one can't have a weak reference to a rational number or an > integer. Hence, these tests result in an error with my patch. > > However, I think that the ability to consider rational numbers as > OBJECTS of a category (rather than elements of an object) and to > construct Hom(2/3, 8/1) is not much of a use. And in fact it does not > seem to be used anywhere in Sage, except in these few doc tests. > > Why were these tests introduced?
Because it's "cool". But I'm personally OK with loosing this feature. The feature we're loosing though is this: objects of a category have to support weak references. Is that feature OK to loose? I can't think of a reason we need that feature. William -- william > > Would you agree with me that these tests should simply be removed? > > Best regards, > Simon > > -- > To post to this group, send an email to [email protected] > To unsubscribe from this group, send an email to > [email protected] > For more options, visit this group at > http://groups.google.com/group/sage-devel > URL: http://www.sagemath.org -- William Stein Professor of Mathematics University of Washington http://wstein.org -- To post to this group, send an email to [email protected] To unsubscribe from this group, send an email to [email protected] For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
