Skip to content

[Merged by Bors] - feat: the Fourier transform is self-adjoint#10833

Closed
sgouezel wants to merge 3 commits intomasterfrom
SG_cleanup_fourier
Closed

[Merged by Bors] - feat: the Fourier transform is self-adjoint#10833
sgouezel wants to merge 3 commits intomasterfrom
SG_cleanup_fourier

Conversation

@sgouezel
Copy link
Copy Markdown
Contributor

Also extend the notation 𝓕 for the Fourier transform to inner product spaces, introduce a notation 𝓕⁻ for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.


Open in Gitpod

@sgouezel sgouezel added awaiting-review t-analysis Analysis (normed *, calculus) labels Feb 21, 2024
@loefflerd loefflerd self-requested a review February 24, 2024 20:40
Copy link
Copy Markdown
Contributor

@loefflerd loefflerd left a comment

Choose a reason for hiding this comment

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

Looks great! I like your custom notation 𝐞 as well – the only change I have to suggest is a couple of further places where the notation can be used. It's a trivial thing, so feel free to ignore if you disagree. Either way, definitely a positive review from me.

Co-authored-by: David Loeffler <d.loeffler.01@cantab.net>
@sgouezel
Copy link
Copy Markdown
Contributor Author

Thanks for the review!

Copy link
Copy Markdown
Member

@jcommelin jcommelin 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 awaiting-review labels Feb 28, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 28, 2024
Also extend the notation `𝓕` for the Fourier transform to inner product spaces, introduce a notation `𝓕⁻` for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 28, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: the Fourier transform is self-adjoint [Merged by Bors] - feat: the Fourier transform is self-adjoint Feb 28, 2024
@mathlib-bors mathlib-bors bot closed this Feb 28, 2024
@mathlib-bors mathlib-bors bot deleted the SG_cleanup_fourier branch February 28, 2024 07:39
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
Also extend the notation `𝓕` for the Fourier transform to inner product spaces, introduce a notation `𝓕⁻` for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.
kbuzzard pushed a commit that referenced this pull request Mar 12, 2024
Also extend the notation `𝓕` for the Fourier transform to inner product spaces, introduce a notation `𝓕⁻` for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
Also extend the notation `𝓕` for the Fourier transform to inner product spaces, introduce a notation `𝓕⁻` for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
Also extend the notation `𝓕` for the Fourier transform to inner product spaces, introduce a notation `𝓕⁻` for the inverse Fourier transform, and generalize a few results from the real line to inner product spaces.
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-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants