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

eliminate all but one way of replacing Rule context #7

Closed
mscarey opened this issue Mar 31, 2019 · 1 comment
Closed

eliminate all but one way of replacing Rule context #7

mscarey opened this issue Mar 31, 2019 · 1 comment

Comments

@mscarey
Copy link
Owner

mscarey commented Mar 31, 2019

currently there's Procedure.new_context() and the entire concept of a Holding object.

Probably I need to stick with Procedure.new_context(). The Holding object at first glance seems to link all Holdings with the same meaning to the same ProceduralRule object, but realistically, there's no way to prevent multiple ProceduralRule objects with the same meaning from being created, so the Holdings class just adds a confusing abstraction that doesn't solve the deduping problem or any other problem.

@mscarey
Copy link
Owner Author

mscarey commented Apr 3, 2019

Eliminated Holding class 03638a0

@mscarey mscarey closed this as completed Apr 3, 2019
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

No branches or pull requests

1 participant