You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When trying to add a list of compound to a container, the add method can take quite long (in the scales of 10,000 atoms). This can due to the recursively nature of the method (each particle is added separately). The culprit seems to be the check at the very end of this method, where we are checking if the new compound (with added molecules) box is bigger than it's bounding box. A solution for this would be have a way to skip this check for the individual add and only have one check at the very end. Optimizing the Compound.get_boundingbox() would be beneficial.
The text was updated successfully, but these errors were encountered:
Bug summary
When trying to add a list of compound to a container, the
add
method can take quite long (in the scales of 10,000 atoms). This can due to the recursively nature of the method (each particle is added separately). The culprit seems to be the check at the very end of this method, where we are checking if the new compound (with added molecules) box is bigger than it's bounding box. A solution for this would be have a way to skip this check for the individual add and only have one check at the very end. Optimizing theCompound.get_boundingbox()
would be beneficial.The text was updated successfully, but these errors were encountered: