Skip to content

refactor: rename RunMetrics to SingleRunMetric and MultipleRunMetric#81

Merged
iraedeus merged 1 commit into
devfrom
iraedeus/refactor-metrics
Apr 18, 2026
Merged

refactor: rename RunMetrics to SingleRunMetric and MultipleRunMetric#81
iraedeus merged 1 commit into
devfrom
iraedeus/refactor-metrics

Conversation

@iraedeus
Copy link
Copy Markdown
Contributor

This pull request refactors the metric base classes to clarify the distinction between metrics that operate on a single run and those that aggregate results over multiple runs. The main change is the renaming of RunMetric to SingleRunMetric for single-run metrics and to MultipleRunMetric for benchmark-level metrics, along with corresponding updates throughout the codebase.

Metric base class renaming and refactoring:

  • Renamed RunMetric to SingleRunMetric in pysatl_cpd/analysis/metrics/single_run_metric.py and updated all single-run metric classes (e.g., ClassificationMetric, ConfusionMatrix, DelayMetric, RunLengthMetric) to inherit from SingleRunMetric instead of RunMetric. [1] [2] [3] [4] [5] [6]

  • Renamed pysatl_cpd/benchmark/metrics/run_metric.py to multiple_run_metric.py, and renamed the base class from RunMetric to MultipleRunMetric for benchmark-level metrics. Updated all relevant imports and class usages. [1] [2] [3]

Codebase consistency and clarity:

  • Updated all imports and type aliases throughout the codebase and in test mocks to use the new SingleRunMetric and MultipleRunMetric names, improving clarity and reducing ambiguity between metric types. [1] [2]

@iraedeus iraedeus requested a review from Desiment April 13, 2026 06:52
@iraedeus iraedeus self-assigned this Apr 13, 2026
@iraedeus iraedeus merged commit 8add146 into dev Apr 18, 2026
16 checks passed
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