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

[TC-FLW-2.1] MaxMeasuredValue attribute value mismatches from specification #27543

Closed
Ashwinigrl opened this issue Jun 29, 2023 · 0 comments · Fixed by #29355
Closed

[TC-FLW-2.1] MaxMeasuredValue attribute value mismatches from specification #27543

Ashwinigrl opened this issue Jun 29, 2023 · 0 comments · Fixed by #29355
Labels
app-clusters Application cluster work bug Something isn't working cert blocker examples needs owner needs triage spec Mismatch between spec and implementation
Milestone

Comments

@Ashwinigrl
Copy link

Ashwinigrl commented Jun 29, 2023

Reproduction steps

  1. Execute the below mentioned command to put DUT into a commissionable state
    ./chip-all-clusters-app

  2. Once DUT reach the commissionable state sent below mentioned command on TH.
    ./chip-tool pairing onnetwork 1 20202021
    Verify the commissioning completed with success on TH(chip-tool) from DUT

  3. TH reads from the DUT the MinMeasuredValue attribute : ./chip-tool flowmeasurement read min-measured-value 1 1

  4. TH reads from the DUT the MaxMeasuredValue attribute : ./chip-tool flowmeasurement read max-measured-value 1 1

Bug prevalence

Everytime

GitHub hash of the SDK that was being used

Master - 7806dd1

Platform

raspi

Platform Version(s)

RPI4 8GB RAM

Summary

As per specifications, MaxMeasuredValue attribute represents the maximum value that can be measured for the MeasuredValue attribute and value should be in the range of MinMeasuredValue+1 to 65534.
In Master commit Minmeasured value is 0. Maxmeasured value should be in the range of 1 to 65534(MinMeasuredValue + 1 to 65534). But the currently Maxmeasured value is 0.

PFA Spec screenshot
image

./chip-tool flowmeasurement read min-measured-value 1 1

[1688026768.988344][1909:1911] CHIP:DMG: 	InteractionModelRevision = 1
[1688026768.988370][1909:1911] CHIP:DMG: }
[1688026768.988574][1909:1911] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0404 Attribute 0x0000_0001 DataVersion: 1380109096
[1688026768.991124][1909:1911] CHIP:TOO:   MinMeasuredValue: 0

./chip-tool flowmeasurement read max-measured-value 1 1

[1688026832.310929][1913:1915] CHIP:DMG: }
[1688026832.311093][1913:1915] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0404 Attribute 0x0000_0002 DataVersion: 1380109096
[1688026832.311166][1913:1915] CHIP:TOO:   MaxMeasuredValue: 0

PFA reference log :
TC-FLW-2.1.txt

@Ashwinigrl Ashwinigrl added bug Something isn't working needs triage labels Jun 29, 2023
@github-project-automation github-project-automation bot moved this to Open Cert Blockers in [Certification] Blockers Jun 30, 2023
@Ashwinigrl Ashwinigrl added examples app-clusters Application cluster work and removed examples labels Jul 10, 2023
@woody-apple woody-apple added the spec Mismatch between spec and implementation label Aug 10, 2023
@cjandhyala cjandhyala added this to the 1.2 milestone Aug 10, 2023
@woody-apple woody-apple modified the milestones: 1.2, Future Sep 19, 2023
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Sep 20, 2023
Per spec, MaxMeasuredValue > MinMeasuredValue, but we were ending up with 0 for both.

Fixes project-chip#27543
@mergify mergify bot closed this as completed in #29355 Sep 20, 2023
mergify bot pushed a commit that referenced this issue Sep 20, 2023
…29355)

Per spec, MaxMeasuredValue > MinMeasuredValue, but we were ending up with 0 for both.

Fixes #27543
@github-project-automation github-project-automation bot moved this from Open Cert Blockers to Complete in [Certification] Blockers Sep 20, 2023
HunsupJung pushed a commit to HunsupJung/connectedhomeip that referenced this issue Oct 23, 2023
…roject-chip#29355)

Per spec, MaxMeasuredValue > MinMeasuredValue, but we were ending up with 0 for both.

Fixes project-chip#27543
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app-clusters Application cluster work bug Something isn't working cert blocker examples needs owner needs triage spec Mismatch between spec and implementation
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants