Changeset 28687

Timestamp:

Jul 18, 2010, 2:59:19 PM (16 years ago)

Author:

eugene

Message:

update psf-convolved model fitting to be a bit more consistent with other psphot fitting, and a bit more flexible; fix the diff stats

Location:

branches/eam_branches/ipp-20100621

Files:

• ippconfig/recipes/psphot.config (modified) (1 diff)
• psModules/src/objects/pmPCM_MinimizeChisq.c (added)
• psModules/src/objects/pmPCMdata.c (added)
• psModules/src/objects/pmSourceFitPCM.c (added)
• psModules/src/objects/pmSourcePhotometry.c (modified) (2 diffs)
• psphot/src/psphotExtendedSourceFits.c (modified) (1 diff)
• psphot/src/psphotSourceFits.c (modified) (3 diffs)

branches/eam_branches/ipp-20100621/ippconfig/recipes/psphot.config

@@ -92 +92 @@
 PSF_FIT_MIN_TOL                     F32   0.01            # Fit tolerance for PSF
 PSF_FIT_MAX_TOL                     F32   2.00            # Fit tolerance for PSF
+
+PSF_FIT_MIN_VALID_FLUX              F32   1e-8            # minimum allow flux for fitted source
+PSF_FIT_MAX_VALID_FLUX              F32   1e+8            # maximum allow flux for fitted source
 
 # the following is used to require a minimum quality of fit before

branches/eam_branches/ipp-20100621/psModules/src/objects/pmSourcePhotometry.c

@@ -390 +390 @@
 # define FLUX_LIMIT 3.0
 
-// return source aperture magnitude
-bool pmSourceMeasureDiffStats (pmSource *source, psImageMaskType maskVal)
+// measure stats that may be using in difference images for distinguishing real sources from bad residuals
+bool pmSourceMeasureDiffStats (pmSource *source, psImageMaskType maskVal, psImageMaskType markVal)
 {
     PS_ASSERT_PTR_NON_NULL(source, false);
@@ -411 +411 @@
     for (int iy = 0; iy < flux->numRows; iy++) {
         for (int ix = 0; ix < flux->numCols; ix++) {
+            // only count up the stats in the unmarked region (ie, the aperture)
+            if (mask->data.PS_TYPE_IMAGE_MASK_DATA[iy][ix] & markVal) {
+                continue;
+            }
             if (mask->data.PS_TYPE_IMAGE_MASK_DATA[iy][ix] & maskVal) {
                 nMask ++;

branches/eam_branches/ipp-20100621/psphot/src/psphotExtendedSourceFits.c

@@ -210 +210 @@
           pmModel *modelFit = NULL;
           if (convolved) {
-              modelFit = psphotPSFConvModel (readout, source, modelType, maskVal, markVal, psfSize);
+              modelFit = psphotFitPCM (readout, source, fitOptions, modelType, maskVal, markVal, psfSize);
               if (!modelFit) {
                   psTrace ("psphot", 5, "failed to fit psf-conv model for object at %f, %f", source->moments->Mx, source->moments->My);

branches/eam_branches/ipp-20100621/psphot/src/psphotSourceFits.c

@@ -476 +476 @@
 }
 
+pmModel *psphotFitPCM (pmReadout *readout, pmSource *source, pmSourceFitOptions *fitOptions, pmModelType modelType, psImageMaskType maskVal, psImageMaskType markVal) {
+
+    pmSourceFitOptions options = *fitOptions;
+
+    NfitPCM ++;
+
+    // maskVal is used to test for rejected pixels, and must include markVal
+    maskVal |= markVal;
+
+    // psTraceSetLevel("psLib.math.psMinimizeLMChi2", 5);
+
+    // use the source moments, etc to guess basic model parameters
+    pmModel *EXT = pmSourceModelGuess (source, modelType);
+    if (!EXT) {
+        psTrace ("psphot", 5, "failed to generate a model for source: moments: %f %f\n", source->moments->Mxx, source->moments->Myy);
+        return NULL;
+    }
+
+    if ((source->moments->Mxx < 1e-3) || (source->moments->Myy < 1e-3)) {
+        psTrace ("psphot", 5, "problem source: moments: %f %f\n", source->moments->Mxx, source->moments->Myy);
+    }
+
+    // for sersic models, use a grid search to choose an index, then float the params there
+    if (modelType == pmModelClassGetType("PS_MODEL_SERSIC")) {
+        psphotFitSersicIndexPCM (source, EXT, fitOptions, maskVal);
+    }
+
+    if (modelType == pmModelClassGetType("PS_MODEL_SERSIC")) {
+        options.mode = PM_SOURCE_FIT_NO_INDEX;
+    } else {
+        options.mode = PM_SOURCE_FIT_EXT;
+    }
+    pmSourceFitPCM (source, EXT, &options, maskVal);
+
+    // psTraceSetLevel("psLib.math.psMinimizeLMChi2", 0);
+    return (EXT);
+}
+
 // note that these should be 1/2n of the standard sersic index
 float indexGuess[] = {0.5, 0.33, 0.25, 0.167, 0.125, 0.083};
+define N_INDEX_GUESS 6
 
 // A sersic model is very sensitive to the index.  attempt to find the index first by grid search in just the index
@@ -491 +530 @@
     options.nIter = 3;
 
-    float xMin, chiSquare[3];
+    float xMin, chiSquare[N_INDEX_GUESS];
     int iMin;
 
-    for (int i = 0; i < 3; i++) {
+    for (int i = 0; i < N_INDEX_GUESS; i++) {
         model->params->data.F32[PM_PAR_7] = indexGuess[i];
         model->modelGuess(model, source);
@@ -516 +555 @@
     return true;
 }
+
+// A sersic model is very sensitive to the index.  attempt to find the index first by grid search in just the index
+// for a sersic model, attempt to fit just the index and normalization with a modest number of iterations
+bool psphotFitSersicIndexPCM (pmSource *source, pmModel *model, pmSourceFitOptions *fitOptions, psImageMaskType maskVal) {
+
+    assert (model->type == pmModelClassGetType("PS_MODEL_SERSIC"));
+
+    pmSourceFitOptions options = *fitOptions;
+    
+    // fit EXT (not PSF) model (set/unset the pixel mask)
+    options.mode = PM_SOURCE_FIT_NO_INDEX;
+    options.nIter = 3;
+
+    float xMin, chiSquare[N_INDEX_GUESS];
+    int iMin;
+
+    for (int i = 0; i < N_INDEX_GUESS; i++) {
+        model->params->data.F32[PM_PAR_7] = indexGuess[i];
+        model->modelGuess(model, source);
+        pmSourceFitPCM (source, model, &options, maskVal);
+        chiSquare[i] = model->chisq;
+        if (i == 0) {
+            xMin = chiSquare[i];
+            iMin = i;
+        } else {
+            if (chiSquare[i] < xMin) {
+                xMin = chiSquare[i];
+                iMin = i;
+            }
+        }
+    }
+
+    model->flags = PM_MODEL_STATUS_NONE; // do not attempt to handle failures here, let the next iteration deal with it
+    model->params->data.F32[PM_PAR_7] = indexGuess[iMin];
+    model->modelGuess(model, source);
+
+    return true;
+}