From a891ad00ef8af1c74180b4e0f3078c3e781cf5fc Mon Sep 17 00:00:00 2001 From: Allison Piper Date: Fri, 31 Jan 2025 00:20:57 +0000 Subject: [PATCH] Fix sccache calculations. sccache started reporting PTX/CUBIN hits. We filter these out as they are not included in the `compile_requests` counter. --- .../actions/workflow-results/prepare-execution-summary.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.github/actions/workflow-results/prepare-execution-summary.py b/.github/actions/workflow-results/prepare-execution-summary.py index ed47eba4981..5c47ffc5d63 100755 --- a/.github/actions/workflow-results/prepare-execution-summary.py +++ b/.github/actions/workflow-results/prepare-execution-summary.py @@ -8,6 +8,11 @@ import re +# sccache started reporting PTX/CUBIN hits. +# We filter these out as they are not included in the `compile_requests` counter. +sccache_languages = ["C/C++", "CUDA"] + + def job_succeeded(job): # The job was successful if the success file exists: return os.path.exists(f'jobs/{job["id"]}/success') @@ -91,7 +96,8 @@ def update_summary_entry(entry, job, job_times=None): if "counts" in cache_hits: counts = cache_hits["counts"] for lang, lang_hits in counts.items(): - hits += lang_hits + if lang in sccache_languages: + hits += lang_hits if "sccache" not in entry: entry["sccache"] = {"requests": requests, "hits": hits} else: