-
Notifications
You must be signed in to change notification settings - Fork 517
WWSTCERT-8745 Add support to frient vibration sensor wiszb 137 #2454
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
base: main
Are you sure you want to change the base?
WWSTCERT-8745 Add support to frient vibration sensor wiszb 137 #2454
Conversation
…ibration-sensor-WISZB-137
…tps://github.com/marcintyminski/SmartThingsEdgeDrivers into Add-support-to-frient-vibration-sensor-WISZB-137
|
Duplicate profile check: Passed - no duplicate profiles detected. |
|
Invitation URL: |
|
Minimum allowed coverage is Generated by 🐒 cobertura-action against ac43af1 |
wkhenon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| if attribute_id == 0x0000 then | ||
| measured_x = axis_value | ||
| device:set_field("measured_x", measured_x) | ||
| log.trace("Updated X value: " .. axis_value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you're satisfied with the behavior, please remove the logging
| ID = 0xFC04, | ||
| ManufacturerSpecificCode = 0x1015, | ||
| attributes = { | ||
| MeasuredValueX = { ID = 0x0000, data_type = data_types.name_to_id_map["Int16"] }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can just use data_types.Int16
| log.trace "Initializing sensor" | ||
| battery_defaults.build_linear_voltage_init(2.3, 3.0)(driver, device) | ||
| --Add the manufacturer-specific attributes to generate their configure reporting and bind requests | ||
| for _, config in pairs(get_cluster_configurations()) do | ||
| device:add_configured_attribute(config) | ||
| end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: we use 2 spaces
Check all that apply
Type of Change
Checklist
Description of Change
Summary of Completed Tests