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

GC reports notebook - read GC Infra gcperfsim output #4388

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

Conversation

markples
Copy link
Member

@markples markples commented Aug 9, 2024

No description provided.

@markples markples self-assigned this Aug 9, 2024
@markples markples requested a review from mrsharm August 9, 2024 23:15
@markples markples changed the title Add support for the GC reports notebook to read GC Infra gcperfsim output GC reports notebook - read GC Infra gcperfsim output Aug 9, 2024
@mrsharm
Copy link
Member

mrsharm commented Aug 20, 2024

Curious - has this been tested with the crank runs for GCPerfSim?

@markples
Copy link
Member Author

Curious - has this been tested with the crank runs for GCPerfSim?

I've been running locally. Are they different?

@mrsharm
Copy link
Member

mrsharm commented Sep 2, 2024

Curious - has this been tested with the crank runs for GCPerfSim?

I've been running locally. Are they different?

Yes, there is a mode that can supplied by passing in --server <Name Of Server> to run these via Crank. Worthwhile just testing if that mode works with your latest changes.

Documentation: https://github.com/dotnet/performance/tree/main/src/benchmarks/gc/GC.Infrastructure#running-gcperfsim-scenarios-on-the-aspnet-machines

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.

2 participants