Wrong index type

[Katterrina]

Hi,

I think there should be faiss.IndexFlatIP instead of faiss.IndexFlatL2 in the aggregation script.

Katerina

[heathcliff233]

Dear Katerina,

Thank you for reading our paper into detail and the question. We have updated our model to v2 to support esm2 and have changed to L2 distance in order to support more optimization methods in faiss package. I will add some explanations to avoid ambiguity.

Best,
Liang

[Katterrina]

Thank you very much for the quick reply! I have a follow up question: If I download model checkpoints using the link from README (link to the checkpoint), am I supposed to use L2 distance or IP?