# validateDrp.py doesn't like the NB0921 filter

XMLWordPrintable

#### Details

• Type: Story
• Status: Invalid
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Team:
SQuaRE

#### Description

Using w_2018_28 and running validateDrp.py on a filter=NB0921 json file generated by matchedVisitMetrics.py gives the following error

 Traceback (most recent call last):  File "/software/lsstsw/stack3_20171023/stack/miniconda3-4.3.21-10a4fa6/Linux64/validate_drp/16.0-4-g0b0e9de+6/bin/validateDrp.py", line 109, in   validate.run(args.repo, **kwargs)  File "/software/lsstsw/stack3_20171023/stack/miniconda3-4.3.21-10a4fa6/Linux64/validate_drp/16.0-4-g0b0e9de+6/python/lsst/validate/drp/validate.py", line 157, in run  print_metrics(job)  File "/software/lsstsw/stack3_20171023/stack/miniconda3-4.3.21-10a4fa6/Linux64/validate_drp/16.0-4-g0b0e9de+6/python/lsst/validate/drp/validate.py", line 482, in print_metrics  metric = job.metrics[metric_set[0]] # Pick the first one for the description  File "/software/lsstsw/stack3_20171023/stack/miniconda3-4.3.21-10a4fa6/Linux64/verify/16.0+1/python/lsst/verify/metricset.py", line 207, in __getitem__  return self._metrics[key] KeyError: Name('release', 'PF1') 

To reproduce, try

 validateDrp.py /datasets/hsc/repo/rerun/RC/w_2018_28/DM-14988/validateDrp/matchedVisitMetrics/9813/NB0921/matchedVisit_NB0921.json --noplot 

#### Activity

Hide
Tim Jenness added a comment -

validate_drp is deprecated so this ticket is no longer valid as-is. Keith Bechtol is this at all relevant for faro?

Show
Tim Jenness added a comment - validate_drp is deprecated so this ticket is no longer valid as-is. Keith Bechtol is this at all relevant for faro?
Hide

At the DRP metrics meeting a month ago Jeffrey Carlin got an action to "Make faro not attempt running on narrow-bands." to https://confluence.lsstcorp.org/display/DM/DRP+Metrics+Monitoring+2022-02-28

I can't find a ticket for that, so maybe we can repurpose this ticket.

Show
Yusra AlSayyad added a comment - At the DRP metrics meeting a month ago Jeffrey Carlin  got an action to "Make faro not attempt running on narrow-bands." to https://confluence.lsstcorp.org/display/DM/DRP+Metrics+Monitoring+2022-02-28 I can't find a ticket for that, so maybe we can repurpose this ticket.
Hide

Also, Jim Bosch suggested that there was a way for faro task to, at QG generation time, throw away any quanta for narrow bands.

Show
Yusra AlSayyad added a comment - Also, Jim Bosch suggested that there was a way for faro task to, at QG generation time, throw away any quanta for narrow bands.
Hide
Tim Jenness added a comment -

Thanks. I'm more than happy for the component to be changed to faro and the description tweaked.

Show
Tim Jenness added a comment - Thanks. I'm more than happy for the component to be changed to faro and the description tweaked.
Hide
Leanne Guy added a comment -

faro does not use validateDrp.py so I think I'd rather close this ticket and make a new ticket for faro

Show
Leanne Guy added a comment - faro does not use validateDrp.py so I think I'd rather close this ticket and make a new ticket for faro
Hide
Tim Jenness added a comment -

Please link the new ticket to this one when you make the new ticket (and then mark this one INVALID).

Show
Tim Jenness added a comment - Please link the new ticket to this one when you make the new ticket (and then mark this one INVALID).

#### People

Assignee:
Unassigned
Reporter:
Hsin-Fang Chiang
Watchers:
Hsin-Fang Chiang, Leanne Guy, Tim Jenness, Yusra AlSayyad