Skip to content

Update DDF for battery bindings for Aubess/Tuya temperature and humidity sensors#6718

Merged
manup merged 7 commits intodresden-elektronik:masterfrom
Finne75:Update-battery-bindings
Mar 16, 2023
Merged

Update DDF for battery bindings for Aubess/Tuya temperature and humidity sensors#6718
manup merged 7 commits intodresden-elektronik:masterfrom
Finne75:Update-battery-bindings

Conversation

@Finne75
Copy link
Contributor

@Finne75 Finne75 commented Feb 1, 2023

Set longer read intervals, inspired by this comment in #6697. Remove deprecated file for Aubess multisensor TZ3000.

@BabaIsYou
Copy link
Contributor

Hi @Finne75, what do you think about changing "swversion" "refresh.interval" in snzb-02 DDF accordingly to 86400 like other DDF in this list ? ;-)

@Finne75
Copy link
Contributor Author

Finne75 commented Feb 1, 2023

Hi @Finne75, what do you think about changing "swversion" "refresh.interval" in snzb-02 DDF accordingly to 86400 like other DDF in this list ? ;-)

Sounds like a great idea :)

@BabaIsYou
Copy link
Contributor

Changes for SNBZ-02 are already included in PR #6699 that add support for another device. Then I suggest you to drop this file off this PR. Sorry about that

@SwoopX
Copy link
Collaborator

SwoopX commented Feb 1, 2023

I'd prefer not to make those changes to battery reporting, as this results in just a single report per 24h. Then you got other people complaining that the devices do not work properly anymore and they want to have more reports...

@BabaIsYou
Copy link
Contributor

I'd prefer not to make those changes to battery reporting, as this results in just a single report per 24h.

          "at": "0x0021",
          "dt": "0x20",
          "min": 3600,
          "max": 84600,
          "change": "0x00000002"

means for me minimum 1 hour, maximum 1 day, or any change of 2 degree ... Am I wrong ?

@Finne75
Copy link
Contributor Author

Finne75 commented Feb 2, 2023

Changes for SNBZ-02 are already included in PR #6699 that add support for another device. Then I suggest you to drop this file off this PR. Sorry about that

No problem. How do I do this, should I make a commit with the original version before your PR?

@Finne75
Copy link
Contributor Author

Finne75 commented Feb 2, 2023

I'd prefer not to make those changes to battery reporting, as this results in just a single report per 24h.

          "at": "0x0021",
          "dt": "0x20",
          "min": 3600,
          "max": 84600,
          "change": "0x00000002"

means for me minimum 1 hour, maximum 1 day, or any change of 2 degree ... Am I wrong ?

How about maximum half (42300) a day as a compromise?

@BabaIsYou
Copy link
Contributor

How about maximum half (42300) a day as a compromise?

With a "refresh.interval": 3600, on the battery resource I don't see why we should change this binding.

@manup manup added this to the v2.21.1-beta milestone Mar 16, 2023
@manup manup changed the title Update DDF for battery bindings for some temp/humidity sensor Update DDF for battery bindings for Aubess/Tuya temperature and humidity sensors Mar 16, 2023
@manup manup merged commit a7cc244 into dresden-elektronik:master Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments