-
Notifications
You must be signed in to change notification settings - Fork 87
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
test: generate unit tests for cpu and cuda #2938
test: generate unit tests for cpu and cuda #2938
Conversation
706b7fc
to
3104632
Compare
2cb9702
to
e8820ec
Compare
e8820ec
to
7e89d12
Compare
ce0448b
to
282445a
Compare
282445a
to
eff0c94
Compare
c4d9766
to
bd0515c
Compare
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.
This is good! It adds the tests that you need, you can toggle them with status
, I just checked it out and tried them (all good), and I see that they're running in CI.
This can be merged; do so whenever you're ready!
Oh, one other thing: you'll want to include these new directories in Don't merge yet—I'll add them. |
Okay, now it can be merged, when you're ready. |
There is another |
Thanks! |
Generated
unit-tests
forcpu-kernels
(4978 tests which run in the CI too) andcuda-kernels
. For now, I have added astatus
in thekernel-test-data.json
to turn off the cuda tests. As I will fix/add the kernels, I will turn the tests for the kernels on.