◐ Shell
reader mode source ↗
Skip to content

bpo-46850: Move _PyInterpreterState_SetEvalFrameFunc() to internal C API#32054

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:eval_func
Apr 1, 2022
Merged

bpo-46850: Move _PyInterpreterState_SetEvalFrameFunc() to internal C API#32054
vstinner merged 1 commit into
python:mainfrom
vstinner:eval_func

Conversation

@vstinner

@vstinner vstinner commented Mar 22, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

Move the private _PyFrameEvalFunction type, and private
_PyInterpreterState_GetEvalFrameFunc() and
_PyInterpreterState_SetEvalFrameFunc() functions to the internal C
API. The _PyFrameEvalFunction callback function type now uses the
_PyInterpreterFrame type which is part of the internal C API.

Update the _PyFrameEvalFunction documentation.

https://bugs.python.org/issue46850

@vstinner

vstinner commented Mar 22, 2022

Copy link
Copy Markdown
Member Author

See also PR #32052 "Remove _PyEval_EvalFrameDefault() function".

@vstinner vstinner mentioned this pull request Mar 22, 2022
Merged
@vstinner

vstinner commented Mar 22, 2022

Copy link
Copy Markdown
Member Author

According to a code search on top 5000 PyPI project (2022-01-26), two debugger projects are affected, the same projects than the projects affected by PR #32052.

debugpy-1.5.1.zip:

src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: # ctypedef PyObject* _PyFrameEvalFunction(PyThreadState* tstate, PyFrameObject *frame, int exc)
src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: # ctypedef PyObject* _PyFrameEvalFunction(PyFrameObject *frame, int exc)
src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: ctypedef PyObject* _PyFrameEvalFunction(...)
src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: _PyFrameEvalFunction eval_frame
src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.template.pyx: state.interp.eval_frame = <_PyFrameEvalFunction *> get_bytecode_while_frame_eval_39
src/debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.template.pyx: state.interp.eval_frame = <_PyFrameEvalFunction *> get_bytecode_while_frame_eval_38

ptvsd-4.3.2.zip:

src/ptvsd/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: ctypedef PyObject* _PyFrameEvalFunction(PyFrameObject *frame, int exc)
src/ptvsd/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.pxd: _PyFrameEvalFunction eval_frame

@jansel

jansel commented Mar 28, 2022 •
edited
Loading

Copy link
Copy Markdown

This (and PEP-523) is used in TorchDynamo:
https://github.com/facebookresearch/torchdynamo

We (the PyTorch team) are planning to include TorchDynamo as part of PyTorch 2.0.

@vstinner

vstinner commented Apr 1, 2022

Copy link
Copy Markdown
Member Author

PR rebased on top of #32052 that I plan to merge first.

@vstinner vstinner changed the title bpo-46850: Remove _PyInterpreterState_SetEvalFrameFunc() Apr 1, 2022
@vstinner
bpo-46850: Move _PyInterpreterState_SetEvalFrameFunc() to internal C API …
cc55444 
Move the private _PyFrameEvalFunction type, and private
_PyInterpreterState_GetEvalFrameFunc() and
_PyInterpreterState_SetEvalFrameFunc() functions to the internal C
API. The _PyFrameEvalFunction callback function type now uses the
_PyInterpreterFrame type which is part of the internal C API.

Update the _PyFrameEvalFunction documentation.
@vstinner vstinner merged commit f877b40 into python:main Apr 1, 2022
@vstinner vstinner deleted the eval_func branch April 1, 2022 08:55
@vstinner vstinner mentioned this pull request Apr 5, 2022
Merged
vstinner added a commit that referenced this pull request Apr 6, 2022
@vstinner
Revert "bpo-46850: Move _PyEval_EvalFrameDefault() to internal C API (G… …
2b4f2f5 
…H-32052)" (GH-32343)

* Revert "bpo-46850: Move _PyInterpreterState_SetEvalFrameFunc() to internal C API (GH-32054)"

This reverts commit f877b40.

* Revert "bpo-46850: Move _PyEval_EvalFrameDefault() to internal C API (GH-32052)"

This reverts commit b9a5522.
@itamarst

itamarst commented Apr 15, 2022

Copy link
Copy Markdown

I rely on this API and (other PEP 523 APIs as well) for https://pythonspeed.com/sciagraph/, so would like it or some equivalent to continue to exist.

@vstinner

vstinner commented Apr 19, 2022

Copy link
Copy Markdown
Member Author

This change was reverted: #32343

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@vstinner @jansel @itamarst @the-knights-who-say-ni @bedevere-bot