From c075d5f94d60b9fca0b013f8c4bdb6f0e193083c Mon Sep 17 00:00:00 2001
From: "J. Bryce Kalmbach" <jbkalmbach@gmail.com>
Date: Wed, 11 Dec 2024 12:32:56 -0800
Subject: [PATCH] Change default pipeline setting to false for rubinTV upload.

---
 doc/versionHistory.rst                               |  1 +
 pipelines/_ingredients/donutVizGroupPipeline.yaml    |  6 +++---
 pipelines/production/comCamDailyProcessing.yaml      | 10 ----------
 .../comCamRapidAnalysisDanishPipeline.yaml           | 12 ++++++++++++
 .../production/comCamRapidAnalysisPipeline.yaml      | 12 ++++++++++++
 5 files changed, 28 insertions(+), 13 deletions(-)

diff --git a/doc/versionHistory.rst b/doc/versionHistory.rst
index 5293323e..2661371b 100644
--- a/doc/versionHistory.rst
+++ b/doc/versionHistory.rst
@@ -13,6 +13,7 @@ Version History
 -------------
 
 * Add donut quality tables to outputs even when there are no donuts that pass so that we can match it up to the donut stamps and understand rejections.
+* Change default pipeline setting to false for rubinTV upload.
 
 .. _lsst.ts.wep-13.2.0:
 
diff --git a/pipelines/_ingredients/donutVizGroupPipeline.yaml b/pipelines/_ingredients/donutVizGroupPipeline.yaml
index 96d5c03d..36ea171b 100644
--- a/pipelines/_ingredients/donutVizGroupPipeline.yaml
+++ b/pipelines/_ingredients/donutVizGroupPipeline.yaml
@@ -14,17 +14,17 @@ tasks:
   plotAOSTask:
     class: lsst.donut.viz.PlotAOSTask
     config:
-      doRubinTVUpload: true
+      doRubinTVUpload: false
   aggregateDonutStampsTask:
     class: lsst.donut.viz.AggregateDonutStampsTask
   plotDonutTask:
     class: lsst.donut.viz.PlotDonutTask
     config:
-      doRubinTVUpload: true
+      doRubinTVUpload: false
   plotPsfZernTask:
     class: lsst.donut.viz.PlotPsfZernTask
     config:
-      doRubinTVUpload: true
+      doRubinTVUpload: false
 
 subsets:
   donutVizGroups:
diff --git a/pipelines/production/comCamDailyProcessing.yaml b/pipelines/production/comCamDailyProcessing.yaml
index ee208b76..667f33b0 100644
--- a/pipelines/production/comCamDailyProcessing.yaml
+++ b/pipelines/production/comCamDailyProcessing.yaml
@@ -9,16 +9,6 @@ tasks:
     class: lsst.ts.wep.task.generateDonutDirectDetectTask.GenerateDonutDirectDetectTask
     config:
       donutSelector.useCustomMagLimit: True
-  plotAOSTask:
-    class: lsst.donut.viz.PlotAOSTask
-    config:
-      doRubinTVUpload: false
-  aggregateDonutStampsTask:
-    class: lsst.donut.viz.AggregateDonutStampsTask
-  plotDonutTask:
-    class: lsst.donut.viz.PlotDonutTask
-    config:
-      doRubinTVUpload: false
 
 # Define pipeline steps
 subsets:
diff --git a/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml b/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml
index 79116854..4b35f076 100644
--- a/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml
+++ b/pipelines/production/comCamRapidAnalysisDanishPipeline.yaml
@@ -21,6 +21,18 @@ tasks:
         ftol: 1.0e-3
         xtol: 1.0e-3
         gtol: 1.0e-3
+  plotAOSTask:
+    class: lsst.donut.viz.PlotAOSTask
+    config:
+      doRubinTVUpload: true
+  plotDonutTask:
+    class: lsst.donut.viz.PlotDonutTask
+    config:
+      doRubinTVUpload: true
+  plotPsfZernTask:
+    class: lsst.donut.viz.PlotPsfZernTask
+    config:
+      doRubinTVUpload: true
 
 # Define pipeline steps
 subsets:
diff --git a/pipelines/production/comCamRapidAnalysisPipeline.yaml b/pipelines/production/comCamRapidAnalysisPipeline.yaml
index 5f86060a..b210c3bb 100644
--- a/pipelines/production/comCamRapidAnalysisPipeline.yaml
+++ b/pipelines/production/comCamRapidAnalysisPipeline.yaml
@@ -18,6 +18,18 @@ tasks:
       estimateZernikes.maskKwargs: { "doMaskBlends": False }
       donutStampSelector.maxSelect: 5
       donutStampSelector.maxFracBadPixels: 2.0e-4
+  plotAOSTask:
+    class: lsst.donut.viz.PlotAOSTask
+    config:
+      doRubinTVUpload: true
+  plotDonutTask:
+    class: lsst.donut.viz.PlotDonutTask
+    config:
+      doRubinTVUpload: true
+  plotPsfZernTask:
+    class: lsst.donut.viz.PlotPsfZernTask
+    config:
+      doRubinTVUpload: true
 
 # Define pipeline steps
 subsets: