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: cleanup file on derivative of Fourier transform #11779

Closed
wants to merge 7 commits into from

Conversation

sgouezel
Copy link
Contributor

Rename mul_L to fourierSMulRight for predictability, expand the file-level docstring, extract a few lemmas from proofs, change fourier_integral to fourierIntegral in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.


I haven't deprecated the definitions and lemmas in the file FourierTransformDeriv when I have changed their names, since as far as I can tell they are not used elsewhere yet. If you think it's important to deprecate them properly, tell me and I'll do it.

Open in Gitpod

@sgouezel sgouezel added awaiting-review The author would like community review of the PR t-analysis Analysis (normed *, calculus) labels Mar 29, 2024
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Copy link
Contributor

@ocfnash ocfnash 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 d+

Mathlib/Analysis/Fourier/FourierTransformDeriv.lean Outdated Show resolved Hide resolved
Mathlib/Analysis/Fourier/FourierTransformDeriv.lean Outdated Show resolved Hide resolved
Mathlib/Analysis/Fourier/FourierTransformDeriv.lean Outdated Show resolved Hide resolved
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 1, 2024

✌️ 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-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Apr 1, 2024
sgouezel and others added 2 commits April 1, 2024 19:47
Co-authored-by: Oliver Nash <github@olivernash.org>
@sgouezel
Copy link
Contributor Author

sgouezel commented Apr 1, 2024

bors r+

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Apr 1, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 1, 2024
Rename `mul_L` to `fourierSMulRight` for predictability, expand the file-level docstring, extract a few lemmas from proofs, change `fourier_integral` to `fourierIntegral` in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 1, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: cleanup file on derivative of Fourier transform [Merged by Bors] - feat: cleanup file on derivative of Fourier transform Apr 1, 2024
@mathlib-bors mathlib-bors bot closed this Apr 1, 2024
@mathlib-bors mathlib-bors bot deleted the SG_cleanup_fourier2 branch April 1, 2024 18:36
Louddy pushed a commit that referenced this pull request Apr 15, 2024
Rename `mul_L` to `fourierSMulRight` for predictability, expand the file-level docstring, extract a few lemmas from proofs, change `fourier_integral` to `fourierIntegral` in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.
atarnoam pushed a commit that referenced this pull request Apr 16, 2024
Rename `mul_L` to `fourierSMulRight` for predictability, expand the file-level docstring, extract a few lemmas from proofs, change `fourier_integral` to `fourierIntegral` in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
Rename `mul_L` to `fourierSMulRight` for predictability, expand the file-level docstring, extract a few lemmas from proofs, change `fourier_integral` to `fourierIntegral` in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.
callesonne pushed a commit that referenced this pull request Apr 22, 2024
Rename `mul_L` to `fourierSMulRight` for predictability, expand the file-level docstring, extract a few lemmas from proofs, change `fourier_integral` to `fourierIntegral` in a few lemma names. There is essentially no new mathematical content. This is a preparation for the study of higher order derivatives in #11776.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated ready-to-merge This PR has been sent to bors. t-analysis Analysis (normed *, calculus)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants