project-chip / project-chip/matter-test-scripts
[SMOKECO] - Follow up: Update Highest priority alarm PIXIT to dynamic values in the code.
@jtrejoespinoza-grid is already working on this.
Since May 6, 2026.
- Dominant language
- Python
- Stars
- 19
- Forks
- 7
- Avg merge
- 3d 17h
- Merged PRs (30d)
- 3
Description
This is a follow up for: https://github.com/project-chip/connectedhomeip/pull/43712
Summary Title:
Update PIXIT values from the HIEST_PRI_ALARM.
Description:
Currently the PIXIT values for HIEST_PRI_ALARM look as they are custom and should be provided as part of the certification but its not 100% clear and how is implemented from the test plan it encourages the testers to play with the values until get a valid set of values for the HIEST_PRI_ALARM which can be harmful.
Alternative from this scenario we think the following can be done to solve this issue:
In step 22 (1st HIEST_PRI_ALARM):
ensure ExpressedState is one of [BatterAlert, InterconnectSmokeAlarm (if available), InterconnectCOAlarm (if available), COState, SmokeState]
save value as last_highest_alarm
In step 25:
if last_highest_alarm was not COState, ensure ExpressedState is last_highest_alarm
else ensure ExpressedState is one of [BatterAlert, InterconnectSmokeAlarm (if available), InterconnectCOAlarm (if available), SmokeState]
save as last_highest_alarm
In step 28:
if last_highest_alarm was not InteronnectCOAlarm, ensure ExpressedState is last_highest_alarm
else ensure ExpressedState is one of [BatterAlert, InterconnectSmokeAlarm (if available), SmokeState]
save as last_highest_alarm
Reference: https://github.com/project-chip/connectedhomeip/pull/43712/changes#r3197619179
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.