Skip to content

[Merged by Bors] - feat(SetTheory/ZFC/Basic): order instances#19946

Closed
vihdzp wants to merge 8 commits intomasterfrom
vi.preorder
Closed

[Merged by Bors] - feat(SetTheory/ZFC/Basic): order instances#19946
vihdzp wants to merge 8 commits intomasterfrom
vi.preorder

Conversation

@vihdzp
Copy link
Copy Markdown
Collaborator

@vihdzp vihdzp commented Dec 13, 2024

Sets form a partial order under the subset relation.


Open in Gitpod

@vihdzp vihdzp added the t-set-theory Set theory label Dec 13, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 13, 2024

PR summary 132d7a7b21

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.SetTheory.ZFC.Basic 391 393 +2 (+0.51%)
Import changes for all files
Files Import difference
Mathlib.SetTheory.ZFC.Basic 2

Declarations diff

+ instance : HasSSubset PSet := ⟨(· < ·)⟩
+ instance : HasSSubset ZFSet := ⟨(· < ·)⟩
+ instance : IsNonstrictStrictOrder PSet (· ⊆ ·) (· ⊂ ·)
+ instance : IsNonstrictStrictOrder ZFSet (· ⊆ ·) (· ⊂ ·)
+ instance : Preorder PSet
+ instance : SetLike ZFSet ZFSet
++ le_def
++ lt_def

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

Copy link
Copy Markdown
Contributor

@alreadydone alreadydone left a comment

Choose a reason for hiding this comment

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

Thanks!
maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by alreadydone.

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

@alreadydone alreadydone left a comment

Choose a reason for hiding this comment

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

Apparently we can introduce a SetLike instance (on ZFSet, not on PSet) and then the PartialOrder from SetLike.instPartialOrder is defeq to this. (There's a PR that demotes SetLike.instPartialOrder from an instance to an abbrev (I suppose) though.)

@vihdzp
Copy link
Copy Markdown
Collaborator Author

vihdzp commented Jan 13, 2025

I guess you're right... ZFC sets are in fact kind of like sets :P

I'll rework this PR to implement that instance instead.

@vihdzp vihdzp added awaiting-author A reviewer has asked the author a question or requested changes. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Jan 13, 2025
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 27, 2025
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Apr 3, 2025
@vihdzp vihdzp removed the awaiting-author A reviewer has asked the author a question or requested changes. label Apr 3, 2025
@vihdzp vihdzp requested a review from b-mehta April 3, 2025 06:06
@alreadydone
Copy link
Copy Markdown
Contributor

Thanks
maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 8, 2025

🚀 Pull request has been placed on the maintainer queue by alreadydone.

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

@b-mehta b-mehta 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 maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels May 9, 2025
mathlib-bors bot pushed a commit that referenced this pull request May 9, 2025
Sets form a partial order under the subset relation.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented May 9, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(SetTheory/ZFC/Basic): order instances [Merged by Bors] - feat(SetTheory/ZFC/Basic): order instances May 9, 2025
@mathlib-bors mathlib-bors bot closed this May 9, 2025
@mathlib-bors mathlib-bors bot deleted the vi.preorder branch May 9, 2025 16:18
tannerduve pushed a commit that referenced this pull request May 13, 2025
Sets form a partial order under the subset relation.
bwehlin pushed a commit to bwehlin/mathlib4 that referenced this pull request May 31, 2025
Sets form a partial order under the subset relation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-set-theory Set theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants