summaryrefslogtreecommitdiffstats
path: root/scripts/lib/build_perf
diff options
context:
space:
mode:
authorMarkus Lehtonen <markus.lehtonen@linux.intel.com>2017-09-15 15:54:55 +0300
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-09-18 11:07:30 +0100
commit873707489fafaa5e1bd43b03b068df74d5956b00 (patch)
tree83845f6b71a8e7d8de569045769c6d232b182197 /scripts/lib/build_perf
parenteaeb6981291226f3f883f01e4827e711f93ec44d (diff)
downloadpoky-873707489fafaa5e1bd43b03b068df74d5956b00.tar.gz
scripts/oe-build-perf-report: tidy up html syntax
Fix some problems in the html syntax of the generated report: - prevent empty rows in the summary table - add one missing column in the results table (From OE-Core rev: 10883bb49ad2f5309883fd352cf320b2e1648615) Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/build_perf')
-rw-r--r--scripts/lib/build_perf/html/report.html36
1 files changed, 20 insertions, 16 deletions
diff --git a/scripts/lib/build_perf/html/report.html b/scripts/lib/build_perf/html/report.html
index 165cbb811c..f983a4a09c 100644
--- a/scripts/lib/build_perf/html/report.html
+++ b/scripts/lib/build_perf/html/report.html
@@ -118,29 +118,32 @@ h3 {
118 {% else %} 118 {% else %}
119 {% set row_style = 'style="background-color: #ffffff"' %} 119 {% set row_style = 'style="background-color: #ffffff"' %}
120 {% endif %} 120 {% endif %}
121 <tr {{ row_style }}><td>{{ test.name }}: {{ test.description }}</td>
122 {% if test.status == 'SUCCESS' %} 121 {% if test.status == 'SUCCESS' %}
123 {% for measurement in test.measurements %} 122 {% for measurement in test.measurements %}
124 {# add empty cell in place of the test name#} 123 <tr {{ row_style }}>
125 {% if loop.index > 1 %}<td></td>{% endif %} 124 {% if loop.index == 1 %}
126 {% if measurement.absdiff > 0 %} 125 <td>{{ test.name }}: {{ test.description }}</td>
127 {% set result_style = "color: red" %} 126 {% else %}
128 {% elif measurement.absdiff == measurement.absdiff %} 127 {# add empty cell in place of the test name#}
129 {% set result_style = "color: green" %} 128 <td></td>
130 {% else %} 129 {% endif %}
131 {% set result_style = "color: orange" %} 130 {% if measurement.absdiff > 0 %}
132 {%endif %} 131 {% set result_style = "color: red" %}
133 <td>{{ measurement.description }}</td> 132 {% elif measurement.absdiff == measurement.absdiff %}
134 <td style="font-weight: bold">{{ measurement.value.mean }}</td> 133 {% set result_style = "color: green" %}
135 <td style="{{ result_style }}">{{ measurement.absdiff_str }}</td> 134 {% else %}
136 <td style="{{ result_style }}">{{ measurement.reldiff }}</td> 135 {% set result_style = "color: orange" %}
137 </tr><tr {{ row_style }}> 136 {%endif %}
137 <td>{{ measurement.description }}</td>
138 <td style="font-weight: bold">{{ measurement.value.mean }}</td>
139 <td style="{{ result_style }}">{{ measurement.absdiff_str }}</td>
140 <td style="{{ result_style }}">{{ measurement.reldiff }}</td>
141 </tr>
138 {% endfor %} 142 {% endfor %}
139 {% else %} 143 {% else %}
140 <td style="font-weight: bold; color: red;">{{test.status }}</td> 144 <td style="font-weight: bold; color: red;">{{test.status }}</td>
141 <td></td> <td></td> <td></td> <td></td> 145 <td></td> <td></td> <td></td> <td></td>
142 {% endif %} 146 {% endif %}
143 </tr>
144 {% endfor %} 147 {% endfor %}
145 </table> 148 </table>
146 149
@@ -186,6 +189,7 @@ h3 {
186 <th>Stdev</th><td>{{ measurement.value.stdev }}</td> 189 <th>Stdev</th><td>{{ measurement.value.stdev }}</td>
187 </tr><tr> 190 </tr><tr>
188 <th><div id="{{ test.name }}_{{ measurement.name }}_chart_png"></div></th> 191 <th><div id="{{ test.name }}_{{ measurement.name }}_chart_png"></div></th>
192 <td></td>
189 </tr> 193 </tr>
190 </table> 194 </table>
191 </td> 195 </td>