start and end mjd from schedule

growth-astro / growth-too-marshal

GROWTH Target of Opportunity Marshal

MIT License

13 stars 12 forks source link

start and end mjd from schedule #121

Closed mcoughlin closed 4 years ago

mcoughlin commented 4 years ago

Does this pull request make any changes to the database? No.

Code changes that affect the database require special attention for data migration. If your change affects the database, describe the migration plan here.

shreyasahasram08 commented 4 years ago

Michael, will test this shortly. Do you think it would be appropriate to also add an end time to Plan in models and Plan.html as a part of this PR? Of course it's still possible to compute the end time pretty easily as it is, but might be useful to have this when quickly glancing through multiple plans.

mcoughlin commented 4 years ago

Ah yes could do. If you want to add it while testing the other bit, I wouldn't complain ;)

lpsinger commented 4 years ago

Can this be closed as a duplicate of #122, which has already been merged?