-
Notifications
You must be signed in to change notification settings - Fork 156
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
Evaluator.instantiate: Not a tylambda: Lambda ... #2781
Labels
Comments
christiaanb
added a commit
that referenced
this issue
Aug 5, 2024
For some eta-reduced 'e', we used to bogusly eta-expand to: \x.(\y. e y) x We now correctly expand to: \x.\y.(e x) y Fixes #2781
Thanks for the small reproducer, I found the culprit quite quickly: #2782 |
christiaanb
added a commit
that referenced
this issue
Aug 19, 2024
For some eta-reduced 'e', we used to bogusly eta-expand to: \x.(\y. e y) x We now correctly expand to: \x.\y.(e x) y Fixes #2781
christiaanb
added a commit
that referenced
this issue
Aug 26, 2024
For some eta-reduced 'e', we used to bogusly eta-expand to: \x.(\y. e y) x We now correctly expand to: \x.\y.(e x) y Fixes #2781
2 tasks
christiaanb
added a commit
that referenced
this issue
Aug 27, 2024
For some eta-reduced 'e', we used to bogusly eta-expand to: \x.(\y. e y) x We now correctly expand to: \x.\y.(e x) y Fixes #2781 (cherry picked from commit f946617) Co-authored-by: Christiaan Baaij <[email protected]>
We've released v1.8.2, which includes a fix for this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Results in:
When I initially discovery this bug, it was trigged by the addition of the type signature on
ugnsStable
.Which was weird because I understood
ugnsStable
to be monomorphic, and thus the extra type signature to be meaningless, see here.But somewhere in minimizing the code it lost that property and now it always produces the error.
The text was updated successfully, but these errors were encountered: