-
Notifications
You must be signed in to change notification settings - Fork 583
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
Extension causes high cpu load #841
Comments
I have been having problems with this as well and I was able to capture a CPU profile while I had 100% CPU from this extension. I was not even using the extension at all, so it was running in the background and looks like it was stuck in garbage collection. The profile was for about 10-20 seconds. |
Same issue. I've attached the automatically generated profile. GitHub.vscode-pull-request-github-unresponsive.cpuprofile.txt |
Ditto ☝️.
|
Same here on Windows 10 Version: 1.62.3 (system setup) GitHub.vscode-pull-request-github-unresponsive.cpuprofile.txt |
awgh |
Same issue here on macOS: System detailsVersion: 1.63.2 Gist - https://gist.github.com/DillonB07/5ae1a3259f1d789d08469f33db39be93 |
I've started looking through several of the profiles in this issue (thanks everyone who added one!). The high CPU seems to occur when several extensions are activating at once. ritwickdey.liveserver in particular seems to coincide with the high CPU, so folks who have that installed might try disabling it. Is everyone seeing this high CPU associated with GitHub Pull Requests and issues shortly after starting or reloading VS Code? |
Yes. I do have the liveserver extension installed but I'll try disabling that and see how it works. I am seeing the issue after starting/reloading VSC though. |
However, disabling the liveserver extension didn't seem to help. CPU Log - https://gist.github.com/DillonB07/72bef869458babec53e19efd0127373b |
Thanks for trying that @DillonB07 and for the new profile. After further digging, it looks like one of the modules we include may be slow (https://www.npmjs.com/package/vscode-extension-telemetry). I'm following up with the owners to see if that's the case and if it can be improved. |
Module has been updated, and I've pulled the update in to this extension. The pre-release build will have the fix tomorrow (the version number will be 0.35.20211228*). If folks in this thread could try it out after that build is ready tomorrow and comment your results it would help confirm whether we've tracked the perf issue down or if the issue should be reopened for further investigation. |
Once I get confirmation that the perf issue is fixed from someone in this thread, I'll release 0.34.3 with the change! |
I haven't got the issue anymore. Seems like it's fixed. Thanks! |
Thanks @DillonB07, I'll release 0.34.3 with the fix. |
Performance
vscode-pull-request-github
0.3.2
Darwin x64 17.7.0
1.31.0-insider
GitHub.vscode-pull-request-github-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load
The text was updated successfully, but these errors were encountered: