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

Disable CModel in forcedPhotCcd

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ci_hsc, obs_subaru
    • Labels:
      None
    • Team:
      External

      Description

      obs_subaru's config for forcedPhotCcd loads CModel, but CModel doesn't yet implement forced photometry when WCSs differ, leading to a fatal error when attempting to use forcedPhotCcd.py to do forced photometry on individual exposures. Use of CModel is disabled for forcedPhotCcd in ci_hsc, which demonstrates that it shouldn't be there in the first place.

        Attachments

          Activity

          No builds found.
          price Paul Price created issue -
          price Paul Price made changes -
          Field Original Value New Value
          Risk Score 0
          Show
          price Paul Price added a comment - https://github.com/lsst/obs_subaru/issues/154
          Hide
          price Paul Price added a comment -

          Jim Bosch, I think you put the line there originally. Are you OK with its removal?

          Show
          price Paul Price added a comment - Jim Bosch , I think you put the line there originally. Are you OK with its removal?
          price Paul Price made changes -
          Reviewers Jim Bosch [ jbosch ]
          Status To Do [ 10001 ] In Review [ 10004 ]
          Hide
          price Paul Price added a comment -

          Seems to be going through Jenkins just fine.

          Show
          price Paul Price added a comment - Seems to be going through Jenkins just fine.
          Hide
          jbosch Jim Bosch added a comment -

          Looks good!  Thanks for taking care of this.

          Show
          jbosch Jim Bosch added a comment - Looks good!  Thanks for taking care of this.
          jbosch Jim Bosch made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Hide
          price Paul Price added a comment -

          Thanks Jim!

          Merged to master.

          Show
          price Paul Price added a comment - Thanks Jim! Merged to master.
          price Paul Price made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            price Paul Price
            Reporter:
            price Paul Price
            Reviewers:
            Jim Bosch
            Watchers:
            Jim Bosch, Paul Price
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.