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

Extend fftvis to allow per-antenna beams #32

Open
tyler-a-cox opened this issue Mar 18, 2025 · 1 comment
Open

Extend fftvis to allow per-antenna beams #32

tyler-a-cox opened this issue Mar 18, 2025 · 1 comment

Comments

@tyler-a-cox
Copy link
Owner

There is nothing in the current implementation of fftvis that would prevent us from simulating arrays with different beams, so we should probably make a PR to allow users to supply per-antenna beams. This doesn't scale very well the number of unique beams. We'd need to do a total of N_beams * (N_beams + 1) / 2 NUFFTs, which definitely does not scale as well as matvis, but still may be faster than matvis if the number of independent beams is small.

@steven-murray
Copy link
Collaborator

Totally agree. I think we could do a check in the setup and raise a warning if the number of beams is "large" to say that this is not efficient and a different simulator should be used.

@tyler-a-cox tyler-a-cox changed the title Extend fftvis to allow multiple beams Extend fftvis to allow per-antenna beams Mar 18, 2025
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