#9054: create a class for basic function_field arithmetic for Sage
-----------------------------------------------------------------------------------------------+
Reporter: was
| Owner: was
Type: enhancement
| Status: needs_work
Priority: major
| Milestone: sage-pending
Component: algebra
| Resolution:
Keywords:
| Work_issues:
Upstream: N/A
| Reviewer: Maarten Derickx, Julian Rueth
Author: William Stein, Robert Bradshaw, Maarten Derickx, Moritz
Minzlaff, Julian Rueth | Merged:
Dependencies: #9094, #11751, #9138
|
-----------------------------------------------------------------------------------------------+
Changes (by jdemeyer):
* status: positive_review => needs_work
Comment:
The commit messages of the patches could be cleaned up:
1. [attachment:trac_9054-julian-combined.patch]: the commit message
''starts'' with {{{* * *}}} instead of something useful.
1. [attachment:trac_9054-review.patch] has no proper commit message.
This improper commit message is also in [attachment:trac_9054-all-
parts.patch], which should be fixed.
1. [attachment:trac_9054-all-parts.patch] "{{{contains parts 1-12,
marteen's additions and final doctest fixes}}}" makes no sense if you
don't know this ticket, the message should makes sense on its own. The
word "function field" does not even appear in the message of this patch!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9054#comment:80>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.