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.

Sign up for GitHub

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

Jump to bottom

[FR] % written "pour cent" not taken into account in brightness. #2064

Open
mdelpire opened this issue Mar 8, 2024 · 1 comment
Open

[FR] % written "pour cent" not taken into account in brightness. #2064

mdelpire opened this issue Mar 8, 2024 · 1 comment

Comments

@mdelpire
Copy link

mdelpire commented Mar 8, 2024

Hi,

I tested assist in French to turn my lights on and it works great. Then I tried to use it to set the brightness of some dimmable lights. The issue is that my speech to text sometimes translate % as "pourcent" (correct orthography) and sometimes as "pour cent" (incorrect orthography but pronounced exactly the same. When the second is picked, I got the message that HA doesn't know the room "50 pour cent" for instance. Is there a way to make it understand as a synonym?

Thanks
slegars56 added a commit to slegars56/intents that referenced this issue Mar 16, 2024
@slegars56
[Fr] Add "pour cent" in <pourcent>
1def1dd 
Related to home-assistant#2064

Whisper is sometimes returning "pourcent" in two words ("pour cent"). This PR adds this value to the related expansion rule
slegars56 added a commit to slegars56/intents that referenced this issue Mar 16, 2024
@slegars56
[Fr] Add "pour cent" in the expansion rule <pourcent>
138b56c 
this PR is related to home-assistant#2064

In Fench, Whisper is sometimes returning "pourcent" in two words.
--> "pour cent" added in the related expansion rule
@slegars56 slegars56 mentioned this issue Mar 16, 2024
Merged
@slegars56
Copy link
Contributor

slegars56 commented Mar 16, 2024 •
edited

Hi @mdelpire, I had the same when testing the cover position with local STT. Pull request done.

jlpouffier pushed a commit that referenced this issue Mar 16, 2024
@slegars56
[Fr] Add "pour cent" in the expansion rule <pourcent> (#2088)
41db757 
this PR is related to #2064

In Fench, Whisper is sometimes returning "pourcent" in two words.
--> "pour cent" added in the related expansion rule
schizza pushed a commit to schizza/intents that referenced this issue Mar 16, 2024
@slegars56 @schizza
[Fr] Add "pour cent" in the expansion rule <pourcent> (home-assistant…
1bb983e 
…#2088)

this PR is related to home-assistant#2064

In Fench, Whisper is sometimes returning "pourcent" in two words.
--> "pour cent" added in the related expansion rule
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdelpire @slegars56