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

Fibaro FGT001 Thermostat: wrong channel for 2nd endpoint battery level #1821

Open
Sail-Away opened this issue Nov 16, 2022 · 2 comments
Open

Comments

@Sail-Away
Copy link

Sail-Away commented Nov 16, 2022

  • OH Version: 3.3.0-SNAPSHOT (#2659)
  • Binding Version: 3.3.0.202112230348

If the problem is associated with a device, please provide the following -:

The channel battery-level2 is supposed to report the battery level of the additional external temperature sensor, if coupled. However, it reports the same value as battery-level and battery-level1, which reflect the thermostats internal battery. In the attached generated xml file you can see that indeed different values are reported for the two endpoints.

The issue also was mentioned in the openhab forum some time ago, without replies.

I suppose this issue could be related to the channel definition in the thing-description file for this device in the repository. In line 115, shouldn't it read
<property name="binding:*:PercentType">COMMAND_CLASS_BATTERY:2</property>
instead of
<property name="binding:*:PercentType">COMMAND_CLASS_BATTERY</property>?

Is there any way to fix this without an update of the binding?

@Sail-Away Sail-Away changed the title Fibarot FGT001 Thermostat: wrong channel for 2nd endpoint battery level Fibaro FGT001 Thermostat: wrong channel for 2nd endpoint battery level Nov 20, 2022
@Sail-Away
Copy link
Author

Update: I was able to fix this for me. I changed the thing-description file fgt001_0_0.xml the way indicated in my first post and replaced the version in the binding in my current installation. After deleting (not removing from the controller) and re-adding the device the channel for endpoint 2 now reports the external sensor's battery level as expected.

For reference, the changes can be found in my repository.

However, I'm not sure if and how this could be corrected from the database.

@Sail-Away
Copy link
Author

@cdjackson Chris, do you you have any thoughts on this? Not a big issue for me, but for future updates it would be great getting around without editing the jar file.

Thanks, Christian

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