Closed
Bug 891293
Opened 11 years ago
Closed 11 years ago
Make compare.py output tabular
Categories
(Testing :: Talos, enhancement)
Testing
Talos
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: MattN, Assigned: MattN)
Details
Attachments
(1 file)
2.97 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
This allows for easy copy-paste into a spreadsheet and makes it easier to visually scan IMO. * I added the heading because many people were confused about the output. * Some pieces of data showed too many decimal places so I fixed them to 1 in order to try reduce noise in the output.
Attachment #772574 -
Flags: review?(jmaher)
Comment 1•11 years ago
|
||
Comment on attachment 772574 [details] [diff] [review] v.1 Add header, use tabs for columns & fix to 1 decimal place Review of attachment 772574 [details] [diff] [review]: ----------------------------------------------------------------- I would be fine getting rid of the original format as it is. We could do something like: test result min max newvalue ts_paint pass 89 102 97 tpaint fail 2000 2202 3141 note the spaces could be \t characters. I am open to other formats which would be easier to use. We could alternatively have a cli flag to output various formats of interest!
Attachment #772574 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 2•11 years ago
|
||
I pushed this for now with some tweaks to the headings to make them closer to your example. https://hg.mozilla.org/build/talos/rev/d40aba1fb449
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•