#9706: New Version of orthogonal Polynomials
-------------------------------+--------------------------------------------
Reporter: maldun | Owner: burcin, maldun
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-5.0
Component: symbolics | Keywords: orthogonal polynomials, symbolics
Author: Stefan Reiterer | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Comment(by maldun):
Replying to [comment:30 kcrisman]:
> I don't have time to review this for a while, but did take a quick look
- thanks for polishing that patch! I don't think we are allowed to import
numpy or scipy like that anymore, but rather have to do it in an
individual function (lest startup times get huge). I don't quite
understand exactly how that works, but anyway such a blanket import
statement probably isn't appropriate, the way I understand what others
have said.
But thanks for giving feedback! I know that this patch isn't easy for
review because the code grew from 650 to about 2300 lines of code. But I'm
happy to get at least some info.
You are right the imports didn't change since I started this ticket and
importing the whole numpy and scipy packages is to much. This isn't a very
good Idea if one thinks about performance either. I changed that now so
that only functions that are really needed are importet. I did this also
for mpmath but the problem with the global import remains.
(see above). Also changed some errors in the discription I missed and
repaired a wrong doctest.
PS: If diffs or more changelogs are needed let me know. I'm keeping track
with git on my machine of the changes.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9706#comment:31>
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.
