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

Onzen (Salt Water System) Support #32

Open
feens opened this issue Oct 10, 2022 · 4 comments
Open

Onzen (Salt Water System) Support #32

feens opened this issue Oct 10, 2022 · 4 comments

Comments

@feens
Copy link

feens commented Oct 10, 2022

I was poking around in the code and other issues/requests seeing if it would be possible to view/control the salt water system on my spa. It looks like it may be possible, but I'm not 100% sure what to look for. I notice it listed in both Output connections are and Actual connections are in the logs, and tried capturing changing some settings while logging as well. I'm a Ruby developer by day but don't have much experience in Python (but I'm sure I could figure it out)...just not sure what to look for or where to start in terms of parsing messages and building a device handler for it, so some guidance would be helpful.

@gazoodle
Copy link
Owner

If the def _scan_outputs(self) function in GeckoAsyncFacade has identified a user demandable feature from the spa pack then you can use that keyed entity to capture an accessor from the spa to control that entity.

If the entity is an "on/off" kind of thing (and looking in the SpaPackStruct.xml, it appears to be so), then it will be pretty straightforward to extend the function to add this to the devices that are controllable.

Personally, I'd duplicate the functionality for GeckoLight (create a new class, duplicate the code where it gets initialized in the _scan_outputs function so that you've added the functionality to the facade class.

Then you need to move over to the HA integration and add the control, again this ought to be as easy as copying the code for the lights.

I don't have time at present to do this, but am happy to have it added to the backlog if it seems like too much to tweak for you?

@patrikrask
Copy link

patrikrask commented Oct 14, 2022 via email

@feens
Copy link
Author

feens commented Oct 22, 2022

@gazoodle I'll have to checks but I don't think it's an on/off. It should be more like the energy settings, where it has a start/time/frequency (aka 4hrs 4 times a day). Not that I have much free time, but I'm happy to plug away at a PR as I have some time, unless you're able to get to it first.

@feens
Copy link
Author

feens commented May 20, 2024

@gazoodle curious if you're around at all to answer a few questions around this. I've started to dig into this now, and I'm also looking to fix the deprecations.

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

No branches or pull requests

3 participants