#6059: [with patch; with positive review] speed regresion in hilbert_symbol
after
#5834
---------------------------+------------------------------------------------
Reporter: tornaria | Owner: tornaria
Type: defect | Status: new
Priority: major | Milestone: sage-4.0
Component: number theory | Keywords: regression
---------------------------+------------------------------------------------
Comment(by cremona):
Looks good to me. Sorry about the regression which was my fault. I
needed an integer in the same square class as a (and b) and did the
obvious thing without thinking about the conseqences regarding
factorization.
Would it be better to check for integrality, since if a and b are integral
then using ZZ(a), ZZ(b) would save the conversion to rational and back?
i.e. do something like try: a=ZZ(a); b=ZZ(b) first.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6059#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
-~----------~----~----~----~------~----~------~--~---
