Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

refactor(measure_theory/set_integral): move set integral into namespace set and add some lemmas#1950

Merged
mergify[bot] merged 6 commits intoleanprover-community:masterfrom
aceg00:set_integral
Feb 5, 2020
Merged

refactor(measure_theory/set_integral): move set integral into namespace set and add some lemmas#1950
mergify[bot] merged 6 commits intoleanprover-community:masterfrom
aceg00:set_integral

Conversation

@aceg00
Copy link
Copy Markdown
Collaborator

@aceg00 aceg00 commented Feb 4, 2020

Split from #1944.

TO CONTRIBUTORS:

Make sure you have:

  • reviewed and applied the coding style: coding, naming
  • reviewed and applied the documentation requirements
  • for tactics:
  • make sure definitions and lemmas are put in the right files
  • make sure definitions and lemmas are not redundant

If this PR is related to a discussion on Zulip, please include a link in the discussion.

For reviewers: code review check list

@aceg00 aceg00 changed the title feat(measure_theory/set_integral): move set integral into namespace set and add some lemmas refactor(measure_theory/set_integral): move set integral into namespace set and add some lemmas Feb 4, 2020
@aceg00 aceg00 added blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. awaiting-review The author would like community review of the PR and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Feb 4, 2020
Copy link
Copy Markdown
Collaborator

@sgouezel sgouezel left a comment

Choose a reason for hiding this comment

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

If you define set.measurable_on (which is a very good idea, I think), shouldn't you also define set.integrable_on?

@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Feb 5, 2020
@aceg00 aceg00 added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Feb 5, 2020
@sgouezel sgouezel added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Feb 5, 2020
@mergify mergify bot merged commit 8786ea6 into leanprover-community:master Feb 5, 2020
butterthebuddha pushed a commit to butterthebuddha/mathlib that referenced this pull request May 15, 2020
…ce set and add some lemmas (leanprover-community#1950)

* move set integral into namespace set and add some lemmas

* Update bochner_integration.lean

* better theorem names

* Update set_integral.lean

* Update set_integral.lean

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
butterthebuddha pushed a commit to butterthebuddha/mathlib that referenced this pull request May 16, 2020
…ce set and add some lemmas (leanprover-community#1950)

* move set integral into namespace set and add some lemmas

* Update bochner_integration.lean

* better theorem names

* Update set_integral.lean

* Update set_integral.lean

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request May 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants