Of course you can!  I also need to add a check on the source "width" (which I currently do by making it the generic BaseLimit class).  The change to make use of the source selectors (and the tasks already defined in sourceSelector.py) will mess up some config parameters/overrides, so I'm not going to do the adaptation until I've got RFC approval (i.e. not in this ticket).  I do suspect that review will come your way (), so you can let me know then if all ends up in line with what you had envisioned.

To justify/explain my choice of signalToNoiseMin=20 for the default, I took the HSC fluxMin=4000 override (default there is 12500 which seems WAY too high!  See also discussion on DM-16785) and looked for the closest equivalent in S/N.  There are two grey dashed vertical lines at flux = 4000 and 12500 in the following histogram of raw Flux:

The hatched histograms have had S/N cuts as indicated in the legend. Clearly, the S/N>20 (pink-ish hatched histogram) is a good match.

This will be for a future RFC, but while I'm at it, to justify preferring a S/N cut over a flux cut, the following histogram of S/N = rawFlux/rawFluxErr shows that the current fluxMin=4000 for HSC allows some very low S/N candidates into the selection (pink-ish hatched histogram):

Would you mind giving this a look?  The PR is here and Jenkins is happy.

See github for review.

Thanks again for the review. Jenkins passed. Merged to master

In DM-32624, it was noted that the fluxes plotted in the (now old) versions of the plots attached to this ticket are actually not representative of what went into the PSF modeling. They show the PsfFlux values from the src catalogs, but, of course, these are not the same as what went into the source modeling (as they aren't available without the PSF model!). Rather, what should be plotted is the PsfFlux values from the icSrc catalogs (where the "PSF" is some best guess place-holder). I have attached the now correct version of the two initial plots posted to this ticket:

So, we see that the difference between the default and obs_subaru fluxMin settings generally does not have much impact.  This is due to the detection thresholds that are in play for the icSrc catalog are set to:

detection.thresholdValue = 5.0

detection.includeThresholdMultiplier = 10.0

This also jives with the fact that there appears a cutoff roughly at S/N~60 for the sources in these catalogs (the odd lower S/N source does sneak in, and there are typically a few negative flux detections). As such, I am now going to advocate (likely on DM-32624) that the default signalToNoiseMin be set to 50 (instead of 20, and not 60 as the LSSTCam-imSim data have a cutoff much closer to 50, although for real data we perhaps want to go a bit higher) to better match the detection settings above.