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

Fix ci_imsim_run.py to print all available cmdline arguments

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ci_imsim
    • Labels:
      None

      Description

      ci_imsim_run.py (the command to process the testdata and run the tests) does not print useful help information. There appears to be some nested set of argument parsers, and the inner ones aren't passing their available args up to the one that parses the h. Please fix this so that users can know what args are available (e.g. -clean or -j, if the latter is supported).

      $ bin/ci_imsim_run.py -h
      		 
      usage: ci_imsim [-h] [--use_skymap_small]
      		 
       
      		 
      Run ci_imsim on testdata_ci_imsim data
      		 
       
      		 
      options:
      		 
        -h, --help          show this help message and exit
      		 
        --use_skymap_small  Use a smaller 2k x 2k pixel patch skymap instead of 4k^2
      		 
                            default

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-33450 ci_imsim has inconsistent default behavior w.r.t. useCiLimits options

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            dtaranu Dan Taranu added a comment -

            You'll be happy to learn that I also removed the NCSA-era ctrl_bps example yamls. In principle it would be nice to have updated USDF versions, but I can't think of a good way to prevent them from bitrotting and haven't used bps at the USDF yet anyway.

            Show
            dtaranu Dan Taranu added a comment - You'll be happy to learn that I also removed the NCSA-era ctrl_bps example yamls. In principle it would be nice to have updated USDF versions, but I can't think of a good way to prevent them from bitrotting and haven't used bps at the USDF yet anyway.
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Thanks for the fix: I confirm that the help output appears now.

            Show
            Parejkoj John Parejko added a comment - Thanks for the fix: I confirm that the help output appears now.
            Hide
            Permalink
            dtaranu Dan Taranu added a comment -

            Passed Jenkins.

            Show
            dtaranu Dan Taranu added a comment - Passed Jenkins .

              People

              Assignee:
              dtaranu Dan Taranu
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              John Parejko
              Watchers:
              Dan Taranu, John Parejko, Kian-Tat Lim, Nate Lust, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.