-
Notifications
You must be signed in to change notification settings - Fork 6
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
Support STP and Yices2 #400
Conversation
Great work. There's one more thing that needs to be done, and that's getting STP support into Rosette. We have two options here:
Given that we want to get STP and Yices2 support into Rosette anyway, I say we take option 1. This means that you should prioritize opening up the STP and Yices2 PR on Rosette ASAP so that we can get their CI working. I suspect they'll be happy to merge after that. This PR will be blocked until they merge. This looks really good! I especially appreciate the detailed PR message. |
Had to manually rerun tests and force total rebuild so that new Rosette got pulled. That manual rerun passed, so merging now. |
This PR adds support for STP and Yices2 to improve our portfolio solving.
It adds
main.rkt
that allows the user to pass in STP or Yices2 as the solver.lakeroad-portfolio.py
that add stp and yices2 as options for portfolio solving.Note that this PR doesn't add functionality for passing flags/solver settings to STP or Yices2; we haven't needed them yet.
Closes #406