Skip to content

Conversation

@anshgupta1234
Copy link
Contributor

This PR adds --nsys and --ncu to the documentation. I wasn't sure how much detail to add, please let me know if there's anything important that I missed :)

@sbryngelson
Copy link
Member

why did you change the CI file

@anshgupta1234
Copy link
Contributor Author

why did you change the CI file

My apologies, I reverted it.

@sbryngelson
Copy link
Member

Please add links to what nsight compute and nsight sys are and their official documentation, and how to use additional flags for them. You also should describe what they are used for.

@anshgupta1234
Copy link
Contributor Author

@sbryngelson Sounds good, I'll add that.

@henryleberre
Copy link
Member

@sbryngelson @anshgupta1234 I pushed to this PR the feature that lets you specify additional arguments to nsys and ncu. For brevity, I won't explain why but it requires that --nsys (or --ncu) be specified as the last command line argument. You can just append to the invocation the arguments you want to have forwarded.

For example you could:

$ ./mfc.sh run case.py [(optional) case file arguments] -N 5 -n 2 --gpu --no-mpi -j $(nproc) -t simulation -e batch .... --nsys [nsys arguments here]

This isn't necessarily realistic but it shows how you can use all the features together.

@sbryngelson
Copy link
Member

@henryleberre thanks! should be useful. can you add this to the documentation as well, @anshgupta1234 ?

@anshgupta1234 anshgupta1234 force-pushed the nsight_docs branch 2 times, most recently from 0b1ac4c to 70e97f7 Compare April 9, 2023 21:51
Update running.md

Update running.md

Co-Authored-By: Henry Le Berre <hberre3@gatech.edu>
@sbryngelson sbryngelson merged commit cf11592 into MFlowCode:master Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants