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

Port HSC meas_extensions_simpleShape package to LSST

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base
    • Labels:
      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

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

            Show
            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.
            Hide
            nlust Nate Lust added a comment -

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

            Show
            nlust Nate Lust added a comment - Simon indicated I should send this your way, thanks for taking a look at it.
            Hide
            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.

            Show
            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.
            Hide
            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

            Show
            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
            Hide
            rowen Russell Owen added a comment -

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

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

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

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel