Repository navigation
[WIP] Clarify transported mass in partial OT - #902
Open
JaylenTang wants to merge 1 commit into
Open
JaylenTang wants to merge 1 commit into
JaylenTang wants to merge 1 commit into
Conversation
JaylenTang
marked this pull request as ready for review
October 26, 2026 14:33
Author
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Types of changes
Documentation and example update.
Motivation and context / Related issue
Related to #890.
Hi, I've put together a first version of the transported-mass explanation proposed in #890.
The guide already gives the partial OT formulation, and the 1D example shows different numbers of transported samples. This extends the existing 2D example to help explain what
mmeans for weighted distributions:m=0.25,0.5,0.75, and1on the same data with a fixed seed.It uses the existing solver and keeps the original entropic and Gromov-Wasserstein sections and attribution. The example also checks transported mass, marginal limits, and the balanced OT endpoint.
I'd appreciate feedback on the scope and whether this fits best in the existing guide and example or in a separate focused example. Thanks!
How has this been tested (if it applies)
Local checks on Python 3.12 with the NumPy CPU backend:
pytest test/ -q --doctest-modules --disable-warnings: 1,524 passed, 134 skipped, 4 expected failures.test/conftest.py: 20 passed.PR checklist
This is a draft for feedback on the documentation and example.