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

Allow multiple instances of the same correspondence. #2

Open
PetrosPapapa opened this issue May 17, 2019 · 1 comment
Open

Allow multiple instances of the same correspondence. #2

PetrosPapapa opened this issue May 17, 2019 · 1 comment
Labels
feature New feature or request low priority We are not in any rush to address this

Comments

@PetrosPapapa
Copy link
Member

See ede9a5f

This still does not allow multiple copies of the same
correspondence (though it is not clear why one would ever want this).

When using inference rules, process names (such as "TimesProc") must
be constant. Otherwise they get refreshed with the latest counter like
every other variable (e.g. "TimesProc8").

I feel there is a better way to do this than adding a prefix for each
correspondence, but this will require some rethinking of the embedding
and quite a bit of refactoring.

@PetrosPapapa PetrosPapapa added low priority We are not in any rush to address this feature New feature or request labels May 17, 2019
@PetrosPapapa
Copy link
Member Author

PetrosPapapa commented May 17, 2019

What if process definitions are polymorphic to the 2 required types? Would this work? This would also fix the mess that is the Cll_terms module as we would simply instantiate the definition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request low priority We are not in any rush to address this
Projects
None yet
Development

No branches or pull requests

1 participant