#12767: Clean up dead links in sage/rings/number_field documentation
----------------------------------+-----------------------------------------
       Reporter:  davidloeffler   |         Owner:  davidloeffler               
      
           Type:  defect          |        Status:  needs_review                
      
       Priority:  major           |     Milestone:  sage-5.1                    
      
      Component:  number fields   |    Resolution:                              
      
       Keywords:  sd40.5          |   Work issues:                              
      
Report Upstream:  N/A             |     Reviewers:  Karl-Dieter Crisman, 
William Stein
        Authors:  David Loeffler  |     Merged in:                              
      
   Dependencies:                  |      Stopgaps:                              
      
----------------------------------+-----------------------------------------

Comment (by was):

 Hi,

 The new code looks fine, though the new v option should be documented in
 the docstring:

 {{{

         INPUT:

         -  ``v`` - (default: None) None, a prime, or a list of
            primes.

            - if v is None, return the maximal order.

            - if v is a prime, return an order that is p-maximal.

            - if v is a list, return an order that is maximal at each
              prime in the list v.
 }}}

 So if somebody adds the above to the docstring, then we should give this a
 positive review.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12767#comment:6>
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.

Reply via email to