dext icon indicating copy to clipboard operation
dext copied to clipboard

performance/optimizations

Open vutran opened this issue 9 years ago • 11 comments

TBD.

vutran avatar Aug 28 '16 06:08 vutran

do we have any way to measure profile performance?

z3t0 avatar Nov 14 '16 04:11 z3t0

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

vutran avatar Nov 14 '16 05:11 vutran

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

z3t0 avatar Nov 14 '16 20:11 z3t0

Yeah, noticed that lag also.

brpaz avatar Nov 14 '16 22:11 brpaz

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

vutran avatar Nov 14 '16 23:11 vutran

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

z3t0 avatar Nov 14 '16 23:11 z3t0

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?

vutran avatar Nov 15 '16 03:11 vutran

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?

z3t0 avatar Nov 16 '16 01:11 z3t0

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

vutran avatar Nov 16 '16 01:11 vutran

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?

z3t0 avatar Nov 16 '16 02:11 z3t0

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.

mrjohnyhats avatar Jan 11 '17 05:01 mrjohnyhats