#14909: Gap package HAP does not load
--------------------------------------+---------------------------
       Reporter:  vbraun              |         Owner:
           Type:  defect              |        Status:  needs_info
       Priority:  major               |     Milestone:  sage-5.12
      Component:  packages: optional  |    Resolution:
       Keywords:                      |     Merged in:
        Authors:  Volker Braun        |     Reviewers:
Report Upstream:  N/A                 |   Work issues:
         Branch:  u/niles/14909       |  Dependencies:
       Stopgaps:                      |
--------------------------------------+---------------------------
Changes (by niles):

 * status:  needs_review => needs_info
 * branch:   => u/niles/14909


Comment:

 I had trouble with this at first, and then I realized one needs to use
 `--optional=sage,gap_packages`; without `sage` there, _only_ the #optional
 lines are run, which makes a lot of tests fail.  (The tests above still
 fail with `--optional=sage,gap_packages` though.)

 I like the idea of adding tests for the gap packages, but maybe the test
 in this patch is too verbose.  It would fail, for example, if the user had
 some extra gap packages installed manually, even if they load correctly.
 Thoughts on changing it to something like the following?

 {{{
     sage: from sage.tests.gap_packages import all_installed_packages
     sage: for name in all_installed_packages():       # optional:
 gap_packages
     ....:     stat = libgap.eval('LoadPackage("{0}")'.format(name))
     ....:     message = "{2} : {0: <10} got '{1}' when loading."
     ....:     if str(stat) == 'true':
     ....:         prefix = ' '
     ....:     else:
     ....:         prefix = 'x'
     ....:         print(message.format(name,stat,prefix))
     x : HAPcryst   got 'fail' when loading.
 }}}

 In fact, maybe I have attached a git branch with this version of the test!
 (Still figuring out the git workflow . . .)


 And another issue:  Why do we have

 {{{
 $ ./sage -t --optional=sage,gap_packages src/sage/tests/gap_packages.py
 Running doctests with ID 2013-07-19-15-20-57-fee4b5ac.
 ...
 Total time for all tests: 7.3 seconds
     cpu time: 7.2 seconds
     cumulative wall time: 7.2 seconds
 }}}

 and

 {{{
 sage: %time import sage.tests.gap_packages
 CPU times: user 5.55 s, sys: 0.05 s, total: 5.61 s
 Wall time: 5.61 s
 }}}

 The rest of the things in sage/tests seem to import instantly . . . is
 there something I'm doing wrong which makes this one so slow?

--
Ticket URL: <http://trac.sagemath.org/ticket/14909#comment:4>
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/groups/opt_out.


Reply via email to