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

Buderus RC25 - "hc1 mode type" incorrect value #273

Closed
emcekk opened this issue Dec 29, 2021 · 8 comments
Closed

Buderus RC25 - "hc1 mode type" incorrect value #273

emcekk opened this issue Dec 29, 2021 · 8 comments
Labels
bug Something isn't working
Milestone

Comments

@emcekk
Copy link

emcekk commented Dec 29, 2021

Bug description
First of all, I don't think that's really important, but maybe can be addressed when fixing other issues

When the RC25 is in Auto mode a "hc1 mode type" parameter appears, but it's value is always set to "day", regardless of the current state. See pictures attached below.

Steps to reproduce
Steps to reproduce the behavior.

Expected behavior
When in auto+day mode a value 'day' should be set to "hc1 mode type"

When in auto+night mode, a value 'night' should be set to "hc1 mode type"

Screenshots
Screenshot_20211229_004826_com.android.chrome_edit_185650429532087.jpg

IMG_20211229_010727.jpg

Device information
Screenshot_20211229_005724_com.android.chrome.jpg

Additional context
Add any other context about the problem here.

@emcekk emcekk added the bug Something isn't working label Dec 29, 2021
@MichaelDvP
Copy link
Contributor

Please check the in terminal the telegram 0xAE in these modes. (read 17 AE)
It was a long discussion about the RC20/RC25 settings here.

@proddy
Copy link
Contributor

proddy commented Jan 8, 2022

@emcekk can you look into this so we know whether to fix in the upcoming version

@emcekk
Copy link
Author

emcekk commented Jan 8, 2022

Sure, will give it a deeper look in the evening. I'm currently using 3.4a9 and this field is just blank Screenshot_20220108_113641_com.android.chrome.jpg

@proddy
Copy link
Contributor

proddy commented Jan 8, 2022

ok, we need to see the data from those telegrams when in night and day so we can figure out how to translate it. or you send me over an RC25 ;)

@MichaelDvP
Copy link
Contributor

The empty field is due to a wrong modytype reference here and here, it should be enum_modetype3.

@proddy
Copy link
Contributor

proddy commented Jan 9, 2022

thanks Michael. I've added it to 3.4a11

@emcekk
Copy link
Author

emcekk commented Jan 9, 2022

After flashing 3.4a11 it seems that the mode is recognized correctly

Screenshot_20220109_233611_com.android.chrome.jpg

Screenshot_20220109_233657_com.android.chrome.jpg

And for auto mode:
Screenshot_20220109_233710_com.android.chrome.jpg

"night" is correct at the moment the screenshot was taken and the thermostat was in "Auto" mode.

Sorry for not providing those telegrams, looks like the newborn is eating up most of my time.

Should I still sniff them for you?

@proddy
Copy link
Contributor

proddy commented Jan 10, 2022

if all works as expected no need to grab those telegrams. You can just close this issue.

@proddy proddy added this to the v3.4 milestone Jan 10, 2022
@emcekk emcekk closed this as completed Jan 10, 2022
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

No branches or pull requests

3 participants