Sinkhorn distance implementation

Open utkutpcgl opened this issue 2 years ago • 1 comments

Thanks for the great contribution!

As mentioned in https://github.com/Megvii-BaseDetection/OTA/issues/17#issue-1484922452, the original sinkhorn implementation is different (page 11 of your paper):

Why is this the case?

Jun 08 '23 12:06 utkutpcgl

By the way here is the original implementation u is updated first. u and v updates seem to be switched (confused?) in this paper:

https://github.com/gpeyre/SinkhornAutoDiff/blob/1e6c8c38974a325bb59cbc5b8cb6af5787b4f58d/sinkhorn_pointcloud.py#L63

Jun 14 '23 16:06 utkutpcgl