rstrouse / nodejs-poolController-dashPanel

A dashboard panel for tagyoureit/nodejs-poolController
33 stars 20 forks source link

Issue with ORP alarm (low priority) #49

Closed mzuniga51 closed 2 years ago

mzuniga51 commented 2 years ago

Robert, not something terribly important but:

I keep my pool at CYA of 50 and FC 5 per TFP recommendation for Salt pools. I have found that a ORP level of 630 or so roughly equates to that of my desired FC. A low value since naturally the CYA distorts the probe reading.

Now you know I'm not using the intellichem for chlorine control but I find it useful to give me that rough ORP estimate of when my chlorine would be high or low. Just a ver rough proxi.

So I set my ORP level at 630 at the Intellichem thinking that an alarm will trigger if the ORP is below 630. In actuality tough, the alarm triggers when the level is below 650 wich seem to be the minimum ORP hard coded into the Intellichem (red bar area in the Chemistry Settings).

I dont use the Intellicenter's notifications but i keep having an "ORP Level Low" warning at the dashPanel that triggers my OCD. So, would you consider adding a way to disable those warnigns as a setting in dashPanel?

Thanks

rstrouse commented 2 years ago

This function already exists in dashPanel. If you go to the configuration page in dashPanel for IntelliChem you will see an alarms tab where you can set the ranges. Sadly you will still get the LSI warnings from IntelliChem but you can control the other limits. image

rstrouse commented 2 years ago

Pull njsPC IntelliChem was overwriting our alarm data.

mzuniga51 commented 2 years ago

replay.zip And here is the alarm still on in the main panel after deactivating the ORP alarm B08944A5-9F0C-47F3-82EB-545BBD04D15C Also I just lost the whole Chemistry Settings window. Nothing happens now when I click on the Intellichem