Missing sensor for Volume Level

[Headkillah]

First, BIG THX for the integration. Your´s one the only that works with my Soundbar Q995B :)

But the question i have is:

I´m able to change the volume, but there´s only a slider. Is it possible that you can build in a sensor for the volume level like in the official SmartThings Integration (sensor.soundbar_volume)?

That would be fantastic :)

[samuelspagl]

@Headkillah Happy to hear that :) Thanks a lot!

I can add a sensor sure. I'll try to do it tomorrow.

[samuelspagl]

@Headkillah I added a sensor. If you want to (and know how to) you can first try the MR and tell me if its implemented as intended.

[Headkillah]

@Headkillah I added a sensor. If you want to (and know how to) you can first try the MR and tell me if its implemented as intended.

Hey :) Sorry for late answer I'm a litle bit busy ... Will test it this weekend, I promise:)

[Headkillah]

Sensor is present, but the value is not quite correct or is apparently calculated differently. For example: In the SmartThings integration, the volume level is displayed as 6%, but your integration shows me a volume level of 20. If I now set it to 7%, for example, the level in your integration is 23.333333333 :D And unfortunately the sensor is not updated immediately, there is a time delay of approx. 10 seconds.

[samuelspagl]

@Headkillah Yeah I saw your comment in the PR, but didn't get to it yet, sorry, the christmas stress is kicking 😄

So the 10 seconds delay is something I probably can't change, I'll take a look at it again, but homeassistant has some constants for the refresh time of each entity type. For the "MediaPlayer" it is 10 seconds. And I can't change that value, at least as far as I know of.

The wrong value is probably because I forgot converting the units 😄

I'll try to do it as fast as I can.

[samuelspagl]

@Headkillah I just committed a new version. It really was just a minor change, sorry for taking so long. I tested in on my instance and it is now displaying the same value as reported in the Smart Things App.

That is what you wished for correct?

[Headkillah]

@samuelspagl Yes Sir it works now like expected with the level :) If you want you can merge it. But there is another problem, will open another issue.