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

Saturation step update #817

Closed
stscijgbot-rstdms opened this issue Aug 9, 2023 · 2 comments
Closed

Saturation step update #817

stscijgbot-rstdms opened this issue Aug 9, 2023 · 2 comments
Assignees
Milestone

Comments

@stscijgbot-rstdms
Copy link
Collaborator

Issue RCAL-618 was created on JIRA by Eddie Schlafly:

Update the saturation step to account for the fact that the saturation reference file gives the saturation levels for a {}read{}, while Roman measures {}resultants{}.  This amounts to multiplying the cut by a factor of t_mean / t_last in each resultant, so that resultants with many reads are treated as saturating earlier.

@stscijgbot-rstdms
Copy link
Collaborator Author

Comment by Eddie Schlafly on JIRA:

PRs up at https://github.com/spacetelescope/romancal/pull/836], [https://github.com/spacetelescope/stcal/pull/188] addressing this issue.

@stscijgbot-rstdms
Copy link
Collaborator Author

Comment by Eddie Schlafly on JIRA:

PR has been merged, closing!

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

No branches or pull requests

3 participants