Skip to content

Names of rows and columns if possible when logging illegal data in assessBounds and assessMatrix#3165

Open
jajhall wants to merge 3 commits into
latestfrom
fix-3164
Open

Names of rows and columns if possible when logging illegal data in assessBounds and assessMatrix#3165
jajhall wants to merge 3 commits into
latestfrom
fix-3164

Conversation

@jajhall

@jajhall jajhall commented Jul 24, 2026

Copy link
Copy Markdown
Member

Closes #3164

@jajhall jajhall changed the title Names of rows and columns if possible when logging illegal data in assessCosts, assessBounds and assessMatrix Names of rows and columns if possible when logging illegal data in assessBounds and assessMatrix Jul 24, 2026
@jajhall jajhall self-assigned this Jul 24, 2026
@jajhall
jajhall requested a review from BenChampion July 24, 2026 22:44
@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.22807% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.19%. Comparing base (d2b07df) to head (416c427).
⚠️ Report is 180 commits behind head on latest.

Files with missing lines Patch % Lines
highs/util/HighsMatrixUtils.cpp 77.77% 4 Missing ⚠️
highs/lp_data/HighsLpUtils.cpp 95.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #3165      +/-   ##
==========================================
+ Coverage   73.16%   73.19%   +0.02%     
==========================================
  Files         430      432       +2     
  Lines      103967   105496    +1529     
  Branches    16718    16988     +270     
==========================================
+ Hits        76069    77215    +1146     
- Misses      27622    28005     +383     
  Partials      276      276              

☔ View full report in Codecov by Harness.
📢 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.

1 participant