Dish array merge main - #1
Conversation
Fixed the markup and updated the h5py URL. Also removed the `setup.py`-based install suggestion.
merging ss/angpos-ext branch
|
@Arnab-half-blood-prince have you checked that simulations at different pointing elevations come out OK after merging the new changes? |
|
Yes. So far, I checked CAS-A. It seems to be at the right position and with right flux value in Jy. |
|
TAU-A and CYG-A are next to check. |
|
OK, great. I think if it looks good we can probably merge it but I'm totally unfamiliar with the changes in the commit history pre-November 2025. |
|
Yes, I think the bulk of changes are made by Liam at the main branch, where many changes happened within caput, cora etc. |
|
These are CYG-A and TAU-A. I think the simulation is working. 
|
|
@Arnab-half-blood-prince Thanks for carrying out this important work. I trust your tests above that show that these changes are fine. As we've discussed elsewhere, it would be great to eventually merge the disharray code into the upstream |
Fix exception message formatting for polarization check.
- Rename drift/telescope/custom_disharray/ to drift/telescope/disharray/, replacing the old simple disharray.py module - Rename CustomDishArray mixin to DishArrayMixin throughout - Preserve old DishArray class in disharray/core.py for backward compatibility - Remove HIRAX-specific code (hirax.py, hextile data file, manager.py entries) per instrument-agnostic policy; HIRAX implementation belongs in a dedicated instrument repo
beam_circular computed 2*jinc(x) = J0(x)+J2(x) = 2*J1(x)/x, which is identical to airy_beam in beams.py. Remove the redundant _jinc and beam_circular helpers and call airy_beam/pointing_offset_separation directly, also dropping the now-unused jn and spherical imports.
5 participants
This branch is updated with the latest changes in
caput,cora.I also merged
ss/anspos-extto this branch, as that gives an accurate telescope sim for different declinations.