#5520: [with patch; needs review] implement Pizer's algorithm for computing
Brandt
Modules and Brandt Matrices
---------------------------+------------------------------------------------
Reporter: was | Owner: craigcitro
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.1
Component: modular forms | Keywords:
---------------------------+------------------------------------------------
Comment(by was):
WAIT -- I tested your trac_5520-rebase-ref1.patch and
{{{
sage: d = BrandtModule(17,19).decomposition()
*** division by zero
/scratch/wstein/build/sage-3.4.1.alpha0/local/bin/sage-sage: line 197:
14046 Aborted sage-ipython "$@" -i
wst...@sage:/scratch/wstein/build/sage-3.4.1.alpha0$
}}}
But then if I revert it everything works fine again. So I do *not* agree
with it.
If I revert *only* the change you made involving mkfrac and gdiv, then
everything works fine.
I'm posting a patch "trac_5520-rebase3.4.1.alpha0-part3.patch" that does
*not* make the mkfrac change, and also has a comment about gotomat. I
have run the full test suite on all of sage with that patch applied and
everything passes.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5520#comment:22>
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
-~----------~----~----~----~------~----~------~--~---
