-
-
Notifications
You must be signed in to change notification settings - Fork 405
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prep for v1.11.0 #3340
Prep for v1.11.0 #3340
Conversation
odow
commented
May 3, 2023
•
edited
Loading
edited
- extension-tests https://github.com/jump-dev/JuMP.jl/actions/runs/4867072087
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3340 +/- ##
==========================================
+ Coverage 97.92% 98.05% +0.12%
==========================================
Files 34 34
Lines 4874 4874
==========================================
+ Hits 4773 4779 +6
+ Misses 101 95 -6 ☔ View full report in Codecov by Sentry. |
cc @pulsipher it's good having the |
@@ -30,11 +30,7 @@ set_silent(model) | |||
@objective(model, Min, [3x1 + x2, -x1 - 2x2]) | |||
@constraint(model, 3x1 - x2 <= 6) | |||
set_optimizer(model, () -> MOA.Optimizer(HiGHS.Optimizer)) | |||
set_attribute( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intended diff?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yip. I just noticed when I checked the versions we were using. Didn't seem worth a separate PR when I was doing this one.
Yes this this will be very nice to have as a warning/safeguard. |