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

[1.3] The implementation of Electrical Power Measurement Cluster is required in Energy-Management app #32089

Closed
12 tasks
KishokG opened this issue Feb 13, 2024 · 1 comment · Fixed by #32095
Closed
12 tasks
Labels

Comments

@KishokG
Copy link

KishokG commented Feb 13, 2024

Feature Area

Other

Test Case

TC-EPM-1.1, 2.1

Reproduction steps

Below Attibutes has to be enebled in Energy-Management-app:

  • ReactiveCurrent
  • ApparentCurrent
  • ReactivePower
  • ApparentPower
  • RMSVoltage
  • RMSCurrent
  • RMSPower
  • Frequency
  • HarmonicCurrents
  • HarmonicPhases
  • PowerFactor
  • NeutralCurrent

For above mentioned Attributes,
Expected outcome: As per spec and test plan, the response should contain the values/struct entries/enum8 values.

Actual Outcome:

[1707809418.630857][9503:9505] CHIP:DMG: StatusIB =
[1707809418.630887][9503:9505] CHIP:DMG: {
[1707809418.630917][9503:9505] CHIP:DMG: status = 0x86 (UNSUPPORTED_ATTRIBUTE),
[1707809418.630945][9503:9505] CHIP:DMG: },

Note:

As per the above attributes implementation, the FeatureMap and AttributeList has to be updated for the same in Energy-Management-app.

Bug prevalence

Whenever I do this

GitHub hash of the SDK that was being used

4597a3b

Platform

raspi - Linux

Anything else?

No response

@jamesharrow
Copy link
Contributor

As discussed on slack, the Energy Management App deliberately doesn't turn on all attributes since most manufacturers won't implement all attributes.

However based on discussion, I will turn these optional features on here (rather than relying on all-clusters-app) which already has them on.

jamesharrow added a commit to jamesharrow/connectedhomeip that referenced this issue Feb 13, 2024
In all-clusters-app, Set the EPM PowerMode attribute to AC. In energy-management-app turned on all features and attributes for testing. Fixed issue in TC_EPM_2_1.py when valid attribute is returned. Improved type checking for HarmonicCurrents and HarmonicPhases
@mergify mergify bot closed this as completed in #32095 Feb 14, 2024
mergify bot pushed a commit that referenced this issue Feb 14, 2024
…energy-management-app (#32095)

* Fixes: #32089
In all-clusters-app, Set the EPM PowerMode attribute to AC. In energy-management-app turned on all features and attributes for testing. Fixed issue in TC_EPM_2_1.py when valid attribute is returned. Improved type checking for HarmonicCurrents and HarmonicPhases

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Fixes #31925 - QA team expect HarmonicCurrents and HarmonicPhases to have 1 entry in the list for testing

* Turned off 3 provisional features which fail the TC_DeviceConformance.py test

* Removed EVSE attributes which are provisional in 1.3 (SOC, PNC, V2X)
@github-project-automation github-project-automation bot moved this from Open Cert Blockers to Complete in [Certification] Blockers Feb 14, 2024
raul-marquez-csa pushed a commit to raul-marquez-csa/connectedhomeip that referenced this issue Feb 16, 2024
…energy-management-app (project-chip#32095)

* Fixes: project-chip#32089
In all-clusters-app, Set the EPM PowerMode attribute to AC. In energy-management-app turned on all features and attributes for testing. Fixed issue in TC_EPM_2_1.py when valid attribute is returned. Improved type checking for HarmonicCurrents and HarmonicPhases

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Fixes project-chip#31925 - QA team expect HarmonicCurrents and HarmonicPhases to have 1 entry in the list for testing

* Turned off 3 provisional features which fail the TC_DeviceConformance.py test

* Removed EVSE attributes which are provisional in 1.3 (SOC, PNC, V2X)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment