Skip to content

brine temperatures T13/14/16/17#194

Draft
flyingmoro wants to merge 1 commit intoveista:masterfrom
flyingmoro:hps_brine
Draft

brine temperatures T13/14/16/17#194
flyingmoro wants to merge 1 commit intoveista:masterfrom
flyingmoro:hps_brine

Conversation

@flyingmoro
Copy link
Copy Markdown

Hello Veista,

i was unable to find some temperatures from the HPS or GEO part of my CompactP2/Geo3, especially i was interested in the ones you see on the pictures from the Compact Remote Control called "T13 Vorlauf Sole" (hps brine input) and "T14 Rücklauf Sole" (hps brine return). It was pretty hard to find the registers for it but after trying mayn different ones i finally found them. They are a bit weired compared to the others as they are so different. But when you look at the attached plot, you can see, that they actually produce reasonable values, when i switch on the Geo part. This pull request is not meant to be merged as is, its just, what currently works for me. Would be great, if you can take a look at it and then we can discuss, how we can exactly put it in, if possible at all.

Regarding T16 and T17 on my remote control display i wasn't sure, how to deal with those because they were already available but the code was a bit unclear to me so i changed the implementation as well but that can be removed.

Just tell me how you like to have it and i will adopt.

signal-2025-11-06-151243

Screenshot 2025-11-06 151429

@veista
Copy link
Copy Markdown
Owner

veista commented Mar 9, 2026

Hi, sorry I do not understand what you are missing. Can you please provide your device type?

@flyingmoro
Copy link
Copy Markdown
Author

Hello veista and thanks for looking into my PR!
Sorry, if i was not clear enough. I have a CompactP2/Geo3 and i found out, that the values for T13/T14 for the Geo3 part, one of them was not giving any values and the other one was missing completely. I am talking about the temperatures of the cold water going into the Geo3 and leaving it. I mean the geothermal heat input and output. With device type, do you mean Compact P2/Geo 3 or the type of my control unit? I think, the latter is called CTS602.

Was i able to explain myself better now?

@flyingmoro
Copy link
Copy Markdown
Author

maybe when you look at the registers i used to create that plot it becomes more clear.


central_heat_heat_ext_set = 1800

hps_input_t17_supply = 1900
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would you edit the register map? These are pretty much 1:1 to Nilan documentation.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

@veista
Copy link
Copy Markdown
Owner

veista commented Mar 19, 2026

maybe when you look at the registers i used to create that plot it becomes more clear.

I understand that you are missing some features or some features do not work for you as intended. You have renamed everything everywhere so I get confused what it was you were trying to do, sorry.

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.

2 participants