#17950: make modular_decomposition an optional spkg
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  ncohen                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.6
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:  graph  |    Reviewers:
  theory                 |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  e72ab22fddbe84da5018eaf39f546a80d2133b2a
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  public/17950           |
   Dependencies:         |
-------------------------+-------------------------------------------------

Old description:

> The code on which Sage relies to compute modular decompositions has been
> displaying a stopgap for a while. It seems that the problem has been
> fixed with an update, but it seems to be architecture-dependent (see
> #13744).
>
> This branch puts some distance between Sage and this code: its source
> code had been copy/pasted into Sage's source code (I did that), and this
> is far from being clean. Thus, we turn this into an optional spkg.
>
> An interesting consequence of this move is that we will (at least) be
> able to use the updated version of the code on platforms for which it
> work. It will also make it easier to install and test on other
> architectures. It also means that we stop shipping in our own source code
> a program which has proved to be unreliable at times.
>
> Nathann
>
> the package:
> http://www.steinertriples.fr/ncohen/modular_decomposition-20150313.tar.bz2

New description:

 The code on which Sage relies to compute modular decompositions has been
 displaying a stopgap for a while. It seems that the problem has been fixed
 with an update, but it seems to be architecture-dependent (see #13744).

 This branch puts some distance between Sage and this code: its source code
 had been copy/pasted into Sage's source code (I did that), and this is far
 from being clean. Thus, we turn this into an optional spkg.

 An interesting consequence of this move is that we will (at least) be able
 to use the updated version of the code on platforms for which it work. It
 will also make it easier to install and test on other architectures. It
 also means that we stop shipping in our own source code a program which
 has proved to be unreliable at times.

 Nathann

 the package:
 http://www.steinertriples.fr/ncohen/modular_decomposition-20100607.tar.bz2

--

Comment (by ncohen):

 I changed the branch and the package's name (please remove the previous
 one from your copy of Sage). It ships the .c files that are currently
 contained in Sage (and return wrong results too). You should not
 experience anything that does not already happen, however.

 Nathann
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=e72ab22fddbe84da5018eaf39f546a80d2133b2a
 e72ab22]||{{{trac #17950: Use the old C code instead}}}||

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