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

Consider not caching varctx #109

Open
jonsterling opened this issue Feb 26, 2018 · 0 comments
Open

Consider not caching varctx #109

jonsterling opened this issue Feb 26, 2018 · 0 comments

Comments

@jonsterling
Copy link
Collaborator

Right now, constructing and deconstructing ABTs is very expensive; there is basically no way to really optimize the ABT interface (this was the conclusion of Tom 7 many years ago), and it is fairly hopeless generally. But with that said, we can try making some changes.

For instance, we currently cache the calculation of free variables; while this may have been an optimization at one point, I think that it may be pretty devastating now. In RedPRL today, there are only two calls to calculate the free variables. One of these occurs between every rule, but by slightly changing RedPRL, it should be semantically correct to remove that call.

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 a pull request may close this issue.

1 participant