atom-ink icon indicating copy to clipboard operation
atom-ink copied to clipboard

Improve package load time

Open pfitzseb opened this issue 8 years ago • 8 comments

ink is sitting at ~200ms for me, which seems pretty excessive (considering that atom-ide-ui for example only takes ~50ms).

pfitzseb avatar Jul 24 '17 10:07 pfitzseb

I get even worse! Granted, I have some other large apps open right now so maybe my hard drive was being used at the time. However, literally all the worst times are related to Juno/Julia/Ink... ☹️

capture

ahogen avatar Aug 03 '17 16:08 ahogen

Ouch, that looks horrible.

pfitzseb avatar Aug 03 '17 17:08 pfitzseb

The load time has become absolutely astounding. The Ink load time is more than double the load time of all the other packages I use that take more than 5ms to load combined, and about 1.5 times more than the load time of all the packages I use combined!

screen shot 2018-07-23 at 11 26 00

dpo avatar Jul 23 '18 18:07 dpo

Oh wow, that's really bad. Will look into it soon.

pfitzseb avatar Jul 23 '18 20:07 pfitzseb

Lazy loading FTW: image

I'll release a new version tomorrow. :)

pfitzseb avatar Jul 23 '18 20:07 pfitzseb

Okay, that was a bit too optimistic -- the best I have now which also doesn't break anything is image

In the screenshot in the post above I accidentally broke deserialization of the terminal, plot pane etc. pp., which obviously makes things much faster.

pfitzseb avatar Jul 24 '18 09:07 pfitzseb

For me, ink was 200 ms for the first boot, but next times around 130 ms.

image

I believe we should disable the linter inside ink and instead contribute to linter-julia to improve the speed a little https://atom.io/packages/linter-julia Setting up a language server would be easy, and we can use the same UI that other languages use in Atom, to save some speed instead of creating a separate UI for julia

aminya avatar Nov 13 '19 06:11 aminya

Atom-ide-ui package that provides almost the same features have 1/4 of the load time of atom-ink: image

aminya avatar Nov 24 '19 22:11 aminya