Core: Sony Bravia media player volume slider not working

[maartenpaauw]

Home Assistant release with the issue:

0.98.2

Last working Home Assistant release (if known):

Operating environment (Hass.io/Docker/Windows/etc.):

Docker on Synology NAS

Component/platform:

Sony Bravia TV media player
https://www.home-assistant.io/components/braviatv/

Description of problem:

I can't set the volume via the slider. It doesn't do anything.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

media_player
  - platform: braviatv
    name: tv
    host: !secret media_player_tv_host

Traceback (if applicable):

Additional information:

The TV: Sony Bravia KDL-42W805A

[probot-home-assistant[bot]]

Hey there @robbiet480, mind taking a look at this issue as its been labeled with a integration (braviatv) you are listed as a codeowner for? Thanks!

[schmittx]

@maartenpaauw If you have an amp connected, the slider won't work. Refer to the braviatv_psk custom component configuration.

[maartenpaauw]

@schmittx I don't have an amp connected. I use the speakers from the television.

[Hrupek]

@maartenpaauw have you resole the issue? I have the same

[maartenpaauw]

No, unfortunately not.

[itn3rd77]

I can't set the volume nor switch the program. The rest is working fine. Any way to debug whats going on?

[Extreme79]

I have the same issue. Slider isn't working to set volume.
Tapping on 🔊 change the volume instead.
I've a pioneer amp (integrated as onkyo) connected to the TV (the slider for that device is working).

[Siytek]

I also have the same problem, +/- buttons work but the slider doesn't.

[dcnielsen90]

The problem resides in braviarc.py. We are sending a float and the tv is expecting an integer in string.
Here is a fix:

247     def set_volume_level(self, volume):
248         """Set volume level, range 0..1."""
249         self.bravia_req_json("sony/audio", self._jdata_build("setAudioVolume", {"target": "speaker",
- 250                                                                                 "volume": volume * 100}))
+ 250                                                                                 "volume": str(int(volume * 100))}))

Now because of this #30964, I don't know how to submit a PR for this. It appears @balloob and @andrey-git are maintainers of this fork based off of this -- maybe they'll be able to implement the fix.

In the mean time you can implement the fix yourself if you're feeling ambitious.

[balloob]

Bravia is unmaintained. I am only on the package because I run Home Assistant. I am not involved.

If someone wants to fork the original package at https://github.com/aparraga/braviarc, figure out what is different in the braviarc-homeassistant package on pypi, add those changes and start maintaining it, that would be great.

Until then, all Bravia RC bugs will remain unsolved.

[dcnielsen90]

I'm looking into it now. We are currently at adac2af Based on my testing, everything looks good up to commit 28b431a. It appears 2719869 broke something with toggling the TV on/off. Regardless -- this will fix this issue and some other erroneous logs. I'll see if I can get a PR in.