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

Variance array resampling differs from jwst #1387

Open
braingram opened this issue Aug 27, 2024 · 1 comment
Open

Variance array resampling differs from jwst #1387

braingram opened this issue Aug 27, 2024 · 1 comment

Comments

@braingram
Copy link
Collaborator

spacetelescope/jwst#8344 described issues and proposed updates for resampling variance arrays in jwst.

spacetelescope/jwst#8437 updated jwst with one of the proposed solutions.

The variance array resampling in romancal doesn't appear to have similar updates and is instead using the older version of the variance array resampling.

@schlafly
Copy link
Collaborator

Thanks Brett. FWIW, I think in Roman we're happy to follow Webb here, e.g., as part of future refactoring to use more common code in stcal. In discussions with Stefano in the past I don't think RTB likes our current variance computations or the new Webb ones and prefers yet another option based on the weight images, but we haven't coded that up and it's only been requested insofar as I've mentioned that the current variance computation seems incorrect to me.

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

2 participants