Change Default Normal Unit to per_kernel

Open MrBurmark opened this issue 2 years ago • 1 comments

As an application developer I think in terms of kernels rather than wave/wavefronts. Per kernel is also the normalization used in nsight compute so using per kernel makes it easier to compare output with ncu which is one of my common use cases.

Feb 14 '23 21:02 MrBurmark

Thanks for the suggestion. The mode "Per kernel comparison to ncu" is not "only" your case. We did put it in our plan.

Feb 16 '23 04:02 feizheng10

Hi @MrBurmark, sorry for the late follow-up. @feizheng10 has put in a change (https://github.com/ROCm/rocprofiler-compute/pull/555) to make the default normalization to be per kernel. Let me know if you have any other concerns.

Feb 28 '25 21:02 sohaibnd

Closing this issue as it is resolved. @MrBurmark Feel free to re-open the issue if you have any follow-up questions/concerns.

Mar 09 '25 20:03 sohaibnd