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

Adjunctions (notions of adjunctions) #32

Merged
merged 5 commits into from
Sep 20, 2023
Merged

Adjunctions (notions of adjunctions) #32

merged 5 commits into from
Sep 20, 2023

Conversation

emilyriehl
Copy link
Collaborator

I started defining some of the adjunction data enumerated in #26. I'm pushing now so that this can be used for the development of limits and colimits (which is underway). There's a lot of work still to be done!

I attempted to follow the naming conventions of the style guide but would appreciate a second opinion.

As auxiliary functions I also

  • defined a primitive quadruple-comp function in our common file.
  • defined whiskering of natural transformations (as a special case of horizontal composition) in the 2-cats file.
  • defined coherence data I'm referring to by the name of "Gray-interchanger".

The Gray-interchanger is a commutative square, built by gluing two hom2 terms together. I refer to one of these as "left" and the other as "right" for lack of obvious better names.

@fizruk
Copy link
Member

fizruk commented Sep 20, 2023

Looks great! I would only suggest to change ladj and radj into left-adj and right-adj everywhere.

@emilyriehl
Copy link
Collaborator Author

@fizruk it looks like I can't review my own pull request. The new commit just makes the requested notation change.

@fizruk fizruk merged commit 77995fe into main Sep 20, 2023
2 checks passed
@fizruk fizruk deleted the adjunctions branch September 20, 2023 13:56
@fizruk fizruk changed the title Adjunctions Adjunctions (notions of adjunctions) Sep 20, 2023
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