[issue8947] Provide as_integer_ratio() method to Decimal

Alexander Belopolsky added the comment: This issue has been superseded by #25928. -- resolution: rejected -> duplicate stage: patch review -> resolved superseder: -> Add Decimal.as_integer_ratio() ___ Python tracker

[issue8947] Provide as_integer_ratio() method to Decimal

Mark Dickinson added the comment: Apologies: the title change was accidental. Reverted. -- ___ Python tracker ___

[issue8947] Provide as_integer_ratio() method to Decimal

Change by Mark Dickinson : -- title: as_integer_ratio Decimal -> Provide as_integer_ratio() method to Decimal ___ Python tracker ___

[issue8947] Provide as_integer_ratio() method to Decimal

Mark Dickinson dicki...@gmail.com added the comment: After discussions on IRC, I'm going to close this as rejected. Convergence of the float and Decimal APIs is a nice idea, but in this case it's not clear that it really works. In particular, for duck typing to be sensible, int and Fraction

[issue8947] Provide as_integer_ratio() method to Decimal

Mark Dickinson dicki...@gmail.com added the comment: Raymond, do you have any thoughts on this proposal? -- nosy: +rhettinger ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8947 ___

[issue8947] Provide as_integer_ratio() method to Decimal

Raymond Hettinger rhettin...@users.sourceforge.net added the comment: -0 I don't think we really need this (we've already got a reasonable conversion to Fraction). In general, we should have a aversion to adding any methods to the decimal API because it is already very fat. Adding more

[issue8947] Provide as_integer_ratio() method to Decimal

Alexander Belopolsky belopol...@users.sourceforge.net added the comment: Raymond, conversion to Fraction does not really help in my use case of supporting timedelta * Decimal by duck typing. I can think of many other use cases where it will be helpful to accept either float or decimal

[issue8947] Provide as_integer_ratio() method to Decimal

Mark Dickinson dicki...@gmail.com added the comment: Updated patch, taking into account comments from merwok and exarkun on #python-dev: - remove doctests for infinity and nan, replace with a sentence explaining what happens for such inputs. - replace 'snAN' with saner spelling 'snan'.

[issue8947] Provide as_integer_ratio() method to Decimal

Alexander Belopolsky belopol...@users.sourceforge.net added the comment: Nice implementation. I wonder if the /5 loop can be eliminated by noting that /5 is *2 followed by decimal shift. (Probably not without fast decimal arithmetics.) A few documentation nits: 1. In Decimal methods there

[issue8947] Provide as_integer_ratio() method to Decimal

Changes by Stefan Krah stefan-use...@bytereef.org: -- nosy: +skrah ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8947 ___ ___ Python-bugs-list

[issue8947] Provide as_integer_ratio() method to Decimal

New submission from Alexander Belopolsky belopol...@users.sourceforge.net: Here is my use case: recently implemented timedelta * float operation starts with converting float to an int ratio. The same algorithm can be used for decimals, but they don't support as_integer_ratio(). --

[issue8947] Provide as_integer_ratio() method to Decimal

Mark Dickinson dicki...@gmail.com added the comment: This seems like a reasonable request to me; I'd like to see a little bit of convergence between the float and Decimal APIs. One difference between the two types that's worth noting is that the float.as_integer_ratio() method can never take

[issue8947] Provide as_integer_ratio() method to Decimal

Changes by Mark Dickinson dicki...@gmail.com: -- assignee: - mark.dickinson ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue8947 ___ ___