porespy icon indicating copy to clipboard operation
porespy copied to clipboard

Published 20 hours ago • verified publisher icon PMEAL

Memory leak in `diffusive_size_factor_dns`

Open jgostick opened this issue 1 year ago • 2 comments

Discussed in https://github.com/PMEAL/porespy/discussions/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 avatar May 16 '23 12:05 jgostick

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

jgostick avatar May 16 '23 12:05 jgostick

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?

jgostick avatar Sep 14 '23 14:09 jgostick