-
Notifications
You must be signed in to change notification settings - Fork 65
androidtv 12 & 13 #340
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
androidtv 12 & 13 #340
Conversation
|
Hi, the PR pass all of the CI checks. Can you merge it and publish new releases please? |
|
|
||
| return constants.CMD_HDMI_INPUT | ||
|
|
||
| def _cmd_volume_set(self): |
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.
I think it would be simpler to take volume_level as a parameter and return the formatted string.
| new_volume = int(min(max(round(self.max_volume * volume_level), 0.0), self.max_volume)) | ||
|
|
||
| await self._adb.shell("media volume --show --stream 3 --set {}".format(new_volume)) | ||
| await self._adb.shell(self._cmd_volume_set().format(new_volume)) |
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.
This change needs to be made in basetv_sync.py, too.
add support for androidtv 12 & 13. Tested. Also changed the way to set the volume from PR #333.
Reformatted code by black formatter