RadioAstronomySoftwareGroup / RadioAstronomySoftwareGroup/pyuvdata
uvbeam fails to read CST files with proper-case headers and does not appear to handle directivities/e-field amplitudes in dBi
@aewallwi is already working on this.
Since Aug 9, 2018.
- Dominant language
- Python
- Stars
- 102
- Forks
- 35
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 14
Description
cst_beam fails to read the following header from a standard CST output file
Theta [deg.] Phi [deg.] Abs(Dir.)[dBi ] Abs(Theta)[dBi ] Phase(Theta)[deg.] Abs(Phi )[dBi ] Phase(Phi )[deg.] Ax.Ratio[dB ]
------------------------------------------------------------------------------------------------------------------------------------------------------
This appears to be for two reasons
- The [dBi] unit indicators are not accomodated by read_cst_beam
- read_cst_beam assumes that all column names are lower case.
read_cst_beam should not break on the [dBi ] tags, and read in the beam with dBi units and should also allow for arbitrary cases.
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.
Assessment
This issue has not been assessed yet.