#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: | 086af29b0632d16e097da3f042a7656417fa517c
u/jaanos/improve_efficiency_of_calling_gap_functions| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by tscrim):
The other ticket is #19607, and I don't have the technical knowledge of
that part of Sage to do a good review of that.
However I have figured out the problem with comment:23, and it is
essentially what you wrote in comment:28. I added the following right at
the offending doctest:
{{{
sage: x.str() # This is why it is returning True
''
sage: x._name # Because this is making use the file
'a_variable_with_a_name_so_very_very_very_long_that_even_by_itself_will_make_expect_use_a_file'
sage: gap(x._name) # At least the variable is stored and retrieved
correctly...
false
}}}
So we could likely to suppress this by doing another test with a shorter
name being passed. I'm guessing we are somehow recycling variable names in
the session, and we either shouldn't or will actually have to handle long
variable names before this can get merged (possibly both too). I'm okay
with suppressing with a note in the doctest referencing this issue. Anyone
familiar with the interface have a better idea of what is going on?
--
Ticket URL: <http://trac.sagemath.org/ticket/19585#comment:32>
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.
