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

Remove Interrupt buttons and commands from Jupyter #7661

Closed
roblourens opened this issue Sep 24, 2021 · 8 comments · Fixed by #13239
Closed

Remove Interrupt buttons and commands from Jupyter #7661

roblourens opened this issue Sep 24, 2021 · 8 comments · Fixed by #13239
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug notebook-workflow Issues that interrupt expected or desirable behavior papercut 🩸 Something affecting the productivity of the team upstream-vscode Blocked on upstream VS code verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

  • "Run all" on a long notebook eg Titanic.ipynb
  • The interrupt button blinks on and off
  • This is actually contributed by Jupyter, and it looks like the when clause changed recently @joyceerhl

Filing this because @kieferrm mentioned it, we could leave the when clause alone (if that is causing the issue), but next month we have this button in vscode itself. See microsoft/vscode#133776 also

@roblourens roblourens added the bug Issue identified by VS Code Team member as probable bug label Sep 24, 2021
@rchiodo rchiodo added this to the October 2021 milestone Sep 27, 2021
@rchiodo rchiodo added papercut 🩸 Something affecting the productivity of the team and removed needs-triage labels Sep 27, 2021
@DonJayamanne DonJayamanne self-assigned this Oct 4, 2021
@DonJayamanne
Copy link
Contributor

Even if we fixed this, i don't see a point, as we're planning on moving this into core in the same iteration!
To be discussed whether we want to fix this in Jupyter or not.

Suggestion

  • VS Code enables the icon only when cells are queued/running
  • Remove interrupt from Jupyter
    • Currently the interrupt only appears in global toolbar & not in editor toolbar (moving to core will fix this - hopefully)

@greazer greazer removed this from the October 2021 milestone Oct 7, 2021
@DonJayamanne DonJayamanne added the upstream-vscode Blocked on upstream VS code label Oct 8, 2021
@DonJayamanne
Copy link
Contributor

The changes will be made in VS Code core.
Leaving this issue open so we remove the custom toolbar icons & commands.

@greazer greazer changed the title Interrupt button blinks Remove Interrupt buttons and commands from Jupyter Oct 8, 2021
@greazer greazer added notebook-workflow Issues that interrupt expected or desirable behavior and removed discuss-at-standup labels Oct 8, 2021
@DonJayamanne DonJayamanne added this to the October 2021 milestone Oct 11, 2021
@rchiodo rchiodo modified the milestones: November 2021, January 2022 Dec 6, 2021
@rchiodo
Copy link
Contributor

rchiodo commented Jan 5, 2022

This requires core changes so it's blocked.

@amunger amunger added this to the August 2022 milestone Jul 29, 2022
@DonJayamanne
Copy link
Contributor

@roblourens any update on when or even if the interrupt button will be added into core.
I coudln't find any issue that tracks that work in vscode repo. Probably there is and I couldn't find it.

@roblourens
Copy link
Member Author

That's microsoft/vscode#133776 - from what I remember, the button is there but does not appear.

@roblourens
Copy link
Member Author

It looks like currently the Jupyter command is still used for the Interactive Window, and the builtin command is used for notebook editors, is there a reason for the @rebornix? Can the builtin command also be used for the IW?

@rebornix
Copy link
Member

@roblourens when removing the interrupt button for notebooks, I found that we have a different set of interrupt commands for IW, and their impl is slightly different, thus I leave them there. We should take some efforts to remove it.

@rebornix rebornix assigned amunger and unassigned DonJayamanne Nov 14, 2022
@amunger amunger modified the milestones: November 2022, December 2022 Nov 28, 2022
@roblourens roblourens removed their assignment Dec 11, 2022
@amunger amunger modified the milestones: January 2023, February 2023 Jan 23, 2023
@amunger amunger modified the milestones: February 2023, March 2023 Feb 21, 2023
@amunger amunger modified the milestones: March 2023, April 2023 Mar 22, 2023
@amunger
Copy link
Contributor

amunger commented Apr 27, 2023

easier repro to verify:

  1. create an Interactive window
  2. ensure there is no interrupt button in the toolbar while nothing is running
  3. run some interruptable code
import time
for i in range(20):
    time.sleep(.5)
    print(i)
  1. ensure interrupt button appears in the toolbar and can be used to interrupt execution

@rebornix rebornix added the verified Verification succeeded label Apr 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug notebook-workflow Issues that interrupt expected or desirable behavior papercut 🩸 Something affecting the productivity of the team upstream-vscode Blocked on upstream VS code verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants