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

Can DLKcat parse the smiles including '.' ? #28

Open
psp3dcg opened this issue Jul 29, 2024 · 2 comments
Open

Can DLKcat parse the smiles including '.' ? #28

psp3dcg opened this issue Jul 29, 2024 · 2 comments

Comments

@psp3dcg
Copy link

psp3dcg commented Jul 29, 2024

I test the DLKcat with the substrate 'CC(=O)[O-].C(C(C(C(C(C=O)O)O)O)O)O.[Na+]', however, I see the code of line 234 ('if smiles != None and "." not in smiles ') in 'prediction_for_input.py' indicates that DLKcat may not be able to process the smiles containing '.'. How to make DLKcat handle the smiles containing '.'? Thank U~

@le-yuan
Copy link
Collaborator

le-yuan commented Jul 31, 2024

Sorry about this, the DLKcat model currently can not support the prediction of kcat values with the input of substrate SMILES containing "."

@jrcasey
Copy link

jrcasey commented Aug 1, 2024

For some added context, the "." symbol in SMILES represents an R-group, making it a class of compounds rather than a single substrate. It would not make sense to predict a turnover number for a reaction with an unknown reactant, so DLKcat is doing what it should with these entries.

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