Skip to content

Issue 1972- added exclude from analysis option to groups with no meter selected#2087

Merged
rmroot merged 2 commits intodevelopfrom
issue-1972
Oct 1, 2025
Merged

Issue 1972- added exclude from analysis option to groups with no meter selected#2087
rmroot merged 2 commits intodevelopfrom
issue-1972

Conversation

@jamlokim
Copy link
Copy Markdown
Contributor

connects #1972

This pull request introduces a new feature that allows users to exclude meter groups from analysis in the group analysis options. It includes UI updates, model changes, and logic updates to support this functionality, as well as minor improvements to styling and error handling.

Feature: Exclude group from analysis

  • Added a checkbox in group-analysis-options.component.html to allow users to exclude a group from analysis, with corresponding styles in group-analysis-options.component.css.
  • Updated the AnalysisGroup interface and group initialization to include the new isGroupExcluded property.
  • Added logic in GroupAnalysisOptionsComponent to set the analysis type to 'skip' when a group is excluded, and to 'regression' when included.

Logic update: Error handling for excluded groups

  • Modified the logic in AnalysisFooterComponent to allow groups with missing meters to be excluded from error checks if they are marked as excluded.

@jamlokim jamlokim requested a review from rmroot September 23, 2025 16:41
@jamlokim jamlokim requested a review from rmroot September 24, 2025 18:59
@rmroot rmroot merged commit 367cdc4 into develop Oct 1, 2025
3 checks passed
@jamlokim jamlokim deleted the issue-1972 branch October 14, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants