-
Notifications
You must be signed in to change notification settings - Fork 17
Description
This is not a bug report or really an "issue" but more of a conversation. I was encouraged to post here. @paganol @ziotom78 @marcobortolami @silviamich @fpiacentini
binner.py adds each data sample to the IQU maps by inverting the 3x3 matrix and assuming there is sinusoidal modulation of the polarization signals at 4f.
info_pix[1, 0] += cur_sample * inv_sigma2
info_pix[2, 0] += cur_sample * cos_over_sigma * inv_sigma
info_pix[2, 1] += cur_sample * sin_over_sigma * inv_sigma
I believe this is a good method for reconstructing the temperature and polarization maps from time-ordered data that consists primarily of a smoothly scan-and-sampled IQU map with additive white noise. However, for more realistic data with more complex features (e.g. non-white features like 1/f noise, line noise, and a strong 2f line), this binning mapmaker is either inappropriate or requires heavy pre-filtering such as a band-pass around 4f for reconstructing polarization maps. It would be good to a) acknowledge clearly the limitations of this method and b) write an alternative mapmaker that doesn't separate IQU on a per-sample basis, but either implicitly or explicitly demodulates the half-wave plate polarization modulation.