#6084: [needs work] Improved p-adic polynomials
---------------------------+------------------------------------------------
Reporter: roed | Owner: roed
Type: enhancement | Status: needs_info
Priority: major | Milestone: sage-4.7.1
Component: padics | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author: David Roe
Merged: | Dependencies:
---------------------------+------------------------------------------------
Changes (by niles):
* status: new => needs_info
Comment:
I think to get this reviewed, some additional organization will be useful
-- here's what I would suggest:
* Give us an introductory description of what this patch accomplishes:
What are the "many different ways of handling precision"? Why was this
needed? What had to be done? What are the main ideas of your
implementation?
* Presumably the patch has been broken into some conceptual pieces -- it
would be useful if you could explain what these are, so that they can be
reviewed somewhat independently if possible.
* I would suggest versioning each separate patch piece (since they will
surely need to be updated), so a naming scheme like `6084_1.n.patch`.
* An always-up-to-date `6084_ALL.patch` will make for easy downloading
and applying (but viewing the separate pieces on trac is very useful for
me)
* Any other ways you can think to break the review down into distinct
pieces will be much appreciated
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6084#comment:15>
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.
