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

Threshold for losses different than original FCGF #14

Open
theycallmefm opened this issue Dec 21, 2020 · 0 comments
Open

Threshold for losses different than original FCGF #14

theycallmefm opened this issue Dec 21, 2020 · 0 comments

Comments

@theycallmefm
Copy link

Hi, you state that you keep the FCGF parameters the same as in the original paper. In your code positive and negative thresholds for contrastive_hardest_negative_loss are like this.

# For FCGF loss we keep the parameters the same as in the original paper

self.pos_thresh = 1.4
self.neg_thresh = 0.1

However in original FCGF code default threshold parameters are opposite. https://github.com/chrischoy/FCGF/blob/f0863a9ba4a29f677c0ddd2248cf4e56b9318add/config.py#L34

trainer_arg.add_argument('--neg_thresh', type=float, default=1.4)
trainer_arg.add_argument('--pos_thresh', type=float, default=0.1)

Did you use your parameters to train FCGF descriptor? If it is a design choice could you elaborate more?

Thank you!

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

1 participant