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

performance/optimizations #5

Open
vutran opened this issue Aug 28, 2016 · 11 comments
Open

performance/optimizations #5

vutran opened this issue Aug 28, 2016 · 11 comments

Comments

@vutran
Copy link
Member

vutran commented Aug 28, 2016

TBD.

@vutran vutran changed the title pref/optimizations performance/optimizations Aug 28, 2016
@z3t0
Copy link
Contributor

z3t0 commented Nov 14, 2016

do we have any way to measure profile performance?

@vutran
Copy link
Member Author

vutran commented Nov 14, 2016

Not at the moment but am open for any thoughts/suggestions on how we can do this.

@z3t0
Copy link
Contributor

z3t0 commented Nov 14, 2016

Okay then I'll take a look into this. Also I noticed that it lags when typing is that intentional?

@brpaz
Copy link
Contributor

brpaz commented Nov 14, 2016

Yeah, noticed that lag also.

@vutran
Copy link
Member Author

vutran commented Nov 14, 2016

I believe I had debounced the query function for a few milliseconds a while back. I'm open for any suggestions on perf.

@z3t0
Copy link
Contributor

z3t0 commented Nov 14, 2016

I guess the first step is to figure out some way to quantitatively profile performance.

I'm still trying to figure out how the build process works, for example what is the purpose of webpack?

The app seems to run a lot faster and without type lag when just doing electron . im not sure why?
@brpaz can you try doing the same where you run it off electron directly?

Also I might just be making things up in my confusion so let me know if all of this is just circumstantial

@vutran
Copy link
Member Author

vutran commented Nov 15, 2016

webpack is primarily used for transpile the renderer bundle mainly for ES6 imports + JSX.

(Note: app/renderer/src transpiles to app/renderer/lib/bundle.js which is loaded in app/renderer/index.html.)

The main process is not transpiled as it is using currently available Node features.

Also, there's a 500ms debounce (here) for handling multiple queries. We can definitely lower to something more appropriate.

Are you running in dev or the build?

@z3t0
Copy link
Contributor

z3t0 commented Nov 16, 2016

Also, there's a 500ms debounce (here) for handling multiple queries. We can definitely lower to something more appropriate.

Is the debounce for the query being processed, or typing into the bar itself? Because the lag isn't for the query but rather when typing.

Are you running in dev or the build?

Not too sure, I'm just running electron . --debug in the root of the project?

@vutran
Copy link
Member Author

vutran commented Nov 16, 2016

The denounce is to query. We may want to move that higher to the renderer so it doesn't hit the IPC calls.

@z3t0
Copy link
Contributor

z3t0 commented Nov 16, 2016

The denounce is to query. We may want to move that higher to the renderer so it doesn't hit the IPC calls.

Oh I see, I'm still not sure however why just running electron . --debug does not cause a typing lag?

@mrjohnyhats
Copy link

Maybe this was just fixed since November, but I don't notice any lag at all when I'm using electron ., so if we launch the app for users with that, performance should be fine.

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

No branches or pull requests

4 participants