mosdef-hub / mbuild

A hierarchical, component based molecule builder
https://mbuild.mosdef.org
Other
171 stars 80 forks source link

Extensive memory usage for fill box #1139

Closed chrisiacovella closed 1 year ago

chrisiacovella commented 1 year ago

Bug summary

When calling fill_box, memory usage will grow rapidly with system size. this is related to prior issues e.g., #1107 related to how large compounds/systems are constructed. The main issue appears to be in the loader for xyz files which was not updated in prior PRs that switched to adding lists of compounds, instead of compounds one at a time.