Steve Easley

Results 13 comments of Steve Easley

I wonder if this PR should be against dev and not your sensor PR. Its not clear what the change is to other reviewers. But maybe I am wrong.

When I commented here I didnt realize you had created PRs. So deleted the comment since I was addressing my questions in the PR. But thanks for the extra explanation....

@msavazzi Looks like you are pointing to the wrong documentation PR.

The documentation that was merged only talks about the binary sensory that was added a few weeks ago (https://github.com/home-assistant/home-assistant.io/pull/31058/files). I see nothing about the new sensors being added in this...

imo if we make it a Sensor Entity now, we can't easily add a Select Entity later without making the integration confusing for users (multiple places for the reported state),...

> Correct, but the point is not to add a Select Entity as is not how the Projector works :) Projector are: > > * Sensor (read only) > *...

Another way to think about this is that things like: - power state (warming, cooling, etc) - lamp time - color space - etc are all read only and make...

@msavazzi I do understand this would be a fair amount of work for you to implement if we do agree this is the correct route, especially since you don’t plan...

Thanks for the PR @anupmurarka. That fixes the two invalid remote codes, and I added a handful of new ones as well. Hopefully will get into the next HA release...