Skip to content

Conversation

@rflamary
Copy link
Collaborator

@rflamary rflamary commented Mar 24, 2022

Types of changes

This PR aim at adding the factored coupling solver of:

Forrow, A., Hütter, J. C., Nitzan, M., Rigollet, P., Schiebinger,
G., & Weed, J. (2019, April). Statistical optimal transport via factored
couplings. In The 22nd International Conference on Artificial
Intelligence and Statistics (pp. 2454-2465). PMLR.

Motivation and context / Related issue

How has this been tested (if it applies)

PR checklist

  • I have read the CONTRIBUTING document.
  • The documentation is up-to-date with the changes I made (check build artifacts).
  • All tests passed, and additional code has been covered with new tests.
  • I have added the PR and Issue fix to the RELEASES.md file.

@codecov
Copy link

codecov bot commented Mar 24, 2022

Codecov Report

Merging #358 (46b8dd7) into master (7671715) will increase coverage by 0.05%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##           master     #358      +/-   ##
==========================================
+ Coverage   93.69%   93.74%   +0.05%     
==========================================
  Files          22       23       +1     
  Lines        5753     5799      +46     
==========================================
+ Hits         5390     5436      +46     
  Misses        363      363              

@rflamary rflamary changed the title [WIP] Add factored coupling (plan and objective) [WIP] Add factored coupling Mar 24, 2022
@rflamary rflamary changed the title [WIP] Add factored coupling [MRG] Add factored coupling Mar 24, 2022
@rflamary rflamary merged commit 82452e0 into master Mar 24, 2022
@rflamary rflamary deleted the factored_coupling branch April 11, 2022 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants