search

desihub / desitarget

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

Add different Table reader options to mtl.update_ledger() #825

Closed geordie666 closed 2 months ago

geordie666 commented 3 months ago

PR #822 added different astropy table reader options for MTL I/O. The alt-MTL mocks specifically use desitarget.mtl.update_ledger() as a worker function, though, and PR #822 neglected to also add the different table reader options to that function.

This PR augments PR #822 by also adding the tabform kwarg to desitarget.mtl.update_ledger() to facilitate different astropy table reader arguments for the alt-MTL mocks.

This is a small PR, and I've carefully checked that I haven't broken anything about the MTL loop on the data side, so I'll merge in a few hours.

coveralls commented 3 months ago

Coverage Status

coverage: 53.05%. remained the same when pulling 6922a0f19489045456e7d0779c884c674b85d976 on ADM-read-mtl-basic into c77b037d012d02ad4a1864c9e42c82bda6ee31de on main.