Details

    • Type: Bug
    • Status: Done
    • Priority: Major
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base, pipe_tasks
    • Labels:
      None
    • Templates:
    • Story Points:
      2
    • Epic Link:
    • Sprint:
      DRP F16-6
    • Team:
      Data Release Production

      Description

      Nicolas Chotard has discovered an inconsistency between the deepCoadd_forced_src catalogs and their accompanying _schema dataset in some processed Megacam data; one of these (I forget which) has a few additional fields, which mostly seem related to aperture correction.

      First step is to add a test for this in pipe_task's testCoadds.py; if that doesn't fail, we can try to reproduce in ci_hsc, and if that fails I'll ask for more details about the specific dataset.

        Activity

        Hide
        jbosch Jim Bosch added a comment -

        Russell Owen, could you take a look at this? The vast majority of the changes are new tests and test support code, along with a one-line fix for the actual bug in meas_base.

        The meas_base and pipe_tasks branches are branched from and diffed against DM-8210, as the rely on some code there that hasn't been merged to master yet. I'll rebase this ticket once that happens.

        Show
        jbosch Jim Bosch added a comment - Russell Owen , could you take a look at this? The vast majority of the changes are new tests and test support code, along with a one-line fix for the actual bug in meas_base. The meas_base and pipe_tasks branches are branched from and diffed against DM-8210 , as the rely on some code there that hasn't been merged to master yet. I'll rebase this ticket once that happens.
        Hide
        jbosch Jim Bosch added a comment -

        DM-8210 has now been merged to master, but I'm not going to rebase this ticket's branches until after review, since I can't change the base of the PR diff.

        Show
        jbosch Jim Bosch added a comment - DM-8210 has now been merged to master, but I'm not going to rebase this ticket's branches until after review, since I can't change the base of the PR diff.
        Hide
        rowen Russell Owen added a comment -

        A very nice enhancement to the tests, in addition to fixing that bug. I had a few trivial requests on github.

        Show
        rowen Russell Owen added a comment - A very nice enhancement to the tests, in addition to fixing that bug. I had a few trivial requests on github.
        Hide
        jbosch Jim Bosch added a comment -

        Merged to master.

        Show
        jbosch Jim Bosch added a comment - Merged to master.

          People

          • Assignee:
            jbosch Jim Bosch
            Reporter:
            jbosch Jim Bosch
            Reviewers:
            Russell Owen
            Watchers:
            Jim Bosch, Russell Owen
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile