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

Update/improve run_all script after data move

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_ctio0m9
    • Labels:
      None
    • Story Points:
      0.2
    • Sprint:
      DRP F17-3
    • Team:
      Data Release Production

      Description

      Update run_all script now that data on lsst-dev has moved. Add commented out line at the end showing how one reruns all long direct visits to help people in future.

        Attachments

          Activity

          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Should be a trivial review, though feel free to pass it on. Just wanted to get it updated and check it to help Augustin get back in the swing of things.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Should be a trivial review, though feel free to pass it on. Just wanted to get it updated and check it to help Augustin get back in the swing of things.
          Hide
          swinbank John Swinbank added a comment - - edited

          Per Slack, I don't see any need for all the &&​s. As far as I've understood what you're aiming for, using set -e at the top of the script should do just as well, even in plain old /bin/sh. Unless you know some particular reason you have to add all these &&​s, please don't.

          Other than that, hiding the "rerun long direct visits" behind a command line option rather than a comment would probably be nicer, but that's not a big deal.

          Show
          swinbank John Swinbank added a comment - - edited Per Slack, I don't see any need for all the && ​s. As far as I've understood what you're aiming for, using set -e at the top of the script should do just as well, even in plain old /bin/sh . Unless you know some particular reason you have to add all these && ​s, please don't. Other than that, hiding the "rerun long direct visits" behind a command line option rather than a comment would probably be nicer, but that's not a big deal.
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          Merged.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Merged.

            People

            Assignee:
            mfisherlevine Merlin Fisher-Levine
            Reporter:
            mfisherlevine Merlin Fisher-Levine
            Reviewers:
            John Swinbank
            Watchers:
            John Swinbank, Merlin Fisher-Levine
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.