BL0937 accuracy/res improvement by slow down readout#1854
Open
listen-obk wants to merge 72 commits into
Open
Conversation
frequency > Hz for low power
Updated pulse handling to account for overflow and changed timestamp calculations.
Added command to set measurement interval dynamically.
dynamic interval
listen-obk
changed the title
Removed the multi-platform CMake workflow configuration.
Added calibration variables for voltage and power measurements.
Refactor pulse counting and measurement intervals for BL0937 driver.
Refactor pulse timestamp handling and measurement calculations for voltage, current, and power. Introduce debug logging for pulse timestamps and measurement changes.
Contributor
|
This looks interesting. What's the effect of OTA updating to this version of you've already got BL0937 configured? Does it breaks stats or keep working as before? Have you tried it on multiple platforms? |
Author
|
Currently I test only on new Tuya plugs with EC6600. OTA from .205 (and to .207) main version worked well but new calibration is required. Because I wanted to see frequency of BL0937 in log and due to restrictions of Scale (raw values in int) I added a factor of 100 for V and 1000 for I and P. There might be better solutions for that. I did not check impact on stats yet - I would expect it will work well. |
32bit only
general uint not for simulator platform
Updated security limits for sample time and count, and enhanced logging for consumption history.
Updated precision handling and logging for power calculations.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
adopt measure intervals to output frequency of bl0937 for more resolution (~1,7W at 1Hz)