Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

DxD instead of NxN in the computation of the covariance matrix of Z? #25

Open
aarontyliu opened this issue Jul 29, 2023 · 2 comments
Open

Comments

@aarontyliu
Copy link

Hi, I came across the implementation of the covariance matrix of Z while trying to use VICReg in my project. I feel like I am missing something here, but I wonder if the implementation should be x @ x.T instead of x.T @ x?

Thanks for any information you can provide.

@steinate
Copy link

Same quetions. Did you figure out? I also think the former implementation is right.

@hfangcat
Copy link

hfangcat commented Nov 1, 2023

The implementation should be x.T @ x, if x's dimension is (B, D), here B: batch_size, D: dimension, because they want the covariance matrix to be a function of the dimension

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants