-
-
Notifications
You must be signed in to change notification settings - Fork 239
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
Color Temperature / Color widget wrongly sends second POST for Color item #2849
Comments
@toomtech as I myself don't use these widgets, perhaps you can provide some info for others about the one you are using? Does it have a name? And/or some config settings? Etc. |
This is all just built in to openHAB and I don't configure any part of the UI manually. The boxes I'm clicking in the gif in linked issue are provided automatically once you define semantics for the items. |
^ |
@andrewfg I have not configured the Locations page nor added the Office widget. At least not directly or manually. That is automatically constructed when I configure the item semantic metadata as outlined here. I'm configuring the item metadata via REST API and the resulting configuration is attached to the previous hue issue here where the relevant bits are Beyond the configuration which appears in the Model tab shown (which is just item configuration) and adding the relevant things and linking their channels to said items, I'm not doing any other configuration. I do not touch anything in the pages tab or anything else that would result in manual UI configuration for this issue. Does that make sense? I found the semantic model confusing for quite a while and I may be confusing my terminology in general, so apologies if so. |
^ |
I'm happy to provide any information you think will help. Just let me know what you need. |
@maniac103 / @lolodomo => handing over to you guys. |
Don't think we can help as it looks like a bug in a Main UI page. I let @florian-h05 confirm. |
Indeed. And to be more specific the issue only occurs on a screen where there are both a |
@florian-h05 if you need it, then @toomtech has a Wireshark pcap that he can (perhaps) share with you. |
I think the root cause of problems here is #1268. |
Yes. That issue has been open since 2022 so I am hoping it can finally be fixed. |
To chime in from a end user perspective, the issue in question makes it difficult to truly replace the hue app (at least in an ergonomic way) with OpenHab MainUI. The hue app itself is very annoying to use once you have more than one bridge to control and it's always nice to consolidate apps to interface with in general obv. If you need anything from me please let me know. I really appreciate everyone's time volunteering on this, can't wait until it's fixed! |
@toomtech many thanks for the pcap. It shows something rather odd. See screenshot below. I am pretty sure that this is an error in Main UI. And to be specific the problem is NOT in the OH standard items slider screen, but IS in your Locations | Office widget.
Your pcap shows the following sequence of events:
Specific observation is that the S value massively changes between events 3. and 4. !! => It makes me suspect that your Client | Locations | Office widget is wrongly sending
HHB
instead ofHSB
. => We need to look into the widget code!Originally posted by @andrewfg in openhab/openhab-addons#17643 (comment)
After the user issues a Color Temperature command, and OH sends respective Color Temperature and Color HSB state update responses, the UI Widget is issuing -- on its own initiative -- an extra Color HSB command. There are two problems here..
The text was updated successfully, but these errors were encountered: