project-chip / project-chip/matter-test-scripts

[SMOKECO] - Follow up: Update Highest priority alarm PIXIT to dynamic values in the code.

Open
#783 0 comments 0 reactions 1 assignee View on GitHub

@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

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.