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

hhmakemodel.py : list index out of range #135

Open
AntoineLabeeuw opened this issue May 15, 2019 · 2 comments
Open

hhmakemodel.py : list index out of range #135

AntoineLabeeuw opened this issue May 15, 2019 · 2 comments

Comments

@AntoineLabeeuw
Copy link

Hi, me and @diegozea are using HH-suite3, v 3.2.0, commit d335e62 on Ubuntu 16.04.5 LTS.

When using hhmakemodel.py with a .hhr coming from hhsearch, it throws an error :

python3 hhmakemodel.py file.hhr <path_to_pdb> out.pir ./

  File "/home/labeeuw/Documents/hh-suite/build/scripts/hhmakemodel.py", line 2400, in <module>
    main()
  File "/home/labeeuw/Documents/hh-suite/build/scripts/hhmakemodel.py", line 2385, in main
    glob_seq = derive_global_seq(processed_query_grid, query_name, query_chain) # derive query sequence
  File "/home/labeeuw/Documents/hh-suite/build/scripts/hhmakemodel.py", line 663, in derive_global_seq
    global_seq.append(processed_query_grid.get_col_residue(col))
  File "/home/labeeuw/Documents/hh-suite/build/scripts/hhmakemodel.py", line 287, in get_col_residue
    if not self.is_empty(row, col):
  File "/home/labeeuw/Documents/hh-suite/build/scripts/hhmakemodel.py", line 197, in is_empty
    return self._cells[row][col] == EMPTY
IndexError: list index out of range

Steps to Reproduce (for bugs)

Attached to this are the files i've got from hhblits and hhsearch
Everything was done with the latest databases available
O00159.zip

@milot-mirdita
Copy link
Member

The structure modeling part of the hh-suite (hhpred and hhmakemodel) is currently unsupported due to a lack of funding and manpower. We would be happy to merge any pull requests that fix it though.

With that said, could you try to explicitly specify which hits it should try to use for modeling (with the -m parameter).

@AntoineLabeeuw
Copy link
Author

Hi
I managed to fix it, the error was that the rows were not of the same sizes, resulting in a list index out of range.
the pull request #154 fixes it, as well as some other errors i encountered, such as the modelling with modeller that didn't work because of the cif chain renaming in the script. the structure modelling part should now work more reliably

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

2 participants