Skip to content

Load time #155

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

Open
gamaanderson opened this issue Jul 7, 2016 · 5 comments
Open

Load time #155

gamaanderson opened this issue Jul 7, 2016 · 5 comments

Comments

@gamaanderson
Copy link
Contributor

as any graphical program artview takes some time to load, but it is kind to annoying me. We should check if there is no unnecessary loading or if there are things we can load just when we need it.

We could add some profiling to generate statistics of which functions are taking to much time. As we work on python it is important to optimise it once in while.

@nguy
Copy link
Owner

nguy commented Jul 7, 2016

I agree completely. We indirectly employ Py-ART lazy dictionary loading, but maybe we could do this more extensively with the internal code.

We should also expand the testing....

@gamaanderson
Copy link
Contributor Author

in my system with empty cache it takes 17s to load pyart and just around 1.5s to the rest of artview. So we can not do much.

from those 17s, around 7s are just to load basemap. What about a lazy loading there @jjhelmus?

@jjhelmus
Copy link
Contributor

@gamaanderson Can you be more specific in what you mean by "loading". Are you looking at the time it takes to import pyart or to generate a figure. The former should not take 17 seconds, that later might if you are using basemap which can be quite slow.

@gamaanderson
Copy link
Contributor Author

I mean import, it does take so much, but for that I must empty the RAM cache. subsequent runs are faster.

@nguy
Copy link
Owner

nguy commented Mar 24, 2017

I did some profiling the other night. The pyart import remains somewhat slow, but I also noted that PyQt was loading very slowly it seemed. I don't have extensive evidence yet, but this is something to keep in mind and test in the future.

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

No branches or pull requests

3 participants