⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@eminyouskn
Copy link
Contributor

Fixes # .

Summary/Motivation:

This PR add the objective bound retreival for MIP when KNITRO is used as solver.

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.

@eminyouskn
Copy link
Contributor Author

@mrmundt Sorry for adding another PR.

@mrmundt
Copy link
Contributor

mrmundt commented Jan 26, 2026

@mrmundt Sorry for adding another PR.

RUDE. (Kidding, thanks for doing this 😄 )

@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.40%. Comparing base (bb1203d) to head (285866c).
⚠️ Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
pyomo/contrib/solver/solvers/knitro/engine.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3826      +/-   ##
==========================================
- Coverage   89.40%   89.40%   -0.01%     
==========================================
  Files         903      903              
  Lines      105380   105386       +6     
==========================================
+ Hits        94214    94218       +4     
- Misses      11166    11168       +2     
Flag Coverage Δ
builders 29.14% <16.66%> (-0.01%) ⬇️
default 86.02% <83.33%> (?)
expensive 34.84% <16.66%> (?)
linux 86.75% <16.66%> (-2.42%) ⬇️
linux_other 86.75% <16.66%> (-0.01%) ⬇️
osx 82.91% <16.66%> (-0.01%) ⬇️
win 84.98% <16.66%> (-0.01%) ⬇️
win_other 84.98% <16.66%> (-0.01%) ⬇️

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.

@blnicho blnicho moved this from Todo to Reviewer Approved in Pyomo 6.10 Jan 27, 2026
@mrmundt mrmundt merged commit c1bf547 into Pyomo:main Jan 27, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in Pyomo 6.10 Jan 27, 2026
@eminyouskn eminyouskn deleted the knitro-obj-bound branch January 27, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants