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

PDF cleanup: explain sigma and pi types #276

Closed
wants to merge 1 commit into from

Conversation

williamdemeo
Copy link
Collaborator

Description

This addresses a couple of items of #258

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • Code is formatted according to CONTRIBUTING.md
  • Self-reviewed the diff

@williamdemeo williamdemeo force-pushed the william/258-agda-sigma-and-pi branch 7 times, most recently from 173554d to 1ded982 Compare November 2, 2023 07:49
@williamdemeo williamdemeo force-pushed the william/258-agda-sigma-and-pi branch from 1ded982 to 111c8bc Compare November 2, 2023 07:51
@WhatisRT
Copy link
Collaborator

WhatisRT commented Nov 2, 2023

I think we don't have any dependent types in the PDF (and if we do we should get rid of them), so I see no reason to include this in the PDF.

@williamdemeo
Copy link
Collaborator Author

Shouldn't we at least explain Σ, which does appear in the PDF?

@WhatisRT
Copy link
Collaborator

WhatisRT commented Nov 3, 2023

No, we should get rid of them (except if they denote summation)

@williamdemeo
Copy link
Collaborator Author

I'm resurrecting this PR that was rejected when it was initially created over a year ago. Perhaps, in its current state, the explanations here are too detailed even for an appendix. However, it was recently suggested that we should, in fact, explain Sigma types at least, so we might find some of this PR useful.

@WhatisRT
Copy link
Collaborator

I'm closing again this because we first need to have a discussion, which I made #691 for and which links to this PR. It can be reopened if the decision is to keep these in the PDF.

@WhatisRT WhatisRT closed this Feb 18, 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.

2 participants