open-mmlab / mmdetection

OpenMMLab Detection Toolbox and Benchmark
https://mmdetection.readthedocs.io
Apache License 2.0
29.61k stars 9.47k forks source link

SyncBN with `norm_eval=True` #9877

Open liming-ai opened 1 year ago

liming-ai commented 1 year ago

Prerequisite

💬 Describe the reimplementation questions

I just realized that SyncBN should set norm_eval=False. However, there is nothing to highlight this in the docs or elsewhere. Should we set a hook to check that?

If we use SyncBN and norm_eval=True at the same time, does it mean that it is equivalent to using BN and norm_eval=True because the value of BN has not been modified during training?

Also, using SyncBN(requires_grad=True) and norm_eval=True at the same time, will it hurt performance?

Environment

It is not relative to the environment.

Expected results

No response

Additional information

No response

cskkxjk commented 1 year ago

I am also confused by the syncbn setting as well!

liming-ai commented 1 year ago

Hi @ZwwWayne,

Could you please check this?