Add backwards-compatibility "decl" column

Add `decl` as a duplicate of `dec` to sourceTable_tract, CcdVisit, and Visit, with a comment about its future removal.
lsst · Jun 2, 2023 · e45c489 · e45c489
1 parent 811be50
commit e45c489
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/python/lsst/pipe/tasks/postprocess.py b/python/lsst/pipe/tasks/postprocess.py
@@ -1462,6 +1462,11 @@ def run(self, visitSummaryRefs):
             # Technically you should join to get the visit from the visit
             # table.
             ccdEntry = ccdEntry.rename(columns={"visit": "visitId"})
+
+            # RFC-924: Temporarily keep a duplicate "decl" entry for backwards
+            # compatibility. To be removed after September 2023.
+            ccdEntry["decl"] = ccdEntry.loc[:, "dec"]
+
             ccdEntry['ccdVisitId'] = [
                 self.config.idGenerator.apply(
                     visitSummaryRef.dataId,
@@ -1567,6 +1572,11 @@ def run(self, visitSummaries):
             raDec = visitInfo.getBoresightRaDec()
             visitEntry["ra"] = raDec.getRa().asDegrees()
             visitEntry["dec"] = raDec.getDec().asDegrees()
+
+            # RFC-924: Temporarily keep a duplicate "decl" entry for backwards
+            # compatibility. To be removed after September 2023.
+            visitEntry["decl"] = visitEntry.loc[:, "dec"]
+
             visitEntry["skyRotation"] = visitInfo.getBoresightRotAngle().asDegrees()
             azAlt = visitInfo.getBoresightAzAlt()
             visitEntry["azimuth"] = azAlt.getLongitude().asDegrees()

diff --git a/schemas/Object.yaml b/schemas/Object.yaml
@@ -26,6 +26,14 @@ funcs:
         functor: CoordColumn
         args: coord_dec
         dataset: meas
+
+    # RFC-924: Temporarily keep a duplicate "decl" entry for backwards
+    # compatibility. To be removed after September 2023.
+    decl:
+        functor: CoordColumn
+        args: coord_dec
+        dataset: meas
+
     # raErr:  not available yet DM-15180
     # decErr: not available yet DM-15180
     # Reference band is same for all measurements

diff --git a/schemas/Source.yaml b/schemas/Source.yaml
@@ -40,6 +40,12 @@ funcs:
     # raErr: not available yet DM-15180
     dec:
         functor: DecColumn
+
+   # RFC-924: Temporarily keep a duplicate "decl" entry for backwards
+   # compatibility. To be removed after September 2023.
+    decl:
+        functor: DecColumn
+
     # decErr: not available yet DM-15180
     # ra_dec_Cov: not available yet
     # One calibrated Calib flux is important: