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 by Bors] - feat(algebraic_geometry/prime_spectrum/basic): intersection of localisations #16860
Conversation
Multramate
commented
Oct 8, 2022
•
edited by github-actions
bot
edited by github-actions
bot
- depends on: [Merged by Bors] - feat(algebraic_geometry/prime_spectrum/maximal): maximal spectrum #16905 [define maximal spectrum]
- depends on: [Merged by Bors] - feat(ring_theory/dedekind_domain/ideal): height_one_spectrum is equivalent to maximal_spectrum #16920 [refactor height one spectrum]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you add the maximal spectrum, I think it needs to come with a lot of API about its relation to the prime spectrum: at a minimum the inclusion map. Maybe also the characterisation of the image as the subset of closed points.
At a later stage, maybe the Zariski topology on the maximal spectrum should be added, and then of course the continuity of the inclusion map.
Yes, I plan to make a basic API right after this, but on a separate PR so as to not cloud this one - is that fine? I also plan to refactor them into a separate file in the same folder called |
Is the content of this PR urgent? If not, then I would prefer that you swap the order of those two PRs. |
I made a basic PR for the maximal spectrum, but I'm not sure if the Zariski topology definition is ideal here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that the comment applies to all three lemmas.
Thanks! |
🚀 Pull request has been placed on the maintainer queue by erdOne. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
bors merge
Pull request successfully merged into master. Build succeeded: |