#4190: [with patch, needs review] division of number field order elements
doesn't
check for membership
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: davidloeffler
Type: defect | Status: assigned
Priority: major | Milestone: sage-3.1.3
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by davidloeffler):
* status: new => assigned
* summary: division of number field order elements doesn't check for
membership => [with patch, needs review]
division of number field order elements doesn't
check for membership
Comment:
OK, here's a very small patch. Turns out order elements are a cdef class,
so I wrote a _div_c_impl.
I wondered whether the return value should be in the order if that makes
sense, and in the fraction field otherwise, but this slowed everything
down by a factor of 2 according to timeit() so I made it always return
fraction field elements.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4190#comment:4>
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
-~----------~----~----~----~------~----~------~--~---
