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

Give a option to make on_level ATTRIBUTE_FLAG_NONVOLATILE (CON-1431) #1169

Open
Haerteleric opened this issue Nov 25, 2024 · 3 comments
Open

Comments

@Haerteleric
Copy link

Is your feature request related to a problem? Please describe.
On Level Attribute (Level Control Cluster) is reset to init values after each power cycle.

Describe the solution you'd like
allow ATTRIBUTE_FLAG_NONVOLATILE for that attribute.

Describe alternatives you've considered
give us a esp_matter func to manually set the ATTRIBUTE_FLAG_NONVOLATILE of a Attribute

Additional context
...

@Haerteleric Haerteleric changed the title Give a option to make on_level Give a option to make on_level ATTRIBUTE_FLAG_NONVOLATILE Nov 25, 2024
@github-actions github-actions bot changed the title Give a option to make on_level ATTRIBUTE_FLAG_NONVOLATILE Give a option to make on_level ATTRIBUTE_FLAG_NONVOLATILE (CON-1431) Nov 25, 2024
@jonsmirl
Copy link
Contributor

Current Level is required by the spec to be non-volatile.
image

I just tried the Espessif Light Demo and it does not appear to be preserving CurrentLevel like it is supposed to do.
You don't need option, this appears to be broken in esp-matter.

@jadhavrohit924
Copy link
Contributor

@Haerteleric We cannot change the quality of an attribute; doing so would violate the Matter spec.

I just tried the Espessif Light Demo and it does not appear to be preserving CurrentLevel like it is supposed to do.

@jonsmirl We will look into this.

@jadhavrohit924
Copy link
Contributor

@jonsmirl Light Demo example has the StartUpCurrentLevel attribute on the LevelControl cluster that's why CurrentLevel's value is being overridden by StartUpCurrentLevel's value. If you don’t want to overwrite the CurrentLevel's value, you can define IGNORE_LEVEL_CONTROL_CLUSTER_START_UP_CURRENT_LEVEL in your application.

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

3 participants