Opened 3 years ago

Closed 2 years ago

#19975 closed enhancement (wontfix)

Improve Linear code from parity-check matrix

Reported by: dlucas Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: coding theory Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #19930 Stopgaps:

Description (last modified by dlucas)

As #19930 introduces a new encoder which computes a generator matrix for codes defined from their parity check matrices, it would be nice to update codes.LinearCodeFromCheckMatrix.

It could be renamed, made a class, and thus use the encoder described above.

Change History (8)

comment:1 Changed 3 years ago by dlucas

  • Dependencies set to #19930

comment:2 follow-up: Changed 3 years ago by jfields

Should the description for this ticket read "...computes a generator matrix for codes defined from their generator parity check matrices" ?

comment:3 in reply to: ↑ 2 Changed 3 years ago by dlucas

  • Description modified (diff)
  • Milestone changed from sage-7.1 to sage-7.4

Replying to jfields:

Should the description for this ticket read "...computes a generator matrix for codes defined from their generator parity check matrices" ?

Yes, nice catch! I changed the description.

comment:4 Changed 3 years ago by danielaugot

Hi David

Which encoder are you mentionning in the description of the ticket ?

Are you suggesting to use the encoder codes.encoders.LinearCodeParityCheckEncoder ?

This encoder is deprecated.

Daniel

comment:5 Changed 3 years ago by dlucas

  • Milestone changed from sage-7.4 to sage-duplicate/invalid/wontfix

Actually, the point was to use the ParityCheckEncoder to as a building brick for some codes, instead of having LinearCodeFromCheckMatrix But as you said, ParityCheckEncoder is deprecated, and #21165 deprecates LinearCodeFromCheckMatrix.

So, let's set this ticket to invalid/won't fix.

David

comment:6 Changed 2 years ago by jsrn

  • Status changed from new to needs_review

on the way to closing

comment:7 Changed 2 years ago by jsrn

  • Status changed from needs_review to positive_review

closing as wont-fix

comment:8 Changed 2 years ago by embray

  • Resolution set to wontfix
  • Status changed from positive_review to closed

Closing tickets in the sage-duplicate/invalid/wontfix module with positive_review (i.e. someone has confirmed they should be closed).

Note: See TracTickets for help on using tickets.