514238 - runtest.py output updates

Status: VERIFIED FIXED

(Tamarin Graveyard :: Tools, enhancement)

Description

[Brent Baker]

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

Comment 1

[Chris Peyer]

Attachment: add csv output ability

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

Comment 2

[Brent Baker]

Attachment: fix spdup when larger is faster

This is a patch that Edwin has provided and rebased to the first patch in this bug

Comment 3

[Brent Baker]

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

Comment 4

[Chris Peyer]

Attachment: Updated patch that incorporates Brents feedback.

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

Comment 5

[Chris Peyer]

Pushed to TR with  2490:3234a1ebaee0 and 2493:54bb214e3883

Comment 6

[Brent Baker]

removing QRB request, bug resolved/verified