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 helper for printing Umpire pool statistics #1179

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

msimberg
Copy link
Collaborator

@msimberg msimberg commented Sep 4, 2024

This is the non-production ready debugging version that I used to print Umpire pool statistics for debugging. It needs cleanup to be ready for merging, but more importantly if we want something like this in master we should:

  • decide how to trigger the printing and when (when finalizing DLA-Future, after every algorithm, with or without synchronization, environment variable)
  • decide which statistics to show and in what form

@msimberg msimberg self-assigned this Sep 4, 2024
@albestro
Copy link
Collaborator

Thanks! We might think of taking chance of this also for tracking memory usage in our benchmarks (which should just add to requirements "easy to be parsed").

@msimberg
Copy link
Collaborator Author

Thanks! We might think of taking chance of this also for tracking memory usage in our benchmarks (which should just add to requirements "easy to be parsed").

I think this is a very good idea. I could see at least three different directions:

  • trigger printing of statistics at the end of each algorithm
  • explicitly call a generic function to print statistics at the end of all miniapps
  • above, but implement a specialized format only for miniapps that's easier to parse

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

Successfully merging this pull request may close these issues.

2 participants