Repository navigation
Conversation
Contributor
Author
|
I missed #1666, which opened while I was working on the report. We reached the same subst-based fix, so I’m closing this duplicate in favor of that PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
etaadds a lambda around an argument, but annotations inside that argument can still contain fixed levels from before the new binder. In the report, the innerNatannotation ends up pointing at the newBoolvariable.Reopen the argument with the existing
substhelper before wrapping it, preserving variables from the surrounding scope. The three eta callers now pass their current depth. The emitted annotation refers tox2again, and the reported program passes--verdict. Neitherbend.tsnor the Lean kernel changes.Added the reported program as a proof regression. Checked it with both
--check-onlyand--verdict, plus five existing eta/dependent-function files with--verdict; all passed with Lean 4.34.0. The kernel matches the base byte for byte (SHA-256cc78aa9ff3491ff1b788797bdcab4cb98107dcb1730ed99834f6933b11d88335). The repository gate passes without changing caps; the related comments and verdict paragraph are shorter.Fixes #1665.