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

Chemprop version incompatible in requirements.txt and setup.py #16

Closed
Stella-zjy opened this issue Jun 21, 2024 · 2 comments
Closed

Chemprop version incompatible in requirements.txt and setup.py #16

Stella-zjy opened this issue Jun 21, 2024 · 2 comments

Comments

@Stella-zjy
Copy link

The chemprop version stated in the requirements.txt is chemprop==1.5.2 while the one stated in the setup.py (or when you setup the environment using pip install synthemol) is chemprop 1.6.1.

The two versions are not compatible with each other. Specifically, the model weights trained using chemprop 1.6.1 may not be applied to a model under chemprop 1.5.2. The following warning occurred when do so:
Warning: Pretrained parameter "readout.1.weight" cannot be found in model parameters.
Warning: Pretrained parameter "readout.1.bias" cannot be found in model parameters.
Warning: Pretrained parameter "readout.4.weight" cannot be found in model parameters.
Warning: Pretrained parameter "readout.4.bias" cannot be found in model parameters.

It will be helpful if you can update the requirements.txt file and change it into chemprop==1.6.1.
Thanks!

@swansonk14
Copy link
Owner

Hi @Stella-zjy,

Thank you for bringing up this issue! I mistakenly put Chemprop version 1.6.1 in the setup.py since I've recently been using that version in my other work, but I used version 1.5.2 to train the Chemprop models here. To resolve this, I changed the setup.py Chemprop version to 1.5.2 (as of SyntheMol version 1.0.4) to fix this. Please try uninstalling and reinstalling SyntheMol to see if that fixes the issue, and please let me know if you're still having any problems.

Best,
Kyle

@swansonk14
Copy link
Owner

Hi @Stella-zjy,

I'm going to close this issue, but please feel free to reopen it if you're still having issues.

Best,
Kyle

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