Skip to content

ot.da.SinkhornTransport : does barycentric transform work for metric!='sqeuclidean'? #206

Closed

Description

@samarth4149

Hi,

I'm unsure whether the transform the way it is currently computed using the coupling matrix would work for a metric which is not the l_2 norm going by equations 13 and 14 here. Would this for instance, work with cosine distance metric?
Cosine also seems to pose the following question : Would the final transformed source examples then be correct up to a multiplicative factor?

Much appreciated!

Activity

  1. rflamary commented on Aug 24, 2020

    @rflamary
    Collaborator

    This is a very good question!

    No it does not! While the euclidean barycentric mapping makes sens it is indeed not the proper barycentric mapping for any loss. We should definitely add this to the documentation and begin implementing for other losses (which is much more complex since it will require to solve optimization problems on several of them)

  2. locked and limited conversation to collaborators on Aug 13, 2025
  3. converted this issue into a discussion #752 on Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions