Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-2141

Add meas_extensions_shapeHSM to lsstsw, lsst_distrib

    Details

      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

            Hide
            tjenness Tim Jenness added a comment -

            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.

            Show
            tjenness Tim Jenness added a comment - 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.
            Hide
            swinbank John Swinbank added a comment -

            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.]

            Show
            swinbank John Swinbank added a comment - 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.]
            Hide
            price Paul Price added a comment -

            We rely on shapeHSM and psfex for HSC production, so I would like ci_hsc to explicitly require them.

            Show
            price Paul Price added a comment - We rely on shapeHSM and psfex for HSC production, so I would like ci_hsc to explicitly require them.
            Hide
            lauren Lauren MacArthur added a comment -

            This looks reasonable to me. Might as well maintain alphabetical ordering in the ci_hsc table file (p < s)?

            Show
            lauren Lauren MacArthur added a comment - This looks reasonable to me. Might as well maintain alphabetical ordering in the ci_hsc table file (p < s)?
            Hide
            price Paul Price added a comment -

            Thank you, all!

            Merged to master.

            Show
            price Paul Price added a comment - Thank you, all! Merged to master.

              People

              • Assignee:
                price Paul Price
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Lauren MacArthur
                Watchers:
                Jim Bosch, John Swinbank, Lauren MacArthur, Paul Price, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel