introduce sinkdiv
simplified API, expand glossary with sinkhorn divergence and LR sinkhorn
#596
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces a few enhancements:
introduces a
sinkdiv
convenience wrapper to compute the Sinkhorn divergence between two point clouds. Using thesinkhorn_divergence
method was a bit clumsy in that case, since it required passing first thePointCloud
geometry type, clearly overkill for most usages.introduces the possiblity to store the
reg_ot_cost
between the second point cloud (in a sinkhorn divergence) and itself and use it as an offset. This is useful when one wants to track the true Sinkhorn divergence between two point clouds but do not want to recompute that term at every iteration, say, of a Gradient descent. that quantity is calledoffset_static_b
corrects a bug in the
plot
class, wherebytitles
were ignored when theshow_lines
flag wasFalse
expands
glossary.rst