ISISNeutronMuon / ISISNeutronMuon/p4pillon

alarm logic for NTEnums

Open
#12 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
2
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Currently there isn't any logic in `p4p_ext` for setting Enums to alarm on a specific index or choice. This is a common use case as Enums are often used as flags to indicate issues and therefore would want to be associated with an alarm state.

### Acceptance Criteria:
- [ ] new rule exists to handle enum alarm logic
- [ ] new rule is included in Enum related handlers if relevant
- [ ] methods are exposed to allow alarm configuration on Enum recipes
- [ ] config_parser knows how to deal with alarmed enums
- [ ] unit tests completed for new code
- [ ] integration tests for alarmed Enums

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing the existing alarm rules and Enum-related handlers in p4p_ext, then inspect config_parser to see how recipes are represented. Define the required Enum alarm configuration entry points and add unit tests plus integration tests for alarmed Enums. Done means the new rule is included where relevant and all listed acceptance criteria are covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.