Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ci_hsc, lsst_distrib, lsstsw, meas_extensions_shapeHSM, obs_subaru
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:DRP X16-3
-
Team:Data Release Production
Description
meas_extensions_shapeHSM has just been resurrected from bitrot, and should be included in our distribution.
Contrary to DM-2140, it should probably not be included in lsst_apps, as it's not clear we want to add a dependency on tmv and GalSim there.
Attachments
Issue Links
Activity
Ok – if obs_subaru is actually using shapeHSM, then I agree a dependency is appropriate. [If it was _just_ convenience -- "I always want to run shapeHSM when I'm processing HSC data" -- it wouldn't be. But that's not the case here.]
We rely on shapeHSM and psfex for HSC production, so I would like ci_hsc to explicitly require them.
Paul Price
added a comment - We rely on shapeHSM and psfex for HSC production, so I would like ci_hsc to explicitly require them. This looks reasonable to me. Might as well maintain alphabetical ordering in the ci_hsc table file (p < s)?
Thank you, all!
Merged to master.
Paul Price
added a comment - Thank you, all!
Merged to master. 
I think lsstsw and therefore Jenkins always enable optional dependencies so I think it's sufficient to mark them as optional in a package when they are optional and ci_hsc will automatically enable them in the Jenkins build.