#17507: minor error with integral.n()
-------------------------------------+-------------------------------------
Reporter: rws | Owner:
Type: defect | Status: needs_info
Priority: minor | Milestone: sage-6.5
Component: calculus | Resolution:
Keywords: | Merged in:
Authors: Ralf Stephan | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/rws/minor_error_with_integral_n__| 87fb1030cd271861aafe2fb42e58ab03df87e34a
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by ncohen):
* status: needs_review => needs_info
Comment:
Hello !
I think that this branch is ready to go, but I have a couple of questions
to ask (I am new to this code):
- What would you think of an alternative error message: "The function to
be integrated depends on 2 variables (x,y), and so cannot be integrated in
one dimension. Please fix additional variables with the 'params'
argument".
It took me some time to understand what exactly the problem was with the
current error message, because I did not understand what 'parameters'
meant.
- It seems dangerous to me that `params` is a list but not a dictionary.
It seems difficult for the user to know for sure which variable will
receive which value. Isn't that a problem ? Sorry, I really do not know
the code.
If you agree to change the error message to the one I proposed, tell me
and I will write a commit.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/17507#comment:8>
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.
