-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Add elevenlabs integration docs #32322
Conversation
It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the |
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
59d65bf
to
1fd4df8
Compare
WalkthroughWalkthroughThe integration of ElevenLabs text-to-speech (TTS) into Home Assistant introduces a new capability for converting text into natural-sounding speech. The Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
@sorgfresser There is a PR missing for the brands repo, can you take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (2)
source/_integrations/elevenlabs.markdown (2)
19-19
: Possible missing preposition.Consider adding "of" for clarity.
- Check the [complete list of supported languages](https://elevenlabs.io/languages) for an overview which languages can be used. + Check the [complete list of supported languages](https://elevenlabs.io/languages) for an overview of which languages can be used.Tools
LanguageTool
[uncategorized] ~19-~19: Possible missing preposition found.
Context: ...levenlabs.io/languages) for an overview which languages can be used. Check your Voic...(AI_EN_LECTOR_MISSING_PREPOSITION)
21-21
: Preposition usage improvement.Consider using "of" for clarity.
- Check your VoiceLab voices for a description on which voices are available for use. + Check your VoiceLab voices for a description of which voices are available for use.Tools
LanguageTool
[uncategorized] ~21-~21: The preposition “of” seems more likely in this position.
Context: ... your VoiceLab voices for a description on which voices are available for use. Ch...(AI_EN_LECTOR_REPLACEMENT_PREPOSITION)
description: Instructions on how to setup ElevenLabs text-to-speech with Home Assistant. | ||
ha_category: | ||
- Text-to-speech | ||
ha_release: 2024.4.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ha_release: 2024.4.3 | |
ha_release: 2024.8 |
ha_integration_type: integration | ||
--- | ||
|
||
The `elevenlabstts` text-to-speech platform uses the official [ElevenLabs SDK](https://github.com/elevenlabs/elevenlabs-python) to read a text with natural sounding voices. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's an integration not a platform
ha_integration_type: integration | ||
--- | ||
|
||
The `elevenlabstts` text-to-speech platform uses the official [ElevenLabs SDK](https://github.com/elevenlabs/elevenlabs-python) to read a text with natural sounding voices. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why it's relevant it's using the official SDK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I just remove the whole line then?
Edit: Done
|
||
## Service speak | ||
|
||
The `tts.speak` service is the modern way to use TTS. Add the `speak` action, select the entity for your ElevenLabs TTS (it's named for the model and voice you created it with), select the media player entity or group to send the TTS audio to, and enter the message to speak. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a new way of naming services
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The DeviceInfo we added? Otherwise I would be glad if you could tell me what exactly you are referring to.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
@joostlek I felt like I do not need the Brands-Pr due to home-assistant/brands#4118 Could you tell me whether this suffices? |
2d71391
to
1716789
Compare
Add elevenlabs integration docs
1716789
to
89798c0
Compare
It should be moved to the core integration folder |
ha_integration_type: integration | ||
--- | ||
|
||
The **ElevenLabs** text-to-speech {% term integrations %} adds support for [ElevenLabs](https://elevenlabs.io/) as text-to-speech to read a text with natural sounding voices. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there any prerequisites? Do I need to have something before I can add the integration? an ElevenLabs account, credentials, API key?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a draft for prerequisites
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @sorgfresser 👍
Thanks for the corrections @c0ffeeca7! |
are you good with the changes, @joostlek ? |
Last question I have, Furthermore looks good |
Co-authored-by: Simon <80467011+sorgfresser@users.noreply.github.com>
Thanks @joostlek and @c0ffeeca7 for the reviews and the improvements! :) |
Proposed change
Add docs for the ElevenLabs integration - this should be merged together with the corresponding integration at core.
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit