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 use HingeLoss instead of BPRLoss ? #15

Open
KID-22 opened this issue Apr 19, 2023 · 1 comment
Open

Why use HingeLoss instead of BPRLoss ? #15

KID-22 opened this issue Apr 19, 2023 · 1 comment

Comments

@KID-22
Copy link

KID-22 commented Apr 19, 2023

This is good job. However, we always use BPRLoss rather than HingeLoss in pairwise learning to rank since the margin of HingeLoss is hard to tune. So I wonder whther you have tried the BPRLoss?

@GanjinZero
Copy link
Owner

No, I haven't tried it. I found set margin=0 in margin loss run very well. I think other losses can achieve similar results (but you should try it).

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