Merge pull request #558 from GATEOverflow/mlperf-inference

Fix the saving of console logs in benchmark-program - fixes #533
mlcommons · Nov 16, 2024 · e456b44 · e456b44
2 parents 155cf61 + 4081528
commit e456b44
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/test-scc24-sdxl.yaml b/.github/workflows/test-scc24-sdxl.yaml
@@ -2,7 +2,7 @@ name: MLPerf inference SDXL (SCC)
 
 on:
   schedule:
-    - cron: "08 18 * * *"
+    - cron: "25 00 * * *"
 
 jobs:
   build_reference:

diff --git a/script/benchmark-program/customize.py b/script/benchmark-program/customize.py
@@ -45,7 +45,7 @@ def preprocess(i):
 
     if os_info['platform'] != 'windows' and str(env.get('CM_SAVE_CONSOLE_LOG', True)).lower() not in  [ "no", "false", "0"]:
         logs_dir = env.get('CM_LOGS_DIR', env['CM_RUN_DIR'])
-        env['CM_RUN_CMD'] += r" 2>&1 ; echo \$? > exitstatus | tee " + q+ os.path.join(logs_dir, "console.out") + q
+        env['CM_RUN_CMD'] += r" 2>&1 | tee " + q+ os.path.join(logs_dir, "console.out") + q +  r" echo \${PIPESTATUS[0]} > exitstatus"
 
     # additional arguments and tags for measuring system informations(only if 'CM_PROFILE_NVIDIA_POWER' is 'on')
     if env.get('CM_PROFILE_NVIDIA_POWER', '') == "on":