Fix max volume behaviour

MycroftAI / skill-volume

Mycroft AI official Volume Skill - control the volume of your Device

https://mycroft.ai/skills

Apache License 2.0

10 stars 30 forks source link

Fix max volume behaviour #52

Closed krisgesling closed 5 years ago

krisgesling commented 5 years ago

Max volume set to 10.
if volume set to 10 or 11, dialog indicates it is the maximum level.
fixed shorthand volume setting eg "volume six" as STT transcribes numbers as whole words.
set logging to be debug level to clean up constant log output.

forslund commented 5 years ago

I'm wondering about the number to letter thing and if that's an indication of a mycroft-core bug...The STT phrase is passed through normalize() which should change numbers in text to numbers... ie normalize('volume six') -> volume 6

Anywho, this looks good. Merging