Description

AA-Turner

opened on Aug 11, 2025

Issue body actions

Feature or enhancement

Proposal:

_interpretersmodule.c currently exclusively uses PyArg_ParseTupleAndKeywords/PyArg_ParseTuple.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs

• GH-137630: Convert _interpreters to use Argument Clinic  #137631
• GH-137630: Argument Clinic: Reduce use of 'as' for renaming in _interpretersmodule.c #137680
• GH-137630: Convert _interpqueues to use Argument Clinic #137685
• GH-137630: Convert _interpchannels to use Argument Clinic #139141

Activity

1. AA-Turner

   added

   type-featureA feature request or enhancementA feature request or enhancement

   topic-subinterpreters

   topic-argument-clinic

   and removed

   type-featureA feature request or enhancementA feature request or enhancement

   on Aug 11, 2025
2. github-project-automation

   added this to Subinterpreterson Aug 11, 2025
3. github-project-automation

   moved this to Todo in Subinterpreterson Aug 11, 2025
4. picnixz

   added

   interpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)

   type-refactorCode refactoring (with no changes in behavior)Code refactoring (with no changes in behavior)

   on Aug 11, 2025
5. AA-Turner

   added 2 commits that reference this issue on Aug 12, 2025

   GH-137630: Convert ``_interpreters`` to use Argument Clinic (#137631)

   be56464

   GH-137630: Argument Clinic: Reduce use of 'as' for renaming in ``_int…

   797c2c3
6. hnnynh commented on Aug 15, 2025

   

   hnnynh

   on Aug 15, 2025

   Contributor

   More actions

   It seems that _interpchannels also needs to be worked on.
   Would it be okay for me to submit a PR(Convert _interpchannels to use Argument Clinic)?
7. corona10 commented on Aug 15, 2025

   

   corona10

   on Aug 15, 2025

   Member

   More actions

   @AA-Turner @hnnynh is a first-time sprint participant at PyCon KR, and my colleague, it would be a good task to be familiar with the CPython code base with this task.
8. AA-Turner commented on Aug 15, 2025

   

   AA-Turner

   on Aug 15, 2025

   MemberAuthor

   More actions

   Thanks for the proposal both, I think we should wait here as there is an open PR waiting for Eric S.'s feedback.

   Hopefully you can find another good issue to work on during the sprint @corona10 @hnnynh!

   A

   Reacted by Donghee Na
9. corona10 commented on Aug 15, 2025

   

   corona10

   on Aug 15, 2025

   Member

   More actions

   @AA-Turner Hi,
   The sprint day has wrapped up today, but I think one of the main purposes of organizing sprints is to find promising contributor candidates. Would it be okay for her to follow up on the _interpchannels module if @ericsnowcurrently approves the other PR?

   Best,
   Donghee
10. AA-Turner commented on Aug 15, 2025

    

    AA-Turner

    on Aug 15, 2025

    MemberAuthor

    More actions

    Converting an entire module to Argument Clinic all at once can be quite tricky, _interpqueues.c wasn't very straightforwards (it involved writing a custom converter, which is fairly advanced even for AC). I'd prefer to do _interpchannels myself for this reason, and to keep consistency.

    For a new contributor, perhaps you would want to try converting _json.c to Argument Clinic instead? I would happy to help review @hnnynh's PR to do this @corona10 :-)

    There are four functions to convert in _json, so hopefully it should not be too hard!

    A

    Reacted by Donghee Na
11. AA-Turner commented on Aug 15, 2025

    

    AA-Turner

    on Aug 15, 2025

    MemberAuthor

    More actions

    @corona10 @hnnynh see #137821

    Reacted by Yoonho Hann
12. Agent-Hellboy

    added 2 commits that reference this issue on Aug 19, 2025

    pythonGH-137630: Convert ``_interpreters`` to use Argument Clinic (py…

    904c95d

    pythonGH-137630: Argument Clinic: Reduce use of 'as' for renaming in …

    d8f7a9b
13. AA-Turner

    added a commit that references this issue on Sep 17, 2025

    GH-137630: Convert ``_interpqueues`` to use Argument Clinic (#137685)

    4b78fe9
14. ZeroIntensity commented on Sep 18, 2025

    

    ZeroIntensity

    on Sep 18, 2025

    Member

    More actions

    @AA-Turner Is this done?
15. AA-Turner commented on Sep 18, 2025

    

    AA-Turner

    on Sep 18, 2025

    MemberAuthor

    More actions

    No (_interpchannels remains).

    A

Issue actions