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

Slight but noticeable keyboard lag with Gitlens #178

Closed
carlca opened this issue Oct 23, 2017 · 13 comments
Closed

Slight but noticeable keyboard lag with Gitlens #178

carlca opened this issue Oct 23, 2017 · 13 comments
Assignees
Labels
bug Something isn't working

Comments

@carlca
Copy link

carlca commented Oct 23, 2017

  • GitLens Version: 5.7.1
  • VSCode Version: 1.18.0 Insider
  • OS Version: macOS 10.13.0 High Sierra

Other extensions loaded:

PeterJausovec.vscode-docker
RomanPeshkov.go-test-outline
Wosi.omnipascal
be5invis.vscode-custom-css
christian-kohler.path-intellisense
codezombiech.gitignore
deerawan.vscode-dash
donjayamanne.githistory
donjayamanne.python
eamodio.gitlens
eg2.vscode-npm-script
emmanuelbeziat.vscode-great-icons
foxundermoon.shell-format
hars.CppSnippets
hnw.vscode-auto-open-markdown-preview
Kiteco.kite
lukehoban.Go
magicstack.MagicPython
ms-vscode.azure-account
ms-vscode.cpptools
ms-vscode.csharp
nisimiko.theme-whitehat
rogalmic.bash-debug
rubbersheep.gi

Steps to Reproduce:

  1. Open VSCode and attempt to edit a .go file
  2. Keyboard lag is slight but enough to be noticeable. I've tried narrowing down the culprit by disabling my extensions in turn and it seems to be Gitlens that is causing the issue.
@egamma
Copy link

egamma commented Oct 23, 2017

@eamodio we now support to profile the extension host pls see https://github.com/Microsoft/vscode/wiki/Performance-Issues#creating-performance-profiles, that might be useful to do in such a case.

@eamodio eamodio self-assigned this Oct 23, 2017
@eamodio eamodio added this to the Soon™ milestone Oct 23, 2017
@eamodio eamodio added the bug Something isn't working label Oct 23, 2017
@eamodio
Copy link
Member

eamodio commented Oct 23, 2017

I'll take a look into this. In the meantime, can you help me determine if the issue goes away for you if you systematically disable some GitLens features?

"gitlens.blame.line.enabled": false
"gitlens.codeLens.enabled": false
"gitlens.gitExplorer.enabled": false
"gitlens.statusBar.enabled": false

If turning off any (or all) of those help, it will also help me pinpoint the issue.

@egamma Yeah -- good point. The profiler was definitely my friend in helping me greatly improve the performance of the full-file blame annotations :)

@carlca
Copy link
Author

carlca commented Oct 23, 2017

I'll try that in an hour or so. I have to cook some food and eat first. Thanks to you and Erich for the replies!

@carlca
Copy link
Author

carlca commented Oct 23, 2017

It's difficult to be certain because the differences in response time are barely perceptible but I thought there was a slight speed up when I changed
"gitlens.codeLens.enabled":
to
false
I hope that helps!

@eamodio
Copy link
Member

eamodio commented Oct 23, 2017

Thank you! I'll check it out

@eamodio
Copy link
Member

eamodio commented Oct 25, 2017

@carlca can you try out the latest GitLens v6 alpha? I've refactored and reduce a bunch of overhead so hopefully it will resolve the lag you are seeing.

Please follow these instructions to install a pre-release. Let me know if you experience any issues -- thanks!

@carlca
Copy link
Author

carlca commented Oct 25, 2017

Something I forgot to mention (sorry!) is that I have been starting VSCode insiders with the arguments --args --disable-gpu . That was because otherwise, the built-in terminal window was not displaying properly. I've just tried it without those arguments and it does seem to be improved. I don't know at the moment if that can be attributed to the new alpha version, though.

@eamodio
Copy link
Member

eamodio commented Oct 25, 2017

Are you still seeing a difference between with and without GitLens? With the new code, there will be some impact (mostly only on the first character typed into a file without any other edits -- but even so it should be very small) it should be extremely small. From profiling I don't even see any GitLens stuff anywhere near the top of the profile.

If you are still seeing issues -- can you try disabling all other extensions and capture a snapshot of the issue, following the steps here: https://github.com/Microsoft/vscode/wiki/Performance-Issues#creating-performance-profiles?

@eamodio eamodio added the pending-release Resolved but not yet released to the stable edition label Oct 25, 2017
@carlca
Copy link
Author

carlca commented Oct 25, 2017 via email

@carlca
Copy link
Author

carlca commented Oct 26, 2017

There's a lot of different tasks on that page https://github.com/Microsoft/vscode/wiki/Performance-Issues#creating-performance-profiles .
Which tasks, in particular, should I concentrate on (don't say all of them!!)

@eamodio
Copy link
Member

eamodio commented Oct 26, 2017

@eamodio
Copy link
Member

eamodio commented Nov 9, 2017

I'm hoping GitLens 6 (out now) resolves this. Please re-open if not.

@eamodio eamodio closed this as completed Nov 9, 2017
@eamodio eamodio removed the pending-release Resolved but not yet released to the stable edition label Nov 9, 2017
@eamodio eamodio removed this from the Soon™ milestone Jul 19, 2019
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants