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

Adapt `validate_drp` to standard python and bin subdir sturcture

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • None
    • None

    Description

      Move Python files into python/lsst namespace convention.
      Decide on where validateCfht.py and validateDecam.py executables should live
      Add package requirements to ups/validate_drp.table

      Attachments

        Activity

          quick review for code reorganizing to match standard lsst/templates

          Probably takes 30 min - 1 hour to run the validation examples all the way if you want to do that. There aren't any changes that should change the validation example behavior, except for the addition of a log file.

          wmwood-vasey Michael Wood-Vasey added a comment - quick review for code reorganizing to match standard lsst/templates Probably takes 30 min - 1 hour to run the validation examples all the way if you want to do that. There aren't any changes that should change the validation example behavior, except for the addition of a log file.
          tjenness Tim Jenness added a comment -

          Thank you for reorganizing. The file validCfht.py seems to be at the top level still and will need its shebang fixing.

          The end of the README.md file looks like it still needs tweaking: there is a problem with the ordered list item numbering and the last section does not seem to reflect the locations of the files.

          Should there be a team associated with this ticket?

          tjenness Tim Jenness added a comment - Thank you for reorganizing. The file validCfht.py seems to be at the top level still and will need its shebang fixing. The end of the README.md file looks like it still needs tweaking: there is a problem with the ordered list item numbering and the last section does not seem to reflect the locations of the files. Should there be a team associated with this ticket?

          Thanks very much, tjenness

          README fixed up. Extraneous validCfht.py removed (a refuge from a rebase on top of another rebased and squashed branch – which taught me my lesson.)

          wmwood-vasey Michael Wood-Vasey added a comment - Thanks very much, tjenness README fixed up. Extraneous validCfht.py removed (a refuge from a rebase on top of another rebased and squashed branch – which taught me my lesson.)

          Merged into master.

          wmwood-vasey Michael Wood-Vasey added a comment - Merged into master.

          People

            wmwood-vasey Michael Wood-Vasey
            wmwood-vasey Michael Wood-Vasey
            Tim Jenness
            Michael Wood-Vasey, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.