refactor: rename RunMetrics to SingleRunMetric and MultipleRunMetric#81
Merged
Conversation
Desiment
approved these changes
Apr 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
RunMetrictoSingleRunMetricfor single-run metrics and toMultipleRunMetricfor benchmark-level metrics, along with corresponding updates throughout the codebase.Metric base class renaming and refactoring:
Renamed
RunMetrictoSingleRunMetricinpysatl_cpd/analysis/metrics/single_run_metric.pyand updated all single-run metric classes (e.g.,ClassificationMetric,ConfusionMatrix,DelayMetric,RunLengthMetric) to inherit fromSingleRunMetricinstead ofRunMetric. [1] [2] [3] [4] [5] [6]Renamed
pysatl_cpd/benchmark/metrics/run_metric.pytomultiple_run_metric.py, and renamed the base class fromRunMetrictoMultipleRunMetricfor benchmark-level metrics. Updated all relevant imports and class usages. [1] [2] [3]Codebase consistency and clarity:
SingleRunMetricandMultipleRunMetricnames, improving clarity and reducing ambiguity between metric types. [1] [2]