thorpelawrence / alexa-spotify-connect

Control Spotify Connect devices with Alexa
MIT License
97 stars 53 forks source link

More precise volume control #80

Closed visadb closed 3 years ago

visadb commented 3 years ago

My sound system is so powerful that the usable volume range is 1-15%, so currently I can set three volume levels: 0 = 0%: mute 1 = 10%: quite loud 2 = 20%: waaaaay too loud

So basically I would like to revert the part of https://github.com/thorpelawrence/alexa-spotify-connect/pull/5 that changes the volume range from 0-99 to 0-10.

thorpelawrence commented 3 years ago

Hi @visadb, interesting point

Some of the options I could think about for solving this are

I'm not really sure what the best approach to do this would be, as reverting the change from near the start would be quite breaking of current use cases

visadb commented 3 years ago

@thorpelawrence I think the extra intent with the word percent would probably be best. That would not change the behaviour of the current 0-10 volume intent and then I would be able to set volumes between 1-9% which would not be possible with the first option.

The decimal option would be more or less equivalent to the percent intent option but IMHO would be less natural to use, and would be a little awkward to implement code-wise as well.

thorpelawrence commented 3 years ago

@visadb I agree that is probably the best option here then, do you think you might be able to implement it and open a pull request? As otherwise I have been quite busy unfortunately so not sure when I would be able to get around to implementing it

visadb commented 3 years ago

@thorpelawrence I'll try to find the time to implement the volume percent intent and open a pull request.

visadb commented 3 years ago

PR #81 solved this issue for me. Thank you for the quick review, merge and deployment, @thorpelawrence!