-
Notifications
You must be signed in to change notification settings - Fork 15
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
Translate coe constants #116
Comments
I've actually been using |
They are not equivalent, since variable (n : Nat)
#check ( (·) (n + n) : Int) -- (fun a => a) (Int.ofNat n + Int.ofNat n) : Int
#check ((↑·) (n + n) : Int) -- (fun a => Int.ofNat a) (n + n) : Int |
Oh yeah, if it's applied then it's a different story. In that case we probably don't want to use either one and instead should use |
See leanprover-community/mathport#116 (comment) The notation is a bit problematic for the pretty-printer, as it doesn't automatically insert parentheses to disambiguate it from the built-in coercion notation. Alternatives: 1. Use a different token. 2. Magically define `(↑ ∙)` to be the eta-reduction of `fun x => ↑x`. PS: the old macro for `↑ x` was redundant since it's been moved to core some time ago.
We also need to translate the |
I tried to do this using The issue is that the |
That is, translate
coe
to(↑ ⬝)
. AndcoeFn
to(· ·)
.The text was updated successfully, but these errors were encountered: