Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MRG] Batch/Lazy Log Sinkhorn Knopp on samples #259

Merged
merged 7 commits into from
Jun 14, 2021
Merged

[MRG] Batch/Lazy Log Sinkhorn Knopp on samples #259

merged 7 commits into from
Jun 14, 2021

Conversation

6Ulm
Copy link
Collaborator

@6Ulm 6Ulm commented Jun 11, 2021

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Motivation and context / Related issue

How has this been tested (if it applies)

Checklist

  • The documentation is up-to-date with the changes I made.
  • I have read the CONTRIBUTING document.
  • All tests passed, and additional code has been covered with new tests.

@rflamary rflamary changed the title Add batch implementation of Sinkhorn Knopp algorithm [WIP] Batch/Lazy of Sinkhorn Knopp on samples Jun 11, 2021
@rflamary rflamary changed the title [WIP] Batch/Lazy of Sinkhorn Knopp on samples [WIP] Batch/Lazy Log Sinkhorn Knopp on samples Jun 11, 2021
@codecov
Copy link

codecov bot commented Jun 11, 2021

Codecov Report

Merging #259 (28dd2fe) into master (982510e) will increase coverage by 0.02%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
+ Coverage   93.07%   93.09%   +0.02%     
==========================================
  Files          17       17              
  Lines        3450     3506      +56     
==========================================
+ Hits         3211     3264      +53     
- Misses        239      242       +3     

ot/bregman.py Show resolved Hide resolved
ot/bregman.py Outdated Show resolved Hide resolved
ot/bregman.py Outdated Show resolved Hide resolved
ot/bregman.py Outdated Show resolved Hide resolved
ot/bregman.py Outdated Show resolved Hide resolved
@rflamary rflamary changed the title [WIP] Batch/Lazy Log Sinkhorn Knopp on samples [MRG] Batch/Lazy Log Sinkhorn Knopp on samples Jun 11, 2021
@rflamary rflamary merged commit 2dbeeda into PythonOT:master Jun 14, 2021
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