Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

High CPU usage in diffs #443

Closed
borekb opened this issue Oct 8, 2018 · 9 comments
Closed

High CPU usage in diffs #443

borekb opened this issue Oct 8, 2018 · 9 comments

Comments

@borekb
Copy link

borekb commented Oct 8, 2018

Hi, I'm cross-posting an issue from microsoft/vscode#57697 (comment) which hints at high CPU usage cause by this extension in case of diffing larger files. Props to @alexandrudima for finding it.

@KnisterPeter
Copy link
Owner

Thanks, I'll have a look at this.

@KCraig-StepStone
Copy link

KCraig-StepStone commented Nov 6, 2018

Have to disable this extension today as it causes my system to lock up with high CPU usage when opening files with more than 4000 lines. Possibly smaller files but that is where I really noticed it since it would prevent me from even moving my mouse around. Through testing I found it is only this plugin that causes this issue.

@Jonnokc
Copy link

Jonnokc commented Nov 16, 2018

+1

@danguilherme
Copy link

I got the same error in VS Code as pointed in #481.
It happened when diffing a yarn.lock file (13k+ lines).

Here's the CPU profile provided by Code, if it helps: KnisterPeter.vscode-github-unresponsive.cpuprofile.txt

@KnisterPeter
Copy link
Owner

@danguilherme Thanks for reporting. Good catch that it happens with large files. It may be related to the LinkProvider and HoverProvider.

I'll look into this when I find some time.

@AnalogMemory
Copy link

I've been encountering this issues as well the past couple days

@KnisterPeter
Copy link
Owner

I disabled issue scanning for now. This should reduce cpu load.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants