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

Color terms in obs_cfht needs some corrections

    Details

    • Type: Bug
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: obs_cfht
    • Labels:
      None
    • Team:
      External

      Description

      There are several problems in obs_cfht/config/colorterms.py:

      1. the naming convention for Megacam --> SDSS colorterms is inadequate as it is called "e2v". It should be renamed "sdss" as in obs_subaru/hsc
      2. there are missing color terms for SDSS corresponding to Megacam filters: u2, g2, r2, i2, z2 and i3.
      3. the current color terms for panstarrs are identical to the sdss ones
      4. there is a color term for u band in panstarrs while this filter doesn't exist

        Attachments

          Activity

          Hide
          boutigny Dominique Boutigny added a comment -

          http://www.cadc-ccda.hia-iha.nrc-cnrc.gc.ca/en/megapipe/docs/filt.html seems to be the reference for Megacam–>SDSS–>Pan-STARRS color terms

          Show
          boutigny Dominique Boutigny added a comment - http://www.cadc-ccda.hia-iha.nrc-cnrc.gc.ca/en/megapipe/docs/filt.html  seems to be the reference for Megacam–>SDSS–>Pan-STARRS color terms
          Hide
          jbosch Jim Bosch added a comment -

          Lauren MacArthur, do you know enough about how our color term code works for this to be an easy thing for you to do?

          If so, please help me to remember to bring it up next time we do DRP pair-coding so we can both get it done and increase the number of people who know how the color-term code works.

          Dominique Boutigny, I'm guessing we'll be able to get to this sometime in the next month - if it'd be particularly helpful to get it sooner, please shout.

          Show
          jbosch Jim Bosch added a comment - Lauren MacArthur , do you know enough about how our color term code works for this to be an easy thing for you to do? If so, please help me to remember to bring it up next time we do DRP pair-coding so we can both get it done and increase the number of people who know how the color-term code works. Dominique Boutigny , I'm guessing we'll be able to get to this sometime in the next month - if it'd be particularly helpful to get it sooner, please shout.
          Hide
          lauren Lauren MacArthur added a comment - - edited

          While I wouldn’t claim to be intimately familiar, it looks like item 1. should be easy enough.  Adding new colour terms should also be easy if the numbers we need can all be gleaned from the linked site.

          Show
          lauren Lauren MacArthur added a comment - - edited While I wouldn’t claim to be intimately familiar, it looks like item 1. should be easy enough.  Adding new colour terms should also be easy if the numbers we need can all be gleaned from the linked site.
          Hide
          boutigny Dominique Boutigny added a comment - - edited

          Céline Combet, Marina Ricci and myself are working on color terms for CFHT Megacam, so I think that we can handle this ticket. We will also take care to implement the new color terms using the column names used in the reference catalogs available on lsst-dev:/datasets/refcats (see discussion on: https://lsstc.slack.com/archives/C2K97E3L5/p1554304959026300)

          Show
          boutigny Dominique Boutigny added a comment - - edited Céline Combet, Marina Ricci and myself are working on color terms for CFHT Megacam, so I think that we can handle this ticket. We will also take care to implement the new color terms using the column names used in the reference catalogs available on lsst-dev:/datasets/refcats (see discussion on:  https://lsstc.slack.com/archives/C2K97E3L5/p1554304959026300 )

            People

            • Assignee:
              boutigny Dominique Boutigny
              Reporter:
              boutigny Dominique Boutigny
              Watchers:
              Dominique Boutigny, Jim Bosch, John Parejko, Lauren MacArthur
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:

                Summary Panel