Closed JLBLine closed 2 months ago
After discussions with the community, it will be clearer to ditch the LINPOL_ZERO_I
option, and instead explicitly set the linear polarisation flux. With this option, the Stokes I is completely ignored for linear polarisation. Adding the columns below should make things explicit. Will need code down the line to respect this.
Column name | Units | Description |
---|---|---|
LIN_NORM_COMP_PL | Jy | (Optional) The linear polarisation reference flux for a power-law pl component model, must be at the reference frequency 200MHz |
LIN_ALPHA_PL | NA | (Optional) The linear polarisation spectral index for a power-law pl component model, must be at the reference frequency 200MHz. |
LIN_NORM_COMP_CPL | Jy | (Optional) The linear polarisation reference flux for a curved power-law cpl component model, must be at the reference frequency 200MHz. |
LIN_ALPHA_CPL | NA | (Optional) The linear polarisation spectral index for a curved power-law cpl component model, must be at the reference frequency 200MHz. |
LIN_CURVE_CPL | NA | (Optional) The linear polarisation curvature q for a curved power-law cpl component model |
Aight, with 5f16b6460891d1d7e2096edebf55297b352dea80 the testing code that writes out example FITS files cmake_testing/wodenpy/skymodel/fits_skymodel_common.write_full_test_skymodel_fits
can now write out all the above columns
OK these columns all get written in test code, and read into the sky model chunks on the python side as of c8688f8f30b01c75ba08e8548b6f4f457e3d35fd
Copy the StokesI version of things for Stokes V so add columns of:
pf
(just use the VPOL_FRAC to multiply Stokes I),pl
to use V_NORM_COMP_PL,V_ALPHA_PL, orcpl
to use V_NORM_COMP_CPL,V_ALPHA_CPL,V_CURVE_CPLMake sure
wodenpy
can read these new things in.Testing need to update
cmake_testing/wodenpy/skymodel/test_read_FITS_skymodel_chunk.py
to check all these news values are read in, and updatecmake_testing/wodenpy/skymodel/common_skymodel_test.py
andcmake_testing/wodenpy/skymodel/read_skymodel_common.py
with new polarisation options. Should probably write a new testtest_read_yaml_radec_count_components.py
as we don't explicitly test that at the moment (it's tested withintest_read_FITS_skymodel_chunk.py
)