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

Openbabel 3.1 has a mistake in generating radical molecules from XYZs #1

Open
xiaoruiDong opened this issue Jun 27, 2021 · 2 comments

Comments

@xiaoruiDong
Copy link
Owner

xiaoruiDong commented Jun 27, 2021

Openbabel 3.1 has a mistake in generating radical molecules from XYZs. This influences TS-GCN notebook that uses openbabel as the backend to perceive XYZs for radicals. A temporary fix here may be

  1. checking the total spin multiplicity and valence of each atom
  2. manually assign the multiplicity according to 1.
@xiaoruiDong
Copy link
Owner Author

openbabel/openbabel#2392

@xiaoruiDong xiaoruiDong added the bug Something isn't working label Jun 27, 2021
xiaoruiDong added a commit that referenced this issue Jun 28, 2021
OBConversion generated molecules have wrong atom attributes, e.g., all zero multiplicity. There may be another workaround, but currently manually assign spin multiplicities for H, C, and O radical sites.
@xiaoruiDong
Copy link
Owner Author

xiaoruiDong commented Sep 4, 2021

This is a bug possibly due to a bad compilation of the anaconda binary (maybe only for macOS). Currently workaround for N, C, H, and O is added.

@xiaoruiDong xiaoruiDong added bugs or issue in dependencies and removed bug Something isn't working labels Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant