Open DenisDsh opened 6 years ago
I agree with you. @SSARCandy
This is my implementation, any advice?
def coral_loss(source, target):
d = source.size(1)
ns, nt = source.size(0), target.size(0)
# source covariance
tmp_s = torch.ones((1, ns)) @ source
cs = (source.t() @ source - (tmp_s.t() @ tmp_s) / ns) / (ns - 1)
# target covariance
tmp_t = torch.ones((1, nt)) @ target
ct = (target.t() @ target - (tmp_t.t() @ tmp_t) / nt) / (nt - 1)
# frobenius norm
loss = (cs - ct).pow(2).sum().sqrt()
loss = loss / (4 * d * d)
return loss
@yaox12 I used to run your code but got the following error.
Traceback (most recent call last):
File "DeepCoral.py", line 117, in
@redhat12345 My code is based on PyTorch>=0.4
, in which torch.tensor
and Variable
are merged together.
@yaox12 Even I use Pytorch=0.4 but got the error:
Traceback (most recent call last):
File "DeepCoral.py", line 147, in
@redhat12345 if the source
and target
are cuda tensors, then torch.ones((1, ns))
should be torch.ones((1, ns)).cuda()
, as well as that of nt
.
I have tried with this loss and find it usually gets NaN
. I have no idea why.
@yaox12, I agree with you. But I think line 14 is:
loss = (cs - ct).pow(2).sum()
.
Because in paper is
$$l_{coral}=\frac{1}{4d^2}||C_s-C_T||^2_F$$
and Frobenius norm is
$$||A||_F=\sqrt{\sum^m\sum^n |a|^2}$$
then
$$||C_s-C_T||^2_F$$
should not have sqrt()
.
And I think writer's code is also right.
Why you think writer's code is also right?
In my opinion, the main problem is the calculation of the covariance, in paper, the covariance is get by dividing by (n-1), but in the code , it is get by dividing by (n), that is " torch.mean(torch.mul((xc - xct), (xc - xct)))" . however, I'm actually not sure which one is the right one.
tldr, no error, this code is "correct" but the magnitude of the loss is not scaled correctly.
If you plot the values produced by this code vs the original method from the paper, you get the same trends but they are scaled differently, i.e., this code makes the magnitude of coral loss different by a ratio of D*D/(B-1)**2, for B batch size and D dimensionality of features.
def coral_loss(source, target):
# source covariance
xs = torch.mean(source, 0, keepdim=True) - source
xs = xs.t() @ xs
# target covariance
xt = torch.mean(target, 0, keepdim=True) - target
xt = xt.t() @ xt
# frobenius norm
loss = torch.mean(torch.mul(xs - xt, xs - xt))
# note: b batch dim, d is feature dim
# original deep coral implementation differs from the above by a ratio of (d * d / (b-1) / (b-1))
# loss / (4 * d * d) * (d * d / (b-1) / (b-1)) simplifies to
b = source.shape[0] - 1 # batch dim
return loss / (4 * b * b)
I noticed that both the covariance and the Frobenius norm are computed differently in your implementation.
You compute the Frobenius norm as below:
# frobenius norm between source and target
loss = torch.mean(torch.mul((xc - xct), (xc - xct)))
However as stated here http://mathworld.wolfram.com/FrobeniusNorm.html , after squaring each element and summing them, should be computed the square root of the sum not the mean of the squared elements.In the original paper the covariances are computed as below : https://arxiv.org/abs/1607.01719
While in your implementation: