-
Notifications
You must be signed in to change notification settings - Fork 73
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
Revamp interpretH
#386
Comments
another approach @googleson78 and I talked about was to stick the machinery from #384 into |
I'm in favour of it. #384 relies on running the interpreter recursively automatically, which |
There's also the question about how such an extension would interact with #239, but my opinion is that we cross that bridge when we come to it. At the very worst, we have to give up unification of |
I think the names should be something like |
interpretH
is:We need to fix it. The most conservative approach would be to implement an alternative
interpretH
-- something like #384 -- that addresses these issues without changinginterpretH
itself; a more radical approach would be to replaceinterpretH
entirely.Either way, this will be part of the ever-expanding list of issues that v2 should cover. We need to do some work on this to figure out what the best alternate interface would be. My proposal is the following:
but that implementation requires an extensive rework of the library's internals. This also shouldn't replace
interpretH
entirely, because there are some powerinterpretH
gains from access to the functorial state that this doesn't have.For v2, we should at the very least implement a simple stop-gap like #384. If we're able to come up with a satisfying replacement for
interpretH
in time for v2, we'll include it; otherwise, we'll have it as a long-term goal.The text was updated successfully, but these errors were encountered: