Hopefully a quick review.

Small refactor of ForcedPhotImageTask to add a run method. ForcedPhotCcd and ForcedPhotCoadd will both inherit this method.

Thank for doing this. I had a few small requested changes on github.

Only because I'm a watcher on this ticket: This implementation looks incomplete because you're just passing the DataRef to run.

You can call the IdFactory in runDataRef and pass it to run. Do any of the children's attachFootprints()  actually need the DataRef? You might be able to change that API.

OK, I see the answer to my question is yes a child needs to fetch the heavy footprints from disk. So maybe everything down and including the call to `attachFootprints()` should be part of `runDataRef` and run just does

    self.measurement.run(measCat, exposure, refCat, refWcs, 

    if self.config.doApCorr:

            self.applyApCorr.run(

                catalog=measCat,

                apCorrMap=exposure.getInfo().getApCorrMap()

            )

    self.catalogCalculation.run(measCat)

Good catch. Sorry I missed that and I agree with Yusra AlSayyad. I'll update my comments on github.

That makes sense to isolate the DataRef from run().

I'm running a new version through Jenkins to make sure I have everything correct in the new code, and will update the branch once that passes.