PEP 699: Remove private dict version field added in PEP 509

Hello! PEP 699 is a PEP to undo the changes in PEP 509. It’s very short so please give it a read and leave any comments in this discourse thread.

CC @vstinner and @markshannon

4 Likes

Is it worth pointing out in the PEP which version the ma_version_tag field was added in? Are we talking about something fairly new or that’s been around for a while.

2 Likes

It’s 3.6. However, it wasn’t actually properly used until 3.8 (at least by CPython) for LOAD_GLOBAL.

This looks good to me. FWIW Cinder does not use the version tag for any optimizations, but does rely on dictionary watchers, which will be simpler to implement if we don’t have to steal bits from the version tag while still updating it (as done in add support for watching writes to selected dictionaries · Issue #91052 · python/cpython · GitHub) but can just eliminate it instead.

I was pinged about this on Github but responding here because it’s probably more useful and more visible.

I don’t think this should cause Cython any problem. Cython only uses that feature on some Python anyway and falls back to public-API-only code for older versions/PyPy/limited API/etc. So it’s just a case of changing the version guard that selects this feature.

The PEP is now online at https://peps.python.org/pep-0699/

Just for reference, Deepmind’s S6 (GitHub - deepmind/s6) used ma_version_tag for hidden classes: s6/util.h at 557187c514e9e9f7e42ad060ecbd50c5483f02c9 · deepmind/s6 · GitHub
(This is just for reference since Deepmind stopped work on S6.)

The header section needs hard returns so it reads like…

PEP: 699
Title: Remove private dict version field added in PEP 509
Author: Ken Jin <kenjin@python.org>

Click on “raw” button on GitHub to see newlines :wink: