As for the defaults, I tried to set them to what they effectively were in interpolateOnePlane (then run!) in interpImage.py. In repair.py, as you pointed out in your initial review, the default fallbackValue was being set differently, so I included the setDefaults to reflect what was being used there. I'm inclined to leave that as is since it is not unreasonable to want different settings for certain defect types (e.g. true defective CCD pixels versus a mask plane). This could certainly be revisited if the interpImage task becomes more frequently implemented and the callers are finding that a different set of defaults would be more appropriate.