-
Notifications
You must be signed in to change notification settings - Fork 182
Bug in CosetLeadersMatFFE #5923
Copy link
Copy link
Open
Labels
kind: bugIssues describing general bugs, and PRs fixing themIssues describing general bugs, and PRs fixing themkind: bug: wrong resultIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing themIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them
Metadata
Metadata
Assignees
Labels
kind: bugIssues describing general bugs, and PRs fixing themIssues describing general bugs, and PRs fixing themkind: bug: wrong resultIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing themIssues describing bugs that result in mathematically or otherwise wrong results, and PRs fixing them
For matrices with entries in 8-bit fields, this function can return a list of coset leaders w/ missing entries.
A minimal example is
where the 2nd and 5th entries of L are unbound.
I discovered this issue in trying to track down a problem with the covering radius computation for error-correcting codes in the Guava package. It could well be the case that Guava is the only place where this function is used. If so, it would seem to be reasonable to remove this function from GAP and recreate the functionality in Guava. (I've done exactly that as a stop-gap measure for the latest Guava release - but my hack loses the faster approach available for 8-bit fields.) If, on the other hand, other parties are also using this function it should be repaired.