Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong assertion in GainTemplate tests #613

Open
giuspugl opened this issue Nov 25, 2022 · 1 comment
Open

Wrong assertion in GainTemplate tests #613

giuspugl opened this issue Nov 25, 2022 · 1 comment
Assignees

Comments

@giuspugl
Copy link
Contributor

giuspugl commented Nov 25, 2022

the result detdata key

result = Unicode(
should encode the TOD calibrated, i.e. corrected by the gain fluctuations , expressed as the linear combination of Legendre polinomials weighted by the amplitudes estimated by the toast mapmaker.
Instead the test
np.testing.assert_allclose(
asserts exactly that the result data to be an array of np.ones local samples.
What we should compare to be close to 1 should be instead the ops.Calibrate.template_matrix.templates[0].data['calibrator_solve_amplitudes']

@giuspugl
Copy link
Contributor Author

Also in

combine = Combine(op=self.op, first=self.det_data, second=projected)

what we should be dividing is not the template by the uncalibrated signal but the fitted gain amplitudes.
Similarly to what we do in
def calibrate(self, signal, amplitudes):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants