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

compute_irmsd_fast() only works on two chains #76

Open
LilySnow opened this issue Jul 11, 2022 · 6 comments
Open

compute_irmsd_fast() only works on two chains #76

LilySnow opened this issue Jul 11, 2022 · 6 comments
Assignees

Comments

@LilySnow
Copy link
Collaborator

LilySnow commented Jul 11, 2022

>>> ref = "ref.pdb"
>>> model = "model.pdb"
>>> sim = StructureSimilarity(model,ref)
>>> irmsd_fast = sim.compute_irmsd_fast()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/lixue/tools/anaconda3/lib/python3.7/site-packages/pdb2sql/StructureSimilarity.py", line 277, in compute_irmsd_fast
    resData = self.compute_izone(cutoff, save_file=False)
  File "/home/lixue/tools/anaconda3/lib/python3.7/site-packages/pdb2sql/StructureSimilarity.py", line 328, in compute_izone
    'exactly two chains are needed for irmsd calculation but we found %d' % len(chains), chains)
ValueError: ('exactly two chains are needed for irmsd calculation but we found 5', ['A', 'B', 'C', 'D', 'E'])

Archive.zip

@github-actions
Copy link

This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Aug 12, 2022
@github-actions
Copy link

This issue was closed because it has been inactive for 7 days since being marked as stale.

@LilySnow
Copy link
Collaborator Author

LilySnow commented Aug 23, 2022

To solve this issue, I am planning to add an option chain_pairs = ['D:E', 'AB:C'] to compute_izone(). This way, pdb2sql knows which interfaces are of interest when calculating i-RMSD for a multiple-chain complex.

@NicoRenaud
Copy link
Member

i've suggested to make a few change to Li's work see #80. This basically allows to use get_contact_atoms using multiple chains as :

db.get_contact_atoms(chain1='AB', chain2='C')

@github-actions
Copy link

This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Sep 26, 2022
@NicoRenaud
Copy link
Member

Related to PR #80 and #79

@github-actions github-actions bot removed the stale label Oct 5, 2022
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

No branches or pull requests

4 participants