search

ska-sa / katdal

Data access library for the MeerKAT radio telescope
BSD 3-Clause "New" or "Revised" License
12 stars 13 forks source link

SPR1-2367: Remove IMAGING_WEIGHT column from MS #356

Closed ludwigschwardt closed 2 years ago

ludwigschwardt commented 2 years ago

This column is initialised full of zeroes and just takes up space (4 bytes per main table entry, i.e. time-frequency-baseline cell, so about 6-12% overhead). It can be recreated in CASA at a later stage.

The column is a leftover of good old blank.ms, which included it along with MODEL_DATA and CORRECTED_DATA. The latter two columns were made optional in the sneaky 41af4b8 as part of #75 in version 0.9, but this one was missed.

Testing an MS produced by the new code would be much appreciated.

Jordatious commented 2 years ago

Looks good! Thanks!

ludwigschwardt commented 2 years ago

check_data.py is also happy (just checked), so merging.