Repository navigation
Add NVTX support for performance analysis - #144
Merged
Merged
Conversation
gigony
force-pushed
the
support_nvtx
branch
2 times, most recently
from
November 10, 2021 02:14
60bc2ff to
8dddd15
Compare
gigony
force-pushed
the
support_nvtx
branch
9 times, most recently
from
November 12, 2021 19:02
0faa489 to
b83c51b
Compare
Member
|
PR ( #141 ) is in. So this is ready for rebasing |
Contributor
Author
@jakirkham |
Merged
jakirkham
approved these changes
Nov 29, 2021
| deps-nvtx3 | ||
| GIT_REPOSITORY https://github.com/NVIDIA/NVTX.git | ||
| GIT_TAG 3c98c8425b0376fd8653aac7cfc6a864f3897752 | ||
| # GIT_SHALLOW TRUE # Uncomment this when the official release of nvtx3-cpp is available. |
Member
There was a problem hiding this comment.
Should we raise an issue to track this?
Contributor
Author
There was a problem hiding this comment.
Thanks @jakirkham!
Created #168 and updated the comment in nvtx3.cmake.
Member
|
Had a couple questions above. Otherwise LGTM |
jakirkham
approved these changes
Nov 30, 2021
jakirkham
approved these changes
Nov 30, 2021
ajschmidt8
approved these changes
Nov 30, 2021
Member
|
CI is passing. Merging. |
Member
|
Thanks AJ! 😄 |
prateek9623
pushed a commit
to AMD-Ecosystem/hipCIM
that referenced
this pull request
Sep 22, 2026
…6515) (rapidsai#144) Promotes the documentation fixes from AIOSS-6515 (merged as rapidsai#142) from `amd-integration-26.06.00` to `release/26.06.00`. Documentation only — no source or build-system changes. ``` 1bc661e docs: apply doc team review feedback d157d0d docs: simplify the GPU architecture note c58f509 docs: fix install/test instructions found during 26.06 validation ``` `README.md`, `docs/installation/installation.rst`, `scripts/README.md` — +102/-44. Reviewed and approved by the doc team on rapidsai#142. **Please approve only — do not merge from the UI.** This will be fast-forwarded manually so the individual commits are preserved (`release/26.06.00` is at `cebc52ce`, the direct parent of `c58f5098`, so a fast-forward applies cleanly). Refs: AIOSS-6515
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Need to rebase once #141 is merged.
Enabling Tracing
Through
.cucim.jsonfile{ "profiler" : { "trace": true } }Through Environment variable
Through API
Profiling with NVIDIA Nsight Systems
Then, execute
nsight-systo open the profile results (my_profile.qdrep).closes #137