[issue18983] Specify time unit for timeit CLI

Changes by Robert Collins : -- nosy: +rbcollins resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ __

[issue18983] Specify time unit for timeit CLI

Roundup Robot added the comment: New changeset ed34dd00405e by Robert Collins in branch 'default': Fix patch attribution for issue 18983. https://hg.python.org/cpython/rev/ed34dd00405e -- ___ Python tracker ___

[issue18983] Specify time unit for timeit CLI

Roundup Robot added the comment: New changeset 1ebf8d5b7d60 by Robert Collins in branch 'default': Issue #18983: Allow selection of output units in timeit. https://hg.python.org/cpython/rev/1ebf8d5b7d60 -- nosy: +python-dev ___ Python tracker

[issue18983] Specify time unit for timeit CLI

Berker Peksag added the comment: I've added a couple of comments on Rietveld. -- nosy: +berker.peksag ___ Python tracker ___ ___ Pytho

[issue18983] Specify time unit for timeit CLI

Vincent Davis added the comment: Anything else need to be done on this patch? -- nosy: +Vincentdavis ___ Python tracker ___ ___ Python

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Anything else need to be done on this patch? -- ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue18983] Specify time unit for timeit CLI

Quentin Pradet added the comment: The branch appears to exist now. -- nosy: +Quentin.Pradet ___ Python tracker ___ ___ Python-bugs-lis

[issue18983] Specify time unit for timeit CLI

Vajrasky Kok added the comment: This feature is for Python 3.5. I think, we will not commit anything to Python 3.5 until Python 3.4 branch is created (in other world, released). So, be patient. :) -- nosy: +vajrasky ___ Python tracker

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: What needs to be done to close this one out? -- ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue18983] Specify time unit for timeit CLI

Serhiy Storchaka added the comment: LGTM. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Just wanted to check in and see if the documentation change I made is sufficient. I tried to copy the other entries for the command line arguments. Let me know if I can improve it in any way. -- ___ Python tracker

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Updated documentation and fixed a few lines that were too long. -- Added file: http://bugs.python.org/file33095/issue18983_v6.patch ___ Python tracker ___

[issue18983] Specify time unit for timeit CLI

Serhiy Storchaka added the comment: Ah, I forgot. This feature should be documented in Doc/library/timeit.rst. -- ___ Python tracker ___ _

[issue18983] Specify time unit for timeit CLI

Serhiy Storchaka added the comment: Besides too long line with sys.stderr.write last patch LGTM. -- ___ Python tracker ___ ___ Python-

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Re-uploaded the patch to remove duplication. -- ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Whoops. Sorry about that. Super embarrassing. -- Added file: http://bugs.python.org/file33068/issue18983_v5.patch ___ Python tracker ___ _

[issue18983] Specify time unit for timeit CLI

Serhiy Storchaka added the comment: The patch is doubled. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Added newline after error message. -- Added file: http://bugs.python.org/file33067/issue18983_v5.patch ___ Python tracker ___

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Updated patch to log to stderr. -- Added file: http://bugs.python.org/file33066/issue18983_v4.patch ___ Python tracker ___ ___

[issue18983] Specify time unit for timeit CLI

Serhiy Storchaka added the comment: It will be better to write error message to sys.stderr (see test_main_exception). -- ___ Python tracker ___ _

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Incorporated updates suggested by serhiy.storchaka -- Added file: http://bugs.python.org/file33064/issue18983_v3.patch ___ Python tracker ___

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Implemented proposed changes to patch. Simplified "for-loop" and implemented invalid input test. -- Added file: http://bugs.python.org/file33058/issue18983_v2.patch ___ Python tracker

[issue18983] Specify time unit for timeit CLI

Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: needs patch -> patch review versions: +Python 3.5 -Python 3.4 ___ Python tracker ___ _

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Just wanted to check to see if there was anything else I should do regarding this issue. -- ___ Python tracker ___ __

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: Updated patch to include unit tests. -- Added file: http://bugs.python.org/file32874/issue18983.patch ___ Python tracker ___ _

[issue18983] Specify time unit for timeit CLI

Julian Gindi added the comment: I created a patch that allows users to specify a time unit output using the "-u" flag. Example usage: python -m timeit -u "msec" '"-".join(str(n) for n in range(100))' >> 1 loops, best of 3: 0.156 msec per loop python -m timeit -u "sec" '"-".join(str(n) for

[issue18983] Specify time unit for timeit CLI

Jakub Stasiak added the comment: My 2 cents - I'd split timeit.main function into processing part and argument parsing + data printing part so that you could use Python to call the first one to get the actual results (timings, numbers of loops etc.) and then present the data to the outside wor

[issue18983] Specify time unit for timeit CLI

Antoine Pitrou added the comment: If you want the output to be machine readable, better go the full way and add a --json option, IMO. -- nosy: +pitrou, tim.peters ___ Python tracker ___

[issue18983] Specify time unit for timeit CLI

New submission from Serhiy Storchaka: Just an idea. Currently the timeit module in command line automatically choice most appropriate time unit for human readable output. But this makes the output less machine readable. I propose to add new option which specifies fixed time unit (usec, msec, s