#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 tornaria):
Replying to [comment:22 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$
> }}}
Auch! Sorry for my sloppiness... It looked innocent and the difference in
runtime looked interesting. I guess the same issue is behind the need for
{{{gtomat}}}.
Having a concrete example where sage crashes without this (gdiv or gtomat)
is useful. Maybe it has to do with the layout of the pari objects in the
stack --- as some of the copy/restack functions assume some specific order
on recursive types. Most of the g* functions ensure returned objects are
in the correct layout, but the internal functions don't.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5520#comment:24>
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
-~----------~----~----~----~------~----~------~--~---
