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

memory limit #18

Open
jorgens opened this issue Jul 25, 2022 · 3 comments
Open

memory limit #18

jorgens opened this issue Jul 25, 2022 · 3 comments

Comments

@jorgens
Copy link

jorgens commented Jul 25, 2022

I am reaching the preset memory limit and cannot seem to find a way to add more memory?

GAPError: Error, reached the pre-set memory limit
(change it with the -o command line option)

Any ideas?

@embray
Copy link
Owner

embray commented Jul 25, 2022

Hi @jorgens thanks for your interest in gappy. Unfortunately I haven't had the availability to work in it in some time though I would really love to get a final "1.0" release out sometime...

This is an interesting question--the memory limit should definitely be more easily customizable but at the moment it's not really. This is how the default memory limits are calculated:

cpdef get_gap_memory_pool_size(unit='m'):

You could try changing it there and re-compiling from source.

@jorgens
Copy link
Author

jorgens commented Jul 26, 2022

Hi @embray, thanks for your reply and no worries! I am just really happy I found your Python GAP integration. It has been extremely helpful in my project! I will have a look at the function you pointed to.

@jorgens jorgens closed this as completed Jul 28, 2022
@embray
Copy link
Owner

embray commented Aug 26, 2022

Thanks for the feedback.. I'm going re-open this as a reminder that the memory pool size should be more easily configurable.

@embray embray reopened this Aug 26, 2022
michaelontiveros added a commit to michaelontiveros/gappy that referenced this issue May 4, 2024
michaelontiveros added a commit to michaelontiveros/gappy that referenced this issue May 4, 2024
This reverts commit cacb42b.
michaelontiveros added a commit to michaelontiveros/gappy that referenced this issue May 4, 2024
This reverts commit 5f3e34a.
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

2 participants