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

Add experimental setting and embed REPL controller code into the Python extension. #23175

Closed
Tracked by #23332
anthonykim1 opened this issue Apr 3, 2024 · 3 comments · Fixed by #23235
Closed
Tracked by #23332
Assignees
Labels
area-repl feature-request Request for new features or functionality triage-needed Needs assignment to the proper sub-team verified Verification succeeded
Milestone

Comments

@anthonykim1
Copy link

We should add experimental setting to add option to use custom REPL as a kernel (instead of the traditional IPython Kernel) when user decides to run their Python code via interactive window.

@anthonykim1 anthonykim1 added feature-request Request for new features or functionality area-repl labels Apr 3, 2024
@anthonykim1 anthonykim1 added this to the April 2024 milestone Apr 3, 2024
@anthonykim1 anthonykim1 self-assigned this Apr 3, 2024
@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Apr 3, 2024
@anthonykim1 anthonykim1 removed the triage-needed Needs assignment to the proper sub-team label Apr 3, 2024
@karthiknadig
Copy link
Member

This is also done in the linked PR, but it is not testable yet.

@karthiknadig karthiknadig removed the feature-request Request for new features or functionality label Apr 17, 2024
@karthiknadig karthiknadig modified the milestones: April 2024, May 2024 Apr 17, 2024
@karthiknadig karthiknadig added the feature-request Request for new features or functionality label Apr 17, 2024
@amunger amunger reopened this May 5, 2024
@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label May 5, 2024
@anthonykim1
Copy link
Author

Should be good with changes merged in #23235

anthonykim1 added a commit that referenced this issue May 16, 2024
Allow users to use Interactive Window UI with Python custom REPL
controller instead of iPykernel.
Closes #23175
Closes #23174 
Closes #23029
Majority of: #23332 

Context menu under Python for running Python REPL code using IW UI
should only appear when user's ```pythonRunREPL``` experiment is
enabled.
anthonykim1 added a commit to anthonykim1/vscode-python that referenced this issue May 16, 2024
Allow users to use Interactive Window UI with Python custom REPL
controller instead of iPykernel.
Closes microsoft#23175
Closes microsoft#23174 
Closes microsoft#23029
Majority of: microsoft#23332 

Context menu under Python for running Python REPL code using IW UI
should only appear when user's ```pythonRunREPL``` experiment is
enabled.
@benibenj benibenj added the verification-needed Verification of issue is requested label May 28, 2024
@anthonykim1
Copy link
Author

This will be tested via: #23484 hence marking as verified so they do not appear in endgame queries.

@anthonykim1 anthonykim1 added verified Verification succeeded and removed verification-needed Verification of issue is requested labels May 28, 2024
DonJayamanne pushed a commit that referenced this issue Jun 24, 2024
Allow users to use Interactive Window UI with Python custom REPL
controller instead of iPykernel.
Closes #23175
Closes #23174 
Closes #23029
Majority of: #23332 

Context menu under Python for running Python REPL code using IW UI
should only appear when user's ```pythonRunREPL``` experiment is
enabled.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-repl feature-request Request for new features or functionality triage-needed Needs assignment to the proper sub-team verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants