-
Notifications
You must be signed in to change notification settings - Fork 48
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
Collect recursively and filter GPU tests using jax_test_gpu
tag
#1091
Conversation
Needs jax-ml/jax#24218. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like JAX unit tests resulted in 20k+ errors and my browser just spent the last 10 mins loading the error log...
Yeah this confuses me because the errors that I see are all of the form
in tests like |
The full command is
I wonder if Nope, |
d10cf20
to
ceff277
Compare
jax-ml/jax#24322 fixes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
//tests/mosaic:flash_attention_gpu
is failing, but I don't think that should block merging this
Fixed in jax-ml/jax#24467. |
No description provided.