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

SKA configuration changes #53

Open
garrelt opened this issue Sep 13, 2024 · 4 comments
Open

SKA configuration changes #53

garrelt opened this issue Sep 13, 2024 · 4 comments

Comments

@garrelt
Copy link
Collaborator

garrelt commented Sep 13, 2024

The design of SKA-Low has been changing over the years and the design that's used in tools21cm is probably no longer valid. Specifically, what will be built is called AA* and the baseline design, AA4, may be built in the future, if funding appears.
The most up to date information about SKA design (station positions, etc.) has been made available by SKAO in a python package, see https://www.skao.int/en/science-users/ska-tools/494/ska-staged-delivery-array-assemblies-and-subarrays
It would be good to make use of this in tools21cm.

@micbia
Copy link
Collaborator

micbia commented Dec 10, 2024

I need to double-check if the new configuration they mention is the same that we used during the SKA Data challenge 3a (which was provided by SKAO). If that is the case, I have already prepared the new configuration in a format that can be implemented in tools21cm

@sambit-giri
Copy link
Owner

If they match, please add those files to the src/tools21cm/input_data folder.

@sambit-giri
Copy link
Owner

I have now added the files provided by @micbia to the above folder, which can be access through the function get_SKA_Low_layout.
These changes are related to commit 0d63a9a.

@sambit-giri
Copy link
Owner

As the new files contain the antennae layout in cartesian corrdinates, contrary to the previous configuration in tools21cm that was in geographic coordinates. However, the calculation of uv tracks get_uv_daily_observation converted the geographic coordinates to cartesian. To have the flexibility to switch between the corrdinates, I have introduced two new functions that are geographic_to_cartesian_coordinate_system and cartesian_to_geographic_coordinate_system.

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

3 participants