-
Notifications
You must be signed in to change notification settings - Fork 144
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
Issue1133 #212
Issue1133 #212
Conversation
…nd lots of renaming, comments, ...)
It was included for the change reverted in the previous commit. This reverts commit 50603bb.
Let me suggest a commit message for merging this. @salome-eriksson, is the following clear enough? [issue1133] Fix bug in invariant synthesis and revise its implementation.It was possible that an unbalanced action passed the balance check. This was a conceptual gap in the original algorithm for which we had to significantly revise the invariant synthesis. We used the opportunity to revise wide parts of its implementation, restructuring and renaming the different components to improve clarity. |
Yes that looks good to me :) |
This is a major revision of the translator code for finding invariants, also fixing a bug reported on discord. See issue1133 for details.