Include an option to read the MTLs using "ascii.ecsv" instead of "ascii.basic"

desihub / desitarget

DESI Targeting

BSD 3-Clause "New" or "Revised" License

18 stars 23 forks source link

Include an option to read the MTLs using "ascii.ecsv" instead of "ascii.basic" #818

Closed geordie666 closed 3 months ago

geordie666 commented 4 months ago

When running the alt-MTLs for mocks, we occasionally trip on reading a file. This appears to be fixed by adopting the ascii.ecsv reader instead of ascii.basic. So, it would be useful to propagate a kwarg throughout desitarget.io that allows this option when reading MTLs.

geordie666 commented 3 months ago

Addressed in #822.