| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…stats is provided
…able-pystats is provided
…en --enable-pystats is provided Signed-off-by: Pablo Galindo <pablogsal@gmail.com>
|
For testing with Python 3.11.x, this is a version ported on top of v3.11.1 . The pystats.h header is not really needed so I removed the references to it. I also added a check for the PYTHON_GC_STATS env var. That keeps stderr cleaner when you are building stuff and not actually wanting stats. Rather than hard-coding /tmp/py_stats you could get that path from an env var too. An alternative idea would be to have a new -X flag that enables the stats. I'm planning to collect stats on some applications but haven't got too far on that yet. Initial indication is that the plots look quite different depending on application. |
Sorry, something went wrong.
For this PR I opted to mirror the workings and contracts of the other interpreter stats collection that were added in the specializing interpreter to not end with many different ways to activate and retrieve statistics. |
Sorry, something went wrong.
If set, youngest generation threshold will be set to value.
Use shorter labels for generations. Show unit as y-axis label on bar charts.
|
I pushed some changes here: https://github.com/nascheme/cpython/commits/gc_stats in case you want to cherry-pick or merge. |
Sorry, something went wrong.
|
@nascheme I have implemented the stats via a -X option where the filename is provided there. This should be a first baseline for the stats collection and we can add public interfaces in the future as I am keeping the containers around. Let me know if you think we should do further modifications. |
Sorry, something went wrong.
There was a problem hiding this comment.
Pull request does not meet quality requirements.
Sorry, something went wrong.
| ObjectStats object_stats; | ||
| } PyStats; | ||
|
|
||
|
|
There was a problem hiding this comment.
Spurious edit.
Sorry, something went wrong.
There was a problem hiding this comment.
Sir Bedevere obtained new powers?
Sorry, something went wrong.
| }; | ||
| gcstate->generation0 = GEN_HEAD(gcstate, 0); | ||
| INIT_HEAD(gcstate->permanent_generation); | ||
|
|
There was a problem hiding this comment.
Spurious edit.
Sorry, something went wrong.
|
This PR is stale because it has been open for 30 days with no activity. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.