#5409: [with patch; positive review] rewrite quaternion algebras -- the current
implementation isn't sufficiently good
-------------------------+--------------------------------------------------
Reporter: was | Owner: tbd
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4
Component: algebra | Keywords:
-------------------------+--------------------------------------------------
Comment(by jvoight):
Great, so then this gets a positive review from me, too. I tried a few
things out and the multiplication, and this last doctest might be useful
(to test cpdef RingElement _mul_), but maybe there's enough already.
{{{
sage: K.<x> = QQ[]
sage: Q_ab = QuaternionAlgebra(Frac(K), 17, -3)
sage: Q = QuaternionAlgebra(17, -3)
sage: for t in xrange(100):
sage: v1 = [ZZ.random_element() for i in range(4)]
sage: v2 = [ZZ.random_element() for i in range(4)]
sage: if list(Q(v1)*Q(v2)) <> list(Q_ab(v1)*Q_ab(v2)):
sage: print "Multiplication is inconsistent", v1, v2
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5409#comment:11>
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
-~----------~----~----~----~------~----~------~--~---
