-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add solver independent quantifier elimination with ultimate eliminator #462
Draft
Anastasia-Gu
wants to merge
82
commits into
master
Choose a base branch
from
add_solver_independent_quantifier_elimination_with_ultimate_eliminator
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
82 commits
Select commit
Hold shift + click to select a range
1cc2659
Add UltimateEliminator dependencies to ivy and the classpath and inte…
e4aa914
add dependencies for Ultimate Eliminator
Anastasia-Gu 6271bf9
add dependencies for Ultimate Eliminator
Anastasia-Gu 349ac26
add dependencies for Ultimate Eliminator
Anastasia-Gu ed9c398
Add more UltimateEliminator dependencies to ivy and the classpath and…
bbd1848
Extend UltimateEliminator test example with hints
6d12d67
added Ultimate Eliminator integration to CVC5 and Z3
Anastasia-Gu 231935a
add Mathsat5QuantifiedFormulaManager;
Anastasia-Gu 2efbf7e
fix ant
Anastasia-Gu d5eacf6
fix ant
Anastasia-Gu 481ebaa
implement bound variables for Mathsat5QuantifiedFormulaManager
Anastasia-Gu ec1a17c
implement quantified formula generation for Mathsat5QuantifiedFormula…
Anastasia-Gu 2dae218
remove quantifiedFormula initialization and fix for loop
Anastasia-Gu 5e58542
initial commit for UltimateEliminator support for Yices2 and Bitwuzla
Anastasia-Gu 0d163bc
add UltimateEliminatorWrapper to AbstractQuantifiedFormulaManager
Anastasia-Gu def1a38
add UltimateEliminatorWrapper
Anastasia-Gu 188bc9e
Improve Mathsat5 quantifier support
e3d2614
put FormulaManager and mkWithoutQuantifier in AbstractQuantifiedFormu…
Anastasia-Gu 100cb72
format code with ant
Anastasia-Gu 29c1b30
fix mkWithoutQuantifier
Anastasia-Gu 02a7e6c
revert
Anastasia-Gu d13c401
fix CI errors
Anastasia-Gu 729e37d
fix CI errors
Anastasia-Gu acaf7da
Rework setup of QuantifierManagerTest so that the setup does not bloc…
3e318c0
Implement a method for Yices2 that allows the creation of bound varia…
3d4b036
Rework Yices2 quantifier creation with bound variables
25be000
Add Yices2 quantifier support in the manager
5ba98cd
Remove Boolector quantifier support (as it can never be supported)
f9e4c5a
Remove Boolector from QuantifierManagerTest
64a70f7
Add bound variables to Yices2 visitor
9a256e9
Disable quantifier tests that Yices2 can't handle
a787c54
Add visitor case for Yices2 quantifiers
d84820f
Use full import list instead of * in Yices2FormulaCreator
efc25cf
Move Yices2 free variable creation method to native test class and di…
96aeb4d
Move Yices2 free and bound variable creation method to creator and ca…
1c96551
Merge branch 'fix_yices2_quantifier' into remove_boolector_quantifiers
75085ff
Format
bf75c61
Merge release 5.0.1 with fix for Yices2 quantifiers and Boolector qua…
6bcc200
add UltimateEliminator support to Princess
Anastasia-Gu 4dd3133
adjust Yices2 to previous changes to AbstractQuantifiedFormulaManager
Anastasia-Gu 0b3175e
fix mkWithoutQuantifier in AbstractQuantifiedFormulaManager
Anastasia-Gu 741b6ca
add some tests and exclude unsupported Solvers
Anastasia-Gu dbd8f66
implement eliminateQuantifiersUltimateEliminator in AbstractQuantifie…
Anastasia-Gu 347d7ad
improve error handling of eliminateQuantifiers
Anastasia-Gu 9d834c9
add tests for solver-independent quantifier elimination
Anastasia-Gu 7200f60
fix some CI issues
Anastasia-Gu a4ad348
refactor eliminateQuantifiers to increase readability
Anastasia-Gu d92ca24
fix warnings
Anastasia-Gu 75bdfc0
revert restriction for dumpFormulaImpl in Mathsat5FormulaManager
Anastasia-Gu 6eae73d
fix CI
Anastasia-Gu 9558318
fix more CI
Anastasia-Gu 0baa28a
fix more more CI
Anastasia-Gu c5af782
shorten method names for checkstyle
Anastasia-Gu 7374a04
remove Mathsat from tests that require native quantification
Anastasia-Gu 71f443d
remove Mathsat from tests that require native quantification
Anastasia-Gu 4032091
remove Mathsat from more tests that require native quantification
Anastasia-Gu 9bead2b
refactor assume messages for tests where Mathsat5 got removed
Anastasia-Gu 822f945
pulled SolverConcurrencyTest from master
Anastasia-Gu 3207380
format code
Anastasia-Gu 7fcae4b
refactor dependencies
Anastasia-Gu dd0c425
remove unused method from UltimateEliminatorWrapper
Anastasia-Gu cdff920
add quantifier elimination tests and refactor method names
Anastasia-Gu 19c675f
remove unused option
Anastasia-Gu fa7ed3e
remove not thrown Exceptions from method signatures
Anastasia-Gu 3c95681
rename logger componentName for UltimateEliminator
Anastasia-Gu 0e3c875
Merge branch 'master' into add_solver_independent_quantifier_eliminat…
Anastasia-Gu ec6d411
Merge branch 'refs/heads/master' into add_solver_independent_quantifi…
Anastasia-Gu c456aea
merge master into branch
Anastasia-Gu f5a14a6
remove unused IOException
Anastasia-Gu cf60709
remove MathSAT5 and Yices2 from quantifier tests
Anastasia-Gu b7bf341
rename ProverOption SOLVER_INDEPENDENT_QUANTIFIER_ELIMINATION_BEFORE …
Anastasia-Gu e973e5f
updated options and option handling
Anastasia-Gu 763f8e1
fix tests
Anastasia-Gu 613eb07
resolve some comments
Anastasia-Gu 7a1e814
fix CI
Anastasia-Gu fb403ff
fix CI
Anastasia-Gu 64d7782
fix CI
Anastasia-Gu 9d86ef4
slf4j-nop.jar to classpath to disable failed job from spotbugs
Anastasia-Gu 8f2b80c
Revert "slf4j-nop.jar to classpath to disable failed job from spotbugs"
Anastasia-Gu db28834
Remove Boolector quantifier support (as it can never be supported)
0baae82
Remove Boolector from quantified tests
7e5766d
Use the delegate to set the options for quantified formula manger in …
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This method does not make sense. A
ProverOption
is clearly meant for the prover.Also, why not just use the integrated configuration framework for options and hand it to the manager from the context?