facebookresearch / facebookresearch/moco
one question about the implementation
Open
- 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
Assessment
This issue has not been assessed yet.