Skip to content

Onvif: Clarified the need to have at least one H.264 video stream present#41570

Merged
c0ffeeca7 merged 6 commits into
home-assistant:currentfrom
fencepost-error:patch-1
Nov 26, 2025
Merged

Onvif: Clarified the need to have at least one H.264 video stream present#41570
c0ffeeca7 merged 6 commits into
home-assistant:currentfrom
fencepost-error:patch-1

Conversation

@fencepost-error
Copy link
Copy Markdown
Contributor

@fencepost-error fencepost-error commented Nov 2, 2025

Proposed change

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant Bot added the current This PR goes into the current branch label Nov 2, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 2, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit ce77ce1
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6926efe487d1a80008551819
😎 Deploy Preview https://deploy-preview-41570--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread source/_integrations/onvif.markdown Outdated
@home-assistant home-assistant Bot marked this pull request as draft November 3, 2025 06:41
@home-assistant
Copy link
Copy Markdown

home-assistant Bot commented Nov 3, 2025

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@c0ffeeca7 c0ffeeca7 changed the title Clarified the need to have at least one H.264 video stream present Onvif: Clarified the need to have at least one H.264 video stream present Nov 3, 2025
Comment thread source/_integrations/onvif.markdown Outdated
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tiny structure tweak. Let me know what you think.

Comment thread source/_integrations/onvif.markdown Outdated
Comment thread source/_integrations/onvif.markdown Outdated
@c0ffeeca7 c0ffeeca7 marked this pull request as ready for review November 26, 2025 12:16
Copilot AI review requested due to automatic review settings November 26, 2025 12:16
@home-assistant home-assistant Bot requested a review from c0ffeeca7 November 26, 2025 12:17
Comment thread source/_integrations/onvif.markdown Outdated
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @fencepost-error 👍

@c0ffeeca7 c0ffeeca7 enabled auto-merge (squash) November 26, 2025 12:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a troubleshooting section to the ONVIF integration documentation to clarify that cameras must have at least one H.264 video stream configured. The documentation explains a common setup issue where newer cameras default to H.265 encoding, which causes the "No usable cameras were found" error message.

Key Changes

  • Adds a new "Troubleshooting" section to the ONVIF integration documentation
  • Documents the H.264 requirement and provides resolution steps for H.265 compatibility issues
Comments suppressed due to low confidence (3)

source/_integrations/onvif.markdown:1

  • The troubleshooting section structure doesn't follow the Home Assistant documentation standard. According to the template, troubleshooting issues should have a descriptive heading (e.g., "Can't set up the device"), followed by subsections for "Symptom", "Description" (optional), and "Resolution". The heading should describe the problem, not be labeled "Symptom: Error message:". Consider restructuring this as:
### Camera not detected during setup

#### Symptom

The ONVIF integration shows an error message "No usable cameras were found".
---

source/_integrations/onvif.markdown:125

  • The subsection "Cause" is not part of the standard troubleshooting structure in Home Assistant documentation. According to the template and observed patterns, troubleshooting sections should follow the structure: issue heading, "Symptom" (optional), "Description" (optional), and "Resolution". The cause information should be integrated into the "Description" subsection or the resolution text. Consider removing this subsection and incorporating the explanation into the resolution or adding a "Description" subsection before "Resolution".
    source/_integrations/onvif.markdown:127
  • [nitpick] Corrected spelling of 'video coding' to 'video compression'. H.265 provides improved video compression efficiency, not 'video coding' (which is a more general term). While 'video coding' is technically correct in a broad sense, 'compression' is more precise and commonly used when comparing H.264 and H.265.


### Symptom: Error message: "No usable cameras were found"

The ONVIF integration shows an error message "No usable cameras were found".
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line duplicates content from the heading. According to the documentation template, the symptom subsection should expand on or clarify the error, not simply repeat the heading. Either remove this redundant line or provide additional context, such as "When trying to set up the integration, you see the error message 'No usable cameras were found'."

Suggested change
The ONVIF integration shows an error message "No usable cameras were found".
When you try to set up the ONVIF integration in Home Assistant, you may see the error message "No usable cameras were found" if the integration cannot detect any compatible video streams from your camera.

Copilot uses AI. Check for mistakes.
Comment on lines +120 to +121
Update the camera configuration to output at least one video stream in H.264 format rather than H.265. One option for doing this is to set a secondary stream to H.264 while leaving the primary stream at the default H.265.

Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The resolution lacks specific guidance on how to update the camera configuration. Consider adding more detailed steps or referencing the camera's documentation, such as: "Update the camera configuration to output at least one video stream in H.264 format. You can usually configure this in the camera's web interface or app under video stream settings. One option is to set a secondary stream to H.264 while leaving the primary stream at H.265. Refer to your camera's documentation for specific instructions on changing encoder settings."

Suggested change
Update the camera configuration to output at least one video stream in H.264 format rather than H.265. One option for doing this is to set a secondary stream to H.264 while leaving the primary stream at the default H.265.
To resolve this issue, you need to update your camera's configuration so that it outputs at least one video stream in H.264 format. Most cameras allow you to do this in their web interface or mobile app, typically under **Video**, **Stream**, or **Encoding** settings.
1. Open your camera's web interface or app.
2. Navigate to the **Video** or **Stream** settings section.
3. Look for options to set the video encoding format for each stream.
4. Change the encoding for at least one stream (often called the secondary or sub stream) to **H.264**. You can leave the primary stream set to **H.265** if you prefer.
5. Save your changes.
If you are unsure how to access these settings or change the encoding format, refer to your camera's user manual or manufacturer's support website for specific instructions.

Copilot uses AI. Check for mistakes.
@c0ffeeca7 c0ffeeca7 merged commit aaf4e0b into home-assistant:current Nov 26, 2025
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants