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

Puzzling behavior of matched apertures #219

Open
wants to merge 1 commit into
base: siaf-updates
Choose a base branch
from

Conversation

mgennaro
Copy link
Contributor

The grism_ts_sw NIRcam apertures are apertures on the SW detectors that "tag along" existing LW apertures. The generate_nircam.py code calls the tools.match_v2v3 method to match the (V2,V3) of the tag along apertures to a corresponding LW one.

In theory the method should also modify the distortion coefficients associated to the SW aperture, once the SW aperture is moved to match the LW one.

I have however seen that explicitly calling the tools.set_reference_point_and_distortion method after the tools.match_v2v3 further modifies the aperture. Several quantities are affected: V2Ref, V3Ref, V3SciXAngle, V3SciYAngle, Idl2SciX00, Idl2SciY00 and this, for example changes the roundtrip erorr.
I wonder if this is the expected behavior. In theory I would be expecting the second call to tools.set_reference_point_and_distortion to have no effect.

For consistency with the treatment of all other apertures distortion coefficient I wonder if we should explicitly call tools.set_reference_point_and_distortion once more, as the generate_nircam code in this PR does.
But possibly first we need to understand what is really going on behind the scenes

@mgennaro mgennaro changed the title Modified generate_nircam flow for grism_ts_sw apertures Puzzling behavior of matched apertures Mar 23, 2022
@Witchblade101
Copy link
Collaborator

@mgennaro I just want to confirm that the new threshold should be 0.2:

test_aperture.test_jwst_aperture_transforms([pre_delivery_siaf], verbose=False, threshold=0.2)

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

Successfully merging this pull request may close these issues.

None yet

2 participants