mfucci / node-matter

Matter protocol client / server for node.js
Apache License 2.0
119 stars 10 forks source link

Initial commit of Water Content Measurement Clusters. #243

Closed JimBuzbee closed 1 year ago

JimBuzbee commented 1 year ago

Contains three clusters: RelativeHumidityCluster, LeafWetnessMeasurementCluster and SoilMoistureMeasurementCluster.

Tested with live weather data in Google Home

This is the last of the "measurement and sensing" clusters from the spec.

Apollon77 commented 1 year ago

This is the last of the "measurement and sensing" clusters from the spec.

Which come next ? ;-)

JimBuzbee commented 1 year ago

Updates in and tested in Google Home.

Next? I've been playing with the Level Control cluster so I can dim my lights. It's a bit more complex (at least for me!) So far I have both Google Home and Alexa showing the light with an intensity adjustment slider, but I haven't seen my commands in the LevelControlClusterHandler ever get called. Currently trying to debug a "InteractionMessenger Error: Unexpected type 20, was expecting 4" message so I must have something mis-defined. Will keep poking at it.

Apollon77 commented 1 year ago

(for level Controller ... keep it simpel for now .. the whole "inter cluster logic" should be left out for the beginning. We need to add support for that ... if you like postthe code in a Draft PR and I have alook

JimBuzbee commented 1 year ago

Regarding the Level Controller - Fixed the "Unexpected type 20," error (had a command ID wrong). But I still don't get execution of my command handlers in the Cluster Handler. Thanks for the offer to take a look - I'll try and make it presentable tomorrow for a draft PR.

@Apollon77 - Draft PR in.

JimBuzbee commented 1 year ago

Updates based on comments