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

add a memory leak test #38

Open
1fish2 opened this issue Mar 5, 2019 · 2 comments
Open

add a memory leak test #38

1fish2 opened this issue Mar 5, 2019 · 2 comments

Comments

@1fish2
Copy link
Contributor

1fish2 commented Mar 5, 2019

... based on wcEcoli/wholecell/tests/utils/test_memory_debug.py.

@prismofeverything
Copy link
Member

I checked out the test in WCM and found it a little heavy to add, hopefully this is sufficient (it catches the issues we had before): https://github.com/CovertLab/arrow/blob/master/arrow/test/test_arrow.py#L179-L205

@1fish2
Copy link
Contributor Author

1fish2 commented Dec 22, 2021

test_memory is useful but it fails when run by itself and sometimes when run within test_arrow.

It's trying to catch native malloc leaks using psutil.Process(os.getpid()).memory_info().rss. RSS measures OS-level non-swapped physical memory allocated in whole pages. Is there a better measure of used objects on the heap?

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