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 Nsight SQLite reader #138

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

lithomas1
Copy link

TODO:

  • Formalize method for handling multiple "traces" in pipit
    • Nsight Systems contains many traces, interesting ones are
      • NVTX events
      • CUDA API trace (all cuda functions called)
      • Memory operations/Kernel Launches
  • CPU/GPU Metrics
    • Some particularly interesting ones of note:
      • CPU Utilization/GPU utilization/Unified memory page faults, etc.
  • Device Metadata
    • e.g. Might be interesting to attach metadata, such as GPU Name, GPU VRAM, # of SMs, etc. to the pipit Trace object (not on dataframe since not all dataframes can handle metadata)
    • Then we might be able to analyze perf across different types of GPUs (e.g. to figure out which GPU is right for which workload).
  • Multi-device?
  • Tests

cc @jhdavis8 since you were interested in the Nsight reader too

also cc @ocnkr since this is based off your previous PR.

@lithomas1 lithomas1 marked this pull request as ready for review October 4, 2024 12:51
@jhdavis8 jhdavis8 requested review from jhdavis8 and movsesyanae and removed request for jhdavis8 October 4, 2024 17:25
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.

1 participant