Skip to content

[Merged by Bors] - feat: port Data.SetLike.Basic#993

Closed
j-loreaux wants to merge 8 commits intomasterfrom
j-loreaux/data.setlike.basic
Closed

[Merged by Bors] - feat: port Data.SetLike.Basic#993
j-loreaux wants to merge 8 commits intomasterfrom
j-loreaux/data.setlike.basic

Conversation

@j-loreaux
Copy link
Copy Markdown
Contributor

@j-loreaux j-loreaux commented Dec 13, 2022

mathlib3 SHA: fc2ed6f838ce7c9b7c7171e58d78eaf7b438fb0e

porting notes:

  1. on SetLike.exists and SetLike.forall, the statement had to be massaged from ∃ x ∈ p, q ⟨x, ‹_›⟩ to ∃ (x : B) (h : x ∈ p), q ⟨x, ‹_›⟩. Is this a bug? Answer: No, this is because ∃ x ∈ p is now sugar for ∃ x, x ∈ p ∧ ... instead of ∃ x, ∃ (h : x ∈ p), ...
  2. some lemmas were tagged with @[mono] but we don't have this attribute yet. I removed them but left porting notes with "TODO" to add them back.

@j-loreaux j-loreaux added awaiting-review mathlib-port This is a port of a theory file from mathlib. labels Dec 13, 2022
@kim-em kim-em added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 13, 2022
@j-loreaux j-loreaux removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 13, 2022
@gebner
Copy link
Copy Markdown
Member

gebner commented Dec 13, 2022

The @[nolint dangerousInstance] attribute don't seem to be necessary, but I have left them there for now. I'm not sure why the linter doesn't trip if these are removed.

Please remove every @[nolint dangerousInstance] attribute. I've added support for out-params to the linter, so when it complains it usually means there's a real issue. And ping me if the linter still complains.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 14, 2022

After reviewing Gabriel's comments,

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Dec 14, 2022

✌️ j-loreaux can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Dec 14, 2022
@kim-em kim-em added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 14, 2022
@mcdoll mcdoll removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 14, 2022
@j-loreaux
Copy link
Copy Markdown
Contributor Author

bors merge

bors bot pushed a commit that referenced this pull request Dec 14, 2022
mathlib3 SHA: fc2ed6f838ce7c9b7c7171e58d78eaf7b438fb0e

porting notes:
1. on `SetLike.exists` and `SetLike.forall`, the statement had to be massaged from `∃ x ∈ p, q ⟨x, ‹_›⟩` to `∃ (x : B) (h : x ∈ p), q ⟨x, ‹_›⟩`. Is this a bug? Answer: No, this is because `∃ x ∈ p` is now sugar for `∃ x, x ∈ p ∧ ...` instead of `∃ x, ∃ (h : x ∈ p), ...`
2. some lemmas were tagged with `@[mono]` but we don't have this attribute yet. I removed them but left porting notes with "TODO" to add them back.

Co-authored-by: Moritz Doll <moritz.doll@googlemail.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 14, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Data.SetLike.Basic [Merged by Bors] - feat: port Data.SetLike.Basic Dec 14, 2022
@bors bors bot closed this Dec 14, 2022
@bors bors bot deleted the j-loreaux/data.setlike.basic branch December 14, 2022 17:04
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 15, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.euclidean_domain.basic`: leanprover-community/mathlib4#919
* `algebra.field.basic`: leanprover-community/mathlib4#975
* `algebra.group.opposite`: leanprover-community/mathlib4#912
* `algebra.group.prod`: leanprover-community/mathlib4#968
* `algebra.group.with_one.units`: leanprover-community/mathlib4#955
* `algebra.group_power.ring`: leanprover-community/mathlib4#979
* `algebra.hom.equiv.type_tags`: leanprover-community/mathlib4#943
* `algebra.hom.ring`: leanprover-community/mathlib4#958
* `algebra.invertible`: leanprover-community/mathlib4#930
* `algebra.order.field.canonical.basic`: leanprover-community/mathlib4#1018
* `algebra.order.field.canonical.defs`: leanprover-community/mathlib4#985
* `algebra.order.field.inj_surj`: leanprover-community/mathlib4#1017
* `algebra.order.group.densely_ordered`: leanprover-community/mathlib4#956
* `algebra.order.group.min_max`: leanprover-community/mathlib4#933
* `algebra.order.group.prod`: leanprover-community/mathlib4#1026
* `algebra.order.group.with_top`: leanprover-community/mathlib4#946
* `algebra.order.hom.monoid`: leanprover-community/mathlib4#944
* `algebra.order.monoid.prod`: leanprover-community/mathlib4#987
* `algebra.order.monoid.to_mul_bot`: leanprover-community/mathlib4#1024
* `algebra.order.ring.abs`: leanprover-community/mathlib4#929
* `algebra.order.ring.cone`: leanprover-community/mathlib4#935
* `algebra.order.sub.basic`: leanprover-community/mathlib4#936
* `algebra.order.sub.with_top`: leanprover-community/mathlib4#932
* `algebra.order.with_zero`: leanprover-community/mathlib4#903
* `combinatorics.quiver.arborescence`: leanprover-community/mathlib4#997
* `combinatorics.quiver.cast`: leanprover-community/mathlib4#990
* `control.traversable.lemmas`: leanprover-community/mathlib4#948
* `data.bool.set`: leanprover-community/mathlib4#960
* `data.int.cast.field`: leanprover-community/mathlib4#1016
* `data.int.cast.lemmas`: leanprover-community/mathlib4#995
* `data.int.cast.prod`: leanprover-community/mathlib4#1015
* `data.int.div`: leanprover-community/mathlib4#1011
* `data.int.dvd.basic`: leanprover-community/mathlib4#996
* `data.int.order.basic`: leanprover-community/mathlib4#938
* `data.nat.cast.basic`: leanprover-community/mathlib4#980
* `data.nat.cast.prod`: leanprover-community/mathlib4#1010
* `data.nat.cast.with_top`: leanprover-community/mathlib4#1019
* `data.nat.gcd.basic`: leanprover-community/mathlib4#965
* `data.nat.order.basic`: leanprover-community/mathlib4#907
* `data.nat.order.lemmas`: leanprover-community/mathlib4#927
* `data.nat.set`: leanprover-community/mathlib4#961
* `data.nat.upto`: leanprover-community/mathlib4#1020
* `data.pequiv`: leanprover-community/mathlib4#1008
* `data.set.basic`: leanprover-community/mathlib4#892
* `data.set.bool_indicator`: leanprover-community/mathlib4#988
* `data.set.image`: leanprover-community/mathlib4#949
* `data.set.n_ary`: leanprover-community/mathlib4#969
* `data.set.opposite`: leanprover-community/mathlib4#983
* `data.set.prod`: leanprover-community/mathlib4#1004
* `data.set.sigma`: leanprover-community/mathlib4#982
* `data.set_like.basic`: leanprover-community/mathlib4#993
* `data.two_pointing`: leanprover-community/mathlib4#984
* `logic.embedding.set`: leanprover-community/mathlib4#986
* `logic.equiv.embedding`: leanprover-community/mathlib4#1021
* `order.directed`: leanprover-community/mathlib4#963
* `order.rel_iso.set`: leanprover-community/mathlib4#1005
* `order.well_founded`: leanprover-community/mathlib4#970
* `ring_theory.coprime.basic`: leanprover-community/mathlib4#899



Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). mathlib-port This is a port of a theory file from mathlib.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants