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

Add getCutout method to Exposure

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP S18-5, AP S18-6
    • Team:
      Alert Production

      Description

      There are many instances when it would be helpful to have a simple way to return a postage stamp, or small cutout, of an exposure with some user-specified center and dimensions. This ticket is to create such a method, ideally in C++.

      A preliminary python implementation lives in u/mrawls/exposure-cutout.

        Attachments

          Issue Links

          is triggering

          Improvement - An improvement or enhancement to an existing feature or task. DM-14707 Imgserv to use the newly implemented getCutout method in Exposure class

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-38152 Make zooniverseCutouts interface for getting footprint-based cutouts

          • Done

          Improvement - An improvement or enhancement to an existing feature or task. DM-14690 Add ability to construct centered boxes

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            mrawls Meredith Rawls created issue -
            krzys Krzysztof Findeisen made changes -
            Field Original Value New Value
            Epic Link DM-12728 [ 36327 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S18-6 [ 686 ]
            Priority Undefined [ 10000 ] Minor [ 4 ]
            Hide
            Permalink
            price Paul Price added a comment -

            I think this should be a constructor overload.

            Show
            price Paul Price added a comment - I think this should be a constructor overload.
            krzys Krzysztof Findeisen made changes -
            Sprint AP S18-6 [ 686 ] AP S18-5 [ 685 ]
            Labels SciencePipelines PairCoding SciencePipelines
            krzys Krzysztof Findeisen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S18-5 [ 685 ] AP S18-5, AP S18-6 [ 685, 686 ]
            krzys Krzysztof Findeisen made changes -
            Risk Score 0
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Hi Jim Bosch, would you be willing to review this? I'm curious to see what you make of the implementation.

            Show
            krzys Krzysztof Findeisen added a comment - Hi Jim Bosch , would you be willing to review this? I'm curious to see what you make of the implementation.
            krzys Krzysztof Findeisen made changes -
            Reviewers Jim Bosch [ jbosch ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            jbosch Jim Bosch made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue relates to DM-14690 [ DM-14690 ]
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Thanks for the quick review. I've made all requested changes except factoring out the box code, which is enough extra work that it's better to do it as DM-14690.

            Show
            krzys Krzysztof Findeisen added a comment - Thanks for the quick review. I've made all requested changes except factoring out the box code, which is enough extra work that it's better to do it as DM-14690 .
            krzys Krzysztof Findeisen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            kennylo Kenny Lo made changes -
            Link This issue is triggering DM-14707 [ DM-14707 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-38152 [ DM-38152 ]

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              mrawls Meredith Rawls
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, Krzysztof Findeisen, Meredith Rawls, Paul Price
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.