[Merged by Bors] - feat: Fourier integral of Gaussians on inner product spaces#11035
[Merged by Bors] - feat: Fourier integral of Gaussians on inner product spaces#11035
Conversation
|
I'm not sure I entirely get the argument about renaming – the Fourier transform is given by the integral for L^1 functions, and Gaussians are about the most L^1 function you can imagine! But if you really want to rename the lemmas then go ahead, I'm not fanatically attached to the old names. As for the inner-product space results: would it be significantly harder to treat the more general case of "ellipsoidal" rather than "spherical" Gaussians, i.e. |
|
About the naming, I expect that in the not so distant future we will define the Fourier transform of a general function Let me try to do the ellipsoidal Gaussian. |
|
Ellipsoidal case done. |
|
Looks great! We should probably change the section title to something like "Fourier integral of the Gaussian function" for consistency with the lemma renamings (the original title is clearly a mistake anyway). Other than that I have nothing further to add. |
|
Good remark. I've changed the section title accordingly, to |
|
🚀 Pull request has been placed on the maintainer queue by loefflerd. |
|
bors r+ |
Also rename (and deprecate) a few statements by changing `fourierTransform` to `fourierIntegral`: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.
|
Pull request successfully merged into master. Build succeeded: |
Also rename (and deprecate) a few statements by changing `fourierTransform` to `fourierIntegral`: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.
Also rename (and deprecate) a few statements by changing `fourierTransform` to `fourierIntegral`: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.
Also rename (and deprecate) a few statements by changing `fourierTransform` to `fourierIntegral`: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.
Also rename (and deprecate) a few statements by changing `fourierTransform` to `fourierIntegral`: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.
Also rename (and deprecate) a few statements by changing
fourierTransformtofourierIntegral: the Fourier transform for general L^2 functions is not given by the Fourier integral, so we should separate cleanly the two of them.