Skip to content

Fix test_is_done without a sync #305

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 16, 2024
Merged

Fix test_is_done without a sync #305

merged 1 commit into from
Dec 16, 2024

Conversation

vzhurba01
Copy link
Collaborator

Event might not have yet completed when a synchronize is skipped
@vzhurba01
Fix test_is_done without a sync
7811065 
Event might not have yet completed when a synchronize is skipped
@vzhurba01 vzhurba01 self-assigned this Dec 16, 2024
@copy-pr-bot copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Dec 16, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.
@vzhurba01
Copy link
Collaborator Author

/ok to test
@leofang
Copy link
Member

leofang commented Dec 16, 2024

Q: Is this for beta 2 or beta 3?
@leofang leofang added bug Something isn't working cuda.core Everything related to the cuda.core module labels Dec 16, 2024
@vzhurba01 vzhurba01 merged commit eeda1ab into NVIDIA:main Dec 16, 2024
30 checks passed
@vzhurba01
Copy link
Collaborator Author

Beta 2 test fix.
@leofang leofang added this to the cuda.core beta 2 milestone Dec 16, 2024
@leofang leofang added the P0 High priority - Must do! label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cuda.core Everything related to the cuda.core module P0 High priority - Must do!
2 participants
@vzhurba01 @leofang