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

Xiaomi aqara temp sensor pressure #12

Closed
pxFIN opened this issue Aug 7, 2022 · 1 comment
Closed

Xiaomi aqara temp sensor pressure #12

pxFIN opened this issue Aug 7, 2022 · 1 comment

Comments

@pxFIN
Copy link

pxFIN commented Aug 7, 2022

Taken from Smartthings CLI logcat:

(Aqara Weather)> emitting event {"attribute_id":"atmosphericPressure","capability_id":"atmosphericPressureMeasurement","component_id":"main","state":{"unit":"kPa","value":104}}
(Aqara Weather)> emitting event {"attribute_id":"atmosPressure","capability_id":"legendabsolute60149.atmosPressure","component_id":"main","state":{"value":1041}}

So there seems to be two events and in Edge driver:

device:emit_event(capabilities.atmosphericPressureMeasurement.atmosphericPressure({value = utils.round(mBar/10), unit = "kPa"}))
device:emit_event(atmos_Pressure.atmosPressure(mBar))

Thus i reckon the last emit in driver is always relevant ?

Future possibility to choose mbar or kPa ? (for example in device settings)

@veonua
Copy link
Owner

veonua commented Aug 7, 2022

feel free to create a PR,

personally, I don't see any value in the pressure measurement personally, especially in different units.

@veonua veonua closed this as completed Aug 22, 2022
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

No branches or pull requests

2 participants