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

„Heizstab setzen“ leads to malfunctions #237

Open
KalaNagHTD opened this issue Oct 13, 2024 · 3 comments
Open

„Heizstab setzen“ leads to malfunctions #237

KalaNagHTD opened this issue Oct 13, 2024 · 3 comments
Assignees
Labels
config Something related to the provided configs

Comments

@KalaNagHTD
Copy link

KalaNagHTD commented Oct 13, 2024

The newly and on my request implemented „Heizstab setzen“ parser which uses the corresponding „relais test“ commands of the HPSU „Ultra“ leads to multiple faults of the control unit over time.
I was able to reproduce the malfunctions of the controller by repeatedly sending the relais test commands using a simple script.
I am sorry for that outcome and highly recommend to remove the parser „Heizstab setzen“ (and the related „Heizstab abfragen“).

regards Karl

@crycode-de
Copy link
Owner

Thank you for your feedback!

As you mentioned in your forum post this issue only appears if "Heizbetrieb" is active. So my idea would be to add some hint about this in the parser name and a mode check in the parser itself. Doing so this function can only be triggered when not heating which may be fine. :)

@crycode-de crycode-de self-assigned this Oct 14, 2024
@crycode-de crycode-de added the config Something related to the provided configs label Oct 14, 2024
@KalaNagHTD
Copy link
Author

KalaNagHTD commented Oct 14, 2024

Well, that is a good idea! So this function can be used at least during summer.
Nevertheless some final testing is recommended, because I never used the heatpump while using the „Heizstab“ function in „Sommer“ mode. So there might have been some hidden malfunctions.

crycode-de added a commit that referenced this issue Nov 2, 2024
This is to prevent malfunctions of the control unit.
Ref #237
crycode-de added a commit that referenced this issue Nov 2, 2024
This is to prevent malfunctions of the control unit.
Ref #237
ioBrokerTranslator pushed a commit to ioBrokerTranslator/ioBroker.canbus that referenced this issue Nov 2, 2024
This is to prevent malfunctions of the control unit.
Ref crycode-de#237
@crycode-de
Copy link
Owner

I've added a check for mode != 3 to fa0a20-set .

Note

This requires the latest adapter version 2.0.0 with some updates and changes how custom parser scripts are interpreted.

Please use with caution and report any issues. :)

ioBrokerTranslator pushed a commit to ioBrokerTranslator/ioBroker.canbus that referenced this issue Nov 3, 2024
This is to prevent malfunctions of the control unit.
Ref crycode-de#237
ioBrokerTranslator pushed a commit to ioBrokerTranslator/ioBroker.canbus that referenced this issue Nov 3, 2024
This is to prevent malfunctions of the control unit.
Ref crycode-de#237
ioBrokerTranslator pushed a commit to ioBrokerTranslator/ioBroker.canbus that referenced this issue Nov 3, 2024
This is to prevent malfunctions of the control unit.
Ref crycode-de#237
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config Something related to the provided configs
Projects
None yet
Development

No branches or pull requests

2 participants