Increase doctest coverage in coding/sd_codes.py to 100%

[benjaminfjones]

As the summary says. As of sage-5.0.beta12 we have:

sd_codes.py
SCORE sd_codes.py: 20% (1 of 5)

Missing documentation:
	 * MS(n):
	 * matA(n):
	 * matId(n):
	 * MS2(n):

---

Apply attachment: trac_12897_sd_codes_coverage.v2.patch to Sage library.

Component: documentation

Keywords: linear code doctest coverage

Author: Benjamin Jones

Reviewer: Punarbasu Purkayastha

Merged: sage-5.1.beta1

Issue created by migration from https://trac.sagemath.org/ticket/12897

[benjaminfjones]

comment:1

After the patch:

> sage -coverage sd_codes.py
----------------------------------------------------------------------
sd_codes.py
SCORE sd_codes.py: 100% (5 of 5)
----------------------------------------------------------------------

> sage -t --optional sd_codes.pysage -t --optional "devel/sage-test/sage/coding/sd_codes.py"
	 [3.0 s]
 
----------------------------------------------------------------------
All tests passed!
Total time for all tests: 3.0 seconds

[benjaminfjones]

Description changed:

--- 
+++ 
@@ -11,3 +11,7 @@
 	 * MS2(n):
 ```
 
+---
+
+Apply [attachment: trac_12897_sd_codes_coverage.patch](https://github.com/sagemath/sage/files/ticket12897/trac_12897_sd_codes_coverage.patch.gz) to Sage library.
+

[ppurka]

comment:3

I have just one comment:

In matA(n) and matId(n) it should be mentioned that the field is GF(2). Also, \delta_ij in matA should be \delta_{ij}.

Other than that, the patch looks good to me.

[benjaminfjones]

Description changed:

--- 
+++ 
@@ -13,5 +13,5 @@
 
 ---
 
-Apply [attachment: trac_12897_sd_codes_coverage.patch](https://github.com/sagemath/sage/files/ticket12897/trac_12897_sd_codes_coverage.patch.gz) to Sage library.
+Apply [attachment: trac_12897_sd_codes_coverage.v2.patch](https://github.com/sagemath/sage-prod/files/10655441/trac_12897_sd_codes_coverage.v2.patch.gz) to Sage library.
 

[benjaminfjones]

comment:5

Ok, good point. The new patch has the changes.

[ppurka]

comment:6

Replying to @benjaminfjones:

Ok, good point. The new patch has the changes.

I think you uploaded the older patch by mistake. :)

[benjaminfjones]

Attachment: trac_12897_sd_codes_coverage.v2.patch.gz