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

Implemented parsing of advanced diagnosis menu contents #20

Merged
merged 12 commits into from
May 28, 2024
Merged

Conversation

tspopp
Copy link
Owner

@tspopp tspopp commented May 28, 2024

See issue #17.

This PR fixes:

  • MITM Mode: Errors were previously not transferred to HMI ⚠️
  • MITM Mode: Race-Condition which leads to dropped bytes in the main channel connection

This PR adds:

  • Process error messages (message id 74)
  • Forward error messages in MITM mode
  • Publish error message content to MQTT
  • Changed and customizable MQTT publish behavior (fixes MQTT - publishing frequency  #19)
  • Possibility to debug and publish raw heat pump messages received on each channel
  • New MQTT topics for main message:
Value MQTT Topic Format Unit Other Information
Present Error Code $root/main/errorCode uint8 0 if there is no present error, else the error-code
Fan Speed $root/main/fanSpeed uint16 rpm Removed since the message contained the pwm level and no rpm value.
Fan-Level PWM $root/main/fanPWM float %
State: PV Input Signal Detected $root/main/statePV bool
Setting: 1st Fan-Level PWM $root/main/settingPWM_1 uint8 %
Setting: 2nd Fan-Level PWM $root/main/settingPWM_2 uint8 %
Setting: 3rd Fan-Level PWM $root/main/settingPWM_3 uint8 %
Setting: Min. Target Temp $root/main/settingMinTargetTemp uint8 °C
Setting: Min. Anti-Legionella Temp $root/main/settingMinLegionellaTemp uint8 °C
Setting: Wattage Heat Element $root/main/settingWattageElement uint16 W
Setting: Boiler Capacity $root/main/settingBoilerCapacity uint16 l
Setting: Boiler Brand $root/main/settingBoilerBrand Enum Atlantic, Thermor, Sauter, No Name
Capability: Has Heat Exchanger $root/main/capabilityHeatExchanger bool
Capability: Has Circulation $root/main/capabilityCirculation bool
Capability: Has PV-Input $root/main/capabilityPVInput bool
Capability: Has Communication $root/main/capabilityCommunication bool
Capability: Has Anti-Dry-Heating $root/main/capabilityAntiDryHeating bool
Raw Message (Debug Mode Only) $root/main/debug string
  • New MQTT Topics for HMI message:
Value MQTT Topic Format Unit Other Information
PV Input Activated $root/hmi/pvInputActivated bool
Raw Message (Debug Mode Only) $root/hmi/debug string
  • New MQTT Topics for error messages:
Value MQTT Topic Format Unit Other Information
Error Number $root/error/%requestId/errorNumber uint8
Error Code $root/error/%requestId/errorCode uint8
Date $root/error/%requestId/date string e.g. 25.5.2024
Time (hh:ss) $root/error/%requestId/time string e.g. 21:29
Water Temperature $root/error/%requestId/waterTemp float
Lower Evaporator Air Temperature $root/error/%requestId/evaporatorAirTempLower float
Upper Evaporator Air Temperature $root/error/%requestId/evaporatorAirTemp float
Fan-Level PWM $root/error/%requestId/fanPWM uint8 %
Total Heatpump Hours $root/error/%requestId/totalHeatpumpHours uint16
Total Heating Element Hours $root/error/%requestId/totalHeatingElemHours uint16
Raw Message (Debug Mode Only) $root/error/debug string

@tspopp tspopp merged commit a248336 into main May 28, 2024
1 check passed
@tspopp tspopp deleted the t/super_secret branch May 28, 2024 20:12
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.

MQTT - publishing frequency
1 participant