Add a TIMESTAMP to the Main Survey Target of Opportunity Ledgers #761
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a
TIMESTAMP
to the ToO Ledgers forsurvey='main'
. The sameTIMESTAMP
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
tosv3
. So, I'll merge it after the weekend.