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 performance profiling suite #546

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

Add performance profiling suite #546

wants to merge 11 commits into from

Conversation

corybarr
Copy link
Contributor

@corybarr corybarr commented Nov 14, 2023

The PR adds a new "Performance Tracing Suite" app to the extension. When run, it will run performance tracing on every USDC file in the tests/testAssets/usd/flattened directory. One JSON trace analysis file is output for every USDC file. They are intended to be used in CI to query summary statistics.

This app extends the Performance Tracing app, which starts at launch and concludes when the app is closed. This new app will stop the at-launch profiling and write it to disk before profiling each USDC file. There will be one more profiling file than the number of USDC files.

Further work needs to be done to make better use of the trace analyses:

  • Integrate with CI
  • Generate reports or otherwise make available for querying of summary statistics
  • Improved USDC files

Optional uses include the following

  • Gain profiling reproducibility by adding UI to select a test file for playback when doing live profiling with Nsight, Superluminal, etc.

@corybarr corybarr marked this pull request as ready for review November 14, 2023 16:10
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