added entities for target temp, pH, ORP - #12
Open
emogenet wants to merge 1 commit into
Open
Conversation
Owner
|
Thank you for sharing. How do set a target temperature? And ph low and high? Can't seem to find in my app |
Author
|
So, for these various target values, I believe you may be able to set them
directly from the control box in the pool technical room: it is more
sophisticated than the app.
Also the box has a password-protected "maintainer mode" area that may (I'm
not sure) let you fine-tune the PIDs for the temp, ph and chlorine (for
example the low and high interval for pH)
I found the name of those variables by:
1. editing your code to make it dump the full data it downloads to JSON
every 5min for a couple of hours
2. feeding the entire JSON file to chatgpt and asked it to locate target
values variable names. it got it right on the first run.
The reason I wanted these as HA entities is I wanted to visualize them on a
grafana dashboard that scrapes HA and constructs a time series.
Now, I can see how the chemistry control algorithm of the da-gen box
operates over time, and seeing the target values in that graph really helps
in understanding how it works.
…On Tue, Dec 2, 2025 at 11:04 PM Tobias Laursen ***@***.***> wrote:
*djerik* left a comment (djerik/dagen-ha#12)
<#12 (comment)>
Thank you for sharing. How do set a target temperature? And ph low and
high? Can't seem to find in my app
—
Reply to this email directly, view it on GitHub
<#12 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJ4QLAOF3I4TZP5H5PKGXEL37YEILAVCNFSM6AAAAACN25VXSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMMBUGE2TIMZRGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Author
|
So ... my bad.
For pH and chlorine, the target values I read off of the da-gen box are
correct.
But the box does not (can't as a matter of fact) control the temperature,
and therefore can't know the target temp.
So I'm not sure what the value I'm reading actually represents.
Maybe it is something that my installer copied by hand in the dagen box
config from the heating system ...
…On Wed, Dec 3, 2025 at 12:43 AM Emmanuel Mogenet ***@***.***> wrote:
So, for these various target values, I believe you may be able to set them
directly from the control box in the pool technical room: it is more
sophisticated than the app.
Also the box has a password-protected "maintainer mode" area that may (I'm
not sure) let you fine-tune the PIDs for the temp, ph and chlorine (for
example the low and high interval for pH)
I found the name of those variables by:
1. editing your code to make it dump the full data it downloads to JSON
every 5min for a couple of hours
2. feeding the entire JSON file to chatgpt and asked it to locate
target values variable names. it got it right on the first run.
The reason I wanted these as HA entities is I wanted to visualize them on
a grafana dashboard that scrapes HA and constructs a time series.
Now, I can see how the chemistry control algorithm of the da-gen box
operates over time, and seeing the target values in that graph really helps
in understanding how it works.
On Tue, Dec 2, 2025 at 11:04 PM Tobias Laursen ***@***.***>
wrote:
> *djerik* left a comment (djerik/dagen-ha#12)
> <#12 (comment)>
>
> Thank you for sharing. How do set a target temperature? And ph low and
> high? Can't seem to find in my app
>
> —
> Reply to this email directly, view it on GitHub
> <#12 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AJ4QLAOF3I4TZP5H5PKGXEL37YEILAVCNFSM6AAAAACN25VXSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMMBUGE2TIMZRGM>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Seeing the real-time values of your pool chemistry in HA is very nice, but it's also useful to be able to see the target values.
This PR adds target values for:
temperature
chlorine
pH (low and high)