Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for issue #92 - Adding proline #94

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

tony-res
Copy link

Fix for PDB files that have IMGT numbering. Issue #92

The original code ends up adding/removing residues when it runs into a residue number of "112A" or "34A".

To fix, I just renumber the residues whenever the label changes. So they'll be just 1 to N residues.

It would be better if we first run the PDB file through some tools that fixes the PDB and renumbers it with only integer values. However, I didn't want to change too much of the code just in case other things might break.

tony-res and others added 2 commits February 26, 2024 12:06

Verified

This commit was signed with the committer’s verified signature.
Xtansia Thomas Farr
Fix for PDB files that have IMGT numbering. The original code ends up adding/removing residues when it runs into a residue number of "112A" or "34A". To fix, I just renumber the residues whenever the label changes. So they'll be just 1 to N residues.

Verified

This commit was signed with the committer’s verified signature.
Xtansia Thomas Farr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant