LG Soundbar is triggered by media player #41121

Snoopy2406 · 2020-10-03T09:32:16Z

The problem

my home assistant keeps my soundbar on and i can't turn it of anymore. When i push the powerbutton on the remote it says goodbye and restarts itself immediately.

Environment

Home Assistant Core release with the issue: 0.115.6
Last working Home Assistant Core release (if known): /
Operating environment (OS/Container/Supervised/Core): HassOS 4.13 on Raspi 4
Integration causing this issue: media player
Link to integration documentation on our website:

Problem-relevant `configuration.yaml`

Traceback/Error logs

2020-10-03 11:10:20 WARNING (MainThread) [homeassistant.helpers.entity] Entity media_player.lg_soundbar_e797e6bc_013a_41c4_8df7_38516a852e9c is incorrectly being triggered for updates while it is disabled. This is a bug in the lg_soundbar integration

Additional information

The soundbar is not integrated as a device. It always gets an entity. I can deactivate the entity and will get the warning in the logs:

When i push the turn off button ot the switch long, the soundbar turns off. But when i'm using it again, home assistent detects it again as a new entity - with another number behind lg_soundbar_...

I tried:
discovery:
ignore:
- lg_soundbar
- lg
- media_player

that didn't work

also tried:

discovery:
ignore:
- google_cast

media_player:

platform: lg_soundbar

that didn't work either

I'm not using the soundbar with ha so i just would love to remove it fully to have a chance to switch it of again

The text was updated successfully, but these errors were encountered:

frenck · 2020-10-03T10:08:05Z

Hi there,

Thanks for opening up an issue. When creating an issue, an issue template is presented. It looks like you've (partly) removed it. This causes your issue report to be incomplete.

Please edit your issue to have the full issue template back again and provide as much of the requested data as possible. Without it, we won't be able to help you.

Thanks already! 👍

amonhk · 2020-10-05T14:39:12Z

Same problem with SK10Y,
if I turn off the soundbar by holding down the off button, it is no longer reachable from the network and it does not wake up by itself.
Unfortunately, when I turn it back on via the remote control, the connection with the homeassitant is not restored and I have to restart the entire home-assistant to resume its functions.

probot-home-assistant · 2020-10-06T08:07:08Z

lg_soundbar documentation
lg_soundbar source
_{^{(message by IssueLinks)}}

Megachip · 2020-10-10T11:12:51Z

@frenck any option to disable lg_soundbar Auto-detection?

Snoopy2406 · 2020-10-13T06:59:56Z

I enabled the logs in more detail, here is what i got:

2020-10-13 06:40:15 INFO (SyncWorker_10) [homeassistant.loader] Loaded lg_soundbar from homeassistant.components.lg_soundbar

2020-10-13 06:40:17 INFO (MainThread) [homeassistant.components.media_player] Setting up media_player.lg_soundbar

2020-10-13 06:46:10 INFO (MainThread) [homeassistant.components.discovery] Found new service: lg_smart_device {'host': '192.XXX.XXX.XX', 'port': XXXX, 'hostname': 'Soundbar-c45d8433-4d89-bb09-5763-xxxxxxxxxxx-fritz-box.local.', 'properties': {'PROTOVER': '1.0', 'UUID': '57f77c6c-71a4-4004-bfac-exxxxxxxxxxxxxx', 'Model_Type': '0', 'ASSISTANT': 'true'}}

2020-10-13 06:46:10 INFO (MainThread) [homeassistant.components.media_player] Setting up media_player.lg_soundbar

2020-10-13 06:46:10 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new media_player.lg_soundbar entity: media_player.lg_soundbar_57f77c6c_71a4_4004_bfac_xxxxxxxxxxxxxxxx

....after disabling the entity manually i got this again...

2020-10-13 08:43:35 WARNING (MainThread) [homeassistant.helpers.entity] Entity media_player.lg_soundbar_57f77c6c_71a4_4004_bfac_xxxxxxxxxxxx is incorrectly being triggered for updates while it is disabled. This is a bug in the lg_soundbar integration

lasconic · 2020-10-14T15:48:27Z

See also google/python-temescal#7

bernimoses · 2020-10-18T15:51:48Z

There are two problems here:

The soundbar uses a different uuid when announcing itself via zeroconf which leads to multiple entities
The current component uses polling which wakes the soundbar from standby and is not necessary with python-temescal version > 0.2 anymore

As unique id i am testing the hostname since that should not change unless the user does so. And so far it looks ok. What did you @amonhk use as unique id for homeassistant?

amonhk · 2020-10-19T07:45:01Z

@bernimoses I have used hostname (self._host)
I also tested the name (self._name) but the expected result did not return.

patatman · 2020-10-29T09:58:46Z

@bernimoses Is there anything I can help with to close this issue?
I have a similar issue explained here: #35825 (comment)

I have a LG DSN11rg, and I'm happy to help test or provide additional information if needed.
For now I disabled the discovery component, so I can switch my soundbar off.

bernimoses · 2020-10-29T17:08:15Z

@patatman both (this and your) issue should be fixed with my pull request #42044. If you want you can test it (via custom_component). But it is currently stuck because of a unique id problem (see pr). Nothing we can do for now if nobody has a satisfying idea.

patatman · 2020-11-02T08:55:47Z

@bernimoses I just tried to use the custom_component, and to be clear how I set it up:
Took your fork/branch and copied the files from your branch: __init__.py, media_player.py and manifest.json and copied them to my custom_components/lg_soundbar directory.
enabled Discovery and Restarted HASS.

Unfortunatly I'm getting this error:

home-assistant   | 2020-11-02 09:48:59 ERROR (MainThread) [homeassistant.components.media_player] Error while setting up lg_soundbar platform for media_player
home-assistant   | Traceback (most recent call last):
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 205, in _async_setup_platform
home-assistant   |     await asyncio.gather(*pending)
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 314, in async_add_entities
home-assistant   |     await asyncio.gather(*tasks)
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 505, in _async_add_entity
home-assistant   |     await entity.add_to_platform_finish()
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 531, in add_to_platform_finish
home-assistant   |     self.async_write_ha_state()
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 296, in async_write_ha_state
home-assistant   |     self._async_write_ha_state()
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 314, in _async_write_ha_state
home-assistant   |     attr = self.capability_attributes
home-assistant   |   File "/usr/src/homeassistant/homeassistant/components/media_player/__init__.py", line 815, in capability_attributes
home-assistant   |     sound_mode_list = self.sound_mode_list
home-assistant   |   File "/config/custom_components/lg_soundbar/media_player.py", line 172, in sound_mode_list
home-assistant   |     modes.append(temescal.equalisers[equaliser])
home-assistant   | IndexError: list index out of range

If I switch input sources on the Soundbar itself, it gives this error in the log:

home-assistant   | 2020-11-02 09:51:56 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
home-assistant   | Traceback (most recent call last):
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 283, in async_update_ha_state
home-assistant   |     self._async_write_ha_state()
home-assistant   |   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 314, in _async_write_ha_state
home-assistant   |     attr = self.capability_attributes
home-assistant   |   File "/usr/src/homeassistant/homeassistant/components/media_player/__init__.py", line 815, in capability_attributes
home-assistant   |     sound_mode_list = self.sound_mode_list
home-assistant   |   File "/config/custom_components/lg_soundbar/media_player.py", line 172, in sound_mode_list
home-assistant   |     modes.append(temescal.equalisers[equaliser])
home-assistant   | IndexError: list index out of range

bernimoses · 2020-11-07T13:09:03Z

@patatman i don't think this should happen if you use the official source and the above mentioned patch. This error is more related to another pull request i created a while ago #38798. But your specific case would need some tuning i guess. I'll updated the pull request.

Zarkstar · 2020-11-12T03:51:30Z

@patatman both (this and your) issue should be fixed with my pull request #42044. If you want you can test it (via custom_component). But it is currently stuck because of a unique id problem (see pr). Nothing we can do for now if nobody has a satisfying idea.

This is probably a useless comment, but... wouldn't a static ip (eventually with a manual setup) solve this problem? I know it would not be a zero-effort discovered component anymore, but the component is practically unusable at this point, so...
I'm not very good with this stuff, so if it's a bad idea shut me up 😅

patatman · 2020-11-12T11:11:33Z

@Zarkstar That's actually not a bad idea.
I've just enabled the static ip, and I'll see what happens to the integration. So far I haven't seen errors in the log.

Zarkstar · 2020-11-28T23:20:46Z

Hi, any news on the implementation? I would love to help test it with my SK10Y but I can't find the documentation on how to implement custom components. I tried copying the media_player.py file, the init file and the manifest file in a folder inside custom_components, but it did not work (the soundbar is still randomly turning on). At this point, maybe the best thing would be to remove the discovery option (allowing users who don't want to connect the soundbar in this problematic stage of integration) fo lg_soundbar and just have a manual configuration in which the user can insert the ip if static, or the MAC address. Thank you in advance for the update

nunocardosonuc · 2020-12-12T01:39:48Z

I have the same problem with a SK09Y.

frenck added the needs-more-information label Oct 3, 2020

frenck added integration: lg_soundbar and removed needs-more-information labels Oct 6, 2020

amonhk mentioned this issue Oct 16, 2020

Thoughts about socket connection life cycle google/python-temescal#7

Open

bernimoses mentioned this issue Oct 18, 2020

Fix duplicate lg_soundbar entities and disable polling #42044

Merged

21 tasks

bernimoses mentioned this issue Nov 7, 2020

Add check and remove temporary fix for lg soundbar eq and source list #38798

Merged

20 tasks

bernimoses mentioned this issue Feb 2, 2021

Ignore not implemented lg_soundbar source/equaliser #45868

Merged

21 tasks

balloob closed this as completed in #42044 Feb 3, 2021

github-actions bot locked and limited conversation to collaborators Mar 5, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

LG Soundbar is triggered by media player #41121

LG Soundbar is triggered by media player #41121

Snoopy2406 commented Oct 3, 2020 •

edited

Loading

frenck commented Oct 3, 2020

amonhk commented Oct 5, 2020

probot-home-assistant bot commented Oct 6, 2020

Megachip commented Oct 10, 2020

Snoopy2406 commented Oct 13, 2020 •

edited

Loading

lasconic commented Oct 14, 2020

bernimoses commented Oct 18, 2020

amonhk commented Oct 19, 2020

patatman commented Oct 29, 2020

bernimoses commented Oct 29, 2020

patatman commented Nov 2, 2020 •

edited

Loading

bernimoses commented Nov 7, 2020

Zarkstar commented Nov 12, 2020

patatman commented Nov 12, 2020

Zarkstar commented Nov 28, 2020 •

edited

Loading

nunocardosonuc commented Dec 12, 2020

LG Soundbar is triggered by media player #41121

LG Soundbar is triggered by media player #41121

Comments

Snoopy2406 commented Oct 3, 2020 • edited Loading

The problem

Environment

Problem-relevant configuration.yaml

Traceback/Error logs

Additional information

frenck commented Oct 3, 2020

amonhk commented Oct 5, 2020

probot-home-assistant bot commented Oct 6, 2020

Megachip commented Oct 10, 2020

Snoopy2406 commented Oct 13, 2020 • edited Loading

lasconic commented Oct 14, 2020

bernimoses commented Oct 18, 2020

amonhk commented Oct 19, 2020

patatman commented Oct 29, 2020

bernimoses commented Oct 29, 2020

patatman commented Nov 2, 2020 • edited Loading

bernimoses commented Nov 7, 2020

Zarkstar commented Nov 12, 2020

patatman commented Nov 12, 2020

Zarkstar commented Nov 28, 2020 • edited Loading

nunocardosonuc commented Dec 12, 2020

Snoopy2406 commented Oct 3, 2020 •

edited

Loading

Problem-relevant `configuration.yaml`

Snoopy2406 commented Oct 13, 2020 •

edited

Loading

patatman commented Nov 2, 2020 •

edited

Loading

Zarkstar commented Nov 28, 2020 •

edited

Loading