-
Notifications
You must be signed in to change notification settings - Fork 15
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
A lot of wrong commands in commands_hpsu.json #24
Comments
do you have a reworked commands file? |
Its not totally wrong. Vanni wtched the can bus and stripped this commands. So they can't be totally wrong. But they should be fixed, thats right. At the moment the pyHPSU is very simple: Send a command....read a command and fiddle out the values. If we will do it correct: |
I'm not saying everything's wrong. But it's not all right either and should be fixed. I don't use pyHPSU but develop my own project using ESP8266/32. A small WiFi box which can be stowed under the cover of the Rotex system and which passes the values to emoncms, mqtt etc.. |
Do you have more infos about your project? |
I've been doing / running this for more than 12 months now and I'm currently thinking about doing this OpenSource or if it will stay ClosedSource. I use an ELM327 for communication. |
Hi, |
Hi,
i think there are a lot of wrong commands in
commands_hpsu.json
and the project is using the constructor in the wrong way.now:
should be
another example:
now:
should be:
the constructor for heat generator 1 is always the same:
so the
commands_hpsu.json
should realy look like this (for Sender ID 190):the constructor should than make this:
example with "t_ext":
and the awaited answer is:
we end up with this:
Caution:
under some circumstances rapidly changing from Sender ID 190, heat generator, get 31 to Sender ID 190, heating Circuit, get 61 could turn the rotex into maintenance mode. After that i realized that Sender ID 190 seems to be the display / RoCon at the heat pump itself. The external RoCon that Rotex / Daikin sells uses Sender ID 69D.
Nearly all commands having a 31 equivalent and using 61 is not necessary.
e.g. "t_ext"
The text was updated successfully, but these errors were encountered: