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

PDBQT parsing for receptors removes charge sign #116

Open
bradakta opened this issue May 3, 2024 · 3 comments
Open

PDBQT parsing for receptors removes charge sign #116

bradakta opened this issue May 3, 2024 · 3 comments

Comments

@bradakta
Copy link

bradakta commented May 3, 2024

As the title says. The _read_receptor_pdbqt_string() function excludes the field containing the sign of partial charges so that all partial charges are read as positive. It would seem that partial_charges = float(line[71:77].strip()) should instead be partial_charges = float(line[70:77].strip()).

I'm guessing this never has a tangible effect on docking, only the python data structure?

@rwxayheee
Copy link

Yes, this used to be [70:76] (with a size of 6) in AutoDock4

@diogomart
Copy link
Contributor

It has been fixed in 403907e but I just realized it also needs to be fixed in

partial_charges = float(line[71:77].strip())

@diogomart
Copy link
Contributor

@bradakta thanks for reporting this. It does not affect docking, these data structures are used only to process docking results, not to create input for docking.

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

3 participants