#5787: [with second patch, needs review] Improve doctest coverage for
sage/modular/hecke (continued)
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: davidloeffler
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-4.0
Component: modular forms | Keywords: doctests
---------------------------+------------------------------------------------
Comment(by davidloeffler):
At a glance, I can spot two small problems: firstly, the new doctest for
{{{__cmp__}}} in submodule.py fails on my machine; secondly, Craig's patch
changes the intersection method (so it works for modular forms spaces as
well as modular symbols) but I also did the same in #4357, so those are
going to conflict.
I'm not entirely sure why the first failure happens: although
{{{__cmp__}}} doctests are always irritatingly machine-dependent when
comparing objects of different types, the comparison order for submodules
of a common ambient module should be consistent, surely? I'm changing this
back to "needs work" until we can get to the bottom of this. Once that's
sorted I'll do a rebased version that combines this with #4357.
David
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5787#comment:5>
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
-~----------~----~----~----~------~----~------~--~---
