sebastianstarke / AI4Animation

Bringing Characters to Life with Computer Brains in Unity
7.83k stars 1.05k forks source link

Deep Phase amplitude (A) formula mismatch #93

Closed Mathux closed 1 year ago

Mathux commented 1 year ago

Hi Sebastian,

Thanks for releasing your great work :)

To compute the amplitude, it seems to be a mismatch between the formula in the paper and in the code:

I think either the code should: amp = torch.sqrt(2 / self.time_range * torch.sum(power, dim=dim)) or the paper should be: A = 2 / N * sqrt(sum(p_ij))

Best, Mathis

Mathux commented 1 year ago

I just found that the 2/N factor is already defined in pi_j, so the formula is correct!