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 usage increases with repeated runs #2

Open
JeffreyMinucci opened this issue Jul 23, 2024 · 2 comments
Open

Memory usage increases with repeated runs #2

JeffreyMinucci opened this issue Jul 23, 2024 · 2 comments
Assignees
Labels
BeePop+ Issues related to the underlying BeePop+ model bug Something isn't working

Comments

@JeffreyMinucci
Copy link
Collaborator

When doing repeated runs, memory use by the thread slowly increases, suggesting a memory leak. Currently if performing many thousands of runs it is recommended to restart the python process periodically to avoid running out of memory.

Most likely, BeePop+ needs to implement a function to properly free memory allocated to objects passed from the wrapper to the c++ interface.

@JeffreyMinucci JeffreyMinucci added the bug Something isn't working label Jul 23, 2024
@JeffreyMinucci JeffreyMinucci self-assigned this Jul 23, 2024
@JeffreyMinucci JeffreyMinucci added the BeePop+ Issues related to the underlying BeePop+ model label Aug 22, 2024
@RobertCurry
Copy link

With each run, VPopLib creates a list of errors and a list of information. I saw this memory growth on my system until I inserted ClearErrorList() and ClearInfoList() at the end of each run. It occurs to me that I could add a function to enable/disable creating these lists - possibly improving runtime.

@RobertCurry
Copy link

Added two interface functions:

EnableErrorReporting(int enable);  // 1 = true, 0 = false
EnableInfoReporting(int enable);  // 1 = true, 0 = false

The default is to enable the creation of Error lists and Information Lists which, as always, could be retrieved by the using app to inspect the data. If set to false, no list is created. This has been updated in my Github repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BeePop+ Issues related to the underlying BeePop+ model bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants