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

Characterization of Rezk Completion #841

Merged
merged 21 commits into from
Aug 11, 2022
Merged

Conversation

kangrongji
Copy link
Contributor

@kangrongji kangrongji commented Jun 9, 2022

This PR contains the definition of Rezk completion, using its universal property, together with the characterization that a weak equivalence towards univalent category is a Rezk completion functor.
(see Cubical.Categories.RezkCompletion.Base)

The existence of Rezk completion will be given in a future PR.

@mortberg mortberg self-requested a review August 10, 2022 14:04
Copy link
Collaborator

@mortberg mortberg left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this contribution! And for splitting the Rezk completion into manageable pieces in different PRs. I'll review and merge them one by one.

PS: apologies for the slow reviews!

@@ -70,6 +70,9 @@ module _ (C : Category ℓC ℓC') (D : Category ℓD ℓD') where
Iso→NatIso α .trans = α .fst
Iso→NatIso α .nIso = FUNCTORIso' _ (α .snd)

NatIso→Iso : {F G : Functor C D} → NatIso F G → CatIso FUNCTOR F G
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is kind of a general comment, but wouldn't it be better if lemmas about CatIso would be called something with CatIso and not just Iso? So this would be NatIso→CatIso

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I change it to FUNCTORIso so it's more informative.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(Look not very pretty though...)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah, the all caps names are not so nice... We should discuss naming convention for CT in an issue

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. Then that's for it now.

Cubical/Categories/RezkCompletion/Base.agda Outdated Show resolved Hide resolved
Cubical/Categories/RezkCompletion/Base.agda Outdated Show resolved Hide resolved
Cubical/Categories/RezkCompletion/Base.agda Outdated Show resolved Hide resolved
Cubical/Foundations/HLevels.agda Show resolved Hide resolved
Copy link
Collaborator

@mortberg mortberg left a comment

Choose a reason for hiding this comment

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

Great! Ready to be merged as soon as the CI is gives us a green light

@kangrongji
Copy link
Contributor Author

Thanks!

@mortberg mortberg merged commit e791bc0 into agda:master Aug 11, 2022
@kangrongji kangrongji deleted the rezk-universal branch August 11, 2022 19:43
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.

None yet

2 participants