-
Notifications
You must be signed in to change notification settings - Fork 1
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
CHEBI:53020 #8
Comments
for some reason the S GROUP is detected twice, we'll need to look at it so it is probably the reason why molecular formula has the 2 in front and the weights also differ from ACD calculations. I expect ACD show smaller values? we'll need to check on that InChI and InChI key values are the ones provided by rdkit by directly calling the InChI software so that's much tricker. |
do you know which version of inchi is ACD using? |
more potential molecules with issues: all seem to have multiple nested SGROUPS (with SPL, parent list). We should maybe not calculate them until we can find a fix. Some of the structures may be incorrect as well. |
using for sg in Chem.GetMolSubstanceGroups(mol):
print(sg.GetPropsAsDict()) |
now fixed in the code |
https://wwwdev.ebi.ac.uk/chebi/beta/CHEBI:53020
It seems to add the number 2 in front of the formula which needs fixing.
The Mass, InChI and InChIKey calculated are different to the ones generated by ACD which is unusual.
The text was updated successfully, but these errors were encountered: