Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-160

Revise naming scheme for coadd images

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      We need to have two dataset names for coadd images: one for the raw coadd produced by assembleCoadd.py, and one for the background-subtracted coadd with detections identified produced by detectCoaddSources.py. The first is deepCoadd in both LSST and HSC, and is not in contention. The second is deepCoadd_calexp_det in LSST and deepCoadd_calexp in HSC. I think the LSST name originates from before HSC transitioned to a naming scheme that I think makes sense, and hence I think it's a bit confused, and unnecessarily long and clumsy. I would like to make the LSST naming scheme consistent with that of HSC because I think it's simpler, clearer, and more people have more experience with it. deepCoadd_calexp currently appears to be unused on the LSST side.

      Now seems to be a good time to do this, before the multiband products become more widely used on the LSST side. The HSC port ticket DM-5288 will do this work, including updating dataset names in the obs_* packages.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            I think this work is actually already scheduled as part of DM-5322: it's just been waiting for DM-4692 to land before we act on it. Does that story address your point, or am I misunderstanding what you're driving at?

            Show
            swinbank John Swinbank added a comment - I think this work is actually already scheduled as part of DM-5322 : it's just been waiting for DM-4692 to land before we act on it. Does that story address your point, or am I misunderstanding what you're driving at?
            Hide
            price Paul Price added a comment -

            Yes, you're quite right. There's overlap between DM-5322 and DM-5288. On whichever ticket the work is done, there's code that needs to come over from HSC for it.

            Show
            price Paul Price added a comment - Yes, you're quite right. There's overlap between DM-5322 and DM-5288 . On whichever ticket the work is done, there's code that needs to come over from HSC for it.
            Hide
            swinbank John Swinbank added a comment -

            I hadn't realised that the DM-5322 work needed further input from HSC. If you think it's better to handle part or all of that work as part of DM-5288, that works for me.

            I'd been intending to go ahead with renaming the dataset without RFCing it, on the basis that it was implicit in the previous RFCed plan for the multiband transition. However, since you are doing things by the book (), it's a strong +1 from me on this.

            Show
            swinbank John Swinbank added a comment - I hadn't realised that the DM-5322 work needed further input from HSC. If you think it's better to handle part or all of that work as part of DM-5288 , that works for me. I'd been intending to go ahead with renaming the dataset without RFCing it, on the basis that it was implicit in the previous RFCed plan for the multiband transition. However, since you are doing things by the book ( ), it's a strong +1 from me on this.
            Hide
            Parejkoj John Parejko added a comment -

            Definitely a good idea. +1

            Show
            Parejkoj John Parejko added a comment - Definitely a good idea. +1
            Hide
            price Paul Price added a comment -

            Adopted as written.

            Show
            price Paul Price added a comment - Adopted as written.
            Hide
            price Paul Price added a comment -

            Implemented as part of DM-5288.

            DM-5322 was recast to simply remove duplicate or unused datasets.

            Show
            price Paul Price added a comment - Implemented as part of DM-5288 . DM-5322 was recast to simply remove duplicate or unused datasets.

              People

              Assignee:
              price Paul Price
              Reporter:
              price Paul Price
              Watchers:
              John Parejko, John Swinbank, Paul Price
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.