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

Counter - SetOption79 to enable reset of counters after telemetry sent #7355

Merged
merged 3 commits into from
Dec 28, 2019

Conversation

andrethomas
Copy link
Contributor

@andrethomas andrethomas commented Dec 28, 2019

Description:

This PR adds functionality to enable the resetting of counters after telemetry data is sent.

This is useful for counting the number of triggers from any given external signal for the entire duration of teleperiod (usually 300 seconds / 5 minutes) from for example the impulses generated by the optically isolated output of energy meters or the number of impulses received in any given telemetry period from a wind speed sensor.

It should default to SetOption79 0 which means this is not enabled by default.

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest dev branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR.
  • The code change is tested and works on core 2.6.1
  • The code change pass travis tests. Your PR cannot be merged unless tests pass
  • I accept the CLA.

Used to enable resetting of the counter values to zero after telemetry data was transmitted.
Add support for resetting counters on tele_period if SetOption78 is enabled.
@arendst
Copy link
Owner

arendst commented Dec 28, 2019

You'll need to move your code down as currently the Domoticz users won't see any change ;-)

@arendst arendst merged commit 172e315 into arendst:development Dec 28, 2019
arendst added a commit that referenced this pull request Dec 28, 2019
Add command ``SetOption79 0/1`` to enable reset of counters at teleperiod time by Andre Thomas (#7355)
@andrethomas andrethomas deleted the counter branch January 2, 2020 20:33
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.

2 participants