-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
bpo-46072: Document --enable-stats option. #30139
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -131,6 +131,17 @@ General Options | |||
|
|||
.. versionadded:: 3.11 | |||
|
|||
.. cmdoption:: --enable-pystats |
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.
342b93f says --with-pystats
. Which one is correct?
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.
This is. @tiran requested that it was changed to --enable-pystats
@@ -131,6 +131,17 @@ General Options | |||
|
|||
.. versionadded:: 3.11 | |||
|
|||
.. cmdoption:: --enable-pystats | |||
|
|||
Turn on internal statistics gathering. |
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.
Would you mind to give a more more details? In the issue, you wrote: "Currently various parts of the VM gather stats: the GC, dicts, the bytecode interpreter, type lookup cache, etc.".
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.
I'd rather not because, as it says, they are internal. We may well add or remove stats when we need them and this documentation will end up being a pain to maintain or just wrong.
Also change the name of the script for parsing the stats as we want to handle all stats not just specialization.
https://bugs.python.org/issue46072