bwoodworth / hassio-addons

11 stars 9 forks source link

IntelliClor control #12

Closed bwoodworth closed 4 years ago

bwoodworth commented 4 years ago

Thanks @parnic and @bwoodworth ... Really great work.

I did look in the library and tried setting up HA according to the state values I found:

I am just now realizing that some of these values would need to go into the switch section, not the sensor section (e.g. the salt levels)...

I will wait for the next release.

@bwoodworth : I also have an IC40 chlorinator and a Variflow pump which I can switch to a Varispeed pump. so if you need any data or want me to test something, just let me know :-)

Originally posted by @sarahmva in https://github.com/bwoodworth/hassio-addons/issues/9#issuecomment-658402593

bwoodworth commented 4 years ago

Can you explain a little more about what you are trying to do? There are already some readings coming into the addon like salt ppm. Are there other values you need to be able to pull in as sensors? Are there settings in the IntelliChlor that you need to be able to set?

bwoodworth commented 4 years ago

I think I see what you want and it should be doable in the current version of the library. I will get something in there soon and you can test.

sarahmva commented 4 years ago

D0CCE5D4-F308-4F7B-8BDB-1DAF53E3791B

This is a screenshot of the ScreenLogic App for the chlorinator.

Level1 and 2 would be the Pool and Spa output. Status I assume is operating status and the Superchlorination is an option to run the cell on 100% for 24 hours (never used it, I just shock the pool).

My preference would be to be able to set the output levels. I am sure others would also like the super option.

bwoodworth commented 4 years ago

In new release v1.21. Please test

sarahmva commented 4 years ago

Unfortunately, the Chlorinator shows all "unkown" for the new values.

I also need to figure out what to add to the configuration to be able to control the salt cell. I need to find a good tool to look into mqtt and the different topics there.

image

bwoodworth commented 4 years ago

What does your HA configuration look like?

bwoodworth commented 4 years ago

Looks like you got it working from the other thread. Closing.

jefframsey0 commented 4 years ago

Not sure if the set salt levels are working right. I am trying to send the value for the pool setting increasing it to 70% from the existing 60 as a test. See anything wrong in my test config? I sent the mqtt message manually and via this test config in hassio.

Error from addon logs:

TypeError: this.setSaltCellConfig is not a function at UnitConnection. (/node_modules/node-screenlogic/set_salt:14:12) at UnitConnection.emit (events.js:315:20) at UnitConnection.onClientMessage (/node_modules/node-screenlogic/index.js:315:14) at Socket. (/node_modules/node-screenlogic/index.js:155:15) at Socket.emit (events.js:315:20) at addChunk (_stream_readable.js:295:12) at readableAddChunk (_stream_readable.js:271:9) at Socket.Readable.push (_stream_readable.js:212:10) at TCP.onStreamRead (internal/stream_base_commons.js:186:23) /node_modules/node-screenlogic/set_salt:14 this.setSaltCellConfig(0, level, isNaN); ^ TypeError: this.setSaltCellConfig is not a function at UnitConnection. (/node_modules/node-screenlogic/set_salt:14:12) at UnitConnection.emit (events.js:315:20) at UnitConnection.onClientMessage (/node_modules/node-screenlogic/index.js:315:14) at Socket. (/node_modules/node-screenlogic/index.js:155:15) at Socket.emit (events.js:315:20) at addChunk (_stream_readable.js:295:12) at readableAddChunk (_stream_readable.js:271:9) at Socket.Readable.push (_stream_readable.js:212:10) at TCP.onStreamRead (internal/stream_base_commons.js:186:23)

My config:

switch:

MQTT state image

bwoodworth commented 4 years ago

Interesting. Have you done a rebuild of the add-on since updating to 1.21?

jefframsey0 commented 4 years ago

Yep, saw the comments about the pump info so did post update.

sarahmva commented 4 years ago

Wow... what is that cool MQTT interface? I am still looking for something usable.

(Sorry @bwoodworth for hijacking this thread.)

jefframsey0 commented 4 years ago

Wow... what is that cool MQTT interface? I am still looking for something usable.

(Sorry @bwoodworth for hijacking this thread.)

Its pretty decent... heres a link https://www.microsoft.com/en-us/p/mqtt-explorer/9pp8sfm082wd?activetab=pivot:overviewtab

bwoodworth commented 4 years ago

I fixed the bug. Please test v1.22

sarahmva commented 4 years ago

Thank you so much. Unfortunately, HA does not (yet) show the update...

bwoodworth commented 4 years ago

Can you restart the addon and check the log for salt cell level 1 and 2? Do they match what you tried to set it to?

sarahmva commented 4 years ago

Can you restart the addon and check the log for salt cell level 1 and 2? Do they match what you tried to set it to?

Ah, sorry for the misunderstanding. For version 1.21 I could update it from the Add-on store. 1.22 is not yet available. there. Should I update manually?

bwoodworth commented 4 years ago

On the Add-On Store page you can reload to get the latest versions, but wait a minute. The fix I made won't work. I am working on another one.

bwoodworth commented 4 years ago

v1.23 released and should fix the issue.

sarahmva commented 4 years ago

Fantastic.... And it works :-) I could change the Values in HA and see them change in my Pentair App... Now I just need to find a way to change the units in climate control... Or use something else...

image

bwoodworth commented 4 years ago

put this in your mqtt device config:

unit_of_measurement: '%'

jefframsey0 commented 4 years ago

working great, thanks!

jefframsey0 commented 4 years ago

Can you restart the addon and check the log for salt cell level 1 and 2? Do they match what you tried to set it to?

tested both levels and everything is working perfectly

bwoodworth commented 4 years ago

Fantastic! Closing.