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 tables for the other MaxMemoryPreload-reported quantities (or something) #1089

Closed
Tracked by #670
makortel opened this issue Nov 12, 2024 · 2 comments · Fixed by cms-sw/cms-bot#2412
Closed
Tracked by #670
Assignees
Labels

Comments

@makortel
Copy link
Collaborator

makortel commented Nov 12, 2024

Add (e.g.) additional separate tables for

  • total memory requested
  • presently used (in the table use name along "used at the end" or "leaked")
  • # allocations calls
@gartung
Copy link
Member

gartung commented Jan 8, 2025

  • add #allocations - #deallocations for "number of allocation calls whose memory leaked". Maybe call it here "#allocation calls leaked"
  • order the quantities
    max memory used
    total memory requested
    #allocation calls
    memory leaked
    #allocation calls leaked
  • the allocation call number counts would benefit from thousand separators

@gartung
Copy link
Member

gartung commented Jan 8, 2025

addressed in cms-sw/cms-bot#2412

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

Successfully merging a pull request may close this issue.

2 participants