Skip to content

gh-100425: Update tutorial docs related to sum() accuracy#101854

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.

Sign up for GitHub

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

Jump to bottom
Merged
rhettinger merged 1 commit into from
Feb 19, 2023
Merged

gh-100425: Update tutorial docs related to sum() accuracy #101854

rhettinger merged 1 commit into from
Feb 19, 2023

Conversation

@neuralstring
Copy link
Contributor

@neuralstringneuralstring commented Feb 12, 2023•
edited by bedevere-bot
Loading

@ghost
Copy link

ghost commented Feb 12, 2023•
edited by ghost
Loading

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-botbedevere-bot added awaiting review docs Documentation in the Doc dir skip news labels Feb 12, 2023
arhadthedev
arhadthedev reviewed Feb 12, 2023
View reviewed changes
Doc/tutorial/stdlib2.rst
Comment on lines -397 to 398
True
>>> sum([0.1]*10) == 1.0
>>> 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 == 1.0
False
Copy link
Member

@arhadthedevarhadthedevFeb 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can confirm:

RunningRelease|x64interpreter... Python3.12.0a5+ (heads/main:dfc2e065a2, Feb122023, 11:14:56) [MSCv.192964bit (AMD64)] onwin32Type"help", "copyright", "credits"or"license"formoreinformation. >>>sum([0.1]*10) ==1.0True>>>0.1+0.1+0.1+0.1+0.1+0.1+0.1+0.1+0.1+0.1==1.0False

@rhettinger as an author of the referenced PR.

@arhadthedevarhadthedev changed the title Update tutorial docs related to sum() accuracygh-100425: Update tutorial docs related to sum() accuracyFeb 12, 2023
@bedevere-botbedevere-bot mentioned this pull request Feb 12, 2023
Closed
@rhettingerrhettinger self-assigned this Feb 15, 2023
rhettinger
rhettinger requested changes Feb 15, 2023
View reviewed changes
Copy link
Contributor

@rhettingerrhettinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add doctest directives throughout this page. It would have been nice for this issue to have been detected automatically.

@bedevere-bot
Copy link

bedevere-bot commented Feb 15, 2023

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@rhettingerrhettinger merged commit 32df540 into python:mainFeb 19, 2023
@Allimuzzaman
Copy link

Allimuzzaman commented Oct 21, 2024

file your system

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arhadthedevarhadthedevarhadthedev left review comments

@rhettingerrhettingerrhettinger requested changes

Assignees

@rhettingerrhettinger

Labels

docsDocumentation in the Doc dirskip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@neuralstring@bedevere-bot@Allimuzzaman@rhettinger@arhadthedev