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

Deployment spends a lot of time in the path() function. #12

Open
benjamin-heasly opened this issue Sep 13, 2016 · 2 comments
Open

Deployment spends a lot of time in the path() function. #12

benjamin-heasly opened this issue Sep 13, 2016 · 2 comments
Milestone

Comments

@benjamin-heasly
Copy link
Collaborator

I noticed that deploying toolboxes can be slow. Even when the toolboxes are already up to date, adding them to the path can take a while.

I checked with the Matlab profiler on a usage that's typical for me: tbUse({'VirtualWorldColorConstancy', 'isetbio'}). About 85% of the time is spent on 16 calls to Matlab's path() function.

I'm not sure why path() is so slow.

Whatever the reason, we might be able to speed up deployments if we collect path entries during deployment, then apply them all in a single call to path().

@benjamin-heasly
Copy link
Collaborator Author

Here is a profiler screenshot.
tbtb-path-is-slow

@DavidBrainard
Copy link
Collaborator

Another thing that is slow is the 'reset','no-matlab' option.

@benjamin-heasly benjamin-heasly modified the milestone: v0.3 Dec 20, 2016
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

2 participants