From 884d13a55fc328e2e1e3948a82b361b30804b818 Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Tue, 17 Oct 2017 14:46:45 -0700 Subject: time.clock() now emits a DeprecationWarning (GH-4020) bpo-31803: time.clock() and time.get_clock_info('clock') now emit a DeprecationWarning warning. Replace time.clock() with time.perf_counter() in tests and demos. Remove also hasattr(time, 'monotonic') in test_time since time.monotonic() is now always available since Python 3.5. --- Doc/library/profile.rst | 2 +- Doc/library/time.rst | 8 ++++++++ Lib/ctypes/test/test_numbers.py | 2 +- Lib/ctypes/test/test_strings.py | 2 +- Lib/profile.py | 2 +- Lib/test/test_time.py | 22 ++++++++++++---------- Lib/turtledemo/bytedesign.py | 2 +- Lib/turtledemo/forest.py | 2 +- Lib/turtledemo/fractalcurves.py | 2 +- Lib/turtledemo/penrose.py | 2 +- Lib/turtledemo/tree.py | 2 +- Lib/turtledemo/wikipedia.py | 2 +- .../2017-10-17-22-55-13.bpo-31803.YLL1gJ.rst | 2 ++ Modules/timemodule.c | 7 +++++++ 14 files changed, 39 insertions(+), 20 deletions(-) create mode 100644 Misc/NEWS.d/next/Library/2017-10-17-22-55-13.bpo-31803.YLL1gJ.rst diff --git a/Doc/library/profile.rst b/Doc/library/profile.rst index 5796e3a..68f24ab 100644 --- a/Doc/library/profile.rst +++ b/Doc/library/profile.rst @@ -575,7 +575,7 @@ procedure can be used to obtain a better constant for a given platform (see The method executes the number of Python calls given by the argument, directly and again under the profiler, measuring the time for both. It then computes the hidden overhead per profiler event, and returns that as a float. For example, -on a 1.8Ghz Intel Core i5 running Mac OS X, and using Python's time.clock() as +on a 1.8Ghz Intel Core i5 running Mac OS X, and using Python's time.process_time() as the timer, the magical number is about 4.04e-6. The object of this exercise is to get a fairly consistent result. If your diff --git a/Doc/library/time.rst b/Doc/library/time.rst index 25a4ab5..c5d1e83 100644 --- a/Doc/library/time.rst +++ b/Doc/library/time.rst @@ -289,6 +289,9 @@ Functions .. function:: perf_counter() + .. index:: + single: benchmarking + Return the value (in fractional seconds) of a performance counter, i.e. a clock with the highest available resolution to measure a short duration. It does include time elapsed during sleep and is system-wide. The reference @@ -300,6 +303,11 @@ Functions .. function:: process_time() + .. index:: + single: CPU time + single: processor time + single: benchmarking + Return the value (in fractional seconds) of the sum of the system and user CPU time of the current process. It does not include time elapsed during sleep. It is process-wide by definition. The reference point of the diff --git a/Lib/ctypes/test/test_numbers.py b/Lib/ctypes/test/test_numbers.py index ba4f563..09eef90 100644 --- a/Lib/ctypes/test/test_numbers.py +++ b/Lib/ctypes/test/test_numbers.py @@ -241,7 +241,7 @@ class c_int_S(_SimpleCData): def run_test(rep, msg, func, arg=None): ## items = [None] * rep items = range(rep) - from time import clock + from time import perf_counter as clock if arg is not None: start = clock() for i in items: diff --git a/Lib/ctypes/test/test_strings.py b/Lib/ctypes/test/test_strings.py index c7bfbda..e28e141 100644 --- a/Lib/ctypes/test/test_strings.py +++ b/Lib/ctypes/test/test_strings.py @@ -194,7 +194,7 @@ class WStringTestCase(unittest.TestCase): def run_test(rep, msg, func, arg): items = range(rep) - from time import clock + from time import perf_counter as clock start = clock() for i in items: func(arg); func(arg); func(arg); func(arg); func(arg) diff --git a/Lib/profile.py b/Lib/profile.py index 5ceeddc..0340a79 100755 --- a/Lib/profile.py +++ b/Lib/profile.py @@ -195,7 +195,7 @@ class Profile: self.t = r[0] + r[1] - t # put back unrecorded delta # Dispatch routine for best timer program (return = scalar, fastest if - # an integer but float works too -- and time.clock() relies on that). + # an integer but float works too -- and time.process_time() relies on that). def trace_dispatch_i(self, frame, event, arg): timer = self.timer diff --git a/Lib/test/test_time.py b/Lib/test/test_time.py index 61dda09..a08fd18 100644 --- a/Lib/test/test_time.py +++ b/Lib/test/test_time.py @@ -9,6 +9,7 @@ import sysconfig import time import threading import unittest +import warnings try: import _testcapi except ImportError: @@ -64,9 +65,11 @@ class TimeTestCase(unittest.TestCase): self.assertTrue(info.adjustable) def test_clock(self): - time.clock() + with self.assertWarns(DeprecationWarning): + time.clock() - info = time.get_clock_info('clock') + with self.assertWarns(DeprecationWarning): + info = time.get_clock_info('clock') self.assertTrue(info.monotonic) self.assertFalse(info.adjustable) @@ -427,8 +430,6 @@ class TimeTestCase(unittest.TestCase): pass self.assertEqual(time.strftime('%Z', tt), tzname) - @unittest.skipUnless(hasattr(time, 'monotonic'), - 'need time.monotonic') def test_monotonic(self): # monotonic() should not go backward times = [time.monotonic() for n in range(100)] @@ -467,8 +468,6 @@ class TimeTestCase(unittest.TestCase): self.assertTrue(info.monotonic) self.assertFalse(info.adjustable) - @unittest.skipUnless(hasattr(time, 'monotonic'), - 'need time.monotonic') @unittest.skipUnless(hasattr(time, 'clock_settime'), 'need time.clock_settime') def test_monotonic_settime(self): @@ -506,12 +505,15 @@ class TimeTestCase(unittest.TestCase): self.assertRaises(ValueError, time.ctime, float("nan")) def test_get_clock_info(self): - clocks = ['clock', 'perf_counter', 'process_time', 'time'] - if hasattr(time, 'monotonic'): - clocks.append('monotonic') + clocks = ['clock', 'monotonic', 'perf_counter', 'process_time', 'time'] for name in clocks: - info = time.get_clock_info(name) + if name == 'clock': + with self.assertWarns(DeprecationWarning): + info = time.get_clock_info('clock') + else: + info = time.get_clock_info(name) + #self.assertIsInstance(info, dict) self.assertIsInstance(info.implementation, str) self.assertNotEqual(info.implementation, '') diff --git a/Lib/turtledemo/bytedesign.py b/Lib/turtledemo/bytedesign.py index b3b095b..1b7452b 100755 --- a/Lib/turtledemo/bytedesign.py +++ b/Lib/turtledemo/bytedesign.py @@ -23,7 +23,7 @@ mode as fast as possible. """ from turtle import Turtle, mainloop -from time import clock +from time import perf_counter as clock # wrapper for any additional drawing routines # that need to know about each other diff --git a/Lib/turtledemo/forest.py b/Lib/turtledemo/forest.py index 7fe080e..55b7da9 100755 --- a/Lib/turtledemo/forest.py +++ b/Lib/turtledemo/forest.py @@ -13,7 +13,7 @@ http://homepage.univie.ac.at/erich.neuwirth/ """ from turtle import Turtle, colormode, tracer, mainloop from random import randrange -from time import clock +from time import perf_counter as clock def symRandom(n): return randrange(-n,n+1) diff --git a/Lib/turtledemo/fractalcurves.py b/Lib/turtledemo/fractalcurves.py index c49f8b8..54ade96 100755 --- a/Lib/turtledemo/fractalcurves.py +++ b/Lib/turtledemo/fractalcurves.py @@ -12,7 +12,7 @@ methods are taken from the PythonCard example scripts for turtle-graphics. """ from turtle import * -from time import sleep, clock +from time import sleep, perf_counter as clock class CurvesTurtle(Pen): # example derived from diff --git a/Lib/turtledemo/penrose.py b/Lib/turtledemo/penrose.py index f73c864..b2a5813 100755 --- a/Lib/turtledemo/penrose.py +++ b/Lib/turtledemo/penrose.py @@ -17,7 +17,7 @@ For more information see: """ from turtle import * from math import cos, pi -from time import clock, sleep +from time import perf_counter as clock, sleep f = (5**0.5-1)/2.0 # (sqrt(5)-1)/2 -- golden ratio d = 2 * cos(3*pi/10) diff --git a/Lib/turtledemo/tree.py b/Lib/turtledemo/tree.py index 71fff35..9998fa8 100755 --- a/Lib/turtledemo/tree.py +++ b/Lib/turtledemo/tree.py @@ -16,7 +16,7 @@ the current pen is cloned. So in the end there are 1024 turtles. """ from turtle import Turtle, mainloop -from time import clock +from time import perf_counter as clock def tree(plist, l, a, f): """ plist is list of pens diff --git a/Lib/turtledemo/wikipedia.py b/Lib/turtledemo/wikipedia.py index d6bbad8..47d0f00 100644 --- a/Lib/turtledemo/wikipedia.py +++ b/Lib/turtledemo/wikipedia.py @@ -14,7 +14,7 @@ parallel. Followed by a complete undo(). """ from turtle import Screen, Turtle, mainloop -from time import clock, sleep +from time import perf_counter as clock, sleep def mn_eck(p, ne,sz): turtlelist = [p] diff --git a/Misc/NEWS.d/next/Library/2017-10-17-22-55-13.bpo-31803.YLL1gJ.rst b/Misc/NEWS.d/next/Library/2017-10-17-22-55-13.bpo-31803.YLL1gJ.rst new file mode 100644 index 0000000..10a88ba --- /dev/null +++ b/Misc/NEWS.d/next/Library/2017-10-17-22-55-13.bpo-31803.YLL1gJ.rst @@ -0,0 +1,2 @@ +time.clock() and time.get_clock_info('clock') now emit a DeprecationWarning +warning. diff --git a/Modules/timemodule.c b/Modules/timemodule.c index 6af9a90..463f5c5 100644 --- a/Modules/timemodule.c +++ b/Modules/timemodule.c @@ -104,6 +104,13 @@ perf_counter(_Py_clock_info_t *info) static PyObject* pyclock(_Py_clock_info_t *info) { + if (PyErr_WarnEx(PyExc_DeprecationWarning, + "time.clock has been deprecated in Python 3.3 and will " + "be removed from Python 3.8: " + "use time.perf_counter or time.process_time " + "instead", 1) < 0) { + return NULL; + } #ifdef MS_WINDOWS return perf_counter(info); #else -- cgit v0.12