OpenMOSS/CoLLiE

Whether lr_scheduler for Lomo is implemented now?

Opened this issue · 4 comments

Hi authors,

Whether lr = trainer.lr_scheduler.step(global_step) for Lomo in the Trainer is implemented?

If so, how to enable it?

Thanks!

I mean lr_scheduler for Lomo, not AdamW. lr_scheduler for Lomo requires the lr_scheduler.step function to accept a global_step parameter.

Sry, and I get you now. This is a bug and we are planning to fix it to align with the api call with PyTorch's lr_scheduler.
Currently, maybe you can pass this lr_scheduler to trainer when training with Lomo.
https://github.com/OpenLMLab/LOMO/blob/24cde8e91feac437809bf7790f4727623dce6a76/src/utils.py#L207

Hi, lr_scheduler for LOMO is implemented and mergd into branch:dev now.
FYI:3b69924