Message237525
I'm not sure it needs fixing: it follows from the definition of using Decimal(num) / Decimal(denom). Plus, it's controllable with a decimal context:
>>> from decimal import localcontext
>>> with localcontext() as ctx:
... ctx.prec = 100
... format(F(1, 3), '.30f')
...
'0.333333333333333333333333333333'
>>>
For all of the tests, I suggest using format(value, str) instead of ''.format(value). It more directly tests Fraction.__format__.
In general I think adding Fraction.__format__ is a good idea, and I think converting to Decimal is reasonable for the specified codes. My only question is what to do when "natively" formatting Fractions themselves. We might want to support field widths, padding, etc. |
|
Date |
User |
Action |
Args |
2015-03-08 10:06:46 | eric.smith | set | recipients:
+ eric.smith, rhettinger, mark.dickinson, ezio.melotti, skrah, martin.panter, serhiy.storchaka, tuomas.suutari |
2015-03-08 10:06:46 | eric.smith | set | messageid: <1425809206.06.0.586241626175.issue23602@psf.upfronthosting.co.za> |
2015-03-08 10:06:46 | eric.smith | link | issue23602 messages |
2015-03-08 10:06:45 | eric.smith | create | |
|