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] ot.lp reorganizing #714

Merged
merged 12 commits into from
Jan 20, 2025
Merged

[MRG] ot.lp reorganizing #714

merged 12 commits into from
Jan 20, 2025

Conversation

eloitanguy
Copy link
Collaborator

@eloitanguy eloitanguy commented Jan 20, 2025

Types of changes

  • create ot.lp.network_flow.py to store ot.emd and related functions which used to be in ot/lp/__init__.py
  • create ot.lp.barycenter_solvers.py to store free_support_barycenter functions

Motivation and context / Related issue

How has this been tested (if it applies)

  • full coverage in test/test_ot.py

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.

@github-actions github-actions bot added the ot.lp label Jan 20, 2025
Copy link

codecov bot commented Jan 20, 2025

Codecov Report

Attention: Patch coverage is 93.70370% with 17 lines in your changes missing coverage. Please review.

Project coverage is 97.08%. Comparing base (200322b) to head (26748eb).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #714   +/-   ##
=======================================
  Coverage   97.08%   97.08%           
=======================================
  Files          98      100    +2     
  Lines       19937    19947   +10     
=======================================
+ Hits        19355    19365   +10     
  Misses        582      582           

RELEASES.md Outdated Show resolved Hide resolved
@eloitanguy eloitanguy changed the title [WIP] ot.lp reorganise + OT barycenters with generic costs [WIP] ot.lp reorganise Jan 20, 2025
@eloitanguy eloitanguy changed the title [WIP] ot.lp reorganise [WIP] ot.lp reorganize Jan 20, 2025
@eloitanguy eloitanguy changed the title [WIP] ot.lp reorganize [MRG] ot.lp reorganize Jan 20, 2025
@rflamary rflamary changed the title [MRG] ot.lp reorganize [MRG] ot.lp reorganizing Jan 20, 2025
@github-actions github-actions bot added the Tests label Jan 20, 2025
@rflamary rflamary merged commit 9d00f96 into PythonOT:master Jan 20, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants