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

Fix summation limit N for explicit solutions #9

Closed
hagenw opened this issue Jan 18, 2016 · 6 comments
Closed

Fix summation limit N for explicit solutions #9

hagenw opened this issue Jan 18, 2016 · 6 comments

Comments

@hagenw
Copy link
Member

hagenw commented Jan 18, 2016

As Till reported in his mail:

In the expansions of the basis functions (4), (5), (6), the usage of N as limit of the summation is misleading as it suggests a finite number of basis functions.
The definition of a Fredholm operator states that the kernel has finite dimensions and its mapping should be infinite.

=> N should be infinite

@hagenw
Copy link
Member Author

hagenw commented Jan 18, 2016

I have looked into [MorseFeshbach1981] -- it should be page 940, not eq. 940! --, there the trick is that no summation limit at all is mentioned, but simply $\sum_{n}$.

Should I change it to the same, or better replace N by (infinity)?

A related question would be if the functions are also correct for the planar and linear case where we don't have a summation, but an integral as the volume is a non compact space.

@fs446
Copy link
Member

fs446 commented Jan 18, 2016

Should I change it to the same, or better replace N by ∞ (infinity)?

change to inf would be good and consistent with other HOA related stuff so far.

A related question would be if the functions are also correct for the planar and linear case where we don't have a summation, but an integral as the volume is a non compact space.

Compact operators and non-compact ones such as the linear/planar deconvolution must and should be treated independently.

@trettberg
Copy link

Should I change it to the same, or better replace N by ∞ (infinity)?

Matter of taste, I guess.
Personally, I like $\sum_{n \in \mathbb{Z}}$ for the abstract case.
When working in a particular space (e.g. circular or spherical harmonics), switch to whatever indexing is convenient.

I completely agree that the planar (and linear) case should be treated independently.

@hagenw
Copy link
Member Author

hagenw commented Jan 19, 2016

Should I change it to the same, or better replace N by ∞ (infinity)?

Matter of taste, I guess.
Personally, I like $\sum_{n \in \mathbb{Z}}$ for the abstract case.
When working in a particular space (e.g. circular or spherical harmonics), switch to whatever indexing is convenient.

Good point, the actual summation indices can be from 0 to \inf or from -\inf to \inf depending on the basis function, correct?
I solved the problem by using $\sum_n$ and state in the description that $n \in \mathbb{Z}$.

I also added the same three equations with \int instead of \sum for the non compact case, see:
http://kkdu.org/explicit_solution/theory/index.html#solution-for-special-geometries-nfc-hoa-and-sdm

There I'm not sure if it is ok to just write dk as integration index, or is there a better option. Are the integration boundaries always -\inf to \inf? At the moment I have omitted them.

@trettberg
Copy link

Caveat: I'm not an expert.

There I'm not sure if it is ok to just write dk as integration index

Probably not. In the abstract setting, we don't know where k lives. We could possibly assume a mesurable space and integrate w.r.t. the measure $\mu$.

Does the mathematical formulation for general infinite domains exist in the SFS literature?
If not then this may be a can of worms...

An option would be to ditch the abstract case altogether and adopt a more "hands-on" signal processing perspective:
Then NFCHOA and SDM are deconvolutions (inverse filtering), carried out in appropriate transform domains.

@hagenw
Copy link
Member Author

hagenw commented Apr 27, 2016

I decided to go with the abstract measure $\mu$. At the moment I still prefer the mathematical approach to motivate the deconvolution. But if we find a better way, we could change it later one. For now I will close this issue.

hagenw added a commit that referenced this issue Apr 27, 2016
@hagenw hagenw closed this as completed in 7188ebf Apr 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants