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
[WIP/draft] Bring Python wrapper using pybind11 back #501
Conversation
We ran into threading issues with callbacks, and presumably into GIL locking issues as well. Therefore, for now, let's just use sync interfaces.
We should use the user callback functionality.
This is a first proof of concept. This now needs to added to the build.
Why crazy - or crazy "just for Python" (or just crazy because you know @JonasVautherin :-) )? I'm biased, but from what I can see the end result of the gRPC/Rx pipeline should be very consistent - which is a happy thing for a tech writer. |
I do this because it makes it is more readable, is consistent with other enums, structs used in a plugin class, and also because it is inconsistent with other plugins and therfore made the pybind stuff hard.
This solves the pip install problem.
All this stuff is just moved to the DronecodeSDK-Python repository.
They are moved to DronecodeSDK-Python/examples.
This is now replaced with https://github.com/Dronecode/DronecodeSDK-Python |
@JonasVautherin we might want to cherry-pick 164801a |
Yep that would be fine for me! |
Done in #595. |
This is another try with pybind11 and based on the work from #37.