Skip to content

Instantly share code, notes, and snippets.

@parejkoj
Last active January 10, 2020 19:27
Show Gist options
  • Save parejkoj/562bc9eae242d2dc54aea4148972a55f to your computer and use it in GitHub Desktop.
Save parejkoj/562bc9eae242d2dc54aea4148972a55f to your computer and use it in GitHub Desktop.

New proposal for handling calibrations in gen3

Definitions

Some definitions, since precision in this terminology can matter a lot.

  • ingest: move/copy/symlink files from outside of a butler repository and butler.ingest them to a butler repository adding Dimension data to the registry as necessary.
  • put: write in-memory objects to a butler repository using butler.put: this is distinct from ingest because the butler handles all registry updates and the objects have to be in memory, not as on-disk files.
  • export/import: read data from one butler repository and put it into a different butler repository.
  • raw: an un-modified image from a camera (including bias, dark, flat, science, etc.).
  • calibrations: data that is used to convert raw science images to processed images (includes master bias/dark/flat images, defect lists, linearity corrections, crosstalk corrections, brighter-fatter kernels).

Proposal

The only ingest that exists will be lsst.obs.base.RawIngestTask. Once raw files are ingested into a butler, cp_pipe is used to generate the calibrations that are used for processing.

If a user does not want to run cp_pipe to produce calibrations themselves (due to disk space or time limitations), they will export the desired calibrations from an existing butler repository and import them into their own. This is the only supported system by which calibrations can get into a repository without running cp_pipe.

To help us transition from gen2, we will provide a version of the gen2convert.py script to get create working gen3 repositories for the existing testdata that have externally-supplied calibration data. This is not a long term solution, but only exists to maintain our existing testdata.

Known issues

butler.export/import support

butler.export is not fully supported currently. According to @JimBosch these exports currently cannot go into a repository in a different directory. There is no ticket for this at present.

Externally-supplied calibrations

This approach does not allow us to ingest externally-supplied calibrations. For existing test datasets (e.g. ap_verify), the work-around would be to make the existing gen2convert code work on any gen2 repo (not just HSC), and use that to bootstrap gen3 repos with existing external master calibration data.

User-adjustments to master calibrations

This proposal does not support user-tweaked master calibrations. For example, someone notices a chip defect that isn't caught by cp_pipe and wants to add it to the defect list. cp_pipe would have to be updated to correctly identify that defect.

It's not clear to me what the current system for dealing with this is. What is the source of truth, the obs_*_data git repo or some particular on-disk butler somewhere? This proposal sidesteps that issue by having one source of truth: cp_pipe.

obs_*_data

The obs_*_data packages would become either obsolete and removed, or would have to contain butler.exported repository subsets instead of "user-curated text files". If we do want a place where users can download master calibrations supported by LSST, we could use these for that purpose.

Camera geometry

Dealing with the Camera Geometry is a fundamental part of this question: linearity, crosstalk, gains, overscan size could each change over time. Potentially anything in the Amplifier object could change over time other than the rawBBox/rawDataBBox.

However, I don't think that decisions about what to do with Camera Geometry affect whether this proposal is workable: we don't currently have the ability to version camera parts of the camera geometry, and updates to those values should come from DM code (e.g. cp_pipe, jointcal, fgcm), which is consistent with this overall proposal (said code will read from a butler, and write back to it).

@kfindeisen
Copy link

kfindeisen commented Jan 10, 2020

Once raw files are ingested into a butler, cp_pipe is used to generate the calibrations that are used for processing. If a user does not want to run cp_pipe to produce calibrations themselves (due to disk space or time limitations), they will export the desired calibrations from an existing butler repository and import them into their own.

So what does this mean for operations? Do we export/import calibrations (and templates) before running the prompt products pipeline? Do we do this at the start of the night, or on every readout?

For existing test datasets (e.g. ap_verify), the work-around would be to make the existing gen2convert code work on any gen2 repo (not just HSC), and use that to bootstrap gen3 repos with existing external master calibration data.

I think it's more important that the ap_verify input resemble the operations environment (hence my previous question) than that it resemble the Gen 2 ap_verify dataset framework.

@parejkoj
Copy link
Author

For prompt products, I would expect that the PP butler repo would have the latest "blessed" versions of the calibrations produced by cp_pipe. They could have gotten there via the standard cp_pipe approach+blessing, or by export/import from some "master" butler repo. This would be done whenever master calibrations are updated (which is probably weekly to monthly-ish).

ap_verify would resemble the operations environment under the above scheme: we never run cp_pipe as part of ap_pipe, we just have a set of "blessed" master calibrations in the repo available for ap_pipe to use (either from an earlier run of cp_pipe in that repo, or from an export/import from another repo). Having a skeleton gen3 repo with master calibrations in it to export/import from to bootstrap the process for ap_verify seems like the right approach to me: it gives you something that looks like a typical ap_pipe starting point (a gen3 butler repo with master calibrations in it).

@mfisherlevine
Copy link

Apparently you can't 👍 a comment on a gist, so consider this a 👍...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment