-
Notifications
You must be signed in to change notification settings - Fork 218
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
Refactor abstract consensus specification #6475
Conversation
lemmy
commented
Sep 12, 2024
•
edited
Loading
edited
- Refactor (simplify) TypeOK to use (more concise) Sequences!Seq operator.
- Define InitialLogs less explicitly.
- Add assumptions about abs.tla's constants (TLAPS will likely need those assumptions should we attempt writing a proof).
- Mitigate state-space explosion during explicit-state model checking by declaring constant Servers to be symmetric.
@lemmy could you add a short description to the PR capture its goal(s)? The overall impression I get from the commit is that this is trying to simplify things, but it adds a net 34 lines. |
I propose not to squash the commits. |
14d63f5
to
80fc286
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good for this to be merged once the ci is green.
I would suggest double checking the impact of refinement checking on the ci times for consensus with reconfiguration once its finishing properly
Signed-off-by: Markus Alexander Kuppe <github.com@lemmster.de>
Signed-off-by: Markus Alexander Kuppe <github.com@lemmster.de>
Signed-off-by: Markus Alexander Kuppe <github.com@lemmster.de>
Signed-off-by: Markus Alexander Kuppe <github.com@lemmster.de>
I removed all commits with semantic changes from this PR. Will continue with those in a separate PR, after this has moved through the merge queue. |