#3337: [with spkg, needs work] Upgrade gap-guava to new 4.4.12/3.9 release
-------------------------+--------------------------------------------------
Reporter: tabbott | Owner: mabshoff
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.1
Component: packages | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Changes (by mabshoff):
* summary: [with spkg, needs review] Upgrade gap-guava to new 4.4.12/3.9
release => [with spkg, needs work] Upgrade gap-
guava to new 4.4.12/3.9 release
Comment:
I am not sure if we need to make all these doctest #random since IIRC
cwitty initializes the GAP rng for each doctest.
Carl: Is this correct or was this a plan to make GAP's rng behave nicely
with the doctesting framework?
And this change also makes me very uncomfortable:
{{{
6294 6294 sage: C = graphs.CubeGraph(4)
6295 6295 sage: G = C.automorphism_group()
6296 6296 sage: M = G.character_table()
6297 sage: M.determinant()
6297 sage: M.determinant() # random sign (only abs.
value is well-defined)
6298 6298 -712483534798848
6299 6299 sage: G.order()
6300 6300 384
}}}
Why is the sign of the determinant of a character table random? In that
case the doctest should check for the absolute value of the determinant
since #random would hide bugs here, i.e. no one would notice if the
determinant would change more than its sign. I would also appreciate
adding that info the the docstring of the character table to make people
aware of the problem.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3337#comment:36>
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
-~----------~----~----~----~------~----~------~--~---
