Skip to content

Define Dagger categories #1201

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

Define Dagger categories #1201

wants to merge 45 commits into from

Conversation

@anshwad10
Copy link
Contributor

@anshwad10 anshwad10 commented Mar 2, 2025 •
edited
Loading

To do:

  • Define dagger categories
  • Define univalent dagger categories
  • Define dagger functors
  • Define the binary product of dagger categories
  • Define the category of dagger functors between two dagger categories
  • Prove that a dagger category is equivalent to its opposite
  • Typecheck

@anshwad10 anshwad10 marked this pull request as draft March 2, 2025 12:17
@anshwad10 anshwad10 marked this pull request as ready for review March 2, 2025 17:41
@anshwad10 anshwad10 marked this pull request as draft March 3, 2025 12:19
@anshwad10 anshwad10 marked this pull request as ready for review March 4, 2025 14:25
@anshwad10
Copy link
Contributor Author

anshwad10 commented Jul 23, 2025

Somebody please review this, I've been waiting for months. @ecavallo? @mortberg?

@ecavallo ecavallo self-assigned this Jul 23, 2025
ecavallo
ecavallo requested changes Jul 23, 2025
View reviewed changes
Copy link
Collaborator

@ecavallo ecavallo left a comment

Choose a reason for hiding this comment

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

Contents look good. I have some minor comments mostly about naming.

A question about conventions. Is there any logic to when you use Dagger (e.g. DaggerStr) vs Dag (e.g. DagCat) vs † vs D in names? If not, can we pick one (†?) to use in all names?

@anshwad10 anshwad10 requested a review from ecavallo July 25, 2025 16:30
@anshwad10
Copy link
Contributor Author

anshwad10 commented Jul 25, 2025

oops I was trying to see what was the requested change and accidentally clicked on re-request review.

@anshwad10
Copy link
Contributor Author

anshwad10 commented Aug 20, 2025

@ecavallo I've made the changes you requested (though GitHub is not showing it for some reason) can this be merged now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ecavallo ecavallo Awaiting requested review from ecavallo

Assignees

@ecavallo ecavallo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anshwad10 @ecavallo