-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create MolecularSystem class #49
Comments
Hey, I think that should go into a separate Topology class - as it is done for the rigid bodies already. The rigid body topology needs massive cleanup, but i think keeping the topology separate is the better idea. Then the system class is responsible for the algorithms and the Topology for the data structures. I also have a topology for crystal systems. |
victor, can you be more specific? your non-existent documentation makes On 19/03/13 17:00, Kyle Sutherland-Cash wrote:
|
I on purpose wasn't more specific regarding my example. I think the current aatopology is not very well structured. What I wanted to say is that I think we should do more or less exactly what Kyle suggested, but in a separate topology class rather than the system class. Then we have the data structures (Molecules, Atoms, Bonds, ...) separated from the choice of algorithms (system class). I also prefer the idea to have it separated into smaller sub components. We should sit have a chat on which parts might be similar in molecular systems, angle axis systems and crystal systems to design a more uniform interface - but by no means keep the existing one. One thing which also should go into the topology is a reference to the current boundary conditions used, e.g. cubic pbc, open system, triclinic, ... |
I am planning to create a MolecularSystem class inheriting from BaseSystem. Its purpose will be to:
The two important classes for this will be MolecularSystem and Atom, which are outlined below:
The text was updated successfully, but these errors were encountered: