Correct RMSD documentation and remove RMSD weight setting. #417
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of work
Updated and corrected RMSD documentation, the equations in the current version are not correct, see the new version below. Also updated the ROG documentation.
I've also removed the weight setting from the RMSD calculation. Here are two reason for why I've decided that the weight setting should be remove from the RMSD calculation.
I've also changed the result name from rmsd_total to rmsd_all to make it clear that rmsd_all is not the sum of the atom rmsds.
Fixes
Closes #381
To test
The weight setting on the RMSD job should be removed. Run RMSD and check the results, this should be the same as the results calculated on the protos branch with equal weight setting.