Skip to content
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

Incorrect video resolution from Synology camera integration #36999

Closed
Spirituss opened this issue Jun 22, 2020 · 10 comments · Fixed by #39838
Closed

Incorrect video resolution from Synology camera integration #36999

Spirituss opened this issue Jun 22, 2020 · 10 comments · Fixed by #39838

Comments

@Spirituss
Copy link

The problem

I use cameras connected to Synology Surveillance 8.2.2-5766 and Synology camera platform integration for them in HA. Lovelace shows stream video from that cameras only in low resolution despite of any settings of the Synology Surveillance.
In Synology Surveillance I tried to change resolution in:

  • Stream Profile (IP camera -> device settings->video)
  • Schedule Recording Settings (IP camera -> recording settings -> stream)
  • Live View Stream Settings (IP camera -> live view settings -> stream)

I see two issues:

  • Even if all of them have high resolution settings, lovelace shows low res stream in small window which neither can't be resized nor be opened in full screen mode in HA. But in case I use streams from those cameras directly connected via ONVIF HA integration, lovelace shows high res video stream in window with correct appropriate pixel size.
  • For ONVIF direct connection stream the video window has Pause button which allows to stop streaming, while Synology camera integration stream has no one.

Environment

  • Home Assistant Core release with the issue: 0.111.0
  • Last working Home Assistant Core release (if known): None
  • Operating environment (Home Assistant/Supervised/Docker/venv): Home Assistant/Docker
  • Integration causing this issue: Synology camera platform
  • Link to integration documentation on our website: https://www.home-assistant.io/integrations/synology/

Problem-relevant configuration.yaml

Configuration.yaml:

stream:
camera:
  - platform: synology 
    url: !secret surveillance_station
    username: !secret surveillance_station_dsm_user
    password: !secret surveillance_station_dsm_password
    timeout: 60
    verify_ssl: false 

Lovelace.yaml:

...
           - entity: camera.syno_camera
              type: picture-entity 
              camera_image: camera.syno_camera
              camera_view: live   
...

Traceback/Error logs

None

Additional information

@probot-home-assistant
Copy link

synology documentation
synology source
(message by IssueLinks)

@shenxn
Copy link
Contributor

shenxn commented Jun 25, 2020

This is duplicated with #29931. We're currently merging the synology camera integration into synology_dsm integration (ProtoThis/python-synology#53) and the problems you mentioned are going to be fixed with the new merged integration.

@Spirituss
Copy link
Author

This is duplicated with #29931. We're currently merging the synology camera integration into synology_dsm integration (ProtoThis/python-synology#53) and the problems you mentioned are going to be fixed with the new merged integration.

@shenxn When are you going to merge ProtoThis/python-synology#53 with synology_dsm integration? It is really useful adjustment.

@shenxn
Copy link
Contributor

shenxn commented Aug 29, 2020

I'll start working on that soon. Waiting for @Quentame to release a new version of the library.

@Quentame
Copy link
Member

I'll start working on that soon. Waiting for @Quentame to release a new version of the library.

You can start working on it by targeting master as dependency 😉

I would like to merge a new feature before release, a little bit more testing and it can be done.

Also, breaking news, I have a RTSP compatible camera, so I can test Surveillance Station ! 🎉

@Quentame
Copy link
Member

Quentame commented Sep 8, 2020

See https://github.com/ProtoThis/python-synology/releases/tag/0.9.0 and #39819 @shenxn 😉

@Spirituss
Copy link
Author

@Quentame

See https://github.com/ProtoThis/python-synology/releases/tag/0.9.0 and #39819 @shenxn 😉

Wonderfull news! Thanks a lot!

@shenxn
Copy link
Contributor

shenxn commented Sep 9, 2020

@Quentame I already have a draft branch merging the camera into synology_dsm. I'll create a new PR today.

@Spirituss
Copy link
Author

Spirituss commented Sep 11, 2020

@Quentame Great news! Will it be included into 0.115?

@Quentame
Copy link
Member

@Quentame Great news! Will it be included into 0.115?

Into 0.116.0 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants