TechnionYP5779 / SimuMole

Computer simulation method that displays the physical movements of atoms and molecules.
5 stars 0 forks source link

Fix PDB merging function #86

Closed Nitzan810 closed 5 years ago

Nitzan810 commented 5 years ago

The current PDB merge method we are using (loading two molecules and saving them with the pymol cmd) always results in a failed simulation even after fixing.

Fortunatly I found a method that works:

  1. Copy both files one after another into a single file.
  2. From the top file delete the lines beggining with "MASTER" or "END".
  3. From the bottom file delete the line beggining with "HEADER".
  4. Fix them with the PDB fixer.

This method might produce ugly looking structures like this: pdb1

Which is why you want to move it through the fixer, which will make it look good again: pdb2

HadadIdan commented 5 years ago

Where have you encountered that? Have you been using the most recent version of the project? Because I know for a fact that the current way we merge and simulate works for Michael.

Nitzan810 commented 5 years ago

How are you merging the files then? From what I saw we use the pymol cmd to load 2 files and then save them. Did we change that?

HadadIdan commented 5 years ago

Try pulling the latest version from master. I suspect you are working on an older version.

Nitzan810 commented 5 years ago

Doesn't work for me, did it through the webserver after doing a pull and I got this exception:

ValueError: No template found for residue 38 (LEU). The set of atoms matches LEU, but the bonds are different.

Edit: did that with the following IDs: 3d3m and 3l6a.

MichaelShohat commented 5 years ago

After googling the error 2 explanations are available:

Try running with 6DY4 5ZKH