#4276: [with patch, needs review] move number fields to new coercion, implement
embeddings
----------------------+-----------------------------------------------------
Reporter: robertwb | Owner: robertwb
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.1
Component: coercion | Resolution:
Keywords: |
----------------------+-----------------------------------------------------
Comment (by davidloeffler):
I downloaded these patches not so much in order to review them but because
I wanted to see if they would conflict with any changes I might make in
order to fix #4193.
Anyway, they apply and build fine (eventually) under 3.1.4. Testing
number_field/ passes except for a minor numerical noise issue:
{{{
**************************************************
File "/home/david/sage-current/tmp/number_field_morphisms.py", line 223:
sage: closest_root(x^3-1, CDF.0)
Expected:
-0.500000000000000 + 0.866025403784439*I
Got:
-0.500000000000000 + 0.866025403784438*I
**********************************************************************
}}}
But I can't really do much to test the new functionality, since I'm not
sure what it's supposed to do. You say above "here is a brief example
(though more can be found in the documentation)" but I can't see any; all
of the various number field constructors now accept an embedding parameter
but there's no explanation or tests in the docstrings. Can we have a bit
of demystification?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4276#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
-~----------~----~----~----~------~----~------~--~---
