Skip to content

Commit

Permalink
Fix genReport
Browse files Browse the repository at this point in the history
Signed-off-by: Jack Luar <[email protected]>
  • Loading branch information
luarss committed Jan 16, 2025
1 parent ad154f5 commit 9a6566c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion flow/util/genReport.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
METRICS_CHECK_FMT = "{}/metadata-{}-check.log"
REGEX_ERROR = re.compile(r"^\[error ?(\w+-\d+)?\]", re.IGNORECASE)
REGEX_WARNING = re.compile(r"^\[warning ?(\w+-\d+)?\]", re.IGNORECASE)
SKIPPED_FLOW_VARIANT_KEYWORDS = ["test", "tune"]
STATUS_GREEN = "Passing"
STATUS_RED = "Failing"

Expand Down Expand Up @@ -248,7 +249,9 @@ def write_summary():
dir_list = log_dir.split(os.sep)
# Handles autotuner folders, which do not have `report.log` natively.
# TODO: Can we log something for autotuner?
if len(dir_list) != 4 or "test-" in dir_list[-1]:
if len(dir_list) != 4 or any(
word in dir_list[-1] for word in SKIPPED_FLOW_VARIANT_KEYWORDS
):
continue
report_dir = log_dir.replace(LOGS_FOLDER, REPORTS_FOLDER)

Expand Down
2 changes: 1 addition & 1 deletion tools/AutoTuner/test/resume_check.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ def setUp(self):
f" --platform {self.platform}"
f" --config {self.config}"
f" --jobs {self.jobs}"
f" --experiment test_resume"
f" --experiment test-resume"
f" tune --iterations {self.iterations} --samples {self.samples}"
f" --resources_per_trial {res_per_trial}"
f" {c}"
Expand Down

0 comments on commit 9a6566c

Please sign in to comment.