Discrepancy in documentation: at what point is vector X in `translate_diff`

[Affie]

https://juliamanifolds.github.io/Manifolds.jl/stable/manifolds/group.html#Manifolds.translate_diff-Tuple{AbstractDecoratorManifold,%20Vararg{Any}}

translate_diff(G::AbstractDecoratorManifold, p, q, X, conv::ActionDirectionAndSide=LeftForwardAction())

• fist sentence and math: "tangent vector X ∈ TqG"

• but in note: "X is stored at the point p ∈ G"

[Affie]

I'm also not sure if the implementation w.r.t. ActionDirectionAndSide is consistent. I'm busy looking at https://juliamanifolds.github.io/Manifolds.jl/stable/manifolds/group.html#Manifolds.translate-Tuple{AbstractDecoratorManifold,%20Vararg{Any}}

It looks like the LeftInvariantRepresentation implementation may be swopped around or RightBackwardAction but I'm not following the ActionDirectionAndSide that well yet.

Edit: From slack the correct call is translate_diff(G, q, p, X, Manifolds.RightBackwardAction())

when you want to differentiate p∘q with respect to p in the direction X (tangent at p).