Skip to content

Commit 2f22e42

Browse files
Replace bare except with except Exception in execution.py
1 parent 302b973 commit 2f22e42

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

sagemaker-train/src/sagemaker/train/evaluate/execution.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -904,7 +904,7 @@ def wait(
904904
if ipython is not None and 'IPKernelApp' in ipython.config:
905905
is_jupyter = True
906906
from IPython.display import display, HTML, clear_output
907-
except:
907+
except Exception:
908908
pass
909909

910910
if is_jupyter:
@@ -958,7 +958,7 @@ def wait(
958958
end = datetime.fromisoformat(step.end_time.replace('Z', '+00:00'))
959959
duration_seconds = (end - start).total_seconds()
960960
duration = f"{duration_seconds:.1f}s"
961-
except:
961+
except Exception:
962962
duration = "N/A"
963963
elif step.start_time:
964964
duration = "Running..."
@@ -982,7 +982,7 @@ def wait(
982982
# Add error indicator if failures exist
983983
if has_failures:
984984
if step.failure_reason:
985-
row_data.append("")
985+
row_data.append("❌")
986986
failed_steps.append(step)
987987
else:
988988
row_data.append("")
@@ -1005,7 +1005,7 @@ def wait(
10051005

10061006
for step in failed_steps:
10071007
content_parts.append(Text("")) # Empty line before each failure
1008-
content_parts.append(Text(f" {step.display_name or step.name}:", style="bold red"))
1008+
content_parts.append(Text(f"• {step.display_name or step.name}:", style="bold red"))
10091009
content_parts.append(Text(f" {step.failure_reason}", style="red"))
10101010

10111011
combined_content = Group(*content_parts)

0 commit comments

Comments
 (0)