-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Performance regression after 9.9.4133 (3194fd5) #4189
Comments
I think there is a high variance on your model (like it is often the case, especially in multithread). If you try to solve it many times with different random seed, you can get various solve time. I attached an histogram of 1k runs with the latest version. Also, solving it with one thread, like with --params subsolvers:"max_lp" seems to work better now. |
You're right, there's a lot of variance in my model. I've been using I went and did some other stuff and just started a new run with 9.9.4133 now. The new linear propagation code seems to be slightly slower on average, and the standard deviation is much higher which I failed to account for in my initial assessment:
I'll leave it running to get a better estimate, and I might try testing a patch to change which vars are pushed first to see if that makes a difference. |
Yes, I think the new code adds more "randomness" which explain the higher variance, especially if you test with different random seed. On another hand, the code should be more robust to future change in other places. |
Can you retry ? I have good results with main. So try with 9.11. |
What version of OR-Tools and what language are you using?
Version: main, 9ed0074, 3194fd5
Language: Java
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
CP-SAT
What operating system (Linux, Windows, ...) and version?
Linux
What did you do?
Steps to reproduce the behavior:
What did you expect to see
log_9ed0074090aacba704afd420234a6083e2644b5b.txt
What did you see instead?
log_3194fd5bb2017957dd3460719ba08affaa803511.txt
The model I used for comparison:
model.zip
Some differences I noticed in the logs:
The text was updated successfully, but these errors were encountered: