-
Notifications
You must be signed in to change notification settings - Fork 10
JSO Compliance : LMTR #201
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #201 +/- ##
===========================================
+ Coverage 61.53% 85.56% +24.02%
===========================================
Files 11 13 +2
Lines 1292 1579 +287
===========================================
+ Hits 795 1351 +556
+ Misses 497 228 -269 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Did you compare with the master branch here? |
b2680ca to
cb48329
Compare
|
@dpo i can not compare with the old version because LMTR uses |
In that case, please open a separate PR that introduces the correct stopping test in LMTR. Then we'll be able to compare. |
LMTR TestsWithout boundsSubsolver : R2On this branch, On the branch with fixed optimality measure, Subsolver : TRDHOn this branch, On the branch with fixed optimality measure, With boundsSubsolver : R2On this branch, On the branch with fixed optimality measure, |
|
@dpo, feel free to review when you have some time ! |
@dpo @MohamedLaghdafHABIBOULLAH
We should merge after #199 and #200.