aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTibor Frank <tifrank@cisco.com>2019-09-05 09:54:39 +0200
committerTibor Frank <tifrank@cisco.com>2019-09-05 08:01:21 +0000
commitc914a1c0ccaae1530a2990be761c0fb31b27ff4e (patch)
treed6d1c152c8572ae9d31d03e464823e99d7af1874
parentc06e2a99bac0d76743ecd65b1a8d9b2f32d0cc39 (diff)
Report: Fix listing order of new vs. see footnote
Change-Id: I01426c3e4d40af0902e7776088e0489e141a2362 Signed-off-by: Tibor Frank <tifrank@cisco.com> (cherry picked from commit 4751baad9a204c27b0566e4faf00bf0b30f7e170)
-rw-r--r--resources/tools/presentation/generator_tables.py64
1 files changed, 56 insertions, 8 deletions
diff --git a/resources/tools/presentation/generator_tables.py b/resources/tools/presentation/generator_tables.py
index 20779dde25..52f9573756 100644
--- a/resources/tools/presentation/generator_tables.py
+++ b/resources/tools/presentation/generator_tables.py
@@ -423,8 +423,32 @@ def table_performance_comparison(table, input_data):
if (len(item) == len(header)) and (item[-3] != "Not tested"):
tbl_lst.append(item)
- # Sort the table according to the relative change
- tbl_lst.sort(key=lambda rel: rel[-1], reverse=True)
+ # Sort the table:
+ # 1. New in CSIT-XXXX
+ # 2. See footnote
+ # 3. Delta
+ tbl_new = list()
+ tbl_see = list()
+ tbl_delta = list()
+ for item in tbl_lst:
+ if "New in CSIT" in item[-1]:
+ tbl_new.append(item)
+ elif "See footnote" in item[-1]:
+ tbl_see.append(item)
+ else:
+ tbl_delta.append(item)
+
+ # Sort the tables:
+ tbl_new.sort(key=lambda rel: rel[0], reverse=False)
+ tbl_see.sort(key=lambda rel: rel[0], reverse=False)
+ tbl_see.sort(key=lambda rel: rel[-1], reverse=False)
+ tbl_delta.sort(key=lambda rel: rel[-1], reverse=True)
+
+ # Put the tables together:
+ tbl_lst = list()
+ tbl_lst.extend(tbl_new)
+ tbl_lst.extend(tbl_see)
+ tbl_lst.extend(tbl_delta)
# Generate csv tables:
csv_file = "{0}.csv".format(table["output-file"])
@@ -441,10 +465,10 @@ def table_performance_comparison(table, input_data):
txt_file.writelines([
"\nFootnotes:\n",
"[1] CSIT-1908 changed test methodology of dot1q tests in "
- "2n-skx testbeds, dot1q encapsulation is now used on both "
+ "2-node testbeds, dot1q encapsulation is now used on both "
"links of SUT.\n",
" Previously dot1q was used only on a single link with the "
- "other link carrying untagged Ethernet frames. This change "
+ "other link carrying untagged Ethernet frames. This changes "
"results\n",
" in slightly lower throughput in CSIT-1908 for these "
"tests. See release notes."
@@ -693,8 +717,32 @@ def table_performance_comparison_nic(table, input_data):
if (len(item) == len(header)) and (item[-3] != "Not tested"):
tbl_lst.append(item)
- # Sort the table according to the relative change
- tbl_lst.sort(key=lambda rel: rel[-1], reverse=True)
+ # Sort the table:
+ # 1. New in CSIT-XXXX
+ # 2. See footnote
+ # 3. Delta
+ tbl_new = list()
+ tbl_see = list()
+ tbl_delta = list()
+ for item in tbl_lst:
+ if "New in CSIT" in item[-1]:
+ tbl_new.append(item)
+ elif "See footnote" in item[-1]:
+ tbl_see.append(item)
+ else:
+ tbl_delta.append(item)
+
+ # Sort the tables:
+ tbl_new.sort(key=lambda rel: rel[0], reverse=False)
+ tbl_see.sort(key=lambda rel: rel[0], reverse=False)
+ tbl_see.sort(key=lambda rel: rel[-1], reverse=False)
+ tbl_delta.sort(key=lambda rel: rel[-1], reverse=True)
+
+ # Put the tables together:
+ tbl_lst = list()
+ tbl_lst.extend(tbl_new)
+ tbl_lst.extend(tbl_see)
+ tbl_lst.extend(tbl_delta)
# Generate csv tables:
csv_file = "{0}.csv".format(table["output-file"])
@@ -711,10 +759,10 @@ def table_performance_comparison_nic(table, input_data):
txt_file.writelines([
"\nFootnotes:\n",
"[1] CSIT-1908 changed test methodology of dot1q tests in "
- "2n-skx testbeds, dot1q encapsulation is now used on both "
+ "2-node testbeds, dot1q encapsulation is now used on both "
"links of SUT.\n",
" Previously dot1q was used only on a single link with the "
- "other link carrying untagged Ethernet frames. This change "
+ "other link carrying untagged Ethernet frames. This changes "
"results\n",
" in slightly lower throughput in CSIT-1908 for these "
"tests. See release notes."