From 2e750763e7084c961ec29c730ca253c5370a6a3f Mon Sep 17 00:00:00 2001 From: cfallows-amd Date: Sat, 6 Sep 2025 02:34:51 -0400 Subject: [PATCH] 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 --- projects/rocprofiler-compute/src/roofline.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/projects/rocprofiler-compute/src/roofline.py b/projects/rocprofiler-compute/src/roofline.py index 8af0cbc518..84b076485d 100644 --- a/projects/rocprofiler-compute/src/roofline.py +++ b/projects/rocprofiler-compute/src/roofline.py @@ -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):