Skip to content

DDF for Moes vibration sensor (_TZ3000_bmfw9ykl)#6753

Merged
manup merged 1 commit intodresden-elektronik:masterfrom
Smanar:tuya_3357
May 25, 2023
Merged

DDF for Moes vibration sensor (_TZ3000_bmfw9ykl)#6753
manup merged 1 commit intodresden-elektronik:masterfrom
Smanar:tuya_3357

Conversation

@Smanar
Copy link
Collaborator

@Smanar Smanar commented Feb 16, 2023

  • Product name: Moes vibration sensor
  • Manufacturer: _TZ3000_bmfw9ykl
  • Model identifier: TS0210

See #6270

WARNING: this device is a vibration sensor, but used as ZHAPresence. The users prefer this mode for the "auto return to false"

@manup
Copy link
Member

manup commented Feb 20, 2023

WARNING: this device is a vibration sensor, but used as ZHAPresence. The users prefer this mode for the "auto return to false"

If it's a vibration sensor it should be ZHAVibration.

I'm currently not remember, but how do we handle this for other ZHAVibration sensors?

@Smanar
Copy link
Collaborator Author

Smanar commented Feb 20, 2023

I'm currently not remember, but how do we handle this for other ZHAVibration sensors?

It's an IAS device, only the presence alarm have special stuff in IAS code. Other just make a new notification at every detection.

@BabaIsYou
Copy link
Contributor

I'm currently not remember, but how do we handle this for other ZHAVibration sensors?

It's an IAS device, only the presence alarm have special stuff in IAS code. Other just make a new notification at every detection.

Could it be something related to this #6167 ?

@manup
Copy link
Member

manup commented Mar 16, 2023

@SwoopX @Smanar @ebaauw I'd like to have some more opinions on this. I understand the reasons but it doesn't feel right to expose a vibration sensor as ZHAPresence.

@SwoopX
Copy link
Collaborator

SwoopX commented Mar 16, 2023

Generally, I do not see any reason why we should do such a transformation. However, in this particular case, I assume it was made to have a certain mechanism kick in, which apparently is limited to presence sensors only.

Feels more like fighting the symptoms rather than the root cause and reminds me on one of my open PRs in this regard 🙂

@manup manup changed the title DDF add support for Moes vibration sensor _TZ3000_bmfw9ykl DDF for Moes vibration sensor (_TZ3000_bmfw9ykl) May 25, 2023
@manup manup added this to the v2.22.1-beta milestone May 25, 2023
@manup
Copy link
Member

manup commented May 25, 2023

Ok screw it, it's a presense sensor now, Tuya being Tuya.

@manup manup merged commit ab7cbdc into dresden-elektronik:master May 25, 2023
@Smanar Smanar deleted the tuya_3357 branch April 30, 2024 15:11
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