You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The standardizer currently assigns a penalty of 5 when there are overlapping atoms or 6 if there are more than six overlapping atoms. This still allows molecules which have overlapping atoms around a chiral center to pass through.
We also don't have any checks for molecules which have chiral centers with chiral atoms (atoms around a chiral center) which lie on the same line.
Here are some examples we found which have either overlapping chiral atoms (atoms around a chiral center) or chiral atoms which lie on the same line:
Since it's not possible to correctly interpret the stereochemistry of those atoms, I believe it should either be removed or the structure itself should be rejected. I'm probably going to add something to remove the stereo at the RDKit level anyway, but I think that it's worth discussion doing something about this during structure import as well.
The text was updated successfully, but these errors were encountered:
thanks for reporting @greglandrum. We'll be fixing those ones for the next ChEMBL release and check how we can best deal with them during the data loading phase
The standardizer currently assigns a penalty of 5 when there are overlapping atoms or 6 if there are more than six overlapping atoms. This still allows molecules which have overlapping atoms around a chiral center to pass through.
We also don't have any checks for molecules which have chiral centers with chiral atoms (atoms around a chiral center) which lie on the same line.
Here are some examples we found which have either overlapping chiral atoms (atoms around a chiral center) or chiral atoms which lie on the same line:
Since it's not possible to correctly interpret the stereochemistry of those atoms, I believe it should either be removed or the structure itself should be rejected. I'm probably going to add something to remove the stereo at the RDKit level anyway, but I think that it's worth discussion doing something about this during structure import as well.
The text was updated successfully, but these errors were encountered: