Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: verify
-
Labels:None
-
Story Points:4
-
Epic Link:
-
Sprint:AP F22-4 (September)
-
Team:Alert Production
-
Urgent?:No
Description
Change the specification for lsst.verify.MetricTask as described in RFC-788. In addition, make the following concrete code changes throughout Science Pipelines:
- Remove MetricTask.runQuantum. Rewrite all known overrides of runQuantum to be based off of (and preferably delegate to) PipelineTask.runQuantum.
- Stop supporting optional inputs in concrete MetricTasks. This code was only being used in unit tests, so it can be removed without harm.
John Parejko discovered that lsst.verify.tasks.TimingMetricTask causes an empty quantum graph if the task being timed was never run. Investigate whether this is related to the introduction of optional inputs and/or NoWorkFound in the Middleware framework. Note that, in his scenario, the pipeline was structurally incapable of generating the expected metadata, while the "return if irrelevant" policy was intended to handle pipeline failures.