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

remove unnecessary implicit arguments in lambdas #53

Open
semorrison opened this issue Nov 23, 2021 · 2 comments
Open

remove unnecessary implicit arguments in lambdas #53

semorrison opened this issue Nov 23, 2021 · 2 comments
Labels
bug Something isn't working requires change to Lean 4

Comments

@semorrison
Copy link
Contributor

In Lean 4, when we write a lambda, the implicit arguments are introduced automatically and do not need to be named. mathport will need to remove these arguments.

See for example https://github.com/leanprover-community/mathlib3port/blob/9f61de0/Mathbin/Combinatorics/Quiver.lean#L58,
where in the map field we need to write fun f => f rather than fun X Y f => f.

@semorrison semorrison added the bug Something isn't working label Nov 23, 2021
@dselsam
Copy link
Collaborator

dselsam commented Dec 6, 2021

Synport won't have type information in general. It might be easier to set_option hackport.disableImplicitLambda true on the Lean4 side and then (semi-auto) refactor once things type-check in Lean4.

@semorrison
Copy link
Contributor Author

Just to make sure I understand, you're suggesting that we add this option in Lean 4 core?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working requires change to Lean 4
Projects
None yet
Development

No branches or pull requests

3 participants