Open bpartridge opened 10 years ago
Hi Brenton,
Sorry for leaving this on the back burner for so long! We could instead add a flag to convert the image into photo-electron units, if that would be useful. I'm a little hesitant to carry around extra image-sized arrays -- that code path is traversed in a lot of projects that don't care about Poisson statistics :)
cheers, --dustin
[Low priority feature contribution]
Since we're incorporating sky noise in the model for the Inference project, it's useful to expose in
tractor.Image
instances the calculations described in http://data.sdss3.org/datamodel/files/BOSS_PHOTOOBJ/frames/RERUN/RUN/CAMCOL/frame.html :This patch simply returns image-size vectors in
get_tractor_image_dr8
(which is also called if DR9 is passed) for the data-numbers and electron-counts. The caller can then do something likeimg, info = ...; img.counts = np.round(info['dn'])
.