agda / agda-categories

A new Categories library for Agda
https://agda.github.io/agda-categories
MIT License
362 stars 68 forks source link

Removing a redundant field in CartesianClosed/Canonical #350

Closed MiddleAdjunction closed 2 years ago

MiddleAdjunction commented 2 years ago

In the canonical definition of CCCs, the field curry-resp-≈ is redundant and can be defined as a function in the record. I personally cannot see a reason justifying the redundancy but it might be there to improve definitional equality in some way (similar to introducing the redundant sym-assoc as opposed to a function using assoc), in which case there possibly needs to be a comment nearby explaining such reason.

sstucki commented 2 years ago

LGTM! When I wrote this module I didn't realize this assumption was redundant. I also don't see any good reason to keep it.