Opened 6 years ago

Closed 6 years ago

#21306 closed enhancement (fixed)

Improve documentation for divisor method in linear_code.py

Reported by: ralfaro Owned by:
Priority: trivial Milestone: sage-7.4
Component: coding theory Keywords: sd75
Cc: Merged in:
Authors: Ricardo Alfaro Reviewers: Fangan Dosso
Report Upstream: N/A Work issues:
Branch: 412f945 (Commits, GitHub, GitLab) Commit: 412f9454adbbb98639421613515290308b44a80b
Dependencies: Stopgaps:

GitHub link to the corresponding issue

Description

There was no explanation of what A_i meant. This is the number of codewords of weight i, so we rewrote the documentation to avoid the A_i syntax.

Change History (5)

comment:1 Changed 6 years ago by ralfaro

Branch: → u/ralfaro/fix_docs_divisor

comment:2 Changed 6 years ago by ralfaro

Commit: → 7b1b3ec5b6042c57a1d1af4d17e8c081b900572f
Status: new → needs_review

The current documentation calls for smallest integer when it should be largest. Also, no reference to what A-i was, so we rewrote the definition without using A_i.

comment:3 Changed 6 years ago by git

Commit: 7b1b3ec5b6042c57a1d1af4d17e8c081b900572f → 412f9454adbbb98639421613515290308b44a80b

Branch pushed to git repo; I updated commit sha1. New commits:

​412f945Change and correct documentation for divisor of a code.

comment:4 Changed 6 years ago by fdosso48

Reviewers: → Fangan Dosso
Status: needs_review → positive_review

I tested the method. It works correctly. Also, the documentation compiles well.

Changed to positive review.

Last edited 6 years ago by fdosso48 (previous) (diff)

comment:5 Changed 6 years ago by vbraun

Branch: u/ralfaro/fix_docs_divisor → 412f9454adbbb98639421613515290308b44a80b
Resolution: → fixed
Status: positive_review → closed
Note: See TracTickets for help on using tickets.