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

Discrete cocartesian fibrations #458

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Discrete cocartesian fibrations #458

wants to merge 5 commits into from

Conversation

TOTBWF
Copy link
Collaborator

@TOTBWF TOTBWF commented Jan 17, 2025

Description

This PR refactors our discrete cartesian fibration module a bit, and implements discrete cocartesian fibrations.

Notes

I've renamed Discrete-fibration to is-discrete-cartesian-fibration, though I'm not 100% sold on this: the name is
quite long! However, it seems like the best option. The name really should be an is- name, as being a discrete fibration is a property of a displayed category.

Checklist

Before submitting a merge request, please check the items below:

  • I've read the contributing guidelines.
  • The imports of new modules have been sorted with support/sort-imports.hs (or nix run --experimental-features nix-command -f . sort-imports).
  • All new code blocks have "agda" as their language.

If your change affects many files without adding substantial content, and
you don't want your name to appear on those pages (for example, treewide
refactorings or reformattings), start the commit message and PR title with chore:.

@TOTBWF TOTBWF requested review from ncfavier and plt-amy January 17, 2025 14:47
@Lavenza
Copy link
Member

Lavenza commented Jan 17, 2025

@ncfavier
Copy link
Member

ncfavier commented Jan 17, 2025

@Lavenza is missing the added page. EDIT: https://github.com/the1lab/1lab-preview-server/pull/1

src/Cat/Displayed/Cartesian/Discrete.lagda.md Outdated Show resolved Hide resolved
src/Cat/Displayed/Cartesian/Discrete.lagda.md Outdated Show resolved Hide resolved
src/Cat/Displayed/Cartesian/Discrete.lagda.md Outdated Show resolved Hide resolved
src/Cat/Displayed/Cartesian/Discrete.lagda.md Outdated Show resolved Hide resolved
src/Cat/Displayed/Cartesian/Discrete.lagda.md Outdated Show resolved Hide resolved
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