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

Loss weights and resultant curve issue #23

Open
TWang1017 opened this issue Dec 7, 2022 · 2 comments
Open

Loss weights and resultant curve issue #23

TWang1017 opened this issue Dec 7, 2022 · 2 comments

Comments

@TWang1017
Copy link

    Hi, thanks a lot for your swift response and your reminder helps a lot.

One more thing, I train on the DTU dataset with augmentation and co-seg deactivated. The training loss looks like below, the SSIM loss dominates the standard unsupervised loss based on the default weight [12xself.reconstr_loss (photo_loss) + 6xself.ssim_loss + 0.05xself.smooth_loss]. In this case, is it sensible to change the weight, like reduce the 6xself.ssim_loss to 1xself.ssim_loss such that it is in the similar range with reconstr_loss?

Also, the training seems not steady, it fluctuates a lot. Any clues why this happens? Thanks in advance for your help.

image

Originally posted by @TWang1017 in #22 (comment)

@ToughStoneX
Copy link
Owner

ToughStoneX commented Dec 7, 2022 via email

@TWang1017
Copy link
Author

TWang1017 commented Dec 7, 2022

Hi, thanks for your explanation. So that is the challenge occurred in the photometric loss, especially the MVS, the illumination changes and the occlusions. Also, the reconst_loss and SSIM loss seems not in the same range. Would that be beneficial to tweak the default loss weight? I replaced the backbone so I guess it is better not to stick with the default weights that is designed for CVPMVSNET. Thanks

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