-
Notifications
You must be signed in to change notification settings - Fork 70
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
Bifibrations #184
Merged
Merged
Bifibrations #184
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
plt-amy
reviewed
Feb 1, 2023
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.
Okay hang on I need to actually render the webpage because that paragraph with all the maths broke my internal TeX
plt-amy
reviewed
Feb 1, 2023
plt-amy
requested changes
Feb 1, 2023
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.
I'm still not done reviewing...
plt-amy
requested changes
Feb 1, 2023
ncfavier
reviewed
Feb 2, 2023
plt-amy
force-pushed
the
bifibrations
branch
from
February 24, 2023 04:47
017218e
to
14703dc
Compare
plt-amy
approved these changes
Feb 24, 2023
plt-amy
added a commit
that referenced
this pull request
Mar 21, 2023
# Description This PR refactors (co)limits to be defined in terms of kan extensions. The main motivation for doing so is to make preservation of limits less annoying to deal with; as it currently stands we need to define special functions for mapping cones, which makes life much harder. ## Notes As noted on discord, representability does not work due to (annoying!) universe level issues. This PR is also based off #184; I'll rebase when the time comes. --------- Co-authored-by: Amélia Liao <me@amelia.how>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR defines bifibrations, and characterizes them as cartesian fibrations with left adjoints to base change.
Doing so cleanly requires a minor refactor of some properties of weak cartesian morphisms, along with some new
definitions:
This PR also adds the (start) of a natural isomorphism API; we should improve this in the future!
Notes
I couldn't come up with a better name for
push-out
. Suggestions would be appreciated!