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

frequency measurement on ADE7953 #6778

Closed
wants to merge 1 commit into from

Conversation

ljakob
Copy link
Contributor

@ljakob ljakob commented Oct 28, 2019

Description:

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 pre-2.6 => tested on 6.7.1
  • The code change pass travis tests. Your PR cannot be merged unless tests pass
  • I accept the CLA.

@arendst arendst added the Awaiting feedback from Project Owner Awaiting feedback / input from Project Owner label Oct 29, 2019
@arendst arendst added this to the v7.1.0 milestone Oct 29, 2019
arendst added a commit that referenced this pull request Oct 30, 2019
Add frequency to ADE7953 energy monitor as used in Shelly 2.5 by ljakob (#6778)
@arendst
Copy link
Owner

arendst commented Oct 30, 2019

Some small adjustments made and merged.

Thx.

@arendst arendst closed this Oct 30, 2019
@arendst arendst removed the Awaiting feedback from Project Owner Awaiting feedback / input from Project Owner label Oct 30, 2019
@ascillato2 ascillato2 added enhancement Type - Enhancement that will be worked on fixed Result - The work on the issue has ended labels Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type - Enhancement that will be worked on fixed Result - The work on the issue has ended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants