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

cleanup items for butler CLI

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      DB_F20_06
    • Team:
      Data Access and Database
    • Urgent?:
      No

      Description

      1. docstrings for define_visits and ingest_raws are missing so butler --help displays not-useful information for these commands

      2. change butler register-instrument so that instrument is an arg instead of an opt butler register-instrument REPO lsst.obs.lsst.Latiss (without the -i). It should take multiple values for INSTR.

      3. If obs_base is set up such that a commands yaml is found but obs_base is not importable, butler --help falls over in a big heap. I think we should protect the doImport in cmd = self.get_command(ctx, subcommand) and issue a “could not import sucommand X” warning but continue on.

      4. in butler --help, sort the commands alphabetically

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          I do have some comments on github but mainly:

          1. repetition of test_help test code.
          2. Use of capLog fixture
          3. Not allowing multiple instruments in register-instrument
          Show
          tjenness Tim Jenness added a comment - I do have some comments on github but mainly: repetition of test_help test code. Use of capLog fixture Not allowing multiple instruments in register-instrument

            People

            Assignee:
            npease Nate Pease [X] (Inactive)
            Reporter:
            npease Nate Pease [X] (Inactive)
            Reviewers:
            Tim Jenness
            Watchers:
            Nate Pease [X] (Inactive), Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.