Skip to content

[Merged by Bors] - feat(Mathlib/Order/Ideal): ideals of sets are stable under directed unions and chains#12651

Closed
samvang wants to merge 5 commits intomasterfrom
idealsunion
Closed

[Merged by Bors] - feat(Mathlib/Order/Ideal): ideals of sets are stable under directed unions and chains#12651
samvang wants to merge 5 commits intomasterfrom
idealsunion

Conversation

@samvang
Copy link
Copy Markdown
Contributor

@samvang samvang commented May 4, 2024

We prove that the directed union of a non-empty directed union of ideals of subsets of a preorder is an ideal, and in particular ideals of subsets of a preorder are stable under unions. In order to prove it, we also add directed_on_sUnion, a variant of directed_on_iUnion.


Open in Gitpod

@github-actions github-actions bot added the new-contributor This PR was made by a contributor with at most 5 merged PRs. Welcome to the community! label May 4, 2024
@Ruben-VandeVelde Ruben-VandeVelde added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels May 4, 2024
@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

And I should say: thanks for the contribution!

@samvang
Copy link
Copy Markdown
Contributor Author

samvang commented May 5, 2024

Thanks very much for the help @Ruben-VandeVelde and @dagurtomas! I implemented your comments just now.

I also did "Resolve conversation" for each comment and changed back to "awaiting-review". I hope this is the correct workflow?

@samvang samvang added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels May 5, 2024
Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

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

Yes, that's the correct workflow. Thanks!

@samvang
Copy link
Copy Markdown
Contributor Author

samvang commented May 6, 2024

I just added the deprecation notice as per @Ruben-VandeVelde 's suggestion.

Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

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

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 7, 2024

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label May 7, 2024
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels May 7, 2024
mathlib-bors bot pushed a commit that referenced this pull request May 7, 2024
…nions and chains (#12651)

We prove that the directed union of a non-empty directed union of ideals of subsets of a preorder is an ideal, and in particular ideals of subsets of a preorder are stable under unions. In order to prove it, we also add `directed_on_sUnion`, a variant of `directed_on_iUnion`.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented May 7, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Mathlib/Order/Ideal): ideals of sets are stable under directed unions and chains [Merged by Bors] - feat(Mathlib/Order/Ideal): ideals of sets are stable under directed unions and chains May 7, 2024
@mathlib-bors mathlib-bors bot closed this May 7, 2024
@mathlib-bors mathlib-bors bot deleted the idealsunion branch May 7, 2024 15:21
apnelson1 pushed a commit that referenced this pull request May 12, 2024
…nions and chains (#12651)

We prove that the directed union of a non-empty directed union of ideals of subsets of a preorder is an ideal, and in particular ideals of subsets of a preorder are stable under unions. In order to prove it, we also add `directed_on_sUnion`, a variant of `directed_on_iUnion`.
callesonne pushed a commit that referenced this pull request May 16, 2024
…nions and chains (#12651)

We prove that the directed union of a non-empty directed union of ideals of subsets of a preorder is an ideal, and in particular ideals of subsets of a preorder are stable under unions. In order to prove it, we also add `directed_on_sUnion`, a variant of `directed_on_iUnion`.
mathlib-bors bot pushed a commit that referenced this pull request May 16, 2024
…d distributive lattice (#12705)

We prove that a disjoint filter and ideal in a bounded distributive lattice can always be separated by a prime ideal. This is a step towards establishing Stone duality (currently in development here: https://github.com/samvang/StoneDualityInLean/)

- [x] depends on: #12651 (ideals of sets are stable under directed unions and chains)
grunweg pushed a commit that referenced this pull request May 17, 2024
…d distributive lattice (#12705)

We prove that a disjoint filter and ideal in a bounded distributive lattice can always be separated by a prime ideal. This is a step towards establishing Stone duality (currently in development here: https://github.com/samvang/StoneDualityInLean/)

- [x] depends on: #12651 (ideals of sets are stable under directed unions and chains)
callesonne pushed a commit that referenced this pull request Jun 4, 2024
…d distributive lattice (#12705)

We prove that a disjoint filter and ideal in a bounded distributive lattice can always be separated by a prime ideal. This is a step towards establishing Stone duality (currently in development here: https://github.com/samvang/StoneDualityInLean/)

- [x] depends on: #12651 (ideals of sets are stable under directed unions and chains)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. new-contributor This PR was made by a contributor with at most 5 merged PRs. Welcome to the community! ready-to-merge This PR has been sent to bors. t-order Order theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants