Skip to content

Conversation

benjaminhuth
Copy link
Member

@benjaminhuth benjaminhuth commented Aug 7, 2025

This improves analysis of device parallelism with nsight-systems.

@github-actions github-actions bot added Component - Examples Affects the Examples module Component - Plugins Affects one or more Plugins Component - Documentation Affects the documentation Track Finding labels Aug 7, 2025
@github-actions github-actions bot added this to the next milestone Aug 7, 2025
@paulgessinger
Copy link
Member

Does this need to be steered from CMake? Could you make this conditional on #include <nvtx3/nvToolsExt.h> being available, or is there a use case where you want to turn it off even if it's present?

Copy link
Contributor

github-actions bot commented Aug 7, 2025

📊: Physics performance monitoring for b0c3ddf

Full contents

physmon summary

@benjaminhuth
Copy link
Member Author

I'm not sure about the performance impact of this tbh. Maybe @stephenswat has some insight in this.

@paulgessinger
Copy link
Member

@benjaminhuth Conclusion? I would prefer if we could avoid making this an explicit option, but ok if we have to.

@stephenswat
Copy link
Member

I am with Paul on this, I would unconditionally enable this and not bother with the CMake flag.

@github-actions github-actions bot added the Stale label Sep 21, 2025
@andiwand andiwand marked this pull request as draft September 22, 2025 13:32
@github-actions github-actions bot removed the Stale label Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component - Documentation Affects the documentation Component - Examples Affects the Examples module Component - Plugins Affects one or more Plugins Track Finding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants