Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented May 24, 2025

The PEP 791 draft says:

The math 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

@skirpichev skirpichev left a comment

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 module does provide access to such functions (at least for most of them). Edit: It's just not exhaustive description of the module content.

@hugovk hugovk changed the title Remove outdated statement from math about C standard Update outdated statement from math about C standard May 27, 2025
@AA-Turner AA-Turner merged commit 21672b6 into python:main May 28, 2025
28 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Docs PRs May 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 request May 28, 2025
(cherry picked from commit 21672b6)

Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Adam Turner <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 28, 2025
(cherry picked from commit 21672b6)

Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Adam Turner <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented May 28, 2025

GH-134825 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label May 28, 2025
@bedevere-app
Copy link

bedevere-app bot commented May 28, 2025

GH-134826 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label May 28, 2025
AA-Turner added a commit that referenced this pull request May 28, 2025
) (#134826)

Update outdated statement from `math` about C standard (GH-134621)
(cherry picked from commit 21672b6)

Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Adam Turner <[email protected]>
AA-Turner added a commit that referenced this pull request May 28, 2025
) (#134825)

Update outdated statement from `math` about C standard (GH-134621)
(cherry picked from commit 21672b6)

Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Adam Turner <[email protected]>
@hugovk hugovk deleted the 3.15-docs-math-rm-c-std branch May 30, 2025 08:08
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants