#5736: [with patch, needs work] Improve doctest coverage for sage/modular/hecke
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: davidloeffler
Type: defect | Status: assigned
Priority: major | Milestone: sage-4.0
Component: modular forms | Keywords:
---------------------------+------------------------------------------------
Comment(by mabshoff):
Ok, I was surprised this patch did break things again, but there are two
issues here:
An extra space has been inserted when printing the basis, for example:
{{{
File
"/scratch/wstein/build/sage-3.4.2.rc0/devel/sage-0/sage/tests/book_stein_modform.py",
line 70:
: M.basis()
Expected:
({Infinity,0}, {-1/8,0}, {-1/9,0})
Got:
({Infinity, 0}, {-1/8, 0}, {-1/9, 0})
}}}
In addition the printing order for modular_symbol_rep() is reversed:
{{{
age -t -long devel/sage/doc/en/bordeaux_2008/modular_symbols.rst
**********************************************************************
File
"/scratch/wstein/build/sage-3.4.2.rc0/devel/sage-0/doc/en/bordeaux_2008/modular_symbols.rst",
line 124:
sage: a.modular_symbol_rep()
Expected:
36*X^2*{-1/6,0} + 12*X*Y*{-1/6,0} + Y^2*{-1/6,0}
Got:
Y^2*{-1/6, 0} + 12*X*Y*{-1/6, 0} + 36*X^2*{-1/6, 0}
**********************************************************************
}}}
So all those four doctests are simple to fix.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5736#comment:21>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
