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.

Sign up for GitHub

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

Jump to bottom

Replace allocation analysis with Profile.Allocs flame graph #1766

Closed
charleskawczynski opened this issue Jun 12, 2023 · 1 comment
Closed

Replace allocation analysis with Profile.Allocs flame graph #1766

charleskawczynski opened this issue Jun 12, 2023 · 1 comment
Assignees
@charleskawczynski
Labels
Performance monitoring 🚀 🔍

Comments

@charleskawczynski
Copy link
Member

We should replace the allocations analysis with Profile.Allocs flame graphs because:
@charleskawczynski charleskawczynski self-assigned this Jun 12, 2023
@charleskawczynski charleskawczynski mentioned this issue Jun 12, 2023
Merged
bors bot added a commit that referenced this issue Jun 13, 2023
@bors @charleskawczynski
Merge #1767
1b1e69f 
1767: Remove allocs monitoring, fix docs, fix parameter tests r=charleskawczynski a=charleskawczynski

This PR:
 - Removes the allocation monitoring (a step towards #1766)
 - Fixes some doc strings for file includes
 - Fixes the parameter tests and includes them in the test suite.

Co-authored-by: Charles Kawczynski <[email protected]>
@charleskawczynski
Copy link
Member Author

Closed by #1767?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@charleskawczynski charleskawczynski

Labels
Performance monitoring 🚀 🔍
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@charleskawczynski