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

Checks for memory usage ahead of run #115

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Checks for memory usage ahead of run #115

wants to merge 2 commits into from

Conversation

kwhitehall
Copy link
Owner

@marcuseng and @diegovb, please have a review. This may be needed to drive your mem issues solutions. Note that right now, the min memory required is a raw heuristic calculated by assuming at least two frames of any dataset used will be required in mem simultaneously.
@cgoodale Thoughts?

@diegovb
Copy link
Contributor

diegovb commented Mar 8, 2016

I think the code as is will not work with the approach we pivoted to, but we can probably repurpose the code as a way to limit the number of files we process at one time.

@kwhitehall
Copy link
Owner Author

@diegovb can you please provide more details on your comment.

@diegovb
Copy link
Contributor

diegovb commented Mar 9, 2016

I am not sure of the details right now but once we flesh out the design and start developing we'll probably merge this into our private repos, depending no how the design ends up.

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

Successfully merging this pull request may close these issues.

2 participants