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

TorchDrug can't use Lr_Scheduler #152

Closed
mrzzmrzz opened this issue Nov 9, 2022 · 4 comments
Closed

TorchDrug can't use Lr_Scheduler #152

mrzzmrzz opened this issue Nov 9, 2022 · 4 comments

Comments

@mrzzmrzz
Copy link

mrzzmrzz commented Nov 9, 2022

Hey, I found a bug that when I load the related TorchDrug modules, I can't use the torch.optim.lr_scheduler.
Look at this picture which comes from the TorchDrug Colab files(Property Prediction). I add one lr_schduler for the optimizer. and it throws an error.

image

However, When I don't load any TorchDrug modules, I can use the optimizer normally.

@mrzzmrzz
Copy link
Author

mrzzmrzz commented Nov 9, 2022

OK, Seems like that the bug is also mentioned in #148. I also tried the MultiStepLr

image

image

Also Throw Error.

@mrzzmrzz
Copy link
Author

mrzzmrzz commented Nov 9, 2022

CosineAnnealingLR

image

@Oxer11
Copy link
Contributor

Oxer11 commented Nov 9, 2022

Hi! You're right. This issue has been reported in #148 and fixed in 4ea99a0. You can clone the latest repo for usage.

Sorry for the inconvenience. We'll update the conda and pip versions later.

@mrzzmrzz
Copy link
Author

mrzzmrzz commented Nov 9, 2022

OK, Thank you For your timely repair work!

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