Skip to content

Comments

Add SET42 - SetPumpFlowrate#776

Open
andiwirs wants to merge 2 commits intoEgyras:masterfrom
andiwirs:l_series_set_pump_flowrate
Open

Add SET42 - SetPumpFlowrate#776
andiwirs wants to merge 2 commits intoEgyras:masterfrom
andiwirs:l_series_set_pump_flowrate

Conversation

@andiwirs
Copy link

Hi, i added the SET42 - SetPumpFlowrate similar to the previously added SET39 - SET41. The corresponding TOP106 for the pump flowrate (J/K/L series) already exists.

This is my first pull request ever, hope it is not too wrong :)

Best regards

@geduxas
Copy link
Contributor

geduxas commented Jan 19, 2026

Have you tested? not all values are settable in pump, don't know about K/L series, but H and J don't have settable flow rate only max pump duty

@andiwirs
Copy link
Author

andiwirs commented Jan 19, 2026

I have tested this on my L-series (but only sending the raw MQTT payload, i did not build the Heishamon binary) and it works. Even during active operation TOP106 changes and i can see the effect on the pump flow rate. I added the (J/K/L series) comment in the PR message from the TOP106 documentation.

This payload to heat_pump/SendRawValue switches to deltaT:
b'\xf1\x6c\x01\x10\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x10\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'

and this to max. pump:
b'\xf1\x6c\x01\x10\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x20\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'

@geduxas
Copy link
Contributor

geduxas commented Jan 19, 2026

Whups, it's actually not a flow rate, its flow rate control operation mode.. you set pump operation mode by which control flow rate .. i think naming definitely should be changed.

@andiwirs
Copy link
Author

For example SetPumpFlowrateMode would imo best comply with TOP106. Would you agree?

@geduxas
Copy link
Contributor

geduxas commented Jan 19, 2026

yes i think it is more right for this.

@andiwirs
Copy link
Author

Ok added the change, hope it fits :)

@andiwirs
Copy link
Author

andiwirs commented Feb 15, 2026

Hello is something needed to get it merged?

In the meantime i built the firmware with my PR and tested it.
heat_pump/commands/SetPumpFlowrateMode via MQTT is working fine for me.
It changes between deltaT and Max Flow and i can see the effect on the pumpflowrate

@IgorYbema
Copy link
Contributor

I am working on releasing v4.0 firmware. I'll add this one also

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.

3 participants