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

DecamNullIsrTask uses incorrect arg order in butler.put

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None

      Description

      runDataRef() calls sensorRef.put("postISRCCD", exposure), when it should call sensorRef.put(exposure, "postISRCCD"). This write is disabled by default, and only causes trouble when doWrite is enabled.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          ctslater Colin Slater added a comment -

          Meredith Rawls, have a moment for a one-liner? We don't currently test processCcd on instcals, so there's not an easy test to add, and it doesn't seem worth the build time/effort to add such a run.

          Show
          ctslater Colin Slater added a comment - Meredith Rawls , have a moment for a one-liner? We don't currently test processCcd on instcals, so there's not an easy test to add, and it doesn't seem worth the build time/effort to add such a run.
          Hide
          Permalink
          mrawls Meredith Rawls added a comment -

          Good catch! Like I said over on GitHub, I agree it's not very feasible to add a test for instcal file operations.

          Show
          mrawls Meredith Rawls added a comment - Good catch! Like I said over on GitHub, I agree it's not very feasible to add a test for instcal file operations.
          Hide
          Permalink
          ctslater Colin Slater added a comment -

          Thanks; merged.

          Show
          ctslater Colin Slater added a comment - Thanks; merged.

            People

            Assignee:
            ctslater Colin Slater
            Reporter:
            ctslater Colin Slater
            Reviewers:
            Meredith Rawls
            Watchers:
            Colin Slater, Meredith Rawls
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.