-
Notifications
You must be signed in to change notification settings - Fork 44
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
T5x loss+perf baselines #256
Conversation
maanug-nv
commented
Sep 21, 2023
- print test metrics
- update baseline scripts to support T5x
- add t5x test baselines
@maanug-nv Could you please update the description with an example of the mechanism in effect? |
@yhtang Which description? PR description, or in one of the scripts, or in the README? |
@yhtang bumping for @maanug-nv 's Q |
Added |
4ba1456
to
9987b10
Compare
I've tuned the relative tolerances a few times, all test cases are passing in this run: https://github.com/NVIDIA/JAX-Toolbox/actions/runs/6475410065 |
The rtol LGTM. Looks like #292 may have introduced conflicts that you have to resolve in |
f7f27b1
to
b852f69
Compare
resolved, if everything passes FFTM |
Manual run of t5x MGMN since t5x build is failing due to a known issue: https://github.com/NVIDIA/JAX-Toolbox/actions/runs/6492103243 |