GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-100425: Update tutorial docs related to sum() accuracy by neuralstring · Pull Request #101854 · python/cpython

Conversation

@neuralstring

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

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Related to #100426

  • Issue: Improve accuracy of builtin sum() for float inputs #100425

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@neuralstring
Update tutorial docs related to sum() accuracy
518c1af 
Related to python#100426
@ghost

ghost commented Feb 12, 2023 •
edited by ghost
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-bot bedevere-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
Comment thread 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

@arhadthedev arhadthedev Feb 12, 2023

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Can confirm:

Running Release|x64 interpreter...
Python 3.12.0a5+ (heads/main:dfc2e065a2, Feb 12 2023, 11:14:56) [MSC v.1929 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> sum([0.1]*10) == 1.0
True
>>> 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 + 0.1 == 1.0
False

@rhettinger as an author of the referenced PR.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@arhadthedev arhadthedev changed the title Update tutorial docs related to sum() accuracy gh-100425: Update tutorial docs related to sum() accuracy Feb 12, 2023
@bedevere-bot bedevere-bot mentioned this pull request Feb 12, 2023
Improve accuracy of builtin sum() for float inputs #100425
Closed
@rhettinger rhettinger self-assigned this Feb 15, 2023
rhettinger
rhettinger requested changes Feb 15, 2023
View reviewed changes

@rhettinger rhettinger left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this 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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@bedevere-bot bedevere-bot removed the awaiting review label Feb 15, 2023
@bedevere-bot

bedevere-bot commented Feb 15, 2023

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-bot bedevere-bot added the awaiting changes label Feb 15, 2023
@rhettinger
rhettinger merged commit 32df540 into python:main Feb 19, 2023
@bedevere-bot bedevere-bot removed the awaiting changes label Feb 19, 2023
@Allimuzzaman

Allimuzzaman commented Oct 21, 2024

Copy link
Copy Markdown

file your system
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arhadthedev arhadthedev arhadthedev left review comments
@rhettinger rhettinger rhettinger requested changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@rhettinger rhettinger

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@neuralstring @bedevere-bot @Allimuzzaman @rhettinger @arhadthedev
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.