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

Pentair ScreenLogic integration: Spa Salt Cell Level can only be set 0%-20%, should be 0%-100% #72233

Closed
sarahmva opened this issue May 20, 2022 · 10 comments · Fixed by #85725
Closed

Comments

@sarahmva
Copy link

The problem

The SPA Salt Cel Level only allows values from 0-20. The pool salt cell level allows 0-100. It shoudl also be possible to set the SPA Salt Cell Level 0-100.

What version of Home Assistant Core has the issue?

2022.5.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

Pentair Screenlogic

Link to integration documentation on our website

https://www.home-assistant.io/integrations/screenlogic

Diagnostics information

config_entry-screenlogic-b04cfbdeb9a5b2991eadf152e5175219.json.txt

Example YAML snippet

This is the YAML of the generated entity:

            {
                "area_id": null,
                "capabilities": {
                    "min": 0.0,
                    "max": 20,
                    "step": 1.0,
                    "mode": "auto"
                },
                "config_entry_id": "b04cfbdeb9a5b2991eadf152e5175219",
                "device_class": null,
                "device_id": "6aa8ab64d081c45073b1a724636c1347",
                "disabled_by": null,
                "entity_category": null,
                "entity_id": "number.pool_spa_scg_level",
                "hidden_by": null,
                "icon": null,
                "id": "b05aad14ae5b9ec90c4487f980a9f030",
                "name": "Pool Spa SCG Level",
                "options": {},
                "original_device_class": null,
                "original_icon": null,
                "original_name": "Pentair: EC-AA-46 Spa SCG Level",
                "platform": "screenlogic",
                "supported_features": 0,
                "unique_id": "00:c0:33:ec:aa:46_scg_level2",
                "unit_of_measurement": "%"
            },

Anything in the logs that might be useful for us?

No response

Additional information

No response

@probot-home-assistant
Copy link

screenlogic documentation
screenlogic source
(message by IssueLinks)

@probot-home-assistant
Copy link

Hey there @dieselrabbit, @bdraco, mind taking a look at this issue as it has been labeled with an integration (screenlogic) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)

@dieselrabbit
Copy link
Contributor

The 20% limit for the Spa setting was based on the documentation for an EasyTouch system (what I have), page 37 in the manual, specifically listing a 0-20% valid range for the Spa setting. Both the EasyTouch and IntelliTouch manuals give the same information about the controller automatically limiting the amount of chlorine for the Spa setting, but the IntelliTouch manual is less specific about an individualized Spa setting range. There was an assumption that the 0-20% applied to both models. From the diagnostics, you appear to have an IntelliTouch system. Is that correct?

I don't have a chlorinator so I am unable to inspect what values are reported via the protocol adapter or what are actually presented to the user, in either the ScreenLogic app or in the controller interface.

If you could set your spa output to 100% via the app or on the controller, confirm that it is showing you it is set to 100%, and post an updated diagnostics, that would be very helpful for me in understanding how the protocol adapter is reporting the spa setting for your system.

@sarahmva
Copy link
Author

sarahmva commented May 22, 2022

Thank you for checking into this. I am wondering if this is due to the older firmware in my intellitouch i5 (1.7 generation). I do not have a spa, I just use the SPA settings for the pool as well so I have more than 3 choices for pump speed.

Here is a screenshot and the diagnostics data with SPA output set to 100.

image
config_entry-screenlogic-b04cfbdeb9a5b2991eadf152e5175219.json (1).txt

"scg": { "scg_present": 201, "scg_status": { "name": "SCG Status", "value": 129 }, "scg_level1": { "name": "Pool SCG Level", "value": 60, "unit": "%" }, "scg_level2": { "name": "Spa SCG Level", "value": 100, "unit": "%" }, "scg_salt_ppm": { "name": "SCG Salt", "value": 3950, "unit": "ppm" }, "scg_flags": 128, "scg_super_chlor_timer": { "name": "SCG Super Chlorination Timer", "value": 0 }

Spa salt level is at 100...

@dieselrabbit
Copy link
Contributor

Thanks for that. How did you set that value? Through the ScreenLogic app app or via the pool controller hardware?

@sarahmva
Copy link
Author

IMG_5129

I did set it through the app. The intellitouch itself has very little possibilities to set anything. I could change it on the salt cell itself and even there I can set between 0 and 100.

@dieselrabbit
Copy link
Contributor

Thanks. The last question is "Is the ability to set the Spa value to 100% unique to IntelliTouch systems, as the documentation seems to indicate?"

If someone reading this with an EasyTouch and a chlorinator could confirm/deny, that'd be helpful. Otherwise I'll have to wait for a free weekend to falsely reconfigure my system and test.

@github-actions github-actions bot added stale and removed stale labels Jul 15, 2022
@chrismdann
Copy link

It would be great to be able to change the salt level. I show the range but can't change it.

@dieselrabbit
Copy link
Contributor

Thanks. The last question is "Is the ability to set the Spa value to 100% unique to IntelliTouch systems, as the documentation seems to indicate?"

If someone reading this with an EasyTouch and a chlorinator could confirm/deny, that'd be helpful. Otherwise I'll have to wait for a free weekend to falsely reconfigure my system and test.

I was finally able to test this. The ScreenLogic app lets me set 100% for both on my EasyTouch. Seems like the documentation is wrong, or it handles that "limit" internally. Either way, this requires an update to the API which is in-progress.

@sarahmva
Copy link
Author

Thanks so much!

@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants