Skip to content

Conversation

@tarikweiss
Copy link
Contributor

No description provided.

Copy link
Collaborator

@Apollon77 Apollon77 left a comment

Choose a reason for hiding this comment

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

There is no need to stick to the Matter Lighting "min/max" requirements for a speaker. I would simply sync the range to the iobroker min/max which should by 0..100 and simply set this as min/maxLevel on the Matter level control cluster and then the values can be used directly without any calculation

@tarikweiss
Copy link
Contributor Author

In coordination with @Apollon77 we decided to keep it the way it currently is. This is, because we seemingly found issues with setting the minLevel/maxLevel for the Level Control Cluster and a bug in the SmartThings Driver for Speakers by Samsung. https://github.com/SmartThingsCommunity/SmartThingsEdgeDrivers/blob/aa630fa655fb4f7827570d8bb414662c22dbd987/drivers/SmartThings/matter-media/src/init.lua#L166

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

Successfully merging this pull request may close these issues.

2 participants