-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Hi, there are some issues about the infoNCEloss #1
Comments
The |
I know, but when this function is used to perform warm-up training, it will be incorrect. If possible, you can check it. |
If you are asking the warm-up stage, I guess this line is what you are looking for: Line 361 in 77cf96b
|
OKay,i ignore this line. Thank you. |
If possible, could you hare me a training log about the PLRemix on webvision, it seems i met some problems when training on this set. |
Hi, there are some issues about the infoNCEloss:
when we define the info_nce_loss in the code as follows:
info_nce_loss = InfoNCELoss(temperature=0.1,
batch_size=args.batch_size * 2,
flat=('flat' in args.crl),
n_views=8 if args.mcrop else 2)
the labels size in the function of infoNCELoss is batch_size x 2 x n_views, while the size of the features are batch_size x n_views. Since the sizes of these two params are not matched, the calculation of the infoNCEloss is not correct.
The text was updated successfully, but these errors were encountered: