Opened 6 years ago

Closed 6 years ago

#22110 closed defect (fixed)

downgrade modular_decomposition package to experimental

Reported by: Dima Pasechnik Owned by:
Priority: critical Milestone: sage-7.5
Component: packages: optional Keywords:
Cc: Merged in:
Authors: Dima Pasechnik Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: 5b4fc8b (Commits, GitHub, GitLab) Commit: 5b4fc8bc33dd2bcfdf2027941565d62bf4b541f1
Dependencies: Stopgaps:

Status badges

Description (last modified by Jeroen Demeyer)

The optional package modular_decomposition either renders Sage broken, or gives a lot of wrong results.

Downgrading it to experimental would at least give more warnings to potential users.

Change History (8)

comment:1 Changed 6 years ago by Dima Pasechnik

Description: modified (diff)

comment:2 Changed 6 years ago by Jeroen Demeyer

Description: modified (diff)
Summary: downgrade optional packages causing errorsdowngrade modular_decomposition package to experimental

Can you elaborate what is the problem with this package?

comment:3 Changed 6 years ago by Dima Pasechnik

It gives wrong results on many inputs. Moreover it is abandoned by the original upstream author(s). See e.g. here. The latest example.

There is also a stopgap, and I think #17950 should have made it experimental already.

PS. There is no implementation available to hook up (neither Java nor Perl implementations are useful for us). Would make a good student/GSoC project, I suppose...

Last edited 6 years ago by Dima Pasechnik (previous) (diff)

comment:4 Changed 6 years ago by Dima Pasechnik

Branch: u/dimpase/moddecexp
Commit: 5b4fc8bc33dd2bcfdf2027941565d62bf4b541f1
Priority: majorcritical
Status: newneeds_review

New commits:

5b4fc8bdowngrade to exp

comment:5 Changed 6 years ago by Jeroen Demeyer

Personally, I think the stopgap is a sufficient solution. But I'm not against the downgrade to experimental.

Let me just check if it still builds properly...

comment:6 Changed 6 years ago by Jeroen Demeyer

Reviewers: Jeroen Demeyer
Status: needs_reviewpositive_review

Author name...

comment:7 Changed 6 years ago by Dima Pasechnik

Authors: Dima Pasechnik

comment:8 Changed 6 years ago by Volker Braun

Branch: u/dimpase/moddecexp5b4fc8bc33dd2bcfdf2027941565d62bf4b541f1
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.