search

merdok / homebridge-webos-tv

Homebridge plugin for LG webOS TVs
MIT License
646 stars 88 forks source link

Make sure that live TV channel is never removed #496

Closed thibaulf closed 10 months ago

thibaulf commented 10 months ago

This fix is to avoid the removal of the LiveTv checkpoint that sometimes occurs during the plugin init

merdok commented 10 months ago

Hi, when does it occur that the live tv input gets removed?

thibaulf commented 10 months ago

On 10 boots, it happens at least 5 times. I have used the plugins for many years and this behavior is relatively new (after a firmware update I think) I'm using a LG TV (49UN7000PUB) running on webos 5.0

merdok commented 10 months ago

Does it only remove the live tv input or all other too? And you mean the input spinner right?

thibaulf commented 10 months ago

Only the live TV disappear in the input spinner

merdok commented 10 months ago

Uffff, this is very strange, can you post some debug log? I think we should investigate why this is happening in the first place before doing some special handling for specific inputs. The debug log might be sometimes too big as it lists all the inputs on your tv, hence if that is happening you can attach it as a file.

thibaulf commented 10 months ago

I agree with you about the debugging: this is always a good thing. However, the live TV input and other HDMI channels are already in a special handling situation when the are added to the list (checkBasicInputs).

I will try to do that. Since this is a "live" system, i will try to get those logs if I can.

merdok commented 10 months ago

Yeah, right. There is the assumption that those inputs are always there as I remember one of the older webostv version did not report them for some reason. I guess logs would be the best approach here so we can try to find the potential issue.

thibaulf commented 10 months ago

Well, I looks like there was something wrong with my setup: cpu usage too high on my poor little pi gen 1...

Issue cannot be reproduced anymore. This PR can be dropped

merdok commented 10 months ago

Thanks for the additional investigation! If the issue pops up again, the please let me know!