#4779: [with patch; needs review] make function for creating random rings and
running automated testing on them
-------------------------+--------------------------------------------------
Reporter: was | Owner: cwitty
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.2.3
Component: misc | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Comment (by mabshoff):
Looks good to me. One tiny issue probably worth fixing: The docstring
says:
{{{
Create a random prime finite field with cardinality at most 10^20.
}}}
But you create
{{{
GF(ZZ.random_element(x=2, y=10**20).next_prime())
}}}
So it is likely, but extremely unlikely that this will happen:
{{{
sage: ZZ.random_element(x=10**20-1, y=10**20).next_prime()
100000000000000000039
}}}
There is an analog issue further down in another docstring IIRC.
Thoughts?
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4779#comment:3>
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
-~----------~----~----~----~------~----~------~--~---
