On 10/21/07, William Stein <[EMAIL PROTECTED]> wrote: > > I think I know what's going on. > > If you install the Gap optional packages, then reset the > workspace you get the behavior I see. If you don't install > the optional packages you don't. Thus one of the packages > changes the print behavior of Gap. > > Indeed, here it is: > > gap> LoadPackage("hap"); > Loading HAP 1.7.5gamma ... > true > gap> PolynomialRing(Rationals,2); > PolynomialRing(..., [ x, x_2 ]) > > My solution will be to slightly change gap.py, so it does *not* > autoload hap into the workspace, then change the doctests > back. Note: In the old gap.py, hap wasn't loaded into the > workspace automatically -- I thought this was a mistake, but > evidently the reason is because hap slightly modifies > how certain things work with gap polynomial rings. Annoying.
I believe this is *illegal* (ie, does not conform to GAP package requirements) and have reported it. I'll let you know when I hear more. If needed, I'll create a new gap_packages*.spkg. Sorry about the hassle. > > OK, I've pushed this out, so hg_sage.pull() will get you the > fixed version of gap.py. > > William > > On 10/21/07, Jaap Spies <[EMAIL PROTECTED]> wrote: > > > > William Stein wrote: > > > On 10/21/07, Jaap Spies <[EMAIL PROTECTED]> wrote: > > > > >> There seems to be some typos in the test of gap.py > > > > > > These are not typos. The behavior of gap-4.4.10 changed from that > > > of gap-4.4.9, and the doctests reflect that change. Evidently for > > > some reason your gap didn't get upgraded. Try > > > > > > sage: !gap > > > > > > > This is on a fresh install (but dito with the upgrade): > > > > GAP4, Version: 4.4.10 of 02-Oct-2007, i686-pc-linux-gnu-gcc > > gap> > > > > > to see what version of gap you now have. > > > > > > Also, you may need to do > > > sage: gap_reset_workspace() > > > > That does not help. Same test failures. > > > > Jaap > > > > > > >> > > >> [EMAIL PROTECTED] sage-2.8.8]$ ./sage -t > > >> devel/sage-main/sage/interfaces/gap.py > > >> sage -t devel/sage-main/sage/interfaces/gap.py > > >> ********************************************************************** > > >> File "gap.py", line 69: > > >> sage: R = gap.PolynomialRing('Rationals', 2); R > > >> Expected: > > >> PolynomialRing(..., [ x, x_2 ]) > > >> Got: > > >> PolynomialRing(..., [ x_1, x_2 ]) > > >> ********************************************************************** > > >> File "gap.py", line 71: > > >> sage: I = R.IndeterminatesOfPolynomialRing(); I > > >> Expected: > > >> [ x, x_2 ] > > >> Got: > > >> [ x_1, x_2 ] > > >> ********************************************************************** > > >> File "gap.py", line 86: > > >> sage: f = gap(str(g)); f > > >> Expected: > > >> -x^5+x_2^2 > > >> Got: > > >> -x_1^5+x_2^2 > > >> ********************************************************************** > > >> 1 items had failures: > > >> 3 of 22 in __main__.example_0 > > >> ***Test Failed*** 3 failures. > > >> For whitespace errors, see the file .doctest_gap.py > > >> [2.4 s] > > >> exit code: 256 > > >> > > >> ---------------------------------------------------------------------- > > >> The following tests failed: > > >> > > >> > > >> sage -t devel/sage-main/sage/interfaces/gap.py > > >> Total time for all tests: 2.4 seconds > > >> [EMAIL PROTECTED] sage-2.8.8]$ > > >> > > >> > > > > > > > > > > > > > > > > > > -- > William Stein > Associate Professor of Mathematics > University of Washington > http://wstein.org > > > > --~--~---------~--~----~------------~-------~--~----~ 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/ -~----------~----~----~----~------~----~------~--~---