[1.3] Inconsistent Description for Flow Measurement, Missing Constraint for Max Value of 0x0002 MaxMeasuredValue

project-chip / connectedhomeip

Matter (formerly Project CHIP) creates more connections between more objects, simplifying development for manufacturers and increasing compatibility for consumers, guided by the Connectivity Standards Alliance.

https://buildwithmatter.com

Apache License 2.0

7.45k stars 1.99k forks source link

[1.3] Inconsistent Description for Flow Measurement, Missing Constraint for Max Value of 0x0002 MaxMeasuredValue #34218

Closed agatah2333 closed 3 months ago

agatah2333 commented 3 months ago

Reproduction steps

The description for Flow Measurement is inconsistent, and the maximum value for 0x0002 MaxMeasuredValue is missing a constraint.

https://github.com/project-chip/connectedhomeip/blob/v1.3.0.0/src/app/zap-templates/zcl/data-model/chip/flow-measurement-cluster.xml

Bug prevalence

each time

GitHub hash of the SDK that was being used

5bb5c9e23d532cea40476fc0bd1d3008522792ba

Platform

other

Platform Version(s)

1.3

Type

Spec Compliance Issue, Core SDK Interopability Issue

Anything else?

No response

bzbarsky-apple commented 3 months ago

It's a readonly attribute, so the min/max in the XML are not used for anything, even if they were provided.

agatah2333 commented 3 months ago

Yes, it is a read-only attribute, but since the XML will affect ZAP, why not add this limitation to avoid vendors setting this value incorrectly?

bzbarsky-apple commented 3 months ago

If the ZAP UI uses the value, we could add this, sure. Does it?