Residue rotation and UA translation axes for terminal residues - #416
Open
ioanaapapa wants to merge 24 commits into
Open
Residue rotation and UA translation axes for terminal residues#416ioanaapapa wants to merge 24 commits into
ioanaapapa wants to merge 24 commits into
Conversation
…d atoms + average of all other heavy atoms in resid
harryswift01
requested changes
Sep 2, 2026
harryswift01
left a comment
Member
There was a problem hiding this comment.
Thank you for sending these changes through! I think this is a really good improvement, and it’s great to see the terminal residue handling becoming more robust and independent of atom indexing. I’ve just left a few comments around some small technical and maintainability points. If you’re able to have a look through them and let me know what you think, that would be great. Overall, I think this is looking really good and is very close to being ready to go in!
Collaborator
Author
|
@harryswift01 Thank you, Harry! I will make the changes. |
…ed from get_residue_axes and get_UA_axes
…ing print statement
Collaborator
Author
|
@harryswift01 I made the changes. Hope it's okay now. I've also re-run the average position calculations with the correct average position and the result was the same as the table above. |
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
Terminal residue axes are no longer dependent on atom indices. The edge atom (i.e. heavy atom bonded to neighbour residue) is chosen alongside the average position of heavy atoms bonded to the edge atom and average position of all other heavy in the residue as the three points used for determining the axes. The centre of rotation (at res level)/translation (at UA level) and axes are then computed from the three points similar to the case of non-terminal residues.
Changes
Terminal residue treatment was changed
Edge cases have been introduced
Tests for terminal residues and edge cases
Documentation
get_residue_axes and get_UA_axes structure change
Impact