cutlass icon indicating copy to clipboard operation
cutlass copied to clipboard

[FEA] The procedural_name of TileDescription

Open qliu93 opened this issue 4 years ago • 4 comments

Hello, I'm reading the scripts in cutlass/tools/library and find it hard to understand why TileDescription gets its name in this format:

https://github.com/NVIDIA/cutlass/blob/master/tools/library/scripts/library.py#L528

Why don't we use

"{}x{}x{}_{}x{}x{}_{}".format(tbm, tbn, tbk, warpm, warpn, warpk, statges)

Thanks

qliu93

qliu93 avatar Jul 06 '21 07:07 qliu93

tbk x stage shows that how many elements in the k dimension are processed in one complete software pipeline.

We can add warp tile size to the kernel name so that we can tell if a kernel is using slice-k or not. The downside is that the kernel name will be even longer. Do you have interest in creating a PR?

hwu36 avatar Jul 06 '21 19:07 hwu36

We can add warp tile size to the kernel name so that we can tell if a kernel is using slice-k or not. The downside is that the kernel name will be even longer. Do you have interest in creating a PR?

Yes, I will try doing it today : )

qliu93 avatar Jul 07 '21 00:07 qliu93

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

github-actions[bot] avatar Jan 14 '22 17:01 github-actions[bot]

This issue has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.

github-actions[bot] avatar Apr 14 '22 18:04 github-actions[bot]