-
Notifications
You must be signed in to change notification settings - Fork 44
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 CUDA flags to draco_info report #1135
Conversation
+ Fix a bug in the `draco_info` report that removed leading slashes for compiler paths. + Also wrap the fully qualified path to `mpirun` + Move some names to "prior contributors" section of the author list. + Minor clean-up in cdi files.
Codecov Report
@@ Coverage Diff @@
## develop #1135 +/- ##
=========================================
- Coverage 89.0% 89.0% -0.1%
=========================================
Files 374 374
Lines 19255 19263 +8
=========================================
+ Hits 17151 17155 +4
- Misses 2104 2108 +4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
broken logic
@RyanWollaeger Can you review and merge this PR? |
@KineticTheory Will do. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Background
draco_info
command so that information about Cuda is also printed.Description of changes
draco_info
report that removed leading slashes for compiler paths.mpirun
Example output
Status