facebookresearch / facebookresearch/moco

one question about the implementation

Open
#87 1 comment 3 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
5.1k
Forks
802
PR merge metrics
No merged PRs in 30d

Description

Hi,

Thanks for the code.

While q and k are used for positive pairs, q and queue are used for negative pairs, what if earlier samples of q are contained in queue ?

Since a mini batch is sampled randomly, an earlier version of q is possibly in the queue. As are the same instances, they should be positive pairs. However, the are treated negative pairs in the implementation code.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.