-
-
Notifications
You must be signed in to change notification settings - Fork 396
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
[docs] debug failure on v1.11 #3843
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3843 +/- ##
==========================================
+ Coverage 97.50% 97.54% +0.03%
==========================================
Files 44 44
Lines 6017 6017
==========================================
+ Hits 5867 5869 +2
+ Misses 150 148 -2 ☔ View full report in Codecov by Sentry. |
So it seems like the TSP tutorial has broken. I have no idea why |
My conclusion is that the lazy constraint algorithm in GLPK adds local lazy, so it's just pretty terrible at repeating existing work. |
Maybe we should use SCIP instead of GLPK but looking at SCIP.jl, it supports |
Or we could just add Gurobi since we have a license now... |
Nothing really, there is a constraint handler mechanism that should be a
superset of the lazy constraint, so one could implement a restricted
version for LazyConstraintCallback's sake. And this would keep OSS solvers
for the tests, for people to be able to try things out.
…On Wed, Oct 9, 2024, 20:53 Oscar Dowson ***@***.***> wrote:
Or we could just add Gurobi since we have a license now...
—
Reply to this email directly, view it on GitHub
<#3843 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB2FDMUOET3CMPUPO5WRB6TZ2V3SDAVCNFSM6AAAAABPRR75OSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBTGEYTEOBZGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This is sufficient for now. The tutorial changes were because of differences in Random number streams. |
Doc build is currently failing on 1.11