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

scheduler in rdm training #41

Open
JustinXu0 opened this issue Nov 5, 2024 · 1 comment
Open

scheduler in rdm training #41

JustinXu0 opened this issue Nov 5, 2024 · 1 comment

Comments

@JustinXu0
Copy link

I noticed that when training RDM, we need to set args.cosine_lr=True to initialize the scheduler in engine_rdm.py. However, the instructions given in the readme defaults to args.cosine_lr=False. I am new in deep learning. I wonder is it correct to keep the learning rate of adamw as a constant during training? Why can it still converge? Looking forward to your reply!

@LTH14
Copy link
Owner

LTH14 commented Nov 5, 2024

You don't need to set args.cosine_lr=True. A constant learning rate should be ok and can still converge -- the convergence of a model does not mean that the learning rate is 0, but instead, the loss stops decreasing.

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