You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So making this component also have access to that code might fix the issue? If not combined what about extracting an API package for making the calls and both depending on that?
The text was updated successfully, but these errors were encountered:
Ohhh, I wouldn't say it is 100% solved the sleep issue.
I have not tested it for a while, but the logic of the sleep testing is very bad. Trust me, I wrote that code 😝 .
Ideally I would take the time, a lot of time, and rework the whole thing into a python package.
This package should be able to do ALL available commands.
Then have a home assistant component, which includes the pairing process. And the dream thing would be: Add the TV as a device, which has multiple additional components including the ambilights as light and other things as media_player and services.
Interesting, to be honest I had been thinking the same myself. I would love to add autodiscover, pairing and config all in the frontend. There are also an increasing number of bug fixes and features from my other repos that could be merged onto one component. If you are interested @nstrelow let me know if and when you plan on rewriting, and I would be happy to help out merge our two components into a superior media_player, and maybe share some of that work load ;)
Is it worth combining this with the media player component (https://github.com/nstrelow/ha_philips_android_tv)
The main thing I'm thinking of is that you say there is an issue here with the TV going to sleep but that looks to be solved in the media player version here: https://github.com/nstrelow/ha_philips_android_tv/blob/0ee38e1974fda8456e715f7f900fe1effb976a30/custom_components/philips_android_tv/media_player.py#L149
So making this component also have access to that code might fix the issue? If not combined what about extracting an API package for making the calls and both depending on that?
The text was updated successfully, but these errors were encountered: