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

Add SPM01V2 data report duration configuration #8843

Merged
merged 3 commits into from
Feb 21, 2025

Conversation

vilaemail
Copy link
Contributor

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.

@vilaemail
Copy link
Contributor Author

@Koenkk Not sure why GitHub is showing an error message for this PR

Merging is blocked
Merge is not an allowed merge method in this repository.
This branch must not contain merge commits.

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)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.numeric('data_report_duration', ea.SET)
.numeric('data_report_interval', ea.SET)

Would this name make more sense?

Copy link
Contributor Author

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.

Copy link
Owner

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.

@Koenkk Koenkk merged commit fbe5296 into Koenkk:master Feb 21, 2025
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Feb 21, 2025

Thanks!

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