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

            Activity

            No builds found.
            boutigny Dominique Boutigny created issue -
            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
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Team DM Science [ 12218 ]
            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.
            jbosch Jim Bosch made changes -
            Epic Link DM-16678 [ 235238 ]
            jbosch Jim Bosch made changes -
            Team DM Science [ 12218 ] Data Release Production [ 10301 ]
            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.
            boutigny Dominique Boutigny made changes -
            Assignee Dominique Boutigny [ boutigny ]
            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 )
            tjenness Tim Jenness made changes -
            Team Data Release Production [ 10301 ] External [ 12117 ]
            tjenness Tim Jenness made changes -
            Epic Link DM-16678 [ 235238 ]
            tjenness Tim Jenness made changes -
            Remote Link This issue links to "Page (Confluence)" [ 35290 ]
            Hide
            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
            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
            tjenness Tim Jenness made changes -
            Assignee Dominique Boutigny [ boutigny ]
            tjenness Tim Jenness made changes -
            Labels SciencePipelines
            tjenness Tim Jenness made changes -
            Team External [ 12117 ]
            Hide
            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.
            jbosch Jim Bosch made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Won't Fix [ 10405 ]

              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.