josejimenezluna / delfta

Δ-QML for medicinal chemistry
GNU Affero General Public License v3.0
96 stars 16 forks source link

Add checks: hydrogens present, types of atoms #20

Closed cisert closed 3 years ago

cisert commented 3 years ago

https://github.com/josejimenezluna/delfta/blob/ca79d57c862e3b1c626e8abc9f5cf74f3a64b620/delfta/calculator.py#L54

cisert commented 3 years ago
cisert commented 3 years ago
josejimenezluna commented 3 years ago

only missing the check for charges now

josejimenezluna commented 3 years ago

@cisert the atom object in pybel has both a formalcharge and a partialcharge attribute.

https://open-babel.readthedocs.io/en/latest/UseTheLibrary/Python_PybelAPI.html

Which should we be checking?

cisert commented 3 years ago

@josejimenezluna: formal charge :) (difference between #protons and #electrons)