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

obs_decam's apPipe config should default to CP calibs for now

    XMLWordPrintable

    Details

    • Story Points:
      0.5
    • Sprint:
      AP F18-4
    • Team:
      Alert Production

      Description

      For now, the majority of ap_pipe use cases with obs_decam use community pipeline (CP) bias and flat calibration products. It is well known that these require a separate set of processCcd configs, and when running processCcd from the command line, users typically provide the obs_decam/config/processCcdCpIsr.py file.

      However, the obs_decam/config/apPipe.py file (which ap_pipe automatically uses! no command-line config specification required!) presently points to obs_decam/config/processCcd.py. This causes the infamous "No locations for get: bias" error because there are no biases, there are instead cpBiases.

      This ticket is to switch the default in obs_decam/config/apPipe.py to point to obs_decam/config/processCcdCpIsr.py and add a comment so future ap_pipe users are hopefully less confused than I was.

        Attachments

          Activity

          No builds found.
          mrawls Meredith Rawls created issue -
          mrawls Meredith Rawls made changes -
          Field Original Value New Value
          Risk Score 0
          mrawls Meredith Rawls made changes -
          Epic Link DM-14447 [ 80385 ]
          mrawls Meredith Rawls made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          mrawls Meredith Rawls made changes -
          Description For now, the majority of ap_pipe use cases with obs_decam use community pipeline (CP) bias and flat calibration products. It is well known that these require a separate set of processCcd configs, and when running processCcd from the command line, users typically provide the {{obs_decam/config/processCcdCpIsr.py}} file (or, if they are not using CP calibrations, they provide the {{obs_decam/config/processCcd.py}} file instead).

          However, the {{obs_decam/config/apPipe.py}} file (which ap_pipe automatically uses! no command-line config specification required!) presently points to {{obs_decam/config/processCcd.py}}. This causes the infamous "No locations for get: bias" error because there are no biases, there are instead cpBiases.

          This ticket is to switch the default in {{obs_decam/config/apPipe.py}} to point to {{obs_decam/config/processCcdCpIsr.py}} and add a comment so future ap_pipe users are hopefully less confused than I was.
          For now, the majority of ap_pipe use cases with obs_decam use community pipeline (CP) bias and flat calibration products. It is well known that these require a separate set of processCcd configs, and when running processCcd from the command line, users typically provide the {{obs_decam/config/processCcdCpIsr.py}} file.

          However, the {{obs_decam/config/apPipe.py}} file (which ap_pipe automatically uses! no command-line config specification required!) presently points to {{obs_decam/config/processCcd.py}}. This causes the infamous "No locations for get: bias" error because there are no biases, there are instead cpBiases.

          This ticket is to switch the default in {{obs_decam/config/apPipe.py}} to point to {{obs_decam/config/processCcdCpIsr.py}} and add a comment so future ap_pipe users are hopefully less confused than I was.
          Hide
          mrawls Meredith Rawls added a comment -

          This small change affects a config file for apPipe, which is not yet in lsst_distrib. I have not run Jenkins, since this config file is never used as far as it is concerned.

          Show
          mrawls Meredith Rawls added a comment - This small change affects a config file for apPipe, which is not yet in lsst_distrib. I have not run Jenkins, since this config file is never used as far as it is concerned.
          mrawls Meredith Rawls made changes -
          Reviewers Chris Morrison [ cmorrison ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          mrawls Meredith Rawls made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            mrawls Meredith Rawls
            Reporter:
            mrawls Meredith Rawls
            Reviewers:
            Chris Morrison [X] (Inactive)
            Watchers:
            Chris Morrison [X] (Inactive), Eric Morganson [X] (Inactive), Krzysztof Findeisen, Meredith Rawls
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.