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

Pumpensolldruckwert (Bosch CS5800i AW) missing #1563

Closed
timostark opened this issue Jan 19, 2024 · 14 comments
Closed

Pumpensolldruckwert (Bosch CS5800i AW) missing #1563

timostark opened this issue Jan 19, 2024 · 14 comments
Labels
enhancement New feature or request
Milestone

Comments

@timostark
Copy link

timostark commented Jan 19, 2024

PROBLEM DESCRIPTION

The setting for the "Pumpensolldruckwert" (differencial pressure of the heat pump - sorry do not know the english word) is not available by default but can be added via custom entity.

Additionally the UOM of this setting (mbar) is not available in custom entities.

EXPECTED BEHAVIOUR

Telegram is available (see screenshot of custom entity below). I tested changing the value on my heat pump and I noted the following telegram beeing responsible for changing the value:

000+03:28:27.248 N 12: [emsesp] boiler(0x08) -B-> All(0x00), RC300Set2(0x02CC), data: 04 (offset 9)

(entity: 0x8, type: 0x02CC, Offset: 9, UINT).

The factor is 50, the UoM is mbar. When working on the heatpump the minimum value is 150mbar, the maximum 750mbar (so without the factor: 3 to 15).

SCREENSHOTS

grafik

grafik

After changing on the heatpump:
grafik

Change is reflected in ESP:
grafik

@proddy proddy added the enhancement New feature or request label Jan 19, 2024
@proddy proddy added this to the v3.6.5 milestone Jan 19, 2024
@proddy
Copy link
Contributor

proddy commented Jan 19, 2024

nice! I'm really happy we added the Custom Entity feature

@timostark
Copy link
Author

ya that is an absolutly great feature.. thanks for that! Small question: When writing back to custom entities is the factor taken into account? I hope that the heatpump does a verification, but I don't want to put e.g. 200*50 mbar as a goal of the pressure on the heatpump :)

@MichaelDvP
Copy link
Contributor

Yes, on writing the factor becomes a divider.
You can also use factor 0.05 and UOM bar, the values are rendered with 2 decimals, e.g. 0.15 bar to 0.75 bar.

@MichaelDvP
Copy link
Contributor

The telegram 0x2CC is confusing.
In #508 it contains hc dependent the dhw priority on thermostat, 0x2CC for hc1, 0x2CE for hc2, not in a row.
In #1554 it contains hc dependent some mixer settings, also 0x2CC for hc1, 0x2CE for hc2
Now the boiler/heatpump has also a telegram 0x2CC,
I think we should check 0x2CC on other devices.
Very uncommon to use the same telegram number for different functions.

@timostark
Copy link
Author

I just checked the write access and it is working just fine - so thanks for the confirmation (Changed it to 400mbar and it was written back to the heatpump pump pressure).

@timostark
Copy link
Author

timostark commented Jan 19, 2024

Addition: When changing the value manuelly I actually see the following two requests:

000+03:28:49.904 N 30: [emsesp] thermostat(0x10) -W-> boiler(0x08), RC300Set2(0x02CC), data: 05 (offset 9)
000+03:28:49.962 N 31: [emsesp] boiler(0x08) -B-> All(0x00), RC300Set2(0x02CC), data: 05 (offset 9)

so basically once from the thermostat to the boiler (same telegram 02CC) and once from the boiler to all..

Would you normally write on the thermostat or the boiler in that case?

@MichaelDvP
Copy link
Contributor

00+03:28:49.904 N 30: [emsesp] thermostat(0x10) -W-> boiler(0x08), RC300Set2(0x02CC), data: 05 (offset 9)

Yes, that's a problem. We actually only check if the telegram is registered on the device and set the values.
Luckily for now the 0x2CC is for mixer offset 0, 1, 2, for thermostat offset 3 and for boiler offset 9, the other offsets are not processed.
But we need to add logic how to process these telegrams and to avoid conflicts.

@proddy I think of these rules to process:

  • src broadcast
  • src write to ems-esp (reply to request)
  • dest, only if src is not ems-esp

@proddy
Copy link
Contributor

proddy commented Jan 20, 2024

@proddy I think of these rules to process:

  • src broadcast
  • src write to ems-esp (reply to request)
  • dest, only if src is not ems-esp

That feels like the right priority order. It's hard to think what could be affected and I'm sure there may be some edge cases to work through later. Let's try this out in dev and see what the feedback is.

MichaelDvP added a commit to MichaelDvP/EMS-ESP32 that referenced this issue Jan 20, 2024
MichaelDvP added a commit to MichaelDvP/EMS-ESP32 that referenced this issue Jan 20, 2024
@MichaelDvP
Copy link
Contributor

Ok, i've changed the telegram mapping and added the differential pressure with UOM mbar.
So @timostark can check the mapping in log.

@eaxus90
Copy link

eaxus90 commented Jan 20, 2024

Many thanks for that! Unfortunately, the custom entity is not working for me. Although it is displayed in the EMS-ESP, it does not work (HA says "unknown").

@proddy
Copy link
Contributor

proddy commented Jan 20, 2024

Many thanks for that! Unfortunately, the custom entity is not working for me. Although it is displayed in the EMS-ESP, it does not work (HA says "unknown").

you'll need to provide more details. What does HA's log say? Do you see it in the Settings->Devices&Services->MQTT?

@eaxus90
Copy link

eaxus90 commented Jan 20, 2024

I'm using the latest version of Home Assistant.
Here is the log: 2024-01-20 15:36:46.814 ERROR (MainThread) [homeassistant.components.mqtt.number] Invalid value for number.custom_pumpdiffpress: 350.0 (range 0.0 - 100.0)

it seems to be an invalid value.

edit: yes, I can see it.
edit2: now, home assistant seems to be able to show up the value. Changing ist't possible though. I think it has to do something with the factor:

image

@MichaelDvP
Copy link
Contributor

The custom entities do not have a min/max setting. The 0-100 seems to be a HA default? When scaling to bar it should work.
For the new boiler/hpsetdiffpress the min/max can be set in customizations, but i have also set default to 150-750mbar in the new PR.

@timostark
Copy link
Author

As implemented i will close that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants