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.

Sign up for GitHub

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

Jump to bottom

[TC-FLDCONC-1.1] ClusterRevision value has to be updated in Formaldehyde Concentration Measurement Cluster as per spec. #27371

Closed
Survensa opened this issue Jun 21, 2023 · 0 comments · Fixed by #27396
Closed

[TC-FLDCONC-1.1] ClusterRevision value has to be updated in Formaldehyde Concentration Measurement Cluster as per spec. #27371

Survensa opened this issue Jun 21, 2023 · 0 comments · Fixed by #27396

Comments

@Survensa
Copy link

Problem:
As per Spec, ClusterRevision attribute value should be 3 but currently getting ClusterRevision value as 1 in SDK .

Expected behavior: ClusterRevision value as 3.

Actual behavior : ClusterRevision value as 1.

Steps to reproduce: 2

Command: ./chip-tool formaldehydeconcentrationmeasurement read cluster-revision 1 1

System configuration:
Chip-tool - RPI-4, 8GB RAM
DUT - RPI, Version- RPI-4, 8GB RAM
Platform : Onnetwork
Commit-ID: f5c4621
Dut application: all-clusters-app

Spec Reference: https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/ConcentrationMeasurement.adoc#ref_ConcentrationMeasurementRevisionHistory

Log: TC-FLDCONC-1.1.txt
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Jun 21, 2023
@bzbarsky-apple
Update concentration measurement cluster revisions to 3.
46fe824 
* Fixes project-chip#27364
* Fixes project-chip#27367
* Fixes project-chip#27368
* Fixes project-chip#27369
* Fixes project-chip#27370
* Fixes project-chip#27371
* Fixes project-chip#27372
* Fixes project-chip#27373
* Fixes project-chip#27375
* Fixes project-chip#27376
@bzbarsky-apple bzbarsky-apple mentioned this issue Jun 21, 2023
Merged
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Jun 21, 2023
@bzbarsky-apple
Update concentration measurement cluster revisions to 3.
81b447f 
* Fixes project-chip#27364
* Fixes project-chip#27367
* Fixes project-chip#27368
* Fixes project-chip#27369
* Fixes project-chip#27370
* Fixes project-chip#27371
* Fixes project-chip#27372
* Fixes project-chip#27373
* Fixes project-chip#27375
* Fixes project-chip#27376
@mergify mergify bot closed this as completed in #27396 Jun 22, 2023
@mergify mergify bot closed this as completed in e149bb7 Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update concentration measurement cluster revisions to 3. bzbarsky-apple/connectedhomeip
1 participant
@Survensa