-
-
Notifications
You must be signed in to change notification settings - Fork 269
refactor(ptz): Migrate from onvif-zeep to onvif-python library #1135
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
Conversation
✅ Deploy Preview for viseron canceled.
|
|
Thanks! I see that |
|
Thank you for your feedback. I use it for corporate projects, and it's worth noting that I also created In this PR, I also made a change to remove the It's also worth noting that the If there are any changes I need to include, please let me know and I will fix them. Thank you again. Note:
|
|
Additionally, since Viseron's ONVIF implementation is limited to |
|
Thanks for the explanation! Sounds good. A proper ONVIF component in Viseron has been requested many times but i have never gotten around to it. Please dont hesitate to ask if you have any questions regarding that. |
|
CI is failing on a conflict for the |
|
Ok I have done the version change. |
|
It looks like the latest CI results are not from my commit error. There it says: Is there anything I can help? |
|
You are correct, its unrelated to your changes. Something changed in Azure which causes that issue. Will merge this now, thanks! |
|
Thank you, I'll start implementing the But thank you again. |
|
Agreed, having it in the UI would be great! I dont own any PTZ cameras myself currently so thats why i havent done it myself yet. |
Replace deprecated
onvif-zeepwith newonvif-pythonRemove
ONVIFServiceimport and update type annotationsAll PTZ functionality remains unchanged
Uses the built-in wsdl file that is included in the
onvif-pythonlibraryBenefits: better device discovery, CLI support, improved error handling
Related issues: