#3118: [with new patch, needs review] update LCM (easy-to-fix buglet)
------------------------------+---------------------------------------------
Reporter: was | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.2
Component: basic arithmetic | Resolution:
Keywords: |
------------------------------+---------------------------------------------
Changes (by cremona):
* summary: [with patches, needs review] update LCM (easy-to-fix buglet)
=> [with new patch, needs review] update LCM
(easy-to-fix buglet)
Comment:
My patch applies _instead_ of the previous two. It implements Paul's
suggested behaviour.
There's one problem left (I tested all of sage.rings and had to make one
change in the multiploynomial polynomial code which actually used the
integers=True flag which is now redundant). But:
{{{
sage: P.<x,y,z> = ZZ[]
sage: gcd(2*(x+y),3*y)
2
}}}
which of course should give 1. This leads to one doctest failure, but I
cannot track it down at the moment.
I (or someone) should also doctest all the rest of Sage as there are
ceratinly places where gcd/lcm are used outside of sage/rings.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3118#comment:2>
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
-~----------~----~----~----~------~----~------~--~---
