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

consolidate injectivity predicate #2025

Merged

Conversation

Alizter
Copy link
Collaborator

@Alizter Alizter commented Jul 15, 2024

In this patch we create a new file Basics/Classes.v which contain some basic classes. In the future we will move some more stuff from canonical_names.v and abstract_algebra.v here that should be more widely available.

For now we introduce the IsInjective predicate more generally for everybody to use and adapt existing proofs to use this.

In this patch we create a new file Basics/Classes.v which contain some
basic classes. In the future we will move some more stuff from
canonical_names.v and abstract_algebra.v here that should be more widely
available.

For now we introduce the IsInjective predicate more generally for
everybody to use and adapt existing proofs to use this.

Signed-off-by: Ali Caglayan <[email protected]>

<!-- ps-id: 7040e298-dbab-4af6-92e2-40604acaac49 -->
Copy link
Collaborator

@jdchristensen jdchristensen left a comment

Choose a reason for hiding this comment

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

Looks good. If some more things move here in the future, maybe some files won't need to depend on the Classes library.

@Alizter Alizter merged commit a3315b0 into HoTT:master Jul 16, 2024
22 checks passed
@Alizter Alizter deleted the ps/rr/consolidate_injectivity_predicate branch July 16, 2024 19:49
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