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 support for POPP Smart Thermostat #5279

Merged
merged 1 commit into from
Jan 22, 2022
Merged

Conversation

SwoopX
Copy link
Collaborator

@SwoopX SwoopX commented Sep 3, 2021

See #5278.

@SwoopX SwoopX linked an issue Sep 3, 2021 that may be closed by this pull request
@J-N-K
Copy link

J-N-K commented Nov 2, 2021

Is there something blocking this PR? I have several requests for support from my application‘s users.

@Mimiix
Copy link
Collaborator

Mimiix commented Nov 2, 2021

@J-N-K Manups time.

@Pe-MaKer
Copy link

@manup @Mimiix
sorry for poking you. Is there any chance of getting this merged soon?
I'm sitting on eight of these thermostats for my entire house and can't control them. The winter is coming. 🥶
I tried to compile it from @SwoopX branch by myself but it's already 500 commits behind the master which brings other troubles.

If I would know how to put the definition for this Popp derivative of the Danfoss Ally in a DDF, then it would be fine for me as well.

Without your support I'm screwed and the WAF of our new home is at 0.

@Pe-MaKer
Copy link

@manup @Mimiix
I don't want to be a pain, but it would ruin our Advent season if I don't finally master the thermostats.
My wife will ensure there's a fire🔥 otherwise.

Can you please take care that this commit by @SwoopX will make it in the next beta?
@J-N-K did you managed to get a solution by yourself?

@Mimiix
Copy link
Collaborator

Mimiix commented Nov 29, 2021

@Pe-MaKer there's more PR s that are waiting longer. So it isn't fair to prioritize above...

@Mimiix
Copy link
Collaborator

Mimiix commented Nov 29, 2021

I'll ask Manuel, but no promises.

@SwoopX
Copy link
Collaborator Author

SwoopX commented Nov 29, 2021

@Pe-MaKer You can program the thermostats meanwhile via deconz GUI and set weekly schedules via the thermostat cluster. You probably should do that with my branch, as it should ensure the RCT on the device gets synced (check the time cluster attributes to be sure).

What you need to know is that you only have 6 transitions per day at your disposal. The time which needs to be entered is minutes since 00:00, heat set point must be multiplied by 100. The last temperature set on a day will be taken over to the next day until the next change is programmed.
The example below has two periods with comfort temperature set, ending up with 4 transitions and applies for Mon - Fri with the following times:

05:50 -> 21°
09:00 -> 17°
17:00 -> 22°
21:00 -> 17°

grafik

When the schedule is programmed by pressing exec, you need to activate the schedules on the thermostat. This can be done by setting attribute 0x0025 as per below.

grafik

You can override the scheduled temp anytime manually on the device and this will remain until the next transition.

My recommendation would be to overshoot the aimed comfort temperature by 2° in the schedule for the beginning, so set 24° if you want to have 22°.

@Pe-MaKer
Copy link

Pe-MaKer commented Nov 30, 2021

Thank you @SwoopX for the detailed information.
I have made my own and current fork of the master, and incorporated all the changes from your commit there.
https://github.com/Pe-MaKer/deconz-rest-plugin/tree/POPP

Unfortunately, the time sync does not seem to work, as you can see from the screenshots.
If the local time is given as Unix Epoch, the time of the termostat is about 35 years
off.

This makes the use of time schedules a challenge ;)

2021-11-30 18_34_15-phoscon - VNC® Viewer for Google Chrome™
2021-11-30 18_35_39-Epoch Converter - Unix Timestamp Converter

@SwoopX
Copy link
Collaborator Author

SwoopX commented Nov 30, 2021

That more feels like the time sensor has not been created. Can you enter sensor search in Phoscon again and read the attributes of the basic cluster? After that, the sync should directly kick in and I know from own experience that it can be annoying like hell.

Edit:
Oh, an I just saw that you haven't applied all changes covered by the PR to your fork. The part allowing for creation of the time sensor is missing in de_web_plugin.cpp.

@SwoopX SwoopX added this to the v2.14.0-beta milestone Jan 9, 2022
@manup manup merged commit 2298403 into dresden-elektronik:master Jan 22, 2022
@SwoopX SwoopX deleted the popp branch February 6, 2022 23:56
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

Successfully merging this pull request may close these issues.

POPP Smart Thermostat (Produktcode: 701721)
5 participants