#12753: is_PrimeField import error
----------------------------------------+-----------------------------------
       Reporter:  bhutz                 |         Owner:  bhutz       
           Type:  defect                |        Status:  needs_review
       Priority:  trivial               |     Milestone:  sage-5.0    
      Component:  number fields         |    Resolution:              
       Keywords:  is_PrimeField import  |   Work issues:              
Report Upstream:  N/A                   |     Reviewers:              
        Authors:                        |     Merged in:              
   Dependencies:                        |      Stopgaps:              
----------------------------------------+-----------------------------------

Comment (by davidloeffler):

 From your patch:
 {{{
             INPUT:
                 - R - a ring or field

             OUTPUT:
                 - TRUE - if R is QQ or a finite field GF(p) for p prime.
                 - FALSE - otherwise
 }}}

 This isn't correct ReST formatting (building this will give a warning
 about "unexpected indent"). Moreover Python's booleans are {{{ True /
 False }}}, not {{{ TRUE / FALSE }}} (it's rude to shout :-) So it would be
 better as
 {{{
     INPUT:

     - R - a ring or field

     OUTPUT:

     - True - if R is `\QQ` or a finite field `GF(p)` for p prime.
     - False - otherwise
 }}}
 Note the blank line after the list headings, no extra indentation (not
 needed for bulleted lists), and the backticks for LaTeX formulae
 ({{{`\QQ`}}} is a predefined macro).

 Sorry to torture you so much over one tiny (and deprecated!) function...

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