-
Notifications
You must be signed in to change notification settings - Fork 69
Improve power status for PowerState capable TVs #300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
I was having issues with the power state on my S90C where once it was turned off it would no longer update until I reloaded the integration. Tested the code changes on this PR and it seems to have fixed the issue! @ollo69 could you please take a look at this PR? 🙏 |
This PR seems like it works for me (same symptom as @starkillerOG ). |
alos works for me. @starkillerOG thanks |
Works on S95B. @ollo69 can you merge this pls? |
So far very early days but works for me on 2x QE50QN90CATXXU 50 TV's was having the issue on both. I would set the TV's up, it would initially show the tv status as on. Then power off the TV, then turn the TV back on. At that stage I would just get an off status reported in HA Now I can power the TV on either from the remote. Hope this gets added in as it seems to be working well, without having to change the default port! @Collo69, it would be awesome if this fix could be merged! |
Any progress on reviewing this PR? It appears to fix a major pain point. |
@ollo69 could you take a look? |
@starkillerOG Thank you! I can confirm that your update fixed this issue for me on a 2024 Frame TV with Firmware 1201. I am still having issues with Art Mode - to which I posted a proposed solution in the issue. I may create a new issue for that, once this one gets merged / resolved. Thanks again! |
I can also vouch for this fix solving my issues. |
This solved my issues as well. I do want to point out that I was still getting notifications popping up every time I turned on my Samsung The Frame TV (2023) asking if I would allow home assistant to access the tv. Going to Settings -> Connection -> External Device Manager -> Device Connect Manager -> Access Notification and changing the value to "First Time Only" did not work at first. The problem was that my integration was connected to the TVs ethernet IP address. Once I removed the TV from the integration and re-added using the Wifi IP address, the "First Time Only" settings works as it should. |
Hi folks, |
@funandez yeah that's what I did, just used the VSCode plugin on Home Assistant and copy-pasted the content into the original one |
this fix works on my Samsung The Frame (32" 2023) |
Interesting, do you still get the art status on yours? mine seems to have stopped reporting |
Tested on a Samsung S95C.
Before the power status would turn on just after the TV was already off, it was unreliable.
With this change the power status is 100% acurate.
Based on updates of the build in samsung tv integration.