Skip to content

Update README to use relative nsys command paths#3

Open
lsawade wants to merge 1 commit intoPrincetonUniversity:masterfrom
lsawade:patch-1
Open

Update README to use relative nsys command paths#3
lsawade wants to merge 1 commit intoPrincetonUniversity:masterfrom
lsawade:patch-1

Conversation

@lsawade
Copy link

@lsawade lsawade commented Nov 25, 2025

Thanks for making this, Jon!

The /usr/local/bin/nsys was using the wrong executable after loading the cudatoolkit. Removing /usr/local/bin/ solves the issues.

Removed absolute path for nsys and nsys-ui commands in README.
@jdh4
Copy link
Collaborator

jdh4 commented Dec 1, 2025

The idea is to use the full path:
https://researchcomputing.princeton.edu/support/knowledge-base/gpu-computing#Profiling

This allows the latest nsys to be used for users that load an older CUDA toolkit. Doesn't loading the module cause you to use the nsys in that module and not the system version when full path is not applied? We can update the system nsys. Usually do this before the hackathon.

@lsawade
Copy link
Author

lsawade commented Dec 24, 2025

Sorry, only saw the reply now. The problem was when you use a new cudatoolkit to run the profiles. You cannot read them with the older nsys. I tried several different version combinations.

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