[issue7029] Improve pybench

2009-10-09 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: Fixed the bug with "timer = timer" in trunk in revision 75293 -- ___ Python tracker ___ ___

[issue7029] Improve pybench

2009-10-07 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: Ok, thanks for the info. We are currently using the modified version in house for monitoring our builds and that's ok for my purposes. I have set the -C 0 and --timer systimes.processtime as default for us. I'll close this now and leave you to do the

[issue7029] Improve pybench

2009-10-06 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: Kristján Valur Jónsson wrote: > > Kristján Valur Jónsson added the comment: > > Here is an updated patch with the superfluous timing function removed. > Please advise me on how you "don't like the implementation" The reporting methods are a complete mess

[issue7029] Improve pybench

2009-10-06 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: And here is the actual patch. -- Added file: http://bugs.python.org/file15056/pybench.patch ___ Python tracker ___ _

[issue7029] Improve pybench

2009-10-06 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: Here is an updated patch with the superfluous timing function removed. Please advise me on how you "don't like the implementation" I'm also considering adding a '--quiet' flag that causes it to only emit the 'total' line. This is useful for automated

[issue7029] Improve pybench

2009-10-01 Thread Kristján Valur Jónsson
Kristján Valur Jónsson added the comment: Hello Marc-Andre. 1) yes, this is nonsense. I thought that "systimes" was some archaeic module that had been discontinued, since I didn't find it in the standard modules (where "time" is). I didn't realize that it was a special helper sibling-module

[issue7029] Improve pybench

2009-10-01 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: Kristján Valur Jónsson wrote: > Thanks for the patch. Here's a quick review (a bit terse, but hope you don't mind)... > The attached patch contains suggested fixes to pybench.py: > 1) add a processtime timer for windows I'm not sure why you added this: th

[issue7029] Improve pybench

2009-10-01 Thread Kristján Valur Jónsson
New submission from Kristján Valur Jónsson : The attached patch contains suggested fixes to pybench.py: 1) add a processtime timer for windows 2) fix a bug in timer selection: timer wasn't put in 'self' 3) Collect 'gross' times for each round 4) show statistics for per-round sums, both for sum of