desihub / desimeter

DESI coordinates and transformations
BSD 3-Clause "New" or "Revised" License
2 stars 4 forks source link

fix desc calib sort order to asc in query cmd #182

Closed joesilber closed 3 years ago

joesilber commented 3 years ago

When I implemented the "--latest" feature earlier, I erroneously added sort order "desc" to the calibration retrieval. However, previously it had been default order, which is implicitly ascending. Then the code which collates calibration data rows to move data rows was grabbing the earliest match rather than the latest, which is wrong. Thanks for Kevin for catching this behavior. Fixed now.