#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:                      |  Dependencies:
       Stopgaps:                      |
--------------------------------------+---------------------------
Changes (by niles):

 * status:  needs_review => needs_info


Comment:

 Indeed, I have verified that all of the import time comes from importing
 libgap.  The new version of the patch looks good, and in particular the
 new test for loading packages is much better.

 Now checking (long) doctests, I find failures in 3 files:

 {{{
 ----------------------------------------------------------------------
 sage -t --long sage/combinat/designs/incidence_structures.py  # 2 doctests
 failed
 sage -t --long sage/combinat/designs/design_catalog.py  # 1 doctest failed
 sage -t --long sage/coding/linear_code.py  # 8 doctests failed
 ----------------------------------------------------------------------
 }}}


 The first two seem to be introduced by #14499; fixing these is now #14950




 The failures in coding/linear_code are all in optional `gap_packages`
 doctests, and they all have to do with the GUAVA package.  My guess is
 that these come from a version change or something similar, because guava
 does load, and some guava-dependent doctests do work, such as

 {{{
 sage: C = HammingCode(5,GF(2))
 sage: C.covering_radius()  # optional - gap_packages (Guava package)
 1
 }}}

 {{{
 sage: C = HammingCode(3,GF(2))
 sage: MS = MatrixSpace(GF(2),1,7)
 sage: F = GF(2); a = F.gen()
 sage: v1 = [a,a,F(0),a,a,F(0),a]
 sage: C.decode(v1,algorithm="guava")  # optional - gap_packages (Guava
 package)
 }}}

 One failure comes from

 {{{
 sage: print bounds_minimum_distance(10,5,GF(2)) # optional - gap_packages
 (Guava package)
 }}}

 This seems to give the same record as expected (see
 
[http://www.sagemath.org/doc/reference/coding/sage/coding/linear_code.html#sage.coding.linear_code.bounds_minimum_distance
 here]), but the entries aren't printed in the correct order, hence causing
 a doctest failure.


 Another failure comes from

 {{{
 sage: MS = MatrixSpace(GF(2),4,7)
 sage: G =
 MS([[1,1,1,0,0,0,0],[1,0,0,1,1,0,0],[0,1,0,1,0,1,0],[1,1,0,1,0,0,1]])
 sage: C = LinearCode(G)
 sage: C.spectrum()
 [1, 0, 0, 7, 7, 0, 0, 1]
 sage: C.spectrum(algorithm="leon")     # optional - gap_packages (Guava
 package)
 sh: /home/johnson.5320/sage/local/gap/latest/pkg/guava-3.12/bin/wtdist: No
 such file or directory
 ---------------------------------------------------------------------------
 RuntimeError                              Traceback (most recent call
 last)
 ...
 RuntimeError: Problem calling Leon's wtdist program. Install
 gap_packages*.spkg and run './configure ../..; make'.
 }}}

 For this last error, I see that the file

 `SAGE_ROOT/local/gap/latest/pkg/guava-3.12/bin/wtdist`

 does not exist, but

 `SAGE_ROOT/local/gap/latest/pkg/guava-3.12/bin/x86_64-unknown-linux-gnu-
 gcc-default64/wtdist`

 does exist.  So probably this is an error coming from rearranging files.
 I haven't checked the other failures here; I am inclined to believe that
 fixing them belongs on a new ticket, but I'm not sure.

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