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

Add --batch-type None to possibilities, disabling any MPI

    Details

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

      Description

      It can be desirable to run ctrl_pool enabled commands (such as constructBias.py from pipe_drivers) without any batch system.

      Please add an option

      --batch-type None

      that simply runs the job in the current process.

        Issue Links

          Activity

          Hide
          price Paul Price added a comment -

          This is a useful feature. I made a few comments about the implementation on the GitHub PR. In particular, the argv parsing has a bug and should be replaced with an existing feature in the BatchArgumentParser.

          Show
          price Paul Price added a comment - This is a useful feature. I made a few comments about the implementation on the GitHub PR. In particular, the argv parsing has a bug and should be replaced with an existing feature in the BatchArgumentParser .
          Hide
          swinbank John Swinbank added a comment -

          Please remember to update the release notes (https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+F16+Release+Notes) when you merge this.

          Show
          swinbank John Swinbank added a comment - Please remember to update the release notes ( https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+F16+Release+Notes ) when you merge this.
          Hide
          swinbank John Swinbank added a comment -

          The PR seems to be merged, but the ticket isn't done. Robert Lupton: is that just an oversight, or are you planning more work here?

          Show
          swinbank John Swinbank added a comment - The PR seems to be merged, but the ticket isn't done. Robert Lupton : is that just an oversight, or are you planning more work here?
          Hide
          rhl Robert Lupton added a comment -

          Sorry, Done. But see also DM-7958 which points out that this ticket didn't handle logging correctly.

          It's still very useful as is, but we should address these points too.

          Show
          rhl Robert Lupton added a comment - Sorry, Done. But see also DM-7958 which points out that this ticket didn't handle logging correctly. It's still very useful as is, but we should address these points too.

            People

            • Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Reviewers:
              Paul Price
              Watchers:
              John Swinbank, Paul Price, Robert Lupton
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Agile