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

Add --show history option to cmdLineTask

    XMLWordPrintable

    Details

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

      Description

      pex_config is able to report where a config parameter is set. Please add a command line option --show history=config.parameter.name to the cmdLineTask parser.

      The implementation will probably want to use something like:

      import lsst.pex.config.history as pch
      		 
      pch.Color.colorize(False)
      		 
      print pch.format(config.calibrate.astrometry.solver, "matchingRadius")

        Attachments

          Issue Links

          is duplicated by

          Improvement - An improvement or enhancement to an existing feature or task. DM-3812 Please provide a command-line way of getting the history of a config parameter

          • Invalid
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            rhl Robert Lupton created issue -
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Team Process Middleware [ 10206 ]
            Hide
            Permalink
            rhl Robert Lupton added a comment -

            You can control the colouring using

                    import lsst.pex.config.history
            		 
                    lsst.pex.config.history.Color.colorize(False)

            DM-7940 provides a way to disable the colouring by piping to cat.

            Show
            rhl Robert Lupton added a comment - You can control the colouring using import lsst.pex.config.history lsst.pex.config.history.Color.colorize(False) DM-7940 provides a way to disable the colouring by piping to cat .
            rhl Robert Lupton made changes -
            Reviewers Russell Owen [ rowen ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Nice feature, but please simplify the implementation as per github comments (which I tried to mark as a review, but could not because I created the pull request).

            Show
            rowen Russell Owen added a comment - Nice feature, but please simplify the implementation as per github comments (which I tried to mark as a review, but could not because I created the pull request).
            rowen Russell Owen made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            Permalink
            rhl Robert Lupton added a comment -

            Merged to master post-code review improvements

            Show
            rhl Robert Lupton added a comment - Merged to master post-code review improvements
            rhl Robert Lupton made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            rhl Robert Lupton made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ]
            tjenness Tim Jenness made changes -
            Description {{pex_config}} is able to report where a config parameter is set. Please add a command line option {{--show history=config.parameter.name}} to the cmdLineTask parser.

            The implementation will probably want to use something like:
            {quote}
            import lsst.pex.config.history as pch
            pch.Color.colorize(False)
            print pch.format(config.calibrate.astrometry.solver, "matchingRadius")
            {quote}
            		{{pex_config}} is able to report where a config parameter is set. Please add a command line option {{--show history=config.parameter.name}} to the cmdLineTask parser.

            The implementation will probably want to use something like:
            {code:python}
            import lsst.pex.config.history as pch
            pch.Color.colorize(False)
            print pch.format(config.calibrate.astrometry.solver, "matchingRadius")
            {code}
            tjenness Tim Jenness made changes -
            Team Process Middleware [ 10206 ] Data Release Production [ 10301 ]
            tjenness Tim Jenness made changes -
            Assignee Robert Lupton [ rhl ]
            swinbank John Swinbank made changes -
            Epic Link DM-6172 [ 24685 ]
            swinbank John Swinbank made changes -
            Sprint DRP F16-5 [ 252 ]
            Story Points 2
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            jbosch Jim Bosch made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            rowen Russell Owen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            rowen Russell Owen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            krzys Krzysztof Findeisen made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            swinbank John Swinbank made changes -
            Remote Link This issue links to "Page (Confluence)" [ 14339 ] This issue links to "Page (Confluence)" [ 14339 ]
            rhl Robert Lupton made changes -
            Link This issue is duplicated by DM-3812 [ DM-3812 ]

              People

              Assignee:
              rhl Robert Lupton
              Reporter:
              rhl Robert Lupton
              Reviewers:
              Russell Owen
              Watchers:
              Robert Lupton, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.