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

Localization algebra #1007

Merged
merged 9 commits into from
Jul 10, 2023
Merged

Localization algebra #1007

merged 9 commits into from
Jul 10, 2023

Conversation

jpoiret
Copy link
Contributor

@jpoiret jpoiret commented Jun 5, 2023

Hi,

This is an attempt to define the localization of an algebra at an arbitrary multiplicatively closed subset of it, compared with the current localization at a multiplicatively closed subset of the base ring. I've had many attempts, trying to use CT and the fact that algebras over a base ring are equivalent to rings under a base ring, but the definitional behavior was really hard to predict and things just didn't compute well. I chose instead to just extend the existing construction in the most straightforward way. I'm still trying to formulate a recursion principle using the universal property, so that we're able to prove things about the localization without resorting to its actual construction.

This depends on a rebased version of #931.

LMKWYT

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.

Discussed in person.

Cubical/Algebra/CommAlgebra/LocalisationAlgebra.agda Outdated Show resolved Hide resolved
Cubical/Algebra/CommAlgebra/LocalisationAlgebra.agda Outdated Show resolved Hide resolved
Cubical/Algebra/CommAlgebra/LocalisationAlgebra.agda Outdated Show resolved Hide resolved
@felixwellen
Copy link
Collaborator

Checking the library took 47min in the CI, we should investigate.

@felixwellen
Copy link
Collaborator

Both master and this branch take 27min on my machine, so it should be fine.

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.

LGTM - just a request for comment.

@jpoiret jpoiret force-pushed the localization-algebra branch from fc4fcab to 54050ab Compare July 6, 2023 08:23
@jpoiret jpoiret force-pushed the localization-algebra branch from 54050ab to 29b7fb5 Compare July 7, 2023 08:47
@felixwellen
Copy link
Collaborator

Great - thanks!

@felixwellen felixwellen merged commit ac9635f into agda:master Jul 10, 2023
@jpoiret jpoiret deleted the localization-algebra branch July 10, 2023 08:59
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.

3 participants