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

Question: RA_max / DEC_max limited to pixel grid? #178

Open
Jordatious opened this issue Mar 30, 2022 · 2 comments
Open

Question: RA_max / DEC_max limited to pixel grid? #178

Jordatious opened this issue Mar 30, 2022 · 2 comments

Comments

@Jordatious
Copy link

Hi all, I note from the docs that when gaussian components are combined, "the maximum of the source" is stored (presumably as RA_max and DEC_max), as well as the centroid. Does the maximum refer to the (interpolated) position of the brightest gaussian component, or simply the position of the brightest pixel?

@darafferty
Copy link
Collaborator

darafferty commented Apr 6, 2022

I had to check the code, but I believe the maximum generally corresponds to the position of a Gaussian fit to the region around the maximum of the source (see https://github.com/lofar-astron/PyBDSF/blob/master/bdsf/gaul2srl.py#L356 where the fit is done). So it should not be limited to the pixel grid.

@Jordatious
Copy link
Author

Thanks for the info @darafferty. My use case here is identifying the central (brightest) position of bright outliers. Currently, I'm taking the position of the brightest Gaussian component within a small cutout as my position of interest. But it sounds like this is the same value as RA_max and DEC_max from the source catalogue. But perhaps the source grouping parameters affect this.

My current workflow is fine, but I wanted to know if the maximum was fixed to the pixel grid. You've answered that, so feel free to close the ticket, but any other clarity you can provide would be helpful!

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

2 participants