RadioAstronomySoftwareGroup / RadioAstronomySoftwareGroup/pyuvdata
rename the `az_za_simple` interpolation function on UVBeam to be explicit about using RectalinearBivariateSpline
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 102
- Forks
- 35
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 14
Description
Now that there's another (similar) option, should be more explicit about the name. This change needs to be done with deprecation.
Maybe we should also consider changing the default interpolation method for az_za pixel coordinates to use the faster az_za_map_coordinates option?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the UVBeam az_za_simple and az_za_map_coordinates entry points and review how interpolation methods are selected for az_za pixel coordinates. Check the project's deprecation process and existing usages before choosing the explicit replacement name. Done means the old name is deprecated and the replacement is clear; any default change should be separately resolved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data, performance
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100