Skip to content
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

Remove support for Attribute change via DPO #77

Merged
merged 2 commits into from
Jul 19, 2024
Merged

Conversation

kris-brown
Copy link
Collaborator

@kris-brown kris-brown commented Jul 19, 2024

This addresses #76

This restricts the kinds of rules one may create in order to have better-defined behavior:

  1. Left legs of DPO (and SPO) rules must be monic, including attribute variables
  2. Variables in $R$ are exclusively either a. in the image of $I\rightarrow R$ or b. assigned an expression to be evaluated at runtime (or c. freevar=true, in which case AttrVars will appear in the result)

Variables in $R$ which are in the image of $I \rightarrow R$ have a unique concrete value associated with them via the match morphism $L \rightarrow G$: that value is preserved in the rewrite.

Extra work now is needed in case one wants to change the value of an attribute. E.g. one can delete the object and recreate it for the time being.

@kris-brown kris-brown linked an issue Jul 19, 2024 that may be closed by this pull request
@kris-brown kris-brown self-assigned this Jul 19, 2024
@kris-brown kris-brown merged commit 35fabeb into main Jul 19, 2024
7 of 8 checks passed
@kris-brown kris-brown deleted the fix_attrvar_pc branch July 19, 2024 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove support for 'changing attribute values' via a DPO rewrite
1 participant