Skip to content

[MRG] Fiw proximal point algorithm - #844

Merged
rflamary merged 2 commits into
PythonOT:masterfrom
thibaut-germain:proximal_update
Aug 28, 2026
Merged

[MRG] Fiw proximal point algorithm#844
rflamary merged 2 commits into
PythonOT:masterfrom
thibaut-germain:proximal_update

Conversation

@thibaut-germain

Copy link
Copy Markdown
Contributor

Types of changes

Fixing sign issue in proximal_bregman_log_plan_batch changing C + inner_reg * log(T) to C - inner_reg * log(T).

Motivation and context / Related issue

Issue #842

How has this been tested (if it applies)

PR checklist

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

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.86%. Comparing base (bb58c31) to head (cd8cd36).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #844   +/-   ##
=======================================
  Coverage   96.86%   96.86%           
=======================================
  Files         128      128           
  Lines       25938    25947    +9     
=======================================
+ Hits        25124    25133    +9     
  Misses        814      814           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rflamary rflamary changed the title Proximal update [MRG] Fiw proximal point algorithm Aug 28, 2026
@rflamary
rflamary merged commit 9439b8a into PythonOT:master Aug 28, 2026
21 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