From 8f4cc7222258d8c706a27693779831a4831cf489 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 | 7 ++++++- 1 file changed, 6 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..c4153d6ed4a 100755 --- a/.github/actions/workflow-results/prepare-execution-summary.py +++ b/.github/actions/workflow-results/prepare-execution-summary.py @@ -7,6 +7,10 @@ import os 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: @@ -91,7 +95,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: