-
Notifications
You must be signed in to change notification settings - Fork 161
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
Advanced PUB/SUB example with multithreaded fast subscribers for realtime processing #34
Advanced PUB/SUB example with multithreaded fast subscribers for realtime processing #34
Conversation
Thank you very much for your pull request @philipp-schmidt! It looks great. Let me take a few days to look at it and test it and I'll post my feedback and test results in this thread. Should be able to merge in in less than a week. Well done. |
Thanks again for you pull request @philipp-schmidt. I have merged it into the pub-sub-broadcast branch. There were only minor merge conflicts, nothing significant. Next steps:
Thanks again for your great example. Great piece of code. I really appreciate your contribution. |
Hi @philipp-schmidt, |
I tested your example programs with multiple combinations of Raspberry Pi's and Macs and multiple combinations of Webcams and PiCameras. Your programs work very well. I did make some changes to make it more PiCamera friendly (like adding a 2 second warmup delay for PiCameras). I will leave the pub-sub-broadcast branch active for a while for reference, but Master contains your example code and documentation and, of course, lists you as a contributor. Thanks again for your contribution...I think your examples will solve the "slow subscriber" problem for the folks in issue #27 . I will put a note in that issue pointing folks to your examples. |
Also, I saw your comment in |
Hi Jeff, looks good to me. Thanks for all the time and work you put into this repository, very much appreciated. Glad I could help! |
Implemented publisher and subscriber examples.
I used OpenCV, as I don't have a PiCam here right now. Should be easy to implement for Pi though.
Let me know if it works and feel free to change the docu.