-
Notifications
You must be signed in to change notification settings - Fork 89
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: new cuda kernels #2922
test: new cuda kernels #2922
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files |
62600e2
to
e64497c
Compare
e64497c
to
a3c59e7
Compare
a3c59e7
to
f668fc5
Compare
0ff16ae
to
a175ff4
Compare
a175ff4
to
bf9e269
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.
It's great to have all these tests, which indicate which high-level functions are working now and ensure that they won't stop working as long as we run tests on a computer with a GPU.
I ran the tests on my GPU:
tests-cuda/test_1276_cuda_num.py ......... [ 5%]
tests-cuda/test_1276_cuda_transfers.py ............... [ 14%]
. [ 14%]
tests-cuda/test_1276_cupy_interop.py . [ 15%]
tests-cuda/test_1276_from_cupy.py ..... [ 18%]
tests-cuda/test_1300_same_for_numba_cuda.py .......... [ 24%]
............. [ 31%]
tests-cuda/test_1381_check_errors.py . [ 32%]
tests-cuda/test_1809_array_cuda_jit.py .............. [ 40%]
tests-cuda/test_2922a_new_cuda_kernels.py ............ [ 47%]
...................................................... [ 79%]
.... [ 82%]
tests-cuda/test_2922b_new_cuda_kernels.py ............ [ 89%]
.................. [100%]
==================== 169 passed in 9.44s =====================
So this is ready to merge! Since I know you won't be adding anything more today, I'll merge it. Any more tests should go into a new PR.
No description provided.