Skip to content
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

Add option to show total time including children in flat profiles #285

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

matt-graham
Copy link
Contributor

Adds a literal string argument flat_time to ConsoleRenderer which can be used to set frame time type displayed (and used as sort key) in flat profiles - "self" for total self time, excluding child frames (the default, equivalent to current behaviour) and "total" for total time, including child frames.

I find being able to view a flat list sorted by total time useful in some situations, but it does result in the less intutitive behaviour that the sum of the times shown no longer bears any relation to total run time.

Rather than a string argument could alternatively use a boolean flag if preferred or change name of options.

@matt-graham
Copy link
Contributor Author

For the remaining pyright error in pre-commit workflow, I get the same error when running pre-commit run --all-files locally on current tip of main so not sure if the failure is related to changes in this PR or a change in upstream packages?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant