search

hacf-fr / home-assistant-core

:house_with_garden: Open source home automation that puts local control and privacy first
https://www.home-assistant.io
Apache License 2.0
7 stars 6 forks source link

synology_dsm camera enhancement #34

Closed S-Przybylski closed 3 years ago

S-Przybylski commented 4 years ago

The problem

Dear @Quentame Since i noticed that the good woking synology camera integration will be removed in HA 0.118, I investigated the enhanced synology_dsm integration in the current HA 0.116.0.dev. (I already posted this in a feature request, but it seems that the module is used seldomly https://community.home-assistant.io/t/synology-dsm-camera-enhancements/229763. Therefore I try to get in contact with you using that issue)

Behaviour of new synology_dsm integration (camera) in contrast to the deprecated one: I found out that the new synology_dsm integration tends to create problems by showing this 8MP streams using HLS (no mjpeg) at all (this seems to be the new default). The delay is worse > 10 sec if its working. My Samsung S7 (ha app and browser) and also my older PC which worked well for presenting the lovelace interface up to now often struggle with the fixed 8MP camera streams - see environment. I also noticed thats its vital to create an administrative user instead of an surveilance station user.

The old integration used mjpeg for presenting (very fast <= 3 sec delay) with medium stream resolution, which is sufficient for a mounted tablet at all.

Perhaps the current documentation could be enhanced to first mention the capability of the camera enhancement and second to explain how to use the camera stuff of the synology_dsm module (perhaps explain how to create a lovelace object and which options (save picture / save movie is available).

Environment

My configuration: I personally use 4 x 8MP cameras with three streams connected to the synology NAS 713+. All three streams are configured: One with 8MP, one with 1MP and one as a kind of thumbnail stream.

Problem-relevant configuration.yaml

none - configured via integration

Traceback/Error logs

none

Additional information

At the end i dream to have an easy to use dashboard for my camera modules:

I have several ideas to improve that module:

What do you think? Do you see any chance to get some ideas into the integration?

Quentame commented 4 years ago

Hi !

Thanks for your feedback on testing and great ideas of improvements !!! :bulb:

I love the idea to add the media_browser capability to Surveillance Station with an history media_player.

The camera platform uses RTSP to get the stream from cameras. Taking a snapshot or record a camera stream is built in the HA camera platorm, so it should work (not tested actually).

I can check the resolution/protocol involed in camera streams, but I don't have an ETA for you since I have multiple integrations to maintain. @shenxn : Would you like to take a look ?

Thanks a lot.

Linked with home-assistant/core#40490 & home-assistant/core#39838

shenxn commented 4 years ago

@Quentame I'll look into it but I'm currently working on another integration that needs quick fix. Since there seems to be a lot of problems, I think it is better to move the removing schedule to a latter release. At least we should fix all those problems before removing the old one.

Quentame commented 4 years ago

@shenxn

Since there seems to be a lot of problems, I think it is better to move the removing schedule to a latter release.

From https://community.home-assistant.io/t/synology-dsm-camera-enhancements/229763 , I see 2 problems:

  1. [issue] Create a configuration item to configure the available resolution for streaming in lovelace (perhaps interactively)
  2. [issue] Create a configuration item to configure the available streaming technology HLS or mjpeg for lovelace (perhaps interactively)
  3. [shipped] Enable actions to save the stream and also pictures in a pre-defined quality (independend what lovelace presents) either built in lovelace or using Synology NAS methods (I am not aware of).
  4. [feature_request] History module (perhaps realized using the new HA media capabilities).

Have you seen more ?

I think it is better to move the removing schedule to a latter release. At least we should fix all those problems before removing the old one.

I agree, we can do that.

shenxn commented 4 years ago

@Quentame So there is one more. Allow users to add only cameras (no system monitoring sensors) with a non-administrative account.

S-Przybylski commented 4 years ago

Dear @shenxn dear @Quentame I love you passion to work on that! Great! From my perspective it could be also an idea, if you additionally put the capability of the old integration into the new one (using HLS and medium resolution) as a parallel second stream (for each camera). Then you get rid of the old integration as planned and the new one has a additional feature. If I am not wrong, its a more or less a different API call (but I am not a progammer).

Regarding the user account: It was a remark not more. I personally can live with that!

S-Przybylski commented 4 years ago

Dear @shenxn dear @Quentame today i notice that the synology_dsm integration doesn't provide the status (HA state recording) if the stream is recorded by the Disk station itself. I only see the state idle. Is it possible to enhance your integration too?

Quentame commented 4 years ago

today i notice that the synology_dsm integration doesn't provide the status (HA state recording) if the stream is recorded by the Disk station itself. I only see the state idle. Is it possible to enhance your integration too?

Tested on my side and working. Be careful of your home mode setting.

S-Przybylski commented 4 years ago

I do not use the home mode feature. Its off.

Quentame commented 4 years ago

if the stream is recorded by the Disk station itself

What do you mean with Disk Station ? Now, when there is the [REC] icon on the Surveillance Station camera, the recording status is well retrieved in HA.

S-Przybylski commented 4 years ago

I mean my Synology DS713+ Interesting, I don't see that in the entities provided by synology_dsm integration (is always idle), while the old synology integration presents the correct status.

github-actions[bot] commented 3 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.