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

obs_base tests should not use daf_butler test configs

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • obs_base
    • None
    • 1
    • Architecture
    • No

    Description

      swinbank has noted that obs_base tests depend on a private butler config test file in DAF_BUTLER_DIR/tests/config. Those configs are not meant to be used outside of daf_butler since they contain unexpected formatters and storage classes designed specifically for internal daf_butler tests. obs_base should not be relying on those configurations. It should be sufficient for obs_base to use a default configuration made by Butler.makeRepo.

      I'm not entirely sure whether I should fix this or whether this should be handled by science pipelines.

      Attachments

        Issue Links

          Activity

            Parejkoj John Parejko added a comment -

            Lets wait on this until DM-20994 and DM-21016 are further along/completed. I'd started to rework some things related to this dependency as part of that work.

            Parejkoj John Parejko added a comment - Lets wait on this until DM-20994 and DM-21016 are further along/completed. I'd started to rework some things related to this dependency as part of that work.

            We should discuss what the timeline for that is.

            The proximate issue is that this makes the Pipelines Docker images clunky/annoying/less useful for development. That's not a crisis, but it'd be nice to see it resolved in the next few weeks. If we can bring DM-20994 and DM-21016 to a conclusion in that timeframe, it'd be great.

            swinbank John Swinbank added a comment - We should discuss what the timeline for that is. The proximate issue is that this makes the Pipelines Docker images clunky/annoying/less useful for development. That's not a crisis , but it'd be nice to see it resolved in the next few weeks. If we can bring DM-20994 and DM-21016 to a conclusion in that timeframe, it'd be great.

            I'm not entirely sure whether I should fix this or whether this should be handled by science pipelines.

            I'd suggest that whoever has time first “should” fix it, but given Parejkoj's comments above let's assume he'll take a look once the issues he refers to are moving.

            swinbank John Swinbank added a comment - I'm not entirely sure whether I should fix this or whether this should be handled by science pipelines. I'd suggest that whoever has time first “should” fix it, but given Parejkoj 's comments above let's assume he'll take a look once the issues he refers to are moving.
            tjenness Tim Jenness added a comment -

            I will take this on. It looks like it's only a couple of lines to fix it and I have obs_base open in my editor.

            tjenness Tim Jenness added a comment - I will take this on. It looks like it's only a couple of lines to fix it and I have obs_base open in my editor.

            Thank you for dealing with this: looks like a simple fix.

            Parejkoj John Parejko added a comment - Thank you for dealing with this: looks like a simple fix.

            People

              tjenness Tim Jenness
              tjenness Tim Jenness
              John Parejko
              John Parejko, John Swinbank, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.