#6371: Implement Riemann theta functions
-----------------------------------------------+----------------------------
Reporter: ncalexan | Owner: cswiercz
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-4.8
Component: numerical | Keywords: riemann theta
klein
Work_issues: | Upstream: N/A
Reviewer: Marco Streng, Bernard Deconinck | Author: Nick
Alexander, Chris Swierczewski
Merged: | Dependencies:
-----------------------------------------------+----------------------------
Comment(by mstreng):
I don't know much about {{{BuiltinFunction}}}, but I suspect that it isn't
meant for what you are trying to do. All the examples that I know of
{{{BuiltinFunction}}} are single fixed functions, independent of any
parameters, such as cos, min, max, {{{IndefiniteIntegral}}}, Gamma,
dirac_delta, etc.
Your objects however depend on a parameter Omega, and are functions only
of z. They are maps {{{theta(Omega): CC^g --> CC : z |--> theta(z)}}}.
This will not be a "built-in function of Sage" for every Omega, so it does
not sound to me like this should be a {{{BuiltinFunction}}} object. Maybe
the memory leak is caused by this.
The function {{{theta: CC^g x H_g --> CC : (z, Omega) |--> theta(z,
Omega)}}} definitely make sense as a built-in function, but not a function
{{{theta(Omega) : CC^g --> C}}} that depends on Omega. In other words, if
you have a function for which Omega is not a parameter but an actual
variable, then that would make a good built-in function.
ps. Could you make sure you break the lines in the patch at 79 characters?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6371#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.
