Stefan Melis
Stefan Melis
> Hey, just checking in on this PR. Do you think its ready for merge soon or do you need any help to finalize it? Hi! The CI pipeline is...
> @smelis When would you be able to merge this PR it's so useful Hi, to be honest I thought it was already merged :) It's not up to me...
> @smelis When would you be able to merge this PR it's so useful I checked the comments, fixed something and pushed the changes. Hope it can be merged now.
Yeah it looks like that's a bug. The check shouldn't have been ```python if not config[CONFIG_PTZ_COMPONENT]: ``` but ```python if not CONFIG_PTZ_COMPONENT in config: ``` Have you tried setting up...
Ok, yeah, the ptz requires at least a 'cameras' property, but as far as I can tell it should work without any camera's defined in it. So, you could try:...
Which config made it work? This one: ``` ptz: cameras: ``` or this one: ``` ptz: cameras: camera_1: onvif_username: # required onvif_password: # required ``` I'd expect it could crash...