10. Ensure changes to use multiple primary beams don't break existing code/tests

JLBLine / WODEN

CUDA code designed to simulate interferometric data using point, Gaussian, and shapelet sky models

https://woden.readthedocs.io/en/latest/

Mozilla Public License 2.0

5 stars 0 forks source link

10. Ensure changes to use multiple primary beams don't break existing code/tests #47

Closed JLBLine closed 2 months ago

JLBLine commented 2 months ago

Given what I'm planning in this milestone, likely many tests will need woden_settings->use_dipamps = 0 added in various place

JLBLine commented 2 months ago

Working on this as I go through the other issues on this Milestone; think I need to change num_ants in anything to do with the beam to num_beams as duplication/misuse of num_ants is getting confusing.

JLBLine commented 2 months ago

Changed a few names of things from two_antennas to multi_beams and a few instances of num_ants to num_beams to get some clarity between how many antennas there are vs how many beams. Pushed in 58afb81a6f7d6cf4c5e2594ab0ce2305069f4e54

JLBLine commented 2 months ago

I fixed these as I broke them during this milestone. As of 9ba45363b02add3841581df12dfba66cea7146ea there are 97 tests called by ctest and they all pass.