Skip to content

Topological modalities #1125

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

Merged
merged 7 commits into from
May 13, 2024
Merged

Conversation

awswan
Copy link
Contributor

@awswan awswan commented May 10, 2024

I needed some results about topological modalities for a project, and I think it makes sense to add these to the library in case anyone else wants to use them. I'm not 100% sure about the naming, so feel free to change if necessary. There are a couple of naming clashes that come from the literature, so there's not much I can do about them: 1. Separated is a general definition that applies to any modality, whereas in the library (and many places in the literature) it is specific to the double negation modality; 2. So far Injective in the library is only referring to functions, I think, but I also use it for injective objects, as in https://ncatlab.org/nlab/show/injective+object .

@felixwellen felixwellen self-assigned this May 13, 2024
Copy link
Collaborator

@felixwellen felixwellen left a comment

Choose a reason for hiding this comment

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

Just a typo and a renaming suggestion, otherwise ready to merge. Thanks for contributing!

@awswan
Copy link
Contributor Author

awswan commented May 13, 2024

Thanks! I've made both those changes.

@felixwellen felixwellen merged commit 5b96b28 into agda:master May 13, 2024
1 check passed
@awswan awswan deleted the topological-modalities branch May 13, 2024 09:27
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