#19047: let QQ.some_elements return more than one element
-------------------------------------+-------------------------------------
       Reporter:  dkrenn             |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  categories         |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Daniel Krenn       |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:  u/dkrenn/rings     |       Commit:
  /QQ-some-elements                  |  9b6b1b7ddb9521a2feb664fc45113430031e87f6
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by behackl):

 I'd change the short description of your `some_elements()` method to

 {{{Return some elements of `\QQ`.}}}

 (just for the sake of consistency; see, e.g. `random_element`).

 Other than that: LGTM, the documentation builds and I'm currently testing
 `make ptestlong`.

--
Ticket URL: <http://trac.sagemath.org/ticket/19047#comment:5>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to