Skip to content

Conversation

@rffontenelle
Copy link
Contributor

@rffontenelle rffontenelle commented Aug 30, 2024

In the python -X cpu_count=n command line, n value is not overridden by anything. It actually overrides the default os.cpu_count()'s return value. So the text should say "override value" instead of "overridden value".

Please backport to 3.13


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

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Aug 30, 2024
@vstinner vstinner added the needs backport to 3.13 bugs and security fixes label Sep 5, 2024
@vstinner vstinner requested a review from corona10 September 5, 2024 22:48
.. versionchanged:: 3.13
If :option:`-X cpu_count <-X>` is given or :envvar:`PYTHON_CPU_COUNT` is set,
:func:`cpu_count` returns the overridden value *n*.
:func:`cpu_count` returns the override value *n*.
Copy link
Contributor

Choose a reason for hiding this comment

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

The language is weird either way, may be 'overriding value'

Copy link
Contributor Author

@rffontenelle rffontenelle Sep 25, 2024

Choose a reason for hiding this comment

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

Hi there. Shouldn't it be like "return value" used in these docs? (English is not my native language, so I could be mixing different things)

@serhiy-storchaka serhiy-storchaka added the needs backport to 3.14 bugs and security fixes label May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes skip issue skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

5 participants