Skip to content
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

Improve power status for PowerState capable TVs #300

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

starkillerOG
Copy link

@starkillerOG starkillerOG commented Feb 3, 2024

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.

@NullScope
Copy link

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? 🙏

@ScottESanDiego
Copy link

This PR seems like it works for me (same symptom as @starkillerOG ).

@kamashkin
Copy link

alos works for me. @starkillerOG thanks

@MindFreeze
Copy link

Works on S95B. @ollo69 can you merge this pls?

@SRJohnston
Copy link

SRJohnston commented Aug 28, 2024

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants