Skip to content

Conversation

@shhyou
Copy link
Contributor

@shhyou shhyou commented Nov 11, 2025

The functions Function.Related.TypeIsomorphism.⊎-comm and Data.Sum.Properties.swap-↔ are the same, thus this PR points one to the other. Function.Related.TypeIsomorphism.∃∃↔∃∃ is similar as well.

To make things parallel, this PR keeps only the name of Function.Related.TypeIsomorphism.×-comm and moves its implementation to Data.Product.Properties.

Finally, this PR adds _,′-↔_ : A ↔ C → B ↔ D → (A × B) ↔ (C × D) that pairs two type isomorphisms.

(The type of _,′-↔_ can't be expressed as _×_ Preserves₂ _↔_ ⟶ _↔_ ⟶ _↔_ because that requires A and C to have identical Levels and similarly B and D would need to have the same Level.)

Copy link
Contributor

@JacquesCarette JacquesCarette left a comment

Choose a reason for hiding this comment

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

I was afraid (from the description) that this might make a mess of the dependency graph. But it seems not - great.

So this looks like a nice simplification.

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