#5834: [with patch, neews review] Improvements to quadratic_forms/extras/py
-----------------------------+----------------------------------------------
Reporter: cremona | Owner: justin
Type: enhancement | Status: new
Priority: minor | Milestone: sage-4.0
Component: quadratic forms | Keywords: quadratic forms
-----------------------------+----------------------------------------------
Comment(by cremona):
Replying to [comment:2 mabshoff]:
> Gonzalo, Jon,
>
> this is your area of expertise. Can either one of you review?
>
> And by the way: "3. I removed random_int_upto(n) since it does the same
as ZZ.random_element(n)." is #5888. As it turned out that if you wanted
some random number larger than 2^53 you ended up with loads of integers
that had a common divisor of 2^47 causing some trouble to Bill Hart today
when he looked at the HNF code in Sage to compare some code he has written
in FLINT.
>
> Cheers,
>
> Michael
Moral: do not reinvent the wheel, especially if your version is square.
Use the wheels provided!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5834#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
-~----------~----~----~----~------~----~------~--~---
