7(+) seconds of unresponsiveness warning after update VS to 15.6

microsoft / VS-PPT

Productivity Power Tools - a set of Visual Studio extensions improving developer productivity.

http://aka.ms/extendvs

Other

887 stars 146 forks source link

7(+) seconds of unresponsiveness warning after update VS to 15.6 #173

Open nphmuller opened 6 years ago

nphmuller commented 6 years ago

In Visual Studio 15.6 the unresponsiveness check has been updated to show to part of VS (or extension) that caused it.

Now I keep getting the message Extension 'Productivity Power Tools 2017 15.0.4' likely caused 7 seconds of unresponsiveness. Disabling it may improve your experience.. Sometimes with 7 seconds, sometimes a bit more.

Is there something that can be changed to keep this unresponsiveness from happening?

justcla commented 6 years ago

Yes, you can uninstall it. That will stop the warnings coming through. :-)

Now I know that might sound silly. But the PPT extension itself is nothing more than an extension installer. Its sole job is to run at startup and check which extensions from the PPT list are missing, and go download and install them. Once all PPT extensions are installed, the PPT installer has no useful purpose and really should just be uninstalled (or disabled).

That should keep the unresponsiveness should happening.

nphmuller commented 6 years ago

@justcla That's fine for me, but isn't there a way to stop the unresponsiveness from happening at all? It probably sounds easier then it is, but isn't it feasible to do the checks in the background, without blocking anything?

justcla commented 6 years ago

Good call, Nick. I've started an internal thread to see if we can push the loading to the background.

It currently loads on startup and checks which extensions are installed (on the main thread), then awaits a background task to download and install missing extensions. We could potentially:

load the package as an AsyncPackage that runs on the background (to delay package loading), and/or
check existing extensions on a background thread (move that part off the main UI thread).

Either of those solutions should reduce UI delays on startup.

@prnadago, what are your thoughts?

giggio commented 6 years ago

I also got the warning, and it was on the options page:

prnadago commented 6 years ago

I just updated the extension on marketplace with a quick patch that runs the task to search for missing extensions in the background. Check it out and let me know if it removes the warning for you! https://marketplace.visualstudio.com/items?itemName=VisualStudioProductTeam.ProductivityPowerPack2017

edit: This patch is only for 'Productivity Power Tools 2017 15.0.4', and not the 'Tools Options Page'. We're still looking into that.