-
Notifications
You must be signed in to change notification settings - Fork 7
Target Phase (non-inverted) + Luby restarts + glues #11
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
base: master
Are you sure you want to change the base?
Conversation
|
This seems to give a lot of speedup. But maybe there is a bug... |
|
Yep, there is a bug :) |
|
Sure! I'll wait for the previous results (cluster's busy...) and then give this one a go! |
|
I think this may be OK now, so when you have time and the cluster is not too busy, it'd be nice to see the performance :) |
aehyvari
left a comment
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.
After running all the benchmarks, this starts to look really good. However, I left one question that puzzled me. I have a version that I'm cleaning for opensmt master, we can continue the discussion there once I manage to make the PR.
|
I'm super-happy to see that this has improved performance! By the way, with in-line distillation, I am pretty sure we could make the solver about 1.5x faster. Maybe I'll have a go at it sometime later today, see if you like it :) It basically tries to simplify the learnt clauses once in a while. It's super effective. I have been hacking my SAT solver to bits lately and it's helped a lot with the performance of my counting tool... I'm back in the game again :) |
We should absolutely try it out, sounds great! |
This may help. I'll hack it a bit more.