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

Error in ligand processing #376

Open
adventureneed opened this issue Dec 13, 2023 · 2 comments
Open

Error in ligand processing #376

adventureneed opened this issue Dec 13, 2023 · 2 comments

Comments

@adventureneed
Copy link

adventureneed commented Dec 13, 2023

Hello,

I have reproduced the same error as here (#369) both in web and in local pdb2pqr environment. I believe, there should be an if statement in mol2.py file, regarding "P" atoms calculation as it should only be done, when "P" atoms are in elements list as my ligand in .mol2 file does not have any kind of P atoms. Could it be fixed in any way?

In PDB2PQR web page

2023-12-13 11:36:43,704 INFO:main.py:670:non_trivial:Processing ligand.
2023-12-13 11:36:43,704 WARNING:main.py:671:non_trivial:Using ZAP9 forcefield for ligand radii.
2023-12-13 11:36:43,704 CRITICAL:main.py:800:main_driver:'P' is not in list
2023-12-13 11:36:43,704 CRITICAL:main.py:801:main_driver:Giving up.

On pdb2pqr local environment

WARNING:Using ZAP9 forcefield for ligand radii.
CRITICAL:'P' is not in list
CRITICAL:Giving up.
Traceback (most recent call last):
  File "/home/psi4conda/lib/python3.6/site-packages/pdb2pqr/main.py", line 790, in main_driver
    is_cif=is_cif,
  File "/home/psi4conda/lib/python3.6/site-packages/pdb2pqr/main.py", line 665, in non_trivial
    ligand.assign_parameters()
  File "/home/psi4conda/lib/python3.6/site-packages/pdb2pqr/ligand/mol2.py", line 331, in assign_parameters
    self.assign_charges()
  File "/home/psi4conda/lib/python3.6/site-packages/pdb2pqr/ligand/mol2.py", line 349, in assign_charges
    atom.charge = atom.formal_charge
  File "/home/psi4conda/lib/python3.6/site-packages/pdb2pqr/ligand/mol2.py", line 295, in formal_charge
    p_atom = self.bonds[0].atoms[elements.index("P")]
ValueError: 'P' is not in list

Using 6A2W PDB structure with its ligands.

@sobolevnrm
Copy link
Member

Can you provide a PDB ID for a structure that generates this error?

@adventureneed
Copy link
Author

adventureneed commented Jan 18, 2024

Hello,

Sorry for the late reply. As I have stated above, it is the 6A2W PDB structure.

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