Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jax-toolbox-triage: minor usability/doc improvements #1125

Merged
merged 5 commits into from
Oct 29, 2024

Conversation

olupton
Copy link
Collaborator

@olupton olupton commented Oct 29, 2024

  • Print the stdout/stderr of the first execution of the test case, which is supposed to fail, at INFO level along with a message encouraging the user to check that it is the correct failure.
  • Print the path to the DEBUG log file at INFO level and, therefore, to the console.
  • Expand the documentation.
  • Add --passing-container and --failing-container arguments, which allow the container-level search to be skipped and non-dated containers to be triaged.

- Print the stdout/stderr of the first execution of the test case, which
  is supposed to fail, at INFO level along with a message encouraging
  the user to check that it is the correct failure.
- Print the path to the DEBUG log file at INFO level and, therefore, to
  the console.
- Expand the documentation.
These allow the container-level search to be skipped entirely, which
enables bisecting at the commit level on containers not from the
ghcr.io/nvidia/jax:CONTAINER-YYYY-MM-DD series.
.github/triage/jax_toolbox_triage/args.py Outdated Show resolved Hide resolved
docs/triage-tool.md Outdated Show resolved Hide resolved
docs/triage-tool.md Outdated Show resolved Hide resolved
@olupton olupton requested a review from nouiz October 29, 2024 15:34
@olupton olupton merged commit 0e4e245 into main Oct 29, 2024
56 of 57 checks passed
@olupton olupton deleted the olupton/triage-tweaks branch October 29, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants