c3-time-domain / SeeChange

A time-domain data reduction pipeline (e.g., for handling images->lightcurves) for surveys like DECam and LS4
BSD 3-Clause "New" or "Revised" License
0 stars 4 forks source link

Merge WCS and ZP into extraction step #257

Closed guynir42 closed 4 months ago

guynir42 commented 6 months ago

To help simplify our versioning / provenance model we should combine all the steps after producing a preprocessed image and before subtraction into a single step. This includes:

I suggest we just continue to call this "extraction" but I'm open to other ideas for the name.

The upside is that we simplify things dramatically (see Will's PR on upstreams/downstreams). The downside is that when we want to only change the astrometric/photometric calibration then we'd have to (a) recalculate stuff like sextractor and (b) have to re-save another source list, psf and background objects, instead of just re-running the crossmatch and saving just the WCS or ZP. I think this is well worth the simplification.

guynir42 commented 4 months ago

Addressed by #296