Skip to content
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

Yices error when invoking sally #53

Open
yav opened this issue Oct 5, 2018 · 2 comments
Open

Yices error when invoking sally #53

yav opened this issue Oct 5, 2018 · 2 comments

Comments

@yav
Copy link

yav commented Oct 5, 2018

Hello,

Consider the following trivial transition system:

(define-state-type S () ())
(define-transition-system TS S (= 1 2) (= 1 1))
(query TS (= 1 1))

When I run sally (master, git commit 0b88fe9) I get:

Yices error (push): the context state does not allow operation

I am guessing that this has something to do with the initial set of states being empty.

@dddejan
Copy link
Member

dddejan commented Oct 9, 2018

Yes, this is not the best error to report.

Just to clarify, which do you think it's better to report:

  • system is valid,
  • system is in deadlock,
  • system is valid with warning "might be deadlock".

@yav
Copy link
Author

yav commented Oct 12, 2018

I am not really sure. As far as I understand, the meaning of query is to check if the property holds in all reachable states described by the system, so I guess valid seems reasonable, given that there are no states.

I haven't played around with sally's support for deadlock detection yet, but perhaps if one of the check-deadlock flags is enabled, it would make sense to also report a deadlock as there is certainly no next-possible state. What seems odd about that is that if we wanted to show a trace that leads to the deadlock, the trace would have to be empty.

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

No branches or pull requests

2 participants