Conversation
|
@haman80 I've cleared the merge conflicts and pushed to Git |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## latest #2870 +/- ##
==========================================
+ Coverage 80.65% 80.87% +0.21%
==========================================
Files 347 347
Lines 86690 86824 +134
==========================================
+ Hits 69920 70216 +296
+ Misses 16770 16608 -162 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @jajhall , |
jajhall
left a comment
There was a problem hiding this comment.
Other than the one error you inherited from me - see comment - this is largely OK.
I'll compile and run it to study it further
…-limit-feasible that expose some issues to fix
@jajhall Thank you so much for the review. I pushed another commit addressing the issues tagged with "fix-2870". Please feel free to take a look and let me know your thoughts. |
jajhall
left a comment
There was a problem hiding this comment.
This addresses my original request for changes, so is good to be merged once the CI tests pass.
@jajhall Thanks again for the review. I fixed the CI failure, which was due to a format-type mismatch in |
Add timeout support to getIIS
Improve handling of unexpected model statuses
Enhancements to HighsIis