Fix Version/s: None
The sky object metric values defined in verify_metrics start with a lowercase 's', whereas faro has been configured to look for an uppercase 's'. This ticket fixes that discrepancy.
- relates to
DM-34489 Add faro flux stat task into RC2/DC2 pipelines
Thanks for the review Jeffrey Carlin, and more importantly, thank you for the catch! I've kicked off a Jenkins run just to be safe. I didn't need to change the task name, but, to maintain consistency with the names in verify_metrics, I changed those too.
Jenkins successfully informed me that I need to also update the task names in all of the pipeline files I've touched (thanks Jenkins). Additional PRs for both drp_pipe and rc2_subset also added. I'm not sure if you want to look at these again or not Jeffrey Carlin? A new Jenkins is currently running with the above PRs in place.
faro PR: https://github.com/lsst/faro/pull/135/files
drp_pipe PR: https://github.com/lsst/drp_pipe/pull/19/files
rc2_subset PR: https://github.com/lsst-dm/rc2_subset/pull/13/files
Oh, right - thanks, Jenkins!
PRs approved - glad you ran Jenkins and caught that!
Looks good - the metrics should (hopefully) appear on the dashboard tomorrow evening.