diff options
author | Pavel Kosov <kosov.pavel@huawei.com> | 2022-01-27 15:45:27 +0400 |
---|---|---|
committer | Pavel Kosov <kosov.pavel@huawei.com> | 2022-01-27 15:45:27 +0400 |
commit | 9846672b95f472e4b914597b8e75fe9fc82345fb (patch) | |
tree | 4ae5aae7633860dd31c7b9aa9413420021570e2a | |
parent | bbf46b2fe6b18957c5758dec66341a31f8418628 (diff) |
[LNT] Fix the graph page to show multiple metrics for the same test
Note: request.args is a dict and duplicate keys are ignored.
Currently the plot args have the following format plot.unused=m.t.f where unused is the test id.
If you select few metrics for the same test (for example compile_time and execution_time) on the Run page and click Graph, the URL will look like /db_default/v4/nts/graph?plot.958=5.958.2&plot.958=5.958.3
The second plot argument with the same key will be ignored. This patch fixes this issue.
OS Laboratory. Huawei Russian Research Institute. Saint-Petersburg
Reviewed By: cmatthews
Differential Revision: https://reviews.llvm.org/D118170
-rw-r--r-- | lnt/server/ui/templates/v4_run.html | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lnt/server/ui/templates/v4_run.html b/lnt/server/ui/templates/v4_run.html index c063f2d..fc1d9d8 100644 --- a/lnt/server/ui/templates/v4_run.html +++ b/lnt/server/ui/templates/v4_run.html @@ -345,7 +345,7 @@ $('.profile-prev-only').tooltip(); <tbody class="searchable"> {% for test_name,test_id,cr in tests %} <tr> - <td><input type="checkbox" name="plot.{{test_id}}" value="{{machine.id}}.{{test_id}}.{{field_index}}"/></td> + <td><input type="checkbox" name="plot.{{test_id}}.{{field_index}}" value="{{machine.id}}.{{test_id}}.{{field_index}}"/></td> <td class="benchmark-name"> <a href="{{graph_base}}&plot.{{test_id}}={{ machine.id}}.{{test_id}}.{{field_index}}"> {{ test_name }} |