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

[RTSP] Convert H265 to H264: H265 not supported by hls.js #84

Merged
merged 5 commits into from
May 7, 2024
Merged

[RTSP] Convert H265 to H264: H265 not supported by hls.js #84

merged 5 commits into from
May 7, 2024

Conversation

Flobul
Copy link
Contributor

@Flobul Flobul commented Apr 30, 2024

Description

hls.js doesn't support H265 codec or browsers are not all compatibles for H265.
So many camera are not displayed on dashboard.

This PR allows to encode H265 video to H264 and have a low delay.
I just created another rtsp-to-hls.sh file and changed video codec h264 option.
I added another checkbox on eqLogic page to select this option (default: old script to copy video)

Suggested changelog entry

Permet d'encoder à la volée les vidéos en codec H265 en H264.

Related issues/external references

https://community.jeedom.com/t/pb-dintegration-dune-camera-jennov-5mp-ptz/113396/2
https://community.jeedom.com/t/camera-poe-annke-h800/87947/12

Fixes #
Fix black screen on H265 codec cameras

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

  • I have checked there is no other PR open for the same change.
  • I have read the Contribution Guidelines.
  • I grant the project the right to include and distribute the code under the GNU.
  • I have added tests to cover my changes.
  • I have verified that the code complies with the projects coding standards.
  • [Required for new sniffs] I have added MD documentation for the sniff.

@zoic21 zoic21 merged commit 38cf02c into jeedom:beta May 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants