DiffusionDet icon indicating copy to clipboard operation
DiffusionDet copied to clipboard

DDIM sampling eta setting

Open pinecho opened this issue 2 years ago • 1 comments

Hi, thanks for your great work. I noticed that you applied DDIM sampling method in your code, with the setting: https://github.com/ShoufaChen/DiffusionDet/blob/1efb36d4b0b721f2d509bc72656235a94d73ab02/diffusiondet/detector.py#L97

I found ETA controls the scale of the variance (0 is DDIM, and 1 is one type of DDPM). from https://github.com/ermongroup/ddim It seems that eta=1 means DDPM. May I ask if your eta is the same as the one from ermongroup?

pinecho avatar Jan 06 '23 16:01 pinecho

Hi,

Thanks for your interest.

We have experimented with several eta values. The results demonstrate that eta has negligible influence on the final results. Therefore, we adopt ddim_sampling_eta = 1. as our default setting.

ShoufaChen avatar Jan 09 '23 01:01 ShoufaChen