#19585: Improve efficiency of calling GAP functions
-------------------------------------+-------------------------------------
Reporter: jaanos | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.10
Component: interfaces | Resolution:
Keywords: GAP functions | Merged in:
interface | Reviewers: Travis Scrimshaw
Authors: Janoš Vidali | Work issues:
Report Upstream: N/A | Commit:
Branch: | 1408680266b88585fa6cb15780e8c8dfce415105
u/jaanos/improve_calling_gap_function-19585| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by jaanos):
* commit: f59926575333e34976901f92c1f070e0c168a0b9 =>
1408680266b88585fa6cb15780e8c8dfce415105
Comment:
Hello again!
OK, I added the check for overlong function calls, and it doesn't seem to
affect runnning times. Such calls would probably never happen, but we can
simulate one with something like
{{{
#!python
gap.function_call("ConjugacyClassesSubgroups",
sage.interfaces.gap.GapElement(gap, 'SymmetricGroup(2)', name =
'a_variable_with_a_very_very_very_long_name'))
}}}
Without the latest commit, this would return a wrong result.
I have also experimented with also making the check for identical objects
in the same `eval` if the function call was short enough, but this
invariably caused slowdowns.
Janoš
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=1408680266b88585fa6cb15780e8c8dfce415105
1408680]||{{{Check if command is too long for combining}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/19585#comment:20>
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.
