#17507: minor error with integral.n()
-------------------------------------+-------------------------------------
Reporter: rws | Owner:
Type: defect | Status: needs_work
Priority: minor | Milestone: sage-6.5
Component: calculus | Resolution:
Keywords: | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/rws/minor_error_with_integral_n__| 87fb1030cd271861aafe2fb42e58ab03df87e34a
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by rws):
* commit: => 87fb1030cd271861aafe2fb42e58ab03df87e34a
Comment:
I cannot find documentation about `InputError`, so `ValueError` is fine.
Looking at the trigger condition however
{{{
if len(params) + 1 != len(vars):
raise ValueError, "Integrand has wrong number of
parameters"
}}}
suggests this is meant to catch problems with the `params` argument to
`numerical_integral`. This is not given by the calling `_evalf_` so maybe
that should be checked upstream too to honor untold preconditions of
`numerical_integral`.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=87fb1030cd271861aafe2fb42e58ab03df87e34a
87fb103]||{{{17507: change error msg, add doctest}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/17507#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.
