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

colorKelvin behavior #10

Open
kaohlive opened this issue Dec 28, 2023 · 2 comments
Open

colorKelvin behavior #10

kaohlive opened this issue Dec 28, 2023 · 2 comments

Comments

@kaohlive
Copy link

Hi,

Thanks for this great module, really helped me implement the local api quickly into my Homey app.
I have everything working perfectly except the color Kelvin part.
In the cloud API I get an missing element of the Kelvin value if the mode is color and a filled version when the mode is color temp.
In your implementation I always get the colorKelvin value.
So it is now impossible to detect when what color mode is used.
Can that be added?

Another related issue, in the cloud version a device has a range for the color Kelvin, and I seem to miss that here. I need it to calculate a percentage because my platform uses color temp percentages. THe ranges seem to differ between Goveee devices.
Is that info that can be exposed in your module also?

Thanks for the great work, I am sure we will make a lot of Homey users happy by adding the local api device support.

Kaoh

@kaohlive
Copy link
Author

Ok, sorry for this, I read the documentation of the local API better. I see there is a range mentioned there, so assuming thats fixed in this api between 2000K-7000K. That answers my first question.
The second one, it states the kelvin value can be 0 to indicate its not the active mode. The other way around is a bit more tricky but based on the app and the messages I conclude its, color rgb is than 255,255,255.
So if I receive a color white I can retrieve the Kelvin from the state and use that. If I receive any other color I can assume I need to ignore the Kelvin. I am going forward with this for now.

Again, great work, love the implementation.

@kaohlive
Copy link
Author

One thing I keep missing, is an update event in case of just colorKelvin is updated.

@kaohlive kaohlive reopened this Dec 29, 2023
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

No branches or pull requests

1 participant