Adapt validate_drp to standard python and bin subdir sturcture

XMLWordPrintable

Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s: None
• Labels:
None
• Story Points:
0.5
• Team:
SQuaRE

Description

Move Python files into python/lsst namespace convention.
Decide on where validateCfht.py and validateDecam.py executables should live

Activity

Hide
Michael Wood-Vasey [X] (Inactive) 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.

Show
Michael Wood-Vasey [X] (Inactive) 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.
Hide
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?

Show
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?
Hide
Michael Wood-Vasey [X] (Inactive) added a comment -

Thanks very much, Tim Jenness

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.)

Show
Michael Wood-Vasey [X] (Inactive) added a comment - Thanks very much, Tim Jenness 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.)
Hide
Michael Wood-Vasey [X] (Inactive) added a comment -

Merged into master.

Show
Michael Wood-Vasey [X] (Inactive) added a comment - Merged into master.

People

Assignee:
Michael Wood-Vasey [X] (Inactive)
Reporter:
Michael Wood-Vasey [X] (Inactive)
Reviewers:
Tim Jenness
Watchers:
Michael Wood-Vasey [X] (Inactive), Tim Jenness