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.

Sign up for GitHub

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

Jump to bottom

Siamese BERT seems to be a cross encoder rather than a bi encoder. #2

Open

rohit1998 opened this issue Nov 4, 2022 · 1 comment

rohit1998 commented Nov 4, 2022

siamese vs cross encoder
cross encoder

I believe you are not implementing a siamese network here but rather a cross encoder in case of Siamese BERT example.

Owner

shahrukhx01 commented Nov 4, 2022

Hi, Thanks for pointing it out. You are correct. I intend to add Siamese BERT to this repo at a later point in time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment