Issue 63- Conformational Entropy Calculation #72
Merged
ioanaapapa merged 2 commits intomainfrom Mar 21, 2025
Merged
Conversation
…get_dihedrals and conformational_entropy functions so that MDAnalysis gives dihedrals only containing heavy atoms. Values obtained for aliphatic residue match the old code now. Some further investigation for aromatic residues is still required to confirm results obtained using this implementation.
…for non-hydrogen atoms within the function is no longer necessary
jimboid
approved these changes
Mar 21, 2025
Member
jimboid
left a comment
There was a problem hiding this comment.
Code looks like it is matching the explainer in the PR. Very thorough analysis supporting this merger.
Member
|
I agree with James. The explanation makes sense and the code looks like what I would expect. |
harryswift01
approved these changes
Mar 21, 2025
Member
harryswift01
left a comment
There was a problem hiding this comment.
This looks like a very clean implementaion, thank you for going through so much detail in the PR comments its very nice to see, I'm happy to approve this.
skfegan
approved these changes
Mar 21, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
This PR addresses issues related to the calculation of the conformational entropy at the united atom and residue levels and resolves the differences between results obtained using the current implementation of the theory at the united atom level and the implementation from the previous release. Differences for each type of residue, prior the commit, with terminal residues being isolated, are shown in the figure below. This PR closes #63

Changes
Heavy atoms universe :
per heavy atom in residue - hence, all dihedrals that the heavy atoms were involved in were found and taken into account.
Impact
Possible future considerations