-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Array Model Output #2704
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
Comments
Models are now compressed. K(Int, 100) is a constant array with value 100. It is the same as [else -> 100]. Since 41 also results in 100, there is no need to list both 41 -> 100 and else -> 100. |
Yea I can see that makes sense, but I wonder if there is a way to get a clean output for an array when more constraints are added. For example
In the older version getting a model for this yields, where I can easily see all elements and indices:
Whereas the new version yields:
Perhaps this is me being silly, but I find the earlier format easier to read and easily assess the state of my array, whereas the newer version requires more unpacking. Is there any way to getting a more easily readable string output, or do I basically need to make a custom parser to do that now? Thanks. |
The new model you display definitely looks convoluted.
I get the following output:
It isn't as convoluted, but you could argue that you prefer to untangle the store sequence into a lookup-table format. |
In another note, you can obtain uncompressed behavior by setting:
It is a design flaw that there are two aliases for the same effect. |
That's awesome. I think this is exactly what I was looking for. Thanks |
Signed-off-by: Nikolaj Bjorner <[email protected]>
This issue exposed that there are two parameters with overlapping functionality.
|
I'm using the python api. I was previously on version 4.5.1.0.post2 and upgraded to version 4.8.6.0. The model output for arrays is drastically different, and I'm not sure how to interpret it. I'm used to seeing something of the form:
But now the output is:
How do I get an output that has index/value pairs like the prior version?
The text was updated successfully, but these errors were encountered: