Updates after JFC review.

lsst-ts · Nov 17, 2024 · fd823c7 · fd823c7
1 parent db76f12
commit fd823c7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pipelines/scienceDirectDetectPipeline.yaml b/pipelines/scienceDirectDetectPipeline.yaml
@@ -37,7 +37,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, 27, 28]
+      estimateZernikes.nollIndices: [4, 5, 6, 7, 8, 9, 10, 11, 14, 15, 20, 21, 22, 27, 28]
       estimateZernikes.saveHistory: False
       estimateZernikes.maskKwargs: {'doMaskBlends': False}
   aggregateZernikeTablesTask:

diff --git a/python/lsst/donut/viz/zernike_pyramid.py b/python/lsst/donut/viz/zernike_pyramid.py
@@ -80,7 +80,7 @@ def zernikePyramid(
     """
     jmin = min(noll_indices)
     jmax = max(noll_indices)
-    jdict = {x: y for x, y in zip(noll_indices - jmin, range(len(noll_indices)))}
+    jdict = {x: y for x, y in zip(noll_indices, range(len(noll_indices)))}
     nmax, _ = galsim.zernike.noll_to_zern(jmax)
     nmin, _ = galsim.zernike.noll_to_zern(jmin)
 
@@ -129,7 +129,7 @@ def shiftAxes(axes, amt):
         scat = ax.scatter(
             xs,
             ys,
-            c=zs[jdict[j - jmin]],
+            c=zs[jdict[j]],
             s=s,
             linewidths=0.5,
             cmap=cmap,