Merge pull request #710 from RosettaCommons/rfalford12/cleanup_meminfo_and_addmemb
Updating Documentation & Organization of AddMembraneMover and MembraneInfo
Its been about a year since I've updated the in-code documentation (@brief, @detials, class description) for AddMembraneMover and MembraneInfo but a lot has changed. Its also becoming harder to figure out where to add things in terms of (1) where data gets stored and (2) at what step to initialize new data in AddMembraneMover.
In this pull request, I updated the documentation for both classes. I also reorganized methods so adding new extensions to these classes is easier.
I've also updated some logic in AddMembraneMover to fail if none of the current edge cases cover an input PDB which is more informative to the user
Integration test Changes:
Code duplication: Catches consistent initialization between constructors (this is good duplication!) and mp_relax_w_ligand is cosmetic