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

Audio URLs #247

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Audio URLs #247

merged 2 commits into from
Sep 9, 2024

Conversation

regulad
Copy link
Contributor

@regulad regulad commented Aug 13, 2024

This adds support for Android IP webcam's Audio URLs. I plan to use this change to enable audio in the Home Assistant integration.

@MartinHjelmare
Copy link
Contributor

How are you planning to use this in Home Assistant?

@regulad
Copy link
Contributor Author

regulad commented Aug 13, 2024

Add streaming support to the Android ip webcam and use ffmpeg to add the AAC stream, a proof of concept is here: https://github.com/regulad/hass-android_ip_webcam-audio/blob/master/custom_components/android_ip_webcam/camera.py

@MartinHjelmare
Copy link
Contributor

I don't think we'll accept that code in the core integration as it has too much AV details and also socket code. That kind of code should be done in a 3rd party library. In a custom integration you can of course do what you want.

Do you still want to pursue this PR?

@regulad
Copy link
Contributor Author

regulad commented Aug 15, 2024

Yes, of course. That is only a proof of concept, I will rework it once this PR is complete and the audio is added to the backend library.

pydroid_ipcam/__init__.py Outdated Show resolved Hide resolved
pydroid_ipcam/__init__.py Outdated Show resolved Hide resolved
pydroid_ipcam/__init__.py Outdated Show resolved Hide resolved
Copy link
Contributor

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Thanks!

@MartinHjelmare MartinHjelmare merged commit c00b346 into home-assistant-libs:master Sep 9, 2024
1 check 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