On Thursday 03 April 2008 10:37, Alex Ghitza wrote: > Note that at the moment 0.digits() does (2) and 0.ndigits() does (1), > which is really bad.
Yes, this is very bad. I was not aware of the ndigits convention until after the digits patch I wrote was included. On this topic, we also need a vote about digits/ndigits disagreement about the defaulted base. One defaults to base 2 and the other to base 10. I vote that they both default to base 10. Somewhat related: The 'bits' and 'ndigits(base=2)' are redundant. This violates the python mantra (but, I personally am not entirely fond of the one-way-to-do-things mantra). I think that 'bits' should be 'nbits'. Sorry, this is a bit of thread-jacking. I'd be happy to write a patch tweaking a few of these things if there were consensus about the tweaking. -- Joel --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---
