[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2017-03-31 Thread Donald Stufft
Changes by Donald Stufft : -- pull_requests: +902 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-23 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-23 Thread Roundup Robot
Roundup Robot added the comment: New changeset 8e6cc952adc6 by Serhiy Storchaka in branch 'default': Issue #28469: timeit now uses the sequence 1, 2, 5, 10, 20, 50,... instead https://hg.python.org/cpython/rev/8e6cc952adc6 -- nosy: +python-dev ___ Pyt

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-23 Thread STINNER Victor
STINNER Victor added the comment: timeit_autorange_numbers.patch: LGTM. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-22 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Following patch takes numbers from the sequence 1, 2, 5, 10, 20, 50, ... It also updates the documentation and tests. -- components: +Library (Lib) stage: -> patch review Added file: http://bugs.python.org/file45186/timeit_autorange_numbers.patch __

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-18 Thread STINNER Victor
STINNER Victor added the comment: > The perf module doesn't report the number of loops by default. Right, it's hidden by default. I consider that it's not an useful information for end users. If you want more details (see the number of inner and outer loops), use --verbose ("python3 -m perf t

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-18 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: The perf module doesn't report the number of loops by default. -- ___ Python tracker ___ ___ Pytho

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-18 Thread STINNER Victor
STINNER Victor added the comment: > "131072 loops" in a report looks not human friendly. In my perf module, I use 2^n syntax for numbers larger than 1024. Syntax accepted in command line arguments. -- ___ Python tracker

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-18 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: "131072 loops" in a report looks not human friendly. I would prefer using a sequence of round numbers: 1, 2, 5, 10, 20, 50, 100, ... -- ___ Python tracker __

[issue28469] timeit: use powers of 2 in autorange(), instead of powers of 10

2016-10-18 Thread STINNER Victor
New submission from STINNER Victor: In the issue #28240, I changed the default repeat from 3 to 5 in timeit. Serhiy wrote (msg277157): "For now default timeit run takes from 0.8 to 8 sec. Adding yet 5 sec makes a user more angry." I propose to use powers to 2, instead of powers of 10, in timei