Skip to content

Conversation

@dominikkallusky
Copy link
Contributor

@dominikkallusky dominikkallusky commented Feb 20, 2025

Fixing bug discovered in: #8587
In rare cases it is possible that the incoming src images is not safe to use by the encoding stream. Adding proper synchronization directives resolves this issue.

@pytorch-bot
Copy link

pytorch-bot bot commented Feb 20, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/8929

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the fix @deekay42

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the fix @deekay42

@NicolasHug NicolasHug merged commit 111aafb into main Feb 21, 2025
25 of 48 checks passed
@NicolasHug NicolasHug deleted the encode_jpeg_cuda_sync branch February 21, 2025 13:56
facebook-github-bot pushed a commit that referenced this pull request Jul 28, 2025
Reviewed By: scotts Differential Revision: D77997051 fbshipit-source-id: 3b0fc22020c8930219f866ccd64679c5095962e4 Co-authored-by: Wieland Morgenstern <wieland.morgenstern@hhi.fraunhofer.de> Co-authored-by: Nicolas Hug <nicolashug@fb.com> Co-authored-by: Nicolas Hug <nh.nicolas.hug@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment