Skip to content

Hot fix/zagor py11#47

Closed
kefeimo wants to merge 4 commits intoVOLTTRON:developfrom
kefeimo:hot-fix/zagor-py11
Closed

Hot fix/zagor py11#47
kefeimo wants to merge 4 commits intoVOLTTRON:developfrom
kefeimo:hot-fix/zagor-py11

Conversation

@kefeimo
Copy link
Contributor

@kefeimo kefeimo commented Oct 28, 2023

TO fix #45.

Note: as a submodule, the deps/pybind11 merge conflict is not trivial to resolve.

Note: for a complete solution we need to update pybind11, see PR: VOLTTRON/pybind11#10

Shawn Greenman and others added 3 commits October 20, 2023 09:37
The ThreadPool owned by DNP3Manager causes a deadlock when it is destroyed without unlocking the GIL.
By returning the DNP3Manager with a custom deleter which unlocks the GIL during destruction the pool can be deleted.
Solution borrowed from: pybind/pybind11#1446
@kefeimo kefeimo closed this Oct 31, 2024
@kefeimo kefeimo deleted the hot-fix/zagor-py11 branch October 31, 2024 15:24
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