search

desihub / desitarget

DESI Targeting
BSD 3-Clause "New" or "Revised" License
18 stars 23 forks source link

Make sure that types are cast properly by read_keyword_from_mtl_header #769

Closed geordie666 closed 2 years ago

geordie666 commented 2 years ago

In #768, there is a partial solution to "clean up" keywords read quickly from .ecsv files using read_ecsv_header(). This "clean up" is needed as reading the ascii is currently fragile for types other than strings.

Once #768 is merged, propagate this "clean up" option to read_keyword_from_mtl_header as the default.

geordie666 commented 2 years ago

This was done in #768 in the end, in which we set read_keyword_from_mtl_header() to always use the default cleanup=True in read_ecsv_header().