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

Terminal performance is down #157432

Closed
abhijit-chikane opened this issue Aug 7, 2022 · 13 comments · Fixed by #159494
Closed

Terminal performance is down #157432

abhijit-chikane opened this issue Aug 7, 2022 · 13 comments · Fixed by #159494
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug perf terminal-shell-integration Shell integration infrastructure, command decorations, etc. verified Verification succeeded

Comments

@abhijit-chikane
Copy link
Contributor

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version:1.70
  • OS Version: windows 10

Steps to Reproduce:

  1. After I upgrade the vscode to 1.70 version it's terminal is seems performing slowly
@vscodenpa
Copy link

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.70.0. Please try upgrading to the latest version and checking whether this issue remains.

Happy Coding!

@meganrogge
Copy link
Contributor

Please see https://github.com/Microsoft/vscode/wiki/Performance-Issues for creating actionable performance problem issues.
/label info-needed terminal

@vscodenpa vscodenpa added info-needed Issue requires more information from poster terminal General terminal issues that don't fall under another label labels Aug 8, 2022
@tinydogio-joshua
Copy link

My team has been seeing a large slow down with the terminal when in a devcontainer's main project directory. It's fast once you leave it.

@tinydogio-joshua
Copy link

tinydogio-joshua commented Aug 9, 2022

I just tried disabling newly enabled shellIntegration setting and it sped back up.

"terminal.integrated.shellIntegration.enabled": false

https://code.visualstudio.com/updates/v1_70#_terminal


Operating System: macOS 12.5
Hardware: MacBook Pro 13-inch, M1, 2020) 16 GB Memory
Visual Studio Code: Version 1.70.0 Universal

@meganrogge meganrogge removed the info-needed Issue requires more information from poster label Aug 9, 2022
@meganrogge meganrogge added confirmation-pending terminal-shell-integration Shell integration infrastructure, command decorations, etc. and removed terminal General terminal issues that don't fall under another label labels Aug 9, 2022
@meganrogge meganrogge added this to the Backlog milestone Aug 9, 2022
@Tyriar
Copy link
Member

Tyriar commented Aug 10, 2022

@abhijitchikane @tinydogio-joshua what shell are you using?

@tinydogio-joshua
Copy link

zsh. My team is all on M1 MacBook Pro's. One person has oh-my-zsh installed. I have rocketship. The rest are stock. This is in a decent sized monorepo in a Node+TypeScript devcontainer. The same behavior does not appear in the stock/empty Node+TypScript devcontainer. I have not tried it locally (outside the container) as we don't work that way.

@Tyriar
Copy link
Member

Tyriar commented Aug 10, 2022

@tinydogio-joshua adding your comment from microsoft/vscode-remote-release#6986 for more context:

@tinydogio-joshua can you be more specific what you mean by "a lot of delays"? Also what shell were you using?

We were seeing 3 to 10 second delays typing in pretty much any command. We're all on M1 MacBook Pro's. zsh shell. I know at least one person has oh-my-zsh installed. I have rocketship but as far as I know the rest of the team hasn't customized their shells. Setting the above setting to false brought the performance back up.

It is in a decent sized monorepo. Doing a stock/empty Node+TypeScript devcontainer does not show the same behavior.

@Tyriar Tyriar modified the milestones: Backlog, August 2022 Aug 10, 2022
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug perf labels Aug 10, 2022
@WindSoilder
Copy link

I get the similar performance issue, using nushell, with MacBook Pro (not M1)

@jakebailey
Copy link
Member

I'm also having bad performance with the new shellIntegration enabled with powershell on Windows 11; it manifests as a huge delay to focus the terminal, or the whole editor freezing and asking if I want to restart it, or, not even being able to launch a new terminal (the pane is blank). Sometimes when it happens, Code.exe is using a large amount of CPU.

The main problem is that it's intermittent and I can't seem go get a profile of it... but I'm trying.

@Tyriar
Copy link
Member

Tyriar commented Aug 29, 2022

I fixed several performance regressions in the last few days: xtermjs/xterm.js#4080, xtermjs/xterm.js#4081, xtermjs/xterm.js#4084. I expect this to be fixed in v1.72 (September)

Tyriar added a commit that referenced this issue Aug 29, 2022
Fixes #159492
Fixes #159493
Fixes #158868
Fixes #158981
Fixes #157432
@aeschli aeschli added the verification-steps-needed Steps to verify are needed for verification label Sep 29, 2022
@mjbvz
Copy link
Collaborator

mjbvz commented Oct 4, 2022

@Tyriar Can you please provides steps to verify this?

@Tyriar
Copy link
Member

Tyriar commented Oct 4, 2022

This is a bit hard to verify, but myself and @jerch have already tested it fairly thoroughly

@Tyriar Tyriar added verified Verification succeeded and removed verification-steps-needed Steps to verify are needed for verification labels Oct 4, 2022
@iansimonson
Copy link

Just commenting to give anecdotal data. In 1.72.0 the example I gave in the duplicate issue #159476 now renders in 12-15 seconds depending on the run for an 11.8 second run with lots of text. Previously it was rendering in 39.7 seconds.

The integrated terminal is definitely usable again, thanks! (at least on my box 😄 )

@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2022
lemanschik pushed a commit to code-oss-dev/code that referenced this issue Nov 25, 2022
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 perf terminal-shell-integration Shell integration infrastructure, command decorations, etc. verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants