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

Fiber displayed category. #1164

Closed
wants to merge 2 commits into from
Closed

Fiber displayed category. #1164

wants to merge 2 commits into from

Conversation

anuyts
Copy link
Contributor

@anuyts anuyts commented Oct 29, 2024

No description provided.

@anuyts anuyts marked this pull request as ready for review October 29, 2024 12:30
@maxsnew
Copy link
Collaborator

maxsnew commented Oct 29, 2024

This can be constructed compositionally using displayed total categories, reindexing and Displayed.Instances.Path. I've already made a similar definition for the fiber up to isomorphism downstream here: https://github.com/maxsnew/cubical-categorical-logic/blob/main/Cubical/Categories/Displayed/Constructions/IsoFiber/Base.agda#L44 . The compositional definition would be preferable because then we get more code reuse.

Also it seems like you're adding a lot of small things to the (displayed) categories library lately, which is great. I think we should have a chat e.g., on Agda zulip about your formalization plans so we can coördinate our efforts a bit better

@anuyts anuyts marked this pull request as draft October 29, 2024 16:07
@anuyts
Copy link
Contributor Author

anuyts commented Oct 29, 2024

Ah yes, I see, using a displayed total category is cleaner indeed.

@jpoiret
Copy link
Contributor

jpoiret commented Feb 3, 2025

Should this PR be closed then?

@anuyts
Copy link
Contributor Author

anuyts commented Feb 3, 2025

Suppose yes, I won't be coming back to it anytime soon.

@anuyts anuyts closed this Feb 3, 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.

3 participants