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.

Sign up for GitHub

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

Jump to bottom

Reports of large memory usage for the host process. #124

Closed
wlandau-lilly opened this issue Oct 31, 2017 · 3 comments
Closed

Reports of large memory usage for the host process. #124

wlandau-lilly opened this issue Oct 31, 2017 · 3 comments
Labels
topic: performance

Comments

@wlandau-lilly
Copy link
Collaborator

Continuing from here in #115. @kendonB, about how much memory does the host process end up consuming? What does that look like in comparison to the size of the .drake cache?.
wlandau-lilly added a commit that referenced this issue Oct 31, 2017
@wlandau-lilly
Address #124
1e5daed 
build_distributed() now returns invisible()
@wlandau-lilly
Copy link
Collaborator Author

wlandau-lilly commented Oct 31, 2017 •
edited
Loading

@kendonB, I think 1e5daed fixes the memory issues. Would you try again?

I am confident that the problem was that build_distributed() was returning the whole runtime configuration list (config in many places in the code).

@kendonB
Copy link
Contributor

kendonB commented Oct 31, 2017

Seems to have worked perfectly!!

@wlandau-lilly
Copy link
Collaborator Author

That's a relief!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
topic: performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kendonB @wlandau-lilly