Another update to the data model for the MTL loop

desihub / desitarget

DESI Targeting

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

19 stars 23 forks source link

Another update to the data model for the MTL loop #698

Closed geordie666 closed 3 years ago

geordie666 commented 3 years ago

This PR updates the expected directory structure for the MTL loop.

Previously, the loop assumed that the spectroscopic pipeline would only maintain one night in the cumulative directory and would unlink previous nights. With this update, the loop instead looks up the (YYYYMMDD) night that corresponds to LASTNIGHT in the redshift tiles.csv file for a given tile and then uses the zbest files in the corresponding directory.

geordie666 commented 3 years ago

This is a (relatively) minor update, and I've checked it works second-time-through-the-loop. I'll merge this in about an hour so I can make a new tag of desitarget and use it to update the BRIGHT/sv2 ledgers in svn.

coveralls commented 3 years ago

Coverage increased (+0.04%) to 60.64% when pulling cd9885a02a8a02e19ba9c656cc60dd56e41b2056 on ADM-dm-MTL into 7ca8df36723a5abef74c4793a0a307c1d3762c79 on master.