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

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

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • ctrl_pool
    • None
    • 2
    • DRP F16-5
    • 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.

      Attachments

        Issue Links

          Activity

            No builds found.
            rhl Robert Lupton created issue -
            rhl Robert Lupton made changes -
            Field Original Value New Value
            Summary Add {{--batch-type None}} to possibilities, disabling any MPI Add --batch-type None to possibilities, disabling any MPI
            rhl Robert Lupton made changes -
            Reviewers Paul Price [ price ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            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.

            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 .
            price Paul Price made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            swinbank John Swinbank made changes -
            Epic Link DM-6172 [ 24685 ]
            swinbank John Swinbank made changes -
            Sprint DRP F16-5 [ 252 ]
            Story Points 2
            Team Data Release Production [ 10301 ]

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

            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.
            rhl Robert Lupton made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            price Paul Price made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            pschella Pim Schellart [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            pschella Pim Schellart [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            rhl Robert Lupton made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]

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

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

            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.

            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.
            rhl Robert Lupton made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggering DM-7958 [ DM-7958 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            jbosch Jim Bosch made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            rowen Russell Owen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            rowen Russell Owen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            krzys Krzysztof Findeisen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14331 ] This issue links to "Page (Confluence)" [ 14331 ]

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.