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 leak in diffusive_size_factor_dns #835

Open
jgostick opened this issue May 16, 2023 Discussed in #834 · 2 comments
Open

Memory leak in diffusive_size_factor_dns #835

jgostick opened this issue May 16, 2023 Discussed in #834 · 2 comments
Assignees

Comments

@jgostick
Copy link
Member

Discussed in #834

Originally posted by oscarhx May 16, 2023
Hello Profs.
As stated in the title, the 64G memory goes out when the number of pore pairs reaching several thousands, far behind the number of pore pairs and conductivity required for AI trainning.

@jgostick
Copy link
Member Author

Thanks for reporting this. It's a new function so was bound to have a few rough edges.

@jgostick
Copy link
Member Author

I think this was coming from openpnm which was used to do the finite difference calculations in the backend. I have pushed a fix for this and it's on pypi. I wonder if you're able to check now and see if the problem is still there?

@ma-sadeghi ma-sadeghi changed the title Memory leak in diffusive_size_factor_DNS Memory leak in diffusive_size_factor_dns Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants