Skip to content

Commit a842004

Browse files
authored
Merge pull request #304 from lsst/tickets/DM-47257
DM-47257: Upload diffim matching kernel metrics
2 parents e3dea21 + af6bb0a commit a842004

File tree

2 files changed

+17
-7
lines changed

2 files changed

+17
-7
lines changed

pipelines/apDetectorVisitQualityCore.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,10 @@ tasks:
7272
nBadPixelsDetectedNegative: ct
7373
sciencePsfSize: pixel
7474
templatePsfSize: pixel
75-
scaleScienceVarianceFactor: ct
76-
scaleTemplateVarianceFactor: ct
75+
scaleScienceVarianceFactor: ''
76+
scaleTemplateVarianceFactor: ''
77+
spatialConditionNum: ''
78+
spatialKernelSum: ''
7779
templateCoveragePercent: percent
7880
python: |
7981
from lsst.analysis.tools.atools import DiffimMetadataMetricTool

python/lsst/analysis/tools/tasks/diffimTaskDetectorVisitMetricsAnalysis.py

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222

2323
__all__ = ("DiffimDetectorVisitMetricsAnalysisConfig", "DiffimDetectorVisitMetricsAnalysisTask")
2424

25+
import lsst.pex.config
2526
import pandas as pd
2627
from lsst.pipe.base import NoWorkFound, connectionTypes
2728

@@ -49,7 +50,11 @@ class DiffimDetectorVisitMetricsAnalysisConnections(
4950
class DiffimDetectorVisitMetricsAnalysisConfig(
5051
AnalysisBaseConfig, pipelineConnections=DiffimDetectorVisitMetricsAnalysisConnections
5152
):
52-
pass
53+
kernelSubtaskName = lsst.pex.config.Field(
54+
dtype=str,
55+
default="makeKernel",
56+
doc="Perform diffim decorrelation to undo pixel correlation due to A&L ",
57+
)
5358

5459

5560
class DiffimDetectorVisitMetricsAnalysisTask(AnalysisPipelineTask):
@@ -64,12 +69,15 @@ def runQuantum(self, butlerQC, inputRefs, outputRefs):
6469
if not metadata:
6570
raise NoWorkFound("No metadata entries for detectAndMeasure.")
6671
inputs.pop("metadataDetect")
72+
6773
subtractTaskName = inputRefs.metadataSubtract.datasetType.name
6874
subtractTaskName = subtractTaskName[: subtractTaskName.find("_")]
6975
metadata |= inputs["metadataSubtract"].metadata[subtractTaskName].to_dict()
76+
kernelSubtaskLabel = subtractTaskName + ":" + self.config.kernelSubtaskName
77+
metadata |= inputs["metadataSubtract"].metadata[kernelSubtaskLabel].to_dict()
7078
inputs.pop("metadataSubtract")
71-
df = pd.DataFrame(metadata)
72-
73-
inputs["data"] = df
74-
outputs = self.run(**inputs)
79+
# Some metadata entries might have different lengths or simply floats.
80+
# Pass the dict in a list to tell Pandas that this is one row in the
81+
# dataframe.
82+
outputs = self.run(data=pd.DataFrame([metadata]))
7583
butlerQC.put(outputs, outputRefs)

0 commit comments

Comments
 (0)