New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-4901: Use yaml configuration files to store camera-specific data ID and ref image information for validation testing. #6
Conversation
wmwv
commented
Feb 13, 2016
- Add YAML config files. Use to store information about match parameters and dataIds to run and analyze.
- Refactor+Centralize to validateDrp.py and runExample.sh to put all of the logic in one place.
- validateDrp.py now accepts a directory and a configuration YAML file.
- The per-camera example scripts now are light wrappers for each camera.
Create test for loading of the config file. Move getCcdKeyName to `util` to make generally available.
Simplified names of (check,plot)AstrometryPhotometry.py to check.py, plot.py Added print.py to handle printing. Include YAML example file for CFHT. Can now run with, e.g.: validateDrp.py CFHT/output examples/runCfht.yaml
test.log | ||
bin/validateCfht.py | ||
bin/validateDecam.py | ||
*.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it actually necessary to ignore all of this stuff? As it stands, it won't do any harm, but it's just going to cause confusion in future when somebody wants to check a JSON file into the repository. Plus I don't see the benefit at the moment -- it doesn't look like anything you're doing is actually creating JSON files, so it's not likely that they'll accidentally be committed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, this was a bit forward-looking to DM-5121, which will create JSON output files.
a3c410b
to
fe11df4
Compare
Create Quick versions for Cfht and Decam for debugging runs. Just one CCD. Note these will fail AM2, because they're just one CCD and AM2 is looking for stars separated by 20 arcminutes. Pass MAPPER, INGEST, PROCESSCCD to runExample. runExample needs to know the CAMERA, MAPPER, the CONFIG_FILE and any specially needed versions of ingestImages.py or processCcd.py Specifically DECam needs these custom versions.
Also remove 'run' prefix from .YAML files.
Add description of Quick examples to README Use hw.logicalcpu on Darwin.
fe11df4
to
a673f37
Compare