durrantlab / wisp

WISP is a trajectory analysis tool that calculates and visualizes allosteric pathways.
https://durrantlab.github.io/wisp/
Academic Free License v3.0
3 stars 0 forks source link

Scripts running for too long #1

Closed gayatripanda5 closed 3 months ago

gayatripanda5 commented 9 months ago

Dear Developers, I want to determine the allosteric pathway connecting two residues in two separate domains of my system. I had setup a trial job using the files attached, and I ran this jib using 16 processors. The script was running for 3 weeks and then I had to kill it as I couldn't understand why is it taking so long. Kindly help me understand what could be the possible reason behind it and as I want to run it for multiple proteins. Kindly suggest a way to increase the efficiency of the script.

The files used files_wisp.tar.gz

aalexmmaldonado commented 8 months ago

Thank you for bringing this to our attention! We will look into this, but it may take some time.

gayatripanda5 commented 8 months ago

Thanks a lot for your response. I'm really looking forward to its resolution since my work is stuck at this step only. Thanks

gayatripanda5 commented 8 months ago

Dear Developers, Could you please help me in resolving this issue?

aalexmmaldonado commented 8 months ago

Hey @gayatripanda5, I am currently reworking the organization of the code. You can track my progress here: #2. I will start troubleshooting your example once I ensure our original test passes with the new format.

I am hoping to have this resolved in a week or two.

gayatripanda5 commented 8 months ago

Thanks for helping me out. I will follow the attached link.

Saleh-OM4R commented 4 months ago

Hi @aalexmmaldonado , I'm trying the wisp.py code and it is taking a long time to run; it gets stuck exactly at the path calculation step. I tried to increase the number of processors, but still it did not help. Was the issue mentioned above fixed or are there still known issues with the code?
Thanks for the tool, Cheers,

aalexmmaldonado commented 3 months ago

Accidentally closed this.

aalexmmaldonado commented 3 months ago

Hey @gayatripanda5, I have been able to look at this again and discovered a few things.

  1. Your PDB has multiple chains but no chain IDs. WISP would automatically assign everything as chain A, which was corrected in #3.
  2. WISP ran for so long because of a combinatorial explosion in your system. WISP did not have a check or warn the user of this. We implemented an estimation of the number of paths, and if it is above some threshold, we terminate the calculation. You can bump this up manually, but it will run for as long as it did.

I will leave this open for now while I look into it more.