Skip to content

CellListsPotential should not contain CellLists as a member #16

Description

@spraharsh

the Cell Lists generated for a particular set of coordinates are stored by the potential itself when using cell lists.
This breaks encapsulation because the Cell Lists are defined given a particular set of coordinates, and the coordinates should not be tied to the potential.
This creates issues while trying to implement energy change calculations, for example, where we have to assume that the cell lists have been calculated correctly for the given coordinates when we want to avoid recalculating them for performance reasons

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions