Skip to content
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-3373 Port HSC code for generation of calibration products #28

Merged
merged 2 commits into from
Mar 8, 2016

Commits on Mar 8, 2016

  1. ingestCalibs: fix output directory for registry

    We aim to have the calib registry end up in the calib root directory. To
    this end, we modify RegisterTask.openRegistry to take a directory instead
    of the butler (the butler was only being used to determine the directory).
    
    Our only access to the calib root directory is through the "--calib"
    command-line argument; if that is not provided, write to the current
    working directory for the user to move into the correct place.
    PaulPrice committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    926ff0d View commit details
    Browse the repository at this point in the history
  2. ingestCalibs: make camera-agnostic

    Several assumptions within ingestCalibs were specific to obs_decam (esp.
    the use of the hard-coded "ccdnum" data identifier keyword). Made these
    assumptions configurable.
    
    This also involved updating the validity range calculations, which have
    been moved to a common place (fixSubsetValidity) instead of being in two
    places (IngestCalibsTask.run and fixSubsetValidity) and requiring users
    to set validStart and validEnd to dummy values.
    
    In the process, updated SQL to use placeholders instead of stringified
    values to proactively prevent escaping problems (similar to SQL injection).
    PaulPrice committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    7bc9153 View commit details
    Browse the repository at this point in the history