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

Near-term calexp cutout API for imgserv

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ImgServ
    • Labels:

      Description

      Need a short-term solution of cutouts in calexps for the light curve viewer (SUIT).

        Attachments

          Issue Links

            Activity

            Hide
            jgates John Gates added a comment -

            Easiest way to do this is to add a parameter for image id number to the current calexp cutout case. If the id number is there, fetch that specific image and then do the cutout. Otherwise fetch an image based on RA/Dec filter. Having both filter and image id in the request should probably be treated as an error.

            Show
            jgates John Gates added a comment - Easiest way to do this is to add a parameter for image id number to the current calexp cutout case. If the id number is there, fetch that specific image and then do the cutout. Otherwise fetch an image based on RA/Dec filter. Having both filter and image id in the request should probably be treated as an error.
            Hide
            ktl Kian-Tat Lim added a comment -

            The current code loads the entire image and then takes a cutout of it. For pixel-level cutouts of id-identified images, I would like to see the Butler's inherent cutout functionality (passing a bounding box into the get call) used instead for efficiency.

            For RA/dec locations and boxes, the WCS is required to transform from sky coordinates to pixel coordinates. We could do two I/Os and load the metadata first to get the WCS, then load only the cutout pixels, or we could just load the entire image as is currently done.

            Show
            ktl Kian-Tat Lim added a comment - The current code loads the entire image and then takes a cutout of it. For pixel-level cutouts of id-identified images, I would like to see the Butler's inherent cutout functionality (passing a bounding box into the get call) used instead for efficiency. For RA/dec locations and boxes, the WCS is required to transform from sky coordinates to pixel coordinates. We could do two I/Os and load the metadata first to get the WCS, then load only the cutout pixels, or we could just load the entire image as is currently done.
            Hide
            gpdf Gregory Dubois-Felsmann added a comment -

            See https://confluence.lsstcorp.org/display/DM/Data+Access+meeting+2017-01-30 for the current proposal on how to provide the cutout-from-ID functionality.

            Show
            gpdf Gregory Dubois-Felsmann added a comment - See https://confluence.lsstcorp.org/display/DM/Data+Access+meeting+2017-01-30 for the current proposal on how to provide the cutout-from-ID functionality.
            Hide
            gpdf Gregory Dubois-Felsmann added a comment -

            The proposal is now captured as endpoints I11 and I12 on the https://confluence.lsstcorp.org/display/DM/API page.

            Show
            gpdf Gregory Dubois-Felsmann added a comment - The proposal is now captured as endpoints I11 and I12 on the https://confluence.lsstcorp.org/display/DM/API page.
            Hide
            kennylo Kenny Lo added a comment -

            Ready for code review - thanks!

            Show
            kennylo Kenny Lo added a comment - Ready for code review - thanks!
            Hide
            jgates John Gates added a comment -

            A couple of minor comments, otherwise looks good.

            Show
            jgates John Gates added a comment - A couple of minor comments, otherwise looks good.
            Hide
            kennylo Kenny Lo added a comment -

            Completed, and deployed to PDAC.

            Show
            kennylo Kenny Lo added a comment - Completed, and deployed to PDAC.

              People

              Assignee:
              kennylo Kenny Lo
              Reporter:
              kennylo Kenny Lo
              Reviewers:
              John Gates
              Watchers:
              Fritz Mueller, Gregory Dubois-Felsmann, John Gates, Kenny Lo, Kian-Tat Lim, Tatiana Goldina, Xiuqin Wu [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.