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 )
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
- Dependencies set to #19930
comment:2 follow-up: ↓ 3 Changed 3 years ago by
comment:3 in reply to: ↑ 2 Changed 3 years ago by
- 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
generatorparity check matrices" ?
Yes, nice catch! I changed the description.
comment:4 Changed 3 years ago by
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
- 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:7 Changed 2 years ago by
- Status changed from needs_review to positive_review
closing as wont-fix
comment:8 Changed 2 years ago by
- 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).
Should the description for this ticket read "...computes a generator matrix for codes defined from their
generatorparity check matrices" ?