-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add minor comments #93
Conversation
f7a0ab9
to
2566fd6
Compare
@@ -14,6 +14,7 @@ \section{\SpecThreeB{} in Alloy}\label{sec:alloy} | |||
restrict the number of checkpoints and votes to 5 and 12, respectively. | |||
Although we introduced similar restrictions with Apalache, Alloy has even | |||
finer level of parameter tuning. | |||
\rs{Is there some inherent limitation of Apalache that we should discuss?} |
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.
No inherent limitation per se, there is just no finer-grained control that Apalache implements or exposes right now. We actually mention it as a possible extension in §1.3.
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.
Yes, we touched upon that in para 1.3. Beyond that, it is hard to explain the difference without explaining the technique implemented by Alloy.
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 think the proposed changes are good. We should probably drop the remaining footnote.
@konnov @thpani If we want to marge this PR into main, we need to remove the pending comments, but I am not sure whether they have been resolved in other PRs. |
See #100 |
No description provided.