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

support r/w floating(single) value type (CON-1077) #863

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

YOGYUI
Copy link
Contributor

@YOGYUI YOGYUI commented Mar 8, 2024

@dhrishi I recently lost my remote and local branch while squashing PR commits because of my mistake, so reopened PR with new branch.
I deeply sorry for inconvenience.

[Related issue]

[Major changes]

  • support read/write floating (single) attribute value type
  • make ConcentrationMeasurement clusters follow Matter specifications (2.10.6)
    • change attribute value type (MeasuredValue, MinMeasuredValue, MaxMeasuredValue, PeakMeasuredValue, AverageMeasuredValue, Uncertainty)
    • remove creating Uncertainty attribute phrase (optional attribute)
    • add creating MeasurementMedium attribute phrase (mandatory attribute)
    • add AverageMeasurement feature config struct

[Minor changes]

  • modify indentation (tab to 4 spaces)

@github-actions github-actions bot changed the title support r/w floating(single) value type support r/w floating(single) value type (CON-1077) Mar 8, 2024
@dhrishi
Copy link
Collaborator

dhrishi commented Mar 9, 2024

Sure. No worries. Thanks @YOGYUI

- concentration measurement clusters: change attr value type (float)
- remove 'add uncentainty attribute'
- add 'measurement medium' attribute
@YOGYUI YOGYUI force-pushed the feature/support-float-attr-val branch from add04e2 to 810a9ca Compare March 12, 2024 12:45
@YOGYUI YOGYUI requested a review from dhrishi March 13, 2024 00:13
Copy link
Contributor

@wqx6 wqx6 left a comment

Choose a reason for hiding this comment

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

LGTM

@lboue
Copy link
Contributor

lboue commented Mar 26, 2024

Can you merge this PR is it's OK?

@YOGYUI YOGYUI marked this pull request as draft March 28, 2024 03:43
@YOGYUI YOGYUI marked this pull request as ready for review March 28, 2024 03:43
@YOGYUI
Copy link
Contributor Author

YOGYUI commented Mar 28, 2024

@dhrishi Any progress?

@dhrishi
Copy link
Collaborator

dhrishi commented Mar 28, 2024

@YOGYUI @lboue Sorry for the delay. We will soon have this in the internal queue and then sync to Github.

@espressif-bot espressif-bot merged commit 434aa0b into espressif:main Apr 2, 2024
1 check passed
@YOGYUI YOGYUI deleted the feature/support-float-attr-val branch April 2, 2024 05:54
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.

None yet

6 participants