#11734: sage_wraps should only read the sources of wrapped functions when
needed.
-------------------------+--------------------------------------------------
Reporter: SimonKing | Owner: jason
Type: defect | Status: needs_review
Priority: blocker | Milestone: sage-4.7.2
Component: misc | Keywords: sage_wraps sources gentoo
startuptime
Work_issues: | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies:
-------------------------+--------------------------------------------------
Comment(by SimonKing):
The tests in sage/misc pass with your patch, and the example from my post
above (the "bugfix") works as well.
If all tests pass, then you patch should be preferred.
It turned out that I am not to blame for the "argspec =
sage_getargspec(wrapped)". The first author of #9976 had introduced an
alternative approach to provide the argument list when building the
documentation -- and I think what we have here is an artefact of that
approach.
Since it was about documentation: Could you please build the documentation
with your patch (I will only be able to do so tomorrow) and look whether
the list of arguments of functions and methods created with @sage_wraps
are correct in the reference manual? See, for example, the
interreduced_basis method of multivariate polynomial ideals.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11734#comment:5>
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.
