#17179: TIDES interface should convert exact parameters to floating points.
-------------------------------------+-------------------------------------
Reporter: mmarco | Owner:
Type: defect | Status: needs_work
Priority: major | Milestone: sage-6.4
Component: interfaces | Resolution:
Keywords: | Merged in:
Authors: Miguel Marco | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/mmarco/ticket/17179 | fb97dfead7efb0375969dc048cf2fe8f752564e8
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Miguel Marco', 'oldvalue': ''}):
* cc: iMark (added)
* author: => Miguel Marco
Comment:
`RR(SR(foo))` is used because strings like `"pi"` or `"-3/2"` are not
accepted by the element constructor of real fields. They are accepted by
the element constructor of SR though. So it was the best way i could find
to create a RealField element from these strings.
The real value of log(10)/log(2) is between 3.3219 and 3.322, so i
prefeared to be conservative in this aspect.
This commit addresses mostly all your comments, but i think that the code
could be greatly improved if we pass the domain to fast_callable, which
would directly convert all constants, avoiding us the work of converting
the exact expressions to the strings that represent them in floating
point.
Besides, there is also some room for improvement in the performance of
`remove_repeated` .
I will work on that, but i am not sure if it should be done in this ticket
(maybe we should change the title?)
--
Ticket URL: <http://trac.sagemath.org/ticket/17179#comment:6>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
