Merge pull request #57 from lsst/tickets/DM-35530

DM-35530: Fix typo in persisted scarlet model data

python/lsst/meas/extensions/scarlet/io.py

@@ -107,7 +107,9 @@ class ScarletFactorizedComponentData:
     extent : `tuple` of `int`
         The `(width, height)` of the component array.
     center : `tuple` of `int`
-        The center of the component.
+        The ``(x, y)`` center of the component.
+        Note: once this is converted into a scarlet `LiteBlend` the source has
+        the traditional c++ `(y, x)` ordering.
     sed : `numpy.ndarray`
         The SED of the component.
     morph : `numpy.ndarray`
@@ -130,7 +132,7 @@ def asDict(self) -> dict:
         return {
             "xy0": self.xy0,
             "extent": self.extent,
-            "center": self.extent,
+            "center": self.center,
             "sed": tuple(self.sed.astype(float)),
             "morph": tuple(self.morph.flatten().astype(float))
         }

tests/test_deblend.py

@@ -188,6 +188,9 @@ def test_deblend_task(self):
 
                     # Get the scarlet model for the source
                     source = [src for src in blend.sources if src.recordId == child.getId()][0]
+                    parentBox = catalog.find(child["parent"]).getFootprint().getBBox()
+                    self.assertEqual(source.center[1], px - parentBox.getMinX())
+                    self.assertEqual(source.center[0], py - parentBox.getMinY())
 
                     if useFlux:
                         # Get the flux re-weighted model and test against