#17507: minor error with integral.n()
-------------------------------------+-------------------------------------
Reporter: rws | Owner:
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-6.5
Component: calculus | Resolution:
Keywords: | Merged in:
Authors: Ralf Stephan | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/minor_error_with_integral_n__|
55517878d1460022fe3b027322ff847e01440233
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by ncohen):
* status: needs_info => needs_review
* commit: 87fb1030cd271861aafe2fb42e58ab03df87e34a =>
55517878d1460022fe3b027322ff847e01440233
* branch: u/rws/minor_error_with_integral_n__ =>
public/minor_error_with_integral_n__
Comment:
Hello,
> As to the message, please go ahead.
It is in the new commit.
> The `expression.variables()` returns a lexically sorted list but the
`function.arguments()` returns the arguments as they are given. This can
get really confusing:
>
> OTOH, giving a dictionary always involves the user more than a list.
What would you think of that: {{{numerical_integral(x*y,0,1,y=3)}}} ?
The 'dictionary' can also be built from **args.
This being said, perhaps the "most simple and clean" prototype would be to
refuse input with more than one variables, and ask the user to input:
{{{
sage: numerical_integral((x*y).subs(y=4),0,1)
(2.0, 2.220446049250313e-14)
}}}
This would not be very welcoming for beginners, though `:-/`
> So maybe we should at least put a warning in the documentation about
this. If you feel up to it, cvould you please?
I would like to fix it, but I do not know if this 'param' argument is not
used in other places of the symbolic functions, in which case it would
call for a global change. I will write to sage-devel about that in a
second.
I set the ticket in its current state as `needs_review`: if we change the
way to fix variables in this functions we will do so in another ticket.
Thanks,
Nathann
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=4a78f46c35a9f781a0d970fb26d04f3184420a81
4a78f46]||{{{trac #17507: Merged with 6.5.beta5}}}||
||[http://git.sagemath.org/sage.git/commit/?id=55517878d1460022fe3b027322ff847e01440233
5551787]||{{{trac #17507: minor error with integral.n()}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/17507#comment:10>
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.
