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

It seem the predictor is just a tranditional Classify model not a network model. Why did you say using LSTM in the predictor model in your paper? #19

Open
lorybaby opened this issue Jan 22, 2019 · 1 comment

Comments

@lorybaby
Copy link

Hi, Dear authors,
It seem the predictor is just a tranditional model not a network model. Why did you say using LSTM in the predictor model in your paper? (Deep reinforcement learning for de novo drug design)
I think only the generator was based on the network model, right? I am so confused when I comparing your code with your paper, please give some help to your readers. If you have update the codes, please let me know.

@Mariewelt
Copy link
Collaborator

Hi @lorybaby

This is an ongoing project, so I'm constantly working on it and adding new features. You can find the demo for training the LSTM model in RecurrentQSAR-example-jak2.ipynb and RecurrentQSAR-example-logP.ipynb ipython-notebooks. I also added a couple of new features for the reinforcement learning part (a follow-up paper will be coming soon).

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

2 participants