Closed eruijsena closed 2 years ago
@eruijsena I agree with all of our changes except for one. In pygromos/files/topology/top.py
you renamed a class so it would have a lower case first letter, when it is convention to have classes have a upper-case first variable letter.
Please fix it and then it should be ok to merge!
Yep, done! And I won't even mention the fact that the two classes above it are lower case! :)
Description
Minor tweaks to go with other pull request (#66) on RE-EDS.
Main change is that consecutive residues with the same name (typically, ions) will be placed in the same force group, which was not the case before.