As part of our weekly brown bag discussion, we did a deep dive into SourceDetectionTask and noticed a number of confusing or misleading docstrings. Here are the notes I took during the brown bag. I'll try to translate these into cleanups shortly, so I don't forget:
DetectionConfig: threshold, thresholdValue
detectFootprints: doSmooth & sigma
convolveImage: under what circumstances would you not want `doSmooth==True`?
Better document relationship between reEstimateBackground and thresholdParity? - `applyThreshold` has a fun pair of `if`s, which might then be removed in `clearUnwantedResults`
remove `makeSourceCatalog = run