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

Wrong decimal interpretation in VERDICHTER HEIZLEISTUNG (. and ,) #79

Closed
zimbo86 opened this issue Dec 10, 2024 · 8 comments · Fixed by #82
Closed

Wrong decimal interpretation in VERDICHTER HEIZLEISTUNG (. and ,) #79

zimbo86 opened this issue Dec 10, 2024 · 8 comments · Fixed by #82
Assignees
Labels
bug Something isn't working

Comments

@zimbo86
Copy link

zimbo86 commented Dec 10, 2024

When VERDICHTER HEIZLEISTUNG contains decimal places, "." is not considerd as decimal marker
A value of 65,529 results in 65529 (65.529)

image

@kr0ner
Copy link
Owner

kr0ner commented Dec 10, 2024

That is a different issue. Values are supposed to be between 0 and 100 ... what you see is close to uint16_t::max().

@kr0ner kr0ner self-assigned this Dec 10, 2024
@kr0ner kr0ner added the bug Something isn't working label Dec 10, 2024
@kr0ner
Copy link
Owner

kr0ner commented Dec 10, 2024

looks like this is happening when hot water is being produced:
image

For the rest of the time it is normal. Two ideas how to fix it ..
1, Ignore everything that is not a response .... 84f42e6
2, Limit the value range of the sensor to [0,100] and drop values that are outside that range

I hope that 1, fixes it. Can you try that?

@zimbo86
Copy link
Author

zimbo86 commented Dec 10, 2024

hmm not really related to hot water in my case..
image

related fhem plot:
image

Will try these fixes and report back, thank you :)

@kr0ner
Copy link
Owner

kr0ner commented Dec 10, 2024

You still use day/night mode, right? Maybe that has an effect 🤷

@zimbo86
Copy link
Author

zimbo86 commented Dec 10, 2024

You still use day/night mode, right? Maybe that has an effect 🤷

yes, but day for HC is 10-24:00, Hot Water 10-11:30 and 17:30-20:30

@kr0ner
Copy link
Owner

kr0ner commented Dec 11, 2024

Ok, got it ... it is not "Heizleistung" but "Heizleistung relativ" ... which already states that values can be negative, too.
Fix therefore is:
image

@zimbo86
Copy link
Author

zimbo86 commented Dec 11, 2024

perfect!
as value turns positiv at 18:00 Uhr this matches exactly.
image

@zimbo86
Copy link
Author

zimbo86 commented Dec 12, 2024

works like a charm
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants