-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
integer optimization bug in python #4670
Comments
A QF_LRA formula at commit f370d8d
Seems a bug in smt.arith.solver=6. Because when I add |
For smtlib2 instance by @amchiclet , both smt.arith.solver=6 and smt.arith.solver=2 can return 189 (at commit f370d8d) |
Thanks, I know what the bug is (a regression), but don't have a fix right now |
Signed-off-by: Nikolaj Bjorner <[email protected]>
* fixing #4670 Signed-off-by: Nikolaj Bjorner <[email protected]> * init Signed-off-by: Nikolaj Bjorner <[email protected]> * arrays Signed-off-by: Nikolaj Bjorner <[email protected]> * arrays Signed-off-by: Nikolaj Bjorner <[email protected]> * arrays Signed-off-by: Nikolaj Bjorner <[email protected]> * na Signed-off-by: Nikolaj Bjorner <[email protected]>
This is fixed |
ba5c9c3, the results of arith.solver=2 and =6 seem different
|
I have version 4.12.2.0 and i have the exact same behavior as described by @amchiclet for the same scenario. Is this an expected behaviour? Thanks. |
Z3 version 4.8.8.0
Python version 3.8.2
Here's the code to reproduce the bug.
The code returns 189, when it should be 290.
Changing the constraint
x>=189
tox>=190
gives me 290.Calling
opt.maximize(x)
twice gives me 290.Removing any constraint except the first two, gives me 290.
The rise4fun web interface returns 290 with this code.
Possibly a bug in the Python port.
The text was updated successfully, but these errors were encountered: