Skip to content
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

chore(gpu): remove device synchronization in drop for CudaVec #1461

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

agnesLeroy
Copy link
Contributor

closes: please link all relevant issues

PR content/description

Check-list:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Relevant issues are marked as resolved/closed, related issues are linked in the description
  • Check for breaking changes (including serialization changes) and add them to commit message following the conventional commit specification

@agnesLeroy agnesLeroy force-pushed the al/remove_device_sync branch 2 times, most recently from 545ed9f to a6a609b Compare August 8, 2024 07:35
@zama-bot zama-bot removed the approved label Aug 8, 2024
@agnesLeroy agnesLeroy force-pushed the al/remove_device_sync branch from a6a609b to befaae1 Compare August 8, 2024 09:09
@agnesLeroy agnesLeroy force-pushed the al/remove_device_sync branch from befaae1 to 39cee66 Compare September 2, 2024 08:17
@zama-bot zama-bot removed the approved label Sep 2, 2024
@agnesLeroy agnesLeroy force-pushed the al/remove_device_sync branch 2 times, most recently from a7fdde2 to ef671d7 Compare September 2, 2024 13:08
@agnesLeroy agnesLeroy force-pushed the al/remove_device_sync branch 2 times, most recently from c3b85ab to 1e45326 Compare September 3, 2024 07:21
@agnesLeroy agnesLeroy merged commit d454b53 into main Sep 5, 2024
67 of 68 checks passed
@agnesLeroy agnesLeroy deleted the al/remove_device_sync branch September 5, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants