#19585: Improve efficiency of calling GAP functions
-------------------------------------+-------------------------------------
       Reporter:  jaanos             |        Owner:
           Type:  enhancement        |       Status:  needs_work
       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_calling_gap_function-19585|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by jaanos):

 * status:  needs_review => needs_work


Comment:

 I've added the doctest. However, now doctesting fails on a seemingly
 unrelated issue:
 {{{
 sage -t src/sage/interfaces/gap.py
 **********************************************************************
 File "src/sage/interfaces/gap.py", line 1001, in
 sage.interfaces.gap.GapElement_generic.bool
 Failed example:
     gap('false').bool()
 Expected:
     False
 Got:
     True
 **********************************************************************
 }}}
 This doesn't happen on the develop branch, and not even with the last
 commit. It seems that the failure has to do with the overlong variable
 name being reused:
 {{{
 #!python
 sage: gap.function_call("ConjugacyClassesSubgroups",
 sage.interfaces.gap.GapElement(gap, 'SymmetricGroup(2)', name =
 
'a_variable_with_a_name_so_very_very_very_long_that_even_by_itself_will_make_expect_use_a_file'))
 [ ConjugacyClassSubgroups(SymmetricGroup( [ 1 .. 2 ] ),Group( [ () ] )),
   ConjugacyClassSubgroups(SymmetricGroup( [ 1 .. 2 ] ),SymmetricGroup( [ 1
 .. 2 ] )) ]
 sage: F = gap('false')
 sage: F.bool()
 True
 sage: F.name()
 
'a_variable_with_a_name_so_very_very_very_long_that_even_by_itself_will_make_expect_use_a_file'
 sage: F = gap('false')
 sage: F.bool()
 False
 sage: F.name()
 '$sage1'
 }}}
 Will look into it.

--
Ticket URL: <http://trac.sagemath.org/ticket/19585#comment:23>
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.

Reply via email to