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

duplicate keys in obs_base/policy/datasets.yaml

    XMLWordPrintable

    Details

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

      Description

      For example: 'processCcd_config' and `characterizeImage_config` both occur twice. Maybe others?

      Not a problem right now since the values are the same for a given key (at least for the two examples above), but could lead to errors in the future if, for example, second entry overrides the first.

      Should probably use a yaml loader that would raise an exception on duplicate keys, e.g., https://gist.github.com/pypt/94d747fe5180851196eb

        Attachments

          Activity

          Hide
          jmeyers314 Joshua Meyers added a comment -

          Also, just noticed that `isr_config` specifically refers to `lsst.obs.subaru.isr.SubaruIsrConfig`, which seems suspicious.

          Show
          jmeyers314 Joshua Meyers added a comment - Also, just noticed that `isr_config` specifically refers to `lsst.obs.subaru.isr.SubaruIsrConfig`, which seems suspicious.
          Hide
          csaunder Clare Saunders added a comment -

          Hey Lee, will you give this a review? This is running on jenkins at https://ci.lsst.codes/job/stack-os-matrix/30746/

          Show
          csaunder Clare Saunders added a comment - Hey Lee, will you give this a review? This is running on jenkins at  https://ci.lsst.codes/job/stack-os-matrix/30746/
          Hide
          lskelvin Lee Kelvin added a comment -

          Looks good, good to merge.

          Show
          lskelvin Lee Kelvin added a comment - Looks good, good to merge.

            People

            Assignee:
            csaunder Clare Saunders
            Reporter:
            jmeyers314 Joshua Meyers
            Reviewers:
            Lee Kelvin
            Watchers:
            Clare Saunders, Joshua Meyers, Lee Kelvin
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins Builds

                No builds found.