Support changing the LHS/RHS terms in performance comparisons

Author: vkarak

reframe/frontend/cli.py

@@ -702,6 +702,14 @@ def main():
         help='The delimiter to use when using `--table-format=csv`',
         envvar='RFM_TABLE_FORMAT_DELIM', configvar='general/table_format_delim'
     )
+    misc_options.add_argument(
+        '--term-lhs', action='store',
+        help='LHS term in performance comparisons'
+    )
+    misc_options.add_argument(
+        '--term-rhs', action='store',
+        help='RHS term in performance comparisons'
+    )
     misc_options.add_argument(
         '-v', '--verbose', action='count',
         help='Increase verbosity level of output',
@@ -1168,8 +1176,10 @@ def restrict_logging():
                     sys.exit(1)
 
             printer.table(
-                reporting.performance_compare(options.performance_compare,
-                                              None, namepatt, *filt)
+                reporting.performance_compare(
+                    options.performance_compare, None, namepatt, *filt,
+                    options.term_lhs, options.term_rhs
+                )
             )
             sys.exit(0)
 
@@ -1769,7 +1779,9 @@ def module_unuse(*paths):
                 try:
                     if rt.get_option('storage/0/enable'):
                         data = reporting.performance_compare(
-                            rt.get_option('general/0/perf_report_spec'), report
+                            rt.get_option('general/0/perf_report_spec'), report,
+                            term_lhs=options.term_lhs,
+                            term_rhs=options.term_rhs
                         )
                     else:
                         data = report.report_data()

reframe/frontend/reporting/__init__.py

@@ -690,10 +690,11 @@ def compare_testcase_data(base_testcases, target_testcases, query):
 
 @time_function
 def performance_compare(cmp, report=None, namepatt=None,
-                        filterA=None, filterB=None):
+                        filterA=None, filterB=None,
+                        term_lhs=None, term_rhs=None):
     with reraise_as(ReframeError, (ValueError,),
                     'could not parse comparison spec'):
-        query = parse_cmp_spec(cmp)
+        query = parse_cmp_spec(cmp, term_lhs, term_rhs)
 
     backend = StorageBackend.default()
     if query.lhs is None:

reframe/frontend/reporting/utility.py

@@ -284,13 +284,16 @@ def __init__(self,
                  rhs: QuerySelectorTestcase,
                  aggregation: Aggregation,
                  groups: List[str],
-                 columns: List[str]):
+                 columns: List[str],
+                 term_lhs: str, term_rhs: str):
         self.__lhs: QuerySelectorTestcase = lhs
         self.__rhs: QuerySelectorTestcase = rhs
         self.__aggregation: Aggregation = aggregation
         self.__tc_group_by: List[str] = groups
         self.__tc_attrs: List[str] = []
         self.__col_variants: Dict[str, List[str]] = {}
+        self.__lhs_term: str = term_lhs or 'lhs'
+        self.__rhs_term: str = term_rhs or 'rhs'
 
         if self.is_compare() and 'pval' not in columns:
             # Always add `pval` if the query is a performance comparison
@@ -339,7 +342,7 @@ def is_compare(self):
     @property
     def lhs_column_suffix(self):
         '''The suffix of the lhs column in a comparison'''
-        return ' (lhs)'
+        return f' ({self.__lhs_term})'
 
     @property
     def lhs_select_suffix(self):
@@ -349,7 +352,7 @@ def lhs_select_suffix(self):
     @property
     def rhs_column_suffix(self):
         '''The suffix of the rhs column in a comparison'''
-        return ' (rhs)'
+        return f' ({self.__rhs_term})'
 
     @property
     def rhs_select_suffix(self):
@@ -405,7 +408,7 @@ def group_by(self) -> List[str]:
 DEFAULT_GROUP_BY = ['name', 'sysenv', 'pvar', 'punit']
 
 
-def parse_cmp_spec(spec):
+def parse_cmp_spec(spec, term_lhs=None, term_rhs=None):
     parts = spec.split('/')
     if len(parts) == 3:
         base_spec, target_spec, aggr, cols = None, *parts
@@ -420,4 +423,5 @@ def parse_cmp_spec(spec):
 
     # Update base columns for listing
     columns = _parse_columns(cols, group_cols + aggr.attributes())
-    return _QueryMatch(base, target, aggr, group_cols, columns)
+    return _QueryMatch(base, target, aggr, group_cols, columns,
+                       term_lhs, term_rhs)