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 the align_mask bug when calculating the prmsd_loss #49

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yaowensnf
Copy link

The shape of atom-wise align_mask is (batch, seq_len*4), need to multiply 4 to the indexes when masking out residues.

@yaowensnf yaowensnf closed this Jul 6, 2023
@yaowensnf yaowensnf reopened this Jul 6, 2023
@yaowensnf yaowensnf marked this pull request as draft July 6, 2023 03:12
@yaowensnf yaowensnf marked this pull request as ready for review July 6, 2023 03:13
@qiz218591
Copy link

Hi, I am getting error, this
ValueError Traceback (most recent call last)
c:\Users\s4735779\Documents\GitHub\IgFold\IgFold.ipynb Cell 6 line 1
10 igfold = IgFoldRunner(num_models=num_models)
12 pred_pdb = os.path.join(pred_dir, f"{name}.pdb")
---> 13 pred = igfold.fold(
14 pred_pdb,
15 sequences=sequences,
16 do_refine=do_refine,
17 use_openmm=True,
18 do_renum=do_renum,
19 )
20 show_pdb(pred_pdb, len(sequences), bb_sticks=False, sc_sticks=True, color="rainbow")

File c:\Users\s4735779\Documents\GitHub\IgFold\igfold\IgFoldRunner.py:106, in IgFoldRunner.fold(self, pdb_file, fasta_file, sequences, template_pdb, ignore_cdrs, ignore_chain, skip_pdb, do_refine, use_openmm, do_renum, truncate_sequences)
91 """
92 Predict antibody structure with IgFold.
93
(...)
103 :param truncate_sequences: Truncate sequences with AbNumber.
104 """
105 start_time = time()
--> 106 model_out = fold(
107 self.antiberty,
108 self.models,
...
(...)
228 do_renum=do_renum,
229 )

ValueError: min() arg is an empty sequence
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...

why?? please solve

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.

2 participants