-
Notifications
You must be signed in to change notification settings - Fork 234
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
[Merged by Bors] - feat: port CategoryTheory.Adjunction.Lifting #4414
Conversation
Mathbin -> Mathlib fix certain import statements move "by" to end of line add import to Mathlib.lean
Oh dear! It's an inscrutable block of rewrites that goes bad part way through. @b-mehta, any chance you'd be able to look at this? |
As
which is pretty much what you would expect from the definition. In Lean 4 the type is totally different: it's
and so after
but I couldn't get the statement to typecheck. If there is just some quick way of making |
….Adjunction.Lifting' into port/CategoryTheory.Adjunction.Lifting
Or just bull ahead naively... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
Co-authored-by: Matthew Ballard <matt@mrb.email> Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Co-authored-by: Matthew Ballard <matt@mrb.email> Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
Co-authored-by: Matthew Ballard <matt@mrb.email> Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
todo:
constructLeftAdjoint
errorthe error likely involves
constructLeftAdjointEquiv_apply
and@[simps (config := { rhsMd := semireducible })]