wackywendell / tess

Python bindings to the voro++ library.
Other
36 stars 10 forks source link

Reduce memory usage #8

Closed wackywendell closed 5 years ago

wackywendell commented 5 years ago

See #7 and before that #4.

Thanks to @pmrv for pointing this out!

wackywendell commented 5 years ago

Following up, I can see the init_mem(8) in the voro++ source in cmd_line.cc, although I have to admit I still don't entirely understand what this variable means - what even are its units? How many blocks are there? I'm not sure. But my guess is that reducing the memory usage like this means at worst the system will have to reallocate more later, which is much better than allocating what looks like it might be N^2 memory for a system of size N. Thanks @prmv!