Remove assert for empty workers - #3134
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## latest #3134 +/- ##
==========================================
- Coverage 73.16% 73.16% -0.01%
==========================================
Files 430 430
Lines 103967 103965 -2
Branches 16718 16717 -1
==========================================
- Hits 76069 76065 -4
- Misses 27622 27624 +2
Partials 276 276 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
odow
left a comment
There was a problem hiding this comment.
I don’t have the ability to review this. As a high level comment: tests, or it didn’t happen. I would love to better understand how we can more robustly test highs, instead of running a few large scale instances and declaring success.
Although the need to test HiGHS more thoroughly on LPs, and to test presolve rules on curated special cases has been exposed in recent weeks - and is being addressed, the testing of HiGHS has always been rather more than "running a few large scale instances and declaring success." @fwesselm is performing regression testing on about 850 MIPs. |
No description provided.