Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update outdated statement frommath about C standard#134621

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
AA-Turner merged 2 commits intopython:mainfromhugovk:3.15-docs-math-rm-c-std
May 28, 2025

Conversation

hugovk
Copy link
Member

@hugovkhugovk commentedMay 24, 2025
edited by github-actionsbot
Loading

ThePEP 791 draft says:

Themath documentation says: “This module provides access to the mathematical functions defined by the C standard.” But, over time the module was populated with functions that aren’t related to the C standard or floating-point arithmetics.

Let's start by removing this outdated sentence from the docs.


📚 Documentation preview 📚:https://cpython-previews--134621.org.readthedocs.build/

Copy link
Contributor

@skirpichevskirpichev left a comment
edited
Loading

Choose a reason for hiding this comment

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

“This module provides access to the mathematical functions defined by the C standard.”

Obviously, -1 to that.

There is nothing wrong with this sentence - the moduledoes provide access to such functions (at least for most of them). Edit: It's just notexhaustive description of the module content.

@hugovkhugovk changed the titleRemove outdated statement frommath about C standardUpdate outdated statement frommath about C standardMay 27, 2025
@AA-TurnerAA-Turner merged commit21672b6 intopython:mainMay 28, 2025
28 checks passed
@github-project-automationgithub-project-automationbot moved this fromTodo toDone inDocs PRsMay 28, 2025
@miss-islington-app
Copy link

Thanks@hugovk for the PR, and@AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestMay 28, 2025
(cherry picked from commit21672b6)Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestMay 28, 2025
(cherry picked from commit21672b6)Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
@bedevere-app
Copy link

GH-134825 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelMay 28, 2025
@bedevere-app
Copy link

GH-134826 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelMay 28, 2025
AA-Turner added a commit that referenced this pull requestMay 28, 2025
) (#134826)Update outdated statement from `math` about C standard (GH-134621)(cherry picked from commit21672b6)Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
AA-Turner added a commit that referenced this pull requestMay 28, 2025
) (#134825)Update outdated statement from `math` about C standard (GH-134621)(cherry picked from commit21672b6)Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
@hugovkhugovk deleted the 3.15-docs-math-rm-c-std branchMay 30, 2025 08:08
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@JelleZijlstraJelleZijlstraJelleZijlstra left review comments

@skirpichevskirpichevskirpichev left review comments

@AA-TurnerAA-TurnerAA-Turner approved these changes

@picnixzpicnixzpicnixz approved these changes

@serhiy-storchakaserhiy-storchakaAwaiting requested review from serhiy-storchaka

Assignees
No one assigned
Labels
docsDocumentation in the Doc dirskip issueskip news
Projects
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@hugovk@JelleZijlstra@skirpichev@AA-Turner@picnixz

[8]ページ先頭

©2009-2025 Movatter.jp