facebookresearch / CODD

CODD ("Temporally Consistent Online Depth Estimation in Dynamic Scenes"), WACV 2023.
Other
62 stars 3 forks source link

KeyError: 'MultiGammaLrUpdaterHook is not in the hook registry' #1

Closed Qjizhi closed 1 year ago

Qjizhi commented 1 year ago

First thanks for your great work! It seems MultiGamma is not supported by mmcv. When I use schedule_stereo_finetune.py, where the lr_config is OneCycle, the training is ok, but the visualization performance is not good.

mli0603 commented 1 year ago

Hi @Qjizhi, thank you for your interest in the project.

I will look into this. I think the customized LR scheduler may have been stripped off during the code clean up. I will let you know once it is fixed.

Qjizhi commented 1 year ago

Sounds great, thanks!

Hi @Qjizhi, thank you for your interest in the project.

I will look into this. I think the customized LR scheduler may have been stripped off during the code clean up. I will let you know once it is fixed.

Qjizhi commented 1 year ago

Hi @Qjizhi, thank you for your interest in the project.

I will look into this. I think the customized LR scheduler may have been stripped off during the code clean up. I will let you know once it is fixed.

I tried some supported lr_config in https://github.com/open-mmlab/mmcv/blob/master/mmcv/runner/hooks/lr_updater.py#L9, However, the model seems doesn't converge.

ywwwer commented 1 year ago

@Qjizhi I added this customized hook back in the latest commit

Qjizhi commented 1 year ago

awsome, thanks! @ywwwer

@Qjizhi I added this customized hook back in the latest commit

mli0603 commented 1 year ago

fixed by PR #2