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

which learning rate is true? #35

Open
howardchina opened this issue Nov 19, 2021 · 2 comments
Open

which learning rate is true? #35

howardchina opened this issue Nov 19, 2021 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@howardchina
Copy link

howardchina commented Nov 19, 2021

Hi,

I find the latest configuration file from mmdetection official released version is different from here.
Which one is better in your experiment?

here:

optimizer = dict(_delete_=True, type='AdamW', lr=0.000025, weight_decay=0.0001)

or official released version from mmdetection:
https://github.com/open-mmlab/mmdetection/blob/master/configs/queryinst/queryinst_r50_fpn_1x_coco.py#L127

looking for your reply!

@howardchina
Copy link
Author

'grad_clip' setting is also different between these two sources.

@vealocia
Copy link
Member

vealocia commented Nov 19, 2021

Hi, @howardchina!
Thanks for your interest in our work.
This repo is the original queryinst implementation. All settings are kept the same as in our paper.
But we find that modifying learning rate and grad_clip leads to a better result under ResNet-101 so in mmdetection, we provide an augmented queryinst as you see.
Both implementations are OK but I suggest you to use the augmented one (mmdetection) for better results.

@vealocia vealocia added the documentation Improvements or additions to documentation label Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants