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

Port HSC meas_extensions_simpleShape package to LSST

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • meas_base
    • None

    Description

      HSC uses a package, meas_extensions_simpleShape, which needs to be ported to LSST. The package is used for basic shape measurements for determining focus, and also serves as a simple guide for writing measurement plugins.

      Attachments

        Issue Links

          Activity

            For my own future reference, if nothing else: meas_extensions_simpleShape doesn't seem to exist in HSC GitHub, but is available at ssh://gituser@hsc-repo.mtk.nao.ac.jp:10022/home/gituser/repositories/meas_extensions_simpleShape.git.

            swinbank John Swinbank added a comment - For my own future reference, if nothing else: meas_extensions_simpleShape doesn't seem to exist in HSC GitHub , but is available at ssh://gituser@hsc-repo.mtk.nao.ac.jp:10022/home/gituser/repositories/meas_extensions_simpleShape.git.
            nlust Nate Lust added a comment -

            Simon indicated I should send this your way, thanks for taking a look at it.

            nlust Nate Lust added a comment - Simon indicated I should send this your way, thanks for taking a look at it.
            rowen Russell Owen added a comment - - edited

            A quick question: afw is listed as a branch, but I can't find any changes on that branch. Is that as expected? If so, please consider deleting that branch.

            rowen Russell Owen added a comment - - edited A quick question: afw is listed as a branch, but I can't find any changes on that branch. Is that as expected? If so, please consider deleting that branch.
            nlust Nate Lust added a comment -

            Sorry about that, I forgot to push my change. It's only one commit where I instantiate a class. Pushed now

            nlust Nate Lust added a comment - Sorry about that, I forgot to push my change. It's only one commit where I instantiate a class. Pushed now
            rowen Russell Owen added a comment -

            This looks really nice. I had a few minor suggestions which I posted as comments on github.

            rowen Russell Owen added a comment - This looks really nice. I had a few minor suggestions which I posted as comments on github.

            nlust: Please remember to add some text describing this algorithm to the release notes.

            swinbank John Swinbank added a comment - nlust : Please remember to add some text describing this algorithm to the release notes .

            People

              nlust Nate Lust
              price Paul Price
              Russell Owen
              John Swinbank, Nate Lust, Paul Price, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.