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

Color terms in obs_cfht needs some corrections

    XMLWordPrintable

    Details

      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

          Issue Links

          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            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 )
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Dominique Boutigny do you still intend to work on this?

            Show
            tjenness Tim Jenness added a comment - Dominique Boutigny do you still intend to work on this?
            Hide
            Permalink
            boutigny Dominique Boutigny added a comment -

            Tim Jenness no I am not planning any work on this

            Show
            boutigny Dominique Boutigny added a comment - Tim Jenness  no I am not planning any work on this
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Calling this Won't Fix.  We're using obs_cfht less and less anyway.

            Show
            jbosch Jim Bosch added a comment - Calling this Won't Fix.  We're using obs_cfht less and less anyway.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              boutigny Dominique Boutigny
              Watchers:
              Dominique Boutigny, Jim Bosch, John Parejko, Lauren MacArthur, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.