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

Update ZY-M100_human_breathing_presence.json #8036

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gituser-rk
Copy link

@gituser-rk gituser-rk commented Nov 21, 2024

Added TS0225 (ZG-205Z/A) from manufacturer _TZE200_2aaelwxk. Still several datapoints missing, like light level ("illuminance_raw").
In Zigbee2MQTT it is this integration: https://www.zigbee2mqtt.io/devices/ZG-205Z_A.html

Added TS0225  (ZG-205Z/A) from manufacturer _TZE200_2aaelwxk. Still several datapoints missing, like light level ("illuminance_raw").
@gituser-rk gituser-rk closed this Nov 21, 2024
@gituser-rk gituser-rk reopened this Nov 21, 2024
@manup
Copy link
Member

manup commented Jan 29, 2025

@Smanar do you think this is the right DDF aka a clone, note the different modelid?

@Smanar
Copy link
Collaborator

Smanar commented Jan 29, 2025

I m checking fastly and from Z2M
_TZE200_2aaelwxk/TS0601
_TZE200_2aaelwxk/TS0225 (Different model, but it seem exist too)
_TZE200_kb5noeto/TS0601

Are clone and use as dpid

  • Dpid = 106 for luminance (problem)
  • Dpid = 1 for presence (ok)
  • Dpid = 4 for motion distance (ok)
  • Dpid = 2 For sensitivity (ok)
  • Dpid = 102 for fading time (ok)

dpid 104 id is medium_motion_detection_distance, not luminance
dpid 101 is motion state, an enum, not a delay
Dpid 9 is not used

So I think this DDF can be updated, to solve the conflict, and add the missing dpid, like the luminosity as he said.
But as there is different dpid, better to create a new DDF

Edit:
The DDF _TZE200_kb5noeto_motion_sensor.json have more commun DPID I think.

@manup
Copy link
Member

manup commented Jan 29, 2025

Thanks for clarification,

But as there is different dpid, better to create a new DDF
Edit:
The DDF _TZE200_kb5noeto_motion_sensor.json have more commun DPID I think.

Due the different data points, my gut feeling is also to make a separate DDF since this isn't really a clone. Further "MmwRadar Human Presence Detector (TS0601)" in this DDF doesn't sound like Human breathing sensor nor motion detector.

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.

3 participants