-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Add SPM01V2 data report duration configuration #8843
Conversation
@Koenkk Not sure why GitHub is showing an error message for this PR
Can you help me figure this out when you have a moment? Thanks. |
@@ -9672,6 +9672,13 @@ export const definitions: DefinitionWithExtend[] = [ | |||
e.produced_energy().withDescription('Total reverse active energy'), | |||
e.power_factor().withUnit('%'), | |||
e.ac_frequency(), | |||
e | |||
.numeric('data_report_duration', ea.SET) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.numeric('data_report_duration', ea.SET) | |
.numeric('data_report_interval', ea.SET) |
Would this name make more sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am ok either way, the reason for the name I used is that in Tuya app that is how it is called when updating firmware - see screenshot here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then let's keep it like this.
Thanks! |
With 3.1.3 firmware for SPM01V2 it is possible to "Data report duration set (Threshold value range 5~3600 seconds)". See here.
This change adds support for that into zigbee2mqtt.
This addresses this issue.