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

Flattening lemma for sequential colimits #972

Conversation

VojtechStep
Copy link
Collaborator

@VojtechStep VojtechStep commented Dec 5, 2023

Resolves #869

@fredrik-bakke
Copy link
Collaborator

You could call it a 2-cell of morphisms of arrows. Prisms would then be triangles of morphisms of arrows.

@VojtechStep
Copy link
Collaborator Author

VojtechStep commented Dec 5, 2023

Oh right, I think what I want is a htpy-hom-arrow https://unimath.github.io/agda-unimath/foundation.morphisms-arrows.html#homotopies-of-morphsims-of-arrows
(and there's a typo in that heading)

@VojtechStep VojtechStep marked this pull request as draft December 5, 2023 20:59
@VojtechStep
Copy link
Collaborator Author

May I just say that rephrasing the whole property from "two coforks with a weird coherence" to "a cofork in the category of morphisms" saved a whole 1 (one!) line, and it's now more conceptual.

Although wrestling the data into shape is a little awkward, so I'll still tweak with it a bit.

@VojtechStep VojtechStep marked this pull request as ready for review December 7, 2023 16:13
@VojtechStep
Copy link
Collaborator Author

I'm pretty happy with the proof, so I'm opening the PR for reviews

@VojtechStep VojtechStep force-pushed the feature/flattening-lemma-sequential-colimits branch 2 times, most recently from 0143711 to 06a3bec Compare December 8, 2023 20:33
Copy link
Collaborator

@fredrik-bakke fredrik-bakke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here are some initial comments from me. I'll see if I have more detailed comments/comments about the actual code tomorrow. Excellent work! ⭐️

@VojtechStep VojtechStep force-pushed the feature/flattening-lemma-sequential-colimits branch from 466a701 to 1da86f6 Compare December 12, 2023 15:58
@fredrik-bakke
Copy link
Collaborator

Here are some initial comments from me. I'll see if I have more detailed comments/comments about the actual code tomorrow. Excellent work! ⭐️

Apologies, time flew today and now I don't think I will have time to review this today. Maybe I can find a little bit of time tomorrow.

@VojtechStep
Copy link
Collaborator Author

Don't worry about it, it's gonna take some time before I actually need this stuff.

@fredrik-bakke
Copy link
Collaborator

Don't worry about it, it's gonna take some time before I actually need this stuff.

I can't help it. 😅 I think this PR is good to merge whenever you are ready.

@VojtechStep VojtechStep enabled auto-merge (squash) December 12, 2023 19:07
@VojtechStep VojtechStep force-pushed the feature/flattening-lemma-sequential-colimits branch from 1da86f6 to 23d46f6 Compare December 12, 2023 19:08
@VojtechStep VojtechStep merged commit c468e60 into UniMath:master Dec 18, 2023
4 checks passed
@VojtechStep VojtechStep deleted the feature/flattening-lemma-sequential-colimits branch January 21, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flattening lemma for sequential colimits
3 participants