Skip to content

Conversation

@kroening
Copy link
Collaborator

This enables writing tests that check the length of the witness trace.

@kroening kroening force-pushed the numbered-trace-states branch from 77c730f to cb41335 Compare April 22, 2024 01:12
This enables writing tests that check the length of the witness trace.
@kroening kroening force-pushed the numbered-trace-states branch from cb41335 to f552b6e Compare April 22, 2024 01:15
@kroening kroening marked this pull request as ready for review April 22, 2024 01:16
--bound 10 --numbered-trace
^\[main\.property\.p0\] cover main\.counter == 1: PROVED$
^Trace:$
^Trace with 2 states:$
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here and above: is "2" the only possible case, or could this change with the solver picking a different model? If so, it should perhaps be with \d+ states.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The desired outcome is that this is the only answer, as the shortest counterexample. It's a TODO item to see whether say incremental SAT could do so deterministically.

@kroening kroening merged commit a68e711 into main Apr 22, 2024
@kroening kroening deleted the numbered-trace-states branch April 22, 2024 15:26
Romy15200 pushed a commit to Romy15200/nws that referenced this pull request Aug 19, 2025
numbered trace output now includes number of states
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants