Opened 15 years ago

Closed 15 years ago

#1810 closed enhancement (fixed)

[with patch, positive review] refactoring to improve finite field reference manual

Reported by: Martin Albrecht Owned by: tba
Priority: major Milestone: sage-2.10.3
Component: documentation Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

The patch removes FiniteField_prime_modn from finite_field.py because it was odd that this implementation was the only showing up in the reference manual. Also, GF is now defined in rings.all rather than in rings.finite_field to avoid that the documentation for it shows up twice. Finally, a more verbose description of the finite field module is given at the top of the finite_field.py file and some doctests were added to FiniteField_prime_modn.

Attachments (5)

finite_field_refactor_doc.patch (46.8 KB) - added by Martin Albrecht 15 years ago.
finite_field_refactor_doc.2.patch (38.0 KB) - added by Martin Albrecht 15 years ago.
rebased against 2.10.2
finite_field_refactor_doc.3.patch (36.9 KB) - added by Martin Albrecht 15 years ago.
rebased against 2.10.3.alpha3
gfp.patch (6.9 KB) - added by Martin Albrecht 15 years ago.
add this on top of the 2.10.3.alpha0 patch, I forgot to hg add a file
guava_fix.patch (1.3 KB) - added by Martin Albrecht 15 years ago.

Download all attachments as: .zip

Change History (11)

Changed 15 years ago by Martin Albrecht

comment:1 Changed 15 years ago by Mike Hansen

I get the following on a clean 2.10.2 install:

applying /home/mike/Desktop/finite_field_refactor_doc.patch
patching file sage/rings/finite_field.py
Hunk #1 succeeded at 1 with fuzz 2 (offset -2 lines).
Hunk #5 FAILED at 147
1 out of 13 hunks FAILED -- saving rejects to file sage/rings/finite_field.py.rej
abort: unable to find sage/rings/padics/eisenstein_extension_generic_element.py or sage/rings/padics/eisenstein_extension_generic_element.py for patching

comment:2 Changed 15 years ago by Martin Albrecht

I am rebasing right now. make test is running and looking good. Then, I'm afraid I'll have to check if I can apply the patch against 2.10.3.alpha0.

Changed 15 years ago by Martin Albrecht

rebased against 2.10.2

Changed 15 years ago by Martin Albrecht

rebased against 2.10.3.alpha3

Changed 15 years ago by Martin Albrecht

Attachment: gfp.patch added

add this on top of the 2.10.3.alpha0 patch, I forgot to hg add a file

comment:3 Changed 15 years ago by Martin Albrecht

Summary: [with patch] refactoring to improve finite field reference manual[with patch, needs review] refactoring to improve finite field reference manual

Add gfp.patch on top of finite_field_refactor_doc.3.patch, I forgot to hg add finite_field_prime_modn.py

Changed 15 years ago by Martin Albrecht

Attachment: guava_fix.patch added

comment:4 Changed 15 years ago by Martin Albrecht

guava_fix.patch needs also to be applied after the other patches

comment:5 Changed 15 years ago by Mike Hansen

Summary: [with patch, needs review] refactoring to improve finite field reference manual[with patch, positive review] refactoring to improve finite field reference manual

Applies cleanly to 2.10.3.alpha0 and passes tests for me.

To apply the patch, do the following,

1) Apply finite_field_refactor_doc.3.patch

2) Apply gfp.patch

3) hg add finite_field_prime_modn.py

4) Apply guava_fix.patch

comment:6 Changed 15 years ago by Michael Abshoff

Resolution: fixed
Status: newclosed

Merged finite_field_refactor_doc.3.patch, gfp.patch and guava_fix.patch in Sage 2.10.3.rc1

Note: See TracTickets for help on using tickets.