Closed
Bug 514238
Opened 15 years ago
Closed 15 years ago
runtest.py output updates
Categories
(Tamarin Graveyard :: Tools, enhancement)
Tamarin Graveyard
Tools
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: brbaker, Assigned: cpeyer)
Details
Attachments
(1 file, 2 obsolete files)
22.98 KB,
patch
|
Details | Diff | Splinter Review |
1) Remove the outputting of html data. This is no longer used and is just dead code. Also remove the --nohtml switch 2) Add the ability to output the summary data as csv via --csvoutput switch
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → cpeyer
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•15 years ago
|
||
This patch adds: - csv output to stdout using the --csv flag - replaces --nohtml with --html and turn html output off by default - fix variance calculation - fix output of memory results when comparing 2 vms with multiple iterations - add info message describing how %diff is calculated
Attachment #398505 -
Flags: review?(brbaker)
Reporter | ||
Comment 2•15 years ago
|
||
This is a patch that Edwin has provided and rebased to the first patch in this bug
Attachment #400076 -
Flags: review?(cpeyer)
Reporter | ||
Comment 3•15 years ago
|
||
Comment on attachment 398505 [details] [diff] [review] add csv output ability - should be able to drop the 'nohtml' from util/runtestBase.py:255 - I sort of liked the older style of output when running a single VM with -i #, was a little more compact than the new style
Attachment #398505 -
Flags: review?(brbaker) → review+
Assignee | ||
Comment 4•15 years ago
|
||
This patch incorporates both previous patches. I changed the display format with multiple iterations back to its more compact format when displaying to screen, but when outputting to csv, individual column headers will be outputted. 'nohtml' command line option also removed from runtestBase.py
Attachment #398505 -
Attachment is obsolete: true
Attachment #400076 -
Attachment is obsolete: true
Attachment #400147 -
Flags: review?(brbaker)
Attachment #400076 -
Flags: review?(cpeyer)
Assignee | ||
Comment 5•15 years ago
|
||
Pushed to TR with 2490:3234a1ebaee0 and 2493:54bb214e3883
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Updated•15 years ago
|
Attachment #400147 -
Flags: review?(brbaker)
Reporter | ||
Comment 6•15 years ago
|
||
removing QRB request, bug resolved/verified
Flags: flashplayer-qrb?
You need to log in
before you can comment on or make changes to this bug.
Description
•