Closed sh0416 closed 3 years ago
z1_z3_cos = cls.sim(z1, z3).unsqueeze(-1)
Could be better, I think.
Sorry I didn't get the question. Were you asking whether the negatives are shared within the batch? The answer is yes.
You should be able to reproduce our result by using the latest commit.
Hi, I wonder that the negative examples are shared when computing cosine similarity.
I think the broadcasting is not needed..
Also, could you give some commit id that reproduces the checkpoint you provided?
Thanks,