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

About change L1 loss to SSIM + L1 loss #165

Open
Beniko95J opened this issue Sep 9, 2022 · 1 comment
Open

About change L1 loss to SSIM + L1 loss #165

Beniko95J opened this issue Sep 9, 2022 · 1 comment

Comments

@Beniko95J
Copy link

Hi, I tried to change the L1 supervision to SSIM + L1 supervsion (0.85 * L_ssim + 0.15 L_l1), but failed to get good results. My motivation is that the original L1 supervision only focuses on a single pixel while discarding the local relationship of pixels, thus maybe better performance can be achieved by adding a structural loss like SSIM into supervision. I calculate the SSIM loss by sampling a batch of patches instead of pixels during training (since the memory consumption increases fast when the patch length is large, I only tried path length = 3, e.g. 9 pixels). I am wondering why the performance gets worse by adding the SSIM loss. Any discussion is welcome.

Thanks!

@JetsonKarl
Copy link

Have you solved this problem now? @Beniko95J

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants