Skip to content

DDF for Aqara Presence Sensor FP1E#7900

Merged
manup merged 6 commits intodresden-elektronik:masterfrom
ebaauw:fp1e
Sep 5, 2024
Merged

DDF for Aqara Presence Sensor FP1E#7900
manup merged 6 commits intodresden-elektronik:masterfrom
ebaauw:fp1e

Conversation

@ebaauw
Copy link
Collaborator

@ebaauw ebaauw commented Aug 23, 2024

See #7885.

Changes:

  • Add new attributes to general.xml;
  • Add new DDF;
  • Re-use config/resetpresence to set AI Learning (see issue);
  • Create new item config/detectionrange. Unfortunately making that updatable from the API requires some trivial C++ changes;
  • Create new read-only item state/distance.

ebaauw added 4 commits August 23, 2024 20:41
Add attributes for Aqara FP1E, see dresden-elektronik#7885
Reset presence is write-only.  Reset it to false on next presence report.
Apparently we still need C++ support to make items updatable.  See dresden-elektronik#7885.
@ebaauw ebaauw requested a review from manup August 23, 2024 21:08
@github-actions
Copy link
Contributor

github-actions bot commented Aug 23, 2024

Hey @ebaauw, thanks for your pull request!

Tip

Modified bundles can be downloaded here.
Relative expire date

DDB changes

Modified

  • xiaomi/lumi_sensor_occupy.agl1.json : Presence Sensor FP1E (PS-SE03) ✔️

Validation

Tip

Everything is fine !

🕗 Updated for commit 1c36846

@ebaauw ebaauw requested a review from SwoopX August 23, 2024 21:11
@ebaauw ebaauw linked an issue Aug 23, 2024 that may be closed by this pull request
1 task
Copy link
Collaborator

@SwoopX SwoopX left a comment

Choose a reason for hiding this comment

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

Guess this one's ok so far. Just added some questions on the DDF.

Comment on lines +39 to +67
"name": "attr/swversion",
"parse": {
"fn": "zcl:attr",
"ep": "0x01",
"cl": "0x0000",
"at": "0x0001",
"eval": "Item.val = '0.0.0_' + ('0000' + Attr.val.toString()).slice(-4)"
},
"read": {
"fn": "zcl:attr",
"ep": "0x01",
"cl": "0x0000",
"at": "0x0001"
},
"refresh.interval": 86400
},
{
"name": "attr/swversion_bis",
"awake": true,
"parse": {
"fn": "xiaomi:special",
"at": "0x00F7",
"idx": "0x08",
"eval": "Item.val = '0.0.0_' + ('0000' + (Attr.val & 0xFF).toString()).slice(-4)"
},
"read": {
"fn": "none"
}
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd probably go with the special reporting here only. Saves some lines and requests towards the device :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not too sure if the sensor actually sends regular reports for the special attribute. I've only ever captures a single report, shortly after resetting the sensor.

"eval": "Item.val = Attr.val"
},
"read": {
"fn": "none"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hm, don't we want to know the current state or isn't it readable? 🤔

Copy link
Collaborator Author

@ebaauw ebaauw Sep 2, 2024

Choose a reason for hiding this comment

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

It is readable.

"eval": "Item.val = Attr.val"
},
"read": {
"fn": "none"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also here, do we want to read?

Copy link
Collaborator Author

@ebaauw ebaauw Sep 2, 2024

Choose a reason for hiding this comment

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

The presence attribute is report-only, the distance can be read.

@ebaauw ebaauw changed the title DDF for Aqara Presence Sensor FP1M DDF for Aqara Presence Sensor FP1E Sep 3, 2024
@manup manup merged commit 8756d93 into dresden-elektronik:master Sep 5, 2024
@manup manup added this to the v2.29.0-beta milestone Sep 5, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2024

This pull request is now merged. The new DDB files have been uploaded to the store.

DDB Files

Modified

  • xiaomi/lumi_sensor_occupy.agl1.json : Presence Sensor FP1E (PS-SE03) : with hash (65bc0127ea)

🕓 Updated for commit 8756d93

@ebaauw ebaauw deleted the fp1e branch September 5, 2024 15:51
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.

Xiaomi Aqara FP1E

3 participants