-
Notifications
You must be signed in to change notification settings - Fork 10
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
Gaussian source modeling #143
Conversation
80db84e
to
eed34d7
Compare
ce89446
to
afb6dc9
Compare
Issue #211 has been made to mark that this is still off by ~10%. I suggest merging it for now until someone can come along and figure out the issue. Does not affect point-source models |
Just checking my understanding since I have not kept up with this discussion. Basically, the setup totally confuses me. If I take a point source and smear it out with a gaussian in image space, this is equivalent to convolving a gaussian centered at zenith with a dirac-delta function located at the position of the point-source. That would indicate to me that in the fourier domain, I should multiply the plane-wave of the point-source by the fourier transform of the image-space gaussian. Based on your March 25 comment, the reverse is being done, i.e. convolving in uv rather than multiplying. Perhaps my premise is incorrect about what the gaussian source modeling setup ought to be. Is it not supposed to be representing an extended source as a sum of gaussians in image space? I scratched some math out and found if you convolve in a plane-wave in uv with a gaussian and then IFT back, you will get a point-source in image space, but the amplitude will be damped by the gaussian (and this gaussian is actually centered at zenith so the damping is strong far away from zenith) rather than smeared by a gaussian located at the point-source. If there were lots of components, I could see how a sum of the damped components might look like a convolution, but be slightly off. Does this change anything? |
@mwilensky768 Not to speak for Nichole here, but I think she misspoke when she said "convolve" in the March 25 comment. The code calculates a plane wave in UV space and then windows (multiplies) by a Gaussian. |
Thanks! I think I see the multiply happening in |
OK, if people are happy to merge this now given the very reasonable performance, I think there just should be a small addition to the documentation that permalinks to this issue as long as it is relevant. That way users can understand what to expect if they try to use this part of the code. Just thinking on the side, no need to investigate this before merging: Units of area in image space can be a source of confusion when imaging. Is it possible that there are two units (say, pixel size and beam) that are very similar in size, and that the normalization is following one unit when it should follow the other? |
We've thought carefully about normalization. One confusing point on normalization: the Gaussian models presented here have an RA extent that is given in what I would call "delta RA" units, corresponding to the change of RA coordinates (in degrees) across the extent of the source. This is not equivalent to the true size of the source in degrees. This is documented in the code. |
Great. Sounds like a mystery for future work. If we can just get some docs (maybe in the keyword dictionary, or examples) that point to this PR, I'd be happy to approve. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just leaving the official review comment here. Requests are mentioned in previous comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, thank you!
6effb8e
to
885a2e7
Compare
Allows model catalogs to include Gaussian shape information for sources or source components. These changes do not affect runs unless the keyword gaussian_source_models is set.