Fix file naming for roofline PDF outputs when using --kernel (#803)

Move check for kernel filtering to add to roofline pdfs- was originally only labelling pdfs with the filtered kernel names from --kernel when --kernel-names was called, we want it at all times when kernels are filtered. 

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
This commit is contained in:
cfallows-amd
2025-09-06 02:34:51 -04:00
committato da GitHub
parent c8ecf77a94
commit 2e750763e7
@@ -298,9 +298,6 @@ class Roofline:
original_kernel_names = []
else:
original_kernel_names = self.__ai_data.get("kernelNames", [])
if self.__run_parameters.get("kernel_filter", False):
for name in sorted(self.__args.kernel):
kernel_list += "_" + name
num_kernels = len(original_kernel_names)
@@ -426,6 +423,9 @@ class Roofline:
# otherwise return HTML to be used in GUI output
if self.__run_parameters["is_standalone"]:
dev_id = str(self.__run_parameters["device_id"])
if self.__run_parameters.get("kernel_filter", False):
for name in sorted(self.__args.kernel):
kernel_list += "_" + name
# Re-save to remove loading MathJax pop up
for i in range(2):