Merge pull request #296 from lsst-ts/update-wep-pipeline-defaults

Update current defaults for WEP pipelines
lsst-ts · Nov 29, 2024 · 98bb9be · 98bb9be
2 parents 3e9fd7d + f883399
commit 98bb9be
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/doc/versionHistory.rst b/doc/versionHistory.rst
@@ -13,6 +13,7 @@ Version History
 -------------
 
 * Increased maxFracBadPixels in pipelines to 8 pixels per 200^2.
+* Updated configs for TIE and Danish production pipelines to reflect current defaults
 
 .. _lsst.ts.wep-13.0.3:
 

diff --git a/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml b/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml
@@ -11,8 +11,11 @@ tasks:
       python: |
         from lsst.ts.wep.task import EstimateZernikesDanishTask
         config.estimateZernikes.retarget(EstimateZernikesDanishTask)
-      donutStampSelector.maxSelect: 2
+      donutStampSelector.maxSelect: 5
       donutStampSelector.maxFracBadPixels: 2.0e-4
+      estimateZernikes.binning: 2
+      estimateZernikes.nollIndices:
+        [4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 20, 21, 22, 27, 28]
 
 # Define pipeline steps
 subsets:

diff --git a/pipelines/production/comCamRapidAnalysisPipeline.yaml b/pipelines/production/comCamRapidAnalysisPipeline.yaml
@@ -8,7 +8,7 @@ tasks:
   calcZernikesTask:
     class: lsst.ts.wep.task.calcZernikesTask.CalcZernikesTask
     config:
-      estimateZernikes.nollIndices: [4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22]
+      estimateZernikes.nollIndices: [4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 18, 19, 20, 21, 22, 27, 28]
       estimateZernikes.convergeTol: 10.0e-9
       estimateZernikes.compGain: 0.75
       estimateZernikes.compSequence: [4, 4, 6, 6, 13, 13, 13, 13]
-Original file line number
+Diff line change
@@ Expand Up / @@ -13,6 +13,7 @@ Version History @@
     -------------
     * Increased maxFracBadPixels in pipelines to 8 pixels per 200^2.
+    * Updated configs for TIE and Danish production pipelines to reflect current defaults
     .. _lsst.ts.wep-13.0.3:
@@ Expand Down @@