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

Adjoint equivalences preserve colimits #454

Closed
wants to merge 1 commit into from

Conversation

Taneb
Copy link
Member

@Taneb Taneb commented Feb 25, 2025

I feel it should be possible to show a weaker version of this, that if C -| D then R preserves limits and L preserves colimits. c.f. https://ncatlab.org/nlab/show/adjoints+preserve+(co-)limits (which would justify the feeling of the author of Categories.Category.Finite that the right adjoint is enough there)

@Taneb Taneb marked this pull request as ready for review February 25, 2025 08:13
@Taneb
Copy link
Member Author

Taneb commented Feb 25, 2025

I've realised that I've misunderstood something here. This module is not preserving limits, but preserving the diagram of limits. Not sure how I want to proceed (but it explains why I kept getting things backwards)

@JacquesCarette
Copy link
Collaborator

The code all seems fine, but I get from your comment that this isn't ready to merge? Or this is, but additional code that you were thinking of doing isn't?

@Taneb
Copy link
Member Author

Taneb commented Feb 25, 2025

Issues:

  1. This isn't what "preserves limits" normally means and I shouldn't have renamed the proof that
  2. I don't know a good name for it otherwise that has a corresponding name for the colimit case
  3. I don't really get how this relates to Category.Finite so I don't know how the comment there should be adjusted
  4. The proof doesn't need adjoint equivalence, a simple adjunction will do (I have a proof of this locally, which I can make a PR for once the previous points are resolved)

@Taneb Taneb closed this Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants