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

why chose "Supervised Contrastive loss" #3

Open
hppy139 opened this issue Dec 3, 2024 · 1 comment
Open

why chose "Supervised Contrastive loss" #3

hppy139 opened this issue Dec 3, 2024 · 1 comment

Comments

@hppy139
Copy link

hppy139 commented Dec 3, 2024

Inspired by <Zhou et al., 2021>, in Section 3.4, you chose to do contrastive learning by using "Supervised Contrastive loss".
But in <Zhou et al., 2021>, I found that "Margin-based Contrastive Loss" may be the better choice.

So, I wonder for why?

@ddehun
Copy link
Owner

ddehun commented Dec 16, 2024

Hi, thank you for your interest in our work!

The decision to use SCL instead of Margin-based CL was primarily a design choice. SCL is simpler to implement due to having fewer hyperparameters, and it already provided reasonable performance gains. Based on this, we decided not to experiment further with other CL objectives. That said, I agree that exploring more advanced or alternative objectives could potentially enhance the overall performance of the models.

Thanks again!

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