Skip to content

Commit 69e53a7

Browse files
committed
only print links if all success
1 parent c1df8b5 commit 69e53a7

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

eval_protocol/pytest/plugin.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -343,16 +343,15 @@ def _print_experiment_links(session):
343343
if EXPERIMENT_LINKS_STASH_KEY is not None and EXPERIMENT_LINKS_STASH_KEY in session.stash:
344344
links = session.stash[EXPERIMENT_LINKS_STASH_KEY]
345345

346-
if links:
346+
# Only print when there is at least one successful link.
347+
# Suppress the entire section if all links are failures (noise).
348+
if any(link.get("status") == "success" for link in links):
347349
print("\n" + "=" * 80, file=sys.__stderr__)
348350
print("🔥 FIREWORKS EXPERIMENT LINKS", file=sys.__stderr__)
349351
print("=" * 80, file=sys.__stderr__)
350352

351353
for link in links:
352-
if link["status"] == "success":
353-
print(f"🔗 Experiment {link['experiment_id']}: {link['job_link']}", file=sys.__stderr__)
354-
else:
355-
print(f"❌ Experiment {link['experiment_id']}: {link['job_link']}", file=sys.__stderr__)
354+
print(f"Experiment {link['experiment_id']}: {link['job_link']}", file=sys.__stderr__)
356355

357356
print("=" * 80, file=sys.__stderr__)
358357
return True

0 commit comments

Comments
 (0)