Skip to content

KKT Transform#3860

Open
ChrisLaliwala wants to merge 10 commits intoPyomo:mainfrom
ChrisLaliwala:kkt-transform
Open

KKT Transform#3860
ChrisLaliwala wants to merge 10 commits intoPyomo:mainfrom
ChrisLaliwala:kkt-transform

Conversation

@ChrisLaliwala
Copy link

Fixes # .

Summary/Motivation:

This PR creates a Pyomo transformation for reformulating models using the KKT conditions.

Changes proposed in this PR:

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@ChrisLaliwala
Copy link
Author

Need to add a couple of more test before PR is ready to be reviewed. Will be done in 1-2 days.

ChrisLaliwala and others added 3 commits February 25, 2026 12:29
Co-authored-by: Bethany Nicholson <blnicho@users.noreply.github.com>
@ChrisLaliwala ChrisLaliwala marked this pull request as ready for review February 25, 2026 17:58
@blnicho blnicho requested a review from emma58 March 3, 2026 19:49
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 97.20670% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.51%. Comparing base (bdb22dd) to head (fce4632).

Files with missing lines Patch % Lines
pyomo/core/plugins/transform/kkt.py 97.20% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3860      +/-   ##
==========================================
- Coverage   89.67%   86.51%   -3.17%     
==========================================
  Files         908      909       +1     
  Lines      106735   106914     +179     
==========================================
- Hits        95717    92495    -3222     
- Misses      11018    14419    +3401     
Flag Coverage Δ
builders 29.06% <13.40%> (-0.02%) ⬇️
default 85.98% <97.20%> (?)
expensive 35.47% <13.40%> (?)
linux ?
linux_other ?
oldsolvers 27.97% <13.40%> (-0.03%) ⬇️
osx ?
win ?
win_other ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

4 participants