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

Startup now feels slower? #55010

Closed
isidorn opened this issue Jul 25, 2018 · 4 comments
Closed

Startup now feels slower? #55010

isidorn opened this issue Jul 25, 2018 · 4 comments
Assignees
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-rapid-render Splash during startup

Comments

@isidorn
Copy link
Contributor

isidorn commented Jul 25, 2018

Refs: #54937

Not sure if related but creating this issue out of concern that the percieved startup with the new splash screen became worse.
Comparing built vscode the difference is not noticable but I think I notice the difference when running vscode out of source. Also I have a fast machine, we should compare built versions on some dog-slow machine.

Could we do some measuring just that we are sure that we are not slowing things down by going to storage / adding html elements on startup?

@isidorn isidorn added the under-discussion Issue is under discussion for relevance, priority, approach label Jul 25, 2018
@jrieken
Copy link
Member

jrieken commented Jul 25, 2018

During development I have measure between 15 and 20ms, mostly due to loading the vscode-uri node module. I'll add perf marks so that everyone can check - accessing local storage might be the bottleneck for heavily used installations.

@jrieken jrieken added the workbench-rapid-render Splash during startup label Jul 25, 2018
jrieken added a commit that referenced this issue Jul 25, 2018
@jrieken
Copy link
Member

jrieken commented Jul 25, 2018

@isidorn I have pushed a change that adds perf marks. Out of dev or when built run Dev: Startup Performance and see at the bottom of the console something like this 'willShowPartsSplash' and 'didShowPartsSplash'

@isidorn
Copy link
Contributor Author

isidorn commented Jul 25, 2018

It is 23 ms on my mac.
It is 12 ms on my windows
And 10ms on linux. For some reason it is slowest on mac. But should get better I guess if you stop using the module. Though these number are not noticable, so it might be just the percieved performance.

@jrieken
Copy link
Member

jrieken commented Sep 11, 2018

Closing as this wasn't reported anywhere else

@jrieken jrieken closed this as completed Sep 11, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-rapid-render Splash during startup
Projects
None yet
Development

No branches or pull requests

2 participants