python / python/cpython

Argument Clinic: add a parametrized converter for durations and timestamps

Offen
#156,263 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@serhiy-storchaka arbeitet bereits daran.

Seit 23.8.2026.

topic-argument-clinic type-feature
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Feature or enhancement

Many functions accept a duration (a timeout, an interval, a delay) or a timestamp, and every one of them converts the argument in the "impl" function. They are declared as object (or double, or a Windows DWORD) and converted with _PyTime_FromSecondsObject(), _PyTime_FromMillisecondsObject(), _PyTime_ObjectToTime_t(), _PyTime_ObjectToTimeval(), _PyTime_ObjectToTimespec() and the corresponding _PyTime_As*() functions.

The conversions differ in several independent aspects:

  • the unit of the argument: seconds (most of them) or milliseconds (select.poll.poll(), select.devpoll.poll());
  • the rounding mode: _PyTime_ROUND_TIMEOUT (select, socket, _thread, faulthandler, time.sleep()), _PyTime_ROUND_CEILING (signal.sigtimedwait(), signal.setitimer(), _queue, _ssl) or _PyTime_ROUND_FLOOR (time.gmtime(), datetime.date.fromtimestamp());
  • the type and unit used by the implementation: PyTime_t (nanoseconds), microseconds or milliseconds as an integer, struct timeval or struct timespec, double seconds, time_t seconds, or Windows DWORD milliseconds;
  • the meaning of None: block forever (select, _queue, _multiprocessing.SemLock), the current time (time.gmtime(), time.localtime(), time.ctime()), not specified (_thread, where the Python default is -1), or not accepted at all (signal.sigtimedwait());
  • the range check: no check, or ValueError with one of several messages -- "timeout must be non-negative", "timeout must be positive or None", "timeout is too large", "timeout value is too large", "timeout must be greater than 0".

The error for a wrong type differs too: select raises "timeout must be a real number or None, not %T", other modules use the message produced by _PyTime_FromSecondsObject().

Timestamps need the same machinery: datetime.date.fromtimestamp() uses _PyTime_ObjectToTime_t(), datetime.datetime.fromtimestamp() uses _PyTime_ObjectToTimeval(), os.utime() uses _PyTime_ObjectToTimespec() for the items of the times tuple, and time.gmtime(), time.localtime() and time.ctime() use a private helper which also treats None as the current time.

Parametrized converters would allow to declare all of them, for example:

    timeout: duration(unit='s', out='ms', round='timeout', accept={float, NoneType})
    timestamp: timestamp(out='time_t', round='floor')

There are 19 parameters named "timeout" and at least 10 more of the same kind (seconds and interval of signal.setitimer(), the argument of time.sleep(), secs of the time functions, timestamp of the datetime constructors). Unifying them will also unify the error messages, which is a user visible change.

Linked PRs
  • gh-156291

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.