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

adding support for no-salt systems and flow rate #51

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

tednv
Copy link

@tednv tednv commented Jan 22, 2024

  • increasing check frequency to 1 minute
  • adding flow rate from api
  • adding support for no salt (oxidizing, etc) systems
  • fixing deprecated unit warnings

i have an oxidation iron filter system that does not use salt but does backwash nightly

the api shows daily usage, average usage, flow rate, and several other stats. integration was working until recent update, which broke because the salt values were not being found. this change adds a checkbox "Uses Salt" to integration configuration, which can be turned off for systems which do not use salt. There are several other ecowater systems which are either plain filters or other types which do not use salt, this should work for those as well

Ted and others added 6 commits January 22, 2024 13:44
- adding flow rate from api
- adding support for no salt (oxidizing, etc) systems
- fixing deprecated unit warnings
@almoney
Copy link

almoney commented Jan 29, 2024

  • increasing check frequency to 1 minute

  • adding flow rate from api

  • adding support for no salt (oxidizing, etc) systems

  • fixing deprecated unit warnings

i have an oxidation iron filter system that does not use salt but does backwash nightly

the api shows daily usage, average usage, flow rate, and several other stats. integration was working until recent update, which broke because the salt values were not being found. this change adds a checkbox "Uses Salt" to integration configuration, which can be turned off for systems which do not use salt. There are several other ecowater systems which are either plain filters or other types which do not use salt, this should work for those as well

Going to 1 minute polling could trigger the Chamberlain Effect and we will lose this integration. 10x load. I actually changed my trigger back to 30 minutes.

@Lk1975
Copy link

Lk1975 commented Sep 29, 2024

I think that one data that I really miss is to know when ( # of days) regeneration has been done

@barleybobs
Copy link
Owner

  • increasing check frequency to 1 minute
  • adding flow rate from api
  • adding support for no salt (oxidizing, etc) systems
  • fixing deprecated unit warnings

i have an oxidation iron filter system that does not use salt but does backwash nightly

the api shows daily usage, average usage, flow rate, and several other stats. integration was working until recent update, which broke because the salt values were not being found. this change adds a checkbox "Uses Salt" to integration configuration, which can be turned off for systems which do not use salt. There are several other ecowater systems which are either plain filters or other types which do not use salt, this should work for those as well

@tednv How does this behave in the latest v4.0.0 version? Do you still have issues with the values not being found?

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

Successfully merging this pull request may close these issues.

4 participants