Issue 13731: Awkward phrasing in Decimal documentation (original) (raw)

Created on 2012-01-07 19:16 by zacherates, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
rephrase.diff zacherates,2012-01-07 19:16 review
Messages (6)
msg150814 - (view) Author: Aaron Maenpaa (zacherates) Date: 2012-01-07 19:16
The paragraph: "The exactness carries over into arithmetic. In decimal floating point, 0.1 + 0.1 + 0.1 - 0.3 is exactly equal to zero. In binary floating point, the result is 5.5511151231257827e-017. While near to zero, the differences prevent reliable equality testing and differences can accumulate. For this reason, decimal is preferred in accounting applications which have strict equality invariants." ... has some awkward phrasing to my ear. I've attached a patch with a proposed alternative.
msg150819 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2012-01-07 21:12
I'm sorry but I think the current wording is better that your proposed revision. When I get a chance, I'll revisit it to see if I can find another way to improve the text.
msg150823 - (view) Author: Aaron Maenpaa (zacherates) Date: 2012-01-07 21:39
That's fine. I'm not particularly attached to that phrasing. The one thing I would push for is to add a comma to "... decimal is preferred in accounting applications which have strict equality invariants." ... since, as far as I can tell, "which have strict equality invariants" is supposed to be a parenthetical statement explaining why accounting applications prefer to use decimal arithmetic, rather than a constraints on the preference for decimal arithmetic to only those accounting applications that have "strict equality invariants".
msg150825 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2012-01-07 22:42
If you can't ascertain the meaning of the sentence, I'll consider making a change. Itherwise, this appears to have degenerated into trivial micro-wordsmithing and I'll close this as not being worth consuming any more of my time.
msg150826 - (view) Author: Aaron Maenpaa (zacherates) Date: 2012-01-07 22:47
I can understand what was meant. You're welcome to close the issue. Sorry for the nitpick.
msg150827 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2012-01-07 22:48
No problem. Thanks for showing an interest in the quality of the documentation.
History
Date User Action Args
2022-04-11 14:57:25 admin set github: 57940
2012-01-07 22:48:26 rhettinger set status: open -> closedresolution: rejectedmessages: +
2012-01-07 22:47:33 zacherates set messages: +
2012-01-07 22:42:38 rhettinger set messages: +
2012-01-07 21:39:12 zacherates set messages: +
2012-01-07 21:12:22 rhettinger set priority: normal -> lowmessages: +
2012-01-07 21:08:37 rhettinger set assignee: docs@python -> rhettingernosy: + rhettinger
2012-01-07 19:16:46 zacherates create