-
Notifications
You must be signed in to change notification settings - Fork 24
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
Renaming structures to more suitable names #194
Comments
A couple of thoughts I have: Validity predicates
Action circuit In essence Action circuit is the circuit that checks that the taiga rules are being preserved, so the most obvious option would be Taiga circuit. Taiga rules - taiga circuit. However, it might feel somewhat redundant and maybe we should have a more functional specifier, for example, Regulation Circuit or Taiga Policy Circuit. I don't particularly like any of these options, but Taiga circuit feels like a good compromise. Predicate vs circuit Action circuit is global and public, it is written once for all users of Taiga. Predicates are written many times, we are even developing a whole toolchain for that. Having action circuit not called a predicate emphasizes this difference as well, but maybe we should come up with a name referring to this fact explicitly. It doesn't matter that much that the action circuit is implemented as a circuit, so we can use other word that refers to the rule checking activity. Often such structures are stable enough, so this will hint to the static nature of the Action circuit. For example, we could call it TransitionPolicyChecker or TaigaPolicyChecker. A TransitionPolicyChecker/TaigaPolicyChecker takes the state transition expressed as input and output notes and checks that it is valid. The output would be a TransitionPolicyProof/TaigaPolicyProof |
This is an issue for discussing what various structures should be renamed to and why. Later in time we will have a meeting, discuss all of the terms and points, and rename the structures. This issue doesn't include renaming caused by higher-level specs naming mismatch, an issue for that already exists
Starting with:
The text was updated successfully, but these errors were encountered: