ryu1kn / vscode-partial-diff

Visual Studio Code Extension. Take a diff of 2 parts of text(s)
https://marketplace.visualstudio.com/items?itemName=ryu1kn.partial-diff
MIT License
184 stars 15 forks source link

Extension causes high cpu load #46

Closed samael205 closed 3 years ago

samael205 commented 4 years ago

ryu1kn.partial-diff-unresponsive.cpuprofile.txt

:warning: Make sure to attach this file from your home-directory: :warning:C:\Users\Becario\ryu1kn.partial-diff-unresponsive.cpuprofile.txt

Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

deavial commented 4 years ago

ryu1kn.partial-diff-unresponsive.cpuprofile.txt

OSX 10.13.6 with Node 10.20.1 as well

liboshu commented 4 years ago

ryu1kn.partial-diff-unresponsive.cpuprofile.txt

iskandersierra commented 4 years ago

ryu1kn.partial-diff-unresponsive.cpuprofile.txt

ryu1kn commented 3 years ago

Additional info on https://github.com/ryu1kn/vscode-partial-diff/issues/48

Developer: Show Running Extensions shows how much time running extensions took when they started.

ryu1kn commented 3 years ago

Just took a look at all 4 uploaded cpu profiles. They all seem to be taking time with loading modules. I'm wondering if this is this case

https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load#code-loading

I'm thinking to try bundling js files with webpack or something as suggested.

ryu1kn commented 3 years ago

Bundle js files with webpack and the extension startup times has improved 154 ms to 7.8 ms (10 times run average), 95% reduction 😮

Please let me know if you still see the issue.