Skip to content

Conversation

@csu-bot-zivid
Copy link
Contributor

@csu-bot-zivid csu-bot-zivid commented Dec 22, 2025

On Windows you can't let another process open and read a file while it's
being created by the NamedTemporaryPath context. So, we need to manage
the lifetime of this temporary file manually.

@csu-bot-zivid csu-bot-zivid force-pushed the 2025-12-22-update-python-samples branch 2 times, most recently from 245383e to 5035eba Compare December 23, 2025 10:26
On Windows you can't let another process open and read a file while it's
being created by the NamedTemporaryPath context. So, we need to manage
the lifetime of this temporary file manually.
@SatjaSivcev SatjaSivcev force-pushed the 2025-12-22-update-python-samples branch 3 times, most recently from 2334a2a to dba7a7d Compare December 23, 2025 10:45
@SatjaSivcev SatjaSivcev changed the title Samples: Automatic updates to public repository Samples: HandEyeGUI: Fix RTDE connection Windows Dec 23, 2025
Copy link
Collaborator

@SatjaSivcev SatjaSivcev left a comment

Choose a reason for hiding this comment

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

lgtm

@SatjaSivcev SatjaSivcev merged commit 4022f7b into master Dec 23, 2025
3 checks passed
@SatjaSivcev SatjaSivcev deleted the 2025-12-22-update-python-samples branch December 23, 2025 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants