search

desihub / desitarget

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

Add a TIMESTAMP to the Main Survey Target of Opportunity Ledgers #761

Closed geordie666 closed 3 years ago

geordie666 commented 3 years ago

This PR adds a TIMESTAMP to the ToO Ledgers for survey='main'. The same TIMESTAMP as used elsewhere in the MTL loop is appropriated.

The justification for this addition is so that any fiber-override ToOs that are included in the future can be modeled in the alternative MTL ledgers, just as with any other target passed to fiberassign. It's possible that fiber-overrides for ToOs won't be necessary, but if they are adopted, having a TIMESTAMP will be crucial.

This is a small change that should be uncontroversial. It only affects a single column in the ToO ledger, and is backwards-compatible from main to sv3. So, I'll merge it after the weekend.

coveralls commented 3 years ago

Coverage Status

Coverage decreased (-0.03%) to 58.392% when pulling 9fe271919acf74181d784a8250ae6e1d1fe41703 on ADM-ToO-time into 015a3e162f8362338b1ba63029913ab1733c6b38 on master.