New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gh-85988: Change documentation for sys.float_info.rounds #99675
gh-85988: Change documentation for sys.float_info.rounds #99675
Conversation
Change the documentation for sys.float_info.rounds to remove references to C99 section 5.2.4.2.2 and instead place the available values inline.
Newlines were not preserved in generated HTML on previous commit. I have changes the list to a comma-separated list of values and their meanings.
…cpython into float-info-rounds-documentation Merge upstream branch into local topic branch
|
Thanks for the PR. A couple of comments: We should keep the "This reflects the value of the system FLT_ROUNDS macro at interpreter startup time" text: there's important information there. In C, The "for floating-point addition" language seems curiously specific (yes, I know it comes directly from the standard). I'd expect the rounding mode to apply to all arithmetic operations (possibly excluding |
Clarify the source of the FLT_ROUNDS value and change 'floating-point addition' to 'floating-point arithmetic' to indicate that the rounding mode applies to all arithmetic operations.
|
Thank you for the feedback, @mdickinson ! I have made a few changes to address your points. Just let me know if you see anything that I might have overlooked, and I'll be glad to sort it out. |
Change the documentation for sys.float_info.rounds to remove references to C99 section 5.2.4.2.2 and instead place the available values inline.