denoising-diffusion-pytorch icon indicating copy to clipboard operation
denoising-diffusion-pytorch copied to clipboard

About recent code commit updates

Open wangqiang9 opened this issue 3 years ago • 5 comments

Hello, I downloaded your latest commit code. I feel that the reconstruction effect is much worse than the code downloaded in March. Have you observed the same phenomenon? Thanks a lot!

wangqiang9 avatar May 03 '22 12:05 wangqiang9

@XDUWQ Hi Wang, so i thought the culprit could be migrating to the next ConvNext blocks, but that was done in January https://github.com/lucidrains/denoising-diffusion-pytorch/commit/91cff459394b784f870f3998023439796c95b7ca

You could always try checking out the specific commit in the repository and doing a comparison

Would be really interested if you figure out what it was

The only other thing I changed was moving linear attention to full attention, but that should lead to an improvement

lucidrains avatar May 03 '22 15:05 lucidrains

@XDUWQ there was also this bug fix https://github.com/lucidrains/denoising-diffusion-pytorch/commit/782c904d3b5240b79fb66929ca8ab9cf6d2496e5 , but i think the patch is the correct way to do it

lucidrains avatar May 03 '22 16:05 lucidrains

@XDUWQ i brought back the old resnet blocks, so you can try turning on and off ConvNext using 'use_convnext`

curious what you find is better!

lucidrains avatar May 04 '22 17:05 lucidrains

@XDUWQ i brought back the old resnet blocks, so you can try turning on and off ConvNext using 'use_convnext`

curious what you find is better!

Thank you for your contribution! I'm trying to use your suggestion…

wangqiang9 avatar May 05 '22 07:05 wangqiang9

So, should we be using a commit from March? Has anyone else verified the change in behavior from March until now? @XDUWQ do you have any examples of the difference in reconstruction quality?

sgbaird avatar Jun 01 '22 18:06 sgbaird