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

Add reporting configuration for temperature and humidity channels #870

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

Conversation

cdjackson
Copy link
Contributor

No description provided.

@cdjackson cdjackson added this to the 4.3 milestone Oct 31, 2024
@FranzSchi
Copy link
Contributor

Hi Chris, this feature looks very valueable to me, similar to the configuration of power measurement.
Are you planning to merge this also in soon?

@cdjackson
Copy link
Contributor Author

Yes, this is all coded, but I've not had time to test. Hopefully I'll do this in the next week or so.

@FranzSchi
Copy link
Contributor

Thanks for your feedback, i thought about if those are also affected by the „long“ reporting config change.

@cdjackson
Copy link
Contributor Author

No - different attributes have different formats

@cdjackson
Copy link
Contributor Author

Unfortunately this won't work until openhab/openhab-webui#2882 is fixed because the thing gets reinitialised rather than updating the configuration.

I will merge this in the coming days in the hope that the UI gets fixed.

@FranzSchi
Copy link
Contributor

FranzSchi commented Nov 21, 2024

Sorry to ask that,
But maybe there can be an improved description for this.
There is the Config
“Report On Change”
The minimum change of the attribute value needed to trigger a device state update

Is it like that that this is an absolute number or is that dependent on the Data type?
So simply said does 5 mean
5 Degrees, 0.5. Degrees?
Or in Volt does 5 Mean
5 Volt or 0.5 volt or 0..05 volt?
Or what I've seen is also Illuminance, i guess here it is the exponent -1?

@cdjackson
Copy link
Contributor Author

This is on my list of things to do, but as it requires a larger, more widespread change to the binding to support the improved descriptions, I won't add it to this PR. I plan on adding a description that will be shown in the UI below the input box.

For many of these channels there is a /100 (so temperature and humidity work this way) so 50 would be 1/2% for example.

@kaikreuzer kaikreuzer removed this from the 4.3 milestone Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants