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

Recompute uvw during averaging #112

Open
landmanbester opened this issue Jul 26, 2024 · 3 comments
Open

Recompute uvw during averaging #112

landmanbester opened this issue Jul 26, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@landmanbester
Copy link
Collaborator

landmanbester commented Jul 26, 2024

The init app has options to do simple averaging in frequency and time based BDA using the averaging functionality in codex-africanus. If I understand correctly, these do not recompute the uvw coordinates from the averaged time and frequency centroids but approximates them with a Euclidean average (is this correct or is it a weighted average @sjperkins @bennahugo?). We should probably rather recompute them as is done in xova. Since chunks are processed in separate processes and we don't have to write back to the MS it should be possible to do this in parallel

@landmanbester landmanbester added the enhancement New feature or request label Jul 26, 2024
@sjperkins
Copy link
Member

Yes it is a weighted average and technically the UVW coordinates should be recomputed.

@sjperkins
Copy link
Member

In general this is something that I'd like to implement in arcae, but that will have to wait until the data processing churn has quietened down.

@landmanbester
Copy link
Collaborator Author

From my tests it doesn't seem to make a big difference if you are doing a moderate amount of averaging. At least for the L-band MeerKAT data I have been working with setting -max-fov 2.2 and using a bda-dorr 0.98 I don't see any difference in the 2 degree fov I am reconstructing and yet I get a factor of about 5 in compression. Big win so thanks. Low priority but thought I would document it for posterity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants