Switch to `v_astra_major`, `v_astra_minor`, `v_astra_patch` and be more semantic

sdss / astra

Analysis framework for SDSS-V/Milky Way Mapper

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

3 stars 0 forks source link

Switch to `v_astra_major`, `v_astra_minor`, `v_astra_patch` and be more semantic #28

Closed andycasey closed 5 days ago

andycasey commented 1 month ago

We might be able to keep this as a single field v_astra but just have it as a three-length tuple, as long as we update all the queries.

The queries affect when looking for new spectra to analyse, or exporting tables, or coadding data, etc etc etc.

andycasey commented 5 days ago

Fixed by setting v_astra as an integer, which encodes the major, minor, and patch version. Unique constraints added for (spectrum_pk, v_astra_major_minor).