-
Notifications
You must be signed in to change notification settings - Fork 217
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
Changing index into Configurations
in MCccfraft.tla
#5779
Conversation
MCccfraft.tla
MCccfraft.tla
Configurations
in MCccfraft.tla
tla-mcconfigs@77905 aka 20231025.60 vs main ewma over 20 builds from 77080 to 77900 Click to see tablemain
tla-mcconfigs
|
This change which should slow down the exhaustive model checking, is making |
Mystery solved, I needed to increase the |
Can we remove |
Yes, I believe we can. I'll try this refactor in a separate PR |
I believe that there is currently an issue in
MCccfraft.tla
where the first configuration inConfigurations
is ignored. Currently, the initial configuration is any subset of servers and then the next configuration is the 2nd set of servers inConfigurations
and so on.I'm not sure yet if this is the best fix (verse using a separate initial state in MCccfraft.tla).I think this is simplest fix so the best for nowThis PR is related to (but separate from) #5778