Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: pipe_analysis
-
Labels:None
-
Story Points:4
-
Epic Link:
-
Sprint:DRP S19-5
-
Team:Data Release Production
Description
DM-10156 changed Calib to PhotoCalib. It looks like some updates in pipe_analysis are needed. Running compareVisitAnalysis.py fails in the following error:
File "/software/lsstsw/stack_20190330/stack/miniconda3-4.5.12-1172c30/Linux64/pipe_base/17.0.1-2-g3e5d191+3/python/lsst/pipe/base/cmdLineTask.py", line 388, in __call__
|
result = self.runTask(task, dataRef, kwargs)
|
File "/software/lsstsw/stack_20190330/stack/miniconda3-4.5.12-1172c30/Linux64/pipe_base/17.0.1-2-g3e5d191+3/python/lsst/pipe/base/cmdLineTask.py", line 447, in runTask
|
return task.runDataRef(dataRef, **kwargs)
|
File "/home/hchiang2/stack/pipe_analysis/python/lsst/pipe/analysis/visitAnalysis.py", line 768, in runDataRef
|
postFix="_commonZp", **plotKwargs1)
|
File "/home/hchiang2/stack/pipe_analysis/python/lsst/pipe/analysis/coaddAnalysis.py", line 1337, in plotMags
|
unitScale=self.unitScale),
|
File "/home/hchiang2/stack/pipe_analysis/python/lsst/pipe/analysis/utils.py", line 332, in __init__
|
self.calib = afwImage.Calib()
|
AttributeError: module 'lsst.afw.image' has no attribute 'Calib'
|
An example to reproduce the above error is compareVisitAnalysis.py /datasets/hsc/repo --calib /datasets/hsc/repo/CALIB --rerun RC/w_2019_14/DM-18300-mosaic:private/user/name --rerun2 RC/w_2019_10/DM-17940 --no-versions --tract=9615 --id visit=26024
Attachments
Issue Links
- relates to
-
DM-19128 Run pipe_analysis & validate_drp with w_2019_14 HSC RC2
- Done
Activity
Would you mind giving this a look? I have confirmed that the compare[Visit/Coadd]Analysis.py scripts now run and that the numbers coming out of the magnitude comparisons (esp. vs ref cats) are all self-consistent.
Code changes look good to me. I didn't try running them but I'd take your words they are fine now 
Hsin-Fang Chiang
added a comment - Code changes look good to me. I didn't try running them but I'd take your words they are fine now Many thanks for another speedy review!
(I forgot a second repo can be chained to --rerun too; description corrected)