Changeset 41533 for trunk/psModules/src/astrom/pmAstrometryWCS.c

Timestamp:

Apr 3, 2021, 2:21:37 PM (5 years ago)

Author:

eugene

Message:

pass in additional orders to psPlaneTransformInvert

File:

• trunk/psModules/src/astrom/pmAstrometryWCS.c (modified) (6 diffs)

trunk/psModules/src/astrom/pmAstrometryWCS.c

@@ -606 +606 @@
     psRegion *region = pmChipPixels (chip);
 
+    // as of r40806, psPlaneTransformInvert supplies the extra order (if non-linear)
     psFree (chip->fromFPA);
-    assert (chip->fromFPA == NULL);
-    chip->fromFPA = psPlaneTransformInvert(NULL, chip->toFPA, *region, 50);
+    chip->fromFPA = psPlaneTransformInvert(NULL, chip->toFPA, *region, 50, 4);
     psFree (region);
 
@@ -666 +666 @@
     psRegion *region = pmChipPixels (chip);
 
+    // as of r40806, psPlaneTransformInvert supplies the extra order (if non-linear)
     psFree (chip->fromFPA);
-    chip->fromFPA = psPlaneTransformInvert(NULL, chip->toFPA, *region, 50);
+    chip->fromFPA = psPlaneTransformInvert(NULL, chip->toFPA, *region, 50, 4);
     psFree (region);
 
@@ -691 +692 @@
         }
     }
+
+    // XXX the way this is coded, updates will NOT have the same behavior as with the
+    // original code: the if ADDITIONAL_WCS_ORDERS is not found, the default
+    // should be 0 (or else we need some way to detect the update version)
 
     // the transformation used the region to define the inversion grid
@@ -701 +706 @@
       additional_orders = config_additional_orders; 
     }
-    
-    psPlaneTransform *myPT = psPlaneTransformAlloc(fpa->toTPA->x->nX+additional_orders, fpa->toTPA->x->nY+additional_orders);
-
-    fpa->fromTPA = psPlaneTransformInvert(myPT, fpa->toTPA, region, 50);
-    psFree (myPT);
+    fpa->fromTPA = psPlaneTransformInvert(NULL, fpa->toTPA, region, 50, additional_orders);
     return true;
 }
@@ -1030 +1031 @@
     }
 
-    psPlaneTransform *outFromFPA = psPlaneTransformInvert(NULL, outToFPA, *outputBounds, 50);
+    // NOTE: the extraOrders value (4) should be ignored since outToFPA is specified to be linear
+    psPlaneTransform *outFromFPA = psPlaneTransformInvert(NULL, outToFPA, *outputBounds, 50, 4);
     if (!outFromFPA) {
         psFree(outToFPA);
@@ -1147 +1149 @@
 
     // this is a linear transformation
-    psPlaneTransform *newFromFPA = psPlaneTransformInvert(NULL, newToFPA, *bounds, 1);
+    psPlaneTransform *newFromFPA = psPlaneTransformInvert(NULL, newToFPA, *bounds, 1, 0);
     if (!newFromFPA) {
         psFree(newToFPA);