You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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
changed the title
Extend fftvis to allow multiple beams
Extend fftvis to allow per-antenna beams
Mar 18, 2025
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.The text was updated successfully, but these errors were encountered: