Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-107784 Document interpid C-API and new Unstable API for fetching __main__ of interpreter #112548

Closed
wants to merge 5 commits into from

Conversation

tonybaloney
Copy link
Contributor

@tonybaloney tonybaloney commented Nov 30, 2023

@tonybaloney
Copy link
Contributor Author

Can be back-ported to Python 3.12

@tonybaloney
Copy link
Contributor Author

@ericsnowcurrently interpid type is opaque, is that correct?

Copy link
Contributor

@blaisep blaisep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change

A interpreter ID identifies a interpreter and may be used as an int.

to

A interpreter ID identifies a interpreter and may be used as an :class:`int`.

@@ -1464,6 +1470,29 @@ function. You can create and destroy them using the following functions:
:c:func:`Py_FinalizeEx` will destroy all sub-interpreters that
haven't been explicitly destroyed at that point.

Interpreter ID Objects
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm giving this some attention so that we can get it merged soon.
This section looks good so far

Interpreter ID Objects
----------------------

A interpreter ID identifies a interpreter and may be used as an int.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hugovk suggested that you link to the class definition,
changing

A interpreter ID identifies a interpreter and may be used as an int.

to

A interpreter ID identifies a interpreter and may be used as an :class:`int`.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
A interpreter ID identifies a interpreter and may be used as an int.
A interpreter ID identifies a interpreter and may be used as an :class:`int`.

Copy link
Contributor

@blaisep blaisep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(added the change as a markdown suggestion to make it more convenient)

Interpreter ID Objects
----------------------

A interpreter ID identifies a interpreter and may be used as an int.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
A interpreter ID identifies a interpreter and may be used as an int.
A interpreter ID identifies a interpreter and may be used as an :class:`int`.

@ericsnowcurrently
Copy link
Member

FYI, I've dropped interpreter ID objects.

@blaisep
Copy link
Contributor

blaisep commented May 21, 2024

FYI, I've dropped interpreter ID objects.
Are you recommending that we close this PR? (Is is obsolete?)

@ericsnowcurrently
Copy link
Member

I'd need to double-check, but, yeah, we'd probably close this PR.

@blaisep
Copy link
Contributor

blaisep commented May 21, 2024

I'd need to double-check, but, yeah, we'd probably close this PR.

Thank you @ericsnowcurrently , I will ask @tonybaloney if he will close this PR.

@willingc
Copy link
Contributor

willingc commented Nov 4, 2024

@tonybaloney I'm going to close this PR. If needed we can reopen. Thanks.

@willingc willingc closed this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants