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

Equivalence of conatural types #330

Merged
merged 3 commits into from
Dec 24, 2024

Conversation

Seiryn21
Copy link
Contributor

@Seiryn21 Seiryn21 commented Dec 3, 2024

@martinescardo Is this what you wanted in #309 ?

@martinescardo
Copy link
Owner

Thanks, @Seiryn21 .

Yes, this is what I wanted!

Let me review it in the next few days.

Thanks for your contribution.

Copy link
Owner

@martinescardo martinescardo left a comment

Choose a reason for hiding this comment

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

Thanks! And sorry for the late review - the pull request came at end of term with too many things to deal at work. Yes, this is what I wanted in the TODO. Would you mind explaining a bit more for the benefit of our readers? Short sentences will be enough.

If you prefer, I can accept the pull request as it is, and add comments myself after that, but it is probably better if you do it.


\end{code}

This implementation of CoNat comes from Cubical, the bisimilarity relation can
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe say from the "Cubical Agda Library".

\end{code}

This implementation of CoNat comes from Cubical, the bisimilarity relation can
be proven to be equivalent to equality, but not in classical Agda
Copy link
Owner

Choose a reason for hiding this comment

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

Add full stop to the sentence.


\begin{code}

CoNat' : Set
Copy link
Owner

Choose a reason for hiding this comment

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

Could we use the TypeTopology conventions, for universes, rather than Set?

data _=C'_ x y where
con : x =C'' y → x =C' y

record _=C_ x y where
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe a few more comments would be in order, in the spirit of TypeTopology, which tried (often, but not always) to explain what's going on. I suppose this is the bisimilarity relation aluded to above.

Copy link
Owner

@martinescardo martinescardo left a comment

Choose a reason for hiding this comment

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

Thanks! Happy Holidays.

@martinescardo
Copy link
Owner

Because your pull request is officially "work in progress", I can't merge it.

@Seiryn21 Seiryn21 marked this pull request as ready for review December 24, 2024 22:16
@Seiryn21
Copy link
Contributor Author

Seiryn21 commented Dec 24, 2024

That should be good, and Happy Holidays to you too !

@martinescardo martinescardo merged commit 085ccbc into martinescardo:master Dec 24, 2024
1 check passed
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