I think I wrote the ordinal_str function, for the output of certain
messages related to roots of unity.  Clearly I did not do a perfect
job:  it uses 'st' for 1 mod 10 except for 11, but I think that should
be: 'st' for 1 mod 10 except 'th' for 11 mod 100.  Similarly for 2 and
3 mod 10.

I just saw that you have already made a patch for this, so I'll review
it right now!

John

On Aug 29, 12:32 pm, Simon King <simon.k...@nuigalway.ie> wrote:
> Hi!
>
> This is a related subject:
>
> Isn't the following a bug:
>
> sage: n = 113
> sage: n.ordinal_str()
> '113rd'
> sage: n = 112
> sage: n.ordinal_str()
> '112nd'
> sage: n = 111
> sage: n.ordinal_str()
> '111st'
>
> I opened the tickethttp://trac.sagemath.org/sage_trac/ticket/6842
> [with patch, needs review]
>
> Cheers,
> Simon
--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-support@googlegroups.com
To unsubscribe from this group, send email to 
sage-support-unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/sage-support
URLs: http://www.sagemath.org
-~----------~----~----~----~------~----~------~--~---

Reply via email to