Remove macro benchmark results if nothing really changed.
Reviewers: mislav.bradac, mferencevic Reviewed By: mferencevic Subscribers: pullbot Differential Revision: https://phabricator.memgraph.io/D913
This commit is contained in:
parent
03c50bada9
commit
0df240f9d5
@ -45,6 +45,7 @@ def compare_values(data_cur, data_prev):
|
||||
for i in range(len(data_cur)):
|
||||
ret.append([])
|
||||
row_cur = data_cur[i]
|
||||
performance_change = False
|
||||
for j in range(len(row_cur)):
|
||||
item_cur = row_cur[j]
|
||||
if type(item_cur) == str:
|
||||
@ -63,8 +64,10 @@ def compare_values(data_cur, data_prev):
|
||||
else:
|
||||
diff = 0.0
|
||||
if diff < -0.05 and item_cur > 0.0005:
|
||||
performance_change = True
|
||||
sign = " {icon arrow-down color=green}"
|
||||
elif diff > 0.05 and item_cur > 0.0005:
|
||||
performance_change = True
|
||||
sign = " {icon arrow-up color=red}"
|
||||
else:
|
||||
sign = ""
|
||||
@ -73,22 +76,27 @@ def compare_values(data_cur, data_prev):
|
||||
else:
|
||||
fmt += "//(new)// {{icon plus color=blue}}"
|
||||
item = fmt.format(item_cur * scale)
|
||||
ret[i].append(item)
|
||||
performance_change = True
|
||||
ret[-1].append(item)
|
||||
if performance_change == False and i > 0: ret.pop()
|
||||
return ret
|
||||
|
||||
def generate_remarkup(data):
|
||||
ret = "==== Macro benchmark summary: ====\n\n"
|
||||
ret += "<table>\n"
|
||||
for row in data:
|
||||
ret += " <tr>\n"
|
||||
for item in row:
|
||||
if row == data[0]:
|
||||
fmt = " <th>{}</th>\n"
|
||||
else:
|
||||
fmt = " <td>{}</td>\n"
|
||||
ret += fmt.format(item)
|
||||
ret += " </tr>\n"
|
||||
ret += "</table>\n"
|
||||
if len(data) > 1:
|
||||
ret += "<table>\n"
|
||||
for row in data:
|
||||
ret += " <tr>\n"
|
||||
for item in row:
|
||||
if row == data[0]:
|
||||
fmt = " <th>{}</th>\n"
|
||||
else:
|
||||
fmt = " <td>{}</td>\n"
|
||||
ret += fmt.format(item)
|
||||
ret += " </tr>\n"
|
||||
ret += "</table>\n"
|
||||
else:
|
||||
ret += "No performance change detected.\n"
|
||||
return ret
|
||||
|
||||
if len(sys.argv) > 4 or len(sys.argv) < 3:
|
||||
|
Loading…
Reference in New Issue
Block a user