Skip to content
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(analysis/convolution): weaken topological assumptions #18012

Closed
wants to merge 10 commits into from

Conversation

sgouezel
Copy link
Collaborator

We redefine locally_integrable: currently, this means "integrable on every compact set", and we change it to "integrable on a neighborhood of any point", to bring it in line with is_locally_finite_measure.

This makes it possible to weaken a lot of assumptions in the file on convolutions, removing second countability or local compactness assumptions.


This is related to #17626, but there is no direct dependence in one direction or the other. Whichever is merged first, I will adapt the other one.

Open in Gitpod

@sgouezel sgouezel requested a review from a team as a code owner December 25, 2022 15:32
@sgouezel sgouezel added awaiting-review The author would like community review of the PR t-analysis Analysis (normed *, calculus) labels Dec 25, 2022
Copy link
Member

@fpvandoorn fpvandoorn left a comment

Choose a reason for hiding this comment

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

Very nice! LGTM

bors d+

@bors
Copy link

bors bot commented Jan 4, 2023

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

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Jan 4, 2023
@sgouezel
Copy link
Collaborator Author

sgouezel commented Jan 5, 2023

bors r+

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Jan 5, 2023
bors bot pushed a commit that referenced this pull request Jan 5, 2023
We redefine `locally_integrable`: currently, this means "integrable on every compact set", and we change it to "integrable on a neighborhood of any point", to bring it in line with `is_locally_finite_measure`.

This makes it possible to weaken a lot of assumptions in the file on convolutions, removing second countability or local compactness assumptions.
@bors
Copy link

bors bot commented Jan 5, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(analysis/convolution): weaken topological assumptions [Merged by Bors] - feat(analysis/convolution): weaken topological assumptions Jan 5, 2023
@bors bors bot closed this Jan 5, 2023
@bors bors bot deleted the SG_locally_integrable branch January 5, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) t-analysis Analysis (normed *, calculus)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants