#8606: floats in exponent do not propagate
--------------------------------+-------------------------------------------
Reporter: zimmerma | Owner: zimmerma
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.4.3
Component: basic arithmetic | Keywords: float, RR
Author: Paul Zimmermann | Upstream: N/A
Reviewer: Burcin Erocal | Merged:
Work_issues: |
--------------------------------+-------------------------------------------
Changes (by burcin):
* status: needs_review => positive_review
Comment:
Replying to [comment:10 zimmerma]:
> thank you Burcin for your review. I have attached a new patch following
your proposal.
> However we still get the same (unhelpful) error message for
{{{5^('a')}}}.
> All doctests still pass.
I was referring to the message `"exponent (=%s) must be an
integer.\nCoerce your numbers to real or complex numbers first."` as
unhelpful. You're right that the message `unsupported operand type(s) for
** or pow(): 'str' and 'str'` can be confusing as well. It just didn't
occur to me since I was staring at the code and expected exactly that.
We could catch the `TypeError` and change the message to "Cannot find a
common domain to perform the operation. Please convert your arguments to
the desired types explicitly." or something similar.
I'm still changing this to positive review since the patch fixes a bug and
a more meaningful error message is just an enhancement.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8606#comment:11>
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 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.
