whisper: Improve support for local custom Whisper models #3988
+15
−2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Updated documentation about `custom_model` usage.
![home-assistant[bot]](https://avatars.githubusercontent.com/in/97978?s=60&v=4){kind=small}
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
home-assistant
bot
added
cla-recheck
cla-signed
and removed
cla-recheck
cla-needed
labels
📝 Walkthrough## Walkthrough
The changes introduce updates to the Whisper add-on for Home Assistant, focusing on enabling and documenting support for local custom Whisper models. The changelog now includes a "2.5.0" section detailing these updates. The documentation has been expanded to provide clear instructions for users on how to use local custom models by copying them into a specific directory and referencing them correctly in the configuration. Additionally, the configuration file has been modified to include a new mapping that allows read-write access to the add-on's configuration directory.
## Changes
| File(s) | Change Summary |
|------------------------|------------------------------------------------------------------------------------------------------------|
| whisper/CHANGELOG.md | Added a "2.5.0" section with entries about local model configuration mapping and documentation updates. |
| whisper/DOCS.md | Expanded documentation for the `custom_model` option, including instructions for using local custom models. |
| whisper/config.yaml | Updated version to 2.5.0 and added a `map` section to enable read-write access to the add-on config directory.|
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant User
participant HomeAssistant
participant WhisperAddon
User->>HomeAssistant: Copy custom Whisper model to /addon_configs/core_whisper/<model-dir>
User->>HomeAssistant: Update config with custom_model: /config/<model-dir>
HomeAssistant->>WhisperAddon: Start add-on with updated config
WhisperAddon->>WhisperAddon: Access custom model from /config/<model-dir>
WhisperAddon-->>User: Use local custom Whisper model for operations |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
whisper/DOCS.md (1)
70-74: Normalize inline code formatting
The path snippet is wrapped in both quotes and backticks. Remove the extra quotes so the code renders cleanly:- "`/config/<your-model-dir>`". For a local model, the path must start with `/config/`, as this is how the add-on accesses your Home Assistant configuration directory through the container's mounted volume. + `/config/<your-model-dir>`. For a local model, the path must start with `/config/`, as this is how the add-on accesses your Home Assistant configuration directory through the container's mounted volume.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
whisper/CHANGELOG.md(1 hunks)whisper/DOCS.md(1 hunks)whisper/config.yaml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (6)
`*/**(html|markdown|md)`: - For instructional content in documentation, use a direct and authoritative tone. Avoid expressions of politeness such as 'may' or 'please', and ensure t...
*/**(html|markdown|md): - For instructional content in documentation, use a direct and authoritative tone. Avoid expressions of politeness such as 'may' or 'please', and ensure the goal of the instruction is fronted.
- Apply the Microsoft Style Guide to ensure documentation maintains clarity and conciseness.
- In step-by-step instructions, front the location phrase in the instructional sentence.
- In step-by-step instructions, front the 'goal' in the instructional sentence.
- In step-by-step instructions, if in doubt what to front, front the 'goal' before the location phrase in the instructional sentence.
- do not hyphenate terms like 'top-right' or 'bottom-left' with 'corner'
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
`*/**(html|markdown|md)`: - Use bold to mark UI strings. - If "" are used to mark UI strings, replace them by bold.
*/**(html|markdown|md): - Use bold to mark UI strings.
- If "" are used to mark UI strings, replace them by bold.
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
`*/**(html|markdown|md)`: - Be brief in your replies and don't add fluff like "thank you for..." and "Please let me know if"
*/**(html|markdown|md): - Be brief in your replies and don't add fluff like "thank you for..." and "Please let me know if"
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
`*/**(html|markdown|md)`: - Use sentence-style capitalization also in headings.
*/**(html|markdown|md): - Use sentence-style capitalization also in headings.
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
`*/**(html|markdown|md)`: do not comment on HTML used for icons
*/**(html|markdown|md): do not comment on HTML used for icons
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
`*/**(html|markdown|md)`: Avoid flagging inline HTML for embedding videos in future reviews for this repository.
*/**(html|markdown|md): Avoid flagging inline HTML for embedding videos in future reviews for this repository.
whisper/CHANGELOG.mdwhisper/config.yamlwhisper/DOCS.md
🔇 Additional comments (2)
whisper/config.yaml (1)
15-17: Add mapping for local‑model support
The newmapentry withtype: addon_configandread_only: falsecorrectly exposes/configfor custom Whisper models. Indentation and placement under the top‑level keys align with the add‑on manifest schema.whisper/CHANGELOG.md (1)
3-7: Document Unreleased changes
The## [Unreleased]section and its bullet entries accurately reflect the addition of the local‑model mapping and the documentation update forcustom_model. The format and placement at the top of the changelog match existing conventions.
agners
changed the title
agners
requested changes
Apr 22, 2025
|
@DT-art1 The backup exclude rule is |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed change
This updates the 'whisper' add-on to clarify how to configure and use a local custom Whisper model.
Changes:
custom_modelusage.Summary by CodeRabbit
Documentation
Chores